@xmachines/play-react 1.0.0-beta.3 → 1.0.0-beta.30

package/dist/useSignalEffect.d.ts

@@ -51,6 +51,12 @@
  * **Implementation note:** We wrap the callback in Signal.Computed because
  * Signal.subtle.Watcher cannot automatically track arbitrary function calls.
  * The Computed handles dependency tracking, and the Watcher monitors it.
+ *
+ * **Memory safety (Phase 29):**
+ * - `disposed` flag prevents post-cleanup callback execution: if cleanup is
+ *   called before a pending microtask fires, the microtask returns early.
+ * - `needsEnqueue` guard dedups rapid synchronous signal changes: only one
+ *   microtask is ever queued per batch of synchronous mutations.
  */
 export declare const useSignalEffect: (callback: () => void | (() => void)) => void;
 //# sourceMappingURL=useSignalEffect.d.ts.map

package/dist/useSignalEffect.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"useSignalEffect.d.ts","sourceRoot":"","sources":["../src/useSignalEffect.ts"],"names":[],"mappings":"AAAA;;;;GAIG;AAWH;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;GAgDG;AACH,eAAO,MAAM,eAAe,GAAI,UAAU,MAAM,IAAI,GAAG,CAAC,MAAM,IAAI,CAAC,KAAG,IAuDrE,CAAC"}
+{"version":3,"file":"useSignalEffect.d.ts","sourceRoot":"","sources":["../src/useSignalEffect.ts"],"names":[],"mappings":"AAAA;;;;GAIG;AAWH;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;GAsDG;AACH,eAAO,MAAM,eAAe,GAAI,UAAU,MAAM,IAAI,GAAG,CAAC,MAAM,IAAI,CAAC,KAAG,IA+DrE,CAAC"}

package/dist/useSignalEffect.js

@@ -3,7 +3,7 @@
  *
  * @packageDocumentation
  */
-import { useEffect, useReducer } from "react";
+import { useEffect, useReducer, useRef } from "react";
 import { Signal } from "@xmachines/play-signals";
 /**
  * Marker symbol returned by effect Computed to satisfy type requirements
@@ -58,39 +58,54 @@ const EFFECT_RUN_MARKER = Symbol("effect-run");
  * **Implementation note:** We wrap the callback in Signal.Computed because
  * Signal.subtle.Watcher cannot automatically track arbitrary function calls.
  * The Computed handles dependency tracking, and the Watcher monitors it.
+ *
+ * **Memory safety (Phase 29):**
+ * - `disposed` flag prevents post-cleanup callback execution: if cleanup is
+ *   called before a pending microtask fires, the microtask returns early.
+ * - `needsEnqueue` guard dedups rapid synchronous signal changes: only one
+ *   microtask is ever queued per batch of synchronous mutations.
  */
 export const useSignalEffect = (callback) => {
+    // Store callback in a ref so the effect closure always calls the latest version
+    // WITHOUT triggering watcher teardown/re-setup on each render (standard React ref pattern)
+    const callbackRef = useRef(callback);
+    // Assignment outside useEffect keeps ref current without triggering re-effect
+    callbackRef.current = callback;
     // Force re-render when signal changes (React needs state change to re-render)
     const [, forceUpdate] = useReducer((x) => x + 1, 0);
     useEffect(() => {
         let cleanup;
         let needsEnqueue = true;
+        let disposed = false;
         // Wrap callback in a Computed to automatically track dependencies
         // The Computed will re-evaluate when any accessed signal changes
         const effect = new Signal.Computed(() => {
             // Run cleanup from previous effect
-            cleanup?.();
-            // Run user callback (may access signals, return cleanup)
-            cleanup = callback();
+            if (typeof cleanup === "function")
+                cleanup();
+            // Run user callback via ref (always latest version, no dep churn)
+            cleanup = callbackRef.current();
             // Return marker value (Computed requires a return value)
             return EFFECT_RUN_MARKER;
         });
         // Create watcher to detect when Computed needs re-evaluation
         const watcher = new Signal.subtle.Watcher(() => {
             // Batching flag prevents multiple microtasks for rapid changes
-            if (needsEnqueue) {
-                needsEnqueue = false;
-                // Schedule microtask to batch updates (prevent React thrashing)
-                queueMicrotask(() => {
-                    needsEnqueue = true;
-                    // Re-evaluate the Computed (runs callback with new signal values)
-                    effect.get();
-                    // Force React re-render
-                    forceUpdate();
-                    // Re-watch for next change
-                    watcher.watch();
-                });
-            }
+            if (disposed || !needsEnqueue)
+                return;
+            needsEnqueue = false;
+            // Schedule microtask to batch updates (prevent React thrashing)
+            queueMicrotask(() => {
+                if (disposed)
+                    return;
+                needsEnqueue = true;
+                // Re-evaluate the Computed (runs callback with new signal values)
+                effect.get();
+                // Force React re-render
+                forceUpdate();
+                // Re-watch for next change
+                watcher.watch();
+            });
         });
         // Watch the Computed signal
         watcher.watch(effect);
@@ -98,10 +113,12 @@ export const useSignalEffect = (callback) => {
         effect.get();
         // Cleanup on unmount
         return () => {
-            cleanup?.();
+            disposed = true;
+            if (typeof cleanup === "function")
+                cleanup();
             // Unwatch to stop receiving signal updates
             watcher.unwatch(effect);
         };
-    }, [callback]);
+    }, []); // ← [] dep array: watcher created ONCE per mount, not once per render
 };
 //# sourceMappingURL=useSignalEffect.js.map

package/dist/useSignalEffect.js.map

@@ -1 +1 @@
-{"version":3,"file":"useSignalEffect.js","sourceRoot":"","sources":["../src/useSignalEffect.ts"],"names":[],"mappings":"AAAA;;;;GAIG;AAEH,OAAO,EAAE,SAAS,EAAE,UAAU,EAAE,MAAM,OAAO,CAAC;AAC9C,OAAO,EAAE,MAAM,EAAE,MAAM,yBAAyB,CAAC;AAEjD;;;GAGG;AACH,MAAM,iBAAiB,GAAG,MAAM,CAAC,YAAY,CAAC,CAAC;AAE/C;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;GAgDG;AACH,MAAM,CAAC,MAAM,eAAe,GAAG,CAAC,QAAmC,EAAQ,EAAE;IAC5E,8EAA8E;IAC9E,MAAM,CAAC,EAAE,WAAW,CAAC,GAAG,UAAU,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,GAAG,CAAC,EAAE,CAAC,CAAC,CAAC;IAEpD,SAAS,CAAC,GAAG,EAAE;QACd,IAAI,OAA4B,CAAC;QACjC,IAAI,YAAY,GAAG,IAAI,CAAC;QAExB,kEAAkE;QAClE,iEAAiE;QACjE,MAAM,MAAM,GAAG,IAAI,MAAM,CAAC,QAAQ,CAAC,GAAG,EAAE;YACvC,mCAAmC;YACnC,OAAO,EAAE,EAAE,CAAC;YAEZ,yDAAyD;YACzD,OAAO,GAAG,QAAQ,EAAE,CAAC;YAErB,yDAAyD;YACzD,OAAO,iBAAiB,CAAC;QAC1B,CAAC,CAAC,CAAC;QAEH,6DAA6D;QAC7D,MAAM,OAAO,GAAG,IAAI,MAAM,CAAC,MAAM,CAAC,OAAO,CAAC,GAAG,EAAE;YAC9C,+DAA+D;YAC/D,IAAI,YAAY,EAAE,CAAC;gBAClB,YAAY,GAAG,KAAK,CAAC;gBACrB,gEAAgE;gBAChE,cAAc,CAAC,GAAG,EAAE;oBACnB,YAAY,GAAG,IAAI,CAAC;oBAEpB,kEAAkE;oBAClE,MAAM,CAAC,GAAG,EAAE,CAAC;oBAEb,wBAAwB;oBACxB,WAAW,EAAE,CAAC;oBAEd,2BAA2B;oBAC3B,OAAO,CAAC,KAAK,EAAE,CAAC;gBACjB,CAAC,CAAC,CAAC;YACJ,CAAC;QACF,CAAC,CAAC,CAAC;QAEH,4BAA4B;QAC5B,OAAO,CAAC,KAAK,CAAC,MAAa,CAAC,CAAC;QAE7B,yCAAyC;QACzC,MAAM,CAAC,GAAG,EAAE,CAAC;QAEb,qBAAqB;QACrB,OAAO,GAAG,EAAE;YACX,OAAO,EAAE,EAAE,CAAC;YACZ,2CAA2C;YAC3C,OAAO,CAAC,OAAO,CAAC,MAAa,CAAC,CAAC;QAChC,CAAC,CAAC;IACH,CAAC,EAAE,CAAC,QAAQ,CAAC,CAAC,CAAC;AAChB,CAAC,CAAC"}
+{"version":3,"file":"useSignalEffect.js","sourceRoot":"","sources":["../src/useSignalEffect.ts"],"names":[],"mappings":"AAAA;;;;GAIG;AAEH,OAAO,EAAE,SAAS,EAAE,UAAU,EAAE,MAAM,EAAE,MAAM,OAAO,CAAC;AACtD,OAAO,EAAE,MAAM,EAAE,MAAM,yBAAyB,CAAC;AAEjD;;;GAGG;AACH,MAAM,iBAAiB,GAAG,MAAM,CAAC,YAAY,CAAC,CAAC;AAE/C;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;GAsDG;AACH,MAAM,CAAC,MAAM,eAAe,GAAG,CAAC,QAAmC,EAAQ,EAAE;IAC5E,gFAAgF;IAChF,2FAA2F;IAC3F,MAAM,WAAW,GAAG,MAAM,CAAC,QAAQ,CAAC,CAAC;IACrC,8EAA8E;IAC9E,WAAW,CAAC,OAAO,GAAG,QAAQ,CAAC;IAE/B,8EAA8E;IAC9E,MAAM,CAAC,EAAE,WAAW,CAAC,GAAG,UAAU,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,GAAG,CAAC,EAAE,CAAC,CAAC,CAAC;IAEpD,SAAS,CAAC,GAAG,EAAE;QACd,IAAI,OAA4B,CAAC;QACjC,IAAI,YAAY,GAAG,IAAI,CAAC;QACxB,IAAI,QAAQ,GAAG,KAAK,CAAC;QAErB,kEAAkE;QAClE,iEAAiE;QACjE,MAAM,MAAM,GAAG,IAAI,MAAM,CAAC,QAAQ,CAAC,GAAG,EAAE;YACvC,mCAAmC;YACnC,IAAI,OAAO,OAAO,KAAK,UAAU;gBAAE,OAAO,EAAE,CAAC;YAE7C,kEAAkE;YAClE,OAAO,GAAG,WAAW,CAAC,OAAO,EAAE,CAAC;YAEhC,yDAAyD;YACzD,OAAO,iBAAiB,CAAC;QAC1B,CAAC,CAAC,CAAC;QAEH,6DAA6D;QAC7D,MAAM,OAAO,GAAG,IAAI,MAAM,CAAC,MAAM,CAAC,OAAO,CAAC,GAAG,EAAE;YAC9C,+DAA+D;YAC/D,IAAI,QAAQ,IAAI,CAAC,YAAY;gBAAE,OAAO;YACtC,YAAY,GAAG,KAAK,CAAC;YACrB,gEAAgE;YAChE,cAAc,CAAC,GAAG,EAAE;gBACnB,IAAI,QAAQ;oBAAE,OAAO;gBACrB,YAAY,GAAG,IAAI,CAAC;gBAEpB,kEAAkE;gBAClE,MAAM,CAAC,GAAG,EAAE,CAAC;gBAEb,wBAAwB;gBACxB,WAAW,EAAE,CAAC;gBAEd,2BAA2B;gBAC3B,OAAO,CAAC,KAAK,EAAE,CAAC;YACjB,CAAC,CAAC,CAAC;QACJ,CAAC,CAAC,CAAC;QAEH,4BAA4B;QAC5B,OAAO,CAAC,KAAK,CAAC,MAAkC,CAAC,CAAC;QAElD,yCAAyC;QACzC,MAAM,CAAC,GAAG,EAAE,CAAC;QAEb,qBAAqB;QACrB,OAAO,GAAG,EAAE;YACX,QAAQ,GAAG,IAAI,CAAC;YAChB,IAAI,OAAO,OAAO,KAAK,UAAU;gBAAE,OAAO,EAAE,CAAC;YAC7C,2CAA2C;YAC3C,OAAO,CAAC,OAAO,CAAC,MAAkC,CAAC,CAAC;QACrD,CAAC,CAAC;IACH,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,sEAAsE;AAC/E,CAAC,CAAC"}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@xmachines/play-react",
-  "version": "1.0.0-beta.3",
+  "version": "1.0.0-beta.30",
   "description": "React renderer for XMachines Play architecture with signal-driven rendering",
   "keywords": [
     "actor",
@@ -26,6 +26,7 @@
   "type": "module",
   "exports": {
     ".": {
+      "source": "./src/index.ts",
       "types": "./dist/index.d.ts",
       "import": "./dist/index.js"
     }
@@ -35,30 +36,46 @@
   },
   "scripts": {
     "build": "tsc --build",
-    "clean": "rm -rf dist *.tsbuildinfo",
-    "typecheck": "tsc --noEmit",
-    "test": "vitest run",
+    "clean": "rm -rf dist *.tsbuildinfo coverage .vitest-attachments test/browser/__screenshots__ node_modules/.vite*",
+    "lint": "oxlint .",
+    "format": "oxfmt .",
+    "test": "vitest",
     "test:vitest": "vitest run",
-    "test:browser": "vitest run --browser.enabled --browser.name=chromium test/browser",
+    "test:browser": "vitest --browser.enabled --browser.name=chromium test/browser",
     "prepublishOnly": "npm run build"
   },
   "dependencies": {
-    "@xmachines/play-actor": "1.0.0-beta.3",
-    "@xmachines/play-catalog": "1.0.0-beta.3",
-    "@xmachines/play-signals": "1.0.0-beta.3"
+    "@xmachines/play": "1.0.0-beta.30",
+    "@xmachines/play-actor": "1.0.0-beta.30",
+    "@xmachines/play-signals": "1.0.0-beta.30"
   },
   "devDependencies": {
+    "@json-render/core": "^0.16.0",
+    "@json-render/react": "^0.16.0",
+    "@json-render/xstate": "^0.16.0",
+    "@testing-library/jest-dom": "^6.9.1",
     "@testing-library/react": "^16.3.2",
     "@types/node": "^25.5.0",
     "@types/react": "^19.2.14",
     "@types/react-dom": "^19.2.3",
-    "jsdom": "^29.0.0",
+    "@xmachines/shared": "1.0.0-beta.30",
+    "@xstate/store": ">=3.17.0",
+    "jsdom": "^29.0.1",
+    "oxfmt": "^0.43.0",
+    "oxlint": "^1.57.0",
     "react": "^19.2.4",
     "react-dom": "^19.2.4",
-    "typescript": "^5.9.3"
+    "typescript": "^5.9.3 || ^6.0.2",
+    "vitest": "^4.1.2",
+    "xstate": "^5.30.0"
   },
   "peerDependencies": {
-    "react": "^18.2.0 || ^19.0.0",
-    "react-dom": "^18.2.0 || ^19.0.0"
+    "@json-render/core": "^0.16.0",
+    "@json-render/react": "^0.16.0",
+    "@json-render/xstate": "^0.16.0",
+    "@xstate/store": ">=3.17.0",
+    "react": "^18.0.0 || ^19.0.0",
+    "react-dom": "^18.0.0 || ^19.0.0",
+    "xstate": "^5.30.0"
   }
 }