experiment with esbuild/rollup for better modern distro

[bencmbrook]

As browsers gain support for all underlying features of Penumbra, we should modernize and simplify our build output to be usable with PnP, ESM, treeshaking, and reduce the amount of poly/ponyfilling. This would switch us off webpack in favor of esbuild, and build distributions with rollup.

TODO:

• Drop webpack and babel
• Update demo/example to import under new dist (can also update to type="module" while at it)
• A little more thinking on web worker best practice
• Reduce poly/ponyfilling and document poly/ponyfill instructions for package consumers (likely a separate PR)

Related Issues

• [none]

Public Changelog

[none]

Security Implications

[none]

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Updated
penumbra-example	✅ Ready (Inspect)	Visit Preview	Jun 15, 2022 at 11:59PM (UTC)

[eligrey]

Suggested change

	'process' in self && process.env.PENUMBRA_LOG_START === 'true';
	typeof process !== 'undefined' && process.env.PENUMBRA_LOG_START === 'true';

slightly more correct (e.g. globalThis.process can be different than self.process)

[bencmbrook]

SG! Also, on the right-hand side, should it be globalThis.process.env.PENUMBRA_LOG_START === 'true';?

[bencmbrook]

FWIW this was more of a type issue than a runtime concern, because the global types for node weren't added in tsconfig. I feel like that's reasonable to omit, but we'd have to update a few things - like this and all the references to the Buffer type (or we could import type { Buffer } from 'buffer' // feross package).

In any case, does this file execute in Node? I was confused when I saw it here

[eligrey]

This file executes in the browser. esbuild can inject variables like process.env.* through its define API. Here's an example of that: https://github.com/transcend-io/consent-manager-ui/blob/main/src/scripts/build.ts#L70-L75

[bencmbrook]

doing a bit of that in #229 (but not for this process.env. var) - i think that branch is better than this one so we could close this out