Skip to content

Route new issues without the blocked actions-cool action (fixes #137) - #138

Draft
skilledwolf wants to merge 1 commit into
ALPSim:masterfrom
skilledwolf:fix/issue-triage-local-parser
Draft

Route new issues without the blocked actions-cool action (fixes #137)#138
skilledwolf wants to merge 1 commit into
ALPSim:masterfrom
skilledwolf:fix/issue-triage-local-parser

Conversation

@skilledwolf

@skilledwolf skilledwolf commented Sep 2, 2026

Copy link
Copy Markdown
Collaborator

Summary

Every new issue made the "Simulation Help Router" workflow fail before its first step ran: the runner cannot download actions-cool/issues-helper ("Repository access blocked"), so the reporter got a failure e-mail and nobody was assigned.

  • read the "Method" dropdown directly from the issue body with a small awk step; the body is passed through an environment variable, so issue text cannot inject shell code
  • set GH_REPO so gh issue edit works in a job without a checkout
  • match the SSE and Worm choices on their abbreviation, so "Quantum Monte Carlo (Worm)" no longer lands in the SSE branch

Issues without a Method field and the "Other" choice keep going to the default assignee, as before.

Testing

  • actionlint .github/workflows/issue-triage.yml
  • ran the workflow's run: scripts locally with a stubbed gh against all six dropdown values, a CRLF body, the body of [Bug]: Bug results in error from failed workflow #137, an empty body and an injection attempt; each routed to the expected maintainer with no side effects

Closes #137

…elper

Every new issue made the "Simulation Help Router" workflow fail before
its first step ran: the runner could not download the third-party
actions-cool/issues-helper action ("Repository access blocked"), so the
issue author received a failure e-mail and nobody was assigned.

Read the "Method" dropdown directly from the issue body (GitHub renders
each form field as a "### <label>" heading followed by the value) with a
small awk script.  The body is passed through an environment variable so
issue text cannot inject shell code.  While here:

- give gh the repository via GH_REPO, which it needs because the job has
  no checkout;
- match the SSE and Worm choices on their abbreviation, so "Quantum Monte
  Carlo (Worm)" no longer falls into the earlier SSE branch.

Issues without a Method field and the "Other" choice keep going to the
default assignee, as before.

Fixes ALPSim#137

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
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.

[Bug]: Bug results in error from failed workflow

1 participant