Skip to content

fix(release): pass base to title sync files - #4505

Merged
matthew-dean merged 1 commit into
alphafrom
fix/release-title-sync-args
Aug 13, 2026
Merged

fix(release): pass base to title sync files#4505
matthew-dean merged 1 commit into
alphafrom
fix/release-title-sync-args

Conversation

@matthew-dean

Copy link
Copy Markdown
Member

Summary

  • pass the release base branch to release-metadata.js sync-files in the release-title sync job
  • add regression assertions that the workflow calls sync-files with <base> <version> <previousVersion>

Why

The auto-created alpha release PR in #4503 failed because the title-sync job called sync-files with only version and previous version, causing 5.0.0-alpha.1 to be interpreted as the release base.

Verification

  • node scripts/test-release-automation.js
  • git diff --check

@dosubot dosubot Bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label Aug 13, 2026
@coderabbitai

coderabbitai Bot commented Aug 13, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: 58b04527-efcd-4268-9e74-b862afdeaf46

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@greptile-apps

greptile-apps Bot commented Aug 13, 2026

Copy link
Copy Markdown

Greptile Summary

This PR fixes release-title synchronization by passing the pull request’s base branch as the first sync-files argument and adds regression assertions for the corrected workflow invocation.

  • Exposes the pull request base as RELEASE_BASE in the title-sync step.
  • Passes base, version, and previous version to release-metadata.js sync-files.
  • Verifies the workflow wiring in the release automation tests.

Confidence Score: 5/5

The PR appears safe to merge, with no actionable defects identified in the corrected workflow invocation or its regression assertions.

The title-sync job is restricted to pull requests targeting master or alpha, and the updated call now matches the release metadata command’s expected base, version, and previous-version argument contract.

Reviews (1): Last reviewed commit: "fix(release): pass base to title sync fi..." | Re-trigger Greptile

@matthew-dean
matthew-dean merged commit 1dc3b47 into alpha Aug 13, 2026
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant