Revert "Improved the generic working of commands recorder"

This reverts commit 260ed6e.

Author: vishnuv688

packages/service/src/index.ts

@@ -10,7 +10,7 @@ import type { WebDriverCommands } from '@wdio/protocols'
 import { SessionCapturer } from './session.js'
 import { TestReporter } from './reporter.js'
 import { DevToolsAppLauncher } from './launcher.js'
-import { getBrowserObject, isUserCommand } from './utils.ts'
+import { getBrowserObject } from './utils.ts'
 import { parse } from 'stack-trace'
 import { type TraceLog, TraceType } from './types.ts'
 
@@ -156,20 +156,25 @@ export default class DevToolsHookService implements Services.ServiceInstance {
          * Smart stack filtering to detect top-level user commands
          */
         const stack = parse(new Error(''))
+        const source = stack.find((frame) =>
+            Boolean(frame.getFileName()) &&
+            !frame.getFileName()?.includes('node_modules')
+        )
 
-        if (isUserCommand(command, stack) && this.#commandStack.length === 0) {
-            const topFrame = stack.find(f =>
-                f.getFileName() && !f.getFileName()!.includes('node_modules')
-            )
+        // If the command is a selector command, we don't want to capture it
+        // as it is not a top-level user command.
+        const isSelectorCommand = command.startsWith('$') || command.includes('LocateNodes')
+
+        if (source && this.#commandStack.length === 0 && !isSelectorCommand) {
             const cmdSig = JSON.stringify({
-                command,
-                args,
-                src: topFrame?.getFileName() + ':' + topFrame?.getLineNumber()
-            })
+              command,
+              args,
+              src: source.getFileName() + ':' + source.getLineNumber()
+            });
 
             if (this.#lastCommandSig !== cmdSig) {
-                this.#commandStack.push(command)
-                this.#lastCommandSig = cmdSig
+                this.#commandStack.push(command);
+                this.#lastCommandSig = cmdSig;
             }
         }
     }

packages/service/src/utils.ts

@@ -1,22 +1,4 @@
-import type { StackFrame } from 'stack-trace'
-
 export function getBrowserObject (elem: WebdriverIO.Element | WebdriverIO.Browser): WebdriverIO.Browser {
   const elemObject = elem as WebdriverIO.Element
   return (elemObject as WebdriverIO.Element).parent ? getBrowserObject(elemObject.parent) : elem as WebdriverIO.Browser
 }
-
-/**
- * Heuristics to decide whether a command is a "user-facing" command.
- * - skip selector helpers ($, $$, shadow$, etc.)
- * - only include commands invoked from user test files (not node_modules)
- */
-export function isUserCommand(command: string, stack: StackFrame[]): boolean {
-    // skip selector helpers or internal "LocateNodes"
-    if (command.startsWith('$') || command.includes('LocateNodes')) return false
-
-    // check if stack contains at least one frame in a test file
-    return stack.some(frame =>
-        frame.getFileName() &&
-        !frame.getFileName()!.includes('node_modules')
-    )
-}