feat(node:dns): polyfill `node:dns` module - Githubissues

unjs / unenv

🕊️ Convert javaScript code to be runtime agnostic

MIT License

342 stars 18 forks source link

feat(node:dns): polyfill `node:dns` module #196

Closed jculvey closed 1 month ago

jculvey commented 1 month ago

Replaces the current auto-mocking of 'dns' to support destructured ESM imports and allow for functional polyfill coverage in the future.

🔗 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 a polyfill for the node:dns module.

Also moves the helpful notImplementedAsync helper to utils from fs so that it can be reused.

📝 Checklist

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

pi0 commented 1 month ago

Crazy idea: https://github.com/unjs/unenv/issues/198