chore: remove multi-review action

[Svtter]

Summary

• Remove multi-review/ directory entirely (action.yml, run-multi-review.py, reviewers/, README.md)
• The opencode github run subprocess approach has a fundamental flaw: stdout contains ONLY CLI boilerplate (permissions, session info, tool calls), never the actual AI review content
• After multiple failed iterations (PR fix(multi-review): improve comment noise filtering #87, fix(multi-review): isolate reviewers in git worktrees #90, feat(multi-review): single comment output + remove standalone reviews #98), we're removing this module to prepare for a replacement approach

Context

See the discussion issue for detailed analysis of why this approach failed and what alternatives we're considering.

🤖 Generated with Claude Code

[github-actions]

有条件合并

总结

PR 删除了 multi-review/ 目录（7 个文件，1185 行），但未清理代码库中对该模块的引用，导致合并后测试和 CI 流程会被破坏。

阻塞项

1. tests/test_multi_review.py — 该测试文件通过 importlib.util.spec_from_file_location 在 multi-review/run-multi-review.py 路径加载模块（test_multi_review.py:12），删除目录后该文件会引发 ModuleNotFoundError，所有测试均无法通过。

2. .github/workflows/multi-review.yml:28 — Workflow 使用 uses: ./multi-review 引用被删除的本地 action，合并后该 workflow 会在 "Run multi-agent review" 步骤失败（action 不存在）。

两个问题都会导致 CI pipeline 直接失败，属于 不可合并 级别的阻塞，需至少修复这两处才能合并。

建议项

1. README.md — 第 34, 105, 117, 127, 180, 236 行多处引用 multi-review action 的使用方式和文档，建议同步清理，避免给用户造成困惑。

2. skills/setup-ci/SKILL.md:22,33-58 及 skills/setup-ci/references/actions-reference.md:30,190,252 — 技能模板和参考文档中包含了生成 multi-review workflow 的模板和权限说明，建议统一移除或标记为已废弃。

opencode session  |  github run

[github-actions]

分析了所有相关引用后，以下是审计结果：

发现遗漏

该 PR 删除了 multi-review/ 目录的 7 个文件，与标题和描述相符。但遗留了多处引用，导致功能不完整。

CRITICAL

1. 测试文件未清理 — tests/test_multi_review.py 在第 12 行通过 importlib.util.spec_from_file_location 直接导入 multi-review/run-multi-review.py。PR 合并后，任何测试运行（例如 pytest tests/）都会因 ModuleNotFoundError 而失败。建议： 删除 tests/test_multi_review.py。

2. CI 工作流引用已删除的 action — .github/workflows/multi-review.yml 第 28 行使用 uses: ./multi-review 指向已删除的目录。合并后该 workflow 将无法解析 action，导致 CI 失败。建议： 删除 .github/workflows/multi-review.yml。

MEDIUM

3. skill 文档指导用户使用已删除的 action — skills/setup-ci/SKILL.md 第 22、33、43、58 行指导用户生成 .github/workflows/opencode-multi-review.yml 并引用 Svtter/opencode-actions/multi-review@v2。用户按此操作会得到损坏的 workflow。建议： 更新 SKILL.md，移除或替换 multi-review 相关内容。

4. skill 参考资料仍包含 multi-review — skills/setup-ci/references/actions-reference.md 第 30、190、252 行包含 multi-review 的 inputs、outputs、权限说明。建议： 删除或更新这些引用。

LOW

5. README.md 多处宣传已删除的特性 — 第 34 行（特性列表）、第 105–132 行（独立章节）、第 180 行（对比表格）、第 236 行（使用示例）均展示了 multi-review action 作为可用功能。用户阅读后会混淆。建议： 清理 README.md 中所有 multi-review 引用。

6. openspec 设计文档变成孤儿文件 — openspec/changes/multi-agent-review/ 目录下的设计文档、proposal、spec、tasks.md 描述了已删除的特性。虽然作为历史记录可能有意保留，但标记为 "完成" 的 tasks 对应代码已不存在。建议： 可选归档或删除该 spec 目录。

opencode session  |  github run

[github-actions]

架构有问题

架构分析摘要

PR 删除了 multi-review/ 目录（7 文件/1185 行），但遗留了大量指向已删除代码的引用，导致架构完整性受损。从结构上看，这是一种"反向散弹式修改"——删除核心模块时未同步清理其消费者。

阻塞项

1. .github/workflows/multi-review.yml:28 — Workflow 使用 uses: ./multi-review 引用即将被删除的本地 action，合并后该 workflow 在 Run multi-agent review 步骤必然失败（action 目录不存在）。这是 CI pipeline 的直接断裂。

2. tests/test_multi_review.py:12 — 测试文件通过 importlib.util.spec_from_file_location 硬编码导入 multi-review/run-multi-review.py（第 12 行），模块删除后该文件在导入阶段即抛出 ModuleNotFoundError，导致整个测试套件（pytest tests/）无法通过。测试是架构契约的一部分，删除实现后保留对实现的直接引用=架构断裂。

这两个问题都导致可执行路径（CI、测试）直接失败，属于不可合并级别。

建议项

1. README.md（第 34, 105, 117, 127, 180, 236 行） — 多处将 multi-review 列为可用 action 并提供使用示例。建议统一清理，避免误导用户。

2. skills/setup-ci/SKILL.md:22,33-58 及 skills/setup-ci/references/actions-reference.md:30,190,252 — 技能模板和参考文档中包含了为项目生成 multi-review workflow 的模板。用户按此指引操作会得到损坏的配置。建议移除或标记为废弃。

3. openspec/changes/multi-agent-review/ — 设计文档描述了已删除特性的完整 spec 和 tasks。作为历史记录可保留，但建议在 README 或目录入口注明该特性已移除。

opencode session  |  github run

[github-actions]

可合并

总结

PR 删除了 multi-review/ 目录及相关引用，共计 12 文件 / 1485 行删除。此前评论指出的遗留引用问题（workflow、测试、README、skill 文档）在 PR 当前 HEAD 中均已清理完毕。

阻塞项：无

此前评论提到的问题均已在当前 HEAD 修复：

• ✅ .github/workflows/multi-review.yml 已删除（不再有损坏的 workflow）
• ✅ tests/test_multi_review.py 已删除（不再有导入错误）
• ✅ README.md 中 multi-review 引用已移除（-32 行）
• ✅ skills/setup-ci/SKILL.md 中 multi-review 模板已移除（-35 行）
• ✅ skills/setup-ci/references/actions-reference.md 中引用已移除（-36 行）

建议项

• openspec/changes/multi-agent-review/ 目录下的设计文档成为孤儿文件。作为历史记录可以保留，但建议在目录入口处注明该特性已移除，避免后续读者困惑。

opencode session  |  github run

[github-actions]

架构合理

PR 是一个干净的模块删除操作：删除了 multi-review/ 目录及其直接附属文件（CI workflow、测试、register action），同时清理了 README 和 skill 文档中的全部外部引用。搜索确认代码库中除 openspec/changes/multi-agent-review/ 外不存在任何对 multi-review 的悬空引用。

阻塞项：无

建议项：

• openspec/changes/multi-agent-review/ 目录包含已删除特性的设计文档、proposal 和 tasks。不影响代码执行，但作为历史文物建议在目录入口注明该特性已移除，避免后续读者困惑。

opencode session  |  github run

[github-actions]

Now I have all the information needed. Let me analyze the PR against its stated requirements.

PR Spec (from title + body):

• Remove multi-review/ directory entirely (action.yml, run-multi-review.py, reviewers/, README.md)

PR Implementation:

• ✅ Deletes multi-review/ directory (7 files)
• ✅ Deletes tests/test_multi_review.py
• ✅ Deletes .github/workflows/multi-review.yml
• ✅ Removes multi-review from README.md
• ✅ Removes multi-review from skills/setup-ci/SKILL.md
• ✅ Removes multi-review from skills/setup-ci/references/actions-reference.md

遗留以下文件未被清理：

---

发现遗漏

该 PR 的删除范围与描述基本一致，但遗留了一个 openspec 设计文档目录未被清理。

MEDIUM

1. openspec/changes/multi-agent-review/ 设计文档成为僵尸文件 — 该目录包含 design.md（详细描述 multi-review 设计目标、决策、架构）、proposal.md（提议新增 multi-review action）、tasks.md（含未勾选的任务如"1.1 创建 multi-review/ action 目录结构"）、specs/multi-agent-pipeline/spec.md 和 specs/reviewer-personas/spec.md。这些文档描述的功能代码已被完全删除，但 openspec 目录本身未做任何处理。任务列表中还有未完成的条目指向已删除的代码，会造成维护者困惑。建议： 删除 openspec/changes/multi-agent-review/ 目录，或至少在其中添加 README 说明该变更已废弃并移除。

opencode session  |  github run