Skip to content

optimize(ui): add the compiler name#1473

Merged
yifancong merged 3 commits intomainfrom
optimize/add-compiler-name
Jan 8, 2026
Merged

optimize(ui): add the compiler name#1473
yifancong merged 3 commits intomainfrom
optimize/add-compiler-name

Conversation

@yifancong
Copy link
Contributor

Summary

image

Related Links

Copilot AI review requested due to automatic review settings January 8, 2026 07:33
@netlify
Copy link

netlify bot commented Jan 8, 2026

Deploy Preview for rsdoctor ready!

Name Link
🔨 Latest commit 1044a26
🔍 Latest deploy log https://app.netlify.com/projects/rsdoctor/deploys/695f7262105b7c0008e85533
😎 Deploy Preview https://deploy-preview-1473--rsdoctor.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.

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 adds support for displaying the webpack/rspack configuration name in the UI. When a build configuration has a name field defined, it will be shown in the Project Overall section and used to customize the browser tab title (e.g., "Rsdoctor - webpack").

Key Changes:

  • Extracts the configuration name from the first config's name field in the data loader
  • Updates type definitions to include an optional name field in the project info response
  • Displays the name in the Project Overall component and dynamically updates the document title

Reviewed changes

Copilot reviewed 5 out of 6 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
pnpm-lock.yaml Adds empty compiled package entries (unrelated to the feature)
packages/utils/src/common/data/index.ts Extracts config name from first configuration when loading project info
packages/types/src/sdk/server/apis/project.ts Adds optional name field to GetProjectInfo API response type
packages/components/src/pages/Overall/index.tsx Passes the name prop to ProjectOverall component
packages/components/src/components/Overall/project.tsx Displays config name as "Builder" field and renames local variable to avoid conflict
packages/components/src/components/Layout/index.tsx Wraps layout with ServerAPIProvider to fetch project info and update document title based on config name
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 requested a review from fansenze January 8, 2026 09:15
@yifancong yifancong merged commit 0ea2179 into main Jan 8, 2026
7 checks passed
@yifancong yifancong deleted the optimize/add-compiler-name branch January 8, 2026 09:19
@yifancong yifancong mentioned this pull request Jan 14, 2026
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