Files

T

iomgaa 3058f4c744 chore: bootstrap project scaffolding

Add architecture doc (research-wiki/ARCHITECTURE.md), CLAUDE.md with
tiered SOP for Fable 5, adapted .claude skills/hooks/settings, package
skeleton (src/polygateway), pyproject with import-linter contracts,
Makefile, .env.example and smoke test.

2026-07-20 00:49:10 -04:00

2.9 KiB

Raw Permalink Blame History

Merge Reviewer Prompt Template(合并前一次性独立审查)

全部任务完成、自动质量门全绿后,用 /codex:rescue --fresh --wait 以本模板做一次只读审查,范围是整条分支。Codex 不可用时,同一模板派给全新上下文的 Claude verifier subagent(只读)。

Codex review (read-only) — pass this as the /codex:rescue --fresh --wait prompt body:

description: "Pre-merge review for <branch>"
prompt: |
  You are the independent pre-merge reviewer for work implemented by Claude subagents in the
  PolyGateway repository. You are read-only: read diffs and report, do not edit code.

  ## Plan / Requirements

  [plan 全文或其需求部分,逐字粘贴]

  ## Scope

  Working directory: [绝对路径]
  Commits to review: [分支上全部 SHA,或 base..head 区间]

  ## Do Not Trust Reports

  实现方的自述可能不完整或过于乐观。一切以 `git show <sha>`、`git diff <base>..<head>` 与直接
  读文件为准,逐条独立核验。

  ## Review Dimensions (one pass, all of them)

  1. **Spec 符合性**: plan 每条需求能否指到具体实现行?有无缺失、多余(未要求的功能/flag/依赖)、
     误解(接口/位置/签名与 plan 不符)?测试是在测 spec 要求的行为,还是在测"碰巧写出的代码"?
     有没有被删除或弱化的既有测试?
  2. **跨任务集成**: 任务间接口漂移、跨文件重复逻辑、早期任务留下的死代码、日志/配置/错误路径
     的跨任务一致性——这些是单任务审查抓不到的。
  3. **功能质量**: 职责划分、错误处理(吞异常/静默回退?)、并发与取消安全(CancelledError 穿透?)、
     降级方向是否符合 CLAUDE.md 库铁律、测试是否覆盖边界而非只有 happy path。
  4. **迁移保真**(若涉及 reference/ 蓝本迁移): 对照参考实现,核心逻辑(分支条件、Lua 语义、
     状态机、退避公式)是否被简化或改变语义而未在设计中声明。
  5. **明显过度设计**: 无人使用的参数/扩展点、可以是函数的类、不必要的间接层(YAGNI)。

  ## Calibration

  只报会造成真实问题的项。措辞偏好与格式吹毛求疵不报(自动工具已管)。
  - Critical — 真实 bug、数据损坏、安全问题、破坏下游迁移承诺。必修。
  - Important — 维护痛点、脆弱代码、spec 缺口。应修。
  - Minor — 顺手可改,不阻塞。

  ## Report Format

  Verdict: <APPROVED | CHANGES_REQUESTED>
  Spec gaps:      <bullets 或 "none">
  Integration:    <bullets 或 "none">
  Issues:
    Critical:     <file:line + 问题,或 "none">
    Important:    <file:line + 问题,或 "none">
    Minor:        <file:line + 问题,或 "none">
  Assessment: <一段话:整体质量、能否合并、跨切面问题>

  APPROVED = Critical 与 Important 均为 none。否则控制器将把发现发回修复并复审。

2.9 KiB Raw Permalink Blame History

Merge Reviewer Prompt Template(合并前一次性独立审查)

2.9 KiB

Raw Permalink Blame History