docs(tasks): 咨询第 0 步 required 修复单编号顺延为 BUG-937/938
Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01JUei7K13cYxLHE3Axe4A45
This commit is contained in:
Jesse_Chen
co-authored by
Claude Fable 5.1
Claude Fable 5.1
parent
c9d7030f4f
commit
f3f77af862
@@ -131,7 +131,7 @@
|
||||
| `TASK-rectification-p0-fix-20260915.md` | `PROGRESS-rectification-p0-fix-20260915.md` | **验收修复单**：`f51e494c` 六条缺陷全部实现且方式正确，但 `page.tsx` 从 1951 涨到 1964 行，撞了 `chart-view-route.test.ts` 的 `<= 1951` 上限（AGENTS.md §6 增长冻结）。全量 fail 32→33，就这一条。门禁红很可能是 staging 停在 `2d7698ea`、6 个提交未部署的原因。修法是把 BUG-705 的十来行接线搬出 page.tsx，不放宽上限 | 待验收 | `codex/rectification-p0-fix-20260915` |
|
||||
| `TASK-settings-dialog-size-and-nav-20260915.md` | — | **复发单**：设置弹窗四个分区尺寸仍随内容跳变（BUG-698，复发自 BUG-554——旧防复发只查「有没有写 height」，查不到「写了没生效」）；首要嫌疑是 `.settings-modal` 的 `dvh` 没有 `vh` 回退，不支持时整条 `height` 作废退化成内容高度，需先复现确认。另按产品要求去掉分区菜单左侧强调条，并拆开与悬停共用的选中态 | 待领取 | `codex/settings-dialog-size-and-nav-20260915` |
|
||||
| `TASK-consult-followup-tool-contract-20260917.md` | `PROGRESS-consult-followup-tool-contract-20260917.md` | 真机：申报时段会话连发「？」「你在说什么鬼」都 `run.failed runtime_contract_incomplete`，回执无任何 `tool` 步骤。根因是 Agent 系统指令写明「简单追问可复用已有 packet / context、不调工具」，而 `contractReady()` 要求每次请求恰好一次成功排盘调用；「已有 packet」跨请求并不存在（缓存只在单次请求内）。本命与窗口两个 Agent 同构。**产品拍板方案 1**：每轮必调工具（BUG-922 删例外句 + BUG-923 第 0 步 `toolChoice: required`）；否决「没调工具就走不扣点纯对话」。第一轮正经问题为何失败留 T4 取证（回执只在 web 容器日志） | 已部署 `dc2f2a16`，但 BUG-923 的第 0 步 `required` 让全部咨询整轮失败（BUG-282 复发），由修复单撤回 | `codex/consult-followup-tool-contract-20260917` |
|
||||
| `TASK-consult-followup-tool-contract-fix-20260917.md` | `PROGRESS-consult-followup-tool-contract-fix-20260917.md` | **P0**。部署 `dc2f2a16` 后再聊天仍 `runtime_contract_incomplete`，回执只有 skill + retry。主根因（推断）：BUG-923 把本命/窗口第 0 步改成 `toolChoice: required`，而首轮固定开 thinking，BUG-282 实证 thinking 模式供应商拒收非 auto 的 tool_choice → 每一轮咨询在第 0 步就被拒（BUG-936，复发自 BUG-282）。次根因（确定）：咨询流不处理 Mastra `error` 块，拒收在回执/事件/日志三处都不可见，合同 retry 照跑同一参数（BUG-937）。T1 撤回 required 只留 activeTools + auto；T2 error 块 → 可诊断错误码、不进合同 retry；T4 部署后取证 | 待领取 | `codex/consult-followup-tool-contract-fix-20260917` |
|
||||
| `TASK-consult-followup-tool-contract-fix-20260917.md` | `PROGRESS-consult-followup-tool-contract-fix-20260917.md` | **P0**。部署 `dc2f2a16` 后再聊天仍 `runtime_contract_incomplete`，回执只有 skill + retry。主根因（推断）：BUG-923 把本命/窗口第 0 步改成 `toolChoice: required`，而首轮固定开 thinking，BUG-282 实证 thinking 模式供应商拒收非 auto 的 tool_choice → 每一轮咨询在第 0 步就被拒（BUG-937，复发自 BUG-282）。次根因（确定）：咨询流不处理 Mastra `error` 块，拒收在回执/事件/日志三处都不可见，合同 retry 照跑同一参数（BUG-938）。T1 撤回 required 只留 activeTools + auto；T2 error 块 → 可诊断错误码、不进合同 retry；T4 部署后取证 | 待领取 | `codex/consult-followup-tool-contract-fix-20260917` |
|
||||
| `TASK-session-list-single-source-20260917.md` | `PROGRESS-session-list-single-source-20260917.md` | 会话列表一处数据源：本地 PG 兼容层 `order()` 只保留最后一键，`/api/sessions` 实际按 `id` 排、与游标不一致；`/` 与次级页两份数据源、`/` 每次回来重启动（产品拍板：首页与三个次级页进同一路由组，侧栏外壳与列表 provider 常驻 layout，不做 store）；空「新对话」落库堆积（首页 50 条里 28 条）；标题类别在后、同名靠墙钟 HH:MM。串行在 composer-guard 单之后。BUG 段 926 起 | 验收未通过 | `e4e73f56`（实现按任务书落地：多键排序 / `(app)` 路由组常驻外壳 / provider / 空会话不入列 + draft 复用 / 标题类别在前；两条让步任务书允许。**P1：4 条针对 `page.tsx` 的源码合同没跟着外壳搬家，`npm test` 在门禁里 `set -e` 下必红，staging 至今停在 `dc2f2a16`**。另：进度记录报「tsc 0 错」实为 2 错，只跑了定向测试。修复单 `TASK-session-list-single-source-fix-20260917.md`） |
|
||||
| `TASK-session-list-single-source-fix-20260917.md` | `PROGRESS-session-list-single-source-fix-20260917.md` | 验收修复单：F1 四条源码合同搬到 `(app)/layout.tsx` 两端对断（BUG-933）；F2 两条陈旧 Python 入口断言（BUG-934，既有欠账）；F3 无活跃会话时输入框静默吞发送（BUG-935）；交付前必须跑全量测试 | 待验收 | `codex/session-list-single-source-fix-20260917` |
|
||||
| `TASK-first-paint-dead-screen-fallback-20260917.md` | — | 真机：首页永远停在「正在载入账户」，兜底全在没跑起来的 bundle 里（BUG-936 investigating）。根 layout 加与 bundle 无关的内联兜底 + 去掉本仓正则后行断言 | 待领取 | — |
|
||||
|
||||
@@ -5,7 +5,7 @@
|
||||
- 基线 commit：`db5b0a21`（`origin/staging` head）。staging 已部署 `dc2f2a16`（`/api/health` 的 `deployment.gitCommit`），即 BUG-922/923 的实现；其后到 head 只有会话列表 / 钉顶等前端改动，与本单无交集。
|
||||
- 分支：`codex/consult-followup-tool-contract-fix-20260917`，`git worktree add -b codex/consult-followup-tool-contract-fix-20260917 .worktrees/consult-followup-tool-contract-fix-20260917 origin/staging`。
|
||||
- 范围：只有前端 `frontend/src/mastra/consultation-tools.ts`、`frontend/src/lib/stream-agent-response.ts`、`frontend/src/lib/agent-observability.ts`（如需扩错误码枚举）与对应测试。不动 Python、不动迁移、不动 Skill、不动 `page.tsx`、不动生时校正 Agent、不动 `contractReady()`。
|
||||
- BUG 段：**BUG-936 起**（基线 `docs/BUG_HISTORY.md` 最大号 BUG-935，开工时再核对一次）。
|
||||
- BUG 段：**BUG-937 起**（基线 `docs/BUG_HISTORY.md` 最大号 BUG-936（另一会话同日已占 936），开工时再核对一次）。
|
||||
- 优先级：**P0**。按 §2 的推断，本命与申报时段两条路线的每一轮咨询在 staging 上都会失败，不只是追问轮。
|
||||
|
||||
## 1. 事故实证
|
||||
@@ -50,7 +50,7 @@
|
||||
## 3. 决策记录
|
||||
|
||||
- **本单撤销 BUG-923 的 `toolChoice: "required"`，回到 BUG-282 / BUG-630 的口径：第 0 步只用 `activeTools` 收窄到排盘工具，`toolChoice` 为 `"auto"`。** BUG-923 记录里「若线上某模型拒收，另立单，不得静默改回 auto 而不改提示词」——本单就是那张单；提示词侧的 BUG-922（每轮必调、packet 不跨请求）**保留不动**，它才是 BUG-922/923 事故里模型跳过工具的真正原因。
|
||||
- 责任说明：`TASK-consult-followup-tool-contract-20260917.md` 的 T2 是本会话（Claude）写的，写时没有检索到 BUG-282 的防复发条款，执行方按单做了。这不是执行方的错。本单在 BUG-936 记录里如实写「复发自 BUG-282」，并说明旧防复发为何没拦住：BUG-282 的约束只落在 `rectification-v9-agent.test.ts` 里锁校正 Agent 的第一步，咨询侧没有任何测试禁止 `required`，任务书作者也没检索到。
|
||||
- 责任说明：`TASK-consult-followup-tool-contract-20260917.md` 的 T2 是本会话（Claude）写的，写时没有检索到 BUG-282 的防复发条款，执行方按单做了。这不是执行方的错。本单在 BUG-937 记录里如实写「复发自 BUG-282」，并说明旧防复发为何没拦住：BUG-282 的约束只落在 `rectification-v9-agent.test.ts` 里锁校正 Agent 的第一步，咨询侧没有任何测试禁止 `required`，任务书作者也没检索到。
|
||||
- **不在本单里尝试「第 0 步关 thinking 再发 `required`」。** Mastra 的 `prepareStep` 返回值支持 `providerOptions` 覆盖，理论上可以在第 0 步关 thinking 换取确定性的工具调用；但本仓没有任何证据证明该供应商在非 thinking 模式接受 `required`，且第 0 步是模型选领域的一步，关掉推理会改变回答质量。这条路要另立研究单、带真实供应商实测，不得在本单顺手试。
|
||||
- 产品负责人 2026-09-17 授权：**先止血（T1）并把错误做成可见（T2）**，不等 T4 取证；T4 的日志证据在部署后补。
|
||||
- 不推翻任何既有红线：`contractReady()` 「恰好一次成功调用」不放宽；BUG-214（失败尝试后重试成功仍过门禁）语义保持；BUG-286（契约未绿即丢弃文本）语义保持；BUG-922 的提示词保持。
|
||||
@@ -66,16 +66,16 @@
|
||||
|
||||
## 5. 任务分解
|
||||
|
||||
### T1 第 0 步撤回 `required`，只留 `activeTools`（BUG-936）
|
||||
### T1 第 0 步撤回 `required`，只留 `activeTools`（BUG-937）
|
||||
|
||||
- `consultationNatalPrepareStep` / `consultationWindowPrepareStep`：第 0 步改回 `{ activeTools: [<排盘工具>], toolChoice: "auto" }`；第 1 步起保持 `{ toolChoice: "auto" }`（保留 `activeTools` 不设，即全部工具可用，与现状一致）。
|
||||
- 在两个函数上方加一条注释指向 BUG-282 / BUG-936：thinking 模式不得发 named / required。
|
||||
- 在两个函数上方加一条注释指向 BUG-282 / BUG-937：thinking 模式不得发 named / required。
|
||||
- 验收：
|
||||
- `frontend/tests/consultation-agentic-runtime.test.ts` 里 BUG-923 加的两条断言改为 `toolChoice: "auto"`，三栏写「原值 `required` / 新值 `auto` / 原因 BUG-282 供应商拒收，BUG-936」。
|
||||
- `frontend/tests/consultation-agentic-runtime.test.ts` 里 BUG-923 加的两条断言改为 `toolChoice: "auto"`，三栏写「原值 `required` / 新值 `auto` / 原因 BUG-282 供应商拒收，BUG-937」。
|
||||
- 新增一条源码契约（放 `consultation-workflow-contract.test.ts`）：`consultation-tools.ts` 全文 `doesNotMatch(/toolChoice:\s*"required"/)`，并 `doesNotMatch(/type:\s*"tool",\s*toolName/)`，把 BUG-282 的约束落到咨询侧。
|
||||
- 既有「natal first step exposes only the chart calculation tool」、`route.ts` 申报时段分支使用 `windowStreamOptions` 的契约（`dc2f2a16` 加的）原样绿——`prepareStep` 本身保留，只是不再 required。
|
||||
|
||||
### T2 咨询流识别 Mastra `error` 块并给出可诊断错误码（BUG-937）
|
||||
### T2 咨询流识别 Mastra `error` 块并给出可诊断错误码（BUG-938）
|
||||
|
||||
- `frontend/src/lib/stream-agent-response.ts` `consumeAttempt`：遇到 `chunk.type === "error"` 时记录失败并**立即结束本次 attempt**，抛出携带闭合错误码的 `Error`：
|
||||
- 错误原文含 `Thinking mode does not support this tool_choice` → `thinking_tool_choice_unsupported`；
|
||||
@@ -96,12 +96,12 @@
|
||||
### T3 记录
|
||||
|
||||
- `docs/BUG_HISTORY.md`：
|
||||
- 新增 **BUG-936**（`toolChoice: required` 复发 BUG-282，两条咨询路线整轮失败）。状态 `resolved`，「复发自：BUG-282」，写明旧防复发为何未拦住（约束只锁在校正 Agent 测试与记录文字里，咨询侧无测试、任务书作者未检索到）。关联 BUG-282、BUG-630、BUG-922、BUG-923。
|
||||
- 新增 **BUG-937**（咨询流不处理 Mastra `error` 块，供应商拒收不可见、合同重试空转）。关联 BUG-282、BUG-214、BUG-268（同为「回执缺字段导致不可观测」）。
|
||||
- **BUG-923** 记录追加一行「最近更新 2026-09-17：`required` 被 BUG-936 撤回，防复发改为 `activeTools` + `auto`」，状态保持 `resolved` 但修复描述要改成实际留下的做法，不得留着「第 0 步 required」当防复发。
|
||||
- 新增 **BUG-937**（`toolChoice: required` 复发 BUG-282，两条咨询路线整轮失败）。状态 `resolved`，「复发自：BUG-282」，写明旧防复发为何未拦住（约束只锁在校正 Agent 测试与记录文字里，咨询侧无测试、任务书作者未检索到）。关联 BUG-282、BUG-630、BUG-922、BUG-923。
|
||||
- 新增 **BUG-938**（咨询流不处理 Mastra `error` 块，供应商拒收不可见、合同重试空转）。关联 BUG-282、BUG-214、BUG-268（同为「回执缺字段导致不可观测」）。
|
||||
- **BUG-923** 记录追加一行「最近更新 2026-09-17：`required` 被 BUG-937 撤回，防复发改为 `activeTools` + `auto`」，状态保持 `resolved` 但修复描述要改成实际留下的做法，不得留着「第 0 步 required」当防复发。
|
||||
- `CHANGELOG.md` 一条：修复部署 `dc2f2a16` 后所有咨询整轮失败；Skill 版本不变。
|
||||
- `docs/tasks/PROGRESS-consult-followup-tool-contract-fix-20260917.md`：测试数字、失败清单与基线逐条比对、gzip 前后、两条断言的三栏说明。
|
||||
- `docs/tasks/README.md`：本单一行，并把 `TASK-consult-followup-tool-contract-20260917.md` 那行状态改为「已部署但被 BUG-936 撤回 required，见修复单」。
|
||||
- `docs/tasks/README.md`：本单一行，并把 `TASK-consult-followup-tool-contract-20260917.md` 那行状态改为「已部署但被 BUG-937 撤回 required，见修复单」。
|
||||
|
||||
### T4 部署后取证（不占 BUG 号，留给产品负责人 / 部署侧）
|
||||
|
||||
@@ -132,7 +132,7 @@ git fetch origin --prune
|
||||
git worktree add -b codex/consult-followup-tool-contract-fix-20260917 .worktrees/consult-followup-tool-contract-fix-20260917 origin/staging
|
||||
cd .worktrees/consult-followup-tool-contract-fix-20260917/frontend
|
||||
./node_modules/.bin/tsc --noEmit && npm run lint && npm test 2>&1 | tail -20 # 记下基线总数与失败清单
|
||||
grep -o "^## BUG-[0-9]*" ../docs/BUG_HISTORY.md | sed 's/## BUG-//' | sort -n | tail -1 # 应为 935
|
||||
grep -o "^## BUG-[0-9]*" ../docs/BUG_HISTORY.md | sed 's/## BUG-//' | sort -n | tail -1 # 应为 936
|
||||
grep -n 'toolChoice: "required"' src/mastra/consultation-tools.ts # 应恰好 2 处，做完为 0
|
||||
grep -n '"error"' src/lib/stream-agent-response.ts # 开工应为 0 处
|
||||
```
|
||||
|
||||
Reference in New Issue
Block a user