Skip to content

fix(workflows): exempt bug-fix from PR-count confirmation - #4636

Merged
KSchlobohm merged 5 commits into
github:mainfrom
KSchlobohm:kschlobohm-sep-18-workflow-exemption
Sep 18, 2026
Merged

KSchlobohm merged 5 commits into
github:mainfrom
KSchlobohm:kschlobohm-sep-18-workflow-exemption

Conversation

@KSchlobohm

Copy link
Copy Markdown
Contributor

Description

Extend the maintenance-workflow clarification from #4607 to bug-fix. Explicitly skip the contributor open-PR count check and confirmation regardless of the submitter's or filing account's count, after completing the assessment-scoped remediation and existing local-check requirements.

Preserve assessment stop paths, draft-only publication, protected-file restrictions, and harness-managed branching, committing, and pushing. No permissions or frontmatter changes. The existing lockfile imports the Markdown instructions at runtime.

Testing

  • New static exemption assertion failed before the prompt change and passed afterward.
  • .venv\Scripts\python.exe -m pytest tests\test_bug_fix_workflow.py tests\test_github_workflows.py -k 'bug_fix or bug_workflow_upgrade or community_upgrade' -q — 18 passed.
  • gh aw compile bug-fix --no-emit --no-check-update — passed.
  • git diff --check — passed.

These checks protect the instruction and existing contracts; no live before/after workflow run demonstrates that the model's count-based refusal is fixed.

AI Disclosure

  • I did not use AI assistance for this contribution
  • I did use AI assistance

AI disclosure: GitHub Copilot (model: GPT-6 Astra) generated the workflow text and tests on behalf of @KSchlobohm in an agent-driven implementation (autonomous code generation), approved by the user for publication. Reasoning effort was not independently reported.

KSchlobohm and others added 3 commits September 16, 2026 23:26
Regenerate the three community submission locks with Copilot CLI 1.0.80 and the compiler defaults. Preserve submission instructions and file allowlists, and cover runtime compatibility and output guards.

Assisted-by: GitHub Copilot (model: GPT-6 Astra, autonomous)

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Merge upstream main, retaining upstream's PR-count exemption tests and preserving fork history. The merged file tree matches upstream main at fcfc7e8.

Assisted-by: GitHub Copilot (model: GPT-6 Astra, autonomous)

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Apply the community maintenance-workflow exemption to bug-fix while preserving assessment gates and harness-managed draft publication. Add static exemption and runtime-import coverage.

Assisted-by: GitHub Copilot (model: GPT-6 Astra, autonomous)

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Copilot AI balanced review requested due to automatic review settings September 18, 2026 20:28
@KSchlobohm
KSchlobohm marked this pull request as ready for review September 18, 2026 20:28
@KSchlobohm
KSchlobohm requested a review from mnriem as a code owner September 18, 2026 20:28

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot review overview

🟢 Approval recommended

The focused instruction change is consistent with analogous workflows and has appropriate regression coverage.

Review effort: Balanced
Findings: None

What changed in this PR

Exempts the repository-owned bug-fix workflow from contributor open-PR count confirmation while preserving existing safeguards.

Changes:

  • Adds the exemption to draft PR publication instructions.
  • Adds static regression coverage and preserves harness-managed Git behavior.
  • Regression evidence was reported but not independently rerun.
File Description
.github/​workflows/​bug-fix.md Adds the PR-count exemption.
tests/​test_github_workflows.py Guards the exemption and publication contract.

💡 Configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Regenerate bug-fix and the three community workflow lockfiles with gh-aw v0.88.7 after the PR-count exemption changes. Only body_hash metadata changes; executable YAML is unchanged.

Assisted-by: GitHub Copilot (model: GPT-6 Astra, autonomous)

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings September 18, 2026 20:44

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot review overview

🟢 Approval recommended

The focused instruction change has regression coverage and preserves the workflow’s existing safety contracts.

Review effort: Balanced
Findings: None

Preserve upstream's community archive-validation workflows and generated configuration while retaining the bug-fix PR-count exemption and refreshed body hash.

Assisted-by: GitHub Copilot (model: GPT-6 Astra, autonomous)

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings September 18, 2026 20:47

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot review overview

🟢 Approval recommended

The focused instruction matches existing maintenance workflows and has appropriate regression coverage.

Review effort: Balanced
Findings: None

@KSchlobohm
KSchlobohm merged commit d4229c0 into github:main Sep 18, 2026
15 checks passed
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.

2 participants