@studio-foundation/cli 0.3.0-beta.1 → 0.3.0-beta.5

package/tests/commands/api.test.ts

@@ -1,110 +0,0 @@
-import { describe, it, expect, vi, afterEach } from 'vitest';
-import { writeFile, rm } from 'node:fs/promises';
-import { join } from 'node:path';
-import { tmpdir } from 'node:os';
-
-// Use a unique fake home dir so tests don't collide with other test files using /tmp
-const FAKE_HOME = join(tmpdir(), `.studio-api-test-${process.pid}`);
-
-vi.mock('node:os', async (importOriginal) => {
-  const actual = await importOriginal<typeof import('node:os')>();
-  return { ...actual, homedir: () => FAKE_HOME };
-});
-
-// Import after mocking
-const { apiStopCommand, apiStatusCommand, writePid, readPid, clearPid, isProcessAlive } = await import('../../src/commands/api.js');
-
-afterEach(async () => {
-  await clearPid();
-  // Remove the .studio dir created by writePid
-  await rm(join(FAKE_HOME, '.studio'), { recursive: true, force: true });
-});
-
-describe('writePid / readPid / clearPid', () => {
-  it('writes and reads PID with port', async () => {
-    await writePid(3700);
-    const entry = await readPid();
-    expect(entry).not.toBeNull();
-    expect(entry!.pid).toBe(process.pid);
-    expect(entry!.port).toBe(3700);
-  });
-
-  it('readPid returns null when no file exists', async () => {
-    const entry = await readPid();
-    expect(entry).toBeNull();
-  });
-
-  it('clearPid removes the file', async () => {
-    await writePid(3700);
-    await clearPid();
-    const entry = await readPid();
-    expect(entry).toBeNull();
-  });
-});
-
-describe('isProcessAlive', () => {
-  it('returns true for the current process', () => {
-    expect(isProcessAlive(process.pid)).toBe(true);
-  });
-
-  it('returns false for a non-existent PID', () => {
-    // PID 999999999 is very unlikely to exist
-    expect(isProcessAlive(999999999)).toBe(false);
-  });
-});
-
-describe('apiStopCommand', () => {
-  it('prints "not running" when no PID file', async () => {
-    const consoleSpy = vi.spyOn(console, 'log').mockImplementation(() => {});
-    await apiStopCommand();
-    expect(consoleSpy).toHaveBeenCalledWith(expect.stringContaining('not running'));
-    consoleSpy.mockRestore();
-  });
-
-  it('prints "stale PID" and clears file when process is dead', async () => {
-    await writePid(3700);
-    // Overwrite with a dead PID
-    const pidFile = join(FAKE_HOME, '.studio', 'api.pid');
-    await writeFile(pidFile, '999999999:3700', 'utf-8');
-
-    const consoleSpy = vi.spyOn(console, 'log').mockImplementation(() => {});
-    await apiStopCommand();
-    expect(consoleSpy).toHaveBeenCalledWith(expect.stringContaining('stale'));
-    expect(await readPid()).toBeNull();
-    consoleSpy.mockRestore();
-  });
-
-  it('sends SIGTERM and clears PID when process is the current process', async () => {
-    await writePid(3700);
-    const killSpy = vi.spyOn(process, 'kill').mockImplementation(() => true);
-    const consoleSpy = vi.spyOn(console, 'log').mockImplementation(() => {});
-
-    await apiStopCommand();
-
-    expect(killSpy).toHaveBeenCalledWith(process.pid, 'SIGTERM');
-    expect(await readPid()).toBeNull();
-    killSpy.mockRestore();
-    consoleSpy.mockRestore();
-  });
-});
-
-describe('apiStatusCommand', () => {
-  it('prints "not running" when no PID file', async () => {
-    const consoleSpy = vi.spyOn(console, 'log').mockImplementation(() => {});
-    await apiStatusCommand({});
-    expect(consoleSpy).toHaveBeenCalledWith(expect.stringContaining('not running'));
-    consoleSpy.mockRestore();
-  });
-
-  it('prints "running" when process is alive and health check succeeds', async () => {
-    await writePid(3700);
-    // Override with current PID (alive) and mock fetch
-    global.fetch = vi.fn().mockResolvedValue({ ok: true }) as unknown as typeof fetch;
-    const consoleSpy = vi.spyOn(console, 'log').mockImplementation(() => {});
-
-    await apiStatusCommand({});
-
-    expect(consoleSpy).toHaveBeenCalledWith(expect.stringContaining('running'));
-    consoleSpy.mockRestore();
-  });
-});

package/tests/commands/config.test.ts

@@ -1,221 +0,0 @@
-import { describe, it, expect, beforeEach, afterEach } from 'vitest';
-import { mkdir, rm, readFile } from 'node:fs/promises';
-import { resolve } from 'node:path';
-import * as yaml from 'js-yaml';
-import {
-  getConfigValue,
-  setConfigValue,
-  maskSecrets,
-  PROVIDERS,
-  validateApiKeyForProvider,
-  addProviderConfig,
-  isProviderConfigured,
-} from '../../src/commands/config.js';
-
-const TMP = resolve(import.meta.dirname, '.tmp-config-cmd-test');
-const STUDIO_DIR = resolve(TMP, '.studio');
-
-beforeEach(async () => { await mkdir(STUDIO_DIR, { recursive: true }); });
-afterEach(async () => { await rm(TMP, { recursive: true, force: true }); });
-
-describe('getConfigValue', () => {
-  it('gets a nested value by dotted path', () => {
-    const config = { defaults: { model: 'claude-haiku' } };
-    expect(getConfigValue(config, 'defaults.model')).toBe('claude-haiku');
-  });
-
-  it('returns undefined for missing path', () => {
-    expect(getConfigValue({}, 'defaults.model')).toBeUndefined();
-  });
-});
-
-describe('setConfigValue', () => {
-  it('sets a nested value by dotted path', () => {
-    const config: Record<string, unknown> = {};
-    setConfigValue(config, 'defaults.model', 'claude-sonnet');
-    expect((config as any).defaults.model).toBe('claude-sonnet');
-  });
-
-  it('merges without destroying sibling keys', () => {
-    const config = { defaults: { provider: 'anthropic', model: 'old' } };
-    setConfigValue(config, 'defaults.model', 'new');
-    expect((config as any).defaults.provider).toBe('anthropic');
-    expect((config as any).defaults.model).toBe('new');
-  });
-});
-
-describe('maskSecrets', () => {
-  it('masks apiKey values', () => {
-    const config = { providers: { anthropic: { apiKey: 'sk-ant-longkey' } } };
-    const masked = maskSecrets(config);
-    expect((masked as any).providers.anthropic.apiKey).toMatch(/\*\*\*/);
-    expect((masked as any).providers.anthropic.apiKey).not.toContain('longkey');
-  });
-
-  it('preserves non-secret values', () => {
-    const config = { defaults: { model: 'claude-haiku' } };
-    expect((maskSecrets(config) as any).defaults.model).toBe('claude-haiku');
-  });
-});
-
-describe('PROVIDERS', () => {
-  it('includes anthropic, openai, google, ollama', () => {
-    const ids = PROVIDERS.map((p) => p.id);
-    expect(ids).toContain('anthropic');
-    expect(ids).toContain('openai');
-    expect(ids).toContain('google');
-    expect(ids).toContain('ollama');
-  });
-
-  it('each provider has id, label, and defaultModel', () => {
-    for (const p of PROVIDERS) {
-      expect(typeof p.id).toBe('string');
-      expect(typeof p.label).toBe('string');
-      expect(typeof p.defaultModel).toBe('string');
-    }
-  });
-});
-
-describe('validateApiKeyForProvider', () => {
-  it('accepts a valid Anthropic key (sk-ant-...)', () => {
-    expect(validateApiKeyForProvider('anthropic', 'sk-ant-api03-abc123')).toBe(true);
-  });
-
-  it('rejects an Anthropic key with wrong prefix', () => {
-    const result = validateApiKeyForProvider('anthropic', 'sk-wrong');
-    expect(typeof result).toBe('string');
-    expect(result).toContain('sk-ant-');
-  });
-
-  it('accepts a valid OpenAI key (sk-...)', () => {
-    expect(validateApiKeyForProvider('openai', 'sk-proj-abc123')).toBe(true);
-  });
-
-  it('rejects an OpenAI key with wrong prefix', () => {
-    const result = validateApiKeyForProvider('openai', 'wrong-key');
-    expect(typeof result).toBe('string');
-    expect(result).toContain('sk-');
-  });
-
-  it('rejects an Anthropic-prefixed key when provider is openai', () => {
-    const result = validateApiKeyForProvider('openai', 'sk-ant-api03-abc');
-    expect(typeof result).toBe('string');
-  });
-
-  it('accepts a valid Google key (AIza...)', () => {
-    expect(validateApiKeyForProvider('google', 'AIzaSyABC123')).toBe(true);
-  });
-
-  it('rejects a Google key with wrong prefix', () => {
-    const result = validateApiKeyForProvider('google', 'wrong-key');
-    expect(typeof result).toBe('string');
-    expect(result).toContain('AIza');
-  });
-
-  it('accepts any value for ollama provider (no validation)', () => {
-    expect(validateApiKeyForProvider('ollama', 'http://localhost:11434')).toBe(true);
-    expect(validateApiKeyForProvider('ollama', '')).toBe(true);
-  });
-
-  it('accepts any value for unknown providers', () => {
-    expect(validateApiKeyForProvider('future-provider', 'any-key')).toBe(true);
-  });
-});
-
-describe('addProviderConfig', () => {
-  const CONFIG_FILE = resolve(STUDIO_DIR, 'config.yaml');
-
-  it('writes provider apiKey to config.yaml', async () => {
-    await addProviderConfig(CONFIG_FILE, 'anthropic', 'sk-ant-key', false);
-
-    const raw = await readFile(CONFIG_FILE, 'utf-8');
-    const parsed = yaml.load(raw) as Record<string, unknown>;
-    const providers = parsed.providers as Record<string, { apiKey: string }>;
-    expect(providers.anthropic.apiKey).toBe('sk-ant-key');
-  });
-
-  it('sets defaults when setDefault=true', async () => {
-    await addProviderConfig(CONFIG_FILE, 'anthropic', 'sk-ant-key', true);
-
-    const raw = await readFile(CONFIG_FILE, 'utf-8');
-    const parsed = yaml.load(raw) as Record<string, unknown>;
-    const defaults = parsed.defaults as { provider: string; model: string };
-    expect(defaults.provider).toBe('anthropic');
-    expect(defaults.model).toBe('claude-sonnet-4-20250514');
-  });
-
-  it('does not touch defaults when setDefault=false', async () => {
-    await addProviderConfig(CONFIG_FILE, 'anthropic', 'sk-ant-key', false);
-
-    const raw = await readFile(CONFIG_FILE, 'utf-8');
-    const parsed = yaml.load(raw) as Record<string, unknown>;
-    expect(parsed.defaults).toBeUndefined();
-  });
-
-  it('preserves existing provider when adding a second', async () => {
-    await addProviderConfig(CONFIG_FILE, 'anthropic', 'sk-ant-key', false);
-    await addProviderConfig(CONFIG_FILE, 'openai', 'sk-openai-key', false);
-
-    const raw = await readFile(CONFIG_FILE, 'utf-8');
-    const parsed = yaml.load(raw) as Record<string, unknown>;
-    const providers = parsed.providers as Record<string, { apiKey: string }>;
-    expect(providers.anthropic.apiKey).toBe('sk-ant-key');
-    expect(providers.openai.apiKey).toBe('sk-openai-key');
-  });
-
-  it('overwrites existing provider key', async () => {
-    await addProviderConfig(CONFIG_FILE, 'anthropic', 'sk-ant-first', false);
-    await addProviderConfig(CONFIG_FILE, 'anthropic', 'sk-ant-second', false);
-
-    const raw = await readFile(CONFIG_FILE, 'utf-8');
-    const parsed = yaml.load(raw) as Record<string, unknown>;
-    const providers = parsed.providers as Record<string, { apiKey: string }>;
-    expect(providers.anthropic.apiKey).toBe('sk-ant-second');
-  });
-
-  it('creates config.yaml if it does not exist', async () => {
-    // STUDIO_DIR exists (from beforeEach) but config.yaml does not
-    await addProviderConfig(CONFIG_FILE, 'openai', 'sk-new-key', false);
-
-    const raw = await readFile(CONFIG_FILE, 'utf-8');
-    const parsed = yaml.load(raw) as Record<string, unknown>;
-    const providers = parsed.providers as Record<string, { apiKey: string }>;
-    expect(providers.openai.apiKey).toBe('sk-new-key');
-  });
-
-  it('uses correct defaultModel for openai', async () => {
-    await addProviderConfig(CONFIG_FILE, 'openai', 'sk-openai-key', true);
-
-    const raw = await readFile(CONFIG_FILE, 'utf-8');
-    const parsed = yaml.load(raw) as Record<string, unknown>;
-    const defaults = parsed.defaults as { model: string };
-    expect(defaults.model).toBe('gpt-4o');
-  });
-
-  it('uses claude-sonnet fallback for unknown provider', async () => {
-    await addProviderConfig(CONFIG_FILE, 'unknown-provider', 'some-key', true);
-
-    const raw = await readFile(CONFIG_FILE, 'utf-8');
-    const parsed = yaml.load(raw) as Record<string, unknown>;
-    const defaults = parsed.defaults as { model: string };
-    expect(defaults.model).toBe('claude-sonnet-4-20250514');
-  });
-});
-
-describe('isProviderConfigured', () => {
-  const CONFIG_FILE = resolve(STUDIO_DIR, 'config.yaml');
-
-  it('returns false when config.yaml does not exist', async () => {
-    expect(await isProviderConfigured(CONFIG_FILE, 'anthropic')).toBe(false);
-  });
-
-  it('returns false when provider not in config', async () => {
-    await addProviderConfig(CONFIG_FILE, 'openai', 'sk-key', false);
-    expect(await isProviderConfigured(CONFIG_FILE, 'anthropic')).toBe(false);
-  });
-
-  it('returns true when provider is in config', async () => {
-    await addProviderConfig(CONFIG_FILE, 'anthropic', 'sk-ant-key', false);
-    expect(await isProviderConfigured(CONFIG_FILE, 'anthropic')).toBe(true);
-  });
-});