Skip to content

Conversation

@ryoppippi
Copy link
Member

@ryoppippi ryoppippi commented Dec 10, 2025

fixes: #194


Summary by cubic

Removed TypeScript and migrated the build to tsgo. The package now ships ESM (.mjs) output and .d.mts type declarations.

  • Dependencies
    • Updated tsdown to 0.17.2, bumped tsgo (@typescript/native-preview) to 7.0.0-dev.20251209.1, and refreshed related tooling (rolldown, rolldown-plugin-dts).
    • Removed typescript from workspace catalogs.

Written for commit 39aef3e. Summary will update automatically on new commits.

Copilot AI review requested due to automatic review settings December 10, 2025 17:18
@ryoppippi ryoppippi requested a review from a team as a code owner December 10, 2025 17:18
@pkg-pr-new
Copy link

pkg-pr-new bot commented Dec 10, 2025

Open in StackBlitz

npm i https://pkg.pr.new/StackOneHQ/stackone-ai-node/@stackone/ai@205

commit: 39aef3e

Copy link

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 modernizes the build toolchain by removing TypeScript as a direct dependency and leveraging tsgo (TypeScript Go) through the updated tsdown bundler. The change enables faster type checking and declaration generation while maintaining full TypeScript compatibility.

Key Changes:

  • Updated tsdown from ^0.15.9 to ^0.17.2, which includes support for tsgo as an alternative to the TypeScript compiler
  • Removed TypeScript from direct devDependencies (it remains available as a transitive dependency for tools that need it)
  • Updated build output extensions from .js/.d.ts to .mjs/.d.mts to better reflect ES module format

Reviewed changes

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

File Description
pnpm-workspace.yaml Updated tsdown version and removed typescript from the catalog
pnpm-lock.yaml Updated dependency resolutions for tsdown, rolldown, and related packages; typescript remains as optional peer dependency for compatible tools
package.json Updated module entry points to use .mjs and .d.mts extensions; removed typescript from devDependencies
Files not reviewed (1)
  • pnpm-lock.yaml: Language not supported

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

Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

No issues found across 3 files

@ryoppippi ryoppippi changed the title chore: remove typescript and use tsgo chore: remove typescript and use tsgo only Dec 10, 2025
@ryoppippi ryoppippi changed the title chore: remove typescript and use tsgo only chore: remove typescrip/update tsdown/update tsgo Dec 10, 2025
Copy link
Contributor

@glebedel glebedel left a comment

Choose a reason for hiding this comment

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

LGTM

@ryoppippi ryoppippi merged commit f0ef982 into main Dec 11, 2025
10 checks passed
@ryoppippi ryoppippi deleted the update-deps branch December 11, 2025 10:14
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.

update tsdown

3 participants