Skip to content

Fixing action job issue in fast-mlsirm#620

Draft
seonghobae with Copilot wants to merge 2 commits into
mainfrom
copilot/fix-action-job-issue
Draft

Fixing action job issue in fast-mlsirm#620
seonghobae with Copilot wants to merge 2 commits into
mainfrom
copilot/fix-action-job-issue

Conversation

Copilot AI commented Jul 24, 2026

Copy link
Copy Markdown
Contributor

Pull request created by AI Agent

Copilot AI added 2 commits July 23, 2026 09:46
…de run is active

When a PR is synchronized, the OpenCode required workflow fires on
pull_request_target and the PR Review Merge Scheduler fires concurrently
on the same event. The scheduler queries the PR's statusCheckRollup and
sees the opencode-review check run as IN_PROGRESS, concluding "OpenCode
review is already in progress" and waiting.

The problem: pull_request_target runs of the OpenCode workflow always
skip the real review job via `github.event_name == 'repository_dispatch'`
gate. Once the pull_request_target run completes (with the review job
SKIPPED), no workflow_run retry fires for required workflows, so the
scheduler never dispatches the real review.

Fix 1 (active_review_run_refs): Exclude pull_request and
pull_request_target event runs from the target repo when classifying
active review runs. These events never execute the review job and must
not be counted as active reviews.

Fix 2 (inspect_pr): When opencode_progress_state returns "running",
call active_opencode_run_refs to verify an actual repository_dispatch
run is active. If only PR-event runs exist, override the state to
"absent" so the review is dispatched immediately instead of stalling.

Adds tests for both fix points with 100% coverage maintained.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because there is no eligible user to bill. To allow Copilot reviews on bot-authored pull requests, enable direct organization billing in your organization's Copilot settings.

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.

3 participants