Skip to content

Fix release notes tag pattern to match actual semver tags (no v prefix)#2664

Merged
piyalbasu merged 2 commits intofeature/new-release-pipelinefrom
copilot/sub-pr-2660-again
Mar 24, 2026
Merged

Fix release notes tag pattern to match actual semver tags (no v prefix)#2664
piyalbasu merged 2 commits intofeature/new-release-pipelinefrom
copilot/sub-pr-2660-again

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Mar 24, 2026

Release notes generation in new-release.yml searched for tags matching v* / ^v[0-9]+\.[0-9]+\.[0-9]+$, but submitProduction.yml creates plain semver tags (e.g. 5.39.0). This caused last_release_tag to always be empty, resulting in release notes spanning the entire repo history.

Changes

  • .github/workflows/new-release.yml: Update tag glob and regex to match plain semver format:
- last_release_tag=$(git tag --list 'v*' --sort=-version:refname | grep -E '^v[0-9]+\.[0-9]+\.[0-9]+$' | head -n 1 || true)
+ last_release_tag=$(git tag --list '[0-9]*' --sort=-version:refname | grep -E '^[0-9]+\.[0-9]+\.[0-9]+$' | head -n 1 || true)

🔒 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 new release pipeline implementation Fix release notes tag pattern to match actual semver tags (no v prefix) Mar 24, 2026
Copilot AI requested a review from piyalbasu March 24, 2026 20:29
@piyalbasu piyalbasu marked this pull request as ready for review March 24, 2026 20:38
@piyalbasu piyalbasu merged commit 29625c6 into feature/new-release-pipeline Mar 24, 2026
2 checks passed
@piyalbasu piyalbasu deleted the copilot/sub-pr-2660-again branch March 24, 2026 20:39
piyalbasu added a commit that referenced this pull request Mar 27, 2026
* first pass at new release pipeline

* Add PR preview and integration test pipelines, remove manifest v2 from createRelease

- Add prPreview.yml: builds extension on every PR commit and publishes as a GitHub release
- Add runIntegrationTests.yml: manual workflow for running tests with INTEGRATION_MODE=true
- Remove manifest v2 update steps from createRelease.yml

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* Rename createRelease to new-release with validation, release notes, and emergency release support

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* Update .github/workflows/runIntegrationTests.yml

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* Fix release notes tag pattern to match actual semver tags (no `v` prefix) (#2664)

* Initial plan

* Fix tag pattern in new-release.yml to match plain semver tags (no v prefix)

Co-authored-by: piyalbasu <6789586+piyalbasu@users.noreply.github.com>
Agent-Logs-Url: https://github.com/stellar/freighter/sessions/774e2d0a-dd3f-44b9-ae49-778d641f5c0c

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: piyalbasu <6789586+piyalbasu@users.noreply.github.com>

* Harden CI workflows against injection and supply-chain attacks

- Validate inputs.branch with regex and pass through env context to prevent command injection
- Replace git add -A with explicit file paths to avoid staging unintended files
- Pin actions/checkout, actions/setup-node, and actions/upload-artifact to full commit SHAs
- Restrict integration test workflow_dispatch to master branch only

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* Add same-repo guard to PR preview workflow

Restrict build-and-release and cleanup jobs to only run for PRs
originating from the same repository, not forks.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* Harden CI/CD workflows: security audit fixes

- Rename new-release.yml to newRelease.yml
- Add validate-branch-from composite action with tag validation
- Replace third-party jossef/action-set-json-field with inline jq
- Pin all actions to commit SHAs, remove mutable tag references
- Add permissions blocks (least-privilege) to all workflows
- Fix concurrency group to scope per-PR instead of global
- Add runtime branch guard on integration test workflow_dispatch
- Move branch input validation before checkout in release workflow
- Add duplicate release tag guard
- Use random heredoc delimiters to prevent output injection
- Use mktemp for temp files, exact grep matches for branch checks
- Reduce fetch-depth to 1 on test workflows
- Move commented-out prPreview.yml to workflow-drafts/
- Remove unnecessary GITHUB_ENV export and token on checkout

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* Fix env var formatting and git fetch error handling per review feedback

Agent-Logs-Url: https://github.com/stellar/freighter/sessions/da96990c-afc7-4c32-8778-9dfb0f1bb116

Co-authored-by: piyalbasu <6789586+piyalbasu@users.noreply.github.com>

* Apply review suggestions: checkout order fix, guard empty commits in release workflow

Agent-Logs-Url: https://github.com/stellar/freighter/sessions/bde2ad2e-5cbf-4d02-bfa7-d25fa69154c2

Co-authored-by: piyalbasu <6789586+piyalbasu@users.noreply.github.com>

* Remove unnecessary actions: write permission from runTests and runIntegrationTests workflows

Agent-Logs-Url: https://github.com/stellar/freighter/sessions/035dca62-de23-4032-ac71-cca8afbcb069

Co-authored-by: piyalbasu <6789586+piyalbasu@users.noreply.github.com>

---------

Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <198982749+Copilot@users.noreply.github.com>
Co-authored-by: piyalbasu <6789586+piyalbasu@users.noreply.github.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.

2 participants