@geminilight/mindos 0.6.71 → 0.6.73

package/tests/unit/cli-build.test.ts

@@ -1,180 +0,0 @@
-import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest';
-import fs from 'fs';
-import os from 'os';
-import path from 'path';
-
-/**
- * Tests for bin/lib/build.js — needsBuild, writeBuildStamp, cleanNextDir, ensureAppDeps.
- *
- * We mock constants.js to point ROOT/BUILD_STAMP/DEPS_STAMP at a temp directory,
- * and mock execSync to avoid real npm install.
- */
-
-let tempDir: string;
-let appDir: string;
-let nextDir: string;
-let buildStamp: string;
-let depsStamp: string;
-
-beforeEach(() => {
-  tempDir = fs.mkdtempSync(path.join(os.tmpdir(), 'mindos-build-test-'));
-  appDir = path.join(tempDir, 'app');
-  nextDir = path.join(appDir, '.next');
-  buildStamp = path.join(nextDir, '.mindos-build-version');
-  depsStamp = path.join(tempDir, 'deps-hash');
-
-  // Create app dir with a package.json
-  fs.mkdirSync(appDir, { recursive: true });
-  fs.writeFileSync(path.join(appDir, 'package.json'), JSON.stringify({ name: 'test-app', version: '1.0.0' }));
-
-  // Create root package.json
-  fs.writeFileSync(path.join(tempDir, 'package.json'), JSON.stringify({ name: 'mindos', version: '0.5.12' }));
-
-  vi.resetModules();
-
-  vi.doMock('../../bin/lib/constants.js', () => ({
-    ROOT: tempDir,
-    BUILD_STAMP: buildStamp,
-    DEPS_STAMP: depsStamp,
-    CONFIG_PATH: path.join(tempDir, 'config.json'),
-    MINDOS_DIR: tempDir,
-    PID_PATH: path.join(tempDir, 'mindos.pid'),
-    LOG_PATH: path.join(tempDir, 'mindos.log'),
-    CLI_PATH: '',
-    NODE_BIN: process.execPath,
-    UPDATE_CHECK_PATH: path.join(tempDir, 'update-check.json'),
-    STANDALONE_SERVER: path.join(tempDir, '_standalone', 'server.js'),
-    STANDALONE_STAMP: path.join(tempDir, '_standalone', '.mindos-build-version'),
-  }));
-});
-
-afterEach(() => {
-  fs.rmSync(tempDir, { recursive: true, force: true });
-  vi.restoreAllMocks();
-});
-
-async function importBuild() {
-  return await import('../../bin/lib/build.js') as {
-    needsBuild: () => boolean;
-    writeBuildStamp: () => void;
-    cleanNextDir: () => void;
-    clearBuildLock: () => void;
-    ensureAppDeps: () => void;
-  };
-}
-
-// ── needsBuild ──────────────────────────────────────────────────────────────
-
-describe('needsBuild', () => {
-  it('returns true when .next directory does not exist', async () => {
-    const { needsBuild } = await importBuild();
-    expect(needsBuild()).toBe(true);
-  });
-
-  it('returns true when .next exists but no build stamp', async () => {
-    fs.mkdirSync(nextDir, { recursive: true });
-    const { needsBuild } = await importBuild();
-    expect(needsBuild()).toBe(true);
-  });
-
-  it('returns false when stamp version matches package.json', async () => {
-    fs.mkdirSync(nextDir, { recursive: true });
-    fs.writeFileSync(buildStamp, '0.5.12', 'utf-8');
-    const { needsBuild } = await importBuild();
-    expect(needsBuild()).toBe(false);
-  });
-
-  it('returns true when stamp version does not match', async () => {
-    fs.mkdirSync(nextDir, { recursive: true });
-    fs.writeFileSync(buildStamp, '0.4.0', 'utf-8');
-    const { needsBuild } = await importBuild();
-    expect(needsBuild()).toBe(true);
-  });
-});
-
-// ── writeBuildStamp ─────────────────────────────────────────────────────────
-
-describe('writeBuildStamp', () => {
-  it('writes current version so needsBuild returns false', async () => {
-    fs.mkdirSync(nextDir, { recursive: true });
-    const { writeBuildStamp, needsBuild } = await importBuild();
-    writeBuildStamp();
-    expect(needsBuild()).toBe(false);
-    expect(fs.readFileSync(buildStamp, 'utf-8')).toBe('0.5.12');
-  });
-});
-
-// ── cleanNextDir ────────────────────────────────────────────────────────────
-
-describe('cleanNextDir', () => {
-  it('removes .next directory', async () => {
-    fs.mkdirSync(nextDir, { recursive: true });
-    fs.writeFileSync(path.join(nextDir, 'test.js'), 'x');
-    const { cleanNextDir } = await importBuild();
-    cleanNextDir();
-    expect(fs.existsSync(nextDir)).toBe(false);
-  });
-
-  it('does not throw when .next does not exist', async () => {
-    const { cleanNextDir } = await importBuild();
-    expect(() => cleanNextDir()).not.toThrow();
-  });
-});
-
-// ── clearBuildLock ──────────────────────────────────────────────────────────
-
-describe('clearBuildLock', () => {
-  it('removes .next/lock file', async () => {
-    fs.mkdirSync(nextDir, { recursive: true });
-    const lockFile = path.join(nextDir, 'lock');
-    fs.writeFileSync(lockFile, '');
-    const { clearBuildLock } = await importBuild();
-    clearBuildLock();
-    expect(fs.existsSync(lockFile)).toBe(false);
-  });
-});
-
-// ── ensureAppDeps ───────────────────────────────────────────────────────────
-
-describe('ensureAppDeps', () => {
-  it('skips install when next is present and deps hash matches', async () => {
-    // Create node_modules/next/package.json
-    const nextPkg = path.join(appDir, 'node_modules', 'next');
-    fs.mkdirSync(nextPkg, { recursive: true });
-    fs.writeFileSync(path.join(nextPkg, 'package.json'), '{}');
-
-    // Write matching deps hash
-    const { createHash } = await import('crypto');
-    const pkgContent = fs.readFileSync(path.join(appDir, 'package.json'));
-    const hash = createHash('sha256').update(pkgContent).digest('hex').slice(0, 16);
-    fs.writeFileSync(depsStamp, hash, 'utf-8');
-
-    // Mock execSync — should NOT be called for npm install
-    const mockExec = vi.fn();
-    vi.doMock('node:child_process', () => ({
-      execSync: mockExec,
-    }));
-    vi.resetModules();
-
-    // Re-mock constants after resetModules
-    vi.doMock('../../bin/lib/constants.js', () => ({
-      ROOT: tempDir,
-      BUILD_STAMP: buildStamp,
-      DEPS_STAMP: depsStamp,
-      CONFIG_PATH: path.join(tempDir, 'config.json'),
-      MINDOS_DIR: tempDir,
-      PID_PATH: path.join(tempDir, 'mindos.pid'),
-      LOG_PATH: path.join(tempDir, 'mindos.log'),
-      CLI_PATH: '',
-      NODE_BIN: process.execPath,
-      UPDATE_CHECK_PATH: path.join(tempDir, 'update-check.json'),
-      STANDALONE_SERVER: path.join(tempDir, '_standalone', 'server.js'),
-      STANDALONE_STAMP: path.join(tempDir, '_standalone', '.mindos-build-version'),
-    }));
-
-    const build = await importBuild();
-    build.ensureAppDeps();
-    // npm install should not have been called
-    expect(mockExec).not.toHaveBeenCalled();
-  });
-});

package/tests/unit/cli-config.test.ts

@@ -1,257 +0,0 @@
-import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest';
-import fs from 'fs';
-import os from 'os';
-import path from 'path';
-
-/**
- * Tests for bin/lib/config.js — loadConfig env mapping.
- *
- * loadConfig() has a module-level `let loaded = false` idempotency guard.
- * We must vi.resetModules() + dynamic import() for each test to get a fresh module.
- */
-
-let tempDir: string;
-let configPath: string;
-
-// All env vars that loadConfig may set
-const MANAGED_ENV_KEYS = [
-  'MIND_ROOT', 'MINDOS_WEB_PORT', 'MINDOS_MCP_PORT',
-  'AUTH_TOKEN', 'WEB_PASSWORD', 'AI_PROVIDER',
-  'ANTHROPIC_API_KEY', 'ANTHROPIC_MODEL',
-  'OPENAI_API_KEY', 'OPENAI_MODEL', 'OPENAI_BASE_URL',
-];
-
-let savedEnv: Record<string, string | undefined>;
-
-beforeEach(() => {
-  tempDir = fs.mkdtempSync(path.join(os.tmpdir(), 'mindos-cfg-test-'));
-  configPath = path.join(tempDir, 'config.json');
-
-  // Save and clear managed env vars
-  savedEnv = {};
-  for (const key of MANAGED_ENV_KEYS) {
-    savedEnv[key] = process.env[key];
-    delete process.env[key];
-  }
-
-  // Reset module registry so loadConfig's `loaded` flag resets
-  vi.resetModules();
-
-  // Mock constants to point CONFIG_PATH to our temp file
-  vi.doMock('../../bin/lib/constants.js', () => ({
-    CONFIG_PATH: configPath,
-    ROOT: path.resolve(__dirname, '..', '..'),
-    MINDOS_DIR: tempDir,
-    PID_PATH: path.join(tempDir, 'mindos.pid'),
-    BUILD_STAMP: path.join(tempDir, '.mindos-build-version'),
-    LOG_PATH: path.join(tempDir, 'mindos.log'),
-    CLI_PATH: '',
-    NODE_BIN: process.execPath,
-    UPDATE_CHECK_PATH: path.join(tempDir, 'update-check.json'),
-    DEPS_STAMP: path.join(tempDir, 'deps-hash'),
-  }));
-});
-
-afterEach(() => {
-  // Restore env
-  for (const key of MANAGED_ENV_KEYS) {
-    if (savedEnv[key] === undefined) delete process.env[key];
-    else process.env[key] = savedEnv[key];
-  }
-  fs.rmSync(tempDir, { recursive: true, force: true });
-  vi.restoreAllMocks();
-});
-
-function writeConfig(config: Record<string, unknown>) {
-  fs.writeFileSync(configPath, JSON.stringify(config, null, 2), 'utf-8');
-}
-
-async function importConfig() {
-  return await import('../../bin/lib/config.js') as {
-    loadConfig: () => void;
-    getStartMode: () => string;
-    isDaemonMode: () => boolean;
-  };
-}
-
-// ── loadConfig env mapping ──────────────────────────────────────────────────
-
-describe('loadConfig — new format (providers)', () => {
-  it('maps anthropic provider fields', async () => {
-    writeConfig({
-      mindRoot: '/tmp/mind',
-      ai: {
-        provider: 'anthropic',
-        providers: {
-          anthropic: { apiKey: 'sk-ant-xxx', model: 'claude-sonnet-4-6' },
-        },
-      },
-    });
-    const { loadConfig } = await importConfig();
-    loadConfig();
-    expect(process.env.AI_PROVIDER).toBe('anthropic');
-    expect(process.env.ANTHROPIC_API_KEY).toBe('sk-ant-xxx');
-    expect(process.env.ANTHROPIC_MODEL).toBe('claude-sonnet-4-6');
-  });
-
-  it('maps openai provider fields (apiKey + model + baseUrl)', async () => {
-    writeConfig({
-      ai: {
-        provider: 'openai',
-        providers: {
-          openai: { apiKey: 'sk-oai-xxx', model: 'gpt-5.4', baseUrl: 'https://custom.api.com/v1' },
-        },
-      },
-    });
-    const { loadConfig } = await importConfig();
-    loadConfig();
-    expect(process.env.AI_PROVIDER).toBe('openai');
-    expect(process.env.OPENAI_API_KEY).toBe('sk-oai-xxx');
-    expect(process.env.OPENAI_MODEL).toBe('gpt-5.4');
-    expect(process.env.OPENAI_BASE_URL).toBe('https://custom.api.com/v1');
-  });
-});
-
-describe('loadConfig — old format (flat keys)', () => {
-  it('maps flat anthropic keys as fallback', async () => {
-    writeConfig({
-      ai: {
-        provider: 'anthropic',
-        anthropicApiKey: 'sk-ant-old',
-        anthropicModel: 'claude-3-opus',
-      },
-    });
-    const { loadConfig } = await importConfig();
-    loadConfig();
-    expect(process.env.ANTHROPIC_API_KEY).toBe('sk-ant-old');
-    expect(process.env.ANTHROPIC_MODEL).toBe('claude-3-opus');
-  });
-
-  it('maps flat openai keys as fallback', async () => {
-    writeConfig({
-      ai: {
-        provider: 'openai',
-        openaiApiKey: 'sk-oai-old',
-        openaiModel: 'gpt-4',
-        openaiBaseUrl: 'https://old.api.com',
-      },
-    });
-    const { loadConfig } = await importConfig();
-    loadConfig();
-    expect(process.env.OPENAI_API_KEY).toBe('sk-oai-old');
-    expect(process.env.OPENAI_MODEL).toBe('gpt-4');
-    expect(process.env.OPENAI_BASE_URL).toBe('https://old.api.com');
-  });
-});
-
-describe('loadConfig — core fields', () => {
-  it('maps port and mcpPort', async () => {
-    writeConfig({ port: 3001, mcpPort: 8788 });
-    const { loadConfig } = await importConfig();
-    loadConfig();
-    expect(process.env.MINDOS_WEB_PORT).toBe('3001');
-    expect(process.env.MINDOS_MCP_PORT).toBe('8788');
-  });
-
-  it('maps mindRoot', async () => {
-    writeConfig({ mindRoot: '/home/user/MindOS/mind' });
-    const { loadConfig } = await importConfig();
-    loadConfig();
-    expect(process.env.MIND_ROOT).toBe('/home/user/MindOS/mind');
-  });
-
-  it('maps authToken', async () => {
-    writeConfig({ authToken: 'tok-abc123' });
-    const { loadConfig } = await importConfig();
-    loadConfig();
-    expect(process.env.AUTH_TOKEN).toBe('tok-abc123');
-  });
-
-  it('maps webPassword', async () => {
-    writeConfig({ webPassword: 'pass123' });
-    const { loadConfig } = await importConfig();
-    loadConfig();
-    expect(process.env.WEB_PASSWORD).toBe('pass123');
-  });
-});
-
-describe('loadConfig — does not override existing env', () => {
-  it('preserves existing AUTH_TOKEN', async () => {
-    process.env.AUTH_TOKEN = 'already-set';
-    writeConfig({ authToken: 'from-config' });
-    const { loadConfig } = await importConfig();
-    loadConfig();
-    expect(process.env.AUTH_TOKEN).toBe('already-set');
-  });
-});
-
-describe('loadConfig — error handling', () => {
-  it('does not throw when config file does not exist', async () => {
-    // configPath does not exist — should not throw
-    const { loadConfig } = await importConfig();
-    expect(() => loadConfig()).not.toThrow();
-  });
-
-  it('does not throw on corrupt JSON (outputs warning)', async () => {
-    fs.writeFileSync(configPath, '{broken json!!!', 'utf-8');
-    const spy = vi.spyOn(console, 'error').mockImplementation(() => {});
-    const { loadConfig } = await importConfig();
-    expect(() => loadConfig()).not.toThrow();
-    expect(spy).toHaveBeenCalled();
-    spy.mockRestore();
-  });
-});
-
-describe('loadConfig — idempotency', () => {
-  it('only loads once (second call is no-op)', async () => {
-    writeConfig({ authToken: 'first' });
-    const { loadConfig } = await importConfig();
-    loadConfig();
-    expect(process.env.AUTH_TOKEN).toBe('first');
-
-    // Overwrite config file and call again — should NOT re-read
-    writeConfig({ authToken: 'second' });
-    loadConfig();
-    expect(process.env.AUTH_TOKEN).toBe('first');
-  });
-});
-
-// ── getStartMode / isDaemonMode ─────────────────────────────────────────────
-
-describe('getStartMode', () => {
-  it('returns "start" when no config', async () => {
-    const { getStartMode } = await importConfig();
-    expect(getStartMode()).toBe('start');
-  });
-
-  it('returns "start" when startMode is "daemon" (CLI uses --daemon flag)', async () => {
-    writeConfig({ startMode: 'daemon' });
-    const { getStartMode } = await importConfig();
-    expect(getStartMode()).toBe('start');
-  });
-
-  it('returns stored startMode for non-daemon values', async () => {
-    writeConfig({ startMode: 'dev' });
-    const { getStartMode } = await importConfig();
-    expect(getStartMode()).toBe('dev');
-  });
-});
-
-describe('isDaemonMode', () => {
-  it('returns true when startMode is "daemon"', async () => {
-    writeConfig({ startMode: 'daemon' });
-    const { isDaemonMode } = await importConfig();
-    expect(isDaemonMode()).toBe(true);
-  });
-
-  it('returns false when startMode is not "daemon"', async () => {
-    writeConfig({ startMode: 'start' });
-    const { isDaemonMode } = await importConfig();
-    expect(isDaemonMode()).toBe(false);
-  });
-
-  it('returns false when no config', async () => {
-    const { isDaemonMode } = await importConfig();
-    expect(isDaemonMode()).toBe(false);
-  });
-});