Cc update review
Skill lane2077/claude-code-harness-zh/skills/cc-update-review
用于 CC 更新集成的质量护栏。会检测只改 Feature Table 文档却没有落实现的情况,并强制要求给出实现方案。仅供内部使用。From its SKILL.md
npx -y skills add lane2077/claude-code-harness-zh --skill cc-update-reviewAssembled from the repository path, not quoted from the project. Check it against their README if it does not work.
One thing to look at
- 1 stars1 stars. Stars are a popularity signal and not a quality one, but at this level it is likely that nobody has read this closely except its author, and you would be relying on your own review.
SKILL.md
7.3 KB, as published. Nobody here has run it
CC Update Review 护栏
这是一个用于 Claude Code 更新集成时的质量护栏,用来防止“只把内容写进 Feature Table 却没有真正落地实现”。 它会自动判断 Feature Table 的新增条目是否伴随了实际实现;如果没有,就强制输出实现方案。
Quick Reference
这个技能会在以下情况触发:
- 在评审 CC 更新集成 PR 时
- 当检测到 Feature Table(
CLAUDE.md/docs/CLAUDE-feature-table.md)新增了新条目时 - 当
/harness-review判断当前 PR 属于 CC 集成 PR 时,作为内部调用触发
以下情况 不会 触发:
- 普通实现工作(
/work) - 只改了 Feature Table 之外的内容
- setup / 初始化类工作
3 类分类
Feature Table 中新增的每一项,都要被归到以下 3 类之一。
(A) 已有实现
定义:与 Feature Table 新增条目对应的 hooks / scripts / agents / skills / core 实现改动,已经包含在同一个 PR 中。
判定条件:
- Feature Table 中提到的功能,确实对应到相关文件改动
- hooks.json、技能 SKILL.md、agent
.md、scripts/.sh、core/src/.ts 中至少有一处 diff
例:
| Feature Table 新增项 | 对应实现修改 | 判定 |
|---|---|---|
PostCompact hook | 新增 hooks/post-compact-handler.sh | A |
MCP Elicitation 支持 | 在 hooks.json 中加入 Elicitation 事件,并新增 elicitation-handler.sh | A |
Worker maxTurns 限制 | 在 agents-v3/worker.md 中加入 maxTurns 字段 | A |
结果:OK,不需要额外动作。
(B) 只写了文档
定义:只在 Feature Table 中新增了一行,但 Harness 侧完全没有任何实现改动;同时它也不属于 CC 自动继承(类别 C)。
判定条件:
- Feature Table 中出现了新条目
- 同一个 PR 里没有任何与 hooks / scripts / agents / skills / core 相关的改动
- 该功能本应由 Harness 提供独特附加价值(例如配置、工作流集成、guardrail 等)
例:
| Feature Table 新增项 | 对应实现修改 | 判定 |
|---|---|---|
PreCompact hook | 无(只有 Feature Table 变更) | B |
Agent Teams | 无(只有 Feature Table 变更) | B |
Desktop Scheduled Tasks | 无(只有 Feature Table 变更) | B |
结果:NG。应阻止 PR,并要求给出实现方案。输出格式见下文。
(C) CC 自动继承
定义:属于 Claude Code 本体的性能改进、bug 修复或内部优化,不需要 Harness 侧额外修改。
判定条件:
- 这是 CC 本体的修复,Harness 没有额外包装或扩展的空间
- 属于性能改进、内存泄漏修复、UI 改善等类型
- 不会影响 Harness 的工作流
例:
| Feature Table 新增项 | 理由 | 判定 |
|---|---|---|
Streaming API memory leak fix | CC 内部的内存泄漏修复,Harness 无需改动 | C |
Compaction image retention | CC 在压缩上下文时保留图片,Harness 无需改动 | C |
Parallel tool call fix | CC 内部的并行执行修复,Harness 会自动受益 | C |
结果:OK。但要在 Feature Table 的列里明确标注“CC 自动继承”。
CC 更新 PR 检查清单
在评审 PR 时,按顺序检查以下内容:
## CC 更新集成检查清单
### 1. 提取 Feature Table 差异
- [ ] 从 `CLAUDE.md` 或 `docs/CLAUDE-feature-table.md` 的 diff 中列出新增条目
### 2. 对各项做分类
- [ ] 把每一条新增项判定为 A / B / C 之一
- [ ] 确认类别 B 的条目数为 0
### 3. 按类别逐项确认
- [ ] (A) 已有实现:对应实现文件是否已正确链接
- [ ] (B) 只写了文档:是否已经给出实现方案(如果不是 0 件,就应阻止 PR)
- [ ] (C) CC 自动继承:Feature Table 中是否明确标注了“CC 自动继承”
### 4. 检查 CHANGELOG
- [ ] 类别 A 的条目是否在 CHANGELOG 中按“之前 / 现在”格式记录
- [ ] 类别 C 的条目是否在 CHANGELOG 中按“CC 自动继承”方式记录
### 分类结果
| # | Feature Table 条目 | 类别 | 对应文件 / 备注 |
|---|-------------------|---------|-------------------|
| 1 | (条目名) | A / B / C | (文件路径或备注) |
| 2 | (条目名) | A / B / C | (文件路径或备注) |
检测到类别 B 时的输出格式
如果检测到 1 件以上的类别 B 条目,就必须按以下格式输出实现方案。 这个输出格式是强制的,不允许省略。
## 检测到类别 B:实现方案
### B-{编号}. {Feature Table 条目名}
**现状**:只写进了 Feature Table,Harness 侧没有实际实现。
**Harness 独有的附加价值**:
{具体说明 Harness 应如何利用这个能力}
**实现方案**:
| 目标文件 | 变更内容 |
|------------|---------|
| `{文件路径}` | {具体变更内容} |
| `{文件路径}` | {具体变更内容} |
**用户体验改善**:
- 之前:{当前用户体验}
- 现在:{实现后的用户体验}
**实现优先级**:{高 / 中 / 低}
**预估工作量**:{小 / 中 / 大}
出力例
## 检测到类别 B:实现方案
### B-1. Desktop Scheduled Tasks
**现状**:只写进了 Feature Table,Harness 侧没有实际实现。
**Harness 独有的附加价值**:
把 Scheduled Tasks 与 Harness 工作流集成起来,用于自动化定期质量检查、
状态同步与记忆整理。
**实现方案**:
| 目标文件 | 变更内容 |
|------------|---------|
| `skills/harness-work/references/scheduled-tasks.md` | 定时任务模板与指南 |
| `scripts/setup-scheduled-tasks.sh` | 初始 setup 脚本 |
| `hooks/hooks.json` | 注册 Cron 触发器 |
**用户体验改善**:
- 之前:用户必须手动执行定期任务
- 现在:Harness 会自动执行定期质量检查并通知结果
**实现优先级**:中
**预估工作量**:中
对“附加价值”列的建议
建议在 Feature Table 中加入下面这列:
| Feature | Skill | Purpose | 附加价值 |
|---|---|---|---|
| PostCompact hook | hooks | 上下文重新注入 | A: 已有实现 |
| Streaming leak fix | all | 内存泄漏修复 | C: CC 自动继承 |
加上这列后,各条目的分类可以一眼看清,也能避免残留类别 B 项目。
相关技能
harness-review- 代码评审(当判定为 CC 集成 PR 时,会内部调用这个技能)harness-work- 实现工作(按照类别 B 的实现方案落地时使用)memory- SSOT 管理(记录分类标准与决策)