fix: keep resolved root dir paths as is

Author: marvinhagemeister

src/resolver.ts

@@ -1,5 +1,6 @@
 import { execFile } from "node:child_process";
 import process from "node:process";
+import path from "node:path";
 import { fileURLToPath } from "node:url";
 import { execAsync } from "./utils.js";
 
@@ -182,7 +183,12 @@ export async function resolveViteSpecifier(
   cache.set(resolved.id, resolved);
 
   // Vite can load this
-  if (resolved.loader === null) return resolved.id;
+  if (
+    resolved.loader === null ||
+    !path.relative(root, resolved.id).startsWith(".")
+  ) {
+    return resolved.id;
+  }
 
   // We must load it
   return toDenoSpecifier(resolved.loader, id, resolved.id);

tests/fixture/deno.lock

@@ -0,0 +1,2 @@
+// will be transformed by a plugin
+export const value = "it doesn't work";

tests/fixture/mapped/foo.ts

@@ -0,0 +1,3 @@
+import { value } from "mapped/foo.ts";
+
+console.log(value);

tests/fixture/resolveInRootDir.ts

@@ -1,8 +1,18 @@
 import { defineConfig } from "vite";
 import deno from "../../src/index";
+import path from "node:path";
 
 export default defineConfig({
-  plugins: [deno()],
+  plugins: [deno(), {
+    name: "mapped-transform",
+    // @ts-ignore not sure
+    transform(code, id) {
+      if (id.startsWith("\0")) return;
+      if (!id.includes("mapped") || path.basename(id) !== "foo.ts") return;
+
+      return code.replace("it doesn't work", "it works");
+    },
+  }],
   build: {
     lib: {
       formats: ["es"],
@@ -17,6 +27,7 @@ export default defineConfig({
         inlineNpm: "inlineNpm.ts",
         inlineJsr: "inlineJsr.ts",
         inlineHttp: "inlineHttp.ts",
+        resolveInRootDir: "resolveInRootDir.ts",
       },
     },
   },

tests/fixture/vite.config.ts

@@ -61,4 +61,9 @@ describe("Deno plugin", () => {
       await runTest(`inlineHttp.js`);
     });
   });
+
+  // https://github.com/denoland/deno-vite-plugin/issues/42
+  it("resolve to file in root dir", async () => {
+    await runTest(`resolveInRootDir.js`);
+  });
 });