Skip to content

Conversation

@benlife5
Copy link
Contributor

A few more changes that I added in the other repos

  • set the default prerelease value to 1 instead of 0 in the inc function
  • sort prereleases below the stable release

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Nov 20, 2025

Walkthrough

The changes modify version management logic in the release utilities. The getVersionChoices function now passes an additional "1" parameter to the semver.inc call when computing the next version. The getLatestTag function has been refactored to improve tag handling: it now collects all git tags, removes empty lines, strips the visual-editor@ prefix from each tag for normalization, sorts them in reverse semantic order using semver.rsort, and returns the highest version with the prefix re-applied. Previously, it relied on git's native descending sort without this normalization process.

Pre-merge checks and finishing touches

✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title 'chore: more release tag fixes' is directly related to the changes made - it addresses the release tag modifications described in the PR objectives.
Description check ✅ Passed The description accurately relates to the changeset, mentioning the two main modifications: adjusting prerelease default value and sorting prereleases below stable releases.
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch utils

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 1d50111 and b75fe60.

📒 Files selected for processing (1)
  • scripts/releaseUtils.ts (2 hunks)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (3)
  • GitHub Check: call_unit_test / unit_tests (18.x)
  • GitHub Check: call_unit_test / unit_tests (20.x)
  • GitHub Check: semgrep/ci
🔇 Additional comments (1)
scripts/releaseUtils.ts (1)

94-104: One-based prerelease numbering via identifierBase is correct

Passing "1" as the fourth argument to semver.inc correctly enables one-based prerelease numbering. This will consistently produce versions like -alpha.1, -beta.1, and -rc.1 across all pre* and prerelease bumps through the inc helper, aligning with the PR objective.

Tip

📝 Customizable high-level summaries are now available in beta!

You can now customize how CodeRabbit generates the high-level summary in your pull requests — including its content, structure, tone, and formatting.

  • Provide your own instructions using the high_level_summary_instructions setting.
  • Format the summary however you like (bullet lists, tables, multi-section layouts, contributor stats, etc.).
  • Use high_level_summary_in_walkthrough to move the summary from the description to the walkthrough section.

Example instruction:

"Divide the high-level summary into five sections:

  1. 📝 Description — Summarize the main change in 50–60 words, explaining what was done.
  2. 📓 References — List relevant issues, discussions, documentation, or related PRs.
  3. 📦 Dependencies & Requirements — Mention any new/updated dependencies, environment variable changes, or configuration updates.
  4. 📊 Contributor Summary — Include a Markdown table showing contributions:
    | Contributor | Lines Added | Lines Removed | Files Changed |
  5. ✔️ Additional Notes — Add any extra reviewer context.
    Keep each section concise (under 200 words) and use bullet or numbered lists for clarity."

Note: This feature is currently in beta for Pro-tier users, and pricing will be announced later.


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 and usage tips.

@benlife5 benlife5 merged commit d59311e into main Nov 20, 2025
13 checks passed
@benlife5 benlife5 deleted the utils branch November 20, 2025 18:08
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.

4 participants