@yeaft/webchat-agent 0.1.752 → 0.1.753

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@yeaft/webchat-agent",
-  "version": "0.1.752",
+  "version": "0.1.753",
   "description": "Remote agent for Yeaft WebChat — connects worker machines to the central server",
   "main": "index.js",
   "type": "module",

package/unify/compact/compactor.js

@@ -62,12 +62,17 @@ export class Compactor {
    *        treats that as a soft failure).
    * @param {() => number|undefined} [opts.getMaxContextTokens]
    *        Returns `config.maxContextTokens` for `shouldCompactHistory`.
+   * @param {() => string|undefined} [opts.getLanguage]
+   *        Returns the live `config.language`. Threaded into
+   *        `compactHistory` so the compactor's summary prompt + the
+   *        "session continued" wrapper render in the user's preferred
+   *        locale instead of always English.
    * @param {(groupId: string, result: CompactedResult) => void} [opts.onCompacted]
    *        Optional sink. Bridge wires this to send the
    *        `unify_history_compacted` WS event. Default: no-op. Can be
    *        replaced post-construction via `setOnCompacted`.
    */
-  constructor({ summarize, getMaxContextTokens, onCompacted } = {}) {
+  constructor({ summarize, getMaxContextTokens, getLanguage, onCompacted } = {}) {
     if (typeof summarize !== 'function') {
       throw new TypeError('Compactor: summarize is required');
     }
@@ -75,6 +80,9 @@ export class Compactor {
     this._getMaxContextTokens = typeof getMaxContextTokens === 'function'
       ? getMaxContextTokens
       : () => undefined;
+    this._getLanguage = typeof getLanguage === 'function'
+      ? getLanguage
+      : () => undefined;
     this._onCompacted = typeof onCompacted === 'function' ? onCompacted : () => {};
     /** @type {Map<string, { inFlight: Promise<void>|null, pending: boolean }>} */
     this._states = new Map();
@@ -186,7 +194,11 @@ export class Compactor {
       const summarize = ({ system, prompt }) =>
         this._summarize({ system, prompt, maxTokens: SUMMARIZER_MAX_TOKENS });
 
-      const result = await compactHistory(snapshot, { summarize, maxContextTokens });
+      const result = await compactHistory(snapshot, {
+        summarize,
+        maxContextTokens,
+        language: this._getLanguage(),
+      });
       if (!result || !result.compacted) {
         if (result && result.error) {
           console.warn(

package/unify/engine.js

@@ -975,13 +975,23 @@ export class Engine {
 
     const archiveIds = [];
 
+    // Language-aware summarizer prompts. The orchestrator-track summary
+    // ends up in the system prompt as a "previous conversation summary"
+    // block, so it needs to match the user's preferred language to avoid
+    // a jarring locale flip mid-context.
+    const isZh = String(this.#config.language || '').toLowerCase().startsWith('zh');
+    const summariserSystem = isZh
+      ? '你是对话摘要器。请用中文写出 2–3 段简明摘要，保留决策、事实与上下文。'
+      : 'You are a conversation summarizer. Summarize concisely in 2–3 paragraphs, preserving decisions, facts, and context.';
+    const summariserPromptPrefix = isZh ? '请概括：\n\n' : 'Summarize:\n\n';
+
     const hooks = {
       summarise: async () => {
         try {
           const result = await adapter.call({
             model: fastConfig.model,
-            system: 'You are a conversation summarizer. Summarize concisely in 2–3 paragraphs, preserving decisions, facts, and context.',
-            messages: [{ role: 'user', content: `Summarize:\n\n${toArchive.map(m => `[${m.role}] ${(m.content || '').slice(0, 500)}`).join('\n\n')}` }],
+            system: summariserSystem,
+            messages: [{ role: 'user', content: `${summariserPromptPrefix}${toArchive.map(m => `[${m.role}] ${(m.content || '').slice(0, 500)}`).join('\n\n')}` }],
             maxTokens: 1024,
           });
           return (result.text || '').trim();

package/unify/history-compact.js

@@ -328,16 +328,22 @@ export function findCutIndex(messages, keepRecent) {
  * `server/db/message-db.js`) recognise it.
  *
  * @param {string} summary
+ * @param {{ language?: string }} [opts]
  * @returns {{role:'user', content:string, _compactSummary: true}}
  */
-export function wrapSummaryAsUserMessage(summary) {
+export function wrapSummaryAsUserMessage(summary, opts = {}) {
   const body = (summary || '').trim() || '(no summary produced)';
-  const content =
-    'This session is being continued from a previous conversation. ' +
-    'The earlier context has been summarized for efficiency.\n\n' +
-    'Summary of conversation so far:\n' +
-    body +
-    '\n\nContinue the conversation from where it left off without asking the user any further questions.';
+  const isZh = String(opts.language || '').toLowerCase().startsWith('zh');
+  const content = isZh
+    ? '本会话延续自之前的对话。早期上下文已经被概括以节省空间。\n\n' +
+      '至此为止的对话摘要：\n' +
+      body +
+      '\n\n请从中断处继续对话，不要再向用户重复确认。'
+    : 'This session is being continued from a previous conversation. ' +
+      'The earlier context has been summarized for efficiency.\n\n' +
+      'Summary of conversation so far:\n' +
+      body +
+      '\n\nContinue the conversation from where it left off without asking the user any further questions.';
   return {
     role: 'user',
     content,
@@ -349,24 +355,36 @@ export function wrapSummaryAsUserMessage(summary) {
  * Build the prompt fed to the fast-model summarizer. Kept in code (not in
  * a template file) because it's small and lives alongside the call site.
  *
+ * The summarizer prompt itself is language-aware: callers pass the live
+ * `config.language` so the produced summary is written in the user's
+ * preferred language. JSON-style structural cues stay English so the
+ * summary remains easy to splice into the next turn regardless of locale.
+ *
  * @param {Array<{role:string, content:string}>} cleanedMessages
+ * @param {{ language?: string }} [opts]
  * @returns {{system: string, prompt: string}}
  */
-export function buildSummaryPrompt(cleanedMessages) {
+export function buildSummaryPrompt(cleanedMessages, opts = {}) {
   const transcript = cleanedMessages
     .map(m => `[${m.role}]\n${m.content}`)
     .join('\n\n---\n\n');
-  const system =
-    'You are a conversation summarizer for a multi-agent group chat. ' +
-    'Produce a concise (4–8 short bullet points) summary of the conversation ' +
-    'so far. Preserve: (1) decisions made, (2) facts learned, (3) the user\'s ' +
-    'current goal, (4) any open questions or pending actions, (5) which VPs ' +
-    'are participating and what each contributed. Do NOT include raw tool ' +
-    'output. Do NOT speculate. Be specific.';
-  const prompt =
-    'Summarize the following conversation. Output ONLY the summary, no ' +
-    'preamble.\n\n' +
-    transcript;
+  const isZh = String(opts.language || '').toLowerCase().startsWith('zh');
+  const system = isZh
+    ? '你是多 agent 群聊的对话摘要器。请用中文写出 4–8 条简明 bullet 摘要。' +
+      '保留：(1) 已做的决策，(2) 已学到的事实，(3) 用户当前目标，' +
+      '(4) 任何未解决的问题或待办事项，(5) 哪些 VP 参与了对话以及各自贡献。' +
+      '不要包含原始工具输出。不要臆测。要具体。'
+    : 'You are a conversation summarizer for a multi-agent group chat. ' +
+      'Produce a concise (4–8 short bullet points) summary of the conversation ' +
+      'so far. Preserve: (1) decisions made, (2) facts learned, (3) the user\'s ' +
+      'current goal, (4) any open questions or pending actions, (5) which VPs ' +
+      'are participating and what each contributed. Do NOT include raw tool ' +
+      'output. Do NOT speculate. Be specific.';
+  const prompt = isZh
+    ? '请概括下面的对话。只输出摘要正文，不要前言。\n\n' + transcript
+    : 'Summarize the following conversation. Output ONLY the summary, no ' +
+      'preamble.\n\n' +
+      transcript;
   return { system, prompt };
 }
 
@@ -384,6 +402,7 @@ export function buildSummaryPrompt(cleanedMessages) {
  *   maxContextTokens?: number,
  *   tokenFraction?: number,
  *   hardTokenCeiling?: number,
+ *   language?: string,
  * }} options
  * @returns {Promise<{
  *   messages: Array<object>,
@@ -407,6 +426,7 @@ export async function compactHistory(messages, options) {
     maxContextTokens,
     tokenFraction,
     hardTokenCeiling,
+    language,
   } = options || {};
 
   if (typeof summarize !== 'function') {
@@ -460,7 +480,7 @@ export async function compactHistory(messages, options) {
 
   let summaryText = '';
   if (cleaned.length > 0) {
-    const { system, prompt } = buildSummaryPrompt(cleaned);
+    const { system, prompt } = buildSummaryPrompt(cleaned, { language });
     try {
       summaryText = (await summarize({ system, prompt })) || '';
     } catch (err) {
@@ -498,7 +518,7 @@ export async function compactHistory(messages, options) {
     }
   }
 
-  const summaryMsg = wrapSummaryAsUserMessage(summaryText);
+  const summaryMsg = wrapSummaryAsUserMessage(summaryText, { language });
 
   // Defensive pair-sanitize: the cut at `cutIdx` lands at a user-message
   // boundary so an `[assistant(toolCalls), tool…]` arc is not split, but

package/unify/session.js

@@ -315,6 +315,13 @@ export async function loadSession(options = {}) {
       engine.summarizeForCompact({ system, prompt, maxTokens }),
     getMaxContextTokens: () =>
       typeof config.maxContextTokens === 'number' ? config.maxContextTokens : undefined,
+    // Live-read: `config.language` is mutated in place by
+    // `engine.setLanguage()` (which broadcastLanguageChange fans out to
+    // every per-VP engine). The compactor must see the post-broadcast
+    // value, not a boot-time snapshot, so the summary prompt + the
+    // "session continued" wrapper render in the user's current locale.
+    getLanguage: () =>
+      typeof config.language === 'string' ? config.language : undefined,
   });
 
   // ─── 9a. Create dream scheduler ────────────

package/unify/tools/registry.js

@@ -61,16 +61,44 @@ function localizeVisibleText(value, language, toolName) {
   ].join('\n');
 }
 
+/**
+ * Walk a JSON Schema `parameters` object and localize the human-readable
+ * `description` strings to the requested language. All other schema bits
+ * (`type`, `enum`, `required`, `items`, nested `properties`, etc.) are
+ * preserved by value.
+ *
+ * Critical correctness rule: a JSON Schema can have a *property named*
+ * `description` whose value is itself a sub-schema (e.g.
+ * `properties: { description: { type: 'string', description: 'Detailed...' }}`).
+ * In that case the OUTER `description` key holds the sub-schema and must
+ * be recursed into; only the INNER `description: 'Detailed...'` value
+ * (which is a string) should be localized.
+ *
+ * Previous bug: this walker localized ANY value under a `description`
+ * key, including sub-schema objects. `localizeVisibleText` then ran
+ * `String(value)` on the object, producing the literal string
+ * `'[object Object]'`. GPT-5's strict schema validator rejected the
+ * resulting `{ description: '[object Object]' }` with
+ * `"'[object Object]' is not of type 'object', 'boolean'"`. This made
+ * FeatureCreate (and any tool whose schema contains a property named
+ * `description`) unusable in zh locale on strict providers.
+ *
+ * The fix: only treat a `description` value as localizable text when it
+ * is actually a string. Object/array values under a `description` key
+ * are sub-schemas and must be recursed into normally.
+ */
 function localizeParameters(parameters, language, toolName) {
   const lang = normalizeLanguage(language);
   if (lang !== 'zh' || !parameters || typeof parameters !== 'object') return parameters;
   if (Array.isArray(parameters)) return parameters.map(v => localizeParameters(v, lang, toolName));
   const out = {};
   for (const [key, value] of Object.entries(parameters)) {
-    if (key === 'description') {
+    if (key === 'description' && typeof value === 'string') {
       out[key] = localizeVisibleText(value, lang, toolName);
-    } else {
+    } else if (value && typeof value === 'object') {
       out[key] = localizeParameters(value, lang, toolName);
+    } else {
+      out[key] = value;
     }
   }
   return out;