Skip to content

fix: enable auto-assign (fix path + drop broken triage label call) - #11

Merged
baxyz merged 2 commits into
mainfrom
fix/enable-auto-assign-and-cleanup
Jul 29, 2026
Merged

fix: enable auto-assign (fix path + drop broken triage label call)#11
baxyz merged 2 commits into
mainfrom
fix/enable-auto-assign-and-cleanup

Conversation

@baxyz

@baxyz baxyz commented Jul 29, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Second file with the same misplacement bug as the manual-fallback workflows (fix: adopt trigger-website-update, fix manual-fallback workflow paths #7): workflows/auto-assign.yml (repo-root, not .github/workflows/) was never registered as a runnable workflow — this repo's own issues/PRs have never been auto-assigned.
  • Rather than just moving the existing bespoke implementation, replaced it with the same reusable-auto-assign.yml wrapper pattern already used by action/typescript/website/devcontainer's auto-assign.yml — for two reasons:
    • The old implementation's "Add labels to issue" step calls addLabels with ['triage'], but no triage label exists in this repo (checked via gh api repos/helpers4/.github/labels) — moving it as-is would have made every issue-opened run fail immediately.
    • Reusing the proven reusable workflow (already exercised by 4 other repos) is more consistent than keeping a diverging bespoke implementation that predates the standardized pattern.

Test plan

  • After merge, open a test issue/PR to confirm auto-assign actually fires (never worked before)

🤖 Generated with Claude Code

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Comment thread .github/workflows/auto-assign.yml Fixed
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@baxyz
baxyz merged commit bfa2316 into main Jul 29, 2026
2 checks passed
@baxyz
baxyz deleted the fix/enable-auto-assign-and-cleanup branch July 29, 2026 22:34
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