Skip to content

feat: bundle more dependencies#185

Merged
chenjiahan merged 1 commit intomainfrom
bundle_deps_0402
Apr 2, 2026
Merged

feat: bundle more dependencies#185
chenjiahan merged 1 commit intomainfrom
bundle_deps_0402

Conversation

@chenjiahan
Copy link
Copy Markdown
Member

@chenjiahan chenjiahan commented Apr 2, 2026

Copilot AI review requested due to automatic review settings April 2, 2026 08:32
Copy link
Copy Markdown

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 updates the build/package setup so more runtime libraries are bundled into the published server build, and TypeScript declarations are generated as a bundled/rolled-up .d.ts output.

Changes:

  • Add an explicit chunk name for the dynamic chokidar import to stabilize bundling output.
  • Switch the server dts configuration to a bundled declarations mode and configure ws env defines to avoid optional native deps.
  • Move previously runtime-declared deps (chokidar, connect-next, ws, @types/ws) out of dependencies (now installed for build via devDependencies) and update the lockfile accordingly.

Reviewed changes

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

File Description
src/server.ts Names the chokidar dynamic import chunk to align with existing dynamic-import bundling patterns.
rslib.config.ts Enables d.ts bundling for the server build and sets ws define flags to avoid optional native dependencies during bundling.
package.json Removes bundled libraries from dependencies, adds @microsoft/api-extractor, and keeps build-time deps in devDependencies.
pnpm-lock.yaml Reflects dependency graph changes from the packaging/build adjustments.
Files not reviewed (1)
  • pnpm-lock.yaml: Language not supported

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

@chenjiahan chenjiahan merged commit 37ef0da into main Apr 2, 2026
7 checks passed
@chenjiahan chenjiahan deleted the bundle_deps_0402 branch April 2, 2026 08:43
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