Skip to content

chore(rsbuild-plugin): migrate to rslib bundleless#4351

Merged
2heal1 merged 9 commits intomainfrom
rslib/rsbuild-plugin
Feb 10, 2026
Merged

chore(rsbuild-plugin): migrate to rslib bundleless#4351
2heal1 merged 9 commits intomainfrom
rslib/rsbuild-plugin

Conversation

@ScriptedAlchemy
Copy link
Copy Markdown
Member

Per-package rslib bundleless migration for rsbuild-plugin.

@changeset-bot
Copy link
Copy Markdown

changeset-bot bot commented Feb 2, 2026

⚠️ No Changeset found

Latest commit: 05eb0b6

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

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

Click here to learn what changesets are, and how to add one.

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

Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 82200a71b0

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment on lines +8 to +9
bundle: false,
outBase: 'src',
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P1 Badge Preserve package.json access for bundleless build

Switching to bundle: false means the build will no longer inline non-code assets like package.json. The entry module (src/cli/index.ts) imports ../../package.json, which will resolve to dist/package.json at runtime after this change, but the output config only copies LICENSE into dist, so consumers importing the package will hit a module-not-found error when that import executes. Consider copying package.json into dist or changing the code to read the package metadata via createRequire/fs from the package root.

Useful? React with 👍 / 👎.

@netlify
Copy link
Copy Markdown

netlify bot commented Feb 2, 2026

Deploy Preview for module-federation-docs ready!

Name Link
🔨 Latest commit 05eb0b6
🔍 Latest deploy log https://app.netlify.com/projects/module-federation-docs/deploys/6989d00ba01c5b00080f2cda
😎 Deploy Preview https://deploy-preview-4351--module-federation-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Feb 7, 2026

Bundle Size Report

1 package(s) changed, 37 unchanged.

Package Total dist Delta ESM gzip Delta
rsbuild-plugin 110.9 kB +16.0 kB (+16.8%) 91 B -6137 B (-98.5%)

Total dist: 6.39 MB (+16.0 kB (+0.2%))
Total ESM gzip: 157.0 kB (-6137 B (-3.7%))

ScriptedAlchemy and others added 5 commits February 6, 2026 23:18
raven-actions/actionlint@v2 runs npm install internally which fails
on pnpm workspace repos due to workspace:* protocol. Switch to direct
binary download which has no npm dependency.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
@2heal1 2heal1 merged commit 0742c60 into main Feb 10, 2026
23 checks passed
@2heal1 2heal1 deleted the rslib/rsbuild-plugin branch February 10, 2026 08:04
@2heal1 2heal1 mentioned this pull request Feb 12, 2026
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