Skip to content

Conversation

Timeless0911
Copy link
Contributor

@Timeless0911 Timeless0911 commented Sep 16, 2025

Summary

Prebundle deps in rsbuild-plugin-dts to reduce deps count.

Related Links

Checklist

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

@Copilot Copilot AI review requested due to automatic review settings September 16, 2025 10:33
Copy link
Contributor

@Copilot 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 prebundles dependencies (magic-string, picocolors, tinyglobby, and tsconfig-paths) in the plugin-dts package to optimize bundle size and performance. The prebundling approach moves these dependencies from runtime to build-time bundling.

Key changes:

  • Configures prebundling setup with external dependency mapping
  • Updates import strategy for tsconfig-paths to use namespace import
  • Moves dependencies from runtime to devDependencies

Reviewed Changes

Copilot reviewed 6 out of 7 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
packages/plugin-dts/tsconfig.json Adds path mappings for prebundled dependencies
packages/plugin-dts/src/utils.ts Changes tsconfig-paths import to namespace import for prebundling compatibility
packages/plugin-dts/rslib.config.ts Configures external mappings for prebundled dependencies
packages/plugin-dts/prebundle.config.mjs Adds prebundle configuration with dependency list and picocolors type fix
packages/plugin-dts/package.json Moves dependencies to devDependencies and adds prebundle script
packages/core/rslib.config.ts Adds comment for external dependencies configuration
Files not reviewed (1)
  • pnpm-lock.yaml: Language not supported

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@Timeless0911 Timeless0911 changed the title chore(plugin-dts): prebundle deps chore(build): prebundle deps in rsbuild-plugin-dts Sep 16, 2025
Copy link

netlify bot commented Sep 16, 2025

Deploy Preview for rslib ready!

Name Link
🔨 Latest commit e06658a
🔍 Latest deploy log https://app.netlify.com/projects/rslib/deploys/68c93ceacbc21300089af891
😎 Deploy Preview https://deploy-preview-1227--rslib.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.

@Timeless0911 Timeless0911 merged commit 4dfb2d1 into main Sep 16, 2025
14 checks passed
@Timeless0911 Timeless0911 deleted the prebundle-plugn-dts branch September 16, 2025 11:26
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.

2 participants