Skip to content

chore(ci): integrate opencode#753

Open
guyutongxue wants to merge 2 commits into
mainfrom
chore-ci-integerate-oc
Open

chore(ci): integrate opencode#753
guyutongxue wants to merge 2 commits into
mainfrom
chore-ci-integerate-oc

Conversation

@guyutongxue

Copy link
Copy Markdown
Member

What Changed

How to Prove It Works

@guyutongxue guyutongxue requested a review from xqm32 as a code owner June 8, 2026 05:38
@opencode-agent

opencode-agent Bot commented Jun 8, 2026

Copy link
Copy Markdown

opencode session  |  github run

@opencode-agent

opencode-agent Bot commented Jun 8, 2026

Copy link
Copy Markdown

Review: chore(ci): integrate opencode

Files: .github/workflows/opencode.yml (+31), .github/workflows/opencode_review.yml (+24)

Issues

  1. pull-requests: read will break comment posting — Both workflows set pull-requests: read, but the opencode action needs to post review comments / issue replies. Without pull-requests: write, posting will fail with a 403. Same issue for issues: write in opencode.yml.

  2. @latest is a floating tag — Both workflows use anomalyco/opencode/github@latest. This will break unexpectedly when a new major version is released. Pin to a specific major version (e.g., @v1) or commit SHA for stability.

  3. No path or branch filter on opencode_review.yml — It triggers on every PR event across all branches. Consider adding paths or branches filters to avoid consuming credits on non-code PRs (docs, config, etc.).

  4. No conditional on opencode.yml fork PRsissue_comment events from forked PRs run with read-only GITHUB_TOKEN and secrets are not available, so DEEPSEEK_API_KEY will be empty. Add a check like github.event.issue.pull_request.head.repo.fork == false to skip forked PRs, or accept this as expected behavior.

Nitpicks

  • opencode.yml: Uses contains + startsWith for both /oc and /opencode. startsWith(github.event.comment.body, '/oc') will also match /octopus, etc. Consider a regex or stricter check.
  • opencode_review.yml has a trailing empty line — minor style inconsistency.

Summary

The integration direction is correct, but permissions and version pinning need fixing before this is ready to merge.

New%20session%20-%202026-06-08T05%3A40%3A53.071Z
opencode session  |  github run

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.

1 participant