Ccdawn pr review

Skill CCDawn/codex-skills/skills/engineering/ccdawn-pr-review

Chinese-first Codex skill pack for intent alignment, dynamic routing, lightweight development, code review, UI design, and AI research workflows.

Install

npx -y skills add CCDawn/codex-skills --skill ccdawn-pr-review

Assembled from the repository path, not quoted from the project. Check it against their README if it does not work.

One thing to look at

4 stars4 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.

What its author says it does

Copied from the file, not written here

Use when the user explicitly requests review of a PR, diff, branch, commit range, merge readiness, or review feedback, or when a high-risk change has reached an explicit pre-integration review gate; do not trigger merely because ordinary development finished.

The file declares its own license as MIT. That is the author’s claim about this one file, and it is not the same thing as the license GitHub reports for the repository, which is listed with the other numbers below.

SKILL.md

5.0 KB, as published. Nobody here has run it

CCDawn PR Review

目标

只读审阅 PR、diff、branch 或 commit range，判断需求覆盖、回归风险和 merge readiness。先给 findings，不用长矩阵掩盖结论；审阅阶段不顺手改代码。

BRT interface

Context Boundary: PR/diff/base-head、需求来源、验证证据、禁止编辑边界、集成目标和排除范围。
Output Contract: risk-ranked findings、简洁结论、证据缺口、修复顺序或 merge route。
Allowed Action: 读取 diff/上下文并运行安全检查；不编辑、移动 HEAD/index、合并、推送或发布。
Success Evidence: diff 与需求已对照，关键证据已检查，每条 finding 绑定位置、影响和验证条件。
Stop Condition: 缺可审对象、审查目标/需求无法推断、关键证据不可得、对象漂移或远程/高风险动作未授权。
Route Out: 对应开发 owner、ccdawn-bug-review、ccdawn-performance-engineering、ccdawn-planning、ccdawn-brt、提交/PR/合并准备或 BLOCKED。

统一调用契约

只处理 BRT interface 范围；不匹配时回 ccdawn-brt 或更具体 owner，复合任务不吞其他 owner。
用户可见内容默认中文；保留技术字面量；只报结论、证据、风险和产出；Route Out 仅以 BRT interface 为准，末行写 下一步建议: <一个具体动作>。

审阅流程

定位 PR、base/head、merge-base 和实际 diff；检查工作区/分支是否漂移。
从用户要求、PR body、issue/spec、现有行为和项目规则中取得需求来源。只有高影响目标无法推断时才回 BRT 集中对齐，不能只做 owner 路由。
阅读变更及必要上下文，核对状态/API/数据/配置/迁移/用户流程和保护边界。
检查最新测试、构建、lint、类型、运行时或手工验收证据；证据不足不能包装成通过。
按 diff 风险选择相关视角，不固定遍历完整清单；只输出由本次变更引入、暴露或会阻塞集成的问题。性能视角只在热路径、规模、查询/I/O、缓存/队列、并发、包体或既有性能契约被触及时启用。
findings 优先；随后给 merge 结论和下一 route。

明显 N+1、循环 I/O、重复全量计算或无界资源增长可用结构/确定性计数形成 finding；声称“更快/回归”必须有可比较 baseline。需要实际 profiling 或 before/after 时以本 skill 为 primary、ccdawn-performance-engineering 为 support，不把每个 PR 变成性能审计。

Findings

P0 BLOCKER：数据丢失、安全事故、核心不可用或不可逆发布风险。
P1 MUST_FIX：重要需求缺失、明确 bug、关键契约/测试/迁移风险。
P2 SHOULD_FIX：边界、错误处理、维护性或局部回归风险。
P3 NICE_TO_HAVE：默认省略；只有能明显降低近期误改或审阅成本时才保留。

每条包含紧凑文件/行号或 diff 位置、问题、影响、建议方向和验证条件。纯风格偏好、无行为影响的命名建议、未被 diff 影响的既有问题不作为 finding；必要时用一句非阻塞备注。没有问题时明确“未发现阻塞性问题”，并说明尚未覆盖的证据边界。

多个问题按依赖和修复成本给执行顺序；用户已要求修复 review findings 时，回最具体 owner 连续处理所有 SAFE_DIRECT 项，不每项询问。设计分叉、高风险动作或 BLOCKED 才暂停。

结论

READY：无 P0/P1，需求与证据足够。
READY_WITH_FIXES：仅有非阻塞 P2/P3。
READY_CONDITIONAL：变更相关证据通过，但仓库 gate 因已在 base 复现的 BASELINE_FAILURE 或与 diff 无关的 ENVIRONMENT_FAILURE 不可运行；必须记录失败命令、base 复现、已通过证据和集成后补验责任。
NEEDS_CHANGES：存在 P0/P1、需求偏离或关键证据缺失。
BLOCKED：没有可审 diff，或审查目标/证据无法取得。

gate 失败先分为 CHANGE_FAILURE / BASELINE_FAILURE / ENVIRONMENT_FAILURE / POLICY_FAILURE / UNKNOWN。只做一次限时环境 probe；无新信号就停止安装或重建环境。未在干净 base/等价事实源复现的失败不能称为 baseline。安全、secret、权限、数据迁移、发布合规及项目明确不可绕过的 gate 不允许条件通过。

输出

Findings:
- P0/P1/P2/P3 [文件:行] 问题；影响；建议；验证条件

结论: READY / READY_WITH_FIXES / READY_CONDITIONAL / NEEDS_CHANGES / BLOCKED
审阅范围与需求来源: ...
验证证据/缺口: ...
执行顺序（仅多个修复项时）: ...
剩余风险: ...
下一步建议: <一个具体动作>

实际提交、推送、合并和发布仍需对应权限；审阅结论不等于自动执行远程动作。