npm - @openchambery/web - Versions diffs - 1.18.2-beta.15 → 1.18.2-beta.17 - Mend

@openchambery/web 1.18.2-beta.15 → 1.18.2-beta.17

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 (57) hide show

package/dist/index.html CHANGED Viewed

@@ -552,7 +552,7 @@
         opacity: 0.72;
       }
     </style>
-    <script type="module" crossorigin src="/assets/main-CKV54Ayt.js"></script>
+    <script type="module" crossorigin src="/assets/main-Cu7sZyJS.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">
@@ -568,7 +568,7 @@
     <link rel="modulepreload" crossorigin href="/assets/gitApiHttp-BxGAFLTf.js">
     <link rel="modulepreload" crossorigin href="/assets/terminalApi-DpiAJlil.js">
     <link rel="modulepreload" crossorigin href="/assets/runtimeConfig-BXCt7ZsQ.js">
-    <link rel="stylesheet" crossorigin href="/assets/src-C8jn0x69.css">
+    <link rel="stylesheet" crossorigin href="/assets/src-DcPZ_rWS.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,7 +74,7 @@
       }
     </style>
-    <script type="module" crossorigin src="/assets/miniChat-D8VtCIh9.js"></script>
+    <script type="module" crossorigin src="/assets/miniChat-CwrVreq-.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">
@@ -90,7 +90,7 @@
     <link rel="modulepreload" crossorigin href="/assets/gitApiHttp-BxGAFLTf.js">
     <link rel="modulepreload" crossorigin href="/assets/terminalApi-DpiAJlil.js">
     <link rel="modulepreload" crossorigin href="/assets/runtimeConfig-BXCt7ZsQ.js">
-    <link rel="stylesheet" crossorigin href="/assets/src-C8jn0x69.css">
+    <link rel="stylesheet" crossorigin href="/assets/src-DcPZ_rWS.css">
   </head>
   <body class="h-full bg-background text-foreground">
     <div id="root" class="h-full">

package/dist/mobile.html CHANGED Viewed

@@ -63,7 +63,7 @@
       })();
     </script>
-    <script type="module" crossorigin src="/assets/mobile-B8jXmmPT.js"></script>
+    <script type="module" crossorigin src="/assets/mobile-SQEAwCJ8.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">
@@ -79,7 +79,7 @@
     <link rel="modulepreload" crossorigin href="/assets/gitApiHttp-BxGAFLTf.js">
     <link rel="modulepreload" crossorigin href="/assets/terminalApi-DpiAJlil.js">
     <link rel="modulepreload" crossorigin href="/assets/runtimeConfig-BXCt7ZsQ.js">
-    <link rel="stylesheet" crossorigin href="/assets/src-C8jn0x69.css">
+    <link rel="stylesheet" crossorigin href="/assets/src-DcPZ_rWS.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.18.2-beta.15",
+  "version": "1.18.2-beta.17",
   "private": false,
   "type": "module",
   "main": "./server/index.js",

package/server/lib/session-title/DOCUMENTATION.md CHANGED Viewed

@@ -5,10 +5,11 @@ conversation's **main subject** (overall feature / goal) with the small model
 (`lib/small-model`), then PATCHes `title` plus `metadata.openchamber.titleRefresh`.
 OpenCode only auto-titles once from the first user message
-(`SessionPrompt.ensureTitle`). Long sessions often need a refresh; this module
-keeps the sidebar title aligned with the durable work being done — not the last
-wrap-up utterance like "commit and push" — throttled to at most once per 5
-minutes per session.
+(`SessionPrompt.ensureTitle`). This module auto-refreshes sparsely (first idle
+of a new session, first newly-sent reply on a fork) and on explicit smart-title
+requests, naming the durable work being done — not the last wrap-up utterance
+like "commit and push". Background auto refreshes still respect a 5-minute
+throttle (`TITLE_THROTTLE_MS`) when a refresh is armed.
 ## Flow
@@ -16,14 +17,17 @@ minutes per session.
    fan-out (`index.js` → `globalMessageStreamHub.subscribeEvent`), same
    pattern as session-assist. Purely event-driven — dormant sessions never
    generate anything.
-2. A newly observed root `session.created` generates its first title immediately
-   on the first `session.status: idle`. A fork title (`(fork #n)`) waits for its
-   first newly-created user message; the matching assistant completion triggers
-   an immediate title refresh that bypasses inherited title metadata and throttle.
-   Later idle transitions arm a 15-second quiet timer; any `busy`/`retry` status
-   or a fresh user `message.updated` clears it.
-   A sidebar smart-title request sets `titleRefresh.requestedAt`; its
-   `session.updated` event arms the same flow immediately.
+  2. Auto title refresh is intentionally sparse (title stability first). A newly
+    observed root `session.created` generates its first title immediately on the
+    first `session.status: idle`. A fork title (`(fork #n)`) waits for its first
+    newly-created user message; the matching assistant completion triggers an
+    immediate title refresh that bypasses inherited title metadata and throttle.
+    Ordinary later idle transitions do **not** arm another refresh. Any
+    `busy`/`retry` status or a fresh user `message.updated` still clears an
+    already-armed timer (so initial/fork timers cancel if the user keeps going).
+    A sidebar smart-title request sets `titleRefresh.requestedAt`; its
+    `session.updated` event arms the same flow immediately (forced / manual
+    refresh is unaffected by the background gate).
   3. On fire:
     - Skip when Settings → Chat → session title refresh is off.
     - Skip system-owned sessions whose metadata carries a non-empty

package/server/lib/session-title/runtime.js CHANGED Viewed

@@ -1,13 +1,14 @@
-// Session title refresh: after a session goes idle, regenerate the sidebar
-// title from the conversation's MAIN SUBJECT (overall feature / goal), not
-// just the last wrap-up utterance. OpenCode only titles once from the first
-// user message; long sessions need refresh, but continuing the same work
+// Session title refresh: regenerate the sidebar title from the conversation's
+// MAIN SUBJECT (overall feature / goal), not just the last wrap-up utterance.
+// OpenCode only titles once from the first user message. Auto refresh is sparse
+// for title stability: first idle of a new session, first newly-sent reply on a
+// fork, plus explicit smart-title / forced refresh. Continuing the same work
 // should keep naming the thing being done — not "commit and push". This
 // watcher throttles to at most one refresh per session every TITLE_THROTTLE_MS
 // and never overwrites a user-renamed title.
 //
-// Purely event-driven: only sessions that transition busy→idle while the
-// server is running ever generate anything. No backfill, no session scans.
+// Purely event-driven: only sessions that arm a refresh while the server is
+// running ever generate anything. No backfill, no session scans.
 import fs from 'fs';
 import os from 'os';
@@ -742,8 +743,6 @@ export const createSessionTitleRuntime = ({
         if (newSessions.delete(status.sessionId)) {
           initialRefreshes.add(status.sessionId);
           armTimer(status.sessionId, status.directory || directoryHint, 0);
-        } else {
-          armTimer(status.sessionId, status.directory || directoryHint, quietMs);
         }
       } else {
         clearTimer(status.sessionId);

package/server/lib/session-title/runtime.test.js CHANGED Viewed

@@ -403,4 +403,229 @@ describe('session-title helpers', () => {
       globalThis.fetch = originalFetch;
     }
   });
+  it('refreshes a new session on first idle, then ignores later idle transitions', async () => {
+    const originalFetch = globalThis.fetch;
+    const session = {
+      id: 'ses-new',
+      title: 'New session - 2026-07-10T12:00:00.000Z',
+      metadata: {},
+    };
+    const messages = [
+      { info: { id: 'u1', role: 'user' }, parts: [{ type: 'text', text: 'Add rate limiting' }] },
+      { info: { id: 'a1', role: 'assistant' }, parts: [{ type: 'text', text: 'Implemented rate limit' }] },
+    ];
+    let generationCalls = 0;
+    let patchedTitle = session.title;
+    globalThis.fetch = async (input, init = {}) => {
+      const url = String(input);
+      if (url.includes('/message')) {
+        return new Response(JSON.stringify(messages), { status: 200 });
+      }
+      if (init.method === 'PATCH') {
+        const body = typeof init.body === 'string' ? JSON.parse(init.body) : init.body;
+        if (typeof body?.title === 'string') {
+          patchedTitle = body.title;
+          session.title = body.title;
+          session.metadata = body.metadata || session.metadata;
+        } else if (body?.metadata) {
+          session.metadata = body.metadata;
+        }
+        return new Response(JSON.stringify(session), { status: 200 });
+      }
+      return new Response(JSON.stringify({ ...session, title: patchedTitle }), { status: 200 });
+    };
+    try {
+      const runtime = createSessionTitleRuntime({
+        buildOpenCodeUrl: (pathname) => `http://opencode${pathname}`,
+        getOpenCodeAuthHeaders: () => ({}),
+        getSmallModelService: async () => ({
+          generateSmallModelText: async () => {
+            generationCalls += 1;
+            return { text: 'Rate limiting', providerID: 'test', modelID: 'test' };
+          },
+        }),
+        now: () => 1_000,
+        isTitleRefreshEnabled: () => true,
+      });
+      runtime.processPayload({
+        type: 'session.created',
+        properties: {
+          directory: '/repo',
+          info: { id: 'ses-new', title: session.title, time: { created: 100 } },
+        },
+      });
+      runtime.processPayload({
+        type: 'session.status',
+        properties: { sessionID: 'ses-new', directory: '/repo', status: { type: 'idle' } },
+      });
+      await new Promise((resolve) => setTimeout(resolve, 25));
+      expect(generationCalls).toBe(1);
+      // Later conversation progress + idle must not auto-refresh again.
+      messages.push(
+        { info: { id: 'u2', role: 'user' }, parts: [{ type: 'text', text: 'Also add retries' }] },
+        { info: { id: 'a2', role: 'assistant' }, parts: [{ type: 'text', text: 'Retries added' }] },
+      );
+      runtime.processPayload({
+        type: 'message.updated',
+        properties: { directory: '/repo', info: { sessionID: 'ses-new', role: 'user', time: { created: 200 } } },
+      });
+      runtime.processPayload({
+        type: 'session.status',
+        properties: { sessionID: 'ses-new', directory: '/repo', status: { type: 'busy' } },
+      });
+      runtime.processPayload({
+        type: 'session.status',
+        properties: { sessionID: 'ses-new', directory: '/repo', status: { type: 'idle' } },
+      });
+      await new Promise((resolve) => setTimeout(resolve, 25));
+      expect(generationCalls).toBe(1);
+      runtime.stop();
+    } finally {
+      globalThis.fetch = originalFetch;
+    }
+  });
+  it('still runs an explicit smart-title forced refresh after initial title', async () => {
+    const originalFetch = globalThis.fetch;
+    const session = {
+      id: 'ses-forced',
+      title: 'Rate limiting',
+      metadata: {
+        openchamber: {
+          titleRefresh: {
+            lastAutoTitle: 'Rate limiting',
+            generatedAt: 500,
+            forMessageID: 'a1',
+          },
+        },
+      },
+    };
+    const messages = [
+      { info: { id: 'u1', role: 'user' }, parts: [{ type: 'text', text: 'Add rate limiting' }] },
+      { info: { id: 'a1', role: 'assistant' }, parts: [{ type: 'text', text: 'Implemented rate limit' }] },
+      { info: { id: 'u2', role: 'user' }, parts: [{ type: 'text', text: 'Switch to token bucket' }] },
+      { info: { id: 'a2', role: 'assistant' }, parts: [{ type: 'text', text: 'Token bucket ready' }] },
+    ];
+    let generationCalls = 0;
+    globalThis.fetch = async (input, init = {}) => {
+      const url = String(input);
+      if (url.includes('/message')) {
+        return new Response(JSON.stringify(messages), { status: 200 });
+      }
+      if (init.method === 'PATCH') {
+        return new Response(JSON.stringify(session), { status: 200 });
+      }
+      return new Response(JSON.stringify(session), { status: 200 });
+    };
+    try {
+      const runtime = createSessionTitleRuntime({
+        buildOpenCodeUrl: (pathname) => `http://opencode${pathname}`,
+        getOpenCodeAuthHeaders: () => ({}),
+        getSmallModelService: async () => ({
+          generateSmallModelText: async () => {
+            generationCalls += 1;
+            return { text: 'Token bucket rate limiting', providerID: 'test', modelID: 'test' };
+          },
+        }),
+        now: () => 1_000,
+        isTitleRefreshEnabled: () => true,
+      });
+      runtime.processPayload({
+        type: 'session.status',
+        properties: { sessionID: 'ses-forced', directory: '/repo', status: { type: 'idle' } },
+      });
+      await new Promise((resolve) => setTimeout(resolve, 25));
+      expect(generationCalls).toBe(0);
+      runtime.processPayload({
+        type: 'session.updated',
+        properties: {
+          directory: '/repo',
+          info: {
+            id: 'ses-forced',
+            title: session.title,
+            metadata: {
+              openchamber: {
+                titleRefresh: {
+                  ...session.metadata.openchamber.titleRefresh,
+                  requestedAt: 1_000,
+                },
+              },
+            },
+          },
+        },
+      });
+      await new Promise((resolve) => setTimeout(resolve, 25));
+      expect(generationCalls).toBe(1);
+      runtime.stop();
+    } finally {
+      globalThis.fetch = originalFetch;
+    }
+  });
+  it('skips automatic initial refresh when session title refresh is disabled', async () => {
+    const originalFetch = globalThis.fetch;
+    const session = {
+      id: 'ses-disabled',
+      title: 'New session - 2026-07-10T12:00:00.000Z',
+      metadata: {},
+    };
+    let generationCalls = 0;
+    globalThis.fetch = async (input, init = {}) => {
+      if (String(input).includes('/message')) {
+        return new Response(JSON.stringify([
+          { info: { id: 'u1', role: 'user' }, parts: [{ type: 'text', text: 'hello' }] },
+          { info: { id: 'a1', role: 'assistant' }, parts: [{ type: 'text', text: 'world' }] },
+        ]), { status: 200 });
+      }
+      if (init.method === 'PATCH') {
+        return new Response(JSON.stringify(session), { status: 200 });
+      }
+      return new Response(JSON.stringify(session), { status: 200 });
+    };
+    try {
+      const runtime = createSessionTitleRuntime({
+        buildOpenCodeUrl: (pathname) => `http://opencode${pathname}`,
+        getOpenCodeAuthHeaders: () => ({}),
+        getSmallModelService: async () => ({
+          generateSmallModelText: async () => {
+            generationCalls += 1;
+            return { text: 'Should not run', providerID: 'test', modelID: 'test' };
+          },
+        }),
+        now: () => 1_000,
+        isTitleRefreshEnabled: () => false,
+      });
+      runtime.processPayload({
+        type: 'session.created',
+        properties: {
+          directory: '/repo',
+          info: { id: 'ses-disabled', title: session.title, time: { created: 100 } },
+        },
+      });
+      runtime.processPayload({
+        type: 'session.status',
+        properties: { sessionID: 'ses-disabled', directory: '/repo', status: { type: 'idle' } },
+      });
+      await new Promise((resolve) => setTimeout(resolve, 25));
+      expect(generationCalls).toBe(0);
+      runtime.stop();
+    } finally {
+      globalThis.fetch = originalFetch;
+    }
+  });
 });