refactor(fileSize): rename showDiff to diff#6688
Conversation
✅ Deploy Preview for rsbuild ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
ℹ️ 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".
There was a problem hiding this comment.
Pull request overview
This PR refactors the file size diff configuration by renaming showDiff to diff for a more concise API, and removes unused diff-related header calculation code.
Key changes:
- Renamed
printFileSize.showDifftoprintFileSize.diffacross type definitions, implementation, and documentation - Removed unused
maxDiffLengthandshowDiffHeaderparameters fromgetHeader()function - Added version history section documenting the
diffoption addition in v1.6.13
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| website/docs/en/config/performance/print-file-size.mdx | Updated type definition, section heading, usage examples, and added version history for the renamed diff option |
| packages/core/src/types/config.ts | Renamed showDiff property to diff in PrintFileSizeOptions type |
| packages/core/src/plugins/fileSize.ts | Updated property references from showDiff to diff, removed unused diff header parameters and logic from getHeader() function |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Summary
Related Links
Checklist