Skip to content

Commit 41c40ff

Browse files
committed
docs(auto-code-review): clarify scenarios not handled by the skill
- Updated the OUT-OF-SCOPE documentation to specify that certain scenarios are not processed by the skill and do not trigger automatic review. - Enhanced the auto-code-review documentation with detailed triggering logic, outlining explicit user actions required to initiate a review.
1 parent f2cffae commit 41c40ff

3 files changed

Lines changed: 21 additions & 2 deletions

File tree

‎skills-engineering/auto-code-review/OUT-OF-SCOPE.md‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@
1212
- 纯文档更新(.md 文件)
1313
- 配置文件微调(单行修改)
1414
- typo 修复、格式化调整
15-
- 这些场景跳过自动审查。
15+
- 这些场景本 skill 不处理,也不自动启动审查。
1616

1717
## 3. 无代码变更的对话
1818

‎skills-engineering/auto-code-review/i18n/en-US/references/out_of_scope.md‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ This skill does **NOT** handle the following scenarios:
1212
- Pure documentation updates (.md files)
1313
- Minor configuration tweaks (single-line changes)
1414
- Typo fixes, formatting adjustments
15-
- These scenarios skip automatic review.
15+
- These scenarios are NOT handled by this skill, and review is NOT started automatically.
1616

1717
## 3. Conversations Without Code Changes
1818

‎skills-engineering/docs/auto-code-review.md‎

Lines changed: 19 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,23 @@
66

77
名称中的 `auto` 表示:用户启动后,工具会自动完成 reviewer 调用、结果归档、知识库同步,以及在用户额外授权时执行修复循环。
88

9+
## 触发逻辑
10+
11+
审查的启动遵循唯一一条规则:
12+
13+
> **只有用户在本轮对话中显式触发,才进入审查;除此之外任何情况都不触发。**
14+
15+
具体判定:
16+
17+
- 触发条件(满足其一即可):用户在本轮请求中明确说出 `/auto-review`、`使用 auto-code-review`、`启动跨模型代码审查`、`/auto-review --fix` 或 `使用 auto-code-review 审查并修复`。
18+
- 不触发条件(任一成立即不进入审查):
19+
- 普通代码生成、修改完成、测试通过;
20+
- “看看代码”“检查一下”等未明确指向跨模型工作流的含糊请求;
21+
- 仅设置 `AUTO_REVIEW_ENABLED=true`;
22+
- 纯问答、纯文档任务或任何非本次请求显式授权的场景。
23+
24+
配置(`enabled: true`、环境变量等)只控制能力是否可用,**不代表当前请求已获得授权**。能力开关不构成、也不能替代用户的显式触发。
25+
926
## 权限模型
1027

1128
审查与修改是两层独立权限:
@@ -19,6 +36,8 @@
1936

2037
## 如何触发
2138

39+
(触发逻辑见上文。以下为显式触发的可用表达。)
40+
2241
明确使用以下表达之一:
2342

2443
- `/auto-review`

0 commit comments

Comments
 (0)