jesse-uxandGrok (agent loop)
360a667e4f
The consultation contract still required await directly before runConsultationWorkflow(toolInput), and a new report test shifted three privacy allowlist lines. Keep the toolInput lock and move that test to the end. No product code change. Co-Authored-By: Grok (agent loop)
47 lines
3.3 KiB
Markdown
47 lines
3.3 KiB
Markdown
# 进度 · 内测容量修复（2026-10-15，第 1 次打回）
|
||
|
||
- 分支：`codex/beta-capacity-20261014`
|
||
- 被验 SHA：`a54f75e5e33775d5dc91f3058f5505a56b498bf2`
|
||
- 修复单：`docs/tasks/TASK-beta-capacity-fix-20261015.md`
|
||
- 原任务书：`docs/tasks/TASK-beta-capacity-20261014.md`
|
||
- 开工时本分支最大 Bug 编号：BUG-1356。本轮新增 BUG-1357。BUG-1351、BUG-1352、BUG-1353 已在本分支文件里，没有重用。
|
||
|
||
## 做了什么
|
||
|
||
- 先把最新 `origin/staging` 合并进本分支（不改写已推送的 `a54f75e5`）。合并只带进修复单和任务索引的一行。
|
||
- 语音合同：仍锁「工作流调用的入参是 `toolInput`」，以及用户轮不塞工具 JSON。不再要求 `await` 紧贴函数名。
|
||
- 隐私白名单：采用修复单推荐的做法 a。把「hourly rate limit returns 429」整段挪到 `personal-report-api.test.ts` 末尾，断言原文不变。`SAFE_LINES` 不改。三行回到 832、860、890，且整行与白名单一致。
|
||
- 业务代码不改。
|
||
|
||
## 断言
|
||
|
||
| 位置 | 原值 | 新值 | 原因 |
|
||
| --- | --- | --- | --- |
|
||
| `consultation-voice-contract.test.ts`「consult user turn no longer dumps tool JSON」 | `/await runConsultationWorkflow\(toolInput/` | `/runConsultationWorkflow\(toolInput/` | 调用包进 `retryOnConsultationRateLimit`，`await` 不再紧贴函数名。仍锁入参是 `toolInput` |
|
||
| `personal-report-api.test.ts` 小时限流测试的位置 | 插在「429 at the daily limit」和「502 model_unavailable」之间 | 文件末尾，断言原文不变 | 插在中部让三行白名单各下移 13 行。挪走后行号复原，不改 `SAFE_LINES` |
|
||
|
||
## 源码文本再搜一遍
|
||
|
||
在 `frontend/tests` 和 `tests` 里核对三条路由被改写的语句：
|
||
|
||
- 只有语音合同把 `await runConsultationWorkflow(toolInput` 写成必须匹配。已按上表改。
|
||
- `consultation-workflow-contract.test.ts` 是在代理分支里断言**不**出现 `await runConsultationWorkflow`，与这次包装无关，没改。
|
||
- 「暂时无法生成解读」仍在非排队失败的分支里，`consult-first-frame-and-pacing-20260928.test.tsx` 继续对得上，没改。
|
||
- `toNextResponse` 多了响应头，没有测试按旧函数签名做源码断言。
|
||
- Python 侧没有按这三处被改写语句做源码文本断言。
|
||
|
||
## 偏离与环境缺口
|
||
|
||
- 没有改数据库，没跑 `test:db`。没有跑前端全量 `npm test`，没有跑 Python 快速门。
|
||
- 并发压测 `frontend/scripts/research/beta-load-smoke.mts` 未跑。没有测试验证码，修复单也写明本机和执行侧都连不上测试服务器，不要登预发。没有改前/改后并发数字，没有服务器占用采样。
|
||
|
||
## 自测
|
||
|
||
| 检查 | 命令 | 结果数字 |
|
||
| --- | --- | --- |
|
||
| 前端相关测试 | `node node_modules/tsx/dist/cli.mjs --test tests/consultation-voice-contract.test.ts tests/runtime-sources-no-test-imports.test.ts tests/personal-report-api.test.ts` | 75 通过，0 失败，0 取消 |
|
||
| 隐私扫描 | `.venv\Scripts\python.exe -m pytest tests/test_repo_privacy_markers.py -q` | 62 通过，0 失败 |
|
||
| 类型检查 | `frontend\node_modules\.bin\tsc --noEmit -p .` | 退出码 0 |
|
||
| 前端 lint | `npm run lint` | 0 错误，123 条原有警告 |
|
||
| 并发压测 | 未跑 `beta-load-smoke.mts` | 没有测试验证码，没有登预发 |
|