castle-web-sdk 0.4.29 → 0.4.30

package/README.md

@@ -735,24 +735,6 @@ if (isEdit()) {
 }
 ```
 
-### `registerParams(path, params)`
-
-Decks never write this call. While `castle-web serve` is running, the
-dev server appends it to every module that declares a `PARAMS` object,
-so a value tuned in the editor's Parameters panel is assigned into that
-object in the deck that is already running instead of restarting it. A
-published deck never gets the call.
-
-For a tune to be visible, read a param at the use site rather than
-destructuring it at module load:
-
-```js
-export const PARAMS = { jumpHeight: 4.2 };
-
-// reads the tuned value every jump
-function jump() { player.vy = -PARAMS.jumpHeight; }
-```
-
 ### `writeFile(path, contents): Promise<void>`
 
 Writes a file in the deck directory. `path` is relative to the deck

package/dist/paramsRegistry.d.ts

@@ -22,9 +22,11 @@ export interface AppliedNotice {
  */
 export declare function appliedNotice(path: string, seq: unknown, ok: boolean): AppliedNotice;
 /**
- * Write tuned values into the module's live `PARAMS`. False when any of them
- * could not land -- an unregistered path, a key the object does not have, a
- * frozen object -- and the rest are still written, because a slot that cannot
- * be applied is no reason to withhold the ones that can.
+ * Write tuned values into every live `PARAMS` registered under the path. False
+ * unless every slot landed on every object -- an unregistered path, a key one
+ * object does not have, a frozen object -- and the rest are still written,
+ * because a slot that cannot be applied is no reason to withhold the ones that
+ * can. An older instance missing a key a later code edit added reports false
+ * too: the run may be reading it, so the editor offers a restart.
  */
 export declare function applyTunedParams(path: string, slots: TunedParam[]): boolean;

package/dist/paramsRegistry.js

@@ -8,7 +8,15 @@
 // path rather than on anything the module says about itself: the path is what
 // the serve's broadcast names, and the module never had to agree to anything.
 //
-// The object is mutated IN PLACE, never replaced. Deck code holds the same
+// A path holds EVERY object ever registered under it, not the latest. The serve
+// appends the call to every transform of the module, including a kit's
+// cache-busting `?t=` re-import, which evaluates a fresh module instance with a
+// fresh `PARAMS` under the same path. An older instance can still be the one
+// running -- physics-2d's play frame keeps the classes it started with and never
+// adopts swapped ones -- and the registry cannot tell which instance a run reads,
+// so the only honest answer is to write the tune into all of them.
+//
+// Each object is mutated IN PLACE, never replaced. Deck code holds the same
 // reference (`PARAMS.jumpHeight` at the use site is the documented way to read
 // one), so handing back a new object would leave every reader on the old one.
 const registered = new Map();
@@ -17,7 +25,12 @@ export function registerParams(path, params) {
         return;
     if (typeof params !== "object" || params === null)
         return;
-    registered.set(path, params);
+    let instances = registered.get(path);
+    if (!instances) {
+        instances = new Set();
+        registered.set(path, instances);
+    }
+    instances.add(params);
 }
 // Own properties only, at every level. A `ParamSlot.id` comes from a key the
 // file wrote in an object literal, so an inherited name (`constructor`,
@@ -65,19 +78,23 @@ export function appliedNotice(path, seq, ok) {
     };
 }
 /**
- * Write tuned values into the module's live `PARAMS`. False when any of them
- * could not land -- an unregistered path, a key the object does not have, a
- * frozen object -- and the rest are still written, because a slot that cannot
- * be applied is no reason to withhold the ones that can.
+ * Write tuned values into every live `PARAMS` registered under the path. False
+ * unless every slot landed on every object -- an unregistered path, a key one
+ * object does not have, a frozen object -- and the rest are still written,
+ * because a slot that cannot be applied is no reason to withhold the ones that
+ * can. An older instance missing a key a later code edit added reports false
+ * too: the run may be reading it, so the editor offers a restart.
  */
 export function applyTunedParams(path, slots) {
-    const target = registered.get(path);
-    if (!target)
+    const instances = registered.get(path);
+    if (!instances || instances.size === 0)
         return false;
     let applied = true;
-    for (const slot of slots) {
-        if (!assign(target, slot.id, slot.after))
-            applied = false;
+    for (const target of instances) {
+        for (const slot of slots) {
+            if (!assign(target, slot.id, slot.after))
+                applied = false;
+        }
     }
     return applied;
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "castle-web-sdk",
-  "version": "0.4.29",
+  "version": "0.4.30",
   "repository": {
     "type": "git",
     "url": "git+https://github.com/castle-xyz/castle-experimental-web.git"