fix: vite resolution things

Author: marvinhagemeister

packages/plugin-vite/src/plugins/commonjs.ts

@@ -6,6 +6,8 @@ import {
   Workspace,
 } from "@deno/loader";
 import * as path from "@std/path";
+import * as babel from "@babel/core";
+import { npmWorkaround } from "./patches/npm_workaround.ts";
 
 interface DenoState {
   type: RequestedModuleType;
@@ -140,6 +142,21 @@ export function deno(): Plugin {
 
       const code = new TextDecoder().decode(result.code);
 
+      // Ensure vite never sees `npm:` specifiers. The load call here
+      // can potentially insert JSX pragmas which refer to `npm:`
+      // specifiers.
+      const res = babel.transformSync(code, {
+        filename: id,
+        babelrc: false,
+        plugins: [npmWorkaround],
+      });
+      if (res?.code) {
+        return {
+          code: res.code,
+          map: res.map,
+        };
+      }
+
       return {
         code,
       };

packages/plugin-vite/src/plugins/deno.ts

@@ -1,6 +1,8 @@
 import type { Plugin } from "vite";
 import * as babel from "@babel/core";
-import { npmWorkaround } from "./npm_workaround.ts";
+import { npmWorkaround } from "./patches/npm_workaround.ts";
+import { cjsPlugin } from "./patches/commonjs.ts";
+import { jsxComments } from "./patches/jsx_comment.ts";
 
 export function patches(): Plugin {
   return {
@@ -12,7 +14,7 @@ export function patches(): Plugin {
       const res = babel.transformSync(code, {
         filename: id,
         babelrc: false,
-        plugins: [npmWorkaround],
+        plugins: [npmWorkaround, cjsPlugin, jsxComments],
       });
 
       if (res?.code) {