Skip to content

fix: carry pr draft status into remediation approval - #78

Merged
svozza merged 1 commit into
mainfrom
fix/command-draft-gate
Sep 16, 2026
Merged

svozza merged 1 commit into
mainfrom
fix/command-draft-gate

Conversation

@svozza

@svozza svozza commented Sep 16, 2026 •

Copy link
Copy Markdown
Owner

The /fix workflow handles issue_comment events, which contain no PR draft status. Its approval assertion therefore treats even trusted authors’ non-draft PRs as drafts and refuses fixes in repositories without required environment reviewers.

Resolve draft status from the PR API during command preparation and pass it to both job routing and the in-job approval assertion. Only literal API false permits a trusted author to bypass approval; draft, missing, and malformed values still require it.

Validation: regression coverage combines author trust with draft states and checks the workflow wiring. GitHub CI passes the full deterministic suite and type checking. Locally, 2,304 tests passed and 2 skipped; four socket-server tests blocked by the sandbox passed on a rerun of all 45 GitHub API tests outside the sandbox. Workflow lint passes with only the installed actionlint version’s missing support for the existing job.workflow_sha and job.workflow_repository fields excluded.

@svozza
svozza deployed to ai-pr-review-runtime September 16, 2026 10:18 — with GitHub Actions Active
@svozza
svozza merged commit 475ab01 into main Sep 16, 2026
8 checks passed

This branch was successfully deployed

1 active deployment
ai-pr-review-runtime — e75b3475 Deployed Sep 16, 2026 by svozza via evals #177
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