№ · 任务清单 CH.07.13
主页 · 📃 任务清单 · 2026-07-13-w29-code-quality-owner

2026-07-13-w29-code-quality-owner

📅 2026-07-13 · 📦 4.7 KB · 📃 任务清单 · 📝 MD 文档 🆕 14d 🏷️ 计划🏷️ 回归🏷️ Owner🏷️ 品质

⟦tag:v2|task|2026-07-13-w29-code-quality-owner⟧ kind: task id: 2026-07-13-w29-code-quality-owner lane: quality-owner title: W29 代码质量长工检查与有界优化(8-12h) objective: 核查 W29 五日计划已落代码的真实质量，在不改变玩法、数值、UI 和存档语义的前提下，修复 1-3 个证据充分的高价值质量根因，并形成可复核的质量审计报告。 scope: AGENTS.md,docs/plans/active/2026-07-13-5日开发计划-v1.md,W29 R7-R14 + T3 已落代码,当前 68 条 tsc --noEmit 基线,三层依赖边界,相关回归测试。 acceptance:

  • 新增 docs/reviews/2026-07-13-w29-code-quality-audit-v1.md,记录完成度、68 条错误分组、架构违规、测试缺口、已修复项和残余风险
  • 开工先冻结基线:cmd /c npm run build,cmd /c npm run lint,W29/T3 定向 tests,git status --short
  • 将 68 条 lint 错误按根因和领域聚类,区分 W29 新增 / 既有基线 / tools 子项目 / UI 活跃 lane
  • 扫描三层边界:data→utils/components,utils→components,数据层函数,逻辑层原始配置,UI 业务计算
  • 只实施 1-3 个可证明无行为变化的 P0/P1 修复;若没有安全修复,允许只交审计报告,禁止为“有改动”制造重构
  • cmd /c npm run build 必须通过;lint 总数不得高于 68,且长工触碰文件不得新增错误
  • 相关定向 tests 全过;改动文件 git diff --check 通过
  • 回执必须给出 before/after 错误数、修复根因、实际文件、验证证据和下一轮 shortlist worker threadId: 019f5a22-01c2-7f92-af6b-5c660d2cde98 current state: verified / callback absorbed blockers: none for audit。R10-followup / doc-sync / R11 尚未收口,因此长工实施阶段必须避开其文件锁。 next action: none；审计报告与两项有界优化已落,后续质量 lane 由新执行长工承接。

callback absorbed:

  • worker thread:019f5a22-01c2-7f92-af6b-5c660d2cde98
  • report:docs/reviews/2026-07-13-w29-code-quality-audit-v1.md
  • verification:build passed,test:w29 39/39,lint 68 -> 68,targeted diff-check passed

prompt(给长工 / 子会话的指令)

角色:[长工]#代码质量-W29收口审计优化@v1.0 单一 Goal:让 W29 已落代码更可验证、更符合三层架构,同时不改变任何玩家可见行为或业务数值。

已确认真值

  • cmd /c npm run build 当前通过。
  • cmd /c npm run lint 当前有 68 条既有错误。
  • R10 仍为 partial / accepted residual risk;真实浏览器截图与 P1 视觉签字缺失。
  • T3 定向测试 5/5 passed;R13 5/5 passed;R14 + R9 11/11 passed。
  • 工作树很脏,所有无关改动都属于用户/其他 lane,不得回退或重写。

Phase A · 质量审计(先做)

  1. 依次读 AGENTS.md、本卡、五日计划、R7-R14/T3 task 卡。
  2. 冻结 build/lint/test/git-status 基线。
  3. 输出 docs/reviews/2026-07-13-w29-code-quality-audit-v1.md:
    • 五日计划代码完成度矩阵
    • 68 条 lint 错误按根因/领域分组
    • 三层依赖违规清单
    • 测试可信度与缺口
    • P0/P1/P2 修复 shortlist,每项含文件边界、风险、验收、回退
  4. 第一检查点先把 shortlist 发回主会话,但无需等待日历门；若某项满足下方实施门,可继续 Phase B。

Phase B · 有界优化

只允许实施同时满足以下条件的 1-3 项:

  • 直接对应当前 lint/build/test/依赖扫描证据。
  • 不改玩法规则、经济数值、剧情内容、玩家文案、UI 布局、存档结构。
  • 不触碰活跃/未收口 lane 的文件锁。
  • 单项最多 8 个文件；若需要跨层搬迁或超过 8 文件,只写建议,不实施。

优先允许目录:

  • src/data/items/**,src/utils/items/**
  • src/data/endgame/**,src/utils/endgame/**
  • src/data/storyEvents/oldLedger.ts,src/data/eventTables/storyline-old-ledger.ts
  • src/data/multiplayer/**,src/utils/multiplayer/**(仅非 UI、非 R10 入口)

强制禁区:

  • src/App.tsx
  • src/components/**
  • src/data/auction/**,src/utils/auctionRules.ts,src/components/auction/**
  • tools/剧情编辑器/**
  • docs/handoff/index.md,docs/handoff/current-dispatch-shortlist.md
  • R10 视觉 checklist、R11 收口文档、产品数值真值文档

验收与回执

  • build 通过。
  • lint after <= 68,并明确 delta。
  • 所有触碰领域定向 tests 全过。
  • targeted diff-check 通过。
  • 回执格式:
[长工 #代码质量-W29收口审计优化@v1.0 回执]
- 状态:完成 / 部分完成 / 阻塞
- 审计报告:<路径>
- 五日计划完成度:<完成 / 部分 / 未执行矩阵>
- lint before/after:<68 -> N>
- 已修复根因:<1-3 项>
- 修改文件:<实际路径>
- build/tests/diff-check:<证据>
- 未修复风险:<按 P0/P1/P2>
- 下一轮 shortlist:<最多 3 项>