Skip to content

perf(memory): disable Rspress file size report#1060

Merged
Huxpro merged 1 commit into
mainfrom
perf/disable-rspress-file-size-report
May 29, 2026
Merged

perf(memory): disable Rspress file size report#1060
Huxpro merged 1 commit into
mainfrom
perf/disable-rspress-file-size-report

Conversation

@SoonIter
Copy link
Copy Markdown
Collaborator

@SoonIter SoonIter commented May 29, 2026

Summary

  • Disable Rspress file size reporting in the Lynx website build.
  • Add a config comment documenting that this reduces peak build memory.

Testing

  • Pre-commit hooks passed for the staged config file.
  • Not run: full npm run build was skipped because you requested direct PR submission.

Disable Rspress file size report to reduce peak build memory

Disable the Rspress file size report in the build configuration by setting printFileSize: false in rspress.config.ts, with an inline comment documenting that this reduces peak memory consumption during the build process.

Review Change Stack

Change-Id: I69840099ed218a4ccdaef210e50288fd62db9a6a
Copilot AI review requested due to automatic review settings May 29, 2026 09:34
@SoonIter SoonIter requested review from Huxpro and colinaaa as code owners May 29, 2026 09:34
@netlify
Copy link
Copy Markdown

netlify Bot commented May 29, 2026

Deploy Preview for lynx-doc ready!

Name Link
🔨 Latest commit ada5e3b
🔍 Latest deploy log https://app.netlify.com/projects/lynx-doc/deploys/6a195dba114ab70008f78970
😎 Deploy Preview https://deploy-preview-1060--lynx-doc.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

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

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 29, 2026

Walkthrough

The build configuration in rspress.config.ts is updated to disable file size report generation by adding printFileSize: false under the performance settings, with an inline comment indicating this reduces peak memory consumption during builds.

Changes

Build Performance Configuration

Layer / File(s) Summary
Disable file size report generation
rspress.config.ts
The printFileSize: false option is added under builderConfig.performance to skip generating the file size report and reduce peak memory usage during builds.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Suggested reviewers

  • Huxpro
  • colinaaa
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Title check ✅ Passed The title 'perf(memory): disable Rspress file size report' directly and clearly summarizes the main change: disabling the file size report to improve build performance/memory usage.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch perf/disable-rspress-file-size-report

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.

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 updates the Rspress/Rsbuild configuration to disable the post-build file size report output, with an inline comment noting that this can reduce peak memory usage during builds.

Changes:

  • Disable file size reporting via builderConfig.performance.printFileSize = false.
  • Add a brief configuration comment explaining the motivation (reduce peak build memory).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@SoonIter SoonIter changed the title perf: disable Rspress file size report perf(memory): disable Rspress file size report May 29, 2026
@Huxpro Huxpro merged commit 843e134 into main May 29, 2026
14 of 15 checks passed
@Huxpro Huxpro deleted the perf/disable-rspress-file-size-report branch May 29, 2026 09:43
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.

3 participants