opengstack 0.13.10 → 0.14.2

package/skills/browse/test/compare-board.test.ts

@@ -1,342 +0,0 @@
-/**
- * Integration test for the design comparison board feedback loop.
- *
- * Tests the DOM polling pattern that plan-design-review, office-hours,
- * and design-consultation use to read user feedback from the comparison board.
- *
- * Flow: generate board HTML → open in browser → verify DOM elements →
- *       simulate user interaction → verify structured JSON feedback.
- *
- * No LLM involved — this is a deterministic functional test.
- */
-
-import { describe, test, expect, beforeAll, afterAll } from 'bun:test';
-import { BrowserManager } from '../src/browser-manager';
-import { handleReadCommand } from '../src/read-commands';
-import { handleWriteCommand } from '../src/write-commands';
-import { generateCompareHtml } from '../../design/src/compare';
-import * as fs from 'fs';
-import * as path from 'path';
-
-let bm: BrowserManager;
-let boardUrl: string;
-let server: ReturnType<typeof Bun.serve>;
-let tmpDir: string;
-
-// Create a minimal 1x1 pixel PNG for test variants
-function createTestPng(filePath: string): void {
-  // Minimal valid PNG: 1x1 red pixel
-  const png = Buffer.from(
-    'iVBORw0KGgoAAAANSUhEUgAAAAEAAAABCAYAAAAfFcSJAAAADUlEQVR42mP8/58BAwAI/AL+hc2rNAAAAABJRU5ErkJggg==',
-    'base64'
-  );
-  fs.writeFileSync(filePath, png);
-}
-
-beforeAll(async () => {
-  // Create test PNG files
-  tmpDir = '/tmp/compare-board-test-' + Date.now();
-  fs.mkdirSync(tmpDir, { recursive: true });
-
-  createTestPng(path.join(tmpDir, 'variant-A.png'));
-  createTestPng(path.join(tmpDir, 'variant-B.png'));
-  createTestPng(path.join(tmpDir, 'variant-C.png'));
-
-  // Generate comparison board HTML using the real compare module
-  const html = generateCompareHtml([
-    path.join(tmpDir, 'variant-A.png'),
-    path.join(tmpDir, 'variant-B.png'),
-    path.join(tmpDir, 'variant-C.png'),
-  ]);
-
-  // Serve the board via HTTP (browse blocks file:// URLs for security)
-  server = Bun.serve({
-    port: 0,
-    fetch() {
-      return new Response(html, { headers: { 'Content-Type': 'text/html' } });
-    },
-  });
-  boardUrl = `http://localhost:${server.port}`;
-
-  // Launch browser and navigate to the board
-  bm = new BrowserManager();
-  await bm.launch();
-  await handleWriteCommand('goto', [boardUrl], bm);
-});
-
-afterAll(() => {
-  try { server.stop(); } catch {}
-  fs.rmSync(tmpDir, { recursive: true, force: true });
-  setTimeout(() => process.exit(0), 500);
-});
-
-// ─── DOM Structure ──────────────────────────────────────────────
-
-describe('Comparison board DOM structure', () => {
-  test('has hidden status element', async () => {
-    const status = await handleReadCommand('js', [
-      'document.getElementById("status").textContent'
-    ], bm);
-    expect(status).toBe('');
-  });
-
-  test('has hidden feedback-result element', async () => {
-    const result = await handleReadCommand('js', [
-      'document.getElementById("feedback-result").textContent'
-    ], bm);
-    expect(result).toBe('');
-  });
-
-  test('has submit button', async () => {
-    const exists = await handleReadCommand('js', [
-      '!!document.getElementById("submit-btn")'
-    ], bm);
-    expect(exists).toBe('true');
-  });
-
-  test('has regenerate button', async () => {
-    const exists = await handleReadCommand('js', [
-      '!!document.getElementById("regen-btn")'
-    ], bm);
-    expect(exists).toBe('true');
-  });
-
-  test('has 3 variant cards', async () => {
-    const count = await handleReadCommand('js', [
-      'document.querySelectorAll(".variant").length'
-    ], bm);
-    expect(count).toBe('3');
-  });
-
-  test('has pick radio buttons for each variant', async () => {
-    const count = await handleReadCommand('js', [
-      'document.querySelectorAll("input[name=\\"preferred\\"]").length'
-    ], bm);
-    expect(count).toBe('3');
-  });
-
-  test('has star ratings for each variant', async () => {
-    const count = await handleReadCommand('js', [
-      'document.querySelectorAll(".stars").length'
-    ], bm);
-    expect(count).toBe('3');
-  });
-});
-
-// ─── Submit Flow ────────────────────────────────────────────────
-
-describe('Submit feedback flow', () => {
-  test('submit without interaction returns empty preferred', async () => {
-    // Reset page state
-    await handleWriteCommand('goto', [boardUrl], bm);
-
-    // Click submit without picking anything
-    await handleReadCommand('js', [
-      'document.getElementById("submit-btn").click()'
-    ], bm);
-
-    // Status should be "submitted"
-    const status = await handleReadCommand('js', [
-      'document.getElementById("status").textContent'
-    ], bm);
-    expect(status).toBe('submitted');
-
-    // Read feedback JSON
-    const raw = await handleReadCommand('js', [
-      'document.getElementById("feedback-result").textContent'
-    ], bm);
-    const feedback = JSON.parse(raw);
-    expect(feedback.preferred).toBeNull();
-    expect(feedback.regenerated).toBe(false);
-    expect(feedback.ratings).toBeDefined();
-  });
-
-  test('submit with pick + rating + comment returns structured JSON', async () => {
-    // Fresh page
-    await handleWriteCommand('goto', [boardUrl], bm);
-
-    // Pick variant B
-    await handleReadCommand('js', [
-      'document.querySelectorAll("input[name=\\"preferred\\"]")[1].click()'
-    ], bm);
-
-    // Rate variant A: 4 stars (click the 4th star)
-    await handleReadCommand('js', [
-      'document.querySelectorAll(".stars")[0].querySelectorAll(".star")[3].click()'
-    ], bm);
-
-    // Rate variant B: 5 stars
-    await handleReadCommand('js', [
-      'document.querySelectorAll(".stars")[1].querySelectorAll(".star")[4].click()'
-    ], bm);
-
-    // Add comment on variant A
-    await handleReadCommand('js', [
-      'document.querySelectorAll(".feedback-input")[0].value = "Good spacing but wrong colors"'
-    ], bm);
-
-    // Add overall feedback
-    await handleReadCommand('js', [
-      'document.getElementById("overall-feedback").value = "Go with B, make the CTA bigger"'
-    ], bm);
-
-    // Submit
-    await handleReadCommand('js', [
-      'document.getElementById("submit-btn").click()'
-    ], bm);
-
-    // Verify status
-    const status = await handleReadCommand('js', [
-      'document.getElementById("status").textContent'
-    ], bm);
-    expect(status).toBe('submitted');
-
-    // Read and verify structured feedback
-    const raw = await handleReadCommand('js', [
-      'document.getElementById("feedback-result").textContent'
-    ], bm);
-    const feedback = JSON.parse(raw);
-
-    expect(feedback.preferred).toBe('B');
-    expect(feedback.ratings.A).toBe(4);
-    expect(feedback.ratings.B).toBe(5);
-    expect(feedback.comments.A).toBe('Good spacing but wrong colors');
-    expect(feedback.overall).toBe('Go with B, make the CTA bigger');
-    expect(feedback.regenerated).toBe(false);
-  });
-
-  test('submit button is disabled after submission', async () => {
-    const disabled = await handleReadCommand('js', [
-      'document.getElementById("submit-btn").disabled'
-    ], bm);
-    expect(disabled).toBe('true');
-  });
-
-  test('success message is visible after submission', async () => {
-    const display = await handleReadCommand('js', [
-      'document.getElementById("success-msg").style.display'
-    ], bm);
-    expect(display).toBe('block');
-  });
-});
-
-// ─── Regenerate Flow ────────────────────────────────────────────
-
-describe('Regenerate flow', () => {
-  test('regenerate button sets status to "regenerate"', async () => {
-    // Fresh page
-    await handleWriteCommand('goto', [boardUrl], bm);
-
-    // Click "Totally different" chiclet then regenerate
-    await handleReadCommand('js', [
-      'document.querySelector(".regen-chiclet[data-action=\\"different\\"]").click()'
-    ], bm);
-    await handleReadCommand('js', [
-      'document.getElementById("regen-btn").click()'
-    ], bm);
-
-    const status = await handleReadCommand('js', [
-      'document.getElementById("status").textContent'
-    ], bm);
-    expect(status).toBe('regenerate');
-
-    // Verify regenerate action in feedback
-    const raw = await handleReadCommand('js', [
-      'document.getElementById("feedback-result").textContent'
-    ], bm);
-    const feedback = JSON.parse(raw);
-    expect(feedback.regenerated).toBe(true);
-    expect(feedback.regenerateAction).toBe('different');
-  });
-
-  test('"More like this" sets regenerate with variant reference', async () => {
-    // Fresh page
-    await handleWriteCommand('goto', [boardUrl], bm);
-
-    // Click "More like this" on variant B
-    await handleReadCommand('js', [
-      'document.querySelectorAll(".more-like-this")[1].click()'
-    ], bm);
-
-    const status = await handleReadCommand('js', [
-      'document.getElementById("status").textContent'
-    ], bm);
-    expect(status).toBe('regenerate');
-
-    const raw = await handleReadCommand('js', [
-      'document.getElementById("feedback-result").textContent'
-    ], bm);
-    const feedback = JSON.parse(raw);
-    expect(feedback.regenerated).toBe(true);
-    expect(feedback.regenerateAction).toBe('more_like_B');
-  });
-
-  test('regenerate with custom text', async () => {
-    // Fresh page
-    await handleWriteCommand('goto', [boardUrl], bm);
-
-    // Type custom regeneration text
-    await handleReadCommand('js', [
-      'document.getElementById("regen-custom-input").value = "V3 layout with V1 colors"'
-    ], bm);
-
-    // Click regenerate (no chiclet selected = custom)
-    await handleReadCommand('js', [
-      'document.getElementById("regen-btn").click()'
-    ], bm);
-
-    const raw = await handleReadCommand('js', [
-      'document.getElementById("feedback-result").textContent'
-    ], bm);
-    const feedback = JSON.parse(raw);
-    expect(feedback.regenerated).toBe(true);
-    expect(feedback.regenerateAction).toBe('V3 layout with V1 colors');
-  });
-});
-
-// ─── Agent Polling Pattern ──────────────────────────────────────
-
-describe('Agent polling pattern (simulates what $B eval does)', () => {
-  test('status is empty before user action', async () => {
-    // Fresh page — simulates agent's first poll
-    await handleWriteCommand('goto', [boardUrl], bm);
-
-    const status = await handleReadCommand('js', [
-      'document.getElementById("status").textContent'
-    ], bm);
-    expect(status).toBe('');
-  });
-
-  test('full polling cycle: empty → submitted → read JSON', async () => {
-    await handleWriteCommand('goto', [boardUrl], bm);
-
-    // Poll 1: empty (user hasn't acted)
-    const poll1 = await handleReadCommand('js', [
-      'document.getElementById("status").textContent'
-    ], bm);
-    expect(poll1).toBe('');
-
-    // User acts: pick A, submit
-    await handleReadCommand('js', [
-      'document.querySelectorAll("input[name=\\"preferred\\"]")[0].click()'
-    ], bm);
-    await handleReadCommand('js', [
-      'document.getElementById("submit-btn").click()'
-    ], bm);
-
-    // Poll 2: submitted
-    const poll2 = await handleReadCommand('js', [
-      'document.getElementById("status").textContent'
-    ], bm);
-    expect(poll2).toBe('submitted');
-
-    // Read feedback (what the agent does after seeing "submitted")
-    const raw = await handleReadCommand('js', [
-      'document.getElementById("feedback-result").textContent'
-    ], bm);
-    const feedback = JSON.parse(raw);
-    expect(feedback.preferred).toBe('A');
-    expect(typeof feedback.ratings).toBe('object');
-    expect(typeof feedback.comments).toBe('object');
-  });
-});

package/skills/browse/test/config.test.ts

@@ -1,316 +0,0 @@
-import { describe, test, expect } from 'bun:test';
-import { resolveConfig, ensureStateDir, readVersionHash, getGitRoot, getRemoteSlug } from '../src/config';
-import * as fs from 'fs';
-import * as path from 'path';
-import * as os from 'os';
-
-describe('config', () => {
-  describe('getGitRoot', () => {
-    test('returns a path when in a git repo', () => {
-      const root = getGitRoot();
-      expect(root).not.toBeNull();
-      expect(fs.existsSync(path.join(root!, '.git'))).toBe(true);
-    });
-  });
-
-  describe('resolveConfig', () => {
-    test('uses git root by default', () => {
-      const config = resolveConfig({});
-      const gitRoot = getGitRoot();
-      expect(gitRoot).not.toBeNull();
-      expect(config.projectDir).toBe(gitRoot);
-      expect(config.stateDir).toBe(path.join(gitRoot!, '.gstack'));
-      expect(config.stateFile).toBe(path.join(gitRoot!, '.gstack', 'browse.json'));
-    });
-
-    test('derives paths from BROWSE_STATE_FILE when set', () => {
-      const stateFile = '/tmp/test-config/.gstack/browse.json';
-      const config = resolveConfig({ BROWSE_STATE_FILE: stateFile });
-      expect(config.stateFile).toBe(stateFile);
-      expect(config.stateDir).toBe('/tmp/test-config/.gstack');
-      expect(config.projectDir).toBe('/tmp/test-config');
-    });
-
-    test('log paths are in stateDir', () => {
-      const config = resolveConfig({});
-      expect(config.consoleLog).toBe(path.join(config.stateDir, 'browse-console.log'));
-      expect(config.networkLog).toBe(path.join(config.stateDir, 'browse-network.log'));
-      expect(config.dialogLog).toBe(path.join(config.stateDir, 'browse-dialog.log'));
-    });
-  });
-
-  describe('ensureStateDir', () => {
-    test('creates directory if it does not exist', () => {
-      const tmpDir = path.join(os.tmpdir(), `browse-config-test-${Date.now()}`);
-      const config = resolveConfig({ BROWSE_STATE_FILE: path.join(tmpDir, '.gstack', 'browse.json') });
-      expect(fs.existsSync(config.stateDir)).toBe(false);
-      ensureStateDir(config);
-      expect(fs.existsSync(config.stateDir)).toBe(true);
-      // Cleanup
-      fs.rmSync(tmpDir, { recursive: true, force: true });
-    });
-
-    test('is a no-op if directory already exists', () => {
-      const tmpDir = path.join(os.tmpdir(), `browse-config-test-${Date.now()}`);
-      const stateDir = path.join(tmpDir, '.gstack');
-      fs.mkdirSync(stateDir, { recursive: true });
-      const config = resolveConfig({ BROWSE_STATE_FILE: path.join(stateDir, 'browse.json') });
-      ensureStateDir(config); // should not throw
-      expect(fs.existsSync(config.stateDir)).toBe(true);
-      // Cleanup
-      fs.rmSync(tmpDir, { recursive: true, force: true });
-    });
-
-    test('adds .gstack/ to .gitignore if not present', () => {
-      const tmpDir = path.join(os.tmpdir(), `browse-gitignore-test-${Date.now()}`);
-      fs.mkdirSync(tmpDir, { recursive: true });
-      fs.writeFileSync(path.join(tmpDir, '.gitignore'), 'node_modules/\n');
-      const config = resolveConfig({ BROWSE_STATE_FILE: path.join(tmpDir, '.gstack', 'browse.json') });
-      ensureStateDir(config);
-      const content = fs.readFileSync(path.join(tmpDir, '.gitignore'), 'utf-8');
-      expect(content).toContain('.gstack/');
-      expect(content).toBe('node_modules/\n.gstack/\n');
-      fs.rmSync(tmpDir, { recursive: true, force: true });
-    });
-
-    test('does not duplicate .gstack/ in .gitignore', () => {
-      const tmpDir = path.join(os.tmpdir(), `browse-gitignore-test-${Date.now()}`);
-      fs.mkdirSync(tmpDir, { recursive: true });
-      fs.writeFileSync(path.join(tmpDir, '.gitignore'), 'node_modules/\n.gstack/\n');
-      const config = resolveConfig({ BROWSE_STATE_FILE: path.join(tmpDir, '.gstack', 'browse.json') });
-      ensureStateDir(config);
-      const content = fs.readFileSync(path.join(tmpDir, '.gitignore'), 'utf-8');
-      expect(content).toBe('node_modules/\n.gstack/\n');
-      fs.rmSync(tmpDir, { recursive: true, force: true });
-    });
-
-    test('handles .gitignore without trailing newline', () => {
-      const tmpDir = path.join(os.tmpdir(), `browse-gitignore-test-${Date.now()}`);
-      fs.mkdirSync(tmpDir, { recursive: true });
-      fs.writeFileSync(path.join(tmpDir, '.gitignore'), 'node_modules');
-      const config = resolveConfig({ BROWSE_STATE_FILE: path.join(tmpDir, '.gstack', 'browse.json') });
-      ensureStateDir(config);
-      const content = fs.readFileSync(path.join(tmpDir, '.gitignore'), 'utf-8');
-      expect(content).toBe('node_modules\n.gstack/\n');
-      fs.rmSync(tmpDir, { recursive: true, force: true });
-    });
-
-    test('logs warning to browse-server.log on non-ENOENT gitignore error', () => {
-      const tmpDir = path.join(os.tmpdir(), `browse-gitignore-test-${Date.now()}`);
-      fs.mkdirSync(tmpDir, { recursive: true });
-      // Create a read-only .gitignore (no .gstack/ entry → would try to append)
-      fs.writeFileSync(path.join(tmpDir, '.gitignore'), 'node_modules/\n');
-      fs.chmodSync(path.join(tmpDir, '.gitignore'), 0o444);
-      const config = resolveConfig({ BROWSE_STATE_FILE: path.join(tmpDir, '.gstack', 'browse.json') });
-      ensureStateDir(config); // should not throw
-      // Verify warning was written to server log
-      const logPath = path.join(config.stateDir, 'browse-server.log');
-      expect(fs.existsSync(logPath)).toBe(true);
-      const logContent = fs.readFileSync(logPath, 'utf-8');
-      expect(logContent).toContain('Warning: could not update .gitignore');
-      // .gitignore should remain unchanged
-      const gitignoreContent = fs.readFileSync(path.join(tmpDir, '.gitignore'), 'utf-8');
-      expect(gitignoreContent).toBe('node_modules/\n');
-      // Cleanup
-      fs.chmodSync(path.join(tmpDir, '.gitignore'), 0o644);
-      fs.rmSync(tmpDir, { recursive: true, force: true });
-    });
-
-    test('skips if no .gitignore exists', () => {
-      const tmpDir = path.join(os.tmpdir(), `browse-gitignore-test-${Date.now()}`);
-      fs.mkdirSync(tmpDir, { recursive: true });
-      const config = resolveConfig({ BROWSE_STATE_FILE: path.join(tmpDir, '.gstack', 'browse.json') });
-      ensureStateDir(config);
-      expect(fs.existsSync(path.join(tmpDir, '.gitignore'))).toBe(false);
-      fs.rmSync(tmpDir, { recursive: true, force: true });
-    });
-  });
-
-  describe('getRemoteSlug', () => {
-    test('returns owner-repo format for current repo', () => {
-      const slug = getRemoteSlug();
-      // This repo has an origin remote — should return a slug
-      expect(slug).toBeTruthy();
-      expect(slug).toMatch(/^[a-zA-Z0-9._-]+-[a-zA-Z0-9._-]+$/);
-    });
-
-    test('parses SSH remote URLs', () => {
-      // Test the regex directly since we can't mock Bun.spawnSync easily
-      const url = 'git@github.com:garrytan/gstack.git';
-      const match = url.match(/[:/]([^/]+)\/([^/]+?)(?:\.git)?$/);
-      expect(match).not.toBeNull();
-      expect(`${match![1]}-${match![2]}`).toBe('garrytan-gstack');
-    });
-
-    test('parses HTTPS remote URLs', () => {
-      const url = 'https://github.com/garrytan/gstack.git';
-      const match = url.match(/[:/]([^/]+)\/([^/]+?)(?:\.git)?$/);
-      expect(match).not.toBeNull();
-      expect(`${match![1]}-${match![2]}`).toBe('garrytan-gstack');
-    });
-
-    test('parses HTTPS remote URLs without .git suffix', () => {
-      const url = 'https://github.com/garrytan/gstack';
-      const match = url.match(/[:/]([^/]+)\/([^/]+?)(?:\.git)?$/);
-      expect(match).not.toBeNull();
-      expect(`${match![1]}-${match![2]}`).toBe('garrytan-gstack');
-    });
-  });
-
-  describe('readVersionHash', () => {
-    test('returns null when .version file does not exist', () => {
-      const result = readVersionHash('/nonexistent/path/browse');
-      expect(result).toBeNull();
-    });
-
-    test('reads version from .version file adjacent to execPath', () => {
-      const tmpDir = path.join(os.tmpdir(), `browse-version-test-${Date.now()}`);
-      fs.mkdirSync(tmpDir, { recursive: true });
-      const versionFile = path.join(tmpDir, '.version');
-      fs.writeFileSync(versionFile, 'abc123def\n');
-      const result = readVersionHash(path.join(tmpDir, 'browse'));
-      expect(result).toBe('abc123def');
-      // Cleanup
-      fs.rmSync(tmpDir, { recursive: true, force: true });
-    });
-  });
-});
-
-describe('resolveServerScript', () => {
-  // Import the function from cli.ts
-  const { resolveServerScript } = require('../src/cli');
-
-  test('uses BROWSE_SERVER_SCRIPT env when set', () => {
-    const result = resolveServerScript({ BROWSE_SERVER_SCRIPT: '/custom/server.ts' }, '', '');
-    expect(result).toBe('/custom/server.ts');
-  });
-
-  test('finds server.ts adjacent to cli.ts in dev mode', () => {
-    const srcDir = path.resolve(__dirname, '../src');
-    const result = resolveServerScript({}, srcDir, '');
-    expect(result).toBe(path.join(srcDir, 'server.ts'));
-  });
-
-  test('throws when server.ts cannot be found', () => {
-    expect(() => resolveServerScript({}, '/nonexistent/$bunfs', '/nonexistent/browse'))
-      .toThrow('Cannot find server.ts');
-  });
-});
-
-describe('resolveNodeServerScript', () => {
-  const { resolveNodeServerScript } = require('../src/cli');
-
-  test('finds server-node.mjs in dist from dev mode', () => {
-    const srcDir = path.resolve(__dirname, '../src');
-    const distFile = path.resolve(srcDir, '..', 'dist', 'server-node.mjs');
-    const fs = require('fs');
-    // Only test if the file exists (it may not be built yet)
-    if (fs.existsSync(distFile)) {
-      const result = resolveNodeServerScript(srcDir, '');
-      expect(result).toBe(distFile);
-    }
-  });
-
-  test('returns null when server-node.mjs does not exist', () => {
-    const result = resolveNodeServerScript('/nonexistent/$bunfs', '/nonexistent/browse');
-    expect(result).toBeNull();
-  });
-
-  test('finds server-node.mjs adjacent to compiled binary', () => {
-    const distDir = path.resolve(__dirname, '../dist');
-    const distFile = path.join(distDir, 'server-node.mjs');
-    const fs = require('fs');
-    if (fs.existsSync(distFile)) {
-      const result = resolveNodeServerScript('/$bunfs/something', path.join(distDir, 'browse'));
-      expect(result).toBe(distFile);
-    }
-  });
-});
-
-describe('version mismatch detection', () => {
-  test('detects when versions differ', () => {
-    const stateVersion = 'abc123';
-    const currentVersion = 'def456';
-    expect(stateVersion !== currentVersion).toBe(true);
-  });
-
-  test('no mismatch when versions match', () => {
-    const stateVersion = 'abc123';
-    const currentVersion = 'abc123';
-    expect(stateVersion !== currentVersion).toBe(false);
-  });
-
-  test('no mismatch when either version is null', () => {
-    const currentVersion: string | null = null;
-    const stateVersion: string | undefined = 'abc123';
-    // Version mismatch only triggers when both are present
-    const shouldRestart = currentVersion !== null && stateVersion !== undefined && currentVersion !== stateVersion;
-    expect(shouldRestart).toBe(false);
-  });
-});
-
-describe('isServerHealthy', () => {
-  const { isServerHealthy } = require('../src/cli');
-  const http = require('http');
-
-  test('returns true for a healthy server', async () => {
-    const server = http.createServer((_req: any, res: any) => {
-      res.writeHead(200, { 'Content-Type': 'application/json' });
-      res.end(JSON.stringify({ status: 'healthy' }));
-    });
-    await new Promise<void>(resolve => server.listen(0, resolve));
-    const port = server.address().port;
-    try {
-      expect(await isServerHealthy(port)).toBe(true);
-    } finally {
-      server.close();
-    }
-  });
-
-  test('returns false for an unhealthy server', async () => {
-    const server = http.createServer((_req: any, res: any) => {
-      res.writeHead(200, { 'Content-Type': 'application/json' });
-      res.end(JSON.stringify({ status: 'unhealthy' }));
-    });
-    await new Promise<void>(resolve => server.listen(0, resolve));
-    const port = server.address().port;
-    try {
-      expect(await isServerHealthy(port)).toBe(false);
-    } finally {
-      server.close();
-    }
-  });
-
-  test('returns false when server is not running', async () => {
-    // Use a port that's almost certainly not in use
-    expect(await isServerHealthy(59999)).toBe(false);
-  });
-
-  test('returns false on non-200 response', async () => {
-    const server = http.createServer((_req: any, res: any) => {
-      res.writeHead(500);
-      res.end('Internal Server Error');
-    });
-    await new Promise<void>(resolve => server.listen(0, resolve));
-    const port = server.address().port;
-    try {
-      expect(await isServerHealthy(port)).toBe(false);
-    } finally {
-      server.close();
-    }
-  });
-});
-
-describe('startup error log', () => {
-  test('write and read error log', () => {
-    const tmpDir = path.join(os.tmpdir(), `browse-error-log-test-${Date.now()}`);
-    fs.mkdirSync(tmpDir, { recursive: true });
-    const errorLogPath = path.join(tmpDir, 'browse-startup-error.log');
-    const errorMsg = 'Cannot find module playwright';
-    fs.writeFileSync(errorLogPath, `2026-03-23T00:00:00.000Z ${errorMsg}\n`);
-    const content = fs.readFileSync(errorLogPath, 'utf-8').trim();
-    expect(content).toContain(errorMsg);
-    expect(content).toMatch(/^\d{4}-\d{2}-\d{2}T/); // ISO timestamp prefix
-    fs.rmSync(tmpDir, { recursive: true, force: true });
-  });
-});