feat: second-cut of using unenv to create a hybrid node.js compatibility setting (via inject)#5878

Merged

petebacondarwin merged 29 commits intomainfrom unenv-wrangler-inject

Jun 6, 2024

+1,140-441

Commits on Jun 6, 2024

feat: first-cut of using unenv to create a hybrid node.js compatibility setting
petebacondarwin
committed
fix up pg cloudflare socket detection
petebacondarwin
committed
fix up unenv integration, use inject instead of various hacks

IgorMinar
authored and
petebacondarwin
committed
make the side-effects of injected globals tree-shakeable

IgorMinar
authored and
petebacondarwin
committed
mark unenv as external so that it doesn't get bundled into wrangler

IgorMinar
authored and
petebacondarwin
committed
don't configure esbuild with a define to rewrite global to globalThis - use unenv instead

IgorMinar
authored and
petebacondarwin
committed
correctly provide the global polyfill if inject is defined as a string

IgorMinar
authored and
petebacondarwin
committed
fix lint issues

IgorMinar
authored and
petebacondarwin
committed
add turbo.json file

IgorMinar
authored and
petebacondarwin
committed
update the activation mechanism to be just a runtime flag 'nodejs_compat_v2' accompanied by 'experimental'

IgorMinar
authored and
petebacondarwin
committed
update to unenv-nightly@1.10.0-1717495514.cea841e

IgorMinar
authored and
petebacondarwin
committed
switch to experimenta:nodejs_compat_v2 activation flag

IgorMinar
authored and
petebacondarwin
committed
fix: ignore aliases for packages that are not installed in the current app

IgorMinar
authored and
petebacondarwin
committed
prettify

IgorMinar
authored and
petebacondarwin
committed
fixup: update unenv to 1.10.0-1717606461.a117952

IgorMinar
authored and
petebacondarwin
committed
post-review fixups
petebacondarwin
committed
Add changeset
petebacondarwin
committed
Update the fixture and add a test
petebacondarwin
committed
Separate out the require call conversion from the alias import handling in the hybrid plugin
petebacondarwin
committed
Make sure we don't match packages that don't exist
petebacondarwin
committed
Fix formatting on the new fixture test
petebacondarwin
committed
Fix deploy test snapshot for change to warning
petebacondarwin
committed
Fix fixture test to be resilient to timezone changes
petebacondarwin
committed
update the fixture to include an built-in alias

IgorMinar
authored and
petebacondarwin
committed
Match externals against the alias rather than the original path
petebacondarwin
committed
Refactor node-compat validation and handling into a helper function
petebacondarwin
committed
Remove extra node-compat validation call, which was messing things up since it mutates compat flags
petebacondarwin
committed
Use a global (Buffer) in the fixture and test it works
petebacondarwin
committed
Use named parameters for the validateNodeCompat() function
petebacondarwin
committed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

feat: second-cut of using unenv to create a hybrid node.js compatibility setting (via inject)#5878

Commits on Jun 6, 2024

feat: first-cut of using unenv to create a hybrid node.js compatibility setting

fix up pg cloudflare socket detection

fix up unenv integration, use inject instead of various hacks

make the side-effects of injected globals tree-shakeable

mark unenv as external so that it doesn't get bundled into wrangler

don't configure esbuild with a define to rewrite global to globalThis - use unenv instead

correctly provide the global polyfill if inject is defined as a string

fix lint issues

add turbo.json file

update the activation mechanism to be just a runtime flag 'nodejs_compat_v2' accompanied by 'experimental'

update to unenv-nightly@1.10.0-1717495514.cea841e

switch to experimenta:nodejs_compat_v2 activation flag

fix: ignore aliases for packages that are not installed in the current app

prettify

fixup: update unenv to 1.10.0-1717606461.a117952

post-review fixups

Add changeset

Update the fixture and add a test

Separate out the require call conversion from the alias import handling in the hybrid plugin

Make sure we don't match packages that don't exist

Fix formatting on the new fixture test

Fix deploy test snapshot for change to warning

Fix fixture test to be resilient to timezone changes

update the fixture to include an built-in alias

Match externals against the alias rather than the original path

Refactor node-compat validation and handling into a helper function

Remove extra node-compat validation call, which was messing things up since it mutates compat flags

Use a global (Buffer) in the fixture and test it works

Use named parameters for the validateNodeCompat() function