wind-agent-cli 1.37.2 → 1.38.0

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

@@ -1,97 +1,97 @@
-import assert from 'node:assert/strict';
-import path from 'node:path';
-import test from 'node:test';
-
-import { createTaskmasterService } from '../taskmaster.service.js';
-
-type ServiceDependencies = Parameters<typeof createTaskmasterService>[0];
-
-function createDependencies(
-  homeDirectory: string,
-  files: Record<string, string>,
-): ServiceDependencies {
-  return {
-    getHomeDirectory: () => homeDirectory,
-    readTextFile: async (filePath) => {
-      const content = files[filePath];
-      if (content === undefined) {
-        throw new Error(`Missing fake file: ${filePath}`);
-      }
-      return content;
-    },
-  };
-}
-
-test('detectMcpServer returns a redacted TaskMaster server status', async () => {
-  const homeDirectory = path.join(path.sep, 'fake-home');
-  const configurationPath = path.join(homeDirectory, '.claude.json');
-  const service = createTaskmasterService(createDependencies(homeDirectory, {
-    [configurationPath]: JSON.stringify({
-      mcpServers: {
-        'task-master-ai': {
-          command: 'npx',
-          args: ['-y', 'task-master-ai'],
-          env: { ANTHROPIC_API_KEY: 'secret-value' },
-        },
-      },
-    }),
-  }));
-
-  assert.deepEqual(await service.detectMcpServer(), {
-    hasMCPServer: true,
-    isConfigured: true,
-    hasApiKeys: true,
-    scope: 'user',
-    config: {
-      command: 'npx',
-      args: ['-y', 'task-master-ai'],
-      url: null,
-      envVars: ['ANTHROPIC_API_KEY'],
-      type: 'stdio',
-    },
-  });
-});
-
-test('detectMcpServer checks the fallback configuration after malformed JSON', async () => {
-  const homeDirectory = path.join(path.sep, 'fake-home');
-  const primaryConfigurationPath = path.join(homeDirectory, '.claude.json');
-  const fallbackConfigurationPath = path.join(homeDirectory, '.claude', 'settings.json');
-  const service = createTaskmasterService(createDependencies(homeDirectory, {
-    [primaryConfigurationPath]: '{ malformed',
-    [fallbackConfigurationPath]: JSON.stringify({
-      projects: {
-        '/workspace/project': {
-          mcpServers: {
-            'project-task-master': { url: 'https://taskmaster.example.test/mcp' },
-          },
-        },
-      },
-    }),
-  }));
-
-  assert.deepEqual(await service.detectMcpServer(), {
-    hasMCPServer: true,
-    isConfigured: true,
-    hasApiKeys: false,
-    scope: 'local',
-    projectPath: '/workspace/project',
-    config: {
-      command: null,
-      args: [],
-      url: 'https://taskmaster.example.test/mcp',
-      envVars: [],
-      type: 'http',
-    },
-  });
-});
-
-test('detectMcpServer reports when no readable Claude configuration exists', async () => {
-  const homeDirectory = path.join(path.sep, 'fake-home');
-  const service = createTaskmasterService(createDependencies(homeDirectory, {}));
-
-  assert.deepEqual(await service.detectMcpServer(), {
-    hasMCPServer: false,
-    reason: 'No Claude configuration file found',
-    hasConfig: false,
-  });
-});
+import assert from 'node:assert/strict';
+import path from 'node:path';
+import test from 'node:test';
+
+import { createTaskmasterService } from '../taskmaster.service.js';
+
+type ServiceDependencies = Parameters<typeof createTaskmasterService>[0];
+
+function createDependencies(
+  homeDirectory: string,
+  files: Record<string, string>,
+): ServiceDependencies {
+  return {
+    getHomeDirectory: () => homeDirectory,
+    readTextFile: async (filePath) => {
+      const content = files[filePath];
+      if (content === undefined) {
+        throw new Error(`Missing fake file: ${filePath}`);
+      }
+      return content;
+    },
+  };
+}
+
+test('detectMcpServer returns a redacted TaskMaster server status', async () => {
+  const homeDirectory = path.join(path.sep, 'fake-home');
+  const configurationPath = path.join(homeDirectory, '.claude.json');
+  const service = createTaskmasterService(createDependencies(homeDirectory, {
+    [configurationPath]: JSON.stringify({
+      mcpServers: {
+        'task-master-ai': {
+          command: 'npx',
+          args: ['-y', 'task-master-ai'],
+          env: { ANTHROPIC_API_KEY: 'secret-value' },
+        },
+      },
+    }),
+  }));
+
+  assert.deepEqual(await service.detectMcpServer(), {
+    hasMCPServer: true,
+    isConfigured: true,
+    hasApiKeys: true,
+    scope: 'user',
+    config: {
+      command: 'npx',
+      args: ['-y', 'task-master-ai'],
+      url: null,
+      envVars: ['ANTHROPIC_API_KEY'],
+      type: 'stdio',
+    },
+  });
+});
+
+test('detectMcpServer checks the fallback configuration after malformed JSON', async () => {
+  const homeDirectory = path.join(path.sep, 'fake-home');
+  const primaryConfigurationPath = path.join(homeDirectory, '.claude.json');
+  const fallbackConfigurationPath = path.join(homeDirectory, '.claude', 'settings.json');
+  const service = createTaskmasterService(createDependencies(homeDirectory, {
+    [primaryConfigurationPath]: '{ malformed',
+    [fallbackConfigurationPath]: JSON.stringify({
+      projects: {
+        '/workspace/project': {
+          mcpServers: {
+            'project-task-master': { url: 'https://taskmaster.example.test/mcp' },
+          },
+        },
+      },
+    }),
+  }));
+
+  assert.deepEqual(await service.detectMcpServer(), {
+    hasMCPServer: true,
+    isConfigured: true,
+    hasApiKeys: false,
+    scope: 'local',
+    projectPath: '/workspace/project',
+    config: {
+      command: null,
+      args: [],
+      url: 'https://taskmaster.example.test/mcp',
+      envVars: [],
+      type: 'http',
+    },
+  });
+});
+
+test('detectMcpServer reports when no readable Claude configuration exists', async () => {
+  const homeDirectory = path.join(path.sep, 'fake-home');
+  const service = createTaskmasterService(createDependencies(homeDirectory, {}));
+
+  assert.deepEqual(await service.detectMcpServer(), {
+    hasMCPServer: false,
+    reason: 'No Claude configuration file found',
+    hasConfig: false,
+  });
+});

package/server/modules/user/index.ts

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

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

@@ -1,61 +1,61 @@
-import assert from 'node:assert/strict';
-import test from 'node:test';
-
-import { createUserService } from '../user.service.js';
-
-type UserDependencies = Parameters<typeof createUserService>[0];
-
-function createDependencies(overrides: Partial<UserDependencies> = {}): UserDependencies {
-  return {
-    users: {
-      getGitConfig: () => undefined,
-      updateGitConfig: () => undefined,
-      completeOnboarding: () => undefined,
-      hasCompletedOnboarding: () => false,
-    },
-    readSystemGitConfig: async () => ({ git_name: null, git_email: null }),
-    applyGlobalGitConfig: async () => undefined,
-    logInfo: () => undefined,
-    logError: () => undefined,
-    ...overrides,
-  };
-}
-
-test('getGitConfig imports system configuration when the repository is empty', async () => {
-  const updates: unknown[][] = [];
-  const service = createUserService(createDependencies({
-    users: {
-      getGitConfig: () => undefined,
-      updateGitConfig: (...args) => updates.push(args),
-      completeOnboarding: () => undefined,
-      hasCompletedOnboarding: () => false,
-    },
-    readSystemGitConfig: async () => ({ git_name: 'Alice', git_email: 'alice@example.com' }),
-  }));
-
-  const result = await service.getGitConfig(7);
-
-  assert.equal(result.gitName, 'Alice');
-  assert.deepEqual(updates, [[7, 'Alice', 'alice@example.com']]);
-});
-
-test('updateGitConfig persists valid input and invokes the Git adapter', async () => {
-  const operations: string[] = [];
-  const service = createUserService(createDependencies({
-    users: {
-      getGitConfig: () => undefined,
-      updateGitConfig: (_id, name, email) => operations.push(`persist:${name}:${email}`),
-      completeOnboarding: () => undefined,
-      hasCompletedOnboarding: () => false,
-    },
-    applyGlobalGitConfig: async (name, email) => {
-      operations.push(`git:${name}:${email}`);
-    },
-  }));
-
-  await service.updateGitConfig(1, 'Alice', 'alice@example.com');
-  assert.deepEqual(operations, [
-    'persist:Alice:alice@example.com',
-    'git:Alice:alice@example.com',
-  ]);
-});
+import assert from 'node:assert/strict';
+import test from 'node:test';
+
+import { createUserService } from '../user.service.js';
+
+type UserDependencies = Parameters<typeof createUserService>[0];
+
+function createDependencies(overrides: Partial<UserDependencies> = {}): UserDependencies {
+  return {
+    users: {
+      getGitConfig: () => undefined,
+      updateGitConfig: () => undefined,
+      completeOnboarding: () => undefined,
+      hasCompletedOnboarding: () => false,
+    },
+    readSystemGitConfig: async () => ({ git_name: null, git_email: null }),
+    applyGlobalGitConfig: async () => undefined,
+    logInfo: () => undefined,
+    logError: () => undefined,
+    ...overrides,
+  };
+}
+
+test('getGitConfig imports system configuration when the repository is empty', async () => {
+  const updates: unknown[][] = [];
+  const service = createUserService(createDependencies({
+    users: {
+      getGitConfig: () => undefined,
+      updateGitConfig: (...args) => updates.push(args),
+      completeOnboarding: () => undefined,
+      hasCompletedOnboarding: () => false,
+    },
+    readSystemGitConfig: async () => ({ git_name: 'Alice', git_email: 'alice@example.com' }),
+  }));
+
+  const result = await service.getGitConfig(7);
+
+  assert.equal(result.gitName, 'Alice');
+  assert.deepEqual(updates, [[7, 'Alice', 'alice@example.com']]);
+});
+
+test('updateGitConfig persists valid input and invokes the Git adapter', async () => {
+  const operations: string[] = [];
+  const service = createUserService(createDependencies({
+    users: {
+      getGitConfig: () => undefined,
+      updateGitConfig: (_id, name, email) => operations.push(`persist:${name}:${email}`),
+      completeOnboarding: () => undefined,
+      hasCompletedOnboarding: () => false,
+    },
+    applyGlobalGitConfig: async (name, email) => {
+      operations.push(`git:${name}:${email}`);
+    },
+  }));
+
+  await service.updateGitConfig(1, 'Alice', 'alice@example.com');
+  assert.deepEqual(operations, [
+    'persist:Alice:alice@example.com',
+    'git:Alice:alice@example.com',
+  ]);
+});

package/server/modules/user/user.module.ts

@@ -1,58 +1,58 @@
-import spawn from 'cross-spawn';
-
-import { userDb } from '@/modules/database/index.js';
-
-import { createUserRouter } from './user.routes.js';
-import { createUserService } from './user.service.js';
-
-type GitCommandResult = { stdout: string };
-
-function runGit(args: string[]): Promise<GitCommandResult> {
-  return new Promise((resolve, reject) => {
-    const child = spawn('git', args, { shell: false });
-    let stdout = '';
-    child.stdout?.on('data', (data: Buffer) => { stdout += data.toString(); });
-    child.on('error', reject);
-    child.on('close', (code) => {
-      if (code === 0) {
-        resolve({ stdout });
-        return;
-      }
-      reject(new Error(`Git command failed with code ${code}`));
-    });
-  });
-}
-
-async function readSystemGitConfig() {
-  const [nameResult, emailResult] = await Promise.all([
-    runGit(['config', '--global', 'user.name']).catch(() => ({ stdout: '' })),
-    runGit(['config', '--global', 'user.email']).catch(() => ({ stdout: '' })),
-  ]);
-  return {
-    git_name: nameResult.stdout.trim() || null,
-    git_email: emailResult.stdout.trim() || null,
-  };
-}
-
-const userService = createUserService({
-  users: {
-    getGitConfig: (userId) => userDb.getGitConfig(userId),
-    updateGitConfig: (userId, gitName, gitEmail) => userDb.updateGitConfig(
-      userId,
-      gitName ?? '',
-      gitEmail ?? '',
-    ),
-    completeOnboarding: (userId) => userDb.completeOnboarding(userId),
-    hasCompletedOnboarding: (userId) => userDb.hasCompletedOnboarding(userId),
-  },
-  readSystemGitConfig,
-  applyGlobalGitConfig: async (gitName, gitEmail) => {
-    await runGit(['config', '--global', 'user.name', gitName]);
-    await runGit(['config', '--global', 'user.email', gitEmail]);
-  },
-  logInfo: (message) => console.log(message),
-  logError: (message, error) => console.error(message, error),
-});
-
-/** User router assembled for the authenticated server mount. */
-export const userRoutes = createUserRouter(userService);
+import spawn from 'cross-spawn';
+
+import { userDb } from '@/modules/database/index.js';
+
+import { createUserRouter } from './user.routes.js';
+import { createUserService } from './user.service.js';
+
+type GitCommandResult = { stdout: string };
+
+function runGit(args: string[]): Promise<GitCommandResult> {
+  return new Promise((resolve, reject) => {
+    const child = spawn('git', args, { shell: false });
+    let stdout = '';
+    child.stdout?.on('data', (data: Buffer) => { stdout += data.toString(); });
+    child.on('error', reject);
+    child.on('close', (code) => {
+      if (code === 0) {
+        resolve({ stdout });
+        return;
+      }
+      reject(new Error(`Git command failed with code ${code}`));
+    });
+  });
+}
+
+async function readSystemGitConfig() {
+  const [nameResult, emailResult] = await Promise.all([
+    runGit(['config', '--global', 'user.name']).catch(() => ({ stdout: '' })),
+    runGit(['config', '--global', 'user.email']).catch(() => ({ stdout: '' })),
+  ]);
+  return {
+    git_name: nameResult.stdout.trim() || null,
+    git_email: emailResult.stdout.trim() || null,
+  };
+}
+
+const userService = createUserService({
+  users: {
+    getGitConfig: (userId) => userDb.getGitConfig(userId),
+    updateGitConfig: (userId, gitName, gitEmail) => userDb.updateGitConfig(
+      userId,
+      gitName ?? '',
+      gitEmail ?? '',
+    ),
+    completeOnboarding: (userId) => userDb.completeOnboarding(userId),
+    hasCompletedOnboarding: (userId) => userDb.hasCompletedOnboarding(userId),
+  },
+  readSystemGitConfig,
+  applyGlobalGitConfig: async (gitName, gitEmail) => {
+    await runGit(['config', '--global', 'user.name', gitName]);
+    await runGit(['config', '--global', 'user.email', gitEmail]);
+  },
+  logInfo: (message) => console.log(message),
+  logError: (message, error) => console.error(message, error),
+});
+
+/** User router assembled for the authenticated server mount. */
+export const userRoutes = createUserRouter(userService);

package/server/modules/user/user.routes.ts

@@ -1,50 +1,50 @@
-import express from 'express';
-
-import type { createUserService } from './user.service.js';
-
-type AuthenticatedRequest = express.Request & { user?: { id?: number | string } };
-
-function readUserId(request: express.Request): number {
-  const rawUserId = (request as AuthenticatedRequest).user?.id;
-  return Number(rawUserId);
-}
-
-/** Creates thin user routes that parse authenticated input and call the service. */
-export function createUserRouter(service: ReturnType<typeof createUserService>): express.Router {
-  const router = express.Router();
-
-  router.get('/git-config', async (req, res, next) => {
-    try {
-      res.json(await service.getGitConfig(readUserId(req)));
-    } catch (error) {
-      next(error);
-    }
-  });
-
-  router.post('/git-config', async (req, res, next) => {
-    try {
-      const body = req.body as { gitName?: unknown; gitEmail?: unknown };
-      res.json(await service.updateGitConfig(readUserId(req), body.gitName, body.gitEmail));
-    } catch (error) {
-      next(error);
-    }
-  });
-
-  router.post('/complete-onboarding', (req, res, next) => {
-    try {
-      res.json(service.completeOnboarding(readUserId(req)));
-    } catch (error) {
-      next(error);
-    }
-  });
-
-  router.get('/onboarding-status', (req, res, next) => {
-    try {
-      res.json(service.getOnboardingStatus(readUserId(req)));
-    } catch (error) {
-      next(error);
-    }
-  });
-
-  return router;
-}
+import express from 'express';
+
+import type { createUserService } from './user.service.js';
+
+type AuthenticatedRequest = express.Request & { user?: { id?: number | string } };
+
+function readUserId(request: express.Request): number {
+  const rawUserId = (request as AuthenticatedRequest).user?.id;
+  return Number(rawUserId);
+}
+
+/** Creates thin user routes that parse authenticated input and call the service. */
+export function createUserRouter(service: ReturnType<typeof createUserService>): express.Router {
+  const router = express.Router();
+
+  router.get('/git-config', async (req, res, next) => {
+    try {
+      res.json(await service.getGitConfig(readUserId(req)));
+    } catch (error) {
+      next(error);
+    }
+  });
+
+  router.post('/git-config', async (req, res, next) => {
+    try {
+      const body = req.body as { gitName?: unknown; gitEmail?: unknown };
+      res.json(await service.updateGitConfig(readUserId(req), body.gitName, body.gitEmail));
+    } catch (error) {
+      next(error);
+    }
+  });
+
+  router.post('/complete-onboarding', (req, res, next) => {
+    try {
+      res.json(service.completeOnboarding(readUserId(req)));
+    } catch (error) {
+      next(error);
+    }
+  });
+
+  router.get('/onboarding-status', (req, res, next) => {
+    try {
+      res.json(service.getOnboardingStatus(readUserId(req)));
+    } catch (error) {
+      next(error);
+    }
+  });
+
+  return router;
+}