Skip to content

接入 OpenCodeReview 作为可选代码审查层 - #74

Merged
sunrisefromdark merged 2 commits into
mainfrom
codex/open-code-review-integration
Jul 6, 2026
Merged

接入 OpenCodeReview 作为可选代码审查层#74
sunrisefromdark merged 2 commits into
mainfrom
codex/open-code-review-integration

Conversation

@sunrisefromdark

@sunrisefromdark sunrisefromdark commented Jul 6, 2026

Copy link
Copy Markdown
Owner

摘要

  • 新增 OpenCodeReview 项目规则,复用现有 codeReviewSkill.md 审查口径
  • 新增本地 OCR wrapper 和 pnpm ocr / code-review:ocr* 命令
  • 新增默认关闭的 PR workflow,支持 fork PR 跳过、非阻断 artifact、固定 OCR 版本和 blocking 开关
  • 更新 pre-push、README、agent-work 文档和 exec-plan 账本,说明自动审查触发条件与启用方式

验证

  • corepack pnpm typecheck
  • corepack pnpm test
  • git diff --check
  • corepack pnpm code-review:preflight
  • OCR_BIN=__missing_ocr__ corepack pnpm ocr
  • OCR_BIN=echo corepack pnpm code-review:ocr -- --exclude src/generated/** --preview

备注

本地未安装 ocr CLI,因此真实 ocr rules check 和 OCR preview 已记录为 skipped。启用 GitHub PR 审查前,需要配置 OCR_REVIEW_ENABLED=true 以及 OCR provider 的 URL、token 和 model。

sunrisefromdark and others added 2 commits July 6, 2026 16:27
- 新增 OpenCodeReview 规则文件,复用现有 codeReviewSkill 并默认排除敏感/生成范围。

- 新增本地 OCR wrapper 与 pnpm 命令,支持 diff review、required gate、scan 和 hard exclude 合并。

- 新增默认关闭的 PR workflow,支持 fork 跳过、非阻断状态 artifact、固定 OCR 版本和 blocking 开关。

- 更新 pre-push、README、agent-work 文档和 exec-plan 账本,说明自动审查触发条件与启用方式。
Co-authored-by: Linli Zhou <kunjoos273@gmail.com>

@JohnWilson273-debei JohnWilson273-debei left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

主人同意狗儿的PR

@sunrisefromdark
sunrisefromdark marked this pull request as ready for review July 6, 2026 09:51
@sunrisefromdark
sunrisefromdark merged commit 323cf67 into main Jul 6, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants