Skip to content

fix(ci): fork PR secrets support + gitignore .omo/#31

Merged
watney1024 merged 2 commits into
mainfrom
zmz_dev
Jul 26, 2026
Merged

fix(ci): fork PR secrets support + gitignore .omo/#31
watney1024 merged 2 commits into
mainfrom
zmz_dev

Conversation

@watney1024

Copy link
Copy Markdown
Collaborator

变更内容

1. AI Code Review 支持 fork PR

  • Trigger 从 pull_request 改为 pull_request_target,使 fork PR 能访问 SENSENOVA_API_KEY
  • Checkout 加 ref: ${{ github.event.pull_request.head.sha || github.sha }},安全锁定 SHA + workflow_dispatch fallback
  • 删除跨仓库评论跳过逻辑(pull_request_target 下 GITHUB_TOKEN 有写权限)

2. .omo/ 目录加入 gitignore

  • 将 AI Agent 会话追踪文件目录 .omo/ 加入 .gitignore
  • 已追踪的 17 个 .omo/ 文件从索引移除(保留本地)

watney1024 and others added 2 commits July 26, 2026 17:52
The .omo/ directory contains AI agent session tracking files that should not be version controlled. Added to .gitignore and removed existing files from the index (kept locally).

Ultraworked with [Sisyphus](https://github.com/code-yeongyu/oh-my-openagent)

Co-authored-by: Sisyphus <clio-agent@sisyphuslabs.ai>
Change trigger from pull_request to pull_request_target so fork PRs can access SENSENOVA_API_KEY. Use SHA checkout (head.sha || github.sha) for security and workflow_dispatch compatibility. Remove cross-repo comment skip guard since GITHUB_TOKEN now has write access.

Ultraworked with [Sisyphus](https://github.com/code-yeongyu/oh-my-openagent)

Co-authored-by: Sisyphus <clio-agent@sisyphuslabs.ai>
@jizhenjun jizhenjun self-assigned this Jul 26, 2026
@jizhenjun
jizhenjun self-requested a review July 26, 2026 11:33
@watney1024
watney1024 merged commit 1aca956 into main Jul 26, 2026
5 of 6 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