npm - @openchambery/web - Versions diffs - 1.18.4-beta.9 → 1.18.4 - Mend

@openchambery/web 1.18.4-beta.9 → 1.18.4

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

package/dist/index.html CHANGED Viewed

@@ -552,7 +552,7 @@
         opacity: 0.72;
       }
     </style>
-    <script type="module" crossorigin src="/assets/main-DOGudNxM.js"></script>
+    <script type="module" crossorigin src="/assets/main-CTzdOaGn.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">
@@ -569,7 +569,7 @@
     <link rel="modulepreload" crossorigin href="/assets/terminalApi-KDUc5mh2.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-Dofg9S27.js">
+    <link rel="modulepreload" crossorigin href="/assets/runtimeConfig-CCrS3B1v.js">
     <link rel="stylesheet" crossorigin href="/assets/src-Dbdlm5x1.css">
   </head>
   <body class="h-full bg-background text-foreground">

package/dist/mini-chat.html CHANGED Viewed

@@ -74,7 +74,7 @@
       }
     </style>
-    <script type="module" crossorigin src="/assets/miniChat-C465ME90.js"></script>
+    <script type="module" crossorigin src="/assets/miniChat-CAbceO-F.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">
@@ -91,7 +91,7 @@
     <link rel="modulepreload" crossorigin href="/assets/terminalApi-KDUc5mh2.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-Dofg9S27.js">
+    <link rel="modulepreload" crossorigin href="/assets/runtimeConfig-CCrS3B1v.js">
     <link rel="stylesheet" crossorigin href="/assets/src-Dbdlm5x1.css">
   </head>
   <body class="h-full bg-background text-foreground">

package/dist/mobile.html CHANGED Viewed

@@ -63,7 +63,7 @@
       })();
     </script>
-    <script type="module" crossorigin src="/assets/mobile-BHRw1m5c.js"></script>
+    <script type="module" crossorigin src="/assets/mobile-BX6ehEeM.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">
@@ -80,7 +80,7 @@
     <link rel="modulepreload" crossorigin href="/assets/terminalApi-KDUc5mh2.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-Dofg9S27.js">
+    <link rel="modulepreload" crossorigin href="/assets/runtimeConfig-CCrS3B1v.js">
     <link rel="stylesheet" crossorigin href="/assets/src-Dbdlm5x1.css">
   </head>
   <body class="h-full bg-background text-foreground">

package/package.json CHANGED Viewed

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

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

@@ -14,8 +14,10 @@ under `metadata.openchamber.assist`.
    `retry` status or a user `message.updated` clears it (the "1 minute of
    quiet" rule).
 3. On fire: fetch the session (skip sub-agent sessions with `parentID`),
-   take the LAST exchange only — the final assistant reply plus the user
-   message it answered (assistant `parentID` → user id) — and call
+   then build a trimmed excerpt via `buildAssistTranscript`: the LAST text
+   block of the last assistant message (earlier blocks, reasoning, and tool
+   output are excluded) plus the latest 3 real user messages (synthetic
+   hidden-instruction parts never count). The excerpt is sent to
    `generateSmallModelText` with the
    session's own provider/model taken from the last assistant message — so
    the utility call spends the same subscription as the conversation.

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

@@ -33,6 +33,8 @@ const isSessionRecapEnabled = () => {
 const IDLE_QUIET_MS = 60_000;
 const TRANSCRIPT_MESSAGE_LIMIT = 12;
+/** Latest real user messages fed to the recap prompt. */
+const TRANSCRIPT_USER_MESSAGE_LIMIT = 3;
 const TRANSCRIPT_PART_CHAR_LIMIT = 6_000;
 const RECAP_CHAR_LIMIT = 320;
 const FETCH_TIMEOUT_MS = 5_000;
@@ -92,13 +94,63 @@ const extractUserMessage = (payload) => {
   };
 };
-const messagePartsToText = (message) => {
+/** Real (non-synthetic) text parts of a message — hidden instructions excluded. */
+const realTextParts = (message) => {
   const parts = Array.isArray(message?.parts) ? message.parts : [];
   return parts
-    .map((part) => (part?.type === 'text' && typeof part.text === 'string' ? part.text : ''))
-    .filter(Boolean)
-    .join('\n')
-    .slice(0, TRANSCRIPT_PART_CHAR_LIMIT);
+    .filter((part) => part?.type === 'text'
+      && typeof part.text === 'string'
+      && part.text.trim().length > 0
+      && part.synthetic !== true)
+    .map((part) => part.text);
+};
+/** Body text of a message: only its LAST text block, char-clamped. */
+const lastMessageText = (message) => {
+  const texts = realTextParts(message);
+  return texts.length > 0 ? texts[texts.length - 1].slice(0, TRANSCRIPT_PART_CHAR_LIMIT) : '';
+};
+/**
+ * Build the recap transcript from a bounded recent-message window.
+ * - Assistant: only the LAST text block of the last assistant message —
+ *   earlier blocks, reasoning, and tool output are token waste for a
+ *   one-line recap.
+ * - User: the latest real user messages (default 3), chronological.
+ */
+export const buildAssistTranscript = (
+  messages,
+  { userLimit = TRANSCRIPT_USER_MESSAGE_LIMIT } = {},
+) => {
+  if (!Array.isArray(messages) || messages.length === 0) {
+    return { transcript: '', assistantInfo: null, userTexts: [], assistantText: '' };
+  }
+  let lastAssistant = null;
+  for (let i = messages.length - 1; i >= 0; i -= 1) {
+    if (messages[i]?.info?.role === 'assistant') {
+      lastAssistant = messages[i];
+      break;
+    }
+  }
+  const lastAssistantInfo = lastAssistant?.info ?? null;
+  if (!lastAssistantInfo?.id) {
+    return { transcript: '', assistantInfo: null, userTexts: [], assistantText: '' };
+  }
+  const userTexts = [];
+  for (let i = messages.length - 1; i >= 0 && userTexts.length < userLimit; i -= 1) {
+    const message = messages[i];
+    if (message?.info?.role !== 'user') continue;
+    const text = lastMessageText(message);
+    if (!text) continue;
+    userTexts.unshift(text);
+  }
+  const assistantText = lastMessageText(lastAssistant);
+  const transcript = [
+    ...userTexts.map((text) => `User:\n${text}`),
+    assistantText ? `Assistant:\n${assistantText}` : '',
+  ].filter(Boolean).join('\n\n');
+  return { transcript, assistantInfo: lastAssistantInfo, userTexts, assistantText };
 };
 export const createSessionAssistRuntime = ({
@@ -169,36 +221,21 @@ export const createSessionAssistRuntime = ({
       return;
     }
-    let lastAssistant = null;
-    for (let i = messages.length - 1; i >= 0; i -= 1) {
-      const info = messages[i]?.info;
-      if (info?.role === 'assistant') {
-        lastAssistant = messages[i];
-        break;
-      }
-    }
-    const lastAssistantInfo = lastAssistant?.info;
+    // Trimmed extraction for a one-line recap: the last assistant text block
+    // plus the latest real user messages. Everything else (earlier blocks,
+    // reasoning, tool output, older turns) is token waste.
+    const { transcript, assistantInfo: lastAssistantInfo, userTexts, assistantText } = buildAssistTranscript(messages);
     if (!lastAssistantInfo?.id) return;
-    // Only the last exchange: the assistant reply plus the user message it
-    // answered (assistant info.parentID → user info.id). Everything else is
-    // token waste for a one-line recap.
-    const parentUserMessage = typeof lastAssistantInfo.parentID === 'string' && lastAssistantInfo.parentID
-      ? messages.find((message) => message?.info?.id === lastAssistantInfo.parentID && message?.info?.role === 'user')
-      : null;
-    const userText = parentUserMessage ? messagePartsToText(parentUserMessage) : '';
-    const assistantText = messagePartsToText(lastAssistant);
-    const transcript = [
-      userText ? `User:\n${userText}` : '',
-      assistantText ? `Assistant:\n${assistantText}` : '',
-    ].filter(Boolean).join('\n\n');
     if (!transcript) return;
     const { generateSmallModelText } = await getSmallModelService();
     // Instruct the language by example, not by description — account-side
     // personalization (e.g. the ChatGPT backend knowing the user's locale)
     // otherwise leaks a different language into the output.
-    const languageSample = (userText || assistantText).slice(0, 200).replace(/\s+/g, ' ').trim();
+    const languageSample = (userTexts[userTexts.length - 1] || assistantText)
+      .slice(0, 200)
+      .replace(/\s+/g, ' ')
+      .trim();
     let generated;
     try {
       generated = await generateSmallModelText({
@@ -206,7 +243,7 @@ export const createSessionAssistRuntime = ({
         // session's own provider unless the user explicitly picked a small
         // model (settings override / opencode config).
         restrictToPreferredProvider: true,
-        prompt: `The latest exchange in the conversation:\n\n${transcript}\n\nWrite recap in the SAME language as this sample from the conversation: "${languageSample}"`,
+        prompt: `The latest conversation excerpt:\n\n${transcript}\n\nWrite recap in the SAME language as this sample from the conversation: "${languageSample}"`,
         system: ASSIST_SYSTEM_PROMPT,
         directory,
         preferredProviderID: typeof lastAssistantInfo.providerID === 'string' ? lastAssistantInfo.providerID : undefined,
@@ -228,7 +265,7 @@ export const createSessionAssistRuntime = ({
     // no Cyrillic/CJK at all, the output must not either.
     const hasCyrillic = (text) => /[\u0400-\u04FF]/.test(text);
     const hasCjk = (text) => /[\u3040-\u30FF\u4E00-\u9FFF\uAC00-\uD7AF]/.test(text);
-    const inputText = `${userText}\n${assistantText}`;
+    const inputText = `${userTexts.join('\n')}\n${assistantText}`;
     const scriptMismatch = (text) => (hasCyrillic(text) && !hasCyrillic(inputText))
       || (hasCjk(text) && !hasCjk(inputText));
     if (recap && scriptMismatch(recap)) {

package/server/lib/session-assist/runtime.test.js ADDED Viewed

@@ -0,0 +1,114 @@
+import { describe, expect, test } from 'vitest';
+import { buildAssistTranscript } from './runtime.js';
+const textPart = (text, { synthetic = false } = {}) => ({
+  type: 'text',
+  text,
+  ...(synthetic ? { synthetic: true } : {}),
+});
+const userMessage = (id, parts) => ({ info: { id, role: 'user' }, parts });
+const assistantMessage = (id, parts) => ({ info: { id, role: 'assistant' }, parts });
+describe('buildAssistTranscript', () => {
+  test('assistant keeps only the LAST text block of the last assistant message', () => {
+    const result = buildAssistTranscript([
+      userMessage('u1', [textPart('first user message')]),
+      assistantMessage('a1', [textPart('old assistant reply')]),
+      userMessage('u2', [textPart('second user message')]),
+      assistantMessage('a2', [
+        textPart('intro before tools'),
+        { type: 'tool', tool: 'bash' },
+        textPart('final body text'),
+      ]),
+    ]);
+    expect(result.assistantInfo.id).toBe('a2');
+    expect(result.assistantText).toBe('final body text');
+    expect(result.transcript).toBe(
+      'User:\nfirst user message\n\nUser:\nsecond user message\n\nAssistant:\nfinal body text',
+    );
+  });
+  test('reasoning parts never enter the transcript', () => {
+    const result = buildAssistTranscript([
+      userMessage('u1', [textPart('请帮我看看')]),
+      assistantMessage('a1', [
+        { type: 'reasoning', text: 'thinking about the problem' },
+        textPart('最终答复'),
+      ]),
+    ]);
+    expect(result.transcript).toBe('User:\n请帮我看看\n\nAssistant:\n最终答复');
+    expect(result.transcript).not.toContain('thinking');
+  });
+  test('user side keeps only the latest 3 real user messages, chronologically', () => {
+    const result = buildAssistTranscript([
+      userMessage('u1', [textPart('question 1')]),
+      assistantMessage('a1', [textPart('answer 1')]),
+      userMessage('u2', [textPart('question 2')]),
+      assistantMessage('a2', [textPart('answer 2')]),
+      userMessage('u3', [textPart('question 3')]),
+      userMessage('u4', [textPart('question 4')]),
+      assistantMessage('a3', [textPart('answer 3')]),
+    ]);
+    expect(result.userTexts).toEqual(['question 2', 'question 3', 'question 4']);
+    expect(result.transcript).toBe(
+      'User:\nquestion 2\n\nUser:\nquestion 3\n\nUser:\nquestion 4\n\nAssistant:\nanswer 3',
+    );
+  });
+  test('synthetic-only user messages do not count toward the user window', () => {
+    const result = buildAssistTranscript([
+      userMessage('u1', [textPart('visible request')]),
+      userMessage('u2', [textPart('hidden instruction', { synthetic: true })]),
+      userMessage('u3', [textPart('real follow-up')]),
+      assistantMessage('a1', [textPart('reply body')]),
+    ]);
+    expect(result.userTexts).toEqual(['visible request', 'real follow-up']);
+    expect(result.transcript).not.toContain('hidden instruction');
+  });
+  test('synthetic instruction part on a real user message is dropped', () => {
+    const result = buildAssistTranscript([
+      userMessage('u1', [
+        textPart('visible request'),
+        textPart('/summary hidden instructions', { synthetic: true }),
+      ]),
+      assistantMessage('a1', [textPart('reply body')]),
+    ]);
+    expect(result.userTexts).toEqual(['visible request']);
+    expect(result.transcript).not.toContain('/summary hidden instructions');
+  });
+  test('fewer than 3 user messages keeps all of them', () => {
+    const result = buildAssistTranscript([
+      userMessage('u1', [textPart('only question')]),
+      assistantMessage('a1', [textPart('only answer')]),
+    ]);
+    expect(result.userTexts).toEqual(['only question']);
+    expect(result.transcript).toBe('User:\nonly question\n\nAssistant:\nonly answer');
+  });
+  test('no assistant message yields an empty transcript', () => {
+    const result = buildAssistTranscript([
+      userMessage('u1', [textPart('unanswered question')]),
+    ]);
+    expect(result.transcript).toBe('');
+    expect(result.assistantInfo).toBeNull();
+    expect(result.userTexts).toEqual([]);
+  });
+  test('assistant with no text parts still summarizes the user side', () => {
+    const result = buildAssistTranscript([
+      userMessage('u1', [textPart('question')]),
+      assistantMessage('a1', [{ type: 'tool', tool: 'bash' }]),
+    ]);
+    expect(result.assistantInfo.id).toBe('a1');
+    expect(result.transcript).toBe('User:\nquestion');
+  });
+  test('empty input yields an empty transcript', () => {
+    expect(buildAssistTranscript([]).transcript).toBe('');
+    expect(buildAssistTranscript(null).transcript).toBe('');
+  });
+});