@openchambery/web 1.19.13-beta.3 → 1.19.13

package/server/lib/assistants/contact-tools.js

@@ -30,6 +30,8 @@ export const NEW_CONVERSATION_TOOL_NAME = 'new_conversation';
 export const CLEAR_CHAT_HISTORY_TOOL_NAME = 'clear_chat_history';
 export const LIST_PROJECTS_TOOL_NAME = 'list_projects';
 export const LIST_SESSIONS_TOOL_NAME = 'list_sessions';
+export const GET_ASSISTANT_SETTINGS_TOOL_NAME = 'get_assistant_settings';
+export const UPDATE_DEFAULT_PROMPT_TOOL_NAME = 'update_default_prompt';
 const CONTACT_TOOL_FENCE = 'openchamber-tool';
 export const ASSIGNED_SESSION_FALLBACK_BUBBLE = 'Opened a coding session.';
 /** Same-turn duplicate assign (different args after a success, or parallel mismatch). */
@@ -38,6 +40,9 @@ export const ASSIGN_DUPLICATE_TURN_MESSAGE = 'This contact turn already opened a
 export const NEW_CONVERSATION_CONFIRM_BUBBLE = 'Memory cleared. Previous messages stay in the chat; I will not use them as context.';
 /** Explicit transcript wipe confirm (clear_chat_history / POST contact/reset). */
 export const CLEAR_CHAT_HISTORY_CONFIRM_BUBBLE = 'Chat history cleared.';
+/** Default-prompt write confirm: persisted settings, next turns only. */
+export const UPDATE_DEFAULT_PROMPT_CONFIRM_BUBBLE = 'Default prompt saved. It applies on later turns (this turn already built its system prompt).';
+export const UPDATE_DEFAULT_PROMPT_UNCHANGED_BUBBLE = 'Default prompt is already that value — nothing changed.';
 const LIST_SESSIONS_LIMIT_DEFAULT = 20;
 const LIST_SESSIONS_LIMIT_MAX = 50;
 
@@ -82,9 +87,23 @@ const CLEAR_CHAT_HISTORY_INTENT = /(?:清空|清除|删除)(?:聊天|对话)记
 const NEW_CONVERSATION_INTENT = /开新对话|新对话|清除记忆|清空(?:聊天|对话)(?!记录)|clear memory|clear chat(?! history)|new conversation|start over|forget (?:everything|this|what we|what i)/giu;
 const LIST_PROJECTS_INTENT = /找项目|查项目|看看项目|有哪些项目|项目列表|list projects|find project|registered project|which project/iu;
 const LIST_SESSIONS_INTENT = /现有对话|现有会话|查会话|找会话|会话列表|有哪些会话|list sessions|find (?:a )?session|existing (?:conversation|session|chat)|active (?:conversation|session)/iu;
+// Self settings: missed-fence retry hints only (not authorization).
+const GET_ASSISTANT_SETTINGS_INTENT = /(?:查看|看看|读|显示)(?:一下)?(?:[^。\n]{0,40}?)(?:助手设定|默认提示词|系统提示词|人设)|(?:助手设定|默认提示词|系统提示词|人设)(?:是什么|怎么样)|(?:get|show|read|view) (?:[\w\u4e00-\u9fff]{0,40} )?(?:assistant settings|default prompt|system prompt)/iu;
+const UPDATE_DEFAULT_PROMPT_INTENT = /(?:改|修改|设置|更新|清空)(?:一下)?(?:[^。\n]{0,40}?)(?:默认提示词|系统提示词|人设)|(?:默认提示词|系统提示词|人设)(?:改成|设为|设置为|更新为|清空)|把(?:[^。\n]{0,40}?)(?:默认提示词|系统提示词|人设)(?:改|设|更新|清空)|(?:update|change|set|clear) (?:[\w\u4e00-\u9fff]{0,40} )?(?:default prompt|system prompt)/iu;
 const INTENT_NEGATION = /不要|别|不用|不开|don't|do\s+not/iu;
 const newConversationParameters = typeboxObject({});
 const clearChatHistoryParameters = typeboxObject({});
+const getAssistantSettingsParameters = typeboxObject({
+  to: typeboxOptional(typeboxString('Target assistant display name. Omit to read this contact. Same as message_assistant `to`.')),
+  name: typeboxOptional(typeboxString('Target assistant display name if `to` is omitted.')),
+  toAssistantID: typeboxOptional(typeboxString('Target assistant id when the display name is ambiguous.')),
+});
+const updateDefaultPromptParameters = typeboxObject({
+  prompt: typeboxString('New default prompt / system persona. Empty string clears it. Persists to Assistant settings; applies on later turns of that contact.'),
+  to: typeboxOptional(typeboxString('Target assistant display name. Omit to update this contact. Example: OpenCode 配置助手.')),
+  name: typeboxOptional(typeboxString('Target assistant display name if `to` is omitted.')),
+  toAssistantID: typeboxOptional(typeboxString('Target assistant id when the display name is ambiguous.')),
+});
 
 const assignParameters = typeboxObject({
   prompt: typeboxString('Coding prompt to kick into the worker OpenCode session. Current-turn user attachments (images/files) are forwarded by the server automatically — do not base64-encode or invent local paths.'),
@@ -402,6 +421,12 @@ export function detectRequestedContactTools(userText, allowedNames = []) {
   if (allowed.has(LIST_SESSIONS_TOOL_NAME) && LIST_SESSIONS_INTENT.test(text)) {
     requested.push(LIST_SESSIONS_TOOL_NAME);
   }
+  // Prefer write intent when both settings phrases match (e.g. 把默认提示词改成 X).
+  if (allowed.has(UPDATE_DEFAULT_PROMPT_TOOL_NAME) && UPDATE_DEFAULT_PROMPT_INTENT.test(text)) {
+    requested.push(UPDATE_DEFAULT_PROMPT_TOOL_NAME);
+  } else if (allowed.has(GET_ASSISTANT_SETTINGS_TOOL_NAME) && GET_ASSISTANT_SETTINGS_INTENT.test(text)) {
+    requested.push(GET_ASSISTANT_SETTINGS_TOOL_NAME);
+  }
   if (allowed.has(CREATE_ASSISTANT_TOOL_NAME) && CREATE_ASSISTANT_INTENT.test(text)) {
     requested.push(CREATE_ASSISTANT_TOOL_NAME);
   }
@@ -485,6 +510,36 @@ export function resolvePeerAssistant(params = {}, assistants = [], currentAssist
   return matches[0];
 }
 
+/**
+ * Resolve a live assistant for settings read/write.
+ * No to/name/toAssistantID → null (caller uses this contact).
+ * Name/id may target self or another live assistant.
+ */
+export function resolveAssistantTarget(params = {}, assistants = []) {
+  const toAssistantID = trim(params.toAssistantID, 256);
+  const toName = trim(params.to, 256) || trim(params.name, 256);
+  const list = Array.isArray(assistants) ? assistants : [];
+  if (!toAssistantID && !toName) return null;
+  if (toAssistantID) {
+    const match = list.find((item) => assistantIdentity(item) === toAssistantID);
+    if (!match) {
+      throw new AssignError('not_found', 'No assistant with that id is available.');
+    }
+    return match;
+  }
+  const matches = list.filter((item) => {
+    const name = typeof item?.name === 'string' ? item.name.trim() : '';
+    return name === toName || name.toLowerCase() === toName.toLowerCase();
+  });
+  if (matches.length === 0) {
+    throw new AssignError('not_found', `No assistant named ${toName} is available.`);
+  }
+  if (matches.length > 1) {
+    throw new AssignError('validation_error', `Several assistants are named ${toName}; use toAssistantID.`);
+  }
+  return matches[0];
+}
+
 /**
  * Serialize a tool parameter schema for the contact system prompt.
  * Accepts plain JSON-schema-like objects and TypeBox schemas (JSON.stringify).
@@ -525,6 +580,8 @@ export function formatContactToolsPrompt(tools) {
     'When they explicitly want to delete the stored chat (清空聊天记录 / clear chat history / delete chat history), call clear_chat_history. Do not use clear_chat_history for ordinary 开新对话 / new conversation wording.',
     'When they want to find a registered project (找项目 / list projects / "openchamber yee"), call list_projects or use the Registered projects block already in context.',
     'When they want existing conversations in a project (现有对话 / list sessions), call list_sessions.',
+    'When they want to view assistant settings / default prompt / system persona (查看助手设定 / 默认提示词 / 系统提示词 / 人设), call get_assistant_settings. Omit `to` for this contact; pass to="OpenCode 配置助手" (or toAssistantID) to read another live assistant.',
+    'When they want to change a default prompt / system persona (改默认提示词 / 设置人设 / 改某助手的默认提示词), call update_default_prompt. That writes Assistant settings and persists — it is not a one-shot message and not new_conversation. Omit `to` for this contact; pass to/name/toAssistantID to update another live assistant without changing this one.',
     'When they want another assistant (建助理 / create an assistant), call create_assistant.',
     'When they want a separate Chat coding session (建会话 / open a session / 开个新会话), call assign_session after matching the project. File and shell work in this assistant\'s working directory uses read, write, edit, and bash — not assign_session.',
     'When they want a scheduled task (排定时任务 / schedule daily ping), call schedule_task.',
@@ -543,6 +600,12 @@ export function formatContactToolsPrompt(tools) {
     `{"name":"${LIST_SESSIONS_TOOL_NAME}","arguments":{"projectPath":"/path/to/repo","query":"login"}}`,
     '```',
     `\`\`\`${CONTACT_TOOL_FENCE}`,
+    `{"name":"${GET_ASSISTANT_SETTINGS_TOOL_NAME}","arguments":{}}`,
+    '```',
+    `\`\`\`${CONTACT_TOOL_FENCE}`,
+    `{"name":"${UPDATE_DEFAULT_PROMPT_TOOL_NAME}","arguments":{"to":"OpenCode 配置助手","prompt":"Be terse and reply in Chinese."}}`,
+    '```',
+    `\`\`\`${CONTACT_TOOL_FENCE}`,
     `{"name":"${CREATE_ASSISTANT_TOOL_NAME}","arguments":{"name":"FlowQA","model":"opencode-go/deepseek-v4-flash"}}`,
     '```',
     `\`\`\`${CONTACT_TOOL_FENCE}`,
@@ -554,12 +617,14 @@ export function formatContactToolsPrompt(tools) {
     `\`\`\`${CONTACT_TOOL_FENCE}`,
     `{"name":"${ASSIGN_SESSION_TOOL_NAME}","arguments":{"prompt":"...","projectPath":"...","model":"provider/model-id"}}`,
     '```',
-    'If the user asked for more than one of these, do them in that order across turns: new_conversation or clear_chat_history, then list_projects, then list_sessions, then create_assistant, then schedule_task, then message_assistant, then assign_session.',
+    'If the user asked for more than one of these, do them in that order across turns: new_conversation or clear_chat_history, then list_projects, then list_sessions, then get_assistant_settings or update_default_prompt, then create_assistant, then schedule_task, then message_assistant, then assign_session.',
     'Prerequisite lookups (list_projects / list_sessions) may run before assign_session in the same turn. After a successful assign_session the turn ends — never call assign_session again, and do not keep looping tools.',
     'new_conversation advances this contact\'s LLM context boundary only. Stored messages and watches remain. It never calls session/new or createNew.',
     'clear_chat_history deletes this contact\'s stored messages, parts, and watches. Use only for explicit wipe intent.',
     'You already receive the registered project catalog each turn. Prefer matching label/path yourself; list_projects refreshes or filters. Never claim you cannot see projects; never ask for a raw path when a name matches.',
     'list_sessions searches the OpenChamber session index for existing chats in a project. A failure is not an empty list — surface the error.',
+    'get_assistant_settings reads live Assistant settings from storage (id, name, defaultPrompt, provider/model, agent, variant, mode, workspacePath, enabled). Omit to/name/toAssistantID for this contact; pass them to read another live assistant. Use it before claiming what a default prompt is.',
+    'update_default_prompt persists defaultPrompt on the target assistant row (empty string clears). Omit to/name/toAssistantID for this contact; pass them to update another live assistant — that does not overwrite this contact. It is Assistant settings — not a one-shot user message and not new_conversation. Takes effect on later turns of that contact only. After success, confirm in one short bubble.',
     'assign_session opens a real OpenChamber/OpenCode session on a registered project (or reuses sessionID). You are not the worker. Optional providerID/modelID/model select the worker only from the connected catalog and never change this contact. Current-turn user attachments are server-forwarded — do not embed base64 or local paths. One successful assign ends this turn.',
     'create_assistant reuses already-connected OpenCode providers (providerID/modelID). Mode is continuous.',
     'schedule_task writes the same payload as PUT /api/projects/:id/scheduled-tasks onto a registered project.',
@@ -646,6 +711,63 @@ const toolFailure = (error, fallbackCode, fallbackMessage) => {
   };
 };
 
+const settingsTargetParams = (params) => Boolean(
+  trim(params?.toAssistantID, 256) || trim(params?.to, 256) || trim(params?.name, 256),
+);
+
+const resolveSettingsAssistantTarget = async (params, listAssistants) => {
+  const toAssistantID = trim(params?.toAssistantID, 256);
+  const toName = trim(params?.to, 256) || trim(params?.name, 256);
+  if (!toAssistantID && !toName) return null;
+  if (typeof listAssistants !== 'function') {
+    throw new AssignError('upstream_error', 'Listing assistants is unavailable.');
+  }
+  const listed = await listAssistants();
+  return resolveAssistantTarget(params, listed);
+};
+
+/** Subset of AssistantDTO fields exposed to get/update settings tools. */
+export function pickAssistantSettings(dto) {
+  if (!dto || typeof dto !== 'object') return null;
+  const id = typeof dto.id === 'string' && dto.id.trim()
+    ? dto.id.trim()
+    : (typeof dto.assistantID === 'string' && dto.assistantID.trim() ? dto.assistantID.trim() : null);
+  if (!id) return null;
+  return {
+    id,
+    name: typeof dto.name === 'string' ? dto.name : '',
+    defaultPrompt: typeof dto.defaultPrompt === 'string' ? dto.defaultPrompt : '',
+    providerID: typeof dto.providerID === 'string' ? dto.providerID : '',
+    modelID: typeof dto.modelID === 'string' ? dto.modelID : '',
+    agent: dto.agent == null ? null : String(dto.agent),
+    variant: dto.variant == null ? null : String(dto.variant),
+    mode: dto.mode === 'stateless' ? 'stateless' : 'continuous',
+    workspacePath: dto.workspacePath == null ? null : String(dto.workspacePath),
+    enabled: Boolean(dto.enabled),
+  };
+}
+
+/** Short readable settings dump for the model (empty defaultPrompt is explicit). */
+export function formatAssistantSettingsContent(settings) {
+  const row = pickAssistantSettings(settings) || settings;
+  if (!row || typeof row !== 'object') return 'Assistant settings unavailable.';
+  const prompt = typeof row.defaultPrompt === 'string' ? row.defaultPrompt : '';
+  const promptLine = prompt === '' ? 'defaultPrompt: (empty)' : `defaultPrompt:\n${prompt}`;
+  return [
+    `Assistant settings for ${row.name || row.id || 'this contact'}:`,
+    `id: ${row.id || ''}`,
+    `name: ${row.name || ''}`,
+    promptLine,
+    `providerID: ${row.providerID || ''}`,
+    `modelID: ${row.modelID || ''}`,
+    `agent: ${row.agent == null ? '(none)' : row.agent}`,
+    `variant: ${row.variant == null ? '(none)' : row.variant}`,
+    `mode: ${row.mode || 'continuous'}`,
+    `workspacePath: ${row.workspacePath == null ? '(managed)' : row.workspacePath}`,
+    `enabled: ${row.enabled ? 'true' : 'false'}`,
+  ].join('\n');
+}
+
 export function createContactTools({
   assignWork,
   createAssistant,
@@ -656,6 +778,8 @@ export function createContactTools({
   listAssistants,
   listProjects,
   listSessions,
+  readAssistantSettings,
+  updateAssistantSettings,
   currentAssistant,
   onCard,
   /** Authoritative current-turn user file parts (server-owned). Forwarded on assign by default. */
@@ -883,6 +1007,93 @@ export function createContactTools({
         }
       },
     },
+    {
+      name: GET_ASSISTANT_SETTINGS_TOOL_NAME,
+      label: 'Get assistant settings',
+      description: [
+        'Read live Assistant settings from storage (not a stale turn snapshot).',
+        'Omit to/name/toAssistantID to read this contact. Pass to="OpenCode 配置助手" or toAssistantID to read another live assistant.',
+        'Returns id, name, defaultPrompt, providerID, modelID, agent, variant, mode, workspacePath, enabled.',
+        'Use when the user asks about 助手设定 / 默认提示词 / 系统提示词 / 人设 — including another assistant\'s.',
+      ].join(' '),
+      parameters: getAssistantSettingsParameters,
+      execute: async (_toolCallId, params) => {
+        try {
+          if (typeof readAssistantSettings !== 'function') {
+            throw new AssignError('upstream_error', 'Reading assistant settings is unavailable.');
+          }
+          const target = settingsTargetParams(params)
+            ? await resolveSettingsAssistantTarget(params, listAssistants)
+            : null;
+          const raw = target
+            ? await readAssistantSettings({ assistantID: assistantIdentity(target) })
+            : await readAssistantSettings();
+          const settings = pickAssistantSettings(raw?.settings ?? raw);
+          if (!settings) {
+            throw new AssignError('upstream_error', 'Assistant settings failed to load.');
+          }
+          return {
+            content: [{ type: 'text', text: formatAssistantSettingsContent(settings) }],
+            details: { settings },
+            terminate: false,
+          };
+        } catch (error) {
+          return toolFailure(error, 'get_assistant_settings_failed', 'Could not read assistant settings.');
+        }
+      },
+    },
+    {
+      name: UPDATE_DEFAULT_PROMPT_TOOL_NAME,
+      label: 'Update default prompt',
+      description: [
+        'Persist defaultPrompt (system persona) to Assistant settings for this contact or another live assistant.',
+        'Required prompt string; empty string clears it. Not a one-shot message — takes effect on later turns of that contact.',
+        'Omit to/name/toAssistantID to update this contact. Pass to="OpenCode 配置助手" to update that assistant without changing this one.',
+        'Use when the user asks to 改默认提示词 / 设置人设 / 改某助手的默认提示词.',
+      ].join(' '),
+      parameters: updateDefaultPromptParameters,
+      execute: async (_toolCallId, params) => {
+        try {
+          if (typeof updateAssistantSettings !== 'function') {
+            throw new AssignError('upstream_error', 'Updating assistant settings is unavailable.');
+          }
+          if (typeof params?.prompt !== 'string') {
+            throw new AssignError('validation_error', 'update_default_prompt requires prompt (string; empty clears).');
+          }
+          const target = settingsTargetParams(params)
+            ? await resolveSettingsAssistantTarget(params, listAssistants)
+            : null;
+          const assistantID = target ? assistantIdentity(target) : null;
+          // Allow empty string (clear). Do not trim here — service normalizes like updateAssistant.
+          const result = await updateAssistantSettings({
+            defaultPrompt: params.prompt,
+            ...(assistantID ? { assistantID } : {}),
+          });
+          const defaultPrompt = typeof result?.defaultPrompt === 'string'
+            ? result.defaultPrompt
+            : params.prompt;
+          const details = {
+            defaultPrompt,
+            ...(assistantID ? { assistantID, name: target?.name || null } : {}),
+          };
+          if (result?.unchanged === true || result?.updated === false) {
+            return {
+              content: [{ type: 'text', text: UPDATE_DEFAULT_PROMPT_UNCHANGED_BUBBLE }],
+              details: { ...details, updated: false, unchanged: true },
+              terminate: false,
+            };
+          }
+          return {
+            content: [{ type: 'text', text: UPDATE_DEFAULT_PROMPT_CONFIRM_BUBBLE }],
+            details: { ...details, updated: true },
+            // Same as create_assistant / schedule_task: keep the turn open for a short confirm.
+            terminate: false,
+          };
+        } catch (error) {
+          return toolFailure(error, 'update_default_prompt_failed', 'Could not update default prompt.');
+        }
+      },
+    },
     {
       name: CREATE_ASSISTANT_TOOL_NAME,
       label: 'Create assistant',

package/server/lib/assistants/contact-tools.test.js

@@ -6,12 +6,16 @@ import {
   CLEAR_CHAT_HISTORY_CONFIRM_BUBBLE,
   CLEAR_CHAT_HISTORY_TOOL_NAME,
   CREATE_ASSISTANT_TOOL_NAME,
+  GET_ASSISTANT_SETTINGS_TOOL_NAME,
   LIST_PROJECTS_TOOL_NAME,
   LIST_SESSIONS_TOOL_NAME,
   MESSAGE_ASSISTANT_TOOL_NAME,
   NEW_CONVERSATION_CONFIRM_BUBBLE,
   NEW_CONVERSATION_TOOL_NAME,
   SCHEDULE_TASK_TOOL_NAME,
+  UPDATE_DEFAULT_PROMPT_CONFIRM_BUBBLE,
+  UPDATE_DEFAULT_PROMPT_TOOL_NAME,
+  UPDATE_DEFAULT_PROMPT_UNCHANGED_BUBBLE,
   confirmBubbleAfterContactReset,
   contactTurnHasSuccessfulReset,
   createContactTools,
@@ -89,6 +93,8 @@ describe('contact tool protocol', () => {
       CLEAR_CHAT_HISTORY_TOOL_NAME,
       LIST_PROJECTS_TOOL_NAME,
       LIST_SESSIONS_TOOL_NAME,
+      GET_ASSISTANT_SETTINGS_TOOL_NAME,
+      UPDATE_DEFAULT_PROMPT_TOOL_NAME,
       CREATE_ASSISTANT_TOOL_NAME,
       SCHEDULE_TASK_TOOL_NAME,
       MESSAGE_ASSISTANT_TOOL_NAME,
@@ -118,6 +124,11 @@ describe('contact tool protocol', () => {
     expect(detectRequestedContactTools('找项目 openchamber yee', tools)).toEqual([LIST_PROJECTS_TOOL_NAME]);
     expect(detectRequestedContactTools('看看现有对话', tools)).toEqual([LIST_SESSIONS_TOOL_NAME]);
     expect(detectRequestedContactTools('list sessions in that project', tools)).toEqual([LIST_SESSIONS_TOOL_NAME]);
+    expect(detectRequestedContactTools('看看我的默认提示词', tools)).toEqual([GET_ASSISTANT_SETTINGS_TOOL_NAME]);
+    expect(detectRequestedContactTools('看看 OpenCode 配置助手的默认提示词', tools)).toEqual([GET_ASSISTANT_SETTINGS_TOOL_NAME]);
+    expect(detectRequestedContactTools('把默认提示词改成简洁中文', tools)).toEqual([UPDATE_DEFAULT_PROMPT_TOOL_NAME]);
+    expect(detectRequestedContactTools('改 OpenCode 配置助手的默认提示词', tools)).toEqual([UPDATE_DEFAULT_PROMPT_TOOL_NAME]);
+    expect(detectRequestedContactTools('update default prompt to be terse', tools)).toEqual([UPDATE_DEFAULT_PROMPT_TOOL_NAME]);
     expect(detectRequestedContactTools('开新对话', tools)).not.toContain(ASSIGN_SESSION_TOOL_NAME);
     expect(detectRequestedContactTools('开新对话', tools)).not.toContain(CLEAR_CHAT_HISTORY_TOOL_NAME);
     expect(detectRequestedContactTools('不要开编码 session', tools)).toEqual([]);
@@ -146,6 +157,8 @@ describe('contact tool protocol', () => {
     expect(prompt).toContain('clear_chat_history');
     expect(prompt).toContain('list_projects');
     expect(prompt).toContain('list_sessions');
+    expect(prompt).toContain('get_assistant_settings');
+    expect(prompt).toContain('update_default_prompt');
     expect(prompt).toContain('create_assistant');
     expect(prompt).toContain('schedule_task');
     expect(prompt).toContain('message_assistant');
@@ -155,11 +168,15 @@ describe('contact tool protocol', () => {
     expect(prompt).toContain('清空聊天记录');
     expect(prompt).toContain('找项目');
     expect(prompt).toContain('现有对话');
+    expect(prompt).toContain('默认提示词');
+    expect(prompt).toContain('助手设定');
     expect(prompt).toContain('建助理');
     expect(prompt).toContain('排定时任务');
     expect(prompt).toContain('说一声');
     expect(prompt).toContain('session/new');
     expect(prompt).toContain('LLM memory only');
+    expect(prompt).toContain('Persists');
+    expect(prompt).toContain('later turns');
     expect(prompt).not.toContain('/card');
     expect(prompt).not.toContain('/dm');
     expect(prompt).toContain('A reply without the tool call does nothing');
@@ -273,6 +290,8 @@ describe('createContactTools', () => {
       CLEAR_CHAT_HISTORY_TOOL_NAME,
       LIST_PROJECTS_TOOL_NAME,
       LIST_SESSIONS_TOOL_NAME,
+      GET_ASSISTANT_SETTINGS_TOOL_NAME,
+      UPDATE_DEFAULT_PROMPT_TOOL_NAME,
       CREATE_ASSISTANT_TOOL_NAME,
       SCHEDULE_TASK_TOOL_NAME,
       MESSAGE_ASSISTANT_TOOL_NAME,
@@ -535,6 +554,138 @@ describe('createContactTools', () => {
     expect(failed.content[0].text).toContain('Session index is unavailable');
     expect(failed.details.sessions).toBeUndefined();
   });
+
+  it('reads assistant settings via live callback and formats empty defaultPrompt', async () => {
+    const readAssistantSettings = vi.fn(async () => ({
+      id: 'asst_host',
+      name: 'DeepSeekQA',
+      defaultPrompt: '',
+      providerID: 'p',
+      modelID: 'm',
+      agent: null,
+      variant: null,
+      mode: 'continuous',
+      workspacePath: null,
+      enabled: true,
+    }));
+    const tools = createContactTools({ readAssistantSettings });
+    const result = await tools.find((tool) => tool.name === GET_ASSISTANT_SETTINGS_TOOL_NAME).execute('call_get', {});
+    expect(readAssistantSettings).toHaveBeenCalledTimes(1);
+    expect(result.details.settings).toMatchObject({
+      id: 'asst_host',
+      name: 'DeepSeekQA',
+      defaultPrompt: '',
+      providerID: 'p',
+      modelID: 'm',
+      enabled: true,
+    });
+    expect(result.content[0].text).toContain('defaultPrompt: (empty)');
+    expect(result.terminate).toBe(false);
+  });
+
+  it('updates default prompt, no-ops when unchanged, and reports callback failures', async () => {
+    const updateAssistantSettings = vi.fn(async ({ defaultPrompt }) => {
+      if (defaultPrompt === 'same') {
+        return { updated: false, unchanged: true, defaultPrompt: 'same' };
+      }
+      return { updated: true, defaultPrompt };
+    });
+    const tools = createContactTools({ updateAssistantSettings });
+    const update = tools.find((tool) => tool.name === UPDATE_DEFAULT_PROMPT_TOOL_NAME);
+
+    const saved = await update.execute('call_set', { prompt: 'Be terse.' });
+    expect(updateAssistantSettings).toHaveBeenCalledWith({ defaultPrompt: 'Be terse.' });
+    expect(saved.details).toEqual({ updated: true, defaultPrompt: 'Be terse.' });
+    expect(saved.content[0].text).toBe(UPDATE_DEFAULT_PROMPT_CONFIRM_BUBBLE);
+    expect(saved.terminate).toBe(false);
+
+    const cleared = await update.execute('call_clear', { prompt: '' });
+    expect(updateAssistantSettings).toHaveBeenCalledWith({ defaultPrompt: '' });
+    expect(cleared.details).toEqual({ updated: true, defaultPrompt: '' });
+    expect(cleared.terminate).toBe(false);
+
+    const unchanged = await update.execute('call_same', { prompt: 'same' });
+    expect(unchanged.details).toEqual({ updated: false, unchanged: true, defaultPrompt: 'same' });
+    expect(unchanged.content[0].text).toBe(UPDATE_DEFAULT_PROMPT_UNCHANGED_BUBBLE);
+    expect(unchanged.terminate).toBe(false);
+
+    const missing = await update.execute('call_missing', {});
+    expect(missing.details.error).toBe('validation_error');
+    expect(missing.terminate).toBe(true);
+
+    updateAssistantSettings.mockRejectedValueOnce(Object.assign(new Error('revision_conflict'), { code: 'revision_conflict' }));
+    const conflicted = await update.execute('call_conflict', { prompt: 'X' });
+    expect(conflicted.details.error).toBe('revision_conflict');
+    expect(conflicted.terminate).toBe(true);
+
+    updateAssistantSettings.mockRejectedValueOnce(Object.assign(new Error('sqlite busy'), { code: 'upstream_error' }));
+    const storageFailed = await update.execute('call_storage', { prompt: 'Y' });
+    expect(storageFailed.details.error).toBe('upstream_error');
+    expect(storageFailed.terminate).toBe(true);
+  });
+
+  it('reads and updates another assistant defaultPrompt by name without changing self', async () => {
+    const peer = {
+      id: 'asst_peer',
+      name: 'OpenCode 配置助手',
+      defaultPrompt: 'old peer',
+      providerID: 'p',
+      modelID: 'm',
+      agent: null,
+      variant: null,
+      mode: 'continuous',
+      workspacePath: null,
+      enabled: true,
+    };
+    const host = {
+      id: 'asst_host',
+      name: 'Host',
+      defaultPrompt: 'host prompt',
+      providerID: 'p',
+      modelID: 'm',
+      agent: null,
+      variant: null,
+      mode: 'continuous',
+      workspacePath: null,
+      enabled: true,
+    };
+    const readAssistantSettings = vi.fn(async (input) => (
+      input?.assistantID === 'asst_peer' ? peer : host
+    ));
+    const updateAssistantSettings = vi.fn(async (patch) => ({
+      updated: true,
+      defaultPrompt: patch.defaultPrompt,
+    }));
+    const listAssistants = vi.fn(async () => [host, peer]);
+    const tools = createContactTools({
+      readAssistantSettings,
+      updateAssistantSettings,
+      listAssistants,
+      currentAssistant: host,
+    });
+
+    const got = await tools.find((tool) => tool.name === GET_ASSISTANT_SETTINGS_TOOL_NAME)
+      .execute('call_peer_get', { to: 'OpenCode 配置助手' });
+    expect(readAssistantSettings).toHaveBeenCalledWith({ assistantID: 'asst_peer' });
+    expect(got.details.settings).toMatchObject({ id: 'asst_peer', name: 'OpenCode 配置助手', defaultPrompt: 'old peer' });
+
+    const saved = await tools.find((tool) => tool.name === UPDATE_DEFAULT_PROMPT_TOOL_NAME)
+      .execute('call_peer_set', { to: 'OpenCode 配置助手', prompt: 'new peer persona' });
+    expect(updateAssistantSettings).toHaveBeenCalledWith({
+      defaultPrompt: 'new peer persona',
+      assistantID: 'asst_peer',
+    });
+    expect(saved.details).toMatchObject({
+      updated: true,
+      defaultPrompt: 'new peer persona',
+      assistantID: 'asst_peer',
+      name: 'OpenCode 配置助手',
+    });
+
+    const missing = await tools.find((tool) => tool.name === UPDATE_DEFAULT_PROMPT_TOOL_NAME)
+      .execute('call_missing_peer', { to: '没有这个助手', prompt: 'x' });
+    expect(missing.details.error).toBe('not_found');
+  });
 });
 
 describe('confirmBubbleAfterContactReset', () => {

package/server/lib/assistants/harness.js

@@ -1,21 +1,29 @@
 import { Agent } from '@earendil-works/pi-agent-core';
 import { isContactSpokenPreamble, splitContactBubbles } from './bubbles.js';
 import {
+  ASSIGN_SESSION_TOOL_NAME,
   CLEAR_CHAT_HISTORY_CONFIRM_BUBBLE,
+  CLEAR_CHAT_HISTORY_TOOL_NAME,
   confirmBubbleAfterContactReset,
   contactTurnClearedChatHistory,
   contactTurnHasSuccessfulReset,
   contactTurnHasToolResult,
+  CREATE_ASSISTANT_TOOL_NAME,
   detectRequestedContactTools,
   extractContactCardsFromMessages,
   formatContactToolsPrompt,
   formatConnectedModelsPrompt,
   formatRegisteredProjectsPrompt,
+  GET_ASSISTANT_SETTINGS_TOOL_NAME,
+  MESSAGE_ASSISTANT_TOOL_NAME,
   MISSED_FENCE_RETRY_USER_TEXT,
   MISSED_TOOL_FAILURE_BUBBLE,
   NEW_CONVERSATION_CONFIRM_BUBBLE,
+  NEW_CONVERSATION_TOOL_NAME,
   parseContactToolCalls,
+  SCHEDULE_TASK_TOOL_NAME,
   stripContactToolFences,
+  UPDATE_DEFAULT_PROMPT_TOOL_NAME,
 } from './contact-tools.js';
 import {
   createPiCodingRuntime,
@@ -78,7 +86,7 @@ export const CONTACT_SYSTEM_PROMPT = [
   'Never write chain-of-thought, plans, tool names, or English narration of what you will do. The user never sees thinking.',
   'Do not expose tool traces, Activity, or editor actions.',
   'You have bash, read, write, and edit in the working directory. Use them for pwd, files, and shell. Never say you have no terminal or cannot read files. Ignore any temporary generator workspace in the environment.',
-  'Understand natural language in any language, including Chinese: 开新对话 / 清除记忆 means new_conversation (LLM memory only, chat history stays), 清空聊天记录 means clear_chat_history (delete transcript), 找项目 means list_projects, 现有对话 means list_sessions, 建助理 means create_assistant, 建会话 / 开个新会话 means assign_session, 排定时任务 means schedule_task, 给 X 说一声 means message_assistant, 发卡片 means emit a card via those tools — never ask the user to type /card or /dm.',
+  'Understand natural language in any language, including Chinese: 开新对话 / 清除记忆 means new_conversation (LLM memory only, chat history stays), 清空聊天记录 means clear_chat_history (delete transcript), 找项目 means list_projects, 现有对话 means list_sessions, 查看助手设定 / 默认提示词 means get_assistant_settings (pass to="Name" for another assistant), 改默认提示词 / 设置人设 / 改某助手的默认提示词 means update_default_prompt (persists that assistant\'s settings, later turns only; pass to="OpenCode 配置助手" to edit another contact without changing this one), 建助理 means create_assistant, 建会话 / 开个新会话 means assign_session, 排定时任务 means schedule_task, 给 X 说一声 means message_assistant, 发卡片 means emit a card via those tools — never ask the user to type /card or /dm.',
   'You receive the registered project catalog every turn. You CAN see those projects. Look them up yourself (fuzzy match label/name/path). Never say you cannot see the registered project list. Never ask for a raw filesystem path when a name matches. If the catalog is empty, tell the user to add a project in Settings.',
   'File and shell work in this working directory uses read, write, edit, and bash. To open a separate Chat coding session: match the project, optionally list_sessions for existing chats, then assign_session with projectPath or sessionID. Optional worker model via providerID/modelID/model from the connected catalog — that does not change this contact. Current-turn user attachments are server-forwarded on assign. One successful assign_session ends the turn — do not call it again.',
   'A reply without the tool call does nothing. Never say 已创建, created, scheduled, or opened unless the tool already returned success.',
@@ -417,6 +425,22 @@ const extractToolResultText = (messages) => {
   return '';
 };
 
+/** Tool confirms that are themselves the user-facing bubble (no card). */
+const TOOL_TEXT_BUBBLE_TOOLS = new Set([
+  NEW_CONVERSATION_TOOL_NAME,
+  CLEAR_CHAT_HISTORY_TOOL_NAME,
+  UPDATE_DEFAULT_PROMPT_TOOL_NAME,
+  GET_ASSISTANT_SETTINGS_TOOL_NAME,
+]);
+
+/** Card / side-effect tools: never paint English toolText into the transcript. */
+const CARD_SIDE_EFFECT_TOOLS = new Set([
+  ASSIGN_SESSION_TOOL_NAME,
+  CREATE_ASSISTANT_TOOL_NAME,
+  SCHEDULE_TASK_TOOL_NAME,
+  MESSAGE_ASSISTANT_TOOL_NAME,
+]);
+
 const extractContactTurnOutcome = (messages, retried) => {
   const list = Array.isArray(messages) ? messages : [];
   let start = 0;
@@ -440,9 +464,22 @@ const extractContactTurnOutcome = (messages, retried) => {
   const coding = isPiCodingToolName(lastToolName);
   const toolText = extractToolResultText(slice);
   const assistant = extractAssistantText(slice);
-  const confirm = hasTool && !coding
-    ? (toolText || assistant)
-    : (assistant || (coding ? toolText : ''));
+  let confirm = '';
+  if (hasTool && !coding) {
+    if (TOOL_TEXT_BUBBLE_TOOLS.has(lastToolName)) {
+      // Confirm-only tools: toolText is the user bubble.
+      confirm = toolText || assistant;
+    } else if (CARD_SIDE_EFFECT_TOOLS.has(lastToolName)) {
+      // Spoken preamble only. Card is enough when present; never English toolText.
+      // Post-tool assistant text (terminate:false tools) may still surface when no spoken.
+      confirm = spoken ? '' : (assistant || '');
+    } else {
+      // list_projects / list_sessions and other app tools keep toolText.
+      confirm = toolText || assistant;
+    }
+  } else {
+    confirm = assistant || (coding ? toolText : '');
+  }
   const parts = [];
   if (spoken) parts.push(spoken);
   if (confirm && confirm !== spoken) parts.push(confirm);
@@ -554,6 +591,19 @@ export async function runContactTurn({
       }
     }
     if (requested.length > 0 && !hasRequestedResult() && !contactTurnHasSuccessfulReset(agent.state.messages)) {
+      // Prefer any spoken/assistant text already in the turn over the English failure fallback.
+      const missed = extractContactTurnOutcome(agent.state.messages, retried);
+      const missedText = stripContactToolFences(missed.text).trim();
+      if (missedText) {
+        const missedBubbles = splitContactBubbles(missedText);
+        return {
+          text: missedText,
+          bubbles: missedBubbles.length > 0 ? missedBubbles : [missedText],
+          cards: [],
+          thinkingLevel: agent.state.thinkingLevel,
+          tools: [...agent.state.tools],
+        };
+      }
       return {
         text: MISSED_TOOL_FAILURE_BUBBLE,
         bubbles: [MISSED_TOOL_FAILURE_BUBBLE],
@@ -580,7 +630,8 @@ export async function runContactTurn({
         tools: [...agent.state.tools],
       };
     }
-    if (!text.trim() && outcome.cards.length === 0) {
+    // Card tools / side-effect tools may finish with cards only (no English toolText bubble).
+    if (!text.trim() && outcome.cards.length === 0 && !outcome.hasTool) {
       const error = new Error('Assistant returned no text');
       error.code = 'upstream_error';
       throw error;