Skip to content

Keep manual release dispatches artifact-only#12

Merged
masarray merged 1 commit into
mainfrom
agent/harden-release-trigger
Jul 11, 2026
Merged

Keep manual release dispatches artifact-only#12
masarray merged 1 commit into
mainfrom
agent/harden-release-trigger

Conversation

@masarray

Copy link
Copy Markdown
Owner

Follow-up hardening for the public release trigger.

  • Manual workflow_dispatch always honors its explicit version input.
  • A release/vX.Y.Z branch publishes only when the event is an actual push.
  • Manual runs started from a release branch remain artifact-only and cannot accidentally republish or replace a public release.

Direct tag releases and versioned release-branch pushes remain unchanged.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: b520b89078

ℹ️ 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".

release:
name: Publish GitHub Release
if: github.ref_type == 'tag' || startsWith(github.ref, 'refs/heads/release/v')
if: github.ref_type == 'tag' || (github.event_name == 'push' && startsWith(github.ref, 'refs/heads/release/v'))

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Gate tag releases on push events too

For a workflow_dispatch run started against a tag ref, github.ref_type is still tag, so this condition still runs the public release job even though the change is meant to make manual dispatches artifact-only. In that same scenario the version step also resolves from the tag before checking workflow_dispatch, so a maintainer trying to produce a manual artifact from a tag can still publish or update the GitHub Release; require github.event_name == 'push' for the tag path as well.

Useful? React with 👍 / 👎.

@masarray
masarray merged commit 7390a43 into main Jul 11, 2026
5 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.

1 participant