Skip to content

Add tsdown#44

Merged
Minhir merged 1 commit into
mainfrom
add-tsdown
Oct 28, 2025
Merged

Add tsdown#44
Minhir merged 1 commit into
mainfrom
add-tsdown

Conversation

@Minhir

@Minhir Minhir commented Oct 28, 2025

Copy link
Copy Markdown
Owner

No description provided.

@Minhir Minhir requested a review from Copilot October 28, 2025 19:05
@Minhir Minhir merged commit 58d6e58 into main Oct 28, 2025
8 checks passed
@Minhir Minhir deleted the add-tsdown branch October 28, 2025 19:07

Copilot AI left a comment

Copy link
Copy Markdown

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 migrates the build system from TypeScript Compiler (tsc) to tsdown, modernizing the build toolchain and package configuration. The project now outputs ES modules and uses a more contemporary build tool based on rolldown.

  • Replaces TypeScript Compiler (tsc) with tsdown for building
  • Updates package.json to properly support ES modules with "type": "module"
  • Changes output directory from lib/ to dist/

Reviewed Changes

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

Show a summary per file
File Description
tsdown.config.ts Adds configuration file for the tsdown build tool, specifying entry point and build options
tsconfig.json Removes outDir option as build output is now managed by tsdown
package.json Updates module system to ESM, changes build script from tsc to tsdown, updates output paths to dist/, adds tsdown dependency, and corrects repository URL format
package-lock.json Adds tsdown and its dependencies (rolldown, babel plugins, oxc-project tools, etc.)
biome.json Removes obsolete lib directory from file includes list
.gitignore Changes ignored output directory from lib/ to dist/

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

Comment thread package.json
"types": "./lib/index.d.ts",
"module": "./dist/index.js",
"exports": {
".": "./dist/index.js",

Copilot AI Oct 28, 2025

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The package exports both 'module' and 'main' fields pointing to the same file. With '"type": "module"', the 'main' field will be treated as an ES module. Consider adding a 'types' field in the exports map (e.g., '"types": "./dist/index.d.ts"') for better TypeScript integration, especially when using modern Node.js resolution.

Suggested change
".": "./dist/index.js",
".": {
"import": "./dist/index.js",
"require": "./dist/index.js",
"types": "./dist/index.d.ts"
},

Copilot uses AI. Check for mistakes.
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