Skip to content

feat: use fix-dts-default-cjs-exports to transform CJS types#1310

Merged
sxzz merged 1 commit intoegoist:mainfrom
userquin:feat-generate-cjs-types-using-fix-dts
Feb 27, 2025
Merged

feat: use fix-dts-default-cjs-exports to transform CJS types#1310
sxzz merged 1 commit intoegoist:mainfrom
userquin:feat-generate-cjs-types-using-fix-dts

Conversation

@userquin
Copy link
Contributor

@userquin userquin commented Feb 26, 2025

I extracted from this unbuild PR the logic to fix default CJS exports to this repository fix-dts-default-cjs-exports.

This PR using the Rollup plugin from the new repository replacing the current plugin here. Here unbuild PR to use the same Rollup plugin.

Looks like tsup, unbuild and pkgroll using rollup-plugin-dts.

Once this PR merged (🤞 🙏 ), I'll update the docs in my repository.

Check userquin/unbuild-cts-issue#2 and userquin/unbuild-cts-issue#3

@codesandbox
Copy link

codesandbox bot commented Feb 26, 2025

Review or Edit in CodeSandbox

Open the branch in Web EditorVS CodeInsiders

Open Preview

@vercel
Copy link

vercel bot commented Feb 26, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
tsup ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 26, 2025 5:29pm

@sxzz sxzz merged commit c654e5f into egoist:main Feb 27, 2025
4 checks passed
@userquin userquin deleted the feat-generate-cjs-types-using-fix-dts branch February 27, 2025 09:11
@belgattitude
Copy link
Contributor

Awesome work, I'd really appreciate a release with this one. Thanks in advance

vaernion pushed a commit to Arbeidstilsynet/brevgen2 that referenced this pull request Dec 3, 2025
This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
| [tsup](https://tsup.egoist.dev/) ([source](https://github.com/egoist/tsup)) | [`8.4.0` -> `8.5.0`](https://renovatebot.com/diffs/npm/tsup/8.4.0/8.5.0) | [![age](https://developer.mend.io/api/mc/badges/age/npm/tsup/8.5.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://developer.mend.io/api/mc/badges/adoption/npm/tsup/8.5.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://developer.mend.io/api/mc/badges/compatibility/npm/tsup/8.4.0/8.5.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://developer.mend.io/api/mc/badges/confidence/npm/tsup/8.4.0/8.5.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) |

---

### Release Notes

<details>
<summary>egoist/tsup (tsup)</summary>

### [`v8.5.0`](https://github.com/egoist/tsup/releases/tag/v8.5.0)

[Compare Source](egoist/tsup@v8.4.0...v8.5.0)

#####    🚀 Features

-   Use `fix-dts-default-cjs-exports` to transform CJS types  -  by [@&#8203;userquin](https://github.com/userquin) in egoist/tsup#1310 [<samp>(c654e)</samp>](egoist/tsup@c654e5f)
-   Allow passing custom swc configuration to swcPlugin  -  by [@&#8203;Romakita](https://github.com/Romakita) in egoist/tsup#1313 [<samp>(fdfd5)</samp>](egoist/tsup@fdfd59a)

#####    🐞 Bug Fixes

-   Make `removeNodeProtocol` work with `shims`  -  by [@&#8203;aryaemami59](https://github.com/aryaemami59) [<samp>(769aa)</samp>](egoist/tsup@769aa49)
-   CopyPublicDir in watch mode  -  by [@&#8203;geeee](https://github.com/geeee) in egoist/tsup#1331 [<samp>(7c1e1)</samp>](egoist/tsup@7c1e13e)

#####     [View changes on GitHub](egoist/tsup@v8.4.0...v8.5.0)

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or PR is renamed to start with "rebase!".

🔕 **Ignore**: Close this PR and you won't be reminded about this update again.

---

This PR has been generated by [Renovate Bot](https://github.com/renovatebot/renovate).

Related work items: #30372
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.

3 participants