@mythicalos/preact-ui 0.2.0 → 0.2.2

package/dist/hooks.js

@@ -64,10 +64,22 @@ export function usePoll(fn, ms, opts = {}) {
         let timer;
         let stopped = false;
         let failures = 0; // consecutive failed reads — drives the ×2 backoff ladder (reset on success)
+        // true from the moment a tick fires its fetch until that fetch settles — the ONE in-flight
+        // read this call site is allowed to have. The visibility handler below must never start a
+        // second tick while this is true; it lets the in-flight tick's own onSettled → schedule()
+        // continue the single chain instead (a prior version fired an immediate tick on visibility
+        // return unconditionally, which could open a second concurrent chain).
+        let pending = false;
         const isHidden = () => typeof document !== "undefined" && document.hidden === true;
         const schedule = () => {
             if (!alive || stopped)
                 return;
+            // Clear any already-armed timer before arming a new one — defense in depth so a stray
+            // double-settle can never leave an earlier timer orphaned (never cleared, still firing).
+            if (timer) {
+                clearTimeout(timer);
+                timer = undefined;
+            }
             // U7 pause: while the document is hidden no timer is armed — the
             // visibilitychange handler below refreshes immediately on return.
             if (isHidden())
@@ -77,26 +89,31 @@ export function usePoll(fn, ms, opts = {}) {
         // diff r3-F1: every response routes through runPollTick — stamped with the epoch at fire
         // time, applied only while that epoch is still live. A stale settle leaves loading alone
         // (the reset already set it for the new epoch) but still rearms via schedule()'s own guards.
-        const tick = () => runPollTick({
-            fetch: () => fnRef.current(),
-            guard,
-            isAlive: () => alive && !stopped,
-            isHidden,
-            onSuccess: (v) => {
-                failures = 0;
-                setData(v);
-                setError(undefined);
-            },
-            onFailure: (message) => {
-                failures += 1;
-                setError(message);
-            },
-            onSettled: (applied) => {
-                if (applied)
-                    setLoading(false);
-                schedule();
-            },
-        });
+        const tick = () => {
+            pending = true;
+            return runPollTick({
+                fetch: () => fnRef.current(),
+                guard,
+                isAlive: () => alive && !stopped,
+                isHidden,
+                onSuccess: (v) => {
+                    failures = 0;
+                    setData(v);
+                    setError(undefined);
+                },
+                onFailure: (message) => {
+                    failures += 1;
+                    setError(message);
+                },
+                onSettled: (applied) => {
+                    if (applied)
+                        setLoading(false);
+                    schedule();
+                },
+            }).finally(() => {
+                pending = false;
+            });
+        };
         const onVisibility = () => {
             if (!alive || stopped)
                 return;
@@ -104,8 +121,10 @@ export function usePoll(fn, ms, opts = {}) {
                 clearTimeout(timer);
                 timer = undefined;
             }
-            if (!isHidden())
-                void tick(); // immediate refresh on visibility return
+            // Immediate refresh on visibility return — but only when idle. A tick already in flight
+            // keeps its place as the single chain: its own onSettled → schedule() picks up from here.
+            if (!isHidden() && !pending)
+                void tick();
         };
         if (typeof document !== "undefined")
             document.addEventListener("visibilitychange", onVisibility);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mythicalos/preact-ui",
-  "version": "0.2.0",
+  "version": "0.2.2",
   "description": "mythicalOS thin Preact bindings over @mythicalos/ui-core — Button, Input/Toggle/Checkbox, MaskedSecretInput, EmptyState, ConfirmDialog/Scrim, Toast/ToastProvider, Chip, Card, Avatar, StatusLine, SearchInput, Banner, Gauge, and the usePoll/useInterval hooks. Render + framework wiring only — every class string, poll-scheduling math, glyph map, and text composition is derived by @mythicalos/ui-core so this binding and its React sibling can never drift. Apache-2.0.",
   "license": "Apache-2.0",
   "type": "module",
@@ -20,28 +20,32 @@
       "default": "./dist/index.js"
     }
   },
-  "files": ["dist", "README.md", "NOTICE"],
+  "files": [
+    "dist",
+    "README.md",
+    "NOTICE"
+  ],
   "peerDependencies": {
     "preact": "^10.22.0",
     "@mythicalos/tokens": "^0.5.11"
   },
   "peerDependenciesMeta": {
     "@mythicalos/tokens": {
-    "optional": true
+      "optional": true
+    }
+  },
+  "dependencies": {
+    "@mythicalos/ui-core": "^0.1.1"
+  },
+  "scripts": {
+    "build": "tsc -p tsconfig.build.json",
+    "typecheck": "tsc --noEmit",
+    "test": "bun test",
+    "prepublishOnly": "bun run build && bun test && bun run typecheck"
+  },
+  "devDependencies": {
+    "preact": "^10.22.0",
+    "preact-render-to-string": "^6.5.13",
+    "typescript": "^5"
   }
-},
-"dependencies": {
-  "@mythicalos/ui-core": "^0.1.0"
-},
-"scripts": {
-  "build": "tsc -p tsconfig.build.json",
-  "typecheck": "tsc --noEmit",
-  "test": "bun test",
-  "prepublishOnly": "bun run build && bun test && bun run typecheck"
-},
-"devDependencies": {
-  "preact": "^10.22.0",
-  "preact-render-to-string": "^6.5.13",
-  "typescript": "^5"
-}
 }