Skip to content

fix: resolve alias in DTS output when unbundle is true#955

Closed
joshuagraber wants to merge 3 commits into
rolldown:mainfrom
joshuagraber:fix/alias-not-resolved-in-dts-unbundle
Closed

fix: resolve alias in DTS output when unbundle is true#955
joshuagraber wants to merge 3 commits into
rolldown:mainfrom
joshuagraber:fix/alias-not-resolved-in-dts-unbundle

Conversation

@joshuagraber

Copy link
Copy Markdown

Fixes #833

  • This PR contains AI-generated code, but I have carefully reviewed it myself. Otherwise, my PR may be closed.
    • AI gen'd code limited to tests and refinement of what I wrote initially.

Description

Resolves issue where TS path aliases are not resolved with unbundle set to true.

Linked Issues

Additional context

  • baseUrl defaulting to . if it doesn't exist in the config is a risk, but should be a low one as TS path aliases require a baseUrl

@netlify

netlify Bot commented May 21, 2026

Copy link
Copy Markdown

Deploy Preview for tsdown-main ready!

Name Link
🔨 Latest commit 4593355
🔍 Latest deploy log https://app.netlify.com/projects/tsdown-main/deploys/6a1f0c69cff316000887378c
😎 Deploy Preview https://deploy-preview-955--tsdown-main.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@pkg-pr-new

pkg-pr-new Bot commented May 21, 2026

Copy link
Copy Markdown

Open in StackBlitz

tsdown

pnpm add https://pkg.pr.new/tsdown@955 -D
npm i https://pkg.pr.new/tsdown@955 -D
yarn add https://pkg.pr.new/tsdown@955.tgz -D

create-tsdown

pnpm add https://pkg.pr.new/create-tsdown@955 -D
npm i https://pkg.pr.new/create-tsdown@955 -D
yarn add https://pkg.pr.new/create-tsdown@955.tgz -D

@tsdown/css

pnpm add https://pkg.pr.new/@tsdown/css@955 -D
npm i https://pkg.pr.new/@tsdown/css@955 -D
yarn add https://pkg.pr.new/@tsdown/css@955.tgz -D

@tsdown/exe

pnpm add https://pkg.pr.new/@tsdown/exe@955 -D
npm i https://pkg.pr.new/@tsdown/exe@955 -D
yarn add https://pkg.pr.new/@tsdown/exe@955.tgz -D

tsdown-migrate

pnpm add https://pkg.pr.new/tsdown-migrate@955 -D
npm i https://pkg.pr.new/tsdown-migrate@955 -D
yarn add https://pkg.pr.new/tsdown-migrate@955.tgz -D

commit: 4593355

@joshuagraber

Copy link
Copy Markdown
Author

@sxzz Hi! I've raised this PR to resolve the aliasing issue described in #833. Is there a process for officially requesting review?

@sxzz

sxzz commented Jun 4, 2026

Copy link
Copy Markdown
Member

baseUrl is removed is TS 6 / 7.

@sxzz sxzz closed this Jun 4, 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.

Path aliases not resolved in DTS output with unbundle: true

2 participants