fix(review): add OpenCode Zen free fallback - #653
Conversation
📝 WalkthroughWalkthroughOpenCode 설정에 ChangesOpenCode 모델 풀 확장
Estimated code review effort: 2 (Simple) | ~10 minutes Possibly related PRs
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In @.github/workflows/opencode-review-dispatch.yml:
- Line 3718: OPENCODE_MODEL_CANDIDATES 위의 주석을 실제 후보 순서에 맞게 수정하세요. DeepSeek v4
Flash Free를 우선 사용하고 North Mini Code Free를 fallback으로 설명하며, 두 free endpoint의 보존 및
개선 예외에 대한 설명도 동일한 우선순위를 반영하도록 정렬하세요.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro Plus
Run ID: bae3e0e5-44a6-4679-b898-2c117435e41a
📒 Files selected for processing (2)
.github/workflows/opencode-review-dispatch.ymltests/test_opencode_agent_contract.py
![opencode-agent[bot]](https://avatars.githubusercontent.com/in/1549082?s=60&v=4){kind=small}
There was a problem hiding this comment.
Pull request overview
OpenCode reviewed the current-head evidence but found unresolved reviewer or review-agent threads before approval.
Findings
1. HIGH .github/workflows/opencode-review.yml:1 - Unresolved reviewer thread blocks automated approval
- Problem: OpenCode reached an APPROVE control result, but the approval step found unresolved, non-outdated human or review-agent thread evidence on the current pull request.
- Root cause: Reviewer and review-agent feedback can arrive after bounded model evidence is prepared, so the approval step must re-query GitHub immediately before publishing an approval.
- Fix: Address or resolve the listed reviewer thread(s), then re-run OpenCode on the current head.
- Regression test: Keep the approval gate querying reviewThreads(first: 100) after model output and before create_pull_review APPROVE, including bot review agents other than OpenCode itself.
Review thread evidence
Latest unresolved reviewer thread evidence
.github/workflows/opencode-review-dispatch.yml line 3718
-
Latest reviewer comment: @coderabbitai at 2026-07-29T16:16:25Z
-
Comment URL: #653 (comment)
-
Comment excerpt: 🔒 Security & Privacy | 🟡 Minor | ⚡ Quick win / <details> / <summary>🧩 Analysis chain</summary> / 🏁 Script executed: / '''shell / #!/bin/bash / set -euo pipefail / file=".github/workflows/opencode-review-dispatch.yml"
-
Result: REQUEST_CHANGES
-
Reason: unresolved reviewer or review-agent thread(s) were present before approval.
-
Head SHA:
dcacf10804cad4af10f346cfa702b0a388c3628d -
Workflow run: 30469752317
-
Workflow attempt: 1
Changed-File Evidence Map
flowchart LR
PR["PR changed files"] --> Evidence["OpenCode bounded evidence"]
Evidence --> S1["Workflow: opencode-review-dispatch.yml"]
S1 --> I1["GitHub Actions review job"]
I1 --> R1["Review risk: Workflow: opencode-review-dispatch.yml"]
R1 --> V1["actionlint plus required checks"]
Evidence --> S2["Test: test_opencode_agent_contract.py"]
S2 --> I2["regression suite"]
I2 --> R2["Review risk: Test: test_opencode_agent_contract.py"]
R2 --> V2["targeted test run"]
OpenCode Review Overview
Pull request overviewOpenCode reviewed the current-head evidence but found unresolved reviewer or review-agent threads before approval. Findings1. HIGH .github/workflows/opencode-review.yml:1 - Unresolved reviewer thread blocks automated approval
Review thread evidenceLatest unresolved reviewer thread evidence
|
|
Superseded by #654, which carries the CodeRabbit-verified final diff and corrected privacy/order comment as one clean commit so the required OpenCode review receives bounded current-head evidence without resolved review history. |
github-project-automation
Bot
moved this from In Progress
to Done
in naruon Platform Roadmap
1 participant
Summary
deepseek-v4-flash-freeand retaining North as fallbackresponse_format, which made OpenCode describe aReadcall as text instead of executing itCloses #648.
Unblocks ContextualWisdomLab/saju-caldav#14.
Supersedes #649 with the same verified final diff as one clean commit.
Runtime evidence
.github/workflows/opencode-review-dispatch.yml, line3364: challenged whether a free JSON-capable reviewer could still use tools; in an isolated OpenCode run, DeepSeek V4 Flash Free invokedReadon a sample file and then returned exact JSON.github/workflows/opencode-review-dispatch.yml, line3720: challenged whether anonymous Zen could receive private-repository content; the existingis_private == falsegate still wraps both free candidates, and the contract test parses that exact orderingVerification
python3 -m pytest -q— 674 passed for the identical final difftests/test_opencode_agent_contract.py— 37 passed on this commitactionlint -shellcheck=, andgit diff --check— passedSummary by CodeRabbit
opencode.jsonc생성 시opencode-free후보 구성이 업데이트되어 모델 선택 풀/우선순위가 달라졌습니다.opencode-free의 DeepSeek/North Mini 대상에 대해 tool 호출 허용 여부와 응답 포맷 옵션 포함 여부를 추가로 확인합니다.