Skip to content

Bugfix/build-in-changelog - #305

Merged
allohamora merged 2 commits into
masterfrom
bugfix/build-in-changelog
Aug 8, 2026
Merged

Bugfix/build-in-changelog#305
allohamora merged 2 commits into
masterfrom
bugfix/build-in-changelog

Conversation

@allohamora

@allohamora allohamora commented Aug 8, 2026

Copy link
Copy Markdown
Owner

Task

Summary by CodeRabbit

  • Enhancements
    • Improved release note organization with clearer categories for performance improvements, reverts, refactoring, tests, styles, documentation, build systems, continuous integration, chores, and miscellaneous changes.
    • Updated category ordering and labels for more consistent, readable release summaries.

Build commits were falling into the catch-all "Other" group. Align
group order and labels with conventional-changelog defaults, in both
the repo's own cliff.toml and the release-workflow scaffolding
template.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@allohamora
allohamora requested a lite review from Copilot August 8, 2026 16:14
@github-actions github-actions Bot changed the title fix: correct git-cliff commit group labels and add build group Bugfix/build-in-changelog Aug 8, 2026
@coderabbitai

coderabbitai Bot commented Aug 8, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

Warning

Review limit reached

@allohamora, you've reached your PR review limit, so we couldn't start this review.

Next review available in: 48 minutes

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

How can I continue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews.

How do review limits work?

CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability.

For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window.

Please refer docs for additional details.

Review details
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: ab47bcb2-726b-4d06-8916-f0e84b39691b

📥 Commits

Reviewing files that changed from the base of the PR and between ce06686 and ac81737.

📒 Files selected for processing (3)
  • __tests__/unit/categories/js/release-workflow/preset/index.spec.ts
  • cliff.toml
  • src/categories/js/release-workflow/preset/default.preset.ts
📝 Walkthrough

Walkthrough

The release workflow updates git-cliff commit categories, labels, and ordering. The default preset, standalone configuration, and unit test now include performance, revert, refactor, test, style, build, CI, chore, and fallback categories.

Changes

Git-cliff category alignment

Layer / File(s) Summary
Category configuration and validation
src/categories/js/release-workflow/preset/default.preset.ts, cliff.toml, __tests__/unit/categories/js/release-workflow/preset/index.spec.ts
The release workflow and git-cliff configuration add and reorder commit categories. Labels include Performance Improvements, Reverts, Code Refactoring, Tests, Styles, Build System, Continuous Integration, Miscellaneous Chores, and Other. The unit test updates its expected parser configuration.

Estimated code review effort: 2 (Simple) | ~10 minutes

Possibly related PRs

  • allohamora/cli#291: Both changes update git-cliff configuration and its unit test. PR #291 changes the documentation regex.
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title identifies the build changelog bugfix, which matches the primary change.
Linked Issues check ✅ Passed The changes add the build commit group in the configuration, preset, and expected test output, satisfying issue #304.
Out of Scope Changes check ✅ Passed The category reordering and label updates support alignment with conventional-changelog defaults and remain within scope.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch bugfix/build-in-changelog

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.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@src/categories/js/release-workflow/preset/default.preset.ts`:
- Around line 53-56: Restore the conventional changelog category order by
assigning Documentation 05, Styles 06, Code Refactoring 07, and Tests 08 in
src/categories/js/release-workflow/preset/default.preset.ts lines 53-56 and
cliff.toml lines 48-51; update the corresponding expected lines in
__tests__/unit/categories/js/release-workflow/preset/index.spec.ts lines 206-209
to match both configurations.
🪄 Autofix

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: d7953437-58a9-4605-838c-e40f2bddb0d4

📥 Commits

Reviewing files that changed from the base of the PR and between 3212e8d and ce06686.

📒 Files selected for processing (3)
  • __tests__/unit/categories/js/release-workflow/preset/index.spec.ts
  • cliff.toml
  • src/categories/js/release-workflow/preset/default.preset.ts

Comment thread src/categories/js/release-workflow/preset/default.preset.ts

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Updates the release-workflow changelog preset and git-cliff configuration to align commit grouping with the Conventional Commits/commitlint “config-conventional” types, specifically adding support for build commits and standardizing section names.

Changes:

  • Add a build commit parser so build: commits appear under a dedicated “Build System” section.
  • Standardize and reorder changelog group headings (e.g., “Performance Improvements”, “Code Refactoring”, “Miscellaneous Chores”).
  • Update the unit test snapshot/expectation to match the new commit_parsers output.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
src/categories/js/release-workflow/preset/default.preset.ts Updates the embedded git-cliff commit_parsers used by the release workflow preset (adds build, renames/reorders groups).
cliff.toml Mirrors the same commit_parsers changes for local/CI git-cliff usage.
tests/unit/categories/js/release-workflow/preset/index.spec.ts Updates expected output to keep the preset test in sync with the new parser configuration.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@allohamora
allohamora merged commit e64cef4 into master Aug 8, 2026
4 checks passed
@allohamora
allohamora deleted the bugfix/build-in-changelog branch August 8, 2026 16:26
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