minimal-agent 0.13.0 → 0.15.0

package/src/collaboration/taskFlow.js

@@ -0,0 +1,115 @@
+/**
+ * 旧项目任务板状态机（兼容层）。
+ *
+ * KakadeAI 新群协作主流程使用 CollaborationEvent，并在一次目标确认后直接运行；
+ * 本文件只负责已有 `conversation/workspace` 数据和旧插件的状态/动作兼容。
+ *
+ * ⚠️ 这张表原先内联在 `minimal-server/src/core/collaboration/projects.ts` 里
+ * （`WORK_ITEM_TRANSITIONS`），只有服务端看得到。前端要预测"按钮该不该亮"就得再抄一份，
+ * 两份必然漂移，漂移的后果是「界面能点、点了被拒」——最糟的一类交互。
+ * 所以兼容代码仍共享这一份，避免旧前后端漂移。
+ *
+ * 相对原表只加了一个状态 `proposed`（方案待审）。「待验收」沿用既有的 `review`
+ * （UI 里本来就叫"评审"），不造同义状态。
+ *
+ * 与 projects.ts 里其余守卫的分工：
+ *   这里管「状态怎么流转」和「谁有资格推动」；
+ *   projects.ts 继续管数据完整性类守卫（验收条件必填、执行者不能评审自己、
+ *   进入执行要有仓库/分支/文件范围、完成前要有已接受的交接单）。两者互补，不重叠。
+ */
+/**
+ * 合法流转表。**服务端与前端共用这一份。**
+ *
+ * `ready → in_progress` 是新旧迁移的默认直连。`submitPlan/approvePlan` 仅在旧任务
+ * 显式选择方案评审时使用，服务端不得再把它强制成所有 Agent 任务的第二次人工确认。
+ */
+export const TASK_TRANSITIONS = {
+    backlog: ['ready'],
+    ready: ['backlog', 'proposed', 'in_progress', 'blocked'],
+    proposed: ['ready', 'in_progress', 'blocked'],
+    in_progress: ['review', 'blocked'],
+    blocked: ['in_progress', 'backlog'],
+    review: ['in_progress', 'done', 'blocked'],
+    done: [],
+};
+const TRANSITION = {
+    assign: 'ready',
+    submitPlan: 'proposed',
+    approvePlan: 'in_progress',
+    rejectPlan: 'ready', // 打回回到已派单：同一张卡、同一个执行者重提方案
+    submitHandoff: 'review',
+    accept: 'done',
+    rework: 'in_progress',
+    block: 'blocked',
+    unblock: 'in_progress',
+};
+const ALLOWED_FROM = {
+    assign: ['backlog'],
+    submitPlan: ['ready'],
+    approvePlan: ['proposed'],
+    rejectPlan: ['proposed'],
+    submitHandoff: ['in_progress'],
+    accept: ['review'],
+    rework: ['review'],
+    block: ['ready', 'proposed', 'in_progress', 'review'],
+    unblock: ['blocked'],
+};
+const AGENT_ACTIONS = ['submitPlan', 'submitHandoff'];
+/** 旧任务板中由人触发的动作；新主流程只保留目标确认和最终验收。 */
+const HUMAN_ACTIONS = ['assign', 'approvePlan', 'rejectPlan', 'accept', 'rework', 'block', 'unblock'];
+/** 打回和返工不写原因，对面无从改起。 */
+const REASON_REQUIRED = ['rejectPlan', 'rework'];
+export function isAgentMember(memberId) {
+    return memberId.startsWith('agent:') || memberId.startsWith('agent-session:');
+}
+export function nextStatus(action) {
+    return TRANSITION[action];
+}
+/** 纯粹的流转合法性（不看是谁）。projects.ts 用它替换原先内联的表。 */
+export function canTransition(from, to) {
+    return from === to || TASK_TRANSITIONS[from].includes(to);
+}
+export function canAct(task, action, actor, input = {}) {
+    if (task.status === 'done')
+        return { allowed: false, reason: '任务已完成，不能再改动' };
+    if (!ALLOWED_FROM[action].includes(task.status)) {
+        if (action === 'approvePlan')
+            return { allowed: false, reason: '还没有待审的方案' };
+        if (action === 'accept' || action === 'rework')
+            return { allowed: false, reason: '任务还没有进入待验收' };
+        if (action === 'submitHandoff')
+            return { allowed: false, reason: '进入执行后才能交付' };
+        return { allowed: false, reason: '当前状态不允许该操作' };
+    }
+    if (AGENT_ACTIONS.includes(action)) {
+        if (actor.kind !== 'agent')
+            return { allowed: false, reason: '该动作由被指派的 agent 提交' };
+        if (actor.memberId !== task.ownerMemberId)
+            return { allowed: false, reason: '只有被指派的 agent 能提交' };
+        return { allowed: true };
+    }
+    if (HUMAN_ACTIONS.includes(action)) {
+        // agent 永远不能替人拍板，否则"人只对方案/结果负责"就是空话。
+        if (actor.kind !== 'human')
+            return { allowed: false, reason: '只有人类成员能拍板' };
+        const isVerdict = action === 'approvePlan' || action === 'rejectPlan' || action === 'accept' || action === 'rework';
+        if (isVerdict && task.reviewerIds.length > 0 && !task.reviewerIds.includes(actor.memberId)) {
+            return { allowed: false, reason: '这张卡指定了评审人，只有评审人能拍板' };
+        }
+        if (REASON_REQUIRED.includes(action) && !input.reason?.trim()) {
+            return { allowed: false, reason: '请填写原因，对方才知道要改什么' };
+        }
+        return { allowed: true };
+    }
+    return { allowed: false, reason: '未知操作' };
+}
+/** @deprecated 新主流程在一次目标确认后直接执行；仅保留导出避免旧插件崩溃。 */
+export function requiresPlanApproval(_ownerMemberId, _from, _to) {
+    return false;
+}
+/** 给 UI：当前这个人/agent 在这张卡上能做哪些动作（决定按钮亮不亮）。 */
+export function taskActionsFor(task, actor) {
+    const all = Object.keys(TRANSITION);
+    // 带理由的动作按"已填理由"预判：按钮要先亮出来，用户才有地方填理由。
+    return all.filter((action) => canAct(task, action, actor, { reason: '预检' }).allowed);
+}

package/src/collaboration/taskFlow.ts

@@ -0,0 +1,169 @@
+/**
+ * 旧项目任务板状态机（兼容层）。
+ *
+ * KakadeAI 新群协作主流程使用 CollaborationEvent，并在一次目标确认后直接运行；
+ * 本文件只负责已有 `conversation/workspace` 数据和旧插件的状态/动作兼容。
+ *
+ * ⚠️ 这张表原先内联在 `minimal-server/src/core/collaboration/projects.ts` 里
+ * （`WORK_ITEM_TRANSITIONS`），只有服务端看得到。前端要预测"按钮该不该亮"就得再抄一份，
+ * 两份必然漂移，漂移的后果是「界面能点、点了被拒」——最糟的一类交互。
+ * 所以兼容代码仍共享这一份，避免旧前后端漂移。
+ *
+ * 相对原表只加了一个状态 `proposed`（方案待审）。「待验收」沿用既有的 `review`
+ * （UI 里本来就叫"评审"），不造同义状态。
+ *
+ * 与 projects.ts 里其余守卫的分工：
+ *   这里管「状态怎么流转」和「谁有资格推动」；
+ *   projects.ts 继续管数据完整性类守卫（验收条件必填、执行者不能评审自己、
+ *   进入执行要有仓库/分支/文件范围、完成前要有已接受的交接单）。两者互补，不重叠。
+ */
+
+/** 任务卡状态。与 protocol.ts 的 `WorkItemStatus` 同一套值。 */
+export type TaskStatus =
+  | 'backlog'      // 待规划：还没指派
+  | 'ready'        // 已派单：等被指派者提方案（执行者是人时可直接开工）
+  | 'proposed'     // 方案待审：**人拍板点 #2**
+  | 'in_progress'  // 执行中
+  | 'review'       // 待验收：**人拍板点 #3**
+  | 'blocked'      // 阻塞
+  | 'done';        // 完成（终态）
+
+/**
+ * 合法流转表。**服务端与前端共用这一份。**
+ *
+ * `ready → in_progress` 是新旧迁移的默认直连。`submitPlan/approvePlan` 仅在旧任务
+ * 显式选择方案评审时使用，服务端不得再把它强制成所有 Agent 任务的第二次人工确认。
+ */
+export const TASK_TRANSITIONS: Record<TaskStatus, TaskStatus[]> = {
+  backlog: ['ready'],
+  ready: ['backlog', 'proposed', 'in_progress', 'blocked'],
+  proposed: ['ready', 'in_progress', 'blocked'],
+  in_progress: ['review', 'blocked'],
+  blocked: ['in_progress', 'backlog'],
+  review: ['in_progress', 'done', 'blocked'],
+  done: [],
+};
+
+export type TaskAction =
+  | 'assign'         // 人：派单
+  | 'submitPlan'     // agent：提方案
+  | 'approvePlan'    // 兼容旧任务：显式方案评审（新主流程不要求）
+  | 'rejectPlan'     // 人：打回方案（须带理由）
+  | 'submitHandoff'  // agent：交交接单 + 自检
+  | 'accept'         // 人：验收通过
+  | 'rework'         // 人：返工（须带理由）
+  | 'block'
+  | 'unblock';
+
+export interface TaskActor {
+  /** 群成员 id：'user:<id>' 或 'agent-session:<id>'。 */
+  memberId: string;
+  kind: 'human' | 'agent';
+}
+
+/** 判定所需的任务卡最小切片（不依赖完整 WorkItem，便于服务端与 UI 共用）。 */
+export interface TaskFlowState {
+  status: TaskStatus;
+  /** 被指派干活的成员。 */
+  ownerMemberId: string;
+  /** 指定评审人；为空表示任意人类成员都可拍板（小团队默认不设卡点）。 */
+  reviewerIds: string[];
+  createdBy: string;
+}
+
+export interface TaskDecision {
+  allowed: boolean;
+  reason?: string;
+}
+
+const TRANSITION: Record<TaskAction, TaskStatus> = {
+  assign: 'ready',
+  submitPlan: 'proposed',
+  approvePlan: 'in_progress',
+  rejectPlan: 'ready',      // 打回回到已派单：同一张卡、同一个执行者重提方案
+  submitHandoff: 'review',
+  accept: 'done',
+  rework: 'in_progress',
+  block: 'blocked',
+  unblock: 'in_progress',
+};
+
+const ALLOWED_FROM: Record<TaskAction, TaskStatus[]> = {
+  assign: ['backlog'],
+  submitPlan: ['ready'],
+  approvePlan: ['proposed'],
+  rejectPlan: ['proposed'],
+  submitHandoff: ['in_progress'],
+  accept: ['review'],
+  rework: ['review'],
+  block: ['ready', 'proposed', 'in_progress', 'review'],
+  unblock: ['blocked'],
+};
+
+const AGENT_ACTIONS: TaskAction[] = ['submitPlan', 'submitHandoff'];
+/** 旧任务板中由人触发的动作；新主流程只保留目标确认和最终验收。 */
+const HUMAN_ACTIONS: TaskAction[] = ['assign', 'approvePlan', 'rejectPlan', 'accept', 'rework', 'block', 'unblock'];
+/** 打回和返工不写原因，对面无从改起。 */
+const REASON_REQUIRED: TaskAction[] = ['rejectPlan', 'rework'];
+
+export function isAgentMember(memberId: string): boolean {
+  return memberId.startsWith('agent:') || memberId.startsWith('agent-session:');
+}
+
+export function nextStatus(action: TaskAction): TaskStatus {
+  return TRANSITION[action];
+}
+
+/** 纯粹的流转合法性（不看是谁）。projects.ts 用它替换原先内联的表。 */
+export function canTransition(from: TaskStatus, to: TaskStatus): boolean {
+  return from === to || TASK_TRANSITIONS[from].includes(to);
+}
+
+export function canAct(
+  task: TaskFlowState,
+  action: TaskAction,
+  actor: TaskActor,
+  input: { reason?: string } = {},
+): TaskDecision {
+  if (task.status === 'done') return { allowed: false, reason: '任务已完成，不能再改动' };
+
+  if (!ALLOWED_FROM[action].includes(task.status)) {
+    if (action === 'approvePlan') return { allowed: false, reason: '还没有待审的方案' };
+    if (action === 'accept' || action === 'rework') return { allowed: false, reason: '任务还没有进入待验收' };
+    if (action === 'submitHandoff') return { allowed: false, reason: '进入执行后才能交付' };
+    return { allowed: false, reason: '当前状态不允许该操作' };
+  }
+
+  if (AGENT_ACTIONS.includes(action)) {
+    if (actor.kind !== 'agent') return { allowed: false, reason: '该动作由被指派的 agent 提交' };
+    if (actor.memberId !== task.ownerMemberId) return { allowed: false, reason: '只有被指派的 agent 能提交' };
+    return { allowed: true };
+  }
+
+  if (HUMAN_ACTIONS.includes(action)) {
+    // agent 永远不能替人拍板，否则"人只对方案/结果负责"就是空话。
+    if (actor.kind !== 'human') return { allowed: false, reason: '只有人类成员能拍板' };
+    const isVerdict = action === 'approvePlan' || action === 'rejectPlan' || action === 'accept' || action === 'rework';
+    if (isVerdict && task.reviewerIds.length > 0 && !task.reviewerIds.includes(actor.memberId)) {
+      return { allowed: false, reason: '这张卡指定了评审人，只有评审人能拍板' };
+    }
+    if (REASON_REQUIRED.includes(action) && !input.reason?.trim()) {
+      return { allowed: false, reason: '请填写原因，对方才知道要改什么' };
+    }
+    return { allowed: true };
+  }
+
+  return { allowed: false, reason: '未知操作' };
+}
+
+/** @deprecated 新主流程在一次目标确认后直接执行；仅保留导出避免旧插件崩溃。 */
+export function requiresPlanApproval(_ownerMemberId: string, _from: TaskStatus, _to: TaskStatus): boolean {
+  return false;
+}
+
+/** 给 UI：当前这个人/agent 在这张卡上能做哪些动作（决定按钮亮不亮）。 */
+export function taskActionsFor(task: TaskFlowState, actor: TaskActor): TaskAction[] {
+  const all = Object.keys(TRANSITION) as TaskAction[];
+  // 带理由的动作按"已填理由"预判：按钮要先亮出来，用户才有地方填理由。
+  return all.filter((action) => canAct(task, action, actor, { reason: '预检' }).allowed);
+}

package/src/config/configDefaults.js

@@ -2,13 +2,26 @@
  * 浏览器与 Node/Bun 都可安全加载的纯配置默认值。
  * 不得在此模块引入 node:*、文件系统或运行时路径逻辑。
  */
+/**
+ * 子任务默认值。
+ *
+ * **为什么 explore(200) 比 general(100) 还宽**（这不是漂移，是刻意的，但此前没写下来）：
+ * explore 的工具面被 `agentTypes.ts::EXPLORE_ALLOWED_TOOLS` 硬裁成严格只读，
+ * 跑再多轮也**毁不掉任何东西**，代价只有 token；general 能写工作区，失控的代价是
+ * 真实损坏，所以反而要收紧。「安全的那个给宽、危险的那个给窄」——
+ * 这与 `resolveAgentType` 未知类型一律回落 explore 是同一条安全默认原则。
+ *
+ * ⚠️ `agentTypes.ts` 的 `EXPLORE_DEF/GENERAL_DEF.maxTurns` 从这里取值，**不要在那边
+ * 再写一份字面量**：那两个数曾经各存一份（都是 200/100），改一处漏一处就会出现
+ * 「有 subagentConfig 时 200、没有时 50」这种只在缺省路径上显形的分裂。
+ */
 export const DEFAULT_SUBAGENTS_CONFIG = {
     enabled: true,
     maxCallsPerTurn: 6,
     maxConcurrent: 3,
     defaultWriteIsolation: 'none',
     timeoutMs: 1_800_000,
-    exploreMaxTurns: 50,
+    exploreMaxTurns: 200,
     generalMaxTurns: 100,
 };
 export const DEFAULT_BROWSER_CONFIG = {
@@ -23,3 +36,30 @@ export const DEFAULT_COMPUTER_USE_CONFIG = {
     maxImagePixels: 1_600_000,
     observationFrames: 1,
 };
+/**
+ * Advisor 默认值——5 处散落魔术数字的「唯一真值」。
+ * 任何 advisor 默认值变更只能改这里，否则就是 numericLimits.ts 注释预言的
+ *「用户改不回值」事故类型（同 DEFAULT_SUBAGENTS_CONFIG / DEFAULT_BROWSER_CONFIG 范式）。
+ *
+ * inputTokenBudget：256K 是 2026-08 的新默认值，从 128K 提到 256K 的理由：
+ *   1. note 上限放宽到 100K 字符（≈100K token）后，128K 预算几乎被 note 吃光，
+ *      历史只能塞几十 K——首句级评审失效。
+ *   2. 主模型普遍升级到 1M 上下文，顾问模型同档位；256K 仍远小于 1M，预算充裕。
+ *   3. fitAdvisorBudget 会自动按「顾问模型 contextWindow - maxTokens - 4096」二次收窄，
+ *      小窗口模型不会被撑爆。
+ *
+ * timeoutMs：120s 是 2026-08 的新默认值，从 180s 压到 120s 的理由：
+ *   180s 在快模型上等太久（用户实际配置中已主动压到 90s 撞墙过），
+ *   60s 在重思考模型上 P95 不够；120s 是 openai / Claude / DeepSeek 系列 P95 的折中。
+ *   重思考专家（DeepSeek-R1、o1/o3 类）用户可上调到 300-600s。
+ */
+export const DEFAULT_ADVISOR_CONFIG = {
+    // 顾问会增加额外模型调用和延迟；必须由用户显式开启。
+    enabled: false,
+    mode: 'auto',
+    modelId: null,
+    contextPolicy: 'full-relevant',
+    shareSensitive: false,
+    inputTokenBudget: 256_000,
+    timeoutMs: 120_000,
+};

package/src/config/configDefaults.ts

@@ -1,4 +1,5 @@
 import type {
+  AdvisorConfig,
   BrowserConfig,
   ComputerUseConfig,
   SubagentsConfig,
@@ -8,13 +9,26 @@ import type {
  * 浏览器与 Node/Bun 都可安全加载的纯配置默认值。
  * 不得在此模块引入 node:*、文件系统或运行时路径逻辑。
  */
+/**
+ * 子任务默认值。
+ *
+ * **为什么 explore(200) 比 general(100) 还宽**（这不是漂移，是刻意的，但此前没写下来）：
+ * explore 的工具面被 `agentTypes.ts::EXPLORE_ALLOWED_TOOLS` 硬裁成严格只读，
+ * 跑再多轮也**毁不掉任何东西**，代价只有 token；general 能写工作区，失控的代价是
+ * 真实损坏，所以反而要收紧。「安全的那个给宽、危险的那个给窄」——
+ * 这与 `resolveAgentType` 未知类型一律回落 explore 是同一条安全默认原则。
+ *
+ * ⚠️ `agentTypes.ts` 的 `EXPLORE_DEF/GENERAL_DEF.maxTurns` 从这里取值，**不要在那边
+ * 再写一份字面量**：那两个数曾经各存一份（都是 200/100），改一处漏一处就会出现
+ * 「有 subagentConfig 时 200、没有时 50」这种只在缺省路径上显形的分裂。
+ */
 export const DEFAULT_SUBAGENTS_CONFIG: SubagentsConfig = {
   enabled: true,
   maxCallsPerTurn: 6,
   maxConcurrent: 3,
   defaultWriteIsolation: 'none',
   timeoutMs: 1_800_000,
-  exploreMaxTurns: 50,
+  exploreMaxTurns: 200,
   generalMaxTurns: 100,
 };
 
@@ -31,3 +45,31 @@ export const DEFAULT_COMPUTER_USE_CONFIG: ComputerUseConfig = {
   maxImagePixels: 1_600_000,
   observationFrames: 1,
 };
+
+/**
+ * Advisor 默认值——5 处散落魔术数字的「唯一真值」。
+ * 任何 advisor 默认值变更只能改这里，否则就是 numericLimits.ts 注释预言的
+ *「用户改不回值」事故类型（同 DEFAULT_SUBAGENTS_CONFIG / DEFAULT_BROWSER_CONFIG 范式）。
+ *
+ * inputTokenBudget：256K 是 2026-08 的新默认值，从 128K 提到 256K 的理由：
+ *   1. note 上限放宽到 100K 字符（≈100K token）后，128K 预算几乎被 note 吃光，
+ *      历史只能塞几十 K——首句级评审失效。
+ *   2. 主模型普遍升级到 1M 上下文，顾问模型同档位；256K 仍远小于 1M，预算充裕。
+ *   3. fitAdvisorBudget 会自动按「顾问模型 contextWindow - maxTokens - 4096」二次收窄，
+ *      小窗口模型不会被撑爆。
+ *
+ * timeoutMs：120s 是 2026-08 的新默认值，从 180s 压到 120s 的理由：
+ *   180s 在快模型上等太久（用户实际配置中已主动压到 90s 撞墙过），
+ *   60s 在重思考模型上 P95 不够；120s 是 openai / Claude / DeepSeek 系列 P95 的折中。
+ *   重思考专家（DeepSeek-R1、o1/o3 类）用户可上调到 300-600s。
+ */
+export const DEFAULT_ADVISOR_CONFIG: AdvisorConfig = {
+  // 顾问会增加额外模型调用和延迟；必须由用户显式开启。
+  enabled: false,
+  mode: 'auto',
+  modelId: null,
+  contextPolicy: 'full-relevant',
+  shareSensitive: false,
+  inputTokenBudget: 256_000,
+  timeoutMs: 120_000,
+};

package/src/config/configDiagnostics.ts

@@ -1,4 +1,4 @@
-import type { Provider } from '../types.js';
+import type { LocalProvider } from '../types.js';
 import type { ModelConfig } from './configFile.js';
 import { type ConfigService, configService } from './configService.js';
 
@@ -27,7 +27,7 @@ function providerFromModel(
     | 'supportsVision'
     | 'promptCache'
   >,
-): Provider {
+): LocalProvider {
   return {
     name: model.displayName ?? model.model,
     apiFormat: model.apiFormat,

package/src/config/configFile.js

@@ -2,11 +2,11 @@ import { randomUUID } from 'node:crypto';
 import { chmod, copyFile, mkdir, readFile, rename, rm, stat, unlink, writeFile } from 'node:fs/promises';
 import { homedir } from 'node:os';
 import { dirname, join } from 'node:path';
-import { execSiblingDir } from '../utils/greenRoot.js';
 import { normalizePromptCacheMode } from '../llm/promptCache.js';
+import { execSiblingDir } from '../utils/greenRoot.js';
+import { DEFAULT_ADVISOR_CONFIG, DEFAULT_BROWSER_CONFIG, DEFAULT_COMPUTER_USE_CONFIG, DEFAULT_SUBAGENTS_CONFIG, } from './configDefaults.js';
 import { CONFIG_LIMITS, normalizeLimited } from './numericLimits.js';
-import { DEFAULT_BROWSER_CONFIG, DEFAULT_COMPUTER_USE_CONFIG, DEFAULT_SUBAGENTS_CONFIG, } from './configDefaults.js';
-export { DEFAULT_BROWSER_CONFIG, DEFAULT_COMPUTER_USE_CONFIG, DEFAULT_SUBAGENTS_CONFIG, } from './configDefaults.js';
+export { DEFAULT_ADVISOR_CONFIG, DEFAULT_BROWSER_CONFIG, DEFAULT_COMPUTER_USE_CONFIG, DEFAULT_SUBAGENTS_CONFIG, } from './configDefaults.js';
 const DEFAULT_CONTEXT_WINDOW = 128_000;
 const LOCK_STALE_MS = 120_000;
 const LOCK_WAIT_MS = 15_000;
@@ -72,6 +72,9 @@ function parseModel(value) {
         contextWindow: positiveNumber(raw.contextWindow) ?? DEFAULT_CONTEXT_WINDOW,
         ...(positiveNumber(raw.maxTokens) !== undefined ? { maxTokens: positiveNumber(raw.maxTokens) } : {}),
         supportsVision: raw.supportsVision !== false,
+        ...(typeof raw.supportsJsonSchema === 'boolean'
+            ? { supportsJsonSchema: raw.supportsJsonSchema }
+            : {}),
         promptCache: {
             mode: normalizePromptCacheMode(raw.promptCache?.mode
                 ?? defaultPromptCacheMode(baseURL)),
@@ -81,6 +84,19 @@ function parseModel(value) {
         updatedAt: positiveNumber(raw.updatedAt) ?? now,
     };
 }
+function parseWorkflowModels(value) {
+    if (!value || typeof value !== 'object' || Array.isArray(value))
+        return undefined;
+    const roles = {};
+    for (const [role, candidate] of Object.entries(value)) {
+        if (!/^[a-z][a-z0-9_-]{0,63}$/i.test(role))
+            continue;
+        const modelId = nonEmpty(candidate);
+        if (modelId)
+            roles[role] = modelId;
+    }
+    return Object.keys(roles).length > 0 ? roles : undefined;
+}
 function parseModels(value) {
     if (!Array.isArray(value))
         return [];
@@ -130,13 +146,14 @@ function parseAdvisor(value) {
         return undefined;
     const raw = value;
     return {
-        enabled: raw.enabled !== false,
+        // 默认关闭；旧配置显式 enabled:true 仍保持用户选择。
+        enabled: raw.enabled === true,
         mode: raw.mode === 'manual' ? 'manual' : 'auto',
         modelId: nonEmpty(raw.modelId) ?? null,
         contextPolicy: 'full-relevant',
         shareSensitive: raw.shareSensitive === true,
-        inputTokenBudget: normalizeLimited(raw.inputTokenBudget, CONFIG_LIMITS.advisorInputTokenBudget, 128_000),
-        timeoutMs: normalizeLimited(raw.timeoutMs, CONFIG_LIMITS.advisorTimeoutMs, 180_000),
+        inputTokenBudget: normalizeLimited(raw.inputTokenBudget, CONFIG_LIMITS.advisorInputTokenBudget, DEFAULT_ADVISOR_CONFIG.inputTokenBudget),
+        timeoutMs: normalizeLimited(raw.timeoutMs, CONFIG_LIMITS.advisorTimeoutMs, DEFAULT_ADVISOR_CONFIG.timeoutMs),
     };
 }
 function parseSubagents(value) {
@@ -147,7 +164,7 @@ function parseSubagents(value) {
         maxConcurrent: normalizeLimited(raw.maxConcurrent, CONFIG_LIMITS.subagentMaxConcurrent, 3),
         defaultWriteIsolation: raw.defaultWriteIsolation === 'worktree' ? 'worktree' : 'none',
         timeoutMs: normalizeLimited(raw.timeoutMs, CONFIG_LIMITS.subagentTimeoutMs, 1_800_000),
-        exploreMaxTurns: normalizeLimited(raw.exploreMaxTurns, CONFIG_LIMITS.subagentExploreMaxTurns, 50),
+        exploreMaxTurns: normalizeLimited(raw.exploreMaxTurns, CONFIG_LIMITS.subagentExploreMaxTurns, 200),
         generalMaxTurns: normalizeLimited(raw.generalMaxTurns, CONFIG_LIMITS.subagentGeneralMaxTurns, 100),
     };
 }
@@ -177,13 +194,16 @@ function parseCurrent(data) {
     if (data.version !== 4)
         return null;
     const advisor = parseAdvisor(data.advisor);
+    const workflowModels = parseWorkflowModels(data.workflowModels);
     const compactRatio = parseCompactRatio(data.compactRatio);
     return {
         version: 4,
         models: parseModels(data.models),
         baseModelId: nonEmpty(data.baseModelId) ?? null,
+        teamServerUrl: nonEmpty(data.teamServerUrl) ?? null,
         ...(compactRatio !== undefined ? { compactRatio } : {}),
         ...(advisor ? { advisor } : {}),
+        ...(workflowModels ? { workflowModels } : {}),
         subagents: parseSubagents(data.subagents),
         browser: parseBrowser(data.browser),
         computerUse: parseComputerUse(data.computerUse),
@@ -227,8 +247,8 @@ function parseLegacyAdvisorConfig(value) {
         provider,
         supportsVision: raw.supportsVision === true,
         shareSensitive: raw.shareSensitive === true,
-        inputTokenBudget: normalizeLimited(raw.inputTokenBudget, CONFIG_LIMITS.advisorInputTokenBudget, 128_000),
-        timeoutMs: normalizeLimited(raw.timeoutMs, CONFIG_LIMITS.advisorTimeoutMs, 180_000),
+        inputTokenBudget: normalizeLimited(raw.inputTokenBudget, CONFIG_LIMITS.advisorInputTokenBudget, DEFAULT_ADVISOR_CONFIG.inputTokenBudget),
+        timeoutMs: normalizeLimited(raw.timeoutMs, CONFIG_LIMITS.advisorTimeoutMs, DEFAULT_ADVISOR_CONFIG.timeoutMs),
     };
 }
 /**
@@ -294,6 +314,7 @@ function buildV4FromLegacy(legacyProvider, legacyAdvisor, subagents, tools, save
         version: 4,
         models,
         baseModelId: baseModel.id,
+        teamServerUrl: null,
         ...(legacyProvider.compactRatio !== undefined ? { compactRatio: legacyProvider.compactRatio } : {}),
         ...(advisor ? { advisor } : {}),
         subagents,
@@ -449,8 +470,10 @@ export async function saveMinimalAgentConfig(config) {
         version: 4,
         models: config.models,
         baseModelId: config.baseModelId,
+        teamServerUrl: config.teamServerUrl ?? null,
         ...(config.compactRatio !== undefined ? { compactRatio: config.compactRatio } : {}),
         advisor: config.advisor,
+        workflowModels: config.workflowModels,
         subagents: config.subagents ?? DEFAULT_SUBAGENTS_CONFIG,
         browser: config.browser ?? DEFAULT_BROWSER_CONFIG,
         computerUse: config.computerUse ?? DEFAULT_COMPUTER_USE_CONFIG,
@@ -472,8 +495,16 @@ export async function updateMinimalAgentConfig(mutate) {
             version: 4,
             models: value.models,
             baseModelId: value.baseModelId,
+            // 沿用现有值而不是重置为 null——大多数配置写入根本不碰这个字段（如加/删模型、
+            // 改 advisor），不该每次都把用户配置的默认服务器地址悄悄抹掉。同一条纪律见
+            // subagents/browser/computerUse。
+            // ⚠️ 必须用 `!== undefined` 而不是 `??`：清空地址走的是显式 `teamServerUrl: null`，
+            // `??` 会把这个合法的 null 也当成"没提供"处理，退回旧值——清空操作因此悄悄失效。
+            // 这正是仓库自己反复强调的 `undefined ≠ 0`（这里是 `undefined ≠ null`）纪律。
+            teamServerUrl: value.teamServerUrl !== undefined ? value.teamServerUrl : (current?.teamServerUrl ?? null),
             ...(value.compactRatio !== undefined ? { compactRatio: value.compactRatio } : {}),
             advisor: value.advisor,
+            workflowModels: value.workflowModels ?? current?.workflowModels,
             subagents: value.subagents ?? current?.subagents ?? DEFAULT_SUBAGENTS_CONFIG,
             browser: value.browser ?? current?.browser ?? DEFAULT_BROWSER_CONFIG,
             computerUse: value.computerUse ?? current?.computerUse ?? DEFAULT_COMPUTER_USE_CONFIG,