refactor(rspeedy/react): remove default output.inlineScripts

[colinaaa]

Summary by CodeRabbit

• Chores
  • Removed automatic inlineScripts configuration override; now relies on existing defaults
  • Cleaned up configuration comments

_{✏️ Tip: You can customize this high-level summary in your review settings.}

Checklist

• Tests updated (or not required).
• Documentation updated (or not required).
• Changeset added, and when a BREAKING CHANGE occurs, it needs to be clearly marked (or not required).

[changeset-bot]

🦋 Changeset detected

Latest commit: bf86c1e

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 0 packages

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[coderabbitai]

📝 Walkthrough

Walkthrough

This PR removes a backward-compatibility default for output.inlineScripts in the React plugin and removes a comment line from the SWC configuration. A new changeset file is added to document these changes.

Changes

Cohort / File(s)	Summary
Changeset Documentation .changeset/deep-ideas-grin.md	Added new changeset file with YAML front-matter markers
React Plugin Config packages/rspeedy/plugin-react/src/pluginReactLynx.ts, packages/rspeedy/plugin-react/src/swc.ts	Removed backward-compatibility block that defaulted output.inlineScripts to true; removed comment from SWC transform config

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

• Key attention area: Verify that removing the output.inlineScripts default doesn't break existing configs that relied on this implicit behavior. Ensure that the new default behavior (when undefined) is documented or handled appropriately in rsbuild configuration.

Possibly related PRs

• test: add tests for output.inlineScript function #1266: Adds tests for function-valued inlineScripts configuration in the same package, alongside this PR's removal of the backward-compat default.
• feat: inlineScripts defaults to false, when chunkSplit strategy is not 'all-in-one' #1504: Related changes to the inlineScripts default/backward-compatibility behavior tied to the chunkSplit strategy.

Suggested labels

framework:React

Suggested reviewers

• gaoachao

Poem

🐰 Hoppy times ahead, we say!
Old defaults hop away,
Comments cleaned, configs aligned,
A cleaner path the team will find! 🎀

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title accurately describes the main change: removing the default output.inlineScripts behavior from the rspeedy/react plugin, which aligns with the primary modification in pluginReactLynx.ts.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

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

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

📢 Thoughts on this report? Let us know!

[codspeed-hq]

CodSpeed Performance Report

Merging #1952 will degrade performances by 6.86%

_{Comparing colinaaa:colin/1124/todos (bf86c1e) with main (4695100)}

Summary

❌ 1 regression
✅ 62 untouched
⏩ 3 skipped¹

⚠️ Please fix the performance issues or acknowledge them on CodSpeed.

Benchmarks breakdown

	Benchmark	BASE	HEAD	Change
❌	basic-performance-small-css	6.9 ms	7.4 ms	-6.86%

Footnotes

1. 3 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports. ↩

[relativeci]

Web Explorer

#6297 Bundle Size — 377.49KiB (0%).

bf86c1e(current) vs 4695100 main#6283(baseline)

Bundle metrics  no changes

	Current #6297	Baseline #6283
Initial JS	146.32KiB	146.32KiB
Initial CSS	32.37KiB	32.37KiB
Cache Invalidation	0%	0%
Chunks	8	8
Assets	8	8
Modules	230	230
Duplicate Modules	16	16
Duplicate Code	2.95%	2.95%
Packages	4	4
Duplicate Packages	0	0

Bundle size by type  no changes

	Current #6297	Baseline #6283
JS	243.18KiB	243.18KiB
Other	101.94KiB	101.94KiB
CSS	32.37KiB	32.37KiB

Bundle analysis report Branch colinaaa:colin/1124/todos Project dashboard

---

^{Generated by RelativeCI Documentation Report issue}