feat(node:process): add additional polyfill coverage for process

jculvey commented 1 month ago

🔗 Linked issue

❓ Type of change

[ ] 📖 Documentation (updates to the documentation, readme, or JSdoc annotations)
[ ] 🐞 Bug fix (a non-breaking change that fixes an issue)
[x] 👌 Enhancement (improving an existing functionality like performance)
[ ] ✨ New feature (a non-breaking change that adds functionality)
[ ] 🧹 Chore (updates to the build process or auxiliary tools and libraries)
[ ] ⚠️ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description

Adds additional polyfill coverage for process.

📝 Checklist

[ ] I have linked an issue or discussion.
[ ] I have updated the documentation accordingly.

jculvey commented 1 month ago

@pi0 please let me know if there's a better way to go about stubbing these or if there's anything here you like to see polyfilled differently.

pi0 commented 1 month ago

Thanks dear @jculvey for nice work on it.

One generic suggestion, if possible, can you try to refactor process assignments into one object assignment? v8 engine does not likes object reshaping (adding props) also it might help to prevent typos like https://github.com/unjs/unenv/pull/185#discussion_r1602241525

unjs / unenv