Skip to content

Add release_tag validation guard to update-from-dispatch workflow#66

Merged
ChronosSF merged 2 commits intosstoychev/add-automated-workflowfrom
copilot/sub-pr-62-again
Mar 27, 2026
Merged

Add release_tag validation guard to update-from-dispatch workflow#66
ChronosSF merged 2 commits intosstoychev/add-automated-workflowfrom
copilot/sub-pr-62-again

Conversation

Copy link
Copy Markdown

Copilot AI commented Mar 27, 2026

Without validation, a missing or malformed release_tag in the repository_dispatch payload silently produces an invalid branch name (e.g. .x) and allows subsequent git commands to proceed against it.

Changes

  • Fail-fast on empty tag — workflow exits immediately with a clear ::error:: message if release_tag is absent or empty
  • Semver pattern validation — rejects tags that don't match ^v?[0-9]+\.[0-9]+\.[0-9]+(-[a-zA-Z0-9.]+)?$ before any branch computation occurs
TAG="${{ github.event.client_payload.release_tag }}"
if [[ -z "$TAG" ]]; then
  echo "::error::release_tag is missing or empty in the repository_dispatch payload."
  exit 1
fi
if [[ ! "$TAG" =~ ^v?[0-9]+\.[0-9]+\.[0-9]+(-[a-zA-Z0-9.]+)?$ ]]; then
  echo "::error::release_tag '$TAG' does not match the expected semver pattern (e.g. v1.2.3 or 1.2.3)."
  exit 1
fi

Accepts v18.2.3, 18.2.3, and pre-release variants like v1.0.0-alpha.1. Rejects .x, bare strings, partial versions, and shell-injection attempts.


🔒 GitHub Advanced Security automatically protects Copilot coding agent pull requests. You can protect all pull requests by enabling Advanced Security for your repositories. Learn more about Advanced Security.

Copilot AI changed the title [WIP] [WIP] Address feedback on automated update workflow - add requested guard Add release_tag validation guard to update-from-dispatch workflow Mar 27, 2026
Copilot AI requested a review from ChronosSF March 27, 2026 16:11
@ChronosSF ChronosSF marked this pull request as ready for review March 27, 2026 16:14
@ChronosSF ChronosSF merged commit 3329186 into sstoychev/add-automated-workflow Mar 27, 2026
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