Skip to content

Conversation

@yifancong
Copy link
Contributor

@yifancong yifancong commented Nov 5, 2025

Summary

chore: rspress repo add diff CI

Demo:
#2721

Related Issue

Checklist

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

Copilot AI review requested due to automatic review settings November 5, 2025 09:51
@netlify
Copy link

netlify bot commented Nov 5, 2025

Deploy Preview for rspress-v2 ready!

Name Link
🔨 Latest commit 0a2e98f
🔍 Latest deploy log https://app.netlify.com/projects/rspress-v2/deploys/690c4ed76518670008eec2df
😎 Deploy Preview https://deploy-preview-2722--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.

@yifancong yifancong changed the title Chore/add diff action chore: rspress repo add diff CI Nov 5, 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 integrates Rsdoctor, a build analysis tool, into the Rspress website project to enable bundle size tracking and build performance monitoring. The integration is conditional (activated via RSDOCTOR environment variable) and includes CI workflow automation for analyzing build artifacts.

  • Added Rsdoctor Rspack plugin configuration to the website build setup
  • Implemented a new GitHub Actions workflow for automated build analysis on pull requests
  • Removed an unused import from the core package configuration

Reviewed Changes

Copilot reviewed 3 out of 4 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
website/rspress.config.ts Added Rsdoctor plugin integration with conditional activation and custom output configuration
website/package.json Added @rsdoctor/rspack-plugin dependency at version 1.3.7
pnpm-lock.yaml Updated lockfile with all Rsdoctor package dependencies and transitive dependencies
packages/core/rslib.config.ts Removed unused path import
.github/workflows/diff.yml Added new CI workflow for build analysis with Rsdoctor action
Files not reviewed (1)
  • pnpm-lock.yaml: Language not supported

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

@yifancong yifancong force-pushed the chore/add-diff-action branch from f675af7 to 44576fd Compare November 5, 2025 10:37
@github-actions
Copy link
Contributor

github-actions bot commented Nov 5, 2025

Rsdoctor Bundle Diff Analysis

⚠️ No baseline data found - Unable to perform comparison analysis

Metric Current Baseline Change
📊 Total Size 14.3 MB - -
📄 JavaScript 13.7 MB - -
🎨 CSS 108.8 KB - -
🌐 HTML 0 B - -
📁 Other Assets 495.8 KB - -

Generated by Rsdoctor Action

@yifancong yifancong enabled auto-merge (squash) November 5, 2025 11:46
@yifancong yifancong force-pushed the chore/add-diff-action branch from e80da4c to 6e4aa1b Compare November 5, 2025 11:48
@yifancong yifancong requested a review from SoonIter November 5, 2025 11:50
@yifancong yifancong closed this Nov 5, 2025
auto-merge was automatically disabled November 5, 2025 12:24

Pull request was closed

@yifancong yifancong reopened this Nov 5, 2025
@Timeless0911 Timeless0911 changed the title chore: rspress repo add diff CI chore: add Rsdoctor bundle diff CI Nov 6, 2025
@SoonIter SoonIter merged commit 44fa541 into main Nov 6, 2025
9 of 10 checks passed
@SoonIter SoonIter changed the title chore: add Rsdoctor bundle diff CI chore(infra): add Rsdoctor bundle diff CI Nov 6, 2025
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.

4 participants