Skip to content

fix: TRAC-514 - accept ticket-first commit format in release flow and use PAT for release PRs#1394

Merged
parthshahp merged 1 commit into
masterfrom
parthshahp/ltrac-529-fix-stencil-cli-github-bot-pr-workflow-run
Jul 23, 2026
Merged

fix: TRAC-514 - accept ticket-first commit format in release flow and use PAT for release PRs#1394
parthshahp merged 1 commit into
masterfrom
parthshahp/ltrac-529-fix-stencil-cli-github-bot-pr-workflow-run

Conversation

@parthshahp

@parthshahp parthshahp commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

Summary

Fixes two release-flow problems in one change (LTRAC-529 / TRAC-514):

1. Ticket-first PR titles broke the release pipeline

Commits using the new BigCommerce format (TICKET-123: type(scope) - subject, as enforced by @bigcommerce/validate-commits) failed in two places:

  • the commitlint gate in the release workflow
  • @semantic-release/commit-analyzer

Fix: new commit-format.cjs exports shared parserOpts accepting both the new ticket-first format and the legacy conventional format. It's wired into commitlint.config.cjs and a new release.config.cjs (replacing .releaserc, which as JSON couldn't share the regex). type-enum is now the union of config-conventional and validate-commits types.

2. LTRAC-529: bot release PRs never ran CI automatically

Release PRs opened by semantic-release-github-pullrequest-fixed with GITHUB_TOKEN don't trigger PR workflows (9.0.2 and 9.0.3 both needed manual re-runs). The plugin prefers env.GH_TOKEN_RELEASE when creating the PR, so release.yml now passes the PAT there — release PRs will be authored by the PAT user and trigger checks automatically.

Testing

  • commitlint passes both formats (verified against 7 real commit subjects from history) and still rejects malformed messages
  • commit-analyzer with the new parserOpts: TRAC-1206: feat - … → minor, feat! → major, fix → patch, chore/docs → no release
  • semantic-release cosmiconfig resolves release.config.cjs; eslint passes

Comment thread commit-format.cjs

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

This whole thing isn't strictly necessary but just a nice to have to be consistent across BC repos. Let me know if we want to keep this scoped to just the GH_TOKEN issue.

@parthshahp
parthshahp merged commit 4d672be into master Jul 23, 2026
16 checks passed
@parthshahp
parthshahp deleted the parthshahp/ltrac-529-fix-stencil-cli-github-bot-pr-workflow-run branch July 23, 2026 12:28
@github-actions

Copy link
Copy Markdown
Contributor

🎉 This PR is included in version 9.1.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants