fix(vite): cover more commonjs cases (#3465)

Fixes #3463
Fixes #3449

Author: marvinhagemeister

deno.json

@@ -47,6 +47,7 @@
     "@std/dotenv": "jsr:@std/dotenv@^0.225.5",
     "@std/http": "jsr:@std/http@^1.0.15",
     "@std/uuid": "jsr:@std/uuid@^1.0.7",
+    "@supabase/postgrest-js": "npm:@supabase/postgrest-js@^1.21.4",
     "@types/node": "npm:@types/node@^24.3.0",
     "docsearch": "https://esm.sh/@docsearch/js@3.5.2?target=es2020",
     "esbuild": "npm:esbuild@0.25.7",
@@ -76,6 +77,7 @@
     "@std/testing": "jsr:@std/testing@^1.0.12",
 
     "@tailwindcss/postcss": "npm:@tailwindcss/postcss@^4.1.10",
+    "redis": "npm:redis@^5.8.2",
     "rollup": "npm:rollup@^4.50.0",
     "tailwindcss": "npm:tailwindcss@^4.1.10",
     "postcss": "npm:postcss@8.5.6",

deno.lock

@@ -0,0 +1,7 @@
+import * as redis from "redis";
+
+export default function Page() {
+  // deno-lint-ignore no-console
+  console.log(redis);
+  return <h1>redis</h1>;
+}

packages/plugin-vite/demo/routes/tests/redis.tsx

@@ -0,0 +1,7 @@
+import * as supabase from "@supabase/postgrest-js";
+
+export default function Page() {
+  // deno-lint-ignore no-console
+  console.log(supabase);
+  return <h1>supabase</h1>;
+}

packages/plugin-vite/demo/routes/tests/supabase_pg.tsx

@@ -53,7 +53,8 @@ export function deno(): Plugin {
     },
     async resolveId(id, importer, options) {
       if (BUILTINS.has(id)) {
-        if (!id.startsWith("node:") && BUILTINS.has(`node:${id}`)) {
+        // `node:` prefix is not included in builtins list.
+        if (!id.startsWith("node:")) {
           id = `node:${id}`;
         }
         return {

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

@@ -34,32 +34,6 @@ export function codeEvalPlugin(
             }
           },
         },
-        CallExpression: {
-          enter(path) {
-            if (
-              t.isMemberExpression(path.node.callee) &&
-              t.isIdentifier(path.node.callee.object) &&
-              t.isIdentifier(path.node.callee.property) &&
-              path.node.callee.object.name === "Object" &&
-              path.node.callee.property.name === "defineProperty" &&
-              path.node.arguments.length >= 2
-            ) {
-              const args = path.node.arguments;
-
-              if (
-                t.isIdentifier(args[0]) && args[0].name === "exports" ||
-                t.isMemberExpression(args[0]) &&
-                  t.isIdentifier(args[0].object) &&
-                  t.isIdentifier(args[0].property) &&
-                  args[0].object.name === "module" &&
-                  args[0].property.name === "exports" &&
-                  t.isStringLiteral(args[1]) && args[1].value === "native"
-              ) {
-                path.remove();
-              }
-            }
-          },
-        },
       },
     };
   };