Skip to content

Comments

feat: enable lazy barrel by default#11841

Merged
ahabhgk merged 1 commit intomainfrom
enable-lazy-barrel-by-default
Oct 11, 2025
Merged

feat: enable lazy barrel by default#11841
ahabhgk merged 1 commit intomainfrom
enable-lazy-barrel-by-default

Conversation

@ahabhgk
Copy link
Contributor

@ahabhgk ahabhgk commented Oct 11, 2025

Summary

Enable lazy barrel by default for v1.6

Related links

Checklist

  • Tests updated (or not required).
  • Documentation updated (or not required).

@ahabhgk ahabhgk requested a review from LingyuCoder as a code owner October 11, 2025 07:40
Copilot AI review requested due to automatic review settings October 11, 2025 07:40
@ahabhgk ahabhgk requested a review from hardfist as a code owner October 11, 2025 07:40
@netlify
Copy link

netlify bot commented Oct 11, 2025

Deploy Preview for rspack ready!

Name Link
🔨 Latest commit 53d9173
🔍 Latest deploy log https://app.netlify.com/projects/rspack/deploys/68ea09d4a0f9200008d86b26
😎 Deploy Preview https://deploy-preview-11841--rspack.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.

@ahabhgk ahabhgk enabled auto-merge (squash) October 11, 2025 07:40
@github-actions github-actions bot added release: feature release: feature related release(mr only) team The issue/pr is created by the member of Rspack. labels Oct 11, 2025
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR enables the lazy barrel experimental feature by default starting from version 1.6.0. The lazy barrel feature optimizes build performance by skipping the building of unused re-export modules in side-effect-free barrel files.

  • Changed the default value of experiments.lazyBarrel from false to true in the core configuration
  • Updated documentation to reflect the new default value and version information
  • Removed explicit lazyBarrel: true settings from test configurations since it's now the default

Reviewed Changes

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

Show a summary per file
File Description
packages/rspack/src/config/defaults.ts Updates the default value of experiments.lazyBarrel from false to true
tests/rspack-test/defaultsCases/default/base.js Updates test expectation to reflect the new default value
website/docs/zh/config/experiments.mdx Updates Chinese documentation with new default value and version note
website/docs/en/config/experiments.mdx Updates English documentation with new default value and version note
packages/rspack-test-tools/src/case/*.ts Removes explicit lazyBarrel: true from test configurations (8 files)

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

@github-actions
Copy link
Contributor

📦 Binary Size-limit

Comparing 53d9173 to refactor: output.charset false by default (#11837) by Fy

🙈 Size remains the same at 47.89MB

@codspeed-hq
Copy link

codspeed-hq bot commented Oct 11, 2025

CodSpeed Performance Report

Merging #11841 will not alter performance

Comparing enable-lazy-barrel-by-default (53d9173) with main (5349110)

Summary

✅ 17 untouched

@ahabhgk ahabhgk merged commit ec22ffb into main Oct 11, 2025
94 of 99 checks passed
@ahabhgk ahabhgk deleted the enable-lazy-barrel-by-default branch October 11, 2025 08:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release: feature release: feature related release(mr only) team The issue/pr is created by the member of Rspack.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants