已开启
docs: #328 统一「集群模式/团队模式」术语表述 #8
grape_bot创建于 2 天前
docs: #328 统一「集群模式/团队模式」术语表述 #8
已开启
合并受阻
2 天前 关联了issue：[文档] README 与快速部署指南中“集群模式/团队模式”表述不一致
grape_bot
2 天前 评论：
2 天前 评论：
[QA Agent] PR #8 审查报告
关联 Issue
- Closes #328 — 状态: open（triaged / documentation / spec-ready）
- 原始需求：统一 README 与《AI 面试天团 — 快速部署指南》中「集群模式/团队模式」术语表述，以「团队模式」为准，补充对应关系说明并指向部署指南第 9 节。
变更摘要
- 共 15 个文件变更，+2527 -1
- 业务文档（与 Issue 直接相关）：
README.md（1+1-）、AI 面试天团 — 快速部署指南.md（3+0-） - 开发记录：
docs/issue-328/{spec,plan,task}.md（3 文件） - ⚠️ 无关新增：
.skills/coding-cli/共 11 文件约 2523 行（SKILL.md + 10 个 references/*.md），PR body 与 Issue 均未描述该部分。
代码审查（5 轴：正确性/可读性/架构/安全/性能）
结论: ✅ 文档变更通过 / ⚠️ 范围越界需处理
文档变更（与 Issue #328 直接相关）— ✅ 通过
| 级别 | 文件 | 行号 | 问题 | 建议 |
|---|---|---|---|---|
| ✅ 通过 | README.md | 80 | 术语「集群模式」→「团队模式」，并追加指向部署指南第 9 节的链接（<AI 面试天团 — 快速部署指南.md#9-启动与运行>，尖括号包裹含空格中文路径） |
符合 Issue 验收标准，markdown 链接格式正确 |
| ✅ 通过 | AI 面试天团 — 快速部署指南.md | 489-491 | 第 9 节步骤 1 后补充「团队模式即早期文档中的集群模式」对应关系说明 | 措辞清晰，与 WorkSwarm 实际界面名称一致 |
- 验收核对（已用 grep 在 PR head 分支核对）：
- README.md 全文「集群模式」= 0 处（已消除）✅
- 部署指南「集群模式」仅在第 491 行对应关系说明处保留（符合预期）✅
- 两文档「团队模式」表述一致 ✅
⚠️ 范围越界（Required 级别）
| 级别 | 文件 | 行号 | 问题 | 建议 |
|---|---|---|---|---|
| Required | .skills/coding-cli/（11 文件 ~2523 行） | 全部 | PR body 仅描述「统一术语 + 补充链接 + 新增 docs/issue-328/」，但实际提交了完整的 coding-cli Skill 定义（SKILL.md + code-review/coding-runbook/grape-cli-*/incremental/new-dev-flow/planning/pr-optimization/spec-driven/test-driven 等 10 个 references）。该部分与 Issue #328（纯文档术语统一）无关，属于夹带提交（drive-by addition）。 | 二选一：(a) 将 .skills/coding-cli/ 拆分到独立 PR，并补 Issue/PR body 说明其用途与验收；(b) 若确需随本 PR 合入，必须在 PR body 补充该部分的改动说明、来源与必要性，并经维护者确认。当前 PR 不应在无说明下混入 2500+ 行无关代码。 |
Skill 审查（针对新增 .skills/coding-cli/SKILL.md，按 skill-review 契约速查）
- 格式规范：frontmatter 含 name/version/description/tools，name(coding-cli) 与目录名一致 ✅
- 完整性：含安全约束/Prerequisites/环境变量/处理流程/操作边界/硬约束/参考章节 ✅
- 引用路径：references/*.md 在本 PR 中已一并新增，路径自洽 ✅
- ⚠️ 仅作格式层面速查，不对其内容做实质评分（与本 Issue 无关，应独立 PR 评审）。
工作流审查（Agent Work Loop 五维）
结论: ⚠️ 需改进
| 维度 | 证据状态 | 严重度 | 发现 | 影响 | 修复 | 验收 |
|---|---|---|---|---|---|---|
| 任务理解 | observed | Required | Issue 明确为「统一术语」纯文档改动，PR 却夹带 coding-cli Skill 全量新增（11 文件），PR body 未说明 | 范围越界，审查/合并风险扩大，追溯困难 | 拆分独立 PR 或在 PR body 补说明并经确认 | 范围与 Issue/PR body 一致 |
| 受控执行 | observed | Info | docs/issue-328/ 下 spec/plan/task 齐备，task.md 自查项 1-5 已勾选 | 流程规范 ✅ | 无 | — |
| 变更验证 | observed | Info | 纯文档改动，PR body 声明 grep 全文检索验收；QA 已在 head 分支复核 grep 结果一致 | 文档验收可信 ✅ | 无 | — |
| 可靠交付 | unobserved | — | 仓库未见 AGENTS.md / .github/workflows（CI 门禁未观察到），无法评估 CI 状态 | 证据不足，不评分 | — | — |
| 经验沉淀 | unobserved | — | 未见本次变更相关的经验记录更新 | 不评分 | — | — |
安全扫描
结论: ✅ 安全
- 文档与 Skill 文件均为静态内容，无密钥/凭据/外部 URL 注入风险。
.skills/coding-cli/SKILL.md明确约束「只用注入的 $GRAPE_SESSION_TOKEN，不探测其它 token」，安全姿态正确。
合并检查
- ✅ 冲突检查通过（mergeable=true，conflict_passed=true）
- ✅ 文档术语变更符合 Issue #328 全部验收标准
- ⚠️ 范围越界（.skills/coding-cli/ 11 文件）未处理前不建议直接合入
- ⚠️ CI 门禁状态未观察到（仓库无 .github/workflows）
建议下一步
- 处理
.skills/coding-cli/范围越界问题（拆分独立 PR 或补 PR body 说明并经维护者确认）。 - 文档变更本身已通过审查，可保留。
- 处理完成后请重新触发 QA 审查。
概述
实现 Issue #328：统一 README 与《AI 面试天团 — 快速部署指南》中对同一操作入口的术语表述，以「团队模式」为准。
改动
测试