Fix CHANGELOG.md for v9.0.0-beta.4 release by justin808 · Pull Request #612 · shakacode/shakapacker

justin808 · 2025-09-28T21:36:58Z

Summary

Restore missing v8.4.0 header that was accidentally removed
Consolidate beta releases into single v9.0.0-beta.4 section
Highlight breaking changes prominently at the top for better visibility

Changes Made

1. Restored v8.4.0 Release Section

Added back the missing v8.4.0 header with correct date (September 8, 2024)
Included all the changes that were in that release

2. Consolidated Beta Releases

Merged v9.0.0.beta.2 changes into v9.0.0-beta.4
Removed separate beta.2 section to reduce clutter

3. Breaking Changes Section

Added prominent ⚠️ Breaking Changes section at the top of v9.0.0-beta.4
Listed all breaking changes with clear migration instructions:
- SWC as default JavaScript transpiler
- CSS Modules named exports configuration
- Configuration rename from webpack_loader to javascript_transpiler

4. Cleanup

Removed doc-only PR (duplicate word fix)
Ensured consistent formatting for all author links and PR references
Updated version comparison links

Test Plan

Verified all PR links are valid
Checked date consistency
Confirmed breaking changes are clearly documented with migration paths

🤖 Generated with Claude Code

Summary by CodeRabbit

Documentation
- Updated changelog for v9.0.0-beta.4, outlining:
  - SWC as the default JS transpiler (Babel opt-in).
  - CSS Modules now use named exports by default; default imports no longer supported.
  - Configuration option renamed to javascript_transpiler.
  - New Rspack-based bundler option.
  - New private_output_path for server-side rendering bundles.
  - Enhanced TypeScript definitions for improved IDE support.
  - Migration guidance and revised changelog anchors/structure.

- Restore missing v8.4.0 header with correct date (September 8, 2024) - Consolidate beta releases into v9.0.0-beta.4 section - Highlight breaking changes prominently at the top with ⚠️ warning - Remove doc-only PRs from changelog - Ensure consistent formatting for authors and PR links 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>

coderabbitai · 2025-09-28T21:37:05Z

Warning

Rate limit exceeded

@justin808 has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 22 minutes and 35 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

📥 Commits

Reviewing files that changed from the base of the PR and between 69419c5 and aa4486b.

📒 Files selected for processing (1)

CHANGELOG.md (2 hunks)

Note

Other AI code review bot(s) detected

CodeRabbit has detected other AI code review bot(s) in this pull request and will avoid duplicating their findings in the review comments. This may lead to a less comprehensive review.

Walkthrough

Updates CHANGELOG.md to add a new v9.0.0-beta.4 section under Unreleased, documenting breaking changes, new options (SWC default, Rspack bundler, private_output_path), configuration rename, CSS Modules default export change, and TypeScript definition enhancements, along with migration notes and adjusted anchors.

Changes

Cohort / File(s)	Summary of Changes
Changelog updates `CHANGELOG.md`	Replaced prior Unreleased block with v9.0.0-beta.4 entry; documented SWC default, Babel optionality, CSS Modules named exports default, rename `webpack_loader`→`javascript_transpiler`, added Rspack option, `private_output_path` for SSR bundles, TypeScript definition improvements, migration guidance, and anchor adjustments.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Suggested reviewers

Judahmeek
tomdracz

Poem

A nibble of notes on the changelog page,
I thump my paws—new version on the stage!
SWC sprints, CSS names align,
Rspack hops in, the paths refine.
With tidy types and routes to trace,
This bunny signs off—ears up, swift pace! 🐇✨

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check	✅ Passed	The title clearly and succinctly describes the primary change of updating the CHANGELOG for the v9.0.0-beta.4 release and aligns with the PR contents centered on documentation fixes for that version.
Docstring Coverage	✅ Passed	No functions found in the changes. Docstring coverage check skipped.

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.}

Copilot

Pull Request Overview

This PR fixes the CHANGELOG.md for the v9.0.0-beta.4 release by consolidating beta releases and improving formatting for better clarity.

Restores the accidentally removed v8.4.0 release section with proper header and date
Consolidates multiple beta releases into a single v9.0.0-beta.4 section with prominent breaking changes
Improves documentation structure with clearer migration instructions and consistent formatting

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

Copilot · 2025-09-28T21:38:27Z

+- Fix: update webpack-dev-server to secure versions (^4.15.2 || ^5.2.2) [PR 585](https://github.com/shakacode/shakapacker/pull/585) by [justin808](https://github.com/justin808)


 ## [v8.3.0] - April 25, 2025


The date for v8.3.0 appears to be in the future (April 25, 2025). This should likely be 2024 to maintain chronological order with the other releases.

Suggested change

## [v8.3.0] - April 25, 2025

## [v8.3.0] - April 25, 2024

claude · 2025-09-28T21:40:44Z

Code Review for PR #612 - Fix CHANGELOG.md for v9.0.0-beta.4 release

✅ Strengths

Clear Breaking Changes Section: The prominent ⚠️ Breaking Changes section at the top of v9.0.0-beta.4 is excellent. It provides clear migration paths for each breaking change, making it easy for users to upgrade.
Well-Structured Migration Guidance: Each breaking change includes specific migration instructions with code examples, which is very helpful for users.
Restored Missing Content: Good catch on restoring the v8.4.0 section that was accidentally removed. This maintains the changelog's historical integrity.
Consolidated Beta Releases: Merging the beta.2 changes into beta.4 reduces clutter and makes the changelog easier to follow.

🔍 Observations & Suggestions

Date Inconsistency: The PR description mentions v8.4.0 was released on "September 8, 2024", which matches the changelog. However, there's a typo in the original changelog for v8.3.0 showing "April 25, 2025" (a future date). While not part of this PR's changes, it should be corrected to 2024.
Version Links: The comparison links at the bottom are properly updated to reflect the new version structure. ✅
PR Link Consistency: All PR links follow the same format and appear to be valid. The author attributions are consistent throughout.
Minor Formatting: Consider adding a brief note in the Unreleased section explaining that it tracks changes since v9.0.0-beta.4, to be even clearer for contributors.

✅ Compliance with Project Guidelines

The file ends with a trailing newline character as required by the project's linting rules
The changes follow the existing changelog format and conventions
No unnecessary documentation has been added

🎯 Recommendation

APPROVED - This PR effectively improves the changelog's clarity and usability. The breaking changes are prominently displayed with clear migration paths, and the restoration of the missing v8.4.0 section maintains changelog integrity. The consolidation of beta releases makes the document more readable.

Consider fixing the v8.3.0 date typo in a follow-up PR (April 25, 2025 → April 25, 2024).

Great work on making the changelog more user-friendly for those upgrading to v9! 🚀

- Remove PR 570 (subresource integrity) from v9.0.0-beta.4 - it belongs to v8.4.0 - Remove PR 576 (peer dependencies) from v9.0.0-beta.4 - it belongs to v8.4.0 - Keep PR 585 (webpack-dev-server security) only in v9.0.0-beta.4 where it belongs - Remove doc-only PR 572 from v8.4.0 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>

claude · 2025-09-28T21:44:14Z

Code Review for PR #612: Fix CHANGELOG.md for v9.0.0-beta.4 release

Great work on consolidating and organizing the changelog! The changes significantly improve readability and clarity for users upgrading to v9. Here's my detailed review:

✅ Strengths

Breaking Changes Section: Excellent decision to add the prominent ⚠️ Breaking Changes section at the top. This makes it immediately clear to users what they need to pay attention to during upgrades.
Clear Migration Paths: The migration instructions for each breaking change are well-structured with concrete code examples and clear options. The SWC migration paths are particularly helpful with both recommended and fallback options.
Version Consolidation: Consolidating the beta releases into a single v9.0.0-beta.4 section reduces clutter and makes the changelog easier to navigate.
Restored v8.4.0 Section: Good catch on restoring the missing v8.4.0 header with the correct date and changes.

🔍 Issues Found

Date Inconsistencies: There are impossible future dates in the changelog:
- v8.3.0 is dated "April 25, 2025"
- v8.2.0 is dated "March 12, 2025"
- v8.1.0 is dated "January 20, 2025"
These should all be 2024 dates since v8.4.0 was released on September 8, 2024.
PR Date Typo: In the original PR body, you mention "v9.0.0.beta.2 - September 25, 2025" which should be 2024.

📝 Suggestions

Version Link Updates: The comparison links at the bottom are correctly updated. Good attention to detail!
Formatting Consistency: The formatting is consistent throughout with proper indentation and bullet points.
Documentation References: The links to upgrade guides and additional documentation (CSS Modules Export Mode) are helpful for users needing more details.

✅ Test Coverage

As mentioned in your test plan, all PR links appear valid and breaking changes are clearly documented with migration paths.

🎯 Overall Assessment

This is a well-structured changelog update that significantly improves the documentation quality. The only critical issue is fixing those future dates from 2025 to 2024. Once those date typos are corrected, this PR will be ready to merge.

Recommendation: Please fix the date issues mentioned above, then this PR is good to go! 👍

🤖 Generated with Claude Code

- v8.4.0: Fix year from 2024 to 2025 (September 8, 2025) - v8.3.0: Fix day from April 25 to April 28, 2025 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>

coderabbitai

Actionable comments posted: 1

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 22dd6e8 and 69419c5.

📒 Files selected for processing (1)

CHANGELOG.md (2 hunks)

🧰 Additional context used

🧠 Learnings (3)

📚 Learning: 2025-09-28T02:23:21.827Z

Learnt from: CR
PR: shakacode/shakapacker#0
File: CLAUDE.md:0-0
Timestamp: 2025-09-28T02:23:21.827Z
Learning: This gem supports both webpack and rspack configurations

Applied to files:

CHANGELOG.md

📚 Learning: 2025-09-28T02:23:21.827Z

Learnt from: CR
PR: shakacode/shakapacker#0
File: CLAUDE.md:0-0
Timestamp: 2025-09-28T02:23:21.827Z
Learning: Test changes with both bundlers (webpack and rspack) when modifying core functionality

Applied to files:

CHANGELOG.md

📚 Learning: 2025-09-28T02:23:21.827Z

Learnt from: CR
PR: shakacode/shakapacker#0
File: CLAUDE.md:0-0
Timestamp: 2025-09-28T02:23:21.827Z
Learning: Applies to lib/shakapacker/**/*.rb : When changing `lib/shakapacker/**/*.rb`, run the corresponding spec at `spec/shakapacker/**/*_spec.rb`

Applied to files:

CHANGELOG.md

claude · 2025-09-28T21:47:24Z

Pull Request Review: Fix CHANGELOG.md for v9.0.0-beta.4 release

Thank you for cleaning up the CHANGELOG! This PR significantly improves its clarity and organization. Here's my review:

✅ Strengths

Excellent organization of breaking changes - The new ⚠️ Breaking Changes section at the top is much more prominent and user-friendly with clear migration paths.
Proper version consolidation - Merging the beta.2 changes into beta.4 reduces clutter and makes the changelog easier to follow.
Restored missing content - Good catch on restoring the v8.4.0 section that was accidentally removed.
Clear migration instructions - Each breaking change includes actionable migration steps for both new and existing projects.

🔍 Minor Issues Found

Date inconsistencies:
- Line 62: ## [v8.4.0] - September 8, 2025 - This should be 2024, not 2025
- Line 71: ## [v8.3.0] - April 28, 2025 - This should be 2024, not 2025
Version header formatting:
- The v9.0.0-beta.4 section is marked as "Unreleased" but has a version header. Consider either:
  - Using a placeholder date like "TBD" or
  - Keeping just "Unreleased" without the version in brackets until release

💡 Suggestions

Consider adding version links for the beta releases in the comparison links at the bottom for better navigation.
The breaking changes section is excellent - consider adding estimated migration times (e.g., "5 minutes" for simple config changes) to help users plan upgrades.

✔️ Code Quality

Clean, well-formatted markdown
Consistent formatting throughout
Proper use of headers and lists
All PR links appear to be correctly formatted

🔒 Security

No security concerns with these documentation changes

🧪 Test Coverage

Good test plan in the PR description covering link validation and consistency checks

⚡ Performance

N/A for documentation changes

Verdict: Approved with minor fixes needed ✅

This is a valuable cleanup that makes the changelog much more user-friendly. Please fix the year typos (2025 → 2024) for the v8.4.0 and v8.3.0 releases before merging.

Great work on improving the project documentation! The breaking changes section in particular will help users upgrade smoothly to v9.

🤖 Generated with Claude Code

Correct v8.4.0 release date from September 8, 2025 to September 8, 2024 to match project chronology and PR summary 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>

claude · 2025-09-28T21:50:59Z

Pull Request Review

Thank you for this important fix to the CHANGELOG. The consolidation and reorganization make the breaking changes much clearer for users upgrading to v9.0.0-beta.4.

✅ Strengths

Excellent Breaking Changes Section: The new ⚠️ Breaking Changes section at the top is prominently placed and provides clear migration paths for each change.
Restored Missing Content: Good catch on restoring the missing v8.4.0 release section that was accidentally removed.
Clear Migration Instructions: Each breaking change includes specific, actionable migration steps with code examples.
Proper File Formatting: The file correctly ends with a trailing newline character as required by project standards.

🔍 Minor Issues

Date Inconsistency: Line 69 shows "April 28, 2025" for v8.3.0, but this is a future date. Should this be "April 28, 2024"?
Beta Version Consistency: The PR consolidates into "v9.0.0-beta.4" but the comparison link format uses both hyphen and dot notation. Consider standardizing.

💡 Suggestions for Improvement

Consider Adding Examples: For the CSS Modules breaking change, it might be helpful to show before/after code examples directly in the changelog rather than just referencing the documentation.
Version Timeline: Since this consolidates multiple beta releases, consider adding a note about when beta.2 and beta.3 were released for historical reference.
Test Plan Coverage: While the PR description mentions verifying links and dates, consider also checking that all referenced PR numbers are valid and link to the correct changes.

✔️ Overall Assessment

This is a well-executed documentation fix that significantly improves the clarity of the breaking changes for users. The organization makes it easy for developers to understand what they need to do when upgrading. The restoration of the missing v8.4.0 section maintains the changelog's completeness.

Recommendation: Approve with minor suggestions - The core changes are solid and improve documentation quality. The date issue should be addressed before merging.

Great work on making the breaking changes more visible and providing clear migration paths! 👍

Correct the release date for version 8.3.0 in CHANGELOG.

claude · 2025-09-28T21:53:54Z

Code Review for PR #612: Fix CHANGELOG.md for v9.0.0-beta.4 release

✅ Overall Assessment

This PR successfully reorganizes and improves the CHANGELOG for the v9.0.0-beta.4 release. The changes make breaking changes more visible and restore missing content.

👍 Strengths

Excellent Breaking Changes Organization
- The new ⚠️ Breaking Changes section is prominently positioned and clearly formatted
- Migration paths are well-documented with specific instructions for both options
- Code examples are provided for clarity
Restored Missing Content
- Successfully restored the v8.4.0 release section with correct date
- Properly maintained historical changelog integrity
Improved Structure
- Logical grouping of related changes
- Better hierarchy with breaking changes prioritized
- Clear subsections (Added, Changed, Fixed)

🔍 Issues Found

Date Inconsistency 🚨
- Line 85: ## [v8.2.0] - March 12, 2025
- Line 91: ## [v8.1.0] - January 20, 2025
- These dates are in the future (2025) while v8.3.0 and v8.4.0 show 2024 dates
- This appears to be a typo that should be corrected to 2024

📝 Suggestions for Improvement

Consider Adding Version Release Date
- The v9.0.0-beta.4 section shows "Unreleased" but since this is a beta release, you might want to add the actual beta release date
PR Link Consistency
- Most PR links follow the format [PR XXX] but some breaking changes descriptions embed the links differently
- Consider standardizing for consistency
Documentation Links
- Good that you reference the CSS Modules documentation
- Consider adding a link to SWC migration guide if one exists

✅ Verification Checklist

✅ PR links are properly formatted and valid
✅ Version comparison links at bottom are correctly updated
✅ Breaking changes are clearly marked and documented
✅ Migration instructions are actionable
⚠️ Date consistency needs fixing (2025 → 2024)

🎯 Recommendation

Approve with minor changes - The PR achieves its goals of improving changelog clarity and restoring missing content. Please fix the date inconsistencies for v8.2.0 and v8.1.0 before merging.

Great work on making the breaking changes more visible and providing clear migration paths! 👏

🤖 Generated with Claude Code

justin808 requested a review from Copilot September 28, 2025 21:38

Copilot AI reviewed Sep 28, 2025

View reviewed changes

coderabbitai Bot requested changes Sep 28, 2025

View reviewed changes

Comment thread CHANGELOG.md Outdated

coderabbitai Bot approved these changes Sep 28, 2025

View reviewed changes

Fix release date for v8.3.0

aa4486b

Correct the release date for version 8.3.0 in CHANGELOG.

justin808 merged commit 5cc9884 into main Sep 28, 2025
53 checks passed

justin808 deleted the justin808/fix-changelog branch September 28, 2025 21:52

This was referenced Oct 8, 2025

Prepare v9.1.0 release #665

Merged

Update CHANGELOG.md with recent PRs #690

Merged

Improve upgrade documentation to clarify dual Gemfile and package.json updates #731

Merged

coderabbitai Bot mentioned this pull request Nov 9, 2025

Improve CSS Modules backward compatibility documentation (#809) #810

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Fix CHANGELOG.md for v9.0.0-beta.4 release#612

Fix CHANGELOG.md for v9.0.0-beta.4 release#612
justin808 merged 5 commits intomainfrom
justin808/fix-changelog

justin808 commented Sep 28, 2025 •

edited by coderabbitai Bot

Loading

Uh oh!

coderabbitai Bot commented Sep 28, 2025 •

edited

Loading

Rate limit exceeded

Other AI code review bot(s) detected

Uh oh!

Copilot AI left a comment

Uh oh!

Copilot AI Sep 28, 2025

Uh oh!

claude Bot commented Sep 28, 2025

Uh oh!

claude Bot commented Sep 28, 2025

Uh oh!

coderabbitai Bot left a comment

Uh oh!

Uh oh!

claude Bot commented Sep 28, 2025

Uh oh!

claude Bot commented Sep 28, 2025

Uh oh!

Uh oh!

claude Bot commented Sep 28, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

		- Fix: update webpack-dev-server to secure versions (^4.15.2 \|\| ^5.2.2) [PR 585](https://github.com/shakacode/shakapacker/pull/585) by [justin808](https://github.com/justin808)


		## [v8.3.0] - April 25, 2025

Uh oh!

Conversation

justin808 commented Sep 28, 2025 • edited by coderabbitai Bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Changes Made

1. Restored v8.4.0 Release Section

2. Consolidated Beta Releases

3. Breaking Changes Section

4. Cleanup

Test Plan

Summary by CodeRabbit

Uh oh!

coderabbitai Bot commented Sep 28, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Rate limit exceeded

Other AI code review bot(s) detected

Walkthrough

Changes

Estimated code review effort

Suggested reviewers

Poem

Pre-merge checks and finishing touches

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Uh oh!

Copilot AI Sep 28, 2025

Choose a reason for hiding this comment

Uh oh!

claude Bot commented Sep 28, 2025

Code Review for PR #612 - Fix CHANGELOG.md for v9.0.0-beta.4 release

✅ Strengths

🔍 Observations & Suggestions

✅ Compliance with Project Guidelines

🎯 Recommendation

Uh oh!

claude Bot commented Sep 28, 2025

Code Review for PR #612: Fix CHANGELOG.md for v9.0.0-beta.4 release

✅ Strengths

🔍 Issues Found

📝 Suggestions

✅ Test Coverage

🎯 Overall Assessment

Uh oh!

coderabbitai Bot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

claude Bot commented Sep 28, 2025

Pull Request Review: Fix CHANGELOG.md for v9.0.0-beta.4 release

✅ Strengths

🔍 Minor Issues Found

💡 Suggestions

✔️ Code Quality

🔒 Security

🧪 Test Coverage

⚡ Performance

Verdict: Approved with minor fixes needed ✅

Uh oh!

claude Bot commented Sep 28, 2025

Pull Request Review

✅ Strengths

🔍 Minor Issues

💡 Suggestions for Improvement

✔️ Overall Assessment

Uh oh!

Uh oh!

claude Bot commented Sep 28, 2025

Code Review for PR #612: Fix CHANGELOG.md for v9.0.0-beta.4 release

✅ Overall Assessment

👍 Strengths

🔍 Issues Found

📝 Suggestions for Improvement

✅ Verification Checklist

🎯 Recommendation

Uh oh!

Reviewers

Assignees

justin808 commented Sep 28, 2025 •

edited by coderabbitai Bot

Loading

coderabbitai Bot commented Sep 28, 2025 •

edited

Loading