@h-ai/cli 0.1.0-alpha.12 → 0.1.0-alpha.15

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
@@ -0,0 +1,109 @@
1
+ ---
2
+ name: hai-pr-review
3
+ description: 审查 Pull Request 的交付质量、scope、风险、验证方式、AI-assisted 说明与合并准备度;当需求涉及 PR review、AI Review 策略、PR 模板、CODEOWNERS 或合并门禁时使用。
4
+ ---
5
+
6
+ # hai-pr-review — PR 审查与 AI Review 规范
7
+
8
+ > PR 是团队协作和 AI 辅助开发的交付边界。AI 可以帮助发现问题,但不能替代确定性 CI 和人类最终责任。
9
+
10
+ ---
11
+
12
+ ## 适用场景
13
+
14
+ - 审查 AI-generated / AI-assisted PR
15
+ - 审查 Pull Request 的 scope、风险、验证方式、issue 关联与合并准备度
16
+ - 判断 AI Review 是 advisory 还是 blocking,以及是否需要人工复核
17
+ - 新增或修改 `.github/pull_request_template.md`
18
+ - 配置 CODEOWNERS、required review、branch protection
19
+ - 设计 AI PR Review 方案
20
+ - 选择 Copilot、CodeRabbit、reviewdog、DangerJS、Claude Code Action、Codex Action 等工具
21
+
22
+ 不负责:代码级质量审查(用 `hai-app-review` / 模块审查技能)、测试用例设计(用 `hai-app-tests`)、workflow 触发/权限/secret 细节(用 `hai-ci`)。
23
+
24
+ ---
25
+
26
+ ## 审查分层
27
+
28
+ ### 第 1 层:确定性检查(必须)
29
+
30
+ - CI required checks 必须全绿。
31
+ - 具体 workflow、权限、secret scan 与 required check 配置由 `hai-ci` 定义。
32
+ - 测试覆盖策略与失败用例设计由 `hai-app-tests` 定义。
33
+
34
+ ### 第 2 层:结构化人工审查(必须)
35
+
36
+ - scope 是否清晰且与 issue 对齐
37
+ - 测试是否覆盖成功路径、失败路径、边界条件
38
+ - 是否符合 hai-framework 约定:HaiResult、i18n、日志、分层、TDD
39
+ - 是否引入密钥、权限扩大、workflow 风险或两仓漂移
40
+
41
+ ### 第 3 层:AI Review(默认 advisory)
42
+
43
+ - AI Review 默认只评论、不阻塞 merge。
44
+ - 只有在 CI 稳定、误报可控、团队明确同意后,才考虑将特定静态规则设为 blocking。
45
+ - AI 的建议必须由人类判断,不得自动 merge。
46
+
47
+ ---
48
+
49
+ ## AI-assisted PR 要求
50
+
51
+ PR 描述必须说明:
52
+
53
+ - 是否使用 AI 辅助
54
+ - AI 负责了哪部分:搜索、实现、测试、review、文档
55
+ - 人类如何验证:命令、截图、日志或 reviewer 重点
56
+ - 是否复制了聊天记录;若有,只能复制脱敏后的结论,禁止包含 token、cookie、私有 URL、账号密码
57
+
58
+ ---
59
+
60
+ ## Issue 关联与上下文交接
61
+
62
+ - PR 必须关联 issue 或说明为什么不需要 issue。
63
+ - PR scope 必须与 issue 的目标、验收标准和明确不做事项一致。
64
+ - AI-assisted PR 必须说明 AI 负责了哪些阶段,以及人类如何复核。
65
+ - 多人或多 agent 协作时,PR 描述必须列出 reviewer focus,避免重复 review。
66
+
67
+ ---
68
+
69
+ ## 自动 Review 工具选择
70
+
71
+ | 工具 | 推荐定位 | 是否适合 blocking |
72
+ | --- | --- | --- |
73
+ | reviewdog | 把 lint/static analyzer 结果贴到 PR | 适合,前提是规则稳定 |
74
+ | DangerJS | PR 元数据、模板、标签、变更范围规则 | 适合,前提是规则明确 |
75
+ | GitHub Copilot Code Review | 通用 AI 代码建议 | 默认 advisory |
76
+ | CodeRabbit | 上下文型 AI review | 默认 advisory |
77
+ | Claude Code Action | 定制化深度 review / 修改建议 | 默认 advisory,严格限制权限 |
78
+ | Codex Action | 定制化 review / 修复建议 | 默认 advisory,严格控制 API key |
79
+
80
+ ---
81
+
82
+ ## 权限与安全红线
83
+
84
+ - ❌ 不在 `pull_request_target` 中运行 AI agent 读取 PR 代码和 secrets。
85
+ - ❌ 不给 AI Review workflow 默认 `contents: write`、`pull-requests: write`,除非只用于评论且触发源可信。
86
+ - ❌ 不允许 AI 自动合并 PR。
87
+ - ❌ 不把私有提示词、聊天原文、token、cookie、连接串写入 PR。
88
+ - ✅ AI Review 的 prompt 必须限制任务范围、禁止执行 secrets、要求引用具体文件/行。
89
+
90
+ ---
91
+
92
+ ## PR Review 检查清单
93
+
94
+ - [ ] PR 关联 issue,scope 单一
95
+ - [ ] PR 模板完整填写
96
+ - [ ] CI required checks 全绿
97
+ - [ ] 新行为有测试,失败路径有覆盖
98
+ - [ ] 代码级质量问题已由 `hai-app-review` / 模块审查技能覆盖
99
+ - [ ] 文档 / skill / README 与代码同步
100
+ - [ ] workflow 权限或 secrets 变化已由 `hai-ci` 审查
101
+ - [ ] AI-assisted 内容已由人类复核
102
+
103
+ ---
104
+
105
+ ## 相关 Skills
106
+
107
+ - `hai-ci` — CI/CD 与 workflow 安全
108
+ - `hai-review-module` / `hai-app-review` — 代码质量审查
109
+ - `hai-framework-sync` — 两仓同步审查
@@ -11,7 +11,7 @@ description: 使用 @h-ai/reldb 进行 SQLite/PostgreSQL/MySQL 的初始化、SQ
11
11
 
12
12
  ## 运行环境
13
13
 
14
- > ⚠️ **服务端模块(Node.js only)。** 浏览器端请通过 API 端点间接访问数据库(参见 hai-kit 契约模式 `kit.fromContract`)。
14
+ > ⚠️ **服务端模块(Node.js only)。** 浏览器端请通过 `@h-ai/api-contract` + `@h-ai/serv` 暴露的 API 端点间接访问数据库。
15
15
 
16
16
  ---
17
17