wind-agent-cli 1.37.2 → 1.37.3

package/server/modules/settings/index.ts

@@ -1,2 +1,2 @@
-// settingsRoutes: used by the server entrypoint to mount protected application-settings endpoints.
-export { settingsRoutes } from './settings.module.js';
+// settingsRoutes: used by the server entrypoint to mount protected application-settings endpoints.
+export { settingsRoutes } from './settings.module.js';

package/server/modules/settings/settings.module.ts

@@ -1,50 +1,50 @@
-import {
-  apiKeysDb,
-  credentialsDb,
-  notificationPreferencesDb,
-  pushSubscriptionsDb,
-} from '@/modules/database/index.js';
-import {
-  createNotificationEvent,
-  getPublicKey,
-  notifyUserIfEnabled,
-} from '@/modules/notifications/index.js';
-
-import { createSettingsRouter } from './settings.routes.js';
-import { createSettingsService } from './settings.service.js';
-
-const settingsService = createSettingsService({
-  apiKeys: {
-    list: (userId) => apiKeysDb.getApiKeys(userId),
-    create: (userId, keyName) => apiKeysDb.createApiKey(userId, keyName),
-    remove: (userId, keyId) => apiKeysDb.deleteApiKey(userId, keyId),
-    toggle: (userId, keyId, isActive) => apiKeysDb.toggleApiKey(userId, keyId, isActive),
-  },
-  credentials: {
-    list: (userId, type) => credentialsDb.getCredentials(userId, type),
-    create: (userId, name, type, value, description) =>
-      credentialsDb.createCredential(userId, name, type, value, description),
-    remove: (userId, credentialId) => credentialsDb.deleteCredential(userId, credentialId),
-    toggle: (userId, credentialId, isActive) =>
-      credentialsDb.toggleCredential(userId, credentialId, isActive),
-  },
-  notifications: {
-    getPreferences: (userId) => notificationPreferencesDb.getPreferences(userId),
-    updatePreferences: (userId, preferences) =>
-      notificationPreferencesDb.updatePreferences(userId, preferences),
-    createEnabledEvent: () => createNotificationEvent({
-      provider: 'system', kind: 'info', code: 'push.enabled',
-      meta: { message: 'Push notifications are now enabled!' }, severity: 'info',
-    }),
-    notifyUser: (userId, event) => notifyUserIfEnabled({ userId, event }),
-  },
-  pushSubscriptions: {
-    save: (userId, endpoint, p256dh, auth) =>
-      pushSubscriptionsDb.saveSubscription(userId, endpoint, p256dh, auth),
-    remove: (endpoint) => pushSubscriptionsDb.removeSubscription(endpoint),
-  },
-  getVapidPublicKey: getPublicKey,
-});
-
-/** Settings router assembled for the authenticated server mount. */
-export const settingsRoutes = createSettingsRouter(settingsService);
+import {
+  apiKeysDb,
+  credentialsDb,
+  notificationPreferencesDb,
+  pushSubscriptionsDb,
+} from '@/modules/database/index.js';
+import {
+  createNotificationEvent,
+  getPublicKey,
+  notifyUserIfEnabled,
+} from '@/modules/notifications/index.js';
+
+import { createSettingsRouter } from './settings.routes.js';
+import { createSettingsService } from './settings.service.js';
+
+const settingsService = createSettingsService({
+  apiKeys: {
+    list: (userId) => apiKeysDb.getApiKeys(userId),
+    create: (userId, keyName) => apiKeysDb.createApiKey(userId, keyName),
+    remove: (userId, keyId) => apiKeysDb.deleteApiKey(userId, keyId),
+    toggle: (userId, keyId, isActive) => apiKeysDb.toggleApiKey(userId, keyId, isActive),
+  },
+  credentials: {
+    list: (userId, type) => credentialsDb.getCredentials(userId, type),
+    create: (userId, name, type, value, description) =>
+      credentialsDb.createCredential(userId, name, type, value, description),
+    remove: (userId, credentialId) => credentialsDb.deleteCredential(userId, credentialId),
+    toggle: (userId, credentialId, isActive) =>
+      credentialsDb.toggleCredential(userId, credentialId, isActive),
+  },
+  notifications: {
+    getPreferences: (userId) => notificationPreferencesDb.getPreferences(userId),
+    updatePreferences: (userId, preferences) =>
+      notificationPreferencesDb.updatePreferences(userId, preferences),
+    createEnabledEvent: () => createNotificationEvent({
+      provider: 'system', kind: 'info', code: 'push.enabled',
+      meta: { message: 'Push notifications are now enabled!' }, severity: 'info',
+    }),
+    notifyUser: (userId, event) => notifyUserIfEnabled({ userId, event }),
+  },
+  pushSubscriptions: {
+    save: (userId, endpoint, p256dh, auth) =>
+      pushSubscriptionsDb.saveSubscription(userId, endpoint, p256dh, auth),
+    remove: (endpoint) => pushSubscriptionsDb.removeSubscription(endpoint),
+  },
+  getVapidPublicKey: getPublicKey,
+});
+
+/** Settings router assembled for the authenticated server mount. */
+export const settingsRoutes = createSettingsRouter(settingsService);

package/server/modules/settings/settings.routes.ts

@@ -1,51 +1,51 @@
-import express from 'express';
-
-import type { createSettingsService } from './settings.service.js';
-
-type AuthenticatedRequest = express.Request & { user?: { id?: number | string } };
-
-function userId(req: express.Request): number {
-  return Number((req as AuthenticatedRequest).user?.id);
-}
-
-function queryString(value: unknown): string | null {
-  return typeof value === 'string' && value.trim() ? value.trim() : null;
-}
-
-/** Creates thin Settings transport handlers around the application service. */
-export function createSettingsRouter(
-  service: ReturnType<typeof createSettingsService>,
-): express.Router {
-  const router = express.Router();
-  const respond = (operation: (req: express.Request) => unknown | Promise<unknown>) =>
-    async (req: express.Request, res: express.Response, next: express.NextFunction) => {
-      try { res.json(await operation(req)); } catch (error) { next(error); }
-    };
-
-  router.get('/api-keys', respond((req) => service.listApiKeys(userId(req))));
-  router.post('/api-keys', respond((req) => service.createApiKey(userId(req), req.body?.keyName)));
-  router.delete('/api-keys/:keyId', respond((req) => service.deleteApiKey(userId(req), Number(req.params.keyId))));
-  router.patch('/api-keys/:keyId/toggle', respond((req) => service.toggleApiKey(
-    userId(req), Number(req.params.keyId), req.body?.isActive,
-  )));
-  router.get('/credentials', respond((req) => service.listCredentials(
-    userId(req), queryString(req.query.type),
-  )));
-  router.post('/credentials', respond((req) => service.createCredential(userId(req), req.body ?? {})));
-  router.delete('/credentials/:credentialId', respond((req) => service.deleteCredential(
-    userId(req), Number(req.params.credentialId),
-  )));
-  router.patch('/credentials/:credentialId/toggle', respond((req) => service.toggleCredential(
-    userId(req), Number(req.params.credentialId), req.body?.isActive,
-  )));
-  router.get('/notification-preferences', respond((req) => service.getNotificationPreferences(userId(req))));
-  router.put('/notification-preferences', respond((req) => service.updateNotificationPreferences(
-    userId(req), req.body ?? {},
-  )));
-  router.get('/push/vapid-public-key', respond(() => service.getVapidPublicKey()));
-  router.post('/push/subscribe', respond((req) => service.subscribeToPush(userId(req), req.body ?? {})));
-  router.post('/push/unsubscribe', respond((req) => service.unsubscribeFromPush(
-    userId(req), req.body?.endpoint,
-  )));
-  return router;
-}
+import express from 'express';
+
+import type { createSettingsService } from './settings.service.js';
+
+type AuthenticatedRequest = express.Request & { user?: { id?: number | string } };
+
+function userId(req: express.Request): number {
+  return Number((req as AuthenticatedRequest).user?.id);
+}
+
+function queryString(value: unknown): string | null {
+  return typeof value === 'string' && value.trim() ? value.trim() : null;
+}
+
+/** Creates thin Settings transport handlers around the application service. */
+export function createSettingsRouter(
+  service: ReturnType<typeof createSettingsService>,
+): express.Router {
+  const router = express.Router();
+  const respond = (operation: (req: express.Request) => unknown | Promise<unknown>) =>
+    async (req: express.Request, res: express.Response, next: express.NextFunction) => {
+      try { res.json(await operation(req)); } catch (error) { next(error); }
+    };
+
+  router.get('/api-keys', respond((req) => service.listApiKeys(userId(req))));
+  router.post('/api-keys', respond((req) => service.createApiKey(userId(req), req.body?.keyName)));
+  router.delete('/api-keys/:keyId', respond((req) => service.deleteApiKey(userId(req), Number(req.params.keyId))));
+  router.patch('/api-keys/:keyId/toggle', respond((req) => service.toggleApiKey(
+    userId(req), Number(req.params.keyId), req.body?.isActive,
+  )));
+  router.get('/credentials', respond((req) => service.listCredentials(
+    userId(req), queryString(req.query.type),
+  )));
+  router.post('/credentials', respond((req) => service.createCredential(userId(req), req.body ?? {})));
+  router.delete('/credentials/:credentialId', respond((req) => service.deleteCredential(
+    userId(req), Number(req.params.credentialId),
+  )));
+  router.patch('/credentials/:credentialId/toggle', respond((req) => service.toggleCredential(
+    userId(req), Number(req.params.credentialId), req.body?.isActive,
+  )));
+  router.get('/notification-preferences', respond((req) => service.getNotificationPreferences(userId(req))));
+  router.put('/notification-preferences', respond((req) => service.updateNotificationPreferences(
+    userId(req), req.body ?? {},
+  )));
+  router.get('/push/vapid-public-key', respond(() => service.getVapidPublicKey()));
+  router.post('/push/subscribe', respond((req) => service.subscribeToPush(userId(req), req.body ?? {})));
+  router.post('/push/unsubscribe', respond((req) => service.unsubscribeFromPush(
+    userId(req), req.body?.endpoint,
+  )));
+  return router;
+}

package/server/modules/settings/settings.service.ts

@@ -1,186 +1,186 @@
-import { AppError } from '@/shared/utils.js';
-
-type ApiKeyRow = Record<string, unknown> & { api_key: string };
-type NotificationPreferences = Record<string, unknown> & {
-  channels?: Record<string, unknown> & { webPush?: boolean };
-};
-
-type SettingsDependencies = {
-  apiKeys: {
-    list(userId: number): ApiKeyRow[];
-    create(userId: number, keyName: string): unknown;
-    remove(userId: number, keyId: number): boolean;
-    toggle(userId: number, keyId: number, isActive: boolean): boolean;
-  };
-  credentials: {
-    list(userId: number, credentialType: string | null): unknown[];
-    create(
-      userId: number,
-      name: string,
-      type: string,
-      value: string,
-      description: string | null,
-    ): unknown;
-    remove(userId: number, credentialId: number): boolean;
-    toggle(userId: number, credentialId: number, isActive: boolean): boolean;
-  };
-  notifications: {
-    getPreferences(userId: number): NotificationPreferences | undefined;
-    updatePreferences(userId: number, preferences: NotificationPreferences): unknown;
-    createEnabledEvent(): unknown;
-    notifyUser(userId: number, event: unknown): void | Promise<void>;
-  };
-  pushSubscriptions: {
-    save(userId: number, endpoint: string, p256dh: string, auth: string): void;
-    remove(endpoint: string): void;
-  };
-  getVapidPublicKey(): string | null;
-};
-
-function requiredString(value: unknown, fieldName: string, code: string): string {
-  const normalizedValue = typeof value === 'string' ? value.trim() : '';
-  if (!normalizedValue) {
-    throw new AppError(`${fieldName} is required`, { code, statusCode: 400 });
-  }
-  return normalizedValue;
-}
-
-function assertFound(found: boolean, resourceName: string, code: string): void {
-  if (!found) {
-    throw new AppError(`${resourceName} not found`, { code, statusCode: 404 });
-  }
-}
-
-/** Creates settings workflows with repositories and notification effects injected. */
-export function createSettingsService(dependencies: SettingsDependencies) {
-  return {
-    listApiKeys(userId: number) {
-      const apiKeys = dependencies.apiKeys.list(userId).map((key) => ({
-        ...key,
-        api_key: `${key.api_key.substring(0, 10)}...`,
-      }));
-      return { apiKeys };
-    },
-    createApiKey(userId: number, keyNameInput: unknown) {
-      const keyName = requiredString(keyNameInput, 'Key name', 'API_KEY_NAME_REQUIRED');
-      return { success: true, apiKey: dependencies.apiKeys.create(userId, keyName) };
-    },
-    deleteApiKey(userId: number, keyId: number) {
-      assertFound(dependencies.apiKeys.remove(userId, keyId), 'API key', 'API_KEY_NOT_FOUND');
-      return { success: true };
-    },
-    toggleApiKey(userId: number, keyId: number, isActive: unknown) {
-      if (typeof isActive !== 'boolean') {
-        throw new AppError('isActive must be a boolean', {
-          code: 'INVALID_ACTIVE_STATE',
-          statusCode: 400,
-        });
-      }
-      assertFound(
-        dependencies.apiKeys.toggle(userId, keyId, isActive),
-        'API key',
-        'API_KEY_NOT_FOUND',
-      );
-      return { success: true };
-    },
-    listCredentials(userId: number, credentialType: string | null) {
-      return { credentials: dependencies.credentials.list(userId, credentialType) };
-    },
-    createCredential(userId: number, input: Record<string, unknown>) {
-      const credentialName = requiredString(
-        input.credentialName,
-        'Credential name',
-        'CREDENTIAL_NAME_REQUIRED',
-      );
-      const credentialType = requiredString(
-        input.credentialType,
-        'Credential type',
-        'CREDENTIAL_TYPE_REQUIRED',
-      );
-      const credentialValue = requiredString(
-        input.credentialValue,
-        'Credential value',
-        'CREDENTIAL_VALUE_REQUIRED',
-      );
-      const description = typeof input.description === 'string'
-        ? input.description.trim() || null
-        : null;
-      return {
-        success: true,
-        credential: dependencies.credentials.create(
-          userId,
-          credentialName,
-          credentialType,
-          credentialValue,
-          description,
-        ),
-      };
-    },
-    deleteCredential(userId: number, credentialId: number) {
-      assertFound(
-        dependencies.credentials.remove(userId, credentialId),
-        'Credential',
-        'CREDENTIAL_NOT_FOUND',
-      );
-      return { success: true };
-    },
-    toggleCredential(userId: number, credentialId: number, isActive: unknown) {
-      if (typeof isActive !== 'boolean') {
-        throw new AppError('isActive must be a boolean', {
-          code: 'INVALID_ACTIVE_STATE',
-          statusCode: 400,
-        });
-      }
-      assertFound(
-        dependencies.credentials.toggle(userId, credentialId, isActive),
-        'Credential',
-        'CREDENTIAL_NOT_FOUND',
-      );
-      return { success: true };
-    },
-    getNotificationPreferences(userId: number) {
-      return { success: true, preferences: dependencies.notifications.getPreferences(userId) };
-    },
-    updateNotificationPreferences(userId: number, preferences: NotificationPreferences) {
-      return {
-        success: true,
-        preferences: dependencies.notifications.updatePreferences(userId, preferences),
-      };
-    },
-    getVapidPublicKey() {
-      return { publicKey: dependencies.getVapidPublicKey() };
-    },
-    subscribeToPush(userId: number, input: Record<string, unknown>) {
-      const endpoint = requiredString(input.endpoint, 'Endpoint', 'PUSH_SUBSCRIPTION_REQUIRED');
-      const keys = typeof input.keys === 'object' && input.keys !== null
-        ? input.keys as Record<string, unknown>
-        : {};
-      const p256dh = requiredString(keys.p256dh, 'p256dh', 'PUSH_SUBSCRIPTION_REQUIRED');
-      const auth = requiredString(keys.auth, 'auth', 'PUSH_SUBSCRIPTION_REQUIRED');
-      dependencies.pushSubscriptions.save(userId, endpoint, p256dh, auth);
-
-      const currentPreferences = dependencies.notifications.getPreferences(userId);
-      if (!currentPreferences?.channels?.webPush) {
-        dependencies.notifications.updatePreferences(userId, {
-          ...currentPreferences,
-          channels: { ...currentPreferences?.channels, webPush: true },
-        });
-      }
-      const event = dependencies.notifications.createEnabledEvent();
-      void dependencies.notifications.notifyUser(userId, event);
-      return { success: true };
-    },
-    unsubscribeFromPush(userId: number, endpointInput: unknown) {
-      const endpoint = requiredString(endpointInput, 'Endpoint', 'PUSH_ENDPOINT_REQUIRED');
-      dependencies.pushSubscriptions.remove(endpoint);
-      const currentPreferences = dependencies.notifications.getPreferences(userId);
-      if (currentPreferences?.channels?.webPush) {
-        dependencies.notifications.updatePreferences(userId, {
-          ...currentPreferences,
-          channels: { ...currentPreferences.channels, webPush: false },
-        });
-      }
-      return { success: true };
-    },
-  };
-}
+import { AppError } from '@/shared/utils.js';
+
+type ApiKeyRow = Record<string, unknown> & { api_key: string };
+type NotificationPreferences = Record<string, unknown> & {
+  channels?: Record<string, unknown> & { webPush?: boolean };
+};
+
+type SettingsDependencies = {
+  apiKeys: {
+    list(userId: number): ApiKeyRow[];
+    create(userId: number, keyName: string): unknown;
+    remove(userId: number, keyId: number): boolean;
+    toggle(userId: number, keyId: number, isActive: boolean): boolean;
+  };
+  credentials: {
+    list(userId: number, credentialType: string | null): unknown[];
+    create(
+      userId: number,
+      name: string,
+      type: string,
+      value: string,
+      description: string | null,
+    ): unknown;
+    remove(userId: number, credentialId: number): boolean;
+    toggle(userId: number, credentialId: number, isActive: boolean): boolean;
+  };
+  notifications: {
+    getPreferences(userId: number): NotificationPreferences | undefined;
+    updatePreferences(userId: number, preferences: NotificationPreferences): unknown;
+    createEnabledEvent(): unknown;
+    notifyUser(userId: number, event: unknown): void | Promise<void>;
+  };
+  pushSubscriptions: {
+    save(userId: number, endpoint: string, p256dh: string, auth: string): void;
+    remove(endpoint: string): void;
+  };
+  getVapidPublicKey(): string | null;
+};
+
+function requiredString(value: unknown, fieldName: string, code: string): string {
+  const normalizedValue = typeof value === 'string' ? value.trim() : '';
+  if (!normalizedValue) {
+    throw new AppError(`${fieldName} is required`, { code, statusCode: 400 });
+  }
+  return normalizedValue;
+}
+
+function assertFound(found: boolean, resourceName: string, code: string): void {
+  if (!found) {
+    throw new AppError(`${resourceName} not found`, { code, statusCode: 404 });
+  }
+}
+
+/** Creates settings workflows with repositories and notification effects injected. */
+export function createSettingsService(dependencies: SettingsDependencies) {
+  return {
+    listApiKeys(userId: number) {
+      const apiKeys = dependencies.apiKeys.list(userId).map((key) => ({
+        ...key,
+        api_key: `${key.api_key.substring(0, 10)}...`,
+      }));
+      return { apiKeys };
+    },
+    createApiKey(userId: number, keyNameInput: unknown) {
+      const keyName = requiredString(keyNameInput, 'Key name', 'API_KEY_NAME_REQUIRED');
+      return { success: true, apiKey: dependencies.apiKeys.create(userId, keyName) };
+    },
+    deleteApiKey(userId: number, keyId: number) {
+      assertFound(dependencies.apiKeys.remove(userId, keyId), 'API key', 'API_KEY_NOT_FOUND');
+      return { success: true };
+    },
+    toggleApiKey(userId: number, keyId: number, isActive: unknown) {
+      if (typeof isActive !== 'boolean') {
+        throw new AppError('isActive must be a boolean', {
+          code: 'INVALID_ACTIVE_STATE',
+          statusCode: 400,
+        });
+      }
+      assertFound(
+        dependencies.apiKeys.toggle(userId, keyId, isActive),
+        'API key',
+        'API_KEY_NOT_FOUND',
+      );
+      return { success: true };
+    },
+    listCredentials(userId: number, credentialType: string | null) {
+      return { credentials: dependencies.credentials.list(userId, credentialType) };
+    },
+    createCredential(userId: number, input: Record<string, unknown>) {
+      const credentialName = requiredString(
+        input.credentialName,
+        'Credential name',
+        'CREDENTIAL_NAME_REQUIRED',
+      );
+      const credentialType = requiredString(
+        input.credentialType,
+        'Credential type',
+        'CREDENTIAL_TYPE_REQUIRED',
+      );
+      const credentialValue = requiredString(
+        input.credentialValue,
+        'Credential value',
+        'CREDENTIAL_VALUE_REQUIRED',
+      );
+      const description = typeof input.description === 'string'
+        ? input.description.trim() || null
+        : null;
+      return {
+        success: true,
+        credential: dependencies.credentials.create(
+          userId,
+          credentialName,
+          credentialType,
+          credentialValue,
+          description,
+        ),
+      };
+    },
+    deleteCredential(userId: number, credentialId: number) {
+      assertFound(
+        dependencies.credentials.remove(userId, credentialId),
+        'Credential',
+        'CREDENTIAL_NOT_FOUND',
+      );
+      return { success: true };
+    },
+    toggleCredential(userId: number, credentialId: number, isActive: unknown) {
+      if (typeof isActive !== 'boolean') {
+        throw new AppError('isActive must be a boolean', {
+          code: 'INVALID_ACTIVE_STATE',
+          statusCode: 400,
+        });
+      }
+      assertFound(
+        dependencies.credentials.toggle(userId, credentialId, isActive),
+        'Credential',
+        'CREDENTIAL_NOT_FOUND',
+      );
+      return { success: true };
+    },
+    getNotificationPreferences(userId: number) {
+      return { success: true, preferences: dependencies.notifications.getPreferences(userId) };
+    },
+    updateNotificationPreferences(userId: number, preferences: NotificationPreferences) {
+      return {
+        success: true,
+        preferences: dependencies.notifications.updatePreferences(userId, preferences),
+      };
+    },
+    getVapidPublicKey() {
+      return { publicKey: dependencies.getVapidPublicKey() };
+    },
+    subscribeToPush(userId: number, input: Record<string, unknown>) {
+      const endpoint = requiredString(input.endpoint, 'Endpoint', 'PUSH_SUBSCRIPTION_REQUIRED');
+      const keys = typeof input.keys === 'object' && input.keys !== null
+        ? input.keys as Record<string, unknown>
+        : {};
+      const p256dh = requiredString(keys.p256dh, 'p256dh', 'PUSH_SUBSCRIPTION_REQUIRED');
+      const auth = requiredString(keys.auth, 'auth', 'PUSH_SUBSCRIPTION_REQUIRED');
+      dependencies.pushSubscriptions.save(userId, endpoint, p256dh, auth);
+
+      const currentPreferences = dependencies.notifications.getPreferences(userId);
+      if (!currentPreferences?.channels?.webPush) {
+        dependencies.notifications.updatePreferences(userId, {
+          ...currentPreferences,
+          channels: { ...currentPreferences?.channels, webPush: true },
+        });
+      }
+      const event = dependencies.notifications.createEnabledEvent();
+      void dependencies.notifications.notifyUser(userId, event);
+      return { success: true };
+    },
+    unsubscribeFromPush(userId: number, endpointInput: unknown) {
+      const endpoint = requiredString(endpointInput, 'Endpoint', 'PUSH_ENDPOINT_REQUIRED');
+      dependencies.pushSubscriptions.remove(endpoint);
+      const currentPreferences = dependencies.notifications.getPreferences(userId);
+      if (currentPreferences?.channels?.webPush) {
+        dependencies.notifications.updatePreferences(userId, {
+          ...currentPreferences,
+          channels: { ...currentPreferences.channels, webPush: false },
+        });
+      }
+      return { success: true };
+    },
+  };
+}