perf: module graph url shortcuts

packages/vite/src/node/server/moduleGraph.ts

@@ -72,6 +72,13 @@ export class ModuleGraph {
     rawUrl: string,
     ssr?: boolean,
   ): Promise<ModuleNode | undefined> {
+    // Quick path, if we already have a module for this rawUrl (even without extension)
+    const cleanedUrl = removeImportQuery(removeTimestampQuery(rawUrl))
+    const mod = this.urlToModuleMap.get(cleanedUrl)
+    if (mod) {
+      return mod
+    }
+
     const [url] = await this.resolveUrl(rawUrl, ssr)
     return this.urlToModuleMap.get(url)
   }
@@ -189,8 +196,15 @@ export class ModuleGraph {
     ssr?: boolean,
     setIsSelfAccepting = true,
   ): Promise<ModuleNode> {
+    // Quick path, if we already have a module for this rawUrl (even without extension)
+    const cleanedUrl = removeImportQuery(removeTimestampQuery(rawUrl))
+    let mod = this.urlToModuleMap.get(cleanedUrl)
+    if (mod) {
+      return mod
+    }
+
     const [url, resolvedId, meta] = await this.resolveUrl(rawUrl, ssr)
-    let mod = this.idToModuleMap.get(resolvedId)
+    mod = this.idToModuleMap.get(resolvedId)
     if (!mod) {
       mod = new ModuleNode(url, setIsSelfAccepting)
       if (meta) mod.meta = meta
@@ -210,6 +224,11 @@ export class ModuleGraph {
     else if (!this.urlToModuleMap.has(url)) {
       this.urlToModuleMap.set(url, mod)
     }
+    // Also register the clean url to the module, so that we can short-circuit
+    // resolving the same url twice
+    if (!this.urlToModuleMap.has(cleanedUrl)) {
+      this.urlToModuleMap.set(cleanedUrl, mod)
+    }
     return mod
   }