From ed9ea0e373fca23eb93e52b78fc2c5da1be15fe0 Mon Sep 17 00:00:00 2001
From: Peter Steinberger <steipete@gmail.com>
Date: Mon, 23 Jun 2025 15:28:03 +0200
Subject: [PATCH] TS fixes

---
 web/eslint.config.js              |  2 +-
 web/src/client/sw.ts              |  6 +++---
 web/src/server/pty/pty-manager.ts |  2 ++
 web/tsconfig.test.json            | 11 +++++++++++
 4 files changed, 17 insertions(+), 4 deletions(-)
 create mode 100644 web/tsconfig.test.json

diff --git a/web/eslint.config.js b/web/eslint.config.js
index cf2d240d..702538d5 100644
--- a/web/eslint.config.js
+++ b/web/eslint.config.js
@@ -14,7 +14,7 @@ module.exports = tseslint.config(
     files: ['src/**/*.ts', 'src/**/*.tsx'],
     languageOptions: {
       parserOptions: {
-        project: ['./tsconfig.json', './tsconfig.client.json'],
+        project: ['./tsconfig.json', './tsconfig.client.json', './tsconfig.sw.json', './tsconfig.test.json'],
         tsconfigRootDir: __dirname,
       },
     },
diff --git a/web/src/client/sw.ts b/web/src/client/sw.ts
index 4716f27d..e67d620d 100644
--- a/web/src/client/sw.ts
+++ b/web/src/client/sw.ts
@@ -168,7 +168,7 @@ async function handlePushNotification(payload: PushNotificationPayload): Promise
       tag: tag || `${NOTIFICATION_TAG_PREFIX}${data.type}-${Date.now()}`,
       requireInteraction: requireInteraction || data.type === 'session-error',
       silent: false,
-      // @ts-ignore - renotify is a valid option but not in TypeScript types
+      // @ts-expect-error - renotify is a valid option but not in TypeScript types
       renotify: true,
       actions: actions || getDefaultActions(data),
       timestamp: data.timestamp,
@@ -176,7 +176,7 @@ async function handlePushNotification(payload: PushNotificationPayload): Promise
 
     // Add vibration pattern for mobile devices
     if ('vibrate' in navigator) {
-      // @ts-ignore - vibrate is a valid option but not in TypeScript types
+      // @ts-expect-error - vibrate is a valid option but not in TypeScript types
       notificationOptions.vibrate = getVibrationPattern(data.type);
     }
 
@@ -402,7 +402,7 @@ async function queueNotification(payload: PushNotificationPayload): Promise<void
 
   // Register for background sync
   try {
-    // @ts-ignore - sync is part of Background Sync API
+    // @ts-expect-error - sync is part of Background Sync API
     await self.registration.sync.register('notification-sync');
   } catch (error) {
     console.warn('[SW] Background sync not supported:', error);
diff --git a/web/src/server/pty/pty-manager.ts b/web/src/server/pty/pty-manager.ts
index 356a3fd8..88a450ad 100644
--- a/web/src/server/pty/pty-manager.ts
+++ b/web/src/server/pty/pty-manager.ts
@@ -323,9 +323,11 @@ export class PtyManager extends EventEmitter {
           logger.debug(`Bell data in session ${session.id}: ${JSON.stringify(data)}`);
 
           // Count total bells and OSC-terminated bells
+          // eslint-disable-next-line no-control-regex
           const totalBells = (data.match(/\x07/g) || []).length;
 
           // Count OSC sequences terminated with bell: \x1b]...\x07
+          // eslint-disable-next-line no-control-regex
           const oscMatches = data.match(/\x1b]([^\x07\x1b]|\x1b[^]])*\x07/g) || [];
           const oscTerminatedBells = oscMatches.length;
 
diff --git a/web/tsconfig.test.json b/web/tsconfig.test.json
new file mode 100644
index 00000000..7d09a431
--- /dev/null
+++ b/web/tsconfig.test.json
@@ -0,0 +1,11 @@
+{
+  "extends": "./tsconfig.json",
+  "compilerOptions": {
+    "types": ["vitest/globals"]
+  },
+  "include": [
+    "src/**/*.test.ts",
+    "src/**/*.spec.ts",
+    "src/test/**/*"
+  ]
+}
\ No newline at end of file