Fix format

Author: ergunsh

src/logger.ts

@@ -38,7 +38,7 @@ export function saveLogsToFileSync(fileName: string): void {
   let fd: number | undefined;
   try {
     fd = fs.openSync(fileName, 'a+');
-  } catch (error: any) {
+  } catch (error) {
     console.error(`Error when opening log file: ${error.message}`);
     process.exit(1);
   }
@@ -48,7 +48,6 @@ export function saveLogsToFileSync(fileName: string): void {
       try {
         fs.writeSync(fd, `${chunks.join(' ')}\n`);
       } catch (e) {
-
       }
     }
   };

src/telemetry/types.ts

@@ -16,7 +16,7 @@ export interface ChromeDevToolsMcpExtension {
   server_shutdown?: ServerShutdown;
 }
 
-export interface ServerShutdown {}
+export type ServerShutdown = Record<string, never>;
 
 export interface ToolInvocation {
   tool_name: string;

src/telemetry/watchdog/main.ts

@@ -12,7 +12,8 @@ import yargs from 'yargs';
 import {hideBin} from 'yargs/helpers';
 
 import {logger, saveLogsToFileSync} from '../../logger.js';
-import {IpcMessageType, OsType} from '../types.js';
+import type { OsType} from '../types.js';
+import {IpcMessageType} from '../types.js';
 
 import {ClearcutSender} from './clearcut-sender.js';
 
@@ -83,7 +84,7 @@ async function main() {
 
   rl.on('line', (line) => {
     try {
-      if (!line.trim()) return;
+      if (!line.trim()) {return;}
 
       const msg = JSON.parse(line);
       if (msg.type === IpcMessageType.EVENT && msg.payload) {

tests/e2e/fixtures/server-fixture.ts

@@ -1,15 +1,23 @@
 
-import {hideBin} from 'yargs/helpers';
-import yargs from 'yargs';
+/**
+ * @license
+ * Copyright 2026 Google LLC
+ * SPDX-License-Identifier: Apache-2.0
+ */
+
 import process from 'node:process';
-import {ClearcutLogger} from '../../../src/telemetry/clearcut-logger.js';
+
+import yargs from 'yargs';
+import {hideBin} from 'yargs/helpers';
+
 import {saveLogsToFile} from '../../../src/logger.js';
+import {ClearcutLogger} from '../../../src/telemetry/clearcut-logger.js';
 
 async function main() {
   const argv = await yargs(hideBin(process.argv))
     .option('action', {
       type: 'string',
-      choices: ['exit-0', 'exit-1', 'crash', 'stay-alive'],
+      choices: ['exit-0', 'exit-1', 'crash'],
       demandOption: true,
     })
     .option('log-file', {
@@ -44,10 +52,6 @@ async function main() {
       case 'crash':
         console.log('Crashing');
         throw new Error('Planned crash');
-      case 'stay-alive':
-        console.log('Staying alive');
-        setInterval(() => {}, 1000); // Keep alive
-        break;
     }
   }, 1000);
 }

tests/e2e/telemetry.test.ts

@@ -4,12 +4,12 @@
  * SPDX-License-Identifier: Apache-2.0
  */
 
-import {describe, it} from 'node:test';
 import assert from 'node:assert';
 import {spawn} from 'node:child_process';
 import fs from 'node:fs';
-import path from 'node:path';
 import os from 'node:os';
+import path from 'node:path';
+import {describe, it} from 'node:test';
 
 describe('Telemetry E2E', () => {
   async function runTelemetryTest(signal: NodeJS.Signals) {
@@ -78,7 +78,7 @@ describe('Telemetry E2E', () => {
       const checkInterval = setInterval(() => {
         try {
           process.kill(watchdogPid!, 0); // Check if running
-        } catch (e) {
+        } catch {
           clearInterval(checkInterval);
           resolve();
           return;
@@ -87,7 +87,11 @@ describe('Telemetry E2E', () => {
 
       setTimeout(() => {
         clearInterval(checkInterval);
-        try { process.kill(watchdogPid!, 'SIGKILL'); } catch {}
+        try {
+          process.kill(watchdogPid!, 'SIGKILL');
+        } catch {
+          // no-op
+        }
         reject(new Error('Timeout waiting for Watchdog exit'));
       }, 5000);
     });
@@ -102,7 +106,7 @@ describe('Telemetry E2E', () => {
     console.log(`[${signal}] Shutdown events verified`);
 
     // Cleanup
-    if (fs.existsSync(logFile)) fs.unlinkSync(logFile);
+    if (fs.existsSync(logFile)) {fs.unlinkSync(logFile);}
   }
 
   async function runFixtureTest(action: 'exit-0' | 'exit-1' | 'crash') {
@@ -165,7 +169,7 @@ describe('Telemetry E2E', () => {
       const checkInterval = setInterval(() => {
         try {
           process.kill(watchdogPid!, 0); 
-        } catch (e) {
+        } catch  {
           clearInterval(checkInterval);
           resolve();
           return;
@@ -174,7 +178,11 @@ describe('Telemetry E2E', () => {
 
       setTimeout(() => {
         clearInterval(checkInterval);
-        try { process.kill(watchdogPid!, 'SIGKILL'); } catch {}
+        try {
+          process.kill(watchdogPid!, 'SIGKILL');
+        } catch {
+          // no-op
+        }
         reject(new Error('Timeout waiting for Watchdog exit'));
       }, 5000);
     });
@@ -187,7 +195,7 @@ describe('Telemetry E2E', () => {
     assert.match(contentAfter, /server_shutdown/);
     console.log(`[${action}] Shutdown events verified`);
 
-    if (fs.existsSync(logFile)) fs.unlinkSync(logFile);
+    if (fs.existsSync(logFile)) {fs.unlinkSync(logFile);}
   }
 
   it('handles clean exit (process.exit(0))', () => runFixtureTest('exit-0'));

tests/telemetry/clearcut-logger.test.ts

@@ -10,10 +10,10 @@ import {describe, it, afterEach, beforeEach} from 'node:test';
 import sinon from 'sinon';
 
 import {ClearcutLogger} from '../../src/telemetry/clearcut-logger.js';
-import {WatchdogClient} from '../../src/telemetry/watchdog-client.js';
 import type {Persistence} from '../../src/telemetry/persistence.js';
 import {FilePersistence} from '../../src/telemetry/persistence.js';
 import {IpcMessageType} from '../../src/telemetry/types.js';
+import {WatchdogClient} from '../../src/telemetry/watchdog-client.js';
 
 describe('ClearcutLogger', () => {
   let mockPersistence: sinon.SinonStubbedInstance<Persistence>;

tests/telemetry/watchdog-client.test.ts

@@ -4,15 +4,17 @@
  * SPDX-License-Identifier: Apache-2.0
  */
 
-import {describe, it, afterEach, beforeEach} from 'node:test';
 import assert from 'node:assert';
-import sinon from 'sinon';
-import {spawn, type ChildProcess} from 'node:child_process';
+import type {spawn} from 'node:child_process';
+import { type ChildProcess} from 'node:child_process';
 import EventEmitter from 'node:events';
 import {Writable} from 'node:stream';
+import {describe, it, afterEach, beforeEach} from 'node:test';
+
+import sinon from 'sinon';
 
-import {WatchdogClient} from '../../src/telemetry/watchdog-client.js';
 import {OsType, IpcMessageType} from '../../src/telemetry/types.js';
+import {WatchdogClient} from '../../src/telemetry/watchdog-client.js';
 
 describe('WatchdogClient', () => {
   let spawnStub: sinon.SinonStub;

tests/telemetry/watchdog/clearcut-sender.test.ts

@@ -4,10 +4,12 @@
  * SPDX-License-Identifier: Apache-2.0
  */
 
+import assert from 'node:assert';
 import crypto from 'node:crypto';
-import sinon from 'sinon';
 import {describe, it, afterEach, beforeEach} from 'node:test';
-import assert from 'node:assert';
+
+import sinon from 'sinon';
+
 import {OsType} from '../../../src/telemetry/types.js';
 import {ClearcutSender} from '../../../src/telemetry/watchdog/clearcut-sender.js';