perf: pass finalizeOtherSpecifiers when only needed

rolldown · Nov 25, 2024 · e43e05b · e43e05b
1 parent 1d8282b
commit e43e05b
Showing 1 changed file with 11 additions and 3 deletions.
diff --git a/packages/vite/src/node/plugins/resolve.ts b/packages/vite/src/node/plugins/resolve.ts
@@ -308,9 +308,17 @@ export function oxcResolvePlugin(
                 }
                 return newId
               },
-          finalizeOtherSpecifiers(resolvedId, rawId) {
-            return ensureVersionQuery(resolvedId, rawId, options, depsOptimizer)
-          },
+          finalizeOtherSpecifiers: !depsOptimizer
+            ? undefined
+            : (resolvedId, rawId) => {
+                const newResolvedId = ensureVersionQuery(
+                  resolvedId,
+                  rawId,
+                  options,
+                  depsOptimizer,
+                )
+                return newResolvedId === resolvedId ? undefined : newResolvedId
+              },
         }) as unknown as Plugin
       },
     ),