wind-agent-cli 1.37.2 → 1.38.0

package/server/modules/notifications/tests/notification-orchestrator.integration.test.ts

@@ -1,55 +1,55 @@
-import assert from 'node:assert/strict';
-import { mkdtemp, rm } from 'node:fs/promises';
-import { tmpdir } from 'node:os';
-import path from 'node:path';
-import test from 'node:test';
-
-import {
-  closeConnection,
-  initializeDatabase,
-  sessionsDb,
-} from '@/modules/database/index.js';
-
-import {
-  buildNotificationPayload,
-} from '../services/notification-orchestrator.service.js';
-
-async function withIsolatedDatabase(runTest: () => void | Promise<void>): Promise<void> {
-  const previousDatabasePath = process.env.DATABASE_PATH;
-  const temporaryDirectory = await mkdtemp(path.join(tmpdir(), 'notification-orchestrator-'));
-  const databasePath = path.join(temporaryDirectory, 'auth.db');
-
-  closeConnection();
-  process.env.DATABASE_PATH = databasePath;
-  await initializeDatabase();
-
-  try {
-    await runTest();
-  } finally {
-    closeConnection();
-    if (previousDatabasePath === undefined) {
-      delete process.env.DATABASE_PATH;
-    } else {
-      process.env.DATABASE_PATH = previousDatabasePath;
-    }
-    await rm(temporaryDirectory, { recursive: true, force: true });
-  }
-}
-
-test('notification payload uses the app session id for a provider session id', async () => {
-  await withIsolatedDatabase(() => {
-    sessionsDb.createAppSession('app-session-1', 'claude', '/workspace/demo');
-    sessionsDb.assignProviderSessionId('app-session-1', 'claude-native-1');
-
-    const payload = buildNotificationPayload({
-      provider: 'claude',
-      sessionId: 'claude-native-1',
-      kind: 'stop',
-      code: 'run.stopped',
-      meta: { stopReason: 'completed' },
-    });
-
-    assert.equal(payload.data.sessionId, 'app-session-1');
-    assert.match(payload.data.tag, /app-session-1/);
-  });
-});
+import assert from 'node:assert/strict';
+import { mkdtemp, rm } from 'node:fs/promises';
+import { tmpdir } from 'node:os';
+import path from 'node:path';
+import test from 'node:test';
+
+import {
+  closeConnection,
+  initializeDatabase,
+  sessionsDb,
+} from '@/modules/database/index.js';
+
+import {
+  buildNotificationPayload,
+} from '../services/notification-orchestrator.service.js';
+
+async function withIsolatedDatabase(runTest: () => void | Promise<void>): Promise<void> {
+  const previousDatabasePath = process.env.DATABASE_PATH;
+  const temporaryDirectory = await mkdtemp(path.join(tmpdir(), 'notification-orchestrator-'));
+  const databasePath = path.join(temporaryDirectory, 'auth.db');
+
+  closeConnection();
+  process.env.DATABASE_PATH = databasePath;
+  await initializeDatabase();
+
+  try {
+    await runTest();
+  } finally {
+    closeConnection();
+    if (previousDatabasePath === undefined) {
+      delete process.env.DATABASE_PATH;
+    } else {
+      process.env.DATABASE_PATH = previousDatabasePath;
+    }
+    await rm(temporaryDirectory, { recursive: true, force: true });
+  }
+}
+
+test('notification payload uses the app session id for a provider session id', async () => {
+  await withIsolatedDatabase(() => {
+    sessionsDb.createAppSession('app-session-1', 'claude', '/workspace/demo');
+    sessionsDb.assignProviderSessionId('app-session-1', 'claude-native-1');
+
+    const payload = buildNotificationPayload({
+      provider: 'claude',
+      sessionId: 'claude-native-1',
+      kind: 'stop',
+      code: 'run.stopped',
+      meta: { stopReason: 'completed' },
+    });
+
+    assert.equal(payload.data.sessionId, 'app-session-1');
+    assert.match(payload.data.tag, /app-session-1/);
+  });
+});

package/server/modules/notifications/vapid-keys.service.ts

@@ -1,38 +1,38 @@
-// @ts-nocheck -- web-push does not provide declarations in this project.
-import webPush from 'web-push';
-
-import { getConnection } from '../database/index.js';
-
-let cachedKeys = null;
-const db = getConnection();
-
-function ensureVapidKeys() {
-  if (cachedKeys) return cachedKeys;
-
-  const row = db.prepare('SELECT public_key, private_key FROM vapid_keys ORDER BY id DESC LIMIT 1').get();
-  if (row) {
-    cachedKeys = { publicKey: row.public_key, privateKey: row.private_key };
-    return cachedKeys;
-  }
-
-  const keys = webPush.generateVAPIDKeys();
-  db.prepare('INSERT INTO vapid_keys (public_key, private_key) VALUES (?, ?)').run(keys.publicKey, keys.privateKey);
-  cachedKeys = keys;
-  return cachedKeys;
-}
-
-function getPublicKey() {
-  return ensureVapidKeys().publicKey;
-}
-
-function configureWebPush() {
-  const keys = ensureVapidKeys();
-  webPush.setVapidDetails(
-    'mailto:noreply@claudecodeui.local',
-    keys.publicKey,
-    keys.privateKey
-  );
-  console.log('Web Push notifications configured');
-}
-
-export { ensureVapidKeys, getPublicKey, configureWebPush };
+// @ts-nocheck -- web-push does not provide declarations in this project.
+import webPush from 'web-push';
+
+import { getConnection } from '../database/index.js';
+
+let cachedKeys = null;
+const db = getConnection();
+
+function ensureVapidKeys() {
+  if (cachedKeys) return cachedKeys;
+
+  const row = db.prepare('SELECT public_key, private_key FROM vapid_keys ORDER BY id DESC LIMIT 1').get();
+  if (row) {
+    cachedKeys = { publicKey: row.public_key, privateKey: row.private_key };
+    return cachedKeys;
+  }
+
+  const keys = webPush.generateVAPIDKeys();
+  db.prepare('INSERT INTO vapid_keys (public_key, private_key) VALUES (?, ?)').run(keys.publicKey, keys.privateKey);
+  cachedKeys = keys;
+  return cachedKeys;
+}
+
+function getPublicKey() {
+  return ensureVapidKeys().publicKey;
+}
+
+function configureWebPush() {
+  const keys = ensureVapidKeys();
+  webPush.setVapidDetails(
+    'mailto:noreply@windcli.local',
+    keys.publicKey,
+    keys.privateKey
+  );
+  console.log('Web Push notifications configured');
+}
+
+export { ensureVapidKeys, getPublicKey, configureWebPush };

package/server/modules/notifications/websocket/desktop-notifications-websocket.service.ts

@@ -1,109 +1,109 @@
-import type { WebSocket } from 'ws';
-
-import {
-  registerDesktopNotificationClient,
-  unregisterDesktopNotificationClient,
-} from '@/modules/notifications/services/desktop-notification-clients.service.js';
-import type { AuthenticatedWebSocketRequest } from '@/shared/types.js';
-import { parseIncomingJsonObject } from '@/shared/utils.js';
-
-type DesktopNotificationRegisterMessage = {
-  type?: unknown;
-  kind?: unknown;
-  deviceId?: unknown;
-  label?: unknown;
-  platform?: unknown;
-  appVersion?: unknown;
-};
-
-function readRequestUserId(request: AuthenticatedWebSocketRequest): number | null {
-  const user = request.user;
-  const rawUserId = typeof user?.id === 'number' || typeof user?.id === 'string'
-    ? user.id
-    : typeof user?.userId === 'number' || typeof user?.userId === 'string'
-      ? user.userId
-      : null;
-  const numericUserId = Number(rawUserId);
-  return Number.isInteger(numericUserId) && numericUserId > 0 ? numericUserId : null;
-}
-
-function readOptionalString(value: unknown): string | null {
-  if (typeof value !== 'string') return null;
-  const normalized = value.trim();
-  return normalized || null;
-}
-
-function sendJson(ws: WebSocket, payload: unknown): void {
-  if (ws.readyState === ws.OPEN) {
-    ws.send(JSON.stringify(payload));
-  }
-}
-
-export function handleDesktopNotificationsConnection(
-  ws: WebSocket,
-  request: AuthenticatedWebSocketRequest
-): void {
-  const userId = readRequestUserId(request);
-  if (!userId) {
-    ws.close(1008, 'Missing authenticated user');
-    return;
-  }
-
-  let registered = false;
-
-  ws.on('message', (rawMessage) => {
-    const data = parseIncomingJsonObject(rawMessage) as DesktopNotificationRegisterMessage | null;
-    if (!data) {
-      return;
-    }
-
-    const type = typeof data.type === 'string' ? data.type : typeof data.kind === 'string' ? data.kind : '';
-    if (type === 'notification_ack') {
-      return;
-    }
-
-    if (type !== 'register' || registered) {
-      return;
-    }
-
-    const deviceId = readOptionalString(data.deviceId);
-    if (!deviceId) {
-      sendJson(ws, {
-        type: 'error',
-        code: 'DEVICE_ID_REQUIRED',
-        message: 'Desktop notification registration requires deviceId.',
-      });
-      ws.close(1008, 'Missing deviceId');
-      return;
-    }
-
-    const device = registerDesktopNotificationClient({
-      userId,
-      deviceId,
-      label: readOptionalString(data.label),
-      platform: readOptionalString(data.platform),
-      appVersion: readOptionalString(data.appVersion),
-      ws,
-    });
-
-    if (!device) {
-      ws.close(1011, 'Registration failed');
-      return;
-    }
-
-    registered = true;
-    sendJson(ws, {
-      type: 'registered',
-      deviceId: device.endpoint_id,
-      enabled: Boolean(device.enabled),
-    });
-  });
-
-  ws.on('close', () => {
-    unregisterDesktopNotificationClient(ws);
-  });
-
-  ws.on('error', () => {
-    unregisterDesktopNotificationClient(ws);
-  });
-}
+import type { WebSocket } from 'ws';
+
+import {
+  registerDesktopNotificationClient,
+  unregisterDesktopNotificationClient,
+} from '@/modules/notifications/services/desktop-notification-clients.service.js';
+import type { AuthenticatedWebSocketRequest } from '@/shared/types.js';
+import { parseIncomingJsonObject } from '@/shared/utils.js';
+
+type DesktopNotificationRegisterMessage = {
+  type?: unknown;
+  kind?: unknown;
+  deviceId?: unknown;
+  label?: unknown;
+  platform?: unknown;
+  appVersion?: unknown;
+};
+
+function readRequestUserId(request: AuthenticatedWebSocketRequest): number | null {
+  const user = request.user;
+  const rawUserId = typeof user?.id === 'number' || typeof user?.id === 'string'
+    ? user.id
+    : typeof user?.userId === 'number' || typeof user?.userId === 'string'
+      ? user.userId
+      : null;
+  const numericUserId = Number(rawUserId);
+  return Number.isInteger(numericUserId) && numericUserId > 0 ? numericUserId : null;
+}
+
+function readOptionalString(value: unknown): string | null {
+  if (typeof value !== 'string') return null;
+  const normalized = value.trim();
+  return normalized || null;
+}
+
+function sendJson(ws: WebSocket, payload: unknown): void {
+  if (ws.readyState === ws.OPEN) {
+    ws.send(JSON.stringify(payload));
+  }
+}
+
+export function handleDesktopNotificationsConnection(
+  ws: WebSocket,
+  request: AuthenticatedWebSocketRequest
+): void {
+  const userId = readRequestUserId(request);
+  if (!userId) {
+    ws.close(1008, 'Missing authenticated user');
+    return;
+  }
+
+  let registered = false;
+
+  ws.on('message', (rawMessage) => {
+    const data = parseIncomingJsonObject(rawMessage) as DesktopNotificationRegisterMessage | null;
+    if (!data) {
+      return;
+    }
+
+    const type = typeof data.type === 'string' ? data.type : typeof data.kind === 'string' ? data.kind : '';
+    if (type === 'notification_ack') {
+      return;
+    }
+
+    if (type !== 'register' || registered) {
+      return;
+    }
+
+    const deviceId = readOptionalString(data.deviceId);
+    if (!deviceId) {
+      sendJson(ws, {
+        type: 'error',
+        code: 'DEVICE_ID_REQUIRED',
+        message: 'Desktop notification registration requires deviceId.',
+      });
+      ws.close(1008, 'Missing deviceId');
+      return;
+    }
+
+    const device = registerDesktopNotificationClient({
+      userId,
+      deviceId,
+      label: readOptionalString(data.label),
+      platform: readOptionalString(data.platform),
+      appVersion: readOptionalString(data.appVersion),
+      ws,
+    });
+
+    if (!device) {
+      ws.close(1011, 'Registration failed');
+      return;
+    }
+
+    registered = true;
+    sendJson(ws, {
+      type: 'registered',
+      deviceId: device.endpoint_id,
+      enabled: Boolean(device.enabled),
+    });
+  });
+
+  ws.on('close', () => {
+    unregisterDesktopNotificationClient(ws);
+  });
+
+  ws.on('error', () => {
+    unregisterDesktopNotificationClient(ws);
+  });
+}

package/server/modules/plugins/index.ts

@@ -1,9 +1,9 @@
-// pluginsRoutes: used by the server entrypoint to mount protected plugin-management endpoints.
-export { pluginsRoutes } from './plugins.module.js';
-
-// startEnabledPluginServers: used by the server entrypoint to start enabled plugin subprocesses.
-export { startEnabledPluginServers } from './plugin-process.service.js';
-// stopAllPlugins: used by the server entrypoint to stop plugin subprocesses during shutdown.
-export { stopAllPlugins } from './plugin-process.service.js';
-// getPluginPort: used by WebSocket setup in the server entrypoint to proxy plugin connections.
-export { getPluginPort } from './plugin-process.service.js';
+// pluginsRoutes: used by the server entrypoint to mount protected plugin-management endpoints.
+export { pluginsRoutes } from './plugins.module.js';
+
+// startEnabledPluginServers: used by the server entrypoint to start enabled plugin subprocesses.
+export { startEnabledPluginServers } from './plugin-process.service.js';
+// stopAllPlugins: used by the server entrypoint to stop plugin subprocesses during shutdown.
+export { stopAllPlugins } from './plugin-process.service.js';
+// getPluginPort: used by WebSocket setup in the server entrypoint to proxy plugin connections.
+export { getPluginPort } from './plugin-process.service.js';