Open garethj2 opened 3 months ago
oak requires use of dynamic imports in order to support multiple runtimes. Deno Deploy creates a bundle of all code, including dynamic imports, to be able to support deployments. deno emit
should also support the same level of functionality, so there is nothing for oak to address.
I've noticed a few things with Oak that prevent it from cleanly being bundled using deno emit.
One is the use of dynamic imports. See line 704 in https://github.com/oakserver/oak/blob/main/application.ts...
Another problem is the use of
npm:
in https://github.com/oakserver/oak/blob/main/deps.ts. This second issue isn't a problem with Oak per say but rather the lack of support for processingnpm:
specifiers in Deno's core caching packages that emit uses. I've raised this as a separate issue but have raised this here just to keep everyone aware.The first issues creates an runtime error when running Deno from a bundle. The second issue prevents the bundle from ever happening in the first place.