Eslint fix

Author: vishnuv688

packages/app/src/app.ts

@@ -52,7 +52,10 @@ export class WebdriverIODevtoolsApplication extends Element {
   connectedCallback(): void {
     super.connectedCallback()
     window.addEventListener('load-trace', this.#loadTrace.bind(this))
-    this.addEventListener('clear-execution-data', this.#clearExecutionData.bind(this))
+    this.addEventListener(
+      'clear-execution-data',
+      this.#clearExecutionData.bind(this)
+    )
   }
 
   render() {

packages/app/src/components/sidebar/explorer.ts

@@ -124,7 +124,9 @@ export class DevtoolsSidebarExplorer extends CollapseableEntry {
     }
 
     // Clear execution data before triggering rerun
-    this.dispatchEvent(new CustomEvent('clear-execution-data', { bubbles: true, composed: true }))
+    this.dispatchEvent(
+      new CustomEvent('clear-execution-data', { bubbles: true, composed: true })
+    )
 
     const payload = {
       ...detail,
@@ -133,7 +135,6 @@ export class DevtoolsSidebarExplorer extends CollapseableEntry {
       specFile: detail.specFile || this.#deriveSpecFile(detail),
       configFile: this.#getConfigPath()
     }
-    console.log('[Explorer] Sending test run request:', payload)
     await this.#postToBackend('/api/tests/run', payload)
   }
 
@@ -199,7 +200,9 @@ export class DevtoolsSidebarExplorer extends CollapseableEntry {
     }
 
     // Clear execution data before triggering rerun
-    this.dispatchEvent(new CustomEvent('clear-execution-data', { bubbles: true, composed: true }))
+    this.dispatchEvent(
+      new CustomEvent('clear-execution-data', { bubbles: true, composed: true })
+    )
 
     void this.#postToBackend('/api/tests/run', {
       uid: '*',
@@ -327,9 +330,11 @@ export class DevtoolsSidebarExplorer extends CollapseableEntry {
     return (
       Boolean(
         ['all', 'none'].includes(this.#testFilter.filterStatus) ||
-        (entry.state === TestState.PASSED && this.#testFilter.filtersPassed) ||
-        (entry.state === TestState.FAILED && this.#testFilter.filtersFailed) ||
-        (entry.state === TestState.SKIPPED && this.#testFilter.filtersSkipped)
+          (entry.state === TestState.PASSED &&
+            this.#testFilter.filtersPassed) ||
+          (entry.state === TestState.FAILED &&
+            this.#testFilter.filtersFailed) ||
+          (entry.state === TestState.SKIPPED && this.#testFilter.filtersSkipped)
       ) &&
       (!this.#testFilter.filterQuery ||
         entryLabelIncludingChildren

packages/app/src/controller/DataManager.ts

@@ -187,14 +187,21 @@ export class DataManagerController implements ReactiveController {
       : ([data] as Record<string, SuiteStatsFragment>[])
 
     for (const chunk of payloads) {
-      if (!chunk) continue
+      if (!chunk) {
+        continue
+      }
 
       for (const suite of Object.values(chunk)) {
-        if (!suite?.start) continue
+        if (!suite?.start) {
+          continue
+        }
 
-        const suiteStartTime = suite.start instanceof Date
-          ? suite.start.getTime()
-          : (typeof suite.start === 'number' ? suite.start : 0)
+        const suiteStartTime =
+          suite.start instanceof Date
+            ? suite.start.getTime()
+            : typeof suite.start === 'number'
+              ? suite.start
+              : 0
 
         // New run detected if we see a newer start timestamp
         if (suiteStartTime > this.#lastSeenRunTimestamp) {
@@ -264,46 +271,44 @@ export class DataManagerController implements ReactiveController {
 
     const suiteMap = new Map<string, SuiteStatsFragment>()
 
-    console.log('[DataManager] Suites update - existing suites:', this.suitesContextProvider.value?.length || 0)
-
     // Populate with existing suites (keeps test list visible)
     ;(this.suitesContextProvider.value || []).forEach((chunk) => {
       Object.entries(chunk as Record<string, SuiteStatsFragment>).forEach(
         ([uid, suite]) => {
           if (suite?.uid) {
             suiteMap.set(uid, suite)
-            console.log('[DataManager] Added existing suite to map:', uid, suite.title)
           }
         }
       )
     })
 
-    console.log('[DataManager] Incoming payloads:', payloads.length)
-
     // Process incoming payloads
     payloads.forEach((chunk) => {
-      if (!chunk) return
+      if (!chunk) {
+        return
+      }
 
       Object.entries(chunk).forEach(([uid, suite]) => {
-        if (!suite?.uid) return
+        if (!suite?.uid) {
+          return
+        }
 
-        console.log('[DataManager] Processing incoming suite:', uid, suite.title)
         const existing = suiteMap.get(uid)
 
         // Always merge to preserve all tests in the suite
         suiteMap.set(uid, existing ? this.#mergeSuite(existing, suite) : suite)
       })
     })
 
-    console.log('[DataManager] Final suite map size:', suiteMap.size)
-
     this.suitesContextProvider.setValue(
       Array.from(suiteMap.entries()).map(([uid, suite]) => ({ [uid]: suite }))
     )
   }
 
   #getTimestamp(date: Date | number | undefined): number {
-    if (!date) return 0
+    if (!date) {
+      return 0
+    }
     return date instanceof Date ? date.getTime() : date
   }
 
@@ -330,10 +335,14 @@ export class DataManagerController implements ReactiveController {
 
     // First merge tests and suites properly
     const mergedTests = this.#mergeTests(existing.tests, incoming.tests)
-    const mergedSuites = this.#mergeChildSuites(existing.suites, incoming.suites)
+    const mergedSuites = this.#mergeChildSuites(
+      existing.suites,
+      incoming.suites
+    )
 
     // Then merge suite properties, ensuring merged tests/suites are preserved
-    const { tests: _incomingTests, suites: _incomingSuites, ...incomingProps } = incoming
+    // eslint-disable-next-line @typescript-eslint/no-unused-vars
+    const { tests, suites, ...incomingProps } = incoming
 
     return {
       ...existing,
@@ -351,23 +360,24 @@ export class DataManagerController implements ReactiveController {
     prev?.forEach((suite) => suite && map.set(suite.uid, suite))
 
     next?.forEach((suite) => {
-      if (!suite) return
+      if (!suite) {
+        return
+      }
       const existing = map.get(suite.uid)
       map.set(suite.uid, existing ? this.#mergeSuite(existing, suite) : suite)
     })
 
     return Array.from(map.values())
   }
 
-  #mergeTests(
-    prev: TestStatsFragment[] = [],
-    next: TestStatsFragment[] = []
-  ) {
+  #mergeTests(prev: TestStatsFragment[] = [], next: TestStatsFragment[] = []) {
     const map = new Map<string, TestStatsFragment>()
     prev?.forEach((test) => test && map.set(test.uid, test))
 
     next?.forEach((test) => {
-      if (!test) return
+      if (!test) {
+        return
+      }
       const existing = map.get(test.uid)
 
       // Check if this test is a rerun (different start time)
@@ -378,7 +388,10 @@ export class DataManagerController implements ReactiveController {
         this.#getTimestamp(test.start) !== this.#getTimestamp(existing.start)
 
       // Replace on rerun, merge on normal update
-      map.set(test.uid, isRerun ? test : existing ? { ...existing, ...test } : test)
+      map.set(
+        test.uid,
+        isRerun ? test : existing ? { ...existing, ...test } : test
+      )
     })
 
     return Array.from(map.values())

packages/backend/src/runner.ts

@@ -45,21 +45,12 @@ const FRAMEWORK_FILTERS: Record<
 > = {
   cucumber: ({ specArg, payload }) => {
     const filters: string[] = []
-    console.log('[Runner] Cucumber filter - payload:', {
-      entryType: payload.entryType,
-      suiteType: payload.suiteType,
-      featureFile: payload.featureFile,
-      featureLine: payload.featureLine,
-      fullTitle: payload.fullTitle,
-      specArg
-    })
 
     // For feature-level suites, run the entire feature file
     if (payload.suiteType === 'feature' && specArg) {
       // Remove any line number from specArg for feature-level execution
       const featureFile = specArg.split(':')[0]
       filters.push('--spec', featureFile)
-      console.log('[Runner] Feature-level execution - running entire file:', featureFile)
       return filters
     }
 
@@ -84,7 +75,10 @@ const FRAMEWORK_FILTERS: Record<
         }
         // Use regex to match the exact "rowNumber: scenarioName" pattern
         // This ensures we only run that specific example row
-        filters.push('--cucumberOpts.name', `^${rowNumber}:\\s*${escapeRegex(scenarioName.trim())}$`)
+        filters.push(
+          '--cucumberOpts.name',
+          `^${rowNumber}:\\s*${escapeRegex(scenarioName.trim())}$`
+        )
         return filters
       }
       // No row number - use plain name filter

packages/service/src/launcher.ts

@@ -43,7 +43,7 @@ export class DevToolsAppLauncher {
       this.#updateCapabilities(caps, {
         port,
         hostname: this.#options.hostname || 'localhost'
-     })
+      })
       this.#browser = await remote({
         automationProtocol: 'devtools',
         capabilities: {