Package exports broken in `3.51.2`

[sondr3]

I have a project where I import a bunch of types to do some type mapping and configuration for sqlite3.capi.sqlite3_preupdate_hook. After upgrading from 3.51.1-build2 to 3.51.2-build3 these imports no longer work and TypeScript is not happy.

import sqlite3InitModule, {
	type WasmPointer,
	type BindingSpec,
	type Database,
	type SAHPoolUtil,
	type CAPI,
	type SqlValue,
	type Sqlite3Static,
} from "@sqlite.org/sqlite-wasm"

I tried doing a cursory look to figure out why, but I was unable to do so before I had to walk my dog 😅 I created a quite silly repro using the same imports I used before. If you run npm run check, TS will complain. If you then downgrade to npm add @sqlite.org/sqlite-wasm@3.51.1-build2 and run it again, it's fine.

[jurerotar]

Thank you for flagging this issue!

I'll take a look if something got crossed during the package restructure that was just done, although index.d.ts was essentially the only file that wasn't touched at all.

I'll keep you updated!

[jurerotar]

Alright, I found the issue. The difference is in the file format. In previous version, the package included index.d.ts and now we create index.d.mts during build.

index.d.ts added all declared types in to global namespace, and the new version keeps them module-scoped.

We have two solutions at this point:

• Rename index.d.mts back to index.d.ts
• Keep index.d.mts, but replace declare keyword with export.

I prefer the second solution, since the first one essentially pollutes the global namespace with types (which means more work for TypeScript to do), while the second one doesn't and still allows users to import whatever they might need.

@tomayac, would you be on board with the second solution? If yes, any specific declarations you'd want to keep private?

[tomayac]

The second solution sounds cleaner to me, too. I can release a new build tomorrow if you get the PR ready. Thank you very much!

[jurerotar]

PR ready here: #139

All previously declared types are now exported, I think this should cover everything 😄

[sondr3]

Thanks, the types now import properly

[tomayac]

Thanks for confirming, @sondr3, and thanks, @jurerotar, for the fix.