arcjet / arcjet-js

Arcjet JS SDKs. Rate limiting, bot protection, email verification & attack defense for Node.js, Next.js, Bun & SvelteKit.
https://arcjet.com
Apache License 2.0
234 stars 5 forks source link

chore(tsconfig)!: Enable verbatim module syntax #1324

Closed blaine-arcjet closed 3 weeks ago

blaine-arcjet commented 1 month ago

This is a breaking change to our tsconfig.

I've enabled the verbatimModuleSyntax config which requires annotating type imports as such so they can be eliminated at build time. This required changes to the source files here and will require changes anywhere else this is used.

Closes #1314

trunk-io[bot] commented 1 month ago

😎 Merged successfully - details.

davidmytton commented 1 month ago

will require changes anywhere else this is used

What changes are needed? Do we need to provide migration guidance to SDK users?

blaine-arcjet commented 1 month ago

Do we need to provide migration guidance to SDK users?

This is not breaking to SDK users. We'll need to make updates on internal projects when we update anything consuming @arcjet/tsconfig.