fix(agent-mention): keep OpenCode dispatch under GitHub payload cap - #1099
fix(agent-mention): keep OpenCode dispatch under GitHub payload cap#1099seonghobae wants to merge 2 commits into
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (4)
Included review availability: Your plan includes up to 1 review per rolling hour; 0 remain after this review. 📝 WalkthroughWalkthroughOpenCode 디스패치 페이로드에서 리뷰 제어값을 제거했습니다. 워크플로는 해당 값을 기본값으로 재구성합니다. 테스트는 축소된 페이로드의 키 집합과 필드 수를 검증합니다. ChangesOpenCode 디스패치 계약
Estimated code review effort: 3 (Moderate) | ~20 minutes Merge Risk: ⚪ Minimal · up to The change narrowly caps agent-mention dispatch payloads while preserving claim binding and adds regression coverage for both transport hops; no actionable merge-blocking risk remains after normal checks and review. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
Bugbot is not enabled for your account, so this pull request was not reviewed. Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs. |
|
Independent review request withdrawn after the existing Noema route returned an unrelated target-token HTTP 403. This PR remains limited to the OpenCode repository-dispatch payload-cap defect. Do not modify or merge |
Incident
route-local-agent-mentionfailed on the trusted@opencode-agentcomment in.github#955(run32026573583, job95379052395) before the wrapper could start. The router sent 14 top-levelclient_payloadproperties; GitHub repository dispatch accepts at most 10.This is a transport-shape defect, not a
seonghobaetrust or permission failure.Scope
Only the agent-mention OpenCode route is changed:
No merge policy, reviewer identity, credential, branch update, auto-merge, scheduler decision logic, or repository allowlist is changed.
.github#955is not part of this PR and must not be merged by this work.Test-first lineage
f43fb27a5820acd0743c2e6759fe0e44a4a5b0d1: the permanent payload-cap regression failed exactly because the router emitted 14 properties.e95c42c116a6c4029bac78ad86e7e209d2b884ba: the router and trusted wrapper transports are narrowed without weakening complete claim binding.Exact-head verification
Agent Mention Router Quality CI run
32029132676, job95384907487passed on the GREEN head:1117 passed, 16 subtests passedagent_mention_router.py: 239/239 statements and 102/102 branches, 100%agent_mention_sweep.py: 162/162 statements and 66/66 branches, 100%compileallandgit diff --check: passedMerge gate
Ready for independent review. Do not bypass required protected checks or approval. Do not merge
.github#955as part of this work.Summary by CodeRabbit
개선 사항
테스트