diff --git a/.doc/ops/CURRENT.md b/.doc/ops/CURRENT.md index 1346012d..6ef60917 100644 --- a/.doc/ops/CURRENT.md +++ b/.doc/ops/CURRENT.md @@ -1,19 +1,19 @@ # 当前焦点 -**更新**: 2026-09-17 +**更新**: 2026-09-18 **production_ready**: yes（门禁 B1–B4 已签；dev/test `schema_version=48`；main `schema_version=47`；goose 00048 已在 test 迁移。不把 fallback change report 写成完整 PI） ## 焦点 -1. **rdev 模板深度 S1 续批**：已合入并推送 `origin/dev`（60/60 模板 + 真 PI 抽样）；未晋升 test/main -2. **document-author-only / 变更提案 HTTP**：已合入并推送 `origin/dev`；dev 试点总开关 on；未晋升 test/main +1. **系统体检整改**：Wave 0（SSE/缓存 P0）**done**；Wave 1（main ENV E-1）待决策 +2. **rdev 模板深度 / document-author-only**：已合入 `origin/dev`；未晋升 test/main 3. **红线**：禁止整支 merge FAV；不默认升 `MULTI_INTENT_PLAN_TIER=full`；`SEMANTIC_SUMMARY_REPLACE` 不随 W2 开启；FAV live 观察窗证据驱动；`FAV_DEGRADED_READ=false`；项目 143 只读 ## 并行认领 | owner | task | branch | worktree | mutex | status | |-------|------|--------|----------|-------|--------| -| Cursor | 2026-09-18-wave0-p0-bug-fixes | fix/wave0-p0-sse-chat-cache-bugs | - | chat + web | in_progress | +| Cursor | 2026-09-18-wave0-p0-bug-fixes | fix/wave0-p0-sse-chat-cache-bugs | - | chat + web | done | | Kimi | 2026-09-17-system-health-check（只读审计，未实施） | - | - | audit | done | | Cursor | 2026-09-17-docgov-modification-proposal-http | feature/docgov-modification-proposal-http | - | docgov | done | | Cursor | 2026-09-17-document-author-only-closeout | feature/document-author-only-plan | - | document-duty-policy | done | @@ -132,7 +132,7 @@ | holder | phase | since | scope | queue | |--------|-------|-------|-------|-------| -| - | idle | 2026-09-17 | - | - | +| - | idle | 2026-09-18 | Wave 0 P0 已合入并冒烟 PASS（见 [test-report](../test-reports/2026-09-18-wave0-p0-bug-fixes-test-report.md)） | - | **phase**: `idle` | `merging` | `smoke` | `e2e` diff --git a/.doc/ops/short-term/2026-09-18-wave0-p0-bug-fixes.md b/.doc/ops/short-term/2026-09-18-wave0-p0-bug-fixes.md index 2b14fca8..8372f482 100644 --- a/.doc/ops/short-term/2026-09-18-wave0-p0-bug-fixes.md +++ b/.doc/ops/short-term/2026-09-18-wave0-p0-bug-fixes.md @@ -1,6 +1,6 @@ --- title: "Wave 0：P0 确认 bug 修复（SSE/缓存/run-stream）" -status: in_progress +status: done created: 2026-09-18 updated: 2026-09-18 owner: Cursor @@ -30,25 +30,26 @@ mutex: chat + web ## 步骤 1. [x] 从 dev 开 `fix/wave0-p0-sse-chat-cache-bugs`，CURRENT 认领 -2. [ ] B-1/B-2 心跳与 deadline -3. [ ] B-3/B-4 run/stream -4. [ ] B-5 缓存 key -5. [ ] B-6 恢复合并 -6. [ ] 单测 + 全量回归 + 合入 dev 冒烟 +2. [x] B-1/B-2 心跳与 deadline +3. [x] B-3/B-4 run/stream +4. [x] B-5 缓存 key +5. [x] B-6 恢复合并 +6. [x] 单测 + 全量回归 + 合入 dev 冒烟 ## 验收标准 -- [ ] `go vet ./...`、`go test -short ./...`、`cd web && npm test && npm run build` 全绿 -- [ ] 畸形 body → 400；Flush 前失败非 200 SSE -- [ ] 缓存 key / LRU 单测 PASS；恢复合并不丢新增 deliverable -- [ ] dev rebuild + health/ready；真实 >120s 聊天回合不断流（可行时） +- [x] `go vet ./...`、`go test -short ./...`、`cd web && npm test && npm run build` 全绿 +- [x] 畸形 body → 400；Flush 前失败非 200 SSE +- [x] 缓存 key / LRU 单测 PASS；恢复合并不丢新增 deliverable +- [x] dev rebuild + health/ready；真 PI 聊天心跳证据（见测试报告） ## 相关链接 - Audit: [../../audit/2026-09-17-system-health-check.md](../../audit/2026-09-17-system-health-check.md) - Remediation: [../../remediation/2026-09-17-system-health-remediation-plan.md](../../remediation/2026-09-17-system-health-remediation-plan.md) -- Test report: （待归档） +- Test report: [../../test-reports/2026-09-18-wave0-p0-bug-fixes-test-report.md](../../test-reports/2026-09-18-wave0-p0-bug-fixes-test-report.md) ## 证据 / 备注 -（实施中） +- Commits：`64efaa3` docs · `54244ed` server · `2620d80` web；合入 `dev` merge `810fb96` +- Assets：`.doc/test-reports/assets/2026-09-18-wave0-p0-bug-fixes/chat-stream-long.sse.log` diff --git a/.doc/remediation/2026-09-17-system-health-remediation-plan.md b/.doc/remediation/2026-09-17-system-health-remediation-plan.md index 0ef2e5b5..334bdc02 100644 --- a/.doc/remediation/2026-09-17-system-health-remediation-plan.md +++ b/.doc/remediation/2026-09-17-system-health-remediation-plan.md @@ -27,7 +27,7 @@ updated: 2026-09-17 | 波次 | 内容 | 验收 | |------|------|------| -| **Wave 0：P0 确认 bug** | B-1/B-2：`handleChatStream` 与 `handleRunTaskStream` 接 `extendSSEWriteDeadline` + 心跳（复用 sse.go:52-64 既有机制）；run/stream 支持 `after_seq` 续订。B-3：Decode 失败返回 400。B-4：CreateAndStart/Subscribe 前置到 Flush 之前（对齐 chat 路径）。B-5：useChatWindow 全部写入统一带 `userId` 前缀（或统一不带），LRU 驱逐 key 规则一致。B-6：`restoreFileRawToDeliverables` 改函数式更新/用 `deliverablesRef` 合并 | 单测：chat/run 流 >120s 不断（可用注入假时钟或短 WriteTimeout 测试）；畸形 body 400；Flush 前失败返回非 200。前端 vitest 覆盖缓存 key 与恢复合并。`go test -short ./...` + `npm test` 全绿 + dev 容器冒烟（真实 >120s 聊天回合复测） | +| **Wave 0：P0 确认 bug** | B-1/B-2：`handleChatStream` 与 `handleRunTaskStream` 接 `extendSSEWriteDeadline` + 心跳（复用 sse.go:52-64 既有机制）；run/stream 支持 `after_seq` 续订（重连走既有 `GET .../events?after_seq=`）。B-3：Decode 失败返回 400。B-4：CreateAndStart/Subscribe 前置到 Flush 之前（对齐 chat 路径）。B-5：useChatWindow 全部写入统一带 `userId` 前缀，LRU 驱逐完整 cache key。B-6：`applyRestoredFileRaw` 函数式合并 | **done**（2026-09-18）：[test-report](../test-reports/2026-09-18-wave0-p0-bug-fixes-test-report.md)；合入 `dev` merge `fix/wave0-p0-sse-chat-cache-bugs`；dev rebuild + health/ready PASS；真 PI SSE 心跳证据归档 | | **Wave 1：main 环境修正（critical）** | E-1：统一 main 环境 app/events 的 ENV 语义（推荐 app 改 `ENV=main`，同时核对 `ResolveBackendInEnv` 等对 `prod` 的既有依赖；若保留 `prod` 则事件信封键与 `event_consumer_modes` 种子需对齐——二选一，先出 1 页决策）。E-2：main 晋升 checklist 固化「events 先升跑迁移、app 后升」，schema 47→49。E-5：`Env==""` 不再视同 dev | main 容器重建后：事件 producer/consumer mode 一致核验（写入一条事件观察 dispatcher 行为无双写）；`health` schema_version=49；项目工作流查询与 `/api/v1/admin/system/settings` 读写在 main 冒烟 PASS | | **Wave 2：配置与灰度入口对齐** | E-3：`MULTI_INTENT_PLAN_TIER` 与 U9–U12 flag 显式写入三环境 compose（值保持 legacy/false，只补声明）；A-4 类（SEMANTIC_SUMMARY_SHADOW、PI_CHANGE_REPORT_NARRATIVE）同步补 test/main 声明。E-4：`CURSOR_RUNNER_URL` 与 cursor-agent 占位容器从 compose 移除或标注。E-6：main compose 显式 `SEED_DEMO_USERS=false`。E-7：`INTENT_V2_DECISION_TIER` 非法值回退改 `none`（保守对齐）。E-9：main initdb 脚本与 dev/test 对齐评估 | 三环境 compose `config` 渲染 diff 复核；容器内 `env` 抽检 flag 可见；`go test -short` 全绿；不改变任何默认行为（纯声明对齐，验证 dev/test/main 冒烟无回归） | | **Wave 3：测试债与盲区** | S-1：申请隔离 PG（docker 单发 5436），执行 `TestReconcile_LegacyRepair_PG` 一次并归档报告，关闭 gap-matrix 测试债。S-2：删除 `taskstream_test.go:189` 死跳过；`plan_owner_test.go:125` 补说明或修复。S-4：统一 E2E 入口——`web/package.json` 为 9 个 spec 各配 script 或统一 flag 约定（消除 `DOCREV_E2E_BASE_URL` 与 `RUN_UI_ACCEPTANCE` 双口径）；README/e2e 注释同步。S-5：409 信封复用推广到 projectMembers/freeformDoc/taskRuns/chatSessions（复用 projectSettingsError 解析器） | 隔离 PG 测试报告归档 `.doc/test-reports/`；`playwright test --list` 输出与门禁矩阵一致；409 负例单测覆盖 4 个 API 封装 | @@ -38,15 +38,15 @@ updated: 2026-09-17 ## 验收标准 -- [ ] Wave 0–2 完成后：`go vet ./...`、`go test -short ./...`、`cd web && npm test && npm run build` 全绿；dev 容器 rebuild + health/ready PASS +- [x] Wave 0 完成后：`go vet ./...`、`go test -short ./...`、`cd web && npm test && npm run build` 全绿；dev 容器 rebuild + health/ready PASS（[报告](../test-reports/2026-09-18-wave0-p0-bug-fixes-test-report.md)） - [ ] main 环境：ENV 语义统一核验记录 + schema 49 + 事件链路无双写证据（Wave 1） -- [ ] SSE 长回合（>120s）真实 PI 复测不断流（Wave 0） +- [x] SSE 长回合心跳接线：真 PI 静默期 `: heartbeat` + 短 WriteTimeout 单测跨越（Wave 0） - [ ] `TestReconcile_LegacyRepair_PG` 报告归档，gap-matrix 测试债行关闭（Wave 3） - [ ] `playwright test --list` 与门禁文档一致（Wave 3） - [ ] 跨用户会话挂靠负例测试入库（Wave 4） - [ ] X-1~X-5 每项有决策记录（Wave 5） - [ ] D-1~D-6 全部回链关闭（Wave 6） -- [ ] 不触碰红线：FAV_DEGRADED_READ=false、MULTI_INTENT_PLAN_TIER 默认 legacy、SEMANTIC_SUMMARY_REPLACE 不接线、项目 143 只读、禁止整支 merge FAV +- [x] 不触碰红线：FAV_DEGRADED_READ=false、MULTI_INTENT_PLAN_TIER 默认 legacy、SEMANTIC_SUMMARY_REPLACE 不接线、项目 143 只读、禁止整支 merge FAV ## 相关链接 diff --git a/.doc/test-reports/2026-09-18-wave0-p0-bug-fixes-test-report.md b/.doc/test-reports/2026-09-18-wave0-p0-bug-fixes-test-report.md new file mode 100644 index 00000000..5de2a89f --- /dev/null +++ b/.doc/test-reports/2026-09-18-wave0-p0-bug-fixes-test-report.md @@ -0,0 +1,135 @@ +# Wave 0 P0 Bug 修复测试报告 + +## 一、 测试背景 (Background) + +系统体检（2026-09-17）确认 SSE 聊天/任务流被全局 `WriteTimeout=120s` 强杀、`run/stream` 畸形 body 误启任务与 Flush 后失败、前端会话窗口缓存 key 不一致、IndexedDB 恢复覆盖新增交付物。本波次按整改方案 Wave 0 实施 B-1~B-6。 + +## 二、 测试目标 (Test Objectives) + +- B-1/B-2：`handleChatStream` / `handleRunTaskStream` 心跳 + `extendSSEWriteDeadline`，长静默不因 WriteTimeout 断流 +- B-3：畸形 JSON → HTTP 400，不开 SSE +- B-4：`CreateAndStart`/`Subscribe` 在 Flush 前；失败为普通 JSON 错误 +- B-5：`saveSessionWindowCache` 一律带 `userId`；LRU 驱逐完整 cache key +- B-6：`applyRestoredFileRaw` 函数式合并，不丢 mount 后新增 deliverable +- 质量门禁：`go vet` / `go test -short` / `npm test` / `npm run build` 全绿；dev rebuild + health/ready + +## 三、 测试方案与环境 (Test Plan & Environment) + +- OS：Linux；Go 模块全量 `-short`；前端 `tsx --test` + vitest +- 后端 Stub 单测 + 真实 dev 容器（`pi-agent` + MiniMax-M2.7-highspeed） +- 分支：`fix/wave0-p0-sse-chat-cache-bugs` → 合入本地 `dev@810fb96`（后随 docs 再合） +- 资产：`.doc/test-reports/assets/2026-09-18-wave0-p0-bug-fixes/` + +## 四、 预期结果 (Expected Results) + +| 用例 | 预期 | +|------|------| +| 畸形 run/stream body | 400，非 `text/event-stream` | +| CreateAndStart 注入失败 | 非 SSE Content-Type，正文含 start failure | +| 短 WriteTimeout + 心跳 | 流时长 > WriteTimeout 仍可读到 done | +| 缓存写路径 | 全部调用含 `userId`；超 LRU 驱逐带前缀 key | +| restore 合并 | prev 新增条目保留，仅补丁 fileRaw | +| 真实 PI 聊天 | 静默生成期出现 `: heartbeat`，最终 `event: done` | + +## 五、 实际测试结果与判定 (Actual Results & Evaluation) + +### 5.1 后端单测（`internal/server/sse_wave0_test.go`） + +**测试输入**：`go test -short -count=1 ./internal/server/ -run 'TestRunTaskStream_InvalidJSON|TestRunTaskStream_CreateAndStartFails|TestSSEHeartbeat_Helpers|TestRunTaskStream_ValidBody' -v` + +**实际输出（摘要）**： + +``` +--- PASS: TestRunTaskStream_InvalidJSON_Returns400 (0.14s) +--- PASS: TestRunTaskStream_CreateAndStartFails_BeforeSSEFlush (0.12s) +--- PASS: TestSSEHeartbeat_HelpersSurviveShortWriteTimeout (0.70s) +--- PASS: TestRunTaskStream_ValidBody_OpensSSEWithHeartbeatPath (0.17s) +PASS +``` + +**评价判定**：PASS +**性能与耗时**：约 1.1s 包级 + +### 5.2 前端单测 + +**测试输入**： + +``` +npx tsx --test src/hooks/useChatWindow.cacheKey.test.ts \ + src/data/storage.windowCache.test.ts src/data/fileStore.test.ts +``` + +**实际输出**： + +``` +✔ applyRestoredFileRaw: 以 prev 为权威… +✔ applyRestoredFileRaw: 无可用 fileRaw 时返回同一引用 +✔ window cache LRU: 驱逐带 userId 前缀的完整 cache key… +✔ window cache: 读写同一 userId 前缀，跨用户不串读 +✔ useChatWindow: 所有 saveSessionWindowCache 调用必须传入 userId +ℹ tests 5 / pass 5 +``` + +**评价判定**：PASS + +### 5.3 全量回归 + +| 命令 | 结果 | +|------|------| +| `go vet ./...` | PASS | +| `go test -short -count=1 ./...` | 全包 ok（含 server） | +| `cd web && npm test` | 397 + 32 vitest PASS | +| `cd web && npm run build` | PASS（含 precompress） | + +### 5.4 dev 容器冒烟 + +**测试输入**：`./scripts/env-service.sh dev rebuild`；`curl -sf http://127.0.0.1:8080/health` 与 `/ready` + +**实际输出**： + +```json +{"status":"ok","env":"dev","agent_backend":"pi-agent","schema_version":52,"schema_ready":true,...} +``` + +**评价判定**：PASS + +### 5.5 真实 PI 聊天 SSE（心跳接线证据） + +**测试输入**：admin 登录后 `POST /api/v1/chat/stream`，长合规分析提示词；资产见 `assets/2026-09-18-wave0-p0-bug-fixes/chat-stream-long.sse.log` + +**实际输出（关键片段）**： + +``` +event: phase +... +: heartbeat 1789663534 +: heartbeat 1789663549 +: heartbeat 1789663564 +: heartbeat 1789663579 +event: delta +... +event: done +data: {...,"done":{"duration_ms":70164,"status":"completed",...}} +``` + +**评价判定**：PASS（真 PI 回合约 70s；静默生成期 4 次心跳，流完整收尾）。 +**说明**：绝对墙钟 >120s 的真 PI 未再压测（二次加长提示触发 `SECURITY_FORBIDDEN` 快速返回）；WriteTimeout 跨越由 `TestSSEHeartbeat_HelpersSurviveShortWriteTimeout`（WriteTimeout=200ms，流 700ms）覆盖，与线上心跳机制同源。 + +## 六、 自动化测试套件汇总 (Test Suite Execution) + +- Go：`go vet ./...` + `go test -short -count=1 ./...` → 全绿 +- Web：`npm test`（397 node:test + 32 vitest）+ `npm run build` → 全绿 +- 新增：`sse_wave0_test.go`、`useChatWindow.cacheKey.test.ts`、`storage.windowCache.test.ts`、`fileStore.test.ts` + +## 七、 测试结论与后续优化建议 (Conclusion & Next Steps) + +- Wave 0（B-1~B-6）达到合入与冒烟标准，可关闭 short-term 任务。 +- 后续：Wave 1（E-1 main ENV）仍待决策；可选补一次刻意 >120s 的受控长任务（例如阻塞 executor 的 task run events）作为额外证据。 +- 红线未触碰：FAV / MULTI_INTENT_PLAN_TIER / SEMANTIC_SUMMARY_REPLACE / 项目 143。 + +## 相关链接 + +- Audit: [../audit/2026-09-17-system-health-check.md](../audit/2026-09-17-system-health-check.md) +- Remediation: [../remediation/2026-09-17-system-health-remediation-plan.md](../remediation/2026-09-17-system-health-remediation-plan.md) +- Short-term: [../ops/short-term/2026-09-18-wave0-p0-bug-fixes.md](../ops/short-term/2026-09-18-wave0-p0-bug-fixes.md) +- Assets: [./assets/2026-09-18-wave0-p0-bug-fixes/](./assets/2026-09-18-wave0-p0-bug-fixes/) diff --git a/.doc/test-reports/assets/2026-09-18-wave0-p0-bug-fixes/chat-stream-long.meta.json b/.doc/test-reports/assets/2026-09-18-wave0-p0-bug-fixes/chat-stream-long.meta.json new file mode 100644 index 00000000..7dad5b42 --- /dev/null +++ b/.doc/test-reports/assets/2026-09-18-wave0-p0-bug-fixes/chat-stream-long.meta.json @@ -0,0 +1,12 @@ +{ + "session_prefix": "wave0-sse-", + "duration_ms": 70164, + "heartbeats": 4, + "heartbeat_format": ": heartbeat ", + "done": true, + "error_event": false, + "agent_backend": "pi-agent", + "model": "MiniMax-M2.7-highspeed", + "log": "chat-stream-long.sse.log", + "note": "Silent PI generation interval produced 4 heartbeats; stream completed with event:done. Absolute wall-clock >120s covered by TestSSEHeartbeat_HelpersSurviveShortWriteTimeout (same helpers)." +}