✨ Chore: Add Namespace builtins Vite Plugin by Natemoo-re

[create-issue-branch]

Original issue description

In order to ensure that all node builtin packages are properly prefixed with the node: prefix a good solution is using the Namespace Builtins Vite plugin from Natemoo-re on github

The best part you ask? well The best part is this is a super simple thing to setup, and should not be hard at all to include within the core integration as a Vite plugin! see the code example below!

import { defineConfig } from 'astro/config';
import { builtinModules as builtins } from 'node:module';

function namespaceBuiltins() {
  return {
    name: 'namespace-builtins',
    enforce: 'pre',
    resolveId(id) {
      if (id[0] === '.' || id[0] === '/') return;

      if (builtins.includes(id)) {
        return { id: `node:${id}`, external: true };
      }
    },
  };
}

// https://astro.build/config
export default defineConfig({
  vite: {
    plugins: [namespaceBuiltins()],
  },
});

Source (Stackblitz)

closes #130

[ghost]

The preview deployment is ready. 🟢

Open Preview | Open Build Logs

Last updated at: 2024-07-01 01:16:50 CET

[ghost]

The preview deployment is ready. 🟢

Open Preview | Open Build Logs

Last updated at: 2024-07-01 01:16:30 CET

[ghost]

The preview deployment is ready. 🟢

Open Preview | Open Build Logs

Last updated at: 2024-07-01 01:16:58 CET

[jdtjenkins]

Yeah I love this, this is some top tier 🧠 shit

[Adammatthiesen]

Yeah I love this, this is some top tier 🧠 shit

I thought it was a Cool find in the Astro Discord. I believe Florian was the one who linked it recently (to deal with a CF thing)

[dreyfus92]

Finally, something small 🫡 LGTM ✅

[Adammatthiesen]

!coauthor

[github-actions]

Co-authored-by: Adam Matthiesen <30383579+Adammatthiesen@users.noreply.github.com>
Co-authored-by: Jacob Jenkins <7649031+jdtjenkins@users.noreply.github.com>
Co-authored-by: Paul Valladares <85648028+dreyfus92@users.noreply.github.com>