Skip to content

feat(ssg): add printFileSize to SSG, and refactor renderPages to a rsbuildPluginSSG#2065

Merged
SoonIter merged 3 commits intomainfrom
syt/file-size-magic
Apr 15, 2025
Merged

feat(ssg): add printFileSize to SSG, and refactor renderPages to a rsbuildPluginSSG#2065
SoonIter merged 3 commits intomainfrom
syt/file-size-magic

Conversation

@SoonIter
Copy link
Member

@SoonIter SoonIter commented Apr 14, 2025

Summary

refactor renderPages to a rsbuildPluginSSG

benefits

  1. add printFileSize to SSG

  2. build within rsbuild lifecycle, can be uploaded by rsbuild-plugin-deploy, such as zephyr

image

drawbacks

Build time now includes the renderPage time. So the value will be larger than before, but it is acceptable.

Related Issue

Checklist

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

@netlify
Copy link

netlify bot commented Apr 14, 2025

Deploy Preview for rspress-v2 ready!

Name Link
🔨 Latest commit a10a347
🔍 Latest deploy log https://app.netlify.com/sites/rspress-v2/deploys/67fe0998e674e70008aa2ea0
😎 Deploy Preview https://deploy-preview-2065--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 site configuration.

@netlify
Copy link

netlify bot commented Apr 14, 2025

Deploy Preview for aquamarine-blini-95325f ready!

Name Link
🔨 Latest commit a10a347
🔍 Latest deploy log https://app.netlify.com/sites/aquamarine-blini-95325f/deploys/67fe0998fc7c1f000832761a
😎 Deploy Preview https://deploy-preview-2065--aquamarine-blini-95325f.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: 91 (🟢 up 6 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 site configuration.

@SoonIter SoonIter force-pushed the syt/file-size-magic branch from 75cb202 to b630573 Compare April 14, 2025 11:05
@SoonIter SoonIter changed the title feat: add printFileSize to SSG, and refactor renderPages to a rsbuildPlugin feat: add printFileSize to SSG, and refactor renderPages to a rsbuildPluginSSG Apr 14, 2025
@SoonIter SoonIter changed the title feat: add printFileSize to SSG, and refactor renderPages to a rsbuildPluginSSG feat(ssg): add printFileSize to SSG, and refactor renderPages to a rsbuildPluginSSG Apr 14, 2025
@SoonIter SoonIter force-pushed the syt/file-size-magic branch from 8e209a2 to 987bd08 Compare April 15, 2025 04:18
@SoonIter SoonIter force-pushed the syt/file-size-magic branch from b495b78 to c411162 Compare April 15, 2025 05:39
@SoonIter SoonIter requested review from Timeless0911, chenjiahan and Copilot and removed request for Timeless0911 April 15, 2025 07:02
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.

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

Comments suppressed due to low confidence (1)

e2e/utils/runCommands.ts:51

  • The updated regex for the 'build' command output may not reliably match the intended success marker across environments. Consider verifying that this pattern consistently detects a successful build or updating it to a more specific pattern.
build: /File (web)/,

SoonIter and others added 2 commits April 15, 2025 15:15
Co-authored-by: Timeless0911 <50201324+Timeless0911@users.noreply.github.com>
@SoonIter SoonIter requested a review from Timeless0911 April 15, 2025 08:14
@SoonIter SoonIter merged commit 547319d into main Apr 15, 2025
11 checks passed
@SoonIter SoonIter deleted the syt/file-size-magic branch April 15, 2025 08:15
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.

3 participants