@devvit/public-api 0.10.23-next-2024-06-12-1b25bb8f2.0 → 0.10.23-next-2024-06-13-cad38246a.0

package/devvit/internals/blocks/BlocksReconciler.js

@@ -212,14 +212,23 @@ export class BlocksReconciler {
         const blockElement = element;
         // Intrinsic elements
         if (typeof blockElement.type === 'string') {
-            const childrens = await Promise.all(blockElement.children.flatMap(async (child, i) => {
+            const childrens = [];
+            /**
+             * Resist the urge to clean this up with flatMap. We need to await each processBlock _in order_.
+             *
+             * Number of times this has caused a major headache: 3
+             *
+             * Please increment the counter above if you had to revert this.
+             */
+            for (let i = 0; i < blockElement.children.length; i++) {
+                const child = blockElement.children[i];
                 if (child === undefined || child === null)
-                    return [];
-                return await this.processBlock(child, idGenerator, [
+                    continue;
+                childrens.push(await this.processBlock(child, idGenerator, [
                     ...path,
                     `${blockElement.type}.${i}`,
-                ]);
-            }));
+                ]));
+            }
             const children = childrens.flat();
             const collapsedChildren = __classPrivateFieldGet(this, _BlocksReconciler_instances, "m", _BlocksReconciler_flatten).call(this, children);
             await this.processProps(blockElement);
@@ -263,6 +272,12 @@ export class BlocksReconciler {
             // Ensure that the number of hooks are same from the previous render.
             if (!this.isInitialRender) {
                 const previousState = this.getPreviousComponentState();
+                /**
+                 * Note: This relies on the magic of {0: "a", 1: "b"} => ["a", "b"] and back under many circumstances.
+                 * It is too magic, but as this component is complicated and deprecated, it is not worth fixing.
+                 *
+                 * In fact previousState is an array.  This is confusing.
+                 */
                 const prevHookCount = Object.keys(previousState).length;
                 if (prevHookCount !== this.currentHookIndex) {
                     throw new Error('Invalid hook call. Hooks can only be called at the top-level of a function component. Make sure that you are not calling hooks inside loops, conditions, or nested functions.');
@@ -412,6 +427,10 @@ export class BlocksReconciler {
         // {"0": "one", "2": "three"} (length: 2; fails consistency check, see line 251)
         // After:
         // ["one", undefined, "three"] (length: 3)
+        /**
+         * Note: This relies on the magic of {0: "a", 1: "b"} => ["a", "b"] and back under many circumstances.
+         * It is too magic, but as this component is complicated and deprecated, it is not worth fixing.
+         */
         for (const key of Object.keys(this.renderState)) {
             this.renderState[key] = Object.values(this.renderState[key]);
         }

package/devvit/internals/context.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"context.d.ts","sourceRoot":"","sources":["../../../src/devvit/internals/context.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,QAAQ,EAAE,MAAM,gBAAgB,CAAC;AAK/C,OAAO,KAAK,EAAE,WAAW,EAAoB,MAAM,wBAAwB,CAAC;AAI5E,wBAAgB,sBAAsB,CACpC,QAAQ,EAAE,QAAQ,EAClB,MAAM,CAAC,EAAE,MAAM,EACf,SAAS,CAAC,EAAE,MAAM,GACjB,WAAW,CA4Bb"}
+{"version":3,"file":"context.d.ts","sourceRoot":"","sources":["../../../src/devvit/internals/context.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,QAAQ,EAAE,MAAM,gBAAgB,CAAC;AAI/C,OAAO,KAAK,EAAE,WAAW,EAAoB,MAAM,wBAAwB,CAAC;AAK5E,wBAAgB,sBAAsB,CACpC,QAAQ,EAAE,QAAQ,EAClB,MAAM,CAAC,EAAE,MAAM,EACf,SAAS,CAAC,EAAE,MAAM,GACjB,WAAW,CA4Bb"}

package/devvit/internals/context.js

@@ -1,6 +1,6 @@
 import { Header } from '@devvit/shared-types/Header.js';
 import { assertNonNull } from '@devvit/shared-types/NonNull.js';
-import pkg from '../../../package.json' assert { type: 'json' };
+import pkg from '../../version.json' assert { type: 'json' };
 let loggedVersion = false;
 export function getContextFromMetadata(metadata, postId, commentId) {
     const subredditId = metadata[Header.Subreddit]?.values[0];

package/meta.json

@@ -11760,11 +11760,11 @@
       "format": "esm"
     },
     "package.json": {
-      "bytes": 2832,
+      "bytes": 2819,
       "imports": []
     },
     "src/devvit/internals/context.ts": {
-      "bytes": 2686,
+      "bytes": 2683,
       "imports": [
         {
           "path": "../shared-types/dist/Header.js",
@@ -11779,7 +11779,7 @@
         {
           "path": "package.json",
           "kind": "import-statement",
-          "original": "../../../package.json"
+          "original": "../../version.json"
         }
       ],
       "format": "esm"
@@ -11894,7 +11894,7 @@
       "format": "esm"
     },
     "src/devvit/internals/blocks/BlocksReconciler.ts": {
-      "bytes": 18194,
+      "bytes": 19088,
       "imports": [
         {
           "path": "../protos/dist/index.js",
@@ -14356,7 +14356,7 @@
           "bytesInOutput": 1973
         },
         "package.json": {
-          "bytesInOutput": 2943
+          "bytesInOutput": 2930
         },
         "src/devvit/internals/context.ts": {
           "bytesInOutput": 1798
@@ -14383,7 +14383,7 @@
           "bytesInOutput": 22653
         },
         "src/devvit/internals/blocks/BlocksReconciler.ts": {
-          "bytesInOutput": 14129
+          "bytesInOutput": 14179
         },
         "src/devvit/internals/custom-post.ts": {
           "bytesInOutput": 697
@@ -14524,7 +14524,7 @@
           "bytesInOutput": 4450
         }
       },
-      "bytes": 14993539
+      "bytes": 14994832
     }
   }
 }

package/meta.min.json

@@ -2990,11 +2990,11 @@
       "format": "esm"
     },
     "package.json": {
-      "bytes": 2832,
+      "bytes": 2819,
       "imports": []
     },
     "src/devvit/internals/context.ts": {
-      "bytes": 2686,
+      "bytes": 2683,
       "imports": [
         {
           "path": "../shared-types/dist/Header.js",
@@ -3009,7 +3009,7 @@
         {
           "path": "package.json",
           "kind": "import-statement",
-          "original": "../../../package.json"
+          "original": "../../version.json"
         }
       ],
       "format": "esm"
@@ -3124,7 +3124,7 @@
       "format": "esm"
     },
     "src/devvit/internals/blocks/BlocksReconciler.ts": {
-      "bytes": 18194,
+      "bytes": 19088,
       "imports": [
         {
           "path": "@devvit/protos",
@@ -4642,7 +4642,7 @@
       "imports": [],
       "exports": [],
       "inputs": {},
-      "bytes": 1135300
+      "bytes": 1136297
     },
     "dist/public-api.min.js": {
       "imports": [
@@ -5268,7 +5268,7 @@
           "bytesInOutput": 848
         },
         "package.json": {
-          "bytesInOutput": 2390
+          "bytesInOutput": 2377
         },
         "src/devvit/internals/context.ts": {
           "bytesInOutput": 940
@@ -5277,7 +5277,7 @@
           "bytesInOutput": 350
         },
         "src/devvit/internals/blocks/BlocksReconciler.ts": {
-          "bytesInOutput": 7132
+          "bytesInOutput": 7153
         },
         "src/apis/ui/helpers/getEffectsFromUIClient.ts": {
           "bytesInOutput": 34
@@ -5427,7 +5427,7 @@
           "bytesInOutput": 2135
         }
       },
-      "bytes": 242962
+      "bytes": 242970
     }
   }
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@devvit/public-api",
-  "version": "0.10.23-next-2024-06-12-1b25bb8f2.0",
+  "version": "0.10.23-next-2024-06-13-cad38246a.0",
   "license": "BSD-3-Clause",
   "repository": {
     "type": "git",
@@ -9,7 +9,7 @@
   "type": "module",
   "main": "./index.js",
   "scripts": {
-    "build": "yarn build:icon-types && yarn build:semantic-colors && tsc && cp -af package.json devvit.tsconfig.json dist/ && yarn build:types && yarn build:min && yarn build:unmin",
+    "build": "yarn build:icon-types && yarn build:semantic-colors && tsc && cp -af devvit.tsconfig.json dist/ && yarn build:types && yarn build:min && yarn build:unmin",
     "build:icon-types": "make-icons src/types/icons.ts",
     "build:min": "esbuild   --bundle --sourcemap=linked --target=es2020 --format=esm  --metafile=dist/meta.min.json --outfile=dist/public-api.min.js --external:@devvit/protos --minify src/index.ts",
     "build:semantic-colors": "node scripts/make-semantic-colors.js",
@@ -30,8 +30,8 @@
   },
   "types": "./index.d.ts",
   "dependencies": {
-    "@devvit/protos": "0.10.23-next-2024-06-12-1b25bb8f2.0",
-    "@devvit/shared-types": "0.10.23-next-2024-06-12-1b25bb8f2.0",
+    "@devvit/protos": "0.10.23-next-2024-06-13-cad38246a.0",
+    "@devvit/shared-types": "0.10.23-next-2024-06-13-cad38246a.0",
     "base64-js": "1.5.1",
     "clone-deep": "4.0.1",
     "core-js": "3.27.2",
@@ -41,7 +41,7 @@
     "@ampproject/filesize": "4.3.0",
     "@devvit/eslint-config": "0.10.22",
     "@devvit/repo-tools": "0.10.22",
-    "@devvit/tsconfig": "0.10.23-next-2024-06-12-1b25bb8f2.0",
+    "@devvit/tsconfig": "0.10.23-next-2024-06-13-cad38246a.0",
     "@microsoft/api-extractor": "7.41.0",
     "@reddit/faceplate-ui": "11.3.3",
     "@types/clone-deep": "4.0.1",
@@ -64,5 +64,5 @@
     }
   },
   "source": "./src/index.ts",
-  "gitHead": "16ba154de06de7f7ce0fcdd8551bfb47c57b5ef8"
+  "gitHead": "f7de895f786949d9b7b476e7d9f1b8fd2124755a"
 }