Build with tsc instead of tsup

[nwalters512]

As reported in #836 (comment), my previous changes from #670 broke the ability for this package to be tree-shaken.

This PR takes the other approach I had proposed in #604: it runs tsc twice, once to produce a CJS build and once to produce an ESM one. It writes a package.json file containing {"type": "commonjs"} to dist/cjs/package.json so that the files in that directory are treated as CJS.

Since #670 introduced an exports field, the internal changes to paths shouldn't be visible to consumers, so this should be safe to release as a patch or minor version. That said, I wouldn't blame you if you wanted to release this as a major version to play it safe.

[codesandbox-ci]

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

[nene]

Thanks a whole lot. This looks mighty promising. Too tired today to check it out. Hopefully will manage tomorrow.

…

On Sun, Mar 16, 2025, 19:32 codesandbox-ci[bot] ***@***.***> wrote: This pull request is automatically built and testable in CodeSandbox <https://codesandbox.io>. To see build info of the built libraries, click here <https://ci.codesandbox.io/status/sql-formatter-org/sql-formatter/pr/842/builds/597741> or the icon next to each commit SHA. — Reply to this email directly, view it on GitHub <#842 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAA43OIPIOMNV6SKEWPUYDT2UWYR5AVCNFSM6AAAAABZD3SD32VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDOMRXGU2TKMJZGM> . You are receiving this because you are subscribed to this thread.Message ID: ***@***.***> [image: codesandbox-ci[bot]]*codesandbox-ci[bot]* left a comment (sql-formatter-org/sql-formatter#842) <#842 (comment)> This pull request is automatically built and testable in CodeSandbox <https://codesandbox.io>. To see build info of the built libraries, click here <https://ci.codesandbox.io/status/sql-formatter-org/sql-formatter/pr/842/builds/597741> or the icon next to each commit SHA. — Reply to this email directly, view it on GitHub <#842 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAA43OIPIOMNV6SKEWPUYDT2UWYR5AVCNFSM6AAAAABZD3SD32VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDOMRXGU2TKMJZGM> . You are receiving this because you are subscribed to this thread.Message ID: ***@***.***> [ { ***@***.***": "http://schema.org", ***@***.***": "EmailMessage", "potentialAction": { ***@***.***": "ViewAction", "target": " #842 (comment)", "url": " #842 (comment)", "name": "View Pull Request" }, "description": "View this Pull Request on GitHub", "publisher": { ***@***.***": "Organization", "name": "GitHub", "url": " https://github.com" } } ]

[nene]

Hmm... I think this doesn't work quite as intended. I generates a file containing:

{type: commonjs}

which isn't valid JSON. I tweaked it a bit: 14e5daf

[nwalters512]

Great catch! Sorry about that.

[nene]

So I merged this in and released as 15.5.0-beta.1. Then fixed the package.json issue and released 15.5.0-beta.2.