fix: normalize test body invocationDetails from stack traces

cli/CHANGELOG.md

@@ -7,6 +7,7 @@ _Released 11/18/2025 (PENDING)_
 
 - Fixed an issue where [`cy.wrap()`](https://docs.cypress.io/api/commands/wrap) would cause infinite recursion and freeze the Cypress App when called with objects containing circular references. Fixes [#24715](https://github.com/cypress-io/cypress/issues/24715). Addressed in [#32917](https://github.com/cypress-io/cypress/pull/32917).
 - Fixed an issue where top changes on test retries could cause attempt numbers to show up more than one time in the reporter and cause attempts to be lost in Test Replay. Addressed in [#32888](https://github.com/cypress-io/cypress/pull/32888).
+- Fixed an issue where stack traces that are used to determine a test's invocation details are sometimes incorrect. Addressed in [#32699](https://github.com/cypress-io/cypress/pull/32699)
 
 **Misc:**
 

packages/app/cypress/e2e/runner/reporter.hooks.cy.ts

@@ -38,25 +38,69 @@ describe('hooks', {
     cy.contains('after all (2)').closest('.collapsible').should('contain', 'afterHook 1')
   })
 
-  it('creates open in IDE button', () => {
-    loadSpec({
-      filePath: 'hooks/basic.cy.js',
-      passCount: 2,
-      hasPreferredIde: true,
+  describe('open in IDE button', () => {
+    it('sends the correct invocation details for before hook', () => {
+      loadSpec({
+        filePath: 'hooks/basic.cy.js',
+        passCount: 2,
+        hasPreferredIde: true,
+      })
+
+      cy.contains('tests 1').click()
+
+      cy.get('.hook-open-in-ide').should('have.length', 4)
+
+      cy.withCtx((ctx, o) => {
+        o.sinon.stub(ctx.actions.file, 'openFile')
+      })
+
+      cy.contains('before all').closest('.hook-header').find('.hook-open-in-ide').invoke('show').click()
+
+      cy.withCtx((ctx, o) => {
+        expect(ctx.actions.file.openFile).to.have.been.calledWith(o.sinon.match(new RegExp(`hooks/basic\.cy\.js$`)), 2, 2)
+      })
     })
 
-    cy.contains('tests 1').click()
+    it('sends the correct invocation details for basic test body', () => {
+      loadSpec({
+        filePath: 'hooks/basic.cy.js',
+        passCount: 2,
+        hasPreferredIde: true,
+      })
+
+      cy.contains('tests 1').click()
+
+      cy.get('.hook-open-in-ide').should('have.length', 4)
+
+      cy.withCtx((ctx, o) => {
+        o.sinon.stub(ctx.actions.file, 'openFile')
+      })
 
-    cy.get('.hook-open-in-ide').should('have.length', 4)
+      cy.contains('test body').closest('.hook-header').find('.hook-open-in-ide').invoke('show').click()
 
-    cy.withCtx((ctx, o) => {
-      o.sinon.stub(ctx.actions.file, 'openFile')
+      cy.withCtx((ctx, o) => {
+        expect(ctx.actions.file.openFile).to.have.been.calledWith(o.sinon.match(new RegExp(`hooks/basic\.cy\.js$`)), 10, 2)
+      })
     })
 
-    cy.get('.hook-open-in-ide').first().invoke('show').click()
+    it.only('sends the correct invocation details for wrapped it', () => {
+      loadSpec({
+        filePath: 'hooks/wrapped-it.cy.js',
+        passCount: 2,
+        hasPreferredIde: true,
+      })
+
+      cy.contains('test 1').click()
+
+      cy.withCtx((ctx, o) => {
+        o.sinon.stub(ctx.actions.file, 'openFile')
+      })
+
+      cy.contains('test body').closest('.hook-header').find('.hook-open-in-ide').invoke('show').click()
 
-    cy.withCtx((ctx, o) => {
-      expect(ctx.actions.file.openFile).to.have.been.calledWith(o.sinon.match(new RegExp(`hooks/basic\.cy\.js$`)), 2, 2)
+      cy.withCtx((ctx, o) => {
+        expect(ctx.actions.file.openFile).to.have.been.calledWith(o.sinon.match(new RegExp(`hooks/wrapped-it\.cy\.js$`)), 5, 1)
+      })
     })
   })
 

packages/driver/cypress/e2e/cypress/invocationDetails.cy.ts

@@ -0,0 +1,157 @@
+import $stackUtils from '../../../src/cypress/stack_utils'
+import $sourceMapUtils from '../../../src/cypress/source_map_utils'
+
+describe('stack_utils getInvocationDetails', () => {
+  context('basic test invocation', () => {
+    it('correctly extracts invocation details for Chrome', { browser: 'chrome' }, function () {
+      // Chrome stack traces for test invocations start with 'at eval' or 'at Suite.eval'
+      const details = $stackUtils.getInvocationDetails(window, $sourceMapUtils.getSourceMapProjectRoot(), 'test')
+
+      expect(details).to.exist
+      expect(details.line).to.be.a('number')
+      expect(details.column).to.be.a('number')
+      expect(details.stack).to.be.a('string')
+
+      // Verify the stack is trimmed to start with the test invocation
+      // Chrome format: "at eval" or "at Suite.eval"
+      const stackLines = details.stack.split('\n')
+      const firstStackLine = stackLines.find((line) => line.trim().startsWith('at'))
+
+      expect(firstStackLine).to.exist
+      expect(firstStackLine.trim()).to.satisfy((line: string) => {
+        return line.startsWith('at eval') || line.startsWith('at Suite.eval')
+      }, 'Chrome stack should start with "at eval" or "at Suite.eval"')
+
+      // Verify that the stack was actually trimmed (should not include Cypress internals before the test invocation)
+      // The trimmed stack should start with the test invocation pattern, not internal Cypress code
+      const hasCypressInternalBeforeInvocation = stackLines.some((line, index) => {
+        const trimmedLine = line.trim()
+
+        return index < stackLines.indexOf(firstStackLine) &&
+          (trimmedLine.includes('cypress:///../driver/src/cypress/runner.ts') ||
+           trimmedLine.includes('cypress:///../driver/src/cypress/mocha.ts'))
+      })
+
+      expect(hasCypressInternalBeforeInvocation).to.be.false
+    })
+
+    it('correctly extracts invocation details for Firefox', { browser: 'firefox' }, function () {
+      // Firefox stack traces for test invocations have no function name before '@'
+      // Format: "@http://localhost:3000/__cypress/tests?p=..."
+      const details = $stackUtils.getInvocationDetails(window, $sourceMapUtils.getSourceMapProjectRoot(), 'test')
+
+      expect(details).to.exist
+      expect(details.line).to.be.a('number')
+      expect(details.column).to.be.a('number')
+      expect(details.stack).to.be.a('string')
+
+      // Verify the stack is trimmed to start with the test invocation
+      // Firefox format: "@" with empty function name before it
+      const stackLines = details.stack.split('\n')
+      const firstStackLine = stackLines.find((line) => line.includes('@'))
+
+      expect(firstStackLine).to.exist
+      const splitAtAt = firstStackLine.split('@')
+
+      expect(splitAtAt.length).to.be.greaterThan(1)
+      expect(splitAtAt[0].trim()).to.equal('', 'Firefox stack should have empty function name before @')
+
+      // Verify that the stack was actually trimmed (should not include Cypress internals before the test invocation)
+      const hasCypressInternalBeforeInvocation = stackLines.some((line, index) => {
+        return index < stackLines.indexOf(firstStackLine) &&
+          (line.includes('cypress:///../driver/src/cypress/runner.ts') ||
+           line.includes('cypress:///../driver/src/cypress/mocha.ts'))
+      })
+
+      expect(hasCypressInternalBeforeInvocation).to.be.false
+    })
+  })
+
+  context('wrapped it function', () => {
+    // Test case for when users re-define Mocha's it function
+    // This creates additional stack frames that need to be trimmed correctly
+    function myIt (name: string, optionsOrFn: any, fn?: () => void) {
+      if (fn) {
+        it(name, optionsOrFn, fn)
+      } else {
+        it(name, optionsOrFn)
+      }
+    }
+
+    myIt('correctly extracts invocation details for wrapped it in Chrome', { browser: 'chrome' }, function () {
+      const details = $stackUtils.getInvocationDetails(window, $sourceMapUtils.getSourceMapProjectRoot(), 'test')
+
+      expect(details).to.exist
+      expect(details.line).to.be.a('number')
+      expect(details.column).to.be.a('number')
+
+      // The stack should be trimmed to the actual test invocation (myIt call)
+      // not the wrapper function call
+      const stackLines = details.stack.split('\n')
+      const firstStackLine = stackLines.find((line) => line.trim().startsWith('at'))
+
+      expect(firstStackLine).to.exist
+      expect(firstStackLine.trim()).to.satisfy((line: string) => {
+        return line.startsWith('at eval') || line.startsWith('at Suite.eval')
+      }, 'Chrome stack should start with "at eval" or "at Suite.eval" even with wrapped it')
+    })
+
+    myIt('correctly extracts invocation details for wrapped it in Firefox', { browser: 'firefox' }, function () {
+      const details = $stackUtils.getInvocationDetails(window, $sourceMapUtils.getSourceMapProjectRoot(), 'test')
+
+      expect(details).to.exist
+      expect(details.line).to.be.a('number')
+      expect(details.column).to.be.a('number')
+
+      // The stack should be trimmed to the actual test invocation
+      const stackLines = details.stack.split('\n')
+      const firstStackLine = stackLines.find((line) => line.includes('@'))
+
+      expect(firstStackLine).to.exist
+      const splitAtAt = firstStackLine.split('@')
+
+      expect(splitAtAt.length).to.be.greaterThan(1)
+      expect(splitAtAt[0].trim()).to.equal('', 'Firefox stack should have empty function name before @ even with wrapped it')
+    })
+  })
+
+  context('nested describes', () => {
+    describe('outer describe', () => {
+      describe('inner describe', () => {
+        it('correctly extracts invocation details in nested describe for Chrome', { browser: 'chrome' }, function () {
+          const details = $stackUtils.getInvocationDetails(window, $sourceMapUtils.getSourceMapProjectRoot(), 'test')
+
+          expect(details).to.exist
+          expect(details.line).to.be.a('number')
+          expect(details.column).to.be.a('number')
+
+          // Stack should still be trimmed correctly even in nested describes
+          const firstStackLine = details.stack.split('\n').find((line) => line.trim().startsWith('at'))
+
+          expect(firstStackLine).to.exist
+          expect(firstStackLine.trim()).to.satisfy((line: string) => {
+            return line.startsWith('at eval') || line.startsWith('at Suite.eval')
+          }, 'Chrome stack should start with "at eval" or "at Suite.eval" in nested describes')
+        })
+
+        it('correctly extracts invocation details in nested describe for Firefox', { browser: 'firefox' }, function () {
+          const details = $stackUtils.getInvocationDetails(window, $sourceMapUtils.getSourceMapProjectRoot(), 'test')
+
+          expect(details).to.exist
+          expect(details.line).to.be.a('number')
+          expect(details.column).to.be.a('number')
+
+          // Stack should still be trimmed correctly even in nested describes
+          const stackLines = details.stack.split('\n')
+          const firstStackLine = stackLines.find((line) => line.includes('@'))
+
+          expect(firstStackLine).to.exist
+          const splitAtAt = firstStackLine.split('@')
+
+          expect(splitAtAt.length).to.be.greaterThan(1)
+          expect(splitAtAt[0].trim()).to.equal('', 'Firefox stack should have empty function name before @ in nested describes')
+        })
+      })
+    })
+  })
+})

packages/driver/src/cypress/mocha.ts

@@ -535,7 +535,7 @@ const patchSuiteAddTest = (specWindow) => {
     const test = args[0]
 
     if (!test.invocationDetails) {
-      test.invocationDetails = $stackUtils.getInvocationDetails(specWindow, $sourceMapUtils.getSourceMapProjectRoot())
+      test.invocationDetails = $stackUtils.getInvocationDetails(specWindow, $sourceMapUtils.getSourceMapProjectRoot(), 'test')
     }
 
     const ret = suiteAddTest.apply(this, args)

packages/driver/src/cypress/stack_utils.ts

@@ -60,6 +60,55 @@ const stackWithLinesRemoved = (stack, cb) => {
   return unsplitStack(messageLines, remainingStackLines)
 }
 
+const stackTrimmedToTestInvocation = (stack: string, specWindow) => {
+  const modifiedStack = stackWithLinesRemoved(stack, (lines: string[]) => {
+    // Guard against Cypress being undefined/null (can happen when users quickly reload tests)
+    if (!specWindow?.Cypress) {
+      return lines
+    }
+
+    if (specWindow.Cypress.isBrowser({ family: 'chromium' })) {
+    // There are cases where there are other lines in the stack trace before the invocation (eg. `context.it.only`, `createRunnable`, etc)
+    // The actual test invocation line starts with either 'at eval' or 'at Suite.eval',
+    // so remove all lines until we reach the test invocation line
+      return _.dropWhile(lines, (line) => {
+        return !(
+          line.trim().startsWith('at eval') ||
+          line.trim().startsWith('at Suite.eval')
+        )
+      })
+    }
+
+    if (specWindow.Cypress.isBrowser({ family: 'firefox' })) {
+      const isTestInvocationLine = (line: string) => {
+        const splitAtAt = line.split('@')
+
+        // firefox stacks traces look like:
+        // functionName@http://localhost:3000/__cypress/tests?p=cypress/support/e2e.js:444:14
+        // @http://localhost:3000/__cypress/tests?p=cypress/e2e/spec.cy.js:43:3
+        // @http://localhost:3000/__cypress/tests?p=cypress/e2e/spec.cy.js:45:12
+        // evalScripts/<@cypress:///../driver/src/cypress/script_utils.ts:38:23
+        //
+        // the actual invocation details will be at the first line with no function name
+        return splitAtAt.length > 1 && splitAtAt[0].trim().length === 0
+      }
+
+      return _.dropWhile(lines, (line) => {
+        return !isTestInvocationLine(line)
+      })
+    }
+
+    return lines
+  })
+
+  // if we removed all the lines then something went wrong. return the original stack instead
+  if (modifiedStack.trim() === 'Error') {
+    return stack
+  }
+
+  return modifiedStack
+}
+
 const stackWithLinesDroppedFromMarker = (stack, marker, includeLast = false) => {
   return stackWithLinesRemoved(stack, (lines) => {
     // drop lines above the marker
@@ -124,7 +173,7 @@ type InvocationDetails = {
 }
 
 // used to determine codeframes for hook/test/etc definitions rather than command invocations
-const getInvocationDetails = (specWindow, sourceMapProjectRoot: string): InvocationDetails | undefined => {
+const getInvocationDetails = (specWindow, sourceMapProjectRoot: string, type?: 'test'): InvocationDetails | undefined => {
   if (specWindow.Error) {
     let stack = (new specWindow.Error()).stack
 
@@ -146,6 +195,11 @@ const getInvocationDetails = (specWindow, sourceMapProjectRoot: string): Invocat
       }
     }
 
+    // if the hook is the test, we will try to remove the lines that are not the actual invocation of the test
+    if (type === 'test') {
+      stack = stackTrimmedToTestInvocation(stack, specWindow)
+    }
+
     const details: Omit<InvocationDetails, 'stack'> = getSourceDetailsForFirstLine(stack, sourceMapProjectRoot) || {}
 
     ;(details as any).stack = stack