perf(module-runner): skip cloning call sites that have no source map

packages/vite/src/module-runner/sourcemap/interceptor.ts

@@ -364,13 +364,19 @@ function wrapCallSite(frame: CallSite, state: State) {
     const line = frame.getLineNumber() ?? 0
     const column = (frame.getColumnNumber() ?? 1) - 1
 
-    const position = mapSourcePosition({
-      name: null,
-      source,
-      line,
-      column,
-    })
+    const unmapped = { name: null, source, line, column }
+    const position = mapSourcePosition(unmapped)
     state.curPosition = position
+    // Most frames have no source map, and cloning one per frame dominates the cost of
+    // building the stack. Skip it when the clone would print the same thing anyway.
+    if (
+      position === unmapped &&
+      !state.nextPosition?.name &&
+      frame.getColumnNumber() != null &&
+      frame.getScriptNameOrSourceURL() === source
+    ) {
+      return frame
+    }
     frame = cloneCallSite(frame)
     const originalFunctionName = frame.getFunctionName
     frame.getFunctionName = function () {

packages/vite/src/node/ssr/runtime/__tests__/server-source-maps.spec.ts

@@ -139,6 +139,17 @@ describe('module runner initialization', async () => {
     ])
   })
 
+  it('prints the file name of a frame without a source map', async () => {
+    // when a script's sourceURL differs from its file name, the file name is what gets printed
+    const error = await getError(() =>
+      runInThisContext(
+        '(function inVm() { throw new Error("vm") })()\n//# sourceURL=renamed.js',
+        { filename: 'original.js', displayErrors: false },
+      ),
+    )
+    expect(error.stack!.split('\n')[1]).toBe('    at inVm (original.js:1:26)')
+  })
+
   it('call site of an imported binding matches Node column', async ({
     runner,
     server,