@openchambery/web 1.19.5-beta.8 → 1.19.5-beta.9

package/dist/assets/{utils-C2R0YoC5.js → utils-D94hxQYq.js}

@@ -1 +1 @@
-import{Xt as e}from"./persistence-DE9RnVcA.js";import{n as t}from"./vendor-class-variance-authority-BLUaYARZ.js";import{t as n}from"./vendor-tailwind-merge-BicNWLTg.js";function r(...e){return n(t(e))}var i=()=>typeof navigator>`u`?!1:/Macintosh|Mac OS X/.test(navigator.userAgent||``),a=()=>typeof navigator>`u`?!1:/Windows/.test(navigator.userAgent||``),o=()=>i()?`common.revealPath.finder`:a()?`common.revealPath.fileExplorer`:`common.revealPath.fileManager`,s=t=>i()&&e()?t.metaKey:t.ctrlKey,c=()=>i()&&e()?`⌘`:`Ctrl`,l=(e,t)=>{let n=e??``,r=Math.max(16,t?.maxLength??45);if(n.length<=r)return n;let i=n.split(`/`);if(i.length<=1)return n;let a=i.pop()??``;if(!a)return n;let o=Math.max(0,r-(a.length+2));if(o<=0)return`…/${a}`;let s=``;for(let e of i){if(!e)continue;let t=s?`${s}/${e}`:e;if(t.length>o)break;s=t}if(!s){let e=i[0]??``;s=e?e.slice(0,o):``}return s?`${s}…/${a}`:`…/${a}`},u=e=>e?e===`/`?`/`:e.replace(/\/+$/,``):``;function d(e,t){if(!e)return``;let n=u(e);if(n===`/`)return`/`;let r=t?u(t):void 0;if(r&&r!==`/`){if(n===r)return`~`;if(n.startsWith(`${r}/`)){let e=n.slice(r.length+1);return e?`~/${e}`:`~`}}return n}function f(e,t){if(!e)return`/`;let n=u(e);if(!n||n===`/`)return`/`;let r=t?u(t):void 0;return r&&r!==`/`&&n===r?`~`:n.split(`/`).pop()||n||`/`}export{o as a,l as c,c as i,f as n,s as o,d as r,i as s,r as t};
+import{Xt as e}from"./persistence-CLUawIlm.js";import{n as t}from"./vendor-class-variance-authority-BLUaYARZ.js";import{t as n}from"./vendor-tailwind-merge-BicNWLTg.js";function r(...e){return n(t(e))}var i=()=>typeof navigator>`u`?!1:/Macintosh|Mac OS X/.test(navigator.userAgent||``),a=()=>typeof navigator>`u`?!1:/Windows/.test(navigator.userAgent||``),o=()=>i()?`common.revealPath.finder`:a()?`common.revealPath.fileExplorer`:`common.revealPath.fileManager`,s=t=>i()&&e()?t.metaKey:t.ctrlKey,c=()=>i()&&e()?`⌘`:`Ctrl`,l=(e,t)=>{let n=e??``,r=Math.max(16,t?.maxLength??45);if(n.length<=r)return n;let i=n.split(`/`);if(i.length<=1)return n;let a=i.pop()??``;if(!a)return n;let o=Math.max(0,r-(a.length+2));if(o<=0)return`…/${a}`;let s=``;for(let e of i){if(!e)continue;let t=s?`${s}/${e}`:e;if(t.length>o)break;s=t}if(!s){let e=i[0]??``;s=e?e.slice(0,o):``}return s?`${s}…/${a}`:`…/${a}`},u=e=>e?e===`/`?`/`:e.replace(/\/+$/,``):``;function d(e,t){if(!e)return``;let n=u(e);if(n===`/`)return`/`;let r=t?u(t):void 0;if(r&&r!==`/`){if(n===r)return`~`;if(n.startsWith(`${r}/`)){let e=n.slice(r.length+1);return e?`~/${e}`:`~`}}return n}function f(e,t){if(!e)return`/`;let n=u(e);if(!n||n===`/`)return`/`;let r=t?u(t):void 0;return r&&r!==`/`&&n===r?`~`:n.split(`/`).pop()||n||`/`}export{o as a,l as c,c as i,f as n,s as o,d as r,i as s,r as t};

package/dist/index.html

@@ -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-4AJr41lz.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-HOGA9fV1.js">
+    <link rel="stylesheet" crossorigin href="/assets/src-Bd_0IB9v.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

@@ -74,25 +74,25 @@
       }
     </style>
 
-    <script type="module" crossorigin src="/assets/miniChat-D3Vj4bki.js"></script>
+    <script type="module" crossorigin src="/assets/miniChat-xTsSFILb.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-HOGA9fV1.js">
+    <link rel="stylesheet" crossorigin href="/assets/src-Bd_0IB9v.css">
   </head>
   <body class="h-full bg-background text-foreground">
     <div id="root" class="h-full">

package/dist/mobile.html

@@ -63,25 +63,25 @@
       })();
     </script>
 
-    <script type="module" crossorigin src="/assets/mobile-6mt3nA3g.js"></script>
+    <script type="module" crossorigin src="/assets/mobile-BCQf1Sdm.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-HOGA9fV1.js">
+    <link rel="stylesheet" crossorigin href="/assets/src-Bd_0IB9v.css">
   </head>
   <body class="h-full bg-background text-foreground">
     <div id="root" class="h-full"></div>

package/package.json

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

package/server/index.js

@@ -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

@@ -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

@@ -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

@@ -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

@@ -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 () => {