npm - @openchambery/web - Versions diffs - 1.19.5-beta.8 → 1.19.5 - Mend

@openchambery/web 1.19.5-beta.8 → 1.19.5

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.

Files changed (94) hide show

package/dist/index.html CHANGED Viewed

@@ -552,25 +552,25 @@
         opacity: 0.72;
       }
     </style>
-    <script type="module" crossorigin src="/assets/main-BNl9icid.js"></script>
+    <script type="module" crossorigin src="/assets/main-Dy2r9c5q.js"></script>
     <link rel="modulepreload" crossorigin href="/assets/rolldown-runtime-C0FnF6B9.js">
     <link rel="modulepreload" crossorigin href="/assets/vendor-base-ui-CnHfUmDN.js">
     <link rel="modulepreload" crossorigin href="/assets/vendor-zustand-B4dlGEM2.js">
-    <link rel="modulepreload" crossorigin href="/assets/useUIStore-DOxLVeGb.js">
+    <link rel="modulepreload" crossorigin href="/assets/useUIStore-BXjMQrel.js">
     <link rel="modulepreload" crossorigin href="/assets/vendor-pathe-vBmvQE8f.js">
     <link rel="modulepreload" crossorigin href="/assets/vendor-react-DLPElpoQ.js">
     <link rel="modulepreload" crossorigin href="/assets/vendor-tanstack-query-core-BWz0ose6.js">
-    <link rel="modulepreload" crossorigin href="/assets/persistence-DE9RnVcA.js">
-    <link rel="modulepreload" crossorigin href="/assets/desktopHosts-BrrYPqKq.js">
-    <link rel="modulepreload" crossorigin href="/assets/desktopRelayRestore-DR60DATT.js">
+    <link rel="modulepreload" crossorigin href="/assets/persistence-CLUawIlm.js">
+    <link rel="modulepreload" crossorigin href="/assets/desktopHosts-DceeVBky.js">
+    <link rel="modulepreload" crossorigin href="/assets/desktopRelayRestore-B0MRdR5y.js">
     <link rel="modulepreload" crossorigin href="/assets/vendor-opencode-sdk-C07HQypa.js">
     <link rel="modulepreload" crossorigin href="/assets/vendor-aparajita-capacitor-secure-storage-CjQdwfjS.js">
-    <link rel="modulepreload" crossorigin href="/assets/platform-MjT8iqKB.js">
-    <link rel="modulepreload" crossorigin href="/assets/terminalApi-Bnx0KvaV.js">
+    <link rel="modulepreload" crossorigin href="/assets/platform-BrncW9ab.js">
+    <link rel="modulepreload" crossorigin href="/assets/terminalApi-D2OzR0I3.js">
     <link rel="modulepreload" crossorigin href="/assets/types-BuiqMmh_.js">
     <link rel="modulepreload" crossorigin href="/assets/capgoAdapter-DCowZ2ab.js">
-    <link rel="modulepreload" crossorigin href="/assets/runtimeConfig-QnEB1ojv.js">
-    <link rel="stylesheet" crossorigin href="/assets/src-CD7-ovJ5.css">
+    <link rel="modulepreload" crossorigin href="/assets/runtimeConfig-D2zUK5uB.js">
+    <link rel="stylesheet" crossorigin href="/assets/src-2eFAiBLq.css">
   </head>
   <body class="h-full bg-background text-foreground">
     <!-- Keep the startup layer outside #root. React replaces every child of

package/dist/mini-chat.html CHANGED Viewed

@@ -74,25 +74,25 @@
       }
     </style>
-    <script type="module" crossorigin src="/assets/miniChat-D3Vj4bki.js"></script>
+    <script type="module" crossorigin src="/assets/miniChat-CQqw310g.js"></script>
     <link rel="modulepreload" crossorigin href="/assets/rolldown-runtime-C0FnF6B9.js">
     <link rel="modulepreload" crossorigin href="/assets/vendor-base-ui-CnHfUmDN.js">
     <link rel="modulepreload" crossorigin href="/assets/vendor-zustand-B4dlGEM2.js">
-    <link rel="modulepreload" crossorigin href="/assets/useUIStore-DOxLVeGb.js">
+    <link rel="modulepreload" crossorigin href="/assets/useUIStore-BXjMQrel.js">
     <link rel="modulepreload" crossorigin href="/assets/vendor-pathe-vBmvQE8f.js">
     <link rel="modulepreload" crossorigin href="/assets/vendor-react-DLPElpoQ.js">
     <link rel="modulepreload" crossorigin href="/assets/vendor-tanstack-query-core-BWz0ose6.js">
-    <link rel="modulepreload" crossorigin href="/assets/persistence-DE9RnVcA.js">
-    <link rel="modulepreload" crossorigin href="/assets/desktopHosts-BrrYPqKq.js">
-    <link rel="modulepreload" crossorigin href="/assets/desktopRelayRestore-DR60DATT.js">
+    <link rel="modulepreload" crossorigin href="/assets/persistence-CLUawIlm.js">
+    <link rel="modulepreload" crossorigin href="/assets/desktopHosts-DceeVBky.js">
+    <link rel="modulepreload" crossorigin href="/assets/desktopRelayRestore-B0MRdR5y.js">
     <link rel="modulepreload" crossorigin href="/assets/vendor-opencode-sdk-C07HQypa.js">
     <link rel="modulepreload" crossorigin href="/assets/vendor-aparajita-capacitor-secure-storage-CjQdwfjS.js">
-    <link rel="modulepreload" crossorigin href="/assets/platform-MjT8iqKB.js">
-    <link rel="modulepreload" crossorigin href="/assets/terminalApi-Bnx0KvaV.js">
+    <link rel="modulepreload" crossorigin href="/assets/platform-BrncW9ab.js">
+    <link rel="modulepreload" crossorigin href="/assets/terminalApi-D2OzR0I3.js">
     <link rel="modulepreload" crossorigin href="/assets/types-BuiqMmh_.js">
     <link rel="modulepreload" crossorigin href="/assets/capgoAdapter-DCowZ2ab.js">
-    <link rel="modulepreload" crossorigin href="/assets/runtimeConfig-QnEB1ojv.js">
-    <link rel="stylesheet" crossorigin href="/assets/src-CD7-ovJ5.css">
+    <link rel="modulepreload" crossorigin href="/assets/runtimeConfig-D2zUK5uB.js">
+    <link rel="stylesheet" crossorigin href="/assets/src-2eFAiBLq.css">
   </head>
   <body class="h-full bg-background text-foreground">
     <div id="root" class="h-full">

package/dist/mobile.html CHANGED Viewed

@@ -63,25 +63,25 @@
       })();
     </script>
-    <script type="module" crossorigin src="/assets/mobile-6mt3nA3g.js"></script>
+    <script type="module" crossorigin src="/assets/mobile-ClMMhPzX.js"></script>
     <link rel="modulepreload" crossorigin href="/assets/rolldown-runtime-C0FnF6B9.js">
     <link rel="modulepreload" crossorigin href="/assets/vendor-base-ui-CnHfUmDN.js">
     <link rel="modulepreload" crossorigin href="/assets/vendor-zustand-B4dlGEM2.js">
-    <link rel="modulepreload" crossorigin href="/assets/useUIStore-DOxLVeGb.js">
+    <link rel="modulepreload" crossorigin href="/assets/useUIStore-BXjMQrel.js">
     <link rel="modulepreload" crossorigin href="/assets/vendor-pathe-vBmvQE8f.js">
     <link rel="modulepreload" crossorigin href="/assets/vendor-react-DLPElpoQ.js">
     <link rel="modulepreload" crossorigin href="/assets/vendor-tanstack-query-core-BWz0ose6.js">
-    <link rel="modulepreload" crossorigin href="/assets/persistence-DE9RnVcA.js">
-    <link rel="modulepreload" crossorigin href="/assets/desktopHosts-BrrYPqKq.js">
-    <link rel="modulepreload" crossorigin href="/assets/desktopRelayRestore-DR60DATT.js">
+    <link rel="modulepreload" crossorigin href="/assets/persistence-CLUawIlm.js">
+    <link rel="modulepreload" crossorigin href="/assets/desktopHosts-DceeVBky.js">
+    <link rel="modulepreload" crossorigin href="/assets/desktopRelayRestore-B0MRdR5y.js">
     <link rel="modulepreload" crossorigin href="/assets/vendor-opencode-sdk-C07HQypa.js">
     <link rel="modulepreload" crossorigin href="/assets/vendor-aparajita-capacitor-secure-storage-CjQdwfjS.js">
-    <link rel="modulepreload" crossorigin href="/assets/platform-MjT8iqKB.js">
-    <link rel="modulepreload" crossorigin href="/assets/terminalApi-Bnx0KvaV.js">
+    <link rel="modulepreload" crossorigin href="/assets/platform-BrncW9ab.js">
+    <link rel="modulepreload" crossorigin href="/assets/terminalApi-D2OzR0I3.js">
     <link rel="modulepreload" crossorigin href="/assets/types-BuiqMmh_.js">
     <link rel="modulepreload" crossorigin href="/assets/capgoAdapter-DCowZ2ab.js">
-    <link rel="modulepreload" crossorigin href="/assets/runtimeConfig-QnEB1ojv.js">
-    <link rel="stylesheet" crossorigin href="/assets/src-CD7-ovJ5.css">
+    <link rel="modulepreload" crossorigin href="/assets/runtimeConfig-D2zUK5uB.js">
+    <link rel="stylesheet" crossorigin href="/assets/src-2eFAiBLq.css">
   </head>
   <body class="h-full bg-background text-foreground">
     <div id="root" class="h-full"></div>

package/package.json CHANGED Viewed

@@ -1,6 +1,6 @@
 {
   "name": "@openchambery/web",
-  "version": "1.19.5-beta.8",
+  "version": "1.19.5",
   "private": false,
   "type": "module",
   "main": "./server/index.js",

package/server/index.js CHANGED Viewed

@@ -342,8 +342,6 @@ const updateUiVisibility = (token, visible, platform) => {
   if (visible === true) clearPendingPushBadge();
   return pushRuntime.updateUiVisibility(token, visible, platform);
 };
-const isAnyUiVisible = (...args) => pushRuntime.isAnyUiVisible(...args);
-const isAnyInteractiveClientVisible = (...args) => pushRuntime.isAnyInteractiveClientVisible(...args);
 const isUiVisible = (...args) => pushRuntime.isUiVisible(...args);
 const ensurePushInitialized = (...args) => pushRuntime.ensurePushInitialized(...args);
 const setPushInitialized = (...args) => pushRuntime.setPushInitialized(...args);
@@ -664,7 +662,6 @@ const notificationTriggerRuntime = createNotificationTriggerRuntime({
   sendPushToAllUiSessions,
   sendApnsToAllUiSessions,
   sendLiveActivityEnd,
-  isAnyInteractiveClientVisible,
   buildOpenCodeUrl,
   getOpenCodeAuthHeaders,
 });

package/server/lib/notifications/DOCUMENTATION.md CHANGED Viewed

@@ -52,7 +52,7 @@ This module provides notification message preparation utilities for the web serv
   - session meta cache for child-session and small-model suppression
   - template resolution and fallback behavior
   - native notification fanout and web push payload fanout
-  - push suppression while any fresh UI visibility heartbeat reports a focused client
+  - push always fans out to every subscribed surface; another client's visibility never suppresses delivery
 ### Push runtime API (push-runtime.js)
 - `createPushRuntime(dependencies)`: creates runtime for web push and UI visibility state.
@@ -62,7 +62,7 @@ This module provides notification message preparation utilities for the web serv
   - `setPushInitialized(value)`
   - `addOrUpdatePushSubscription(uiSessionToken, subscription, userAgent)`
   - `removePushSubscription(uiSessionToken, endpoint)`
-  - `sendPushToAllUiSessions(payload, options?)`
+  - `sendPushToAllUiSessions(payload)`
   - `updateUiVisibility(token, visible)`
   - `isAnyUiVisible()`
   - `isUiVisible(token)`

package/server/lib/notifications/push-runtime.js CHANGED Viewed

@@ -221,8 +221,7 @@ export const createPushRuntime = (deps) => {
     }
   };
-  const sendPushToAllUiSessions = async (payload, options = {}) => {
-    const requireNoSse = options.requireNoSse === true;
+  const sendPushToAllUiSessions = async (payload) => {
     const store = await readPushSubscriptionsFromDisk();
     const sessions = store.subscriptionsBySession || {};
     const subscriptionsByEndpoint = new Map();
@@ -239,14 +238,6 @@ export const createPushRuntime = (deps) => {
     }
     await Promise.all(Array.from(subscriptionsByEndpoint.values()).map(async (sub) => {
-      if (requireNoSse) {
-        // Mobile PWA subscriptions follow the same presence model as native push: suppress only
-        // when an interactive (desktop/web) client is visible. The phone PWA's own foreground is
-        // handled in the service worker (focused-client check), so it won't double-notify.
-        // Non-mobile (desktop/web) subscriptions keep the existing any-visible gate.
-        const suppressed = isMobilePlatform(sub.platform) ? isAnyInteractiveClientVisible() : isAnyUiVisible();
-        if (suppressed) return;
-      }
       await sendPushToSubscription(sub, payload);
     }));
   };
@@ -278,10 +269,9 @@ export const createPushRuntime = (deps) => {
     return false;
   };
-  // True when at least one NON-mobile client (desktop/web/vscode) is currently visible. Used to
-  // suppress native push to the phone: an active desktop already shows the notification, so the
-  // phone doesn't need it. Deliberately based on the desktop's visibility (reliable), never the
-  // phone's own (a backgrounded WKWebView can't report "hidden" before iOS suspends it).
+  // True when at least one NON-mobile client (desktop/web/vscode) is currently visible.
+  // Not a push gate — delivery always fans out. Kept as a visibility query for tests and
+  // diagnostics. Deliberately based on the desktop's visibility, never the phone's own.
   const isAnyInteractiveClientVisible = () => {
     const now = Date.now();
     pruneUiVisibility(now);

package/server/lib/notifications/runtime.js CHANGED Viewed

@@ -12,7 +12,6 @@ export const createNotificationTriggerRuntime = (deps) => {
     sendPushToAllUiSessions,
     sendApnsToAllUiSessions,
     sendLiveActivityEnd,
-    isAnyInteractiveClientVisible,
     buildOpenCodeUrl,
     getOpenCodeAuthHeaders,
   } = deps;
@@ -68,25 +67,17 @@ export const createNotificationTriggerRuntime = (deps) => {
   };
   // Fan a notification out to every delivery channel: browser web-push (full templated
-  // payload) and native iOS APNs (generic model-based text). Both share the dedup tag and
-  // `requireNoSse` focus gate; a failure in one channel must not block the other.
-  const fanoutPush = (payload, options) => {
-    // Presence-aware routing: if any interactive (non-mobile) client — desktop/web/vscode — is
-    // currently visible, it already shows the in-app notification, so skip the native push to the
-    // phone. Gated on the desktop's visibility (reliable), never the phone's own. When we skip we
-    // also skip toApnsGenericPayload, so the badge isn't incremented for an undelivered push.
-    const interactiveVisible = isAnyInteractiveClientVisible?.() === true;
-    return Promise.all([
-      Promise.resolve(sendPushToAllUiSessions?.(payload, options)).catch((error) => {
-        console.warn('[Push] web-push fanout failed:', error?.message ?? error);
-      }),
-      interactiveVisible
-        ? Promise.resolve()
-        : Promise.resolve(sendApnsToAllUiSessions?.(toApnsGenericPayload(payload), options)).catch((error) => {
-            console.warn('[APNs] fanout failed:', error?.message ?? error);
-          }),
-    ]);
-  };
+  // payload) and native iOS APNs (generic model-based text). Both share the dedup tag;
+  // a failure in one channel must not block the other. Visibility on any other client
+  // never suppresses delivery — every subscribed surface gets the push.
+  const fanoutPush = (payload, options) => Promise.all([
+    Promise.resolve(sendPushToAllUiSessions?.(payload, options)).catch((error) => {
+      console.warn('[Push] web-push fanout failed:', error?.message ?? error);
+    }),
+    Promise.resolve(sendApnsToAllUiSessions?.(toApnsGenericPayload(payload), options)).catch((error) => {
+      console.warn('[APNs] fanout failed:', error?.message ?? error);
+    }),
+  ]);
   let getIsWindowFocused = typeof deps.getIsWindowFocused === 'function'
     ? deps.getIsWindowFocused

package/server/lib/notifications/runtime.test.js CHANGED Viewed

@@ -28,7 +28,6 @@ const createRuntime = (overrides = {}) => {
     sendPushToAllUiSessions,
     sendApnsToAllUiSessions,
     sendLiveActivityEnd,
-    isAnyInteractiveClientVisible: () => false,
     buildOpenCodeUrl: (path) => `http://opencode.test${path}`,
     getOpenCodeAuthHeaders: () => ({}),
     getIsWindowFocused: () => false,
@@ -224,14 +223,13 @@ describe('notification trigger live activity end', () => {
     expect(sendApnsToAllUiSessions).not.toHaveBeenCalled();
   });
-  it('ends the live activity even when an interactive client is visible', async () => {
+  it('still sends native push when another client is visible', async () => {
     vi.stubGlobal('fetch', vi.fn(async () => rootSessionResponse()));
-    const { runtime, sendLiveActivityEnd, sendApnsToAllUiSessions } = createRuntime({
-      isAnyInteractiveClientVisible: () => true,
-    });
+    const { runtime, sendLiveActivityEnd, sendApnsToAllUiSessions, sendPushToAllUiSessions } = createRuntime();
     await runtime.maybeSendPushForTrigger(completionPayload());
     expect(sendLiveActivityEnd).toHaveBeenCalledWith({ sessionId: 'ses_root', status: 'complete' });
-    expect(sendApnsToAllUiSessions).not.toHaveBeenCalled();
+    expect(sendPushToAllUiSessions).toHaveBeenCalledTimes(1);
+    expect(sendApnsToAllUiSessions).toHaveBeenCalledTimes(1);
   });
   it('suppresses live activity end for child and small-model sessions', async () => {