Skip to content

feat: enable rspack native watcher by default#2472

Merged
GiveMe-A-Name merged 3 commits intomainfrom
enable-native-watcher-by-default
Sep 2, 2025
Merged

feat: enable rspack native watcher by default#2472
GiveMe-A-Name merged 3 commits intomainfrom
enable-native-watcher-by-default

Conversation

@GiveMe-A-Name
Copy link
Copy Markdown
Member

@GiveMe-A-Name GiveMe-A-Name commented Aug 15, 2025

Summary

Rspack native watcher pass all rspress e2e tests. We can enable rspack native watcher in rspress by default.

User can disable native watcher by setting RSPRESS_NATIVE_WATCHER as 'false' if we found any problems.

Related Issue

Checklist

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

Copilot AI review requested due to automatic review settings August 15, 2025 04:42
@netlify
Copy link
Copy Markdown

netlify Bot commented Aug 15, 2025

Deploy Preview for rspress-v2 ready!

Name Link
🔨 Latest commit 5974c31
🔍 Latest deploy log https://app.netlify.com/projects/rspress-v2/deploys/68b6648f5c255d0008d9ddbc
😎 Deploy Preview https://deploy-preview-2472--rspress-v2.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.

@netlify
Copy link
Copy Markdown

netlify Bot commented Aug 15, 2025

Deploy Preview for rspress ready!

Name Link
🔨 Latest commit 5acb44b
🔍 Latest deploy log https://app.netlify.com/projects/rspress/deploys/689ec457229dc10007ab0c96
😎 Deploy Preview https://deploy-preview-2472--rspress.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 79 (🔴 down 15 from production)
Accessibility: 97 (no change from production)
Best Practices: 92 (🟢 up 9 from production)
SEO: 100 (no change from production)
PWA: -
View the detailed breakdown and full score reports

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

@GiveMe-A-Name GiveMe-A-Name changed the title feat: rspack enable native watcher by default feat: enable rspack native watcher by default Aug 15, 2025
@GiveMe-A-Name GiveMe-A-Name marked this pull request as draft August 15, 2025 04:43
Copy link
Copy Markdown
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 rspack's native watcher by default while providing an environment variable override mechanism. The native watcher is expected to improve file watching performance compared to the default JavaScript-based watcher.

  • Adds rspack experiments configuration to enable native watcher by default
  • Provides RSPRESS_NATIVE_WATCHER environment variable to disable the feature when needed

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

Comment thread packages/core/src/node/initRsbuild.ts
Comment thread packages/core/src/node/initRsbuild.ts Outdated
@GiveMe-A-Name GiveMe-A-Name force-pushed the enable-native-watcher-by-default branch from a7f2fc0 to 5acb44b Compare August 15, 2025 05:23
@GiveMe-A-Name GiveMe-A-Name marked this pull request as ready for review August 15, 2025 06:48
@GiveMe-A-Name GiveMe-A-Name enabled auto-merge (squash) August 15, 2025 06:57
Comment thread packages/core/src/node/initRsbuild.ts Outdated
@GiveMe-A-Name GiveMe-A-Name marked this pull request as draft August 15, 2025 08:58
auto-merge was automatically disabled August 15, 2025 08:58

Pull request was converted to draft

Comment thread packages/core/src/node/initRsbuild.ts
@GiveMe-A-Name GiveMe-A-Name changed the title feat: enable rspack native watcher by default WIP: feat: enable rspack native watcher by default Aug 25, 2025
@GiveMe-A-Name GiveMe-A-Name changed the title WIP: feat: enable rspack native watcher by default feat: enable rspack native watcher by default Sep 1, 2025
@GiveMe-A-Name GiveMe-A-Name force-pushed the enable-native-watcher-by-default branch from 696d02e to a9130c9 Compare September 2, 2025 02:14
@GiveMe-A-Name GiveMe-A-Name marked this pull request as ready for review September 2, 2025 02:15
@SoonIter SoonIter force-pushed the enable-native-watcher-by-default branch from a9130c9 to 5974c31 Compare September 2, 2025 03:29
@GiveMe-A-Name GiveMe-A-Name merged commit cff72ce into main Sep 2, 2025
8 checks passed
@GiveMe-A-Name GiveMe-A-Name deleted the enable-native-watcher-by-default branch September 2, 2025 04:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants