chore: Address usability issues

- Removing `disableDebugger`
- Introducing `#lastActionTimestamp`
- Adding `--fix` flag in eslint scripts

Author: Winify

.gitignore

@@ -34,9 +34,9 @@ selenium-video-*.webm
 nightwatch-video-*.webm
 packages/nightwatch-devtools/nightwatch-video-*.webm
 
-# trace.zip output (mode: 'trace')
+# trace output (mode: 'trace')
 trace-*.zip
-trace-*/
+examples/**/trace-*/
 
 # vitest --coverage output
 coverage/

packages/core/package.json

@@ -38,7 +38,7 @@
   },
   "types": "./src/index.ts",
   "scripts": {
-    "lint": "eslint ."
+    "lint": "eslint . --fix"
   },
   "license": "MIT",
   "devDependencies": {

packages/core/src/trace-exporter.ts

@@ -165,15 +165,12 @@ function buildActionEvents(
     const endMs = Math.max(startMs + 1, rawEndMs)
     const rawArgs = cmd.args as unknown[]
     let params: Record<string, unknown>
-    if (
-      action.class === 'Element' &&
-      action.method === 'fill' &&
-      rawArgs.length >= 2
-    ) {
+    const isValueMethod = FILL_METHODS.has(action.method)
+    if (action.class === 'Element' && isValueMethod && rawArgs.length >= 2) {
       params = { selector: rawArgs[0], value: rawArgs[1] }
     } else if (
       action.class === 'Element' &&
-      action.method === 'fill' &&
+      isValueMethod &&
       rawArgs.length === 1
     ) {
       params = { value: rawArgs[0] }

packages/service/package.json

@@ -31,7 +31,7 @@
   "scripts": {
     "dev": "vite build --watch",
     "build": "tsc && vite build",
-    "lint": "eslint .",
+    "lint": "eslint . --fix",
     "prepublishOnly": "pnpm build"
   },
   "dependencies": {

packages/service/src/index.ts

@@ -219,20 +219,23 @@ export default class DevToolsHookService implements Services.ServiceInstance {
     if (this.#options.mode !== 'trace' || !this.#browser) {
       return
     }
+    const stamp = this.#lastActionTimestamp()
+    const snap = await captureActionSnapshot(this.#browser, '__final__')
+    if (snap) {
+      snap.timestamp = stamp
+      this.#actionSnapshots.push(snap)
+    }
+  }
+
+  #lastActionTimestamp(): number {
     const commands = this.#sessionCapturer.commandsLog
-    let stamp = Date.now()
     for (let i = commands.length - 1; i >= 0; i--) {
       const cmd = commands[i]!
       if (mapCommandToAction(cmd.command)) {
-        stamp = cmd.timestamp
-        break
+        return cmd.timestamp
       }
     }
-    const snap = await captureActionSnapshot(this.#browser, '__final__')
-    if (snap) {
-      snap.timestamp = stamp
-      this.#actionSnapshots.push(snap)
-    }
+    return Date.now()
   }
 
   private resetStack() {
@@ -320,16 +323,7 @@ export default class DevToolsHookService implements Services.ServiceInstance {
       ) {
         const snap = await captureActionSnapshot(this.#browser, command)
         if (snap) {
-          const commands = this.#sessionCapturer.commandsLog
-          let stamp = 0
-          for (let i = commands.length - 1; i >= 0; i--) {
-            const cmd = commands[i]!
-            if (mapCommandToAction(cmd.command)) {
-              stamp = cmd.timestamp
-              break
-            }
-          }
-          snap.timestamp = stamp
+          snap.timestamp = this.#lastActionTimestamp()
           this.#actionSnapshots.push(snap)
         }
       }

packages/service/src/launcher.ts

@@ -110,12 +110,8 @@ export class DevToolsAppLauncher {
   }
 
   async onPrepare(_: never, caps: ExtendedCapabilities[]) {
-    if (this.#options.mode === 'trace' || this.#options.disableDebugger) {
-      log.info(
-        this.#options.mode === 'trace'
-          ? 'Trace mode — skipping backend and Chrome window'
-          : 'Debugger disabled — skipping backend and Chrome window'
-      )
+    if (this.#options.mode === 'trace') {
+      log.info('Trace mode — skipping backend and Chrome window')
       return
     }
     try {

packages/service/src/types.ts

@@ -69,12 +69,6 @@ export interface ServiceOptions {
   screencast?: ScreencastOptions
   /** `live` (default) launches the DevTools UI; `trace` skips it. */
   mode?: DevToolsMode
-  /**
-   * Skip launching the devtools dashboard backend and Chrome UI window
-   * (default: false). Use when only trace recording is needed — no
-   * debug dashboard, no extra Chrome window, no backend server.
-   */
-  disableDebugger?: boolean
   /** Trace output layout — `zip` (default) writes a single archive,
    *  `ndjson-directory` unpacks into `trace-<id>/`. Only applies in trace mode. */
   traceFormat?: TraceFormat