feat: add AI code review on PRs

[best]

Closes #37

自动 AI 代码审查

每个 PR 开出后，GitHub Actions 自动运行结构化 checklist 评论，分析：

检查项	检测内容
✅/⚠️ 类型安全	any 用法、@ts-ignore、不安全类型转换
✅/⚠️ 错误处理	空 catch 块、async 代码缺失错误处理
✅/⚠️ 测试覆盖	是否有测试文件更新
✅/⚠️ API 一致性	新增/移除的 public exports
✅/⚠️ 破坏性变更	exports 删除、BREAKING CHANGE 标记

设计原则

• 初期为 informational（不阻断合并）
• 纯 shell script 实现，无外部 AI API 依赖
• 使用 gh pr review --comment 提交评论
• permissions: pull-requests: write 已配置

[github-actions]

🤖 AI Code Review

Automated review for PR #39 — informational only, does not block merge.

Checklist

	Item	Notes
⚠️	Type Safety	1 use(s) of any detected; unsafe as unknown cast detected; @ts-ignore/@ts-nocheck directive found
✅	Error Handling	No issues detected
⚠️	Test Coverage	No test files changed; consider adding/updating tests
✅	API Consistency	No public API changes
⚠️	Breaking Changes	BREAKING CHANGE noted in diff

Stats

• 📁 Files changed: 1 (0 TypeScript)
• ➕ Lines added: 167
• 🧪 Test files updated: 0

---

3 check(s) need attention.

_{Powered by openlinkos/agent · AI PR Review workflow}

[best]

🤖 开发经理 AI Review — PR #39 AI Code Review Workflow

Informational only, 不阻断合并。

Checklist

	项目	说明
✅	类型安全	Shell 脚本，无类型问题
✅	错误处理	`
✅	测试覆盖	Workflow 本身即为验证机制
✅	API 一致性	新增 workflow，不影响现有接口
✅	破坏性变更	无

具体建议

1. 实现定性说明：当前实现是静态 shell 规则检查，非真正 AI 分析。能检测模式匹配（any、空 catch 等），但无法理解代码语义逻辑。建议在 workflow 注释或 README 中说明这一点，避免误解。

2. 误判风险：

   • any 检测会误报注释里的 any
   • empty catch 正则可能漏掉多行空 catch
   • test file 检测用 file 数量，不检测覆盖率变化

3. 未来方向：配合开发经理巡逻时的真实 AI review（gh pr diff + LLM 分析 + gh pr review --comment），本 workflow 可作为快速静态门，两者互补。

总体评价

作为静态检查 CI gate 有价值，实现质量良好。建议补充说明与真实 AI review 的定位差异。

---

_{由开发经理巡逻 AI Review · openlinkos/agent}