chore(snapshots): refresh required gate label handling#169
Conversation
Advance the github-workflows snapshot to v1 dc64da5 and teach archon-setup update to remove broad required-gate label triggers while preserving customized caller inputs.\n\nCloses #168
2ea3b04 to
c06f626
Compare
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: c06f626d4f
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| const typedPullRequestRe = | ||
| /( pull_request:\r?\n branches: \[main\]\r?\n)( types:\s*\r?\n \[\s*\r?\n(?: [A-Za-z_]+,\s*\r?\n)+ \]\s*\r?\n| types: \[[^\]\r\n]*\]\s*\r?\n)/; |
There was a problem hiding this comment.
When a managed repo-required-gate.yml has a valid YAML types: list that is not the old exact flow format—for example block sequence style (types:\n - opened\n - labeled) or the existing flow bracket layout without a trailing comma on the last item—this regex does not match, and the fallback refuses to insert because a types: key is already present. In that customized-caller scenario archon-setup update reports the file unchanged and leaves labeled/unlabeled triggers in place, so label-only PR changes can still rerun and cancel the required gate this change is meant to repair.
Useful? React with 👍 / 👎.
1 participant
Summary
github-workflowssnapshot tov1commitdc64da5after fix(required-gate): ignore unrelated label changes github-workflows#58.archon-setup updateto remove broad required-gate label triggers while preserving customized caller inputs.Closes #168
Verification
npm testnode --test test/updateManagedFiles.test.mjsactionlint src/snapshots/github-workflows/repo-required-gate.ymlgit diff --checkVerification Notes
npm testpassed 281 tests with 0 failures. The focused updater test passed 9 tests, including the new required-gate label-trigger repair case.actionlintreturned no diagnostics for the refreshed required-gate snapshot.git diff --checkreturned no whitespace errors.Docs / Changelog
Updated
CHANGELOG.md,docs/ecosystem-overview.md, anddocs/ecosystem-status.mdto record thegithub-workflows@v1movement and snapshot refresh.