feat/server-esm

[MasterKale]

This PR aims to add an ESM build to @simplewebauthn/server, alongside the existing CJS build.

Fixes #338.

[nicksrandall]

Suggested change

	"moduleResolution": "node16",
	"moduleResolution": "bundler",

You might find that the bundler option works well here for your use case.

[MasterKale]

Why can't I find any docs on how bundler is different from node16? Not even the TypeScript docs explain the new option:

https://www.typescriptlang.org/tsconfig#moduleResolution

Can you shed any light on how this will help this ESM effort?

[nicksrandall]

https://devblogs.microsoft.com/typescript/announcing-typescript-5-0/#moduleresolution-bundler

It allows you to optionally omit the file extension in imports but keep the other benefits of node16.

[MasterKale]

https://devblogs.microsoft.com/typescript/announcing-typescript-5-0/#moduleresolution-bundler

Oh my god you have saved me so much time, thank you! 🤩

Edit: Tempering my enthusiasm a bit, bundler might not be a silver bullet, but it's still worth looking into:

On the other hand, if you’re writing a library that’s meant to be published on npm, using the bundler option can hide compatibility issues that may arise for your users who aren’t using a bundler. So in these cases, using the node16 or nodenext resolution options is likely to be a better path.

[MasterKale]

Superseded by #425.

[spendres]

Consider renaming deno directory to esm.

…

On Mon, Aug 21, 2023 at 8:15 PM Matthew Miller ***@***.***> wrote: Superseded by #425 <#425> . — Reply to this email directly, view it on GitHub <#366 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAEVLYFWA2KURYSYIVDVVJDXWP2Z3ANCNFSM6AAAAAAVFF7HBA> . You are receiving this because you are subscribed to this thread.Message ID: ***@***.***>