perf(resolver): stat absolute paths once in fast path (#229)

* test: add test for links

* add test for non js modules

* perf(resolver): stat absolute paths once in fast path

src/resolve.ts

@@ -1,4 +1,4 @@
-import { existsSync, realpathSync } from "node:fs";
+import { statSync } from "node:fs";
 import { pathToFileURL } from "node:url";
 import { joinURL } from "ufo";
 import { isAbsolute, normalize } from "pathe";
@@ -27,7 +27,7 @@ function _tryModuleResolve(
   id: string,
   url: URL,
   conditions: any,
-): any | undefined {
+): URL | undefined {
   try {
     return moduleResolve(id, url, conditions);
   } catch (error) {
@@ -49,10 +49,17 @@ function _resolve(id: string, options: ResolveOptions = {}): string {
   }
 
   // Skip resolve for absolute paths
-  if (isAbsolute(id) && existsSync(id)) {
-    // Resolve realPath and normalize slash
-    const realPath = realpathSync(fileURLToPath(id));
-    return pathToFileURL(realPath).toString();
+  if (isAbsolute(id)) {
+    try {
+      const stat = statSync(id);
+      if (stat.isFile()) {
+        return pathToFileURL(id).toString();
+      }
+    } catch (error) {
+      if (error.code !== "ENOENT") {
+        throw error;
+      }
+    }
   }
 
   // Condition set
@@ -82,7 +89,7 @@ function _resolve(id: string, options: ResolveOptions = {}): string {
     }
   }
 
-  let resolved;
+  let resolved: URL | undefined;
   for (const url of urls) {
     // Try simple resolve
     resolved = _tryModuleResolve(id, url, conditionsSet);
@@ -120,9 +127,8 @@ function _resolve(id: string, options: ResolveOptions = {}): string {
     throw error;
   }
 
-  // Resolve realPath and normalize slash
-  const realPath = realpathSync(fileURLToPath(resolved));
-  return pathToFileURL(realPath).toString();
+  // Normalize (TODO: simplify)
+  return pathToFileURL(fileURLToPath(resolved)).toString();
 }
 
 export function resolveSync(id: string, options?: ResolveOptions): string {

test.mjs

@@ -0,0 +1 @@
+console.log(import.meta.resolve.toString())

test/fixture/hello.link.mjs

@@ -0,0 +1 @@
+hello.mjs

test/fixture/test.link.txt

@@ -0,0 +1 @@
+test.txt

test/fixture/test.txt

@@ -0,0 +1 @@
+Test file

test/resolve.test.ts

@@ -36,6 +36,18 @@ describe("resolveSync", () => {
       }
     });
   }
+
+  it("follows symlinks", () => {
+    const resolved = resolveSync("./fixture/hello.link", {
+      url: import.meta.url,
+    });
+    expect(fileURLToPath(resolved)).match(/fixture\/hello\.mjs$/);
+
+    const resolved2 = resolveSync("./fixture/test.link.txt", {
+      url: import.meta.url,
+    });
+    expect(fileURLToPath(resolved2)).match(/fixture\/test.txt$/);
+  });
 });
 
 describe("resolvePathSync", () => {