bundler tests!

[paperdave]

what i think is important

• 17 splitting tests, mostly due to missing variables / imports.
  • default/TopLevelAwaitAllowedImportWithSplitting also affected
• import def2, * as ns2 from './c' in test edgecase/ImportNamespaceAndDefault
• require inside of trycatch is inlined with an error. default/RequireWithCallInsideTry
• default/HashbangBundle we do not add hashbangs like #!/usr/bin/env bun
• top level await in ES modules breaks default/TopLevelAwaitAllowedImportWithoutSplitting as the __esm is not given an async function
• define does not work with optional chaining default/DefineOptionalChain
• require a module that export * from external.edgecase/ExternalES6ConvertedToCommonJSSimplified. complex case default/ExternalES6ConvertedToCommonJS

less important. skipped

• top level await in a commonjs module does not error. default/TopLevelAwaitForbiddenRequire
• identifier minification does not ignore comments

[colinhacks]

Some other things

• Bun.build doesn't write to disk yet?
• Set NODE_ENV=development by default unless otherwise specified
• Reflect entrypoint hierarchy in outdir. Currently building ['./index.ts', './nested/index.ts'] will flatten the file hierarchy and try to generate two <outdir>/index.js files.
• Currently [dir] resolves to absolute path instead of relative path when used in naming
• Some story around asset handling, e.g.:

import logo from "./logo.svg"

export function Logo(){
  return <img src={logo} />
}

This should either a) throw (aka require a plugin to work) or b) convert the import into an absolute path.

Later:

• Fix --server-components & integrate with bun-plugin-server-components

[Jarred-Sumner]

This should either a) throw (aka require a plugin to work) or b) convert the import into an absolute path.

B, except it should become a relative path to a copied & hashed asset or, if a --public-url is provided, it should be joined with that URL so that you can host it on a CDN.