Skip to content

fix(noema): skip workflow runs without PR context - #435

Merged
seonghobae merged 2 commits into
mainfrom
fix/noema-workflow-run-without-pr
Jul 11, 2026
Merged

seonghobae merged 2 commits into
mainfrom
fix/noema-workflow-run-without-pr

Conversation

@seonghobae

Copy link
Copy Markdown
Contributor

Summary

  • skip Noema review early when a workflow_run event has no associated PR number
  • keep fail-closed behavior for real PR review paths with a visible error reason
  • add a workflow contract test so token exchange cannot run before PR context exists

Failure evidence

Verification

  • py -m pytest tests/test_required_workflow_queue_contract.py tests/test_noema_review_gate.py -q
  • py -m pytest -q
  • npx -y @colbymchenry/[email protected] status

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