wind-agent-cli 1.37.2 → 1.37.3

package/server/modules/providers/tests/claude-auth.test.ts

@@ -1,150 +1,150 @@
-import assert from 'node:assert/strict';
-import { mkdir, mkdtemp, rm, writeFile } from 'node:fs/promises';
-import os from 'node:os';
-import path from 'node:path';
-import test from 'node:test';
-
-import { ClaudeProviderAuth } from '@/modules/providers/list/claude/claude-auth.provider.js';
-
-// checkCredentials() is private, but unlike getStatus() it never shells out to the
-// `claude` CLI — it only reads env vars and ~/.claude files. Calling it directly
-// (TypeScript's `private` has no runtime effect) tests the priority order without
-// depending on `claude` being installed in the test environment.
-type CheckCredentialsResult = {
-  authenticated: boolean;
-  email: string | null;
-  method: string | null;
-  error?: string;
-};
-
-const checkCredentials = (auth: ClaudeProviderAuth): Promise<CheckCredentialsResult> =>
-  (auth as unknown as { checkCredentials: () => Promise<CheckCredentialsResult> }).checkCredentials();
-
-const ENV_KEYS = ['CLAUDE_CODE_OAUTH_TOKEN', 'ANTHROPIC_API_KEY', 'ANTHROPIC_AUTH_TOKEN'] as const;
-
-const withEnv = async (
-  overrides: Partial<Record<(typeof ENV_KEYS)[number], string>>,
-  fn: () => Promise<void>,
-) => {
-  const original: Partial<Record<(typeof ENV_KEYS)[number], string>> = {};
-  for (const key of ENV_KEYS) {
-    original[key] = process.env[key];
-    const value = overrides[key];
-    if (value === undefined) {
-      delete process.env[key];
-    } else {
-      process.env[key] = value;
-    }
-  }
-  try {
-    await fn();
-  } finally {
-    for (const key of ENV_KEYS) {
-      if (original[key] === undefined) {
-        delete process.env[key];
-      } else {
-        process.env[key] = original[key];
-      }
-    }
-  }
-};
-
-const withTempHome = async (fn: (homeDir: string) => Promise<void>) => {
-  const homeDir = await mkdtemp(path.join(os.tmpdir(), 'claude-auth-test-'));
-  const originalHome = process.env.HOME;
-  process.env.HOME = homeDir;
-  try {
-    await fn(homeDir);
-  } finally {
-    if (originalHome === undefined) {
-      delete process.env.HOME;
-    } else {
-      process.env.HOME = originalHome;
-    }
-    await rm(homeDir, { recursive: true, force: true });
-  }
-};
-
-const writeCredentialsFile = async (homeDir: string, body: unknown) => {
-  const claudeDir = path.join(homeDir, '.claude');
-  await mkdir(claudeDir, { recursive: true });
-  await writeFile(path.join(claudeDir, '.credentials.json'), JSON.stringify(body));
-};
-
-const writeSettingsFile = async (homeDir: string, env: Record<string, string>) => {
-  const claudeDir = path.join(homeDir, '.claude');
-  await mkdir(claudeDir, { recursive: true });
-  await writeFile(path.join(claudeDir, 'settings.json'), JSON.stringify({ env }));
-};
-
-test('checkCredentials: CLAUDE_CODE_OAUTH_TOKEN set is authenticated via environment, even with a stale credentials file', async () => {
-  await withTempHome(async (homeDir) => {
-    await writeCredentialsFile(homeDir, {
-      claudeAiOauth: { accessToken: 'stale-token', expiresAt: 1_000_000_000_000 }, // long expired
-    });
-
-    await withEnv({ CLAUDE_CODE_OAUTH_TOKEN: 'test-oauth-token' }, async () => {
-      const status = await checkCredentials(new ClaudeProviderAuth());
-      assert.equal(status.authenticated, true);
-      assert.equal(status.method, 'environment');
-    });
-  });
-});
-
-test('checkCredentials: CLAUDE_CODE_OAUTH_TOKEN configured via settings.json env block is authenticated via environment', async () => {
-  await withTempHome(async (homeDir) => {
-    await writeSettingsFile(homeDir, { CLAUDE_CODE_OAUTH_TOKEN: 'test-oauth-token-from-settings' });
-    await writeCredentialsFile(homeDir, {
-      claudeAiOauth: { accessToken: 'stale-token', expiresAt: 1_000_000_000_000 }, // long expired
-    });
-
-    await withEnv({}, async () => {
-      const status = await checkCredentials(new ClaudeProviderAuth());
-      assert.equal(status.authenticated, true);
-      assert.equal(status.method, 'environment');
-    });
-  });
-});
-
-test('checkCredentials: no CLAUDE_CODE_OAUTH_TOKEN, valid credentials file falls back to credentials_file', async () => {
-  await withTempHome(async (homeDir) => {
-    await writeCredentialsFile(homeDir, {
-      claudeAiOauth: { accessToken: 'valid-token', expiresAt: Date.now() + 60 * 60 * 1000 },
-      email: 'someone@example.com',
-    });
-
-    await withEnv({}, async () => {
-      const status = await checkCredentials(new ClaudeProviderAuth());
-      assert.equal(status.authenticated, true);
-      assert.equal(status.method, 'credentials_file');
-      assert.equal(status.email, 'someone@example.com');
-    });
-  });
-});
-
-test('checkCredentials: no CLAUDE_CODE_OAUTH_TOKEN, expired credentials file reports not authenticated', async () => {
-  await withTempHome(async (homeDir) => {
-    await writeCredentialsFile(homeDir, {
-      claudeAiOauth: { accessToken: 'stale-token', expiresAt: 1_000_000_000_000 },
-    });
-
-    await withEnv({}, async () => {
-      const status = await checkCredentials(new ClaudeProviderAuth());
-      assert.equal(status.authenticated, false);
-      assert.match(status.error ?? '', /expired/i);
-    });
-  });
-});
-
-test('checkCredentials: ANTHROPIC_API_KEY takes precedence over CLAUDE_CODE_OAUTH_TOKEN', async () => {
-  await withTempHome(async () => {
-    await withEnv(
-      { ANTHROPIC_API_KEY: 'test-api-key', CLAUDE_CODE_OAUTH_TOKEN: 'test-oauth-token' },
-      async () => {
-        const status = await checkCredentials(new ClaudeProviderAuth());
-        assert.equal(status.authenticated, true);
-        assert.equal(status.method, 'api_key');
-      },
-    );
-  });
-});
+import assert from 'node:assert/strict';
+import { mkdir, mkdtemp, rm, writeFile } from 'node:fs/promises';
+import os from 'node:os';
+import path from 'node:path';
+import test from 'node:test';
+
+import { ClaudeProviderAuth } from '@/modules/providers/list/claude/claude-auth.provider.js';
+
+// checkCredentials() is private, but unlike getStatus() it never shells out to the
+// `claude` CLI — it only reads env vars and ~/.claude files. Calling it directly
+// (TypeScript's `private` has no runtime effect) tests the priority order without
+// depending on `claude` being installed in the test environment.
+type CheckCredentialsResult = {
+  authenticated: boolean;
+  email: string | null;
+  method: string | null;
+  error?: string;
+};
+
+const checkCredentials = (auth: ClaudeProviderAuth): Promise<CheckCredentialsResult> =>
+  (auth as unknown as { checkCredentials: () => Promise<CheckCredentialsResult> }).checkCredentials();
+
+const ENV_KEYS = ['CLAUDE_CODE_OAUTH_TOKEN', 'ANTHROPIC_API_KEY', 'ANTHROPIC_AUTH_TOKEN'] as const;
+
+const withEnv = async (
+  overrides: Partial<Record<(typeof ENV_KEYS)[number], string>>,
+  fn: () => Promise<void>,
+) => {
+  const original: Partial<Record<(typeof ENV_KEYS)[number], string>> = {};
+  for (const key of ENV_KEYS) {
+    original[key] = process.env[key];
+    const value = overrides[key];
+    if (value === undefined) {
+      delete process.env[key];
+    } else {
+      process.env[key] = value;
+    }
+  }
+  try {
+    await fn();
+  } finally {
+    for (const key of ENV_KEYS) {
+      if (original[key] === undefined) {
+        delete process.env[key];
+      } else {
+        process.env[key] = original[key];
+      }
+    }
+  }
+};
+
+const withTempHome = async (fn: (homeDir: string) => Promise<void>) => {
+  const homeDir = await mkdtemp(path.join(os.tmpdir(), 'claude-auth-test-'));
+  const originalHome = process.env.HOME;
+  process.env.HOME = homeDir;
+  try {
+    await fn(homeDir);
+  } finally {
+    if (originalHome === undefined) {
+      delete process.env.HOME;
+    } else {
+      process.env.HOME = originalHome;
+    }
+    await rm(homeDir, { recursive: true, force: true });
+  }
+};
+
+const writeCredentialsFile = async (homeDir: string, body: unknown) => {
+  const claudeDir = path.join(homeDir, '.claude');
+  await mkdir(claudeDir, { recursive: true });
+  await writeFile(path.join(claudeDir, '.credentials.json'), JSON.stringify(body));
+};
+
+const writeSettingsFile = async (homeDir: string, env: Record<string, string>) => {
+  const claudeDir = path.join(homeDir, '.claude');
+  await mkdir(claudeDir, { recursive: true });
+  await writeFile(path.join(claudeDir, 'settings.json'), JSON.stringify({ env }));
+};
+
+test('checkCredentials: CLAUDE_CODE_OAUTH_TOKEN set is authenticated via environment, even with a stale credentials file', async () => {
+  await withTempHome(async (homeDir) => {
+    await writeCredentialsFile(homeDir, {
+      claudeAiOauth: { accessToken: 'stale-token', expiresAt: 1_000_000_000_000 }, // long expired
+    });
+
+    await withEnv({ CLAUDE_CODE_OAUTH_TOKEN: 'test-oauth-token' }, async () => {
+      const status = await checkCredentials(new ClaudeProviderAuth());
+      assert.equal(status.authenticated, true);
+      assert.equal(status.method, 'environment');
+    });
+  });
+});
+
+test('checkCredentials: CLAUDE_CODE_OAUTH_TOKEN configured via settings.json env block is authenticated via environment', async () => {
+  await withTempHome(async (homeDir) => {
+    await writeSettingsFile(homeDir, { CLAUDE_CODE_OAUTH_TOKEN: 'test-oauth-token-from-settings' });
+    await writeCredentialsFile(homeDir, {
+      claudeAiOauth: { accessToken: 'stale-token', expiresAt: 1_000_000_000_000 }, // long expired
+    });
+
+    await withEnv({}, async () => {
+      const status = await checkCredentials(new ClaudeProviderAuth());
+      assert.equal(status.authenticated, true);
+      assert.equal(status.method, 'environment');
+    });
+  });
+});
+
+test('checkCredentials: no CLAUDE_CODE_OAUTH_TOKEN, valid credentials file falls back to credentials_file', async () => {
+  await withTempHome(async (homeDir) => {
+    await writeCredentialsFile(homeDir, {
+      claudeAiOauth: { accessToken: 'valid-token', expiresAt: Date.now() + 60 * 60 * 1000 },
+      email: 'someone@example.com',
+    });
+
+    await withEnv({}, async () => {
+      const status = await checkCredentials(new ClaudeProviderAuth());
+      assert.equal(status.authenticated, true);
+      assert.equal(status.method, 'credentials_file');
+      assert.equal(status.email, 'someone@example.com');
+    });
+  });
+});
+
+test('checkCredentials: no CLAUDE_CODE_OAUTH_TOKEN, expired credentials file reports not authenticated', async () => {
+  await withTempHome(async (homeDir) => {
+    await writeCredentialsFile(homeDir, {
+      claudeAiOauth: { accessToken: 'stale-token', expiresAt: 1_000_000_000_000 },
+    });
+
+    await withEnv({}, async () => {
+      const status = await checkCredentials(new ClaudeProviderAuth());
+      assert.equal(status.authenticated, false);
+      assert.match(status.error ?? '', /expired/i);
+    });
+  });
+});
+
+test('checkCredentials: ANTHROPIC_API_KEY takes precedence over CLAUDE_CODE_OAUTH_TOKEN', async () => {
+  await withTempHome(async () => {
+    await withEnv(
+      { ANTHROPIC_API_KEY: 'test-api-key', CLAUDE_CODE_OAUTH_TOKEN: 'test-oauth-token' },
+      async () => {
+        const status = await checkCredentials(new ClaudeProviderAuth());
+        assert.equal(status.authenticated, true);
+        assert.equal(status.method, 'api_key');
+      },
+    );
+  });
+});

package/server/modules/providers/tests/claude-sessions.test.ts

@@ -1,61 +1,61 @@
-import assert from 'node:assert/strict';
-import test from 'node:test';
-
-import { ClaudeSessionsProvider } from '@/modules/providers/list/claude/claude-sessions.provider.js';
-
-const SESSION_ID = 'session-1';
-
-const SKILL_BODY = [
-  'Base directory for this skill: /tmp/claude/bundled-skills/2.1.220/abc123/claude-api',
-  '',
-  '# Building LLM-Powered Applications with Claude',
-  '',
-  'This skill helps you build LLM-powered applications with Claude.',
-].join('\n');
-
-test('claude: injected skill bodies are hidden even without the isMeta flag', () => {
-  const provider = new ClaudeSessionsProvider();
-
-  // The live SDK stream omits `isMeta`, so the payload has to be recognised by
-  // its content or it renders as a giant user bubble mid-run.
-  const live = provider.normalizeMessage(
-    {
-      uuid: 'u1',
-      timestamp: '2026-07-28T10:00:00.000Z',
-      message: { role: 'user', content: [{ type: 'text', text: SKILL_BODY }] },
-    },
-    SESSION_ID,
-  );
-  assert.deepEqual(live, []);
-
-  const persisted = provider.normalizeMessage(
-    {
-      uuid: 'u2',
-      timestamp: '2026-07-28T10:00:00.000Z',
-      isMeta: true,
-      message: { role: 'user', content: [{ type: 'text', text: SKILL_BODY }] },
-    },
-    SESSION_ID,
-  );
-  assert.deepEqual(persisted, []);
-});
-
-test('claude: the Skill tool result itself still reaches the UI', () => {
-  const provider = new ClaudeSessionsProvider();
-
-  const messages = provider.normalizeMessage(
-    {
-      uuid: 'u3',
-      timestamp: '2026-07-28T10:00:00.000Z',
-      message: {
-        role: 'user',
-        content: [{ type: 'tool_result', tool_use_id: 'toolu_1', content: 'Launching skill: claude-api' }],
-      },
-    },
-    SESSION_ID,
-  );
-
-  assert.equal(messages.length, 1);
-  assert.equal(messages[0].kind, 'tool_result');
-  assert.equal(messages[0].toolId, 'toolu_1');
-});
+import assert from 'node:assert/strict';
+import test from 'node:test';
+
+import { ClaudeSessionsProvider } from '@/modules/providers/list/claude/claude-sessions.provider.js';
+
+const SESSION_ID = 'session-1';
+
+const SKILL_BODY = [
+  'Base directory for this skill: /tmp/claude/bundled-skills/2.1.220/abc123/claude-api',
+  '',
+  '# Building LLM-Powered Applications with Claude',
+  '',
+  'This skill helps you build LLM-powered applications with Claude.',
+].join('\n');
+
+test('claude: injected skill bodies are hidden even without the isMeta flag', () => {
+  const provider = new ClaudeSessionsProvider();
+
+  // The live SDK stream omits `isMeta`, so the payload has to be recognised by
+  // its content or it renders as a giant user bubble mid-run.
+  const live = provider.normalizeMessage(
+    {
+      uuid: 'u1',
+      timestamp: '2026-07-28T10:00:00.000Z',
+      message: { role: 'user', content: [{ type: 'text', text: SKILL_BODY }] },
+    },
+    SESSION_ID,
+  );
+  assert.deepEqual(live, []);
+
+  const persisted = provider.normalizeMessage(
+    {
+      uuid: 'u2',
+      timestamp: '2026-07-28T10:00:00.000Z',
+      isMeta: true,
+      message: { role: 'user', content: [{ type: 'text', text: SKILL_BODY }] },
+    },
+    SESSION_ID,
+  );
+  assert.deepEqual(persisted, []);
+});
+
+test('claude: the Skill tool result itself still reaches the UI', () => {
+  const provider = new ClaudeSessionsProvider();
+
+  const messages = provider.normalizeMessage(
+    {
+      uuid: 'u3',
+      timestamp: '2026-07-28T10:00:00.000Z',
+      message: {
+        role: 'user',
+        content: [{ type: 'tool_result', tool_use_id: 'toolu_1', content: 'Launching skill: claude-api' }],
+      },
+    },
+    SESSION_ID,
+  );
+
+  assert.equal(messages.length, 1);
+  assert.equal(messages[0].kind, 'tool_result');
+  assert.equal(messages[0].toolId, 'toolu_1');
+});