wind-agent-cli 1.37.2 → 1.37.3

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

@@ -1,54 +1,54 @@
-import assert from 'node:assert/strict';
-import test from 'node:test';
-
-import { createSettingsService } from '../settings.service.js';
-
-type Dependencies = Parameters<typeof createSettingsService>[0];
-
-function dependencies(overrides: Partial<Dependencies> = {}): Dependencies {
-  return {
-    apiKeys: { list: () => [], create: () => ({}), remove: () => false, toggle: () => false },
-    credentials: { list: () => [], create: () => ({}), remove: () => false, toggle: () => false },
-    notifications: {
-      getPreferences: () => undefined,
-      updatePreferences: () => ({}),
-      createEnabledEvent: () => ({}),
-      notifyUser: () => undefined,
-    },
-    pushSubscriptions: { save: () => undefined, remove: () => undefined },
-    getVapidPublicKey: () => null,
-    ...overrides,
-  };
-}
-
-test('listApiKeys redacts secret values through the service boundary', () => {
-  const service = createSettingsService(dependencies({
-    apiKeys: {
-      list: () => [{ id: 1, api_key: '1234567890-secret' }],
-      create: () => ({}), remove: () => false, toggle: () => false,
-    },
-  }));
-  assert.equal(service.listApiKeys(1).apiKeys[0]?.api_key, '1234567890...');
-});
-
-test('subscribeToPush persists the subscription and enables Web Push', () => {
-  const operations: string[] = [];
-  const service = createSettingsService(dependencies({
-    pushSubscriptions: {
-      save: (_id, endpoint) => operations.push(`save:${endpoint}`),
-      remove: () => undefined,
-    },
-    notifications: {
-      getPreferences: () => ({ channels: { webPush: false } }),
-      updatePreferences: () => { operations.push('preferences'); return {}; },
-      createEnabledEvent: () => ({ code: 'push.enabled' }),
-      notifyUser: () => { operations.push('notify'); },
-    },
-  }));
-
-  service.subscribeToPush(1, {
-    endpoint: 'https://push.example.test',
-    keys: { p256dh: 'key', auth: 'auth' },
-  });
-  assert.deepEqual(operations, ['save:https://push.example.test', 'preferences', 'notify']);
-});
+import assert from 'node:assert/strict';
+import test from 'node:test';
+
+import { createSettingsService } from '../settings.service.js';
+
+type Dependencies = Parameters<typeof createSettingsService>[0];
+
+function dependencies(overrides: Partial<Dependencies> = {}): Dependencies {
+  return {
+    apiKeys: { list: () => [], create: () => ({}), remove: () => false, toggle: () => false },
+    credentials: { list: () => [], create: () => ({}), remove: () => false, toggle: () => false },
+    notifications: {
+      getPreferences: () => undefined,
+      updatePreferences: () => ({}),
+      createEnabledEvent: () => ({}),
+      notifyUser: () => undefined,
+    },
+    pushSubscriptions: { save: () => undefined, remove: () => undefined },
+    getVapidPublicKey: () => null,
+    ...overrides,
+  };
+}
+
+test('listApiKeys redacts secret values through the service boundary', () => {
+  const service = createSettingsService(dependencies({
+    apiKeys: {
+      list: () => [{ id: 1, api_key: '1234567890-secret' }],
+      create: () => ({}), remove: () => false, toggle: () => false,
+    },
+  }));
+  assert.equal(service.listApiKeys(1).apiKeys[0]?.api_key, '1234567890...');
+});
+
+test('subscribeToPush persists the subscription and enables Web Push', () => {
+  const operations: string[] = [];
+  const service = createSettingsService(dependencies({
+    pushSubscriptions: {
+      save: (_id, endpoint) => operations.push(`save:${endpoint}`),
+      remove: () => undefined,
+    },
+    notifications: {
+      getPreferences: () => ({ channels: { webPush: false } }),
+      updatePreferences: () => { operations.push('preferences'); return {}; },
+      createEnabledEvent: () => ({ code: 'push.enabled' }),
+      notifyUser: () => { operations.push('notify'); },
+    },
+  }));
+
+  service.subscribeToPush(1, {
+    endpoint: 'https://push.example.test',
+    keys: { p256dh: 'key', auth: 'auth' },
+  });
+  assert.deepEqual(operations, ['save:https://push.example.test', 'preferences', 'notify']);
+});

package/server/modules/system/index.ts

@@ -1,2 +1,2 @@
-// createSystemModule: used by the server entrypoint to mount protected system update routes.
-export { createSystemModule } from './system.module.js';
+// createSystemModule: used by the server entrypoint to mount protected system update routes.
+export { createSystemModule } from './system.module.js';

package/server/modules/system/system.module.ts

@@ -1,62 +1,62 @@
-import os from 'node:os';
-
-import spawn from 'cross-spawn';
-import type { Router } from 'express';
-
-import { createSystemRouter } from './system.routes.js';
-import { createSystemUpdateService } from './system.service.js';
-
-type SystemModuleOptions = {
-  appRoot: string;
-  installMode: 'git' | 'npm';
-  isPlatform: boolean;
-};
-
-function runShellCommand(
-  command: string,
-  workingDirectory: string,
-  environment: NodeJS.ProcessEnv,
-  onOutput: (output: string) => void,
-  onErrorOutput: (errorOutput: string) => void,
-): Promise<{ exitCode: number | null; output: string; errorOutput: string }> {
-  return new Promise((resolve, reject) => {
-    const childProcess = spawn('sh', ['-c', command], {
-      cwd: workingDirectory,
-      env: environment,
-    });
-    let output = '';
-    let errorOutput = '';
-
-    childProcess.stdout?.on('data', (data: Buffer) => {
-      const text = data.toString();
-      output += text;
-      onOutput(text);
-    });
-    childProcess.stderr?.on('data', (data: Buffer) => {
-      const text = data.toString();
-      errorOutput += text;
-      onErrorOutput(text);
-    });
-    childProcess.once('error', reject);
-    childProcess.once('close', (exitCode) => {
-      resolve({ exitCode, output, errorOutput });
-    });
-  });
-}
-
-/**
- * Builds the authenticated system router for the server entrypoint using the
- * installation details it already resolves for health and startup metadata.
- */
-export function createSystemModule(options: SystemModuleOptions): Router {
-  const systemUpdateService = createSystemUpdateService({
-    ...options,
-    homeDirectory: os.homedir(),
-    environment: process.env,
-    runShellCommand,
-    logInfo: (message, detail) => console.log(message, detail ?? ''),
-    logError: (message, detail) => console.error(message, detail ?? ''),
-  });
-
-  return createSystemRouter(systemUpdateService);
-}
+import os from 'node:os';
+
+import spawn from 'cross-spawn';
+import type { Router } from 'express';
+
+import { createSystemRouter } from './system.routes.js';
+import { createSystemUpdateService } from './system.service.js';
+
+type SystemModuleOptions = {
+  appRoot: string;
+  installMode: 'git' | 'npm';
+  isPlatform: boolean;
+};
+
+function runShellCommand(
+  command: string,
+  workingDirectory: string,
+  environment: NodeJS.ProcessEnv,
+  onOutput: (output: string) => void,
+  onErrorOutput: (errorOutput: string) => void,
+): Promise<{ exitCode: number | null; output: string; errorOutput: string }> {
+  return new Promise((resolve, reject) => {
+    const childProcess = spawn('sh', ['-c', command], {
+      cwd: workingDirectory,
+      env: environment,
+    });
+    let output = '';
+    let errorOutput = '';
+
+    childProcess.stdout?.on('data', (data: Buffer) => {
+      const text = data.toString();
+      output += text;
+      onOutput(text);
+    });
+    childProcess.stderr?.on('data', (data: Buffer) => {
+      const text = data.toString();
+      errorOutput += text;
+      onErrorOutput(text);
+    });
+    childProcess.once('error', reject);
+    childProcess.once('close', (exitCode) => {
+      resolve({ exitCode, output, errorOutput });
+    });
+  });
+}
+
+/**
+ * Builds the authenticated system router for the server entrypoint using the
+ * installation details it already resolves for health and startup metadata.
+ */
+export function createSystemModule(options: SystemModuleOptions): Router {
+  const systemUpdateService = createSystemUpdateService({
+    ...options,
+    homeDirectory: os.homedir(),
+    environment: process.env,
+    runShellCommand,
+    logInfo: (message, detail) => console.log(message, detail ?? ''),
+    logError: (message, detail) => console.error(message, detail ?? ''),
+  });
+
+  return createSystemRouter(systemUpdateService);
+}

package/server/modules/system/system.routes.ts

@@ -1,21 +1,21 @@
-import express from 'express';
-
-import type { createSystemUpdateService } from './system.service.js';
-
-/** Creates thin system routes that delegate update execution to the service. */
-export function createSystemRouter(
-  systemUpdateService: ReturnType<typeof createSystemUpdateService>,
-): express.Router {
-  const router = express.Router();
-
-  router.post('/update', async (_request, response, next) => {
-    try {
-      const result = await systemUpdateService.updateSystem();
-      response.status(result.success ? 200 : 500).json(result);
-    } catch (error) {
-      next(error);
-    }
-  });
-
-  return router;
-}
+import express from 'express';
+
+import type { createSystemUpdateService } from './system.service.js';
+
+/** Creates thin system routes that delegate update execution to the service. */
+export function createSystemRouter(
+  systemUpdateService: ReturnType<typeof createSystemUpdateService>,
+): express.Router {
+  const router = express.Router();
+
+  router.post('/update', async (_request, response, next) => {
+    try {
+      const result = await systemUpdateService.updateSystem();
+      response.status(result.success ? 200 : 500).json(result);
+    } catch (error) {
+      next(error);
+    }
+  });
+
+  return router;
+}

package/server/modules/system/system.service.ts

@@ -1,76 +1,76 @@
-type SystemUpdateCommandResult = {
-  exitCode: number | null;
-  output: string;
-  errorOutput: string;
-};
-
-type SystemUpdateDependencies = {
-  appRoot: string;
-  homeDirectory: string;
-  installMode: 'git' | 'npm';
-  isPlatform: boolean;
-  environment: NodeJS.ProcessEnv;
-  runShellCommand(
-    command: string,
-    workingDirectory: string,
-    environment: NodeJS.ProcessEnv,
-    onOutput: (output: string) => void,
-    onErrorOutput: (errorOutput: string) => void,
-  ): Promise<SystemUpdateCommandResult>;
-  logInfo(message: string, detail?: string): void;
-  logError(message: string, detail?: string): void;
-};
-
-/**
- * Creates the update workflow used by the system module and its focused tests.
- * Runtime-specific process spawning stays behind the injected command adapter.
- */
-export function createSystemUpdateService(dependencies: SystemUpdateDependencies) {
-  return {
-    /** Selects and executes the correct update workflow for this installation. */
-    async updateSystem() {
-      const updateCommand = dependencies.isPlatform
-        ? 'npm run update:platform'
-        : dependencies.installMode === 'git'
-          ? 'git checkout main && git pull && npm install'
-          : 'npm install -g @cloudcli-ai/cloudcli@latest';
-      const workingDirectory = dependencies.isPlatform || dependencies.installMode === 'git'
-        ? dependencies.appRoot
-        : dependencies.homeDirectory;
-
-      dependencies.logInfo('Starting system update from directory:', workingDirectory);
-
-      try {
-        const result = await dependencies.runShellCommand(
-          updateCommand,
-          workingDirectory,
-          dependencies.environment,
-          (output) => dependencies.logInfo('Update output:', output),
-          (errorOutput) => dependencies.logError('Update error:', errorOutput),
-        );
-
-        if (result.exitCode === 0) {
-          return {
-            success: true as const,
-            output: result.output || 'Update completed successfully',
-            message: 'Update completed. Please restart the server to apply changes.',
-          };
-        }
-
-        return {
-          success: false as const,
-          error: 'Update command failed',
-          output: result.output,
-          errorOutput: result.errorOutput,
-        };
-      } catch (error) {
-        const message = error instanceof Error ? error.message : String(error);
-        dependencies.logError('Update process error:', message);
-        return {
-          success: false as const,
-          error: message,
-        };
-      }
-    },
-  };
-}
+type SystemUpdateCommandResult = {
+  exitCode: number | null;
+  output: string;
+  errorOutput: string;
+};
+
+type SystemUpdateDependencies = {
+  appRoot: string;
+  homeDirectory: string;
+  installMode: 'git' | 'npm';
+  isPlatform: boolean;
+  environment: NodeJS.ProcessEnv;
+  runShellCommand(
+    command: string,
+    workingDirectory: string,
+    environment: NodeJS.ProcessEnv,
+    onOutput: (output: string) => void,
+    onErrorOutput: (errorOutput: string) => void,
+  ): Promise<SystemUpdateCommandResult>;
+  logInfo(message: string, detail?: string): void;
+  logError(message: string, detail?: string): void;
+};
+
+/**
+ * Creates the update workflow used by the system module and its focused tests.
+ * Runtime-specific process spawning stays behind the injected command adapter.
+ */
+export function createSystemUpdateService(dependencies: SystemUpdateDependencies) {
+  return {
+    /** Selects and executes the correct update workflow for this installation. */
+    async updateSystem() {
+      const updateCommand = dependencies.isPlatform
+        ? 'npm run update:platform'
+        : dependencies.installMode === 'git'
+          ? 'git checkout main && git pull && npm install'
+          : 'npm install -g @cloudcli-ai/cloudcli@latest';
+      const workingDirectory = dependencies.isPlatform || dependencies.installMode === 'git'
+        ? dependencies.appRoot
+        : dependencies.homeDirectory;
+
+      dependencies.logInfo('Starting system update from directory:', workingDirectory);
+
+      try {
+        const result = await dependencies.runShellCommand(
+          updateCommand,
+          workingDirectory,
+          dependencies.environment,
+          (output) => dependencies.logInfo('Update output:', output),
+          (errorOutput) => dependencies.logError('Update error:', errorOutput),
+        );
+
+        if (result.exitCode === 0) {
+          return {
+            success: true as const,
+            output: result.output || 'Update completed successfully',
+            message: 'Update completed. Please restart the server to apply changes.',
+          };
+        }
+
+        return {
+          success: false as const,
+          error: 'Update command failed',
+          output: result.output,
+          errorOutput: result.errorOutput,
+        };
+      } catch (error) {
+        const message = error instanceof Error ? error.message : String(error);
+        dependencies.logError('Update process error:', message);
+        return {
+          success: false as const,
+          error: message,
+        };
+      }
+    },
+  };
+}