@runchr/gstack-antigravity 0.1.0 → 0.1.2

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

@@ -0,0 +1,250 @@
+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);
+  });
+});

package/.agents/skills/gstack/browse/test/cookie-import-browser.test.ts

@@ -0,0 +1,397 @@
+/**
+ * Unit tests for cookie-import-browser.ts
+ *
+ * Uses a fixture SQLite database with cookies encrypted using a known test key.
+ * Mocks Keychain access to return the test password.
+ *
+ * Test key derivation (matches real Chromium pipeline):
+ *   password = "test-keychain-password"
+ *   key = PBKDF2(password, "saltysalt", 1003, 16, sha1)
+ *
+ * Encryption: AES-128-CBC with IV = 16 × 0x20, prefix "v10"
+ * First 32 bytes of plaintext = HMAC-SHA256 tag (random for tests)
+ * Remaining bytes = actual cookie value
+ */
+
+import { describe, test, expect, beforeAll, afterAll, mock } from 'bun:test';
+import { Database } from 'bun:sqlite';
+import * as crypto from 'crypto';
+import * as fs from 'fs';
+import * as path from 'path';
+import * as os from 'os';
+
+// ─── Test Constants ─────────────────────────────────────────────
+
+const TEST_PASSWORD = 'test-keychain-password';
+const TEST_KEY = crypto.pbkdf2Sync(TEST_PASSWORD, 'saltysalt', 1003, 16, 'sha1');
+const IV = Buffer.alloc(16, 0x20);
+const CHROMIUM_EPOCH_OFFSET = 11644473600000000n;
+
+// Fixture DB path
+const FIXTURE_DIR = path.join(import.meta.dir, 'fixtures');
+const FIXTURE_DB = path.join(FIXTURE_DIR, 'test-cookies.db');
+
+// ─── Encryption Helper ──────────────────────────────────────────
+
+function encryptCookieValue(value: string): Buffer {
+  // 32-byte HMAC tag (random for test) + actual value
+  const hmacTag = crypto.randomBytes(32);
+  const plaintext = Buffer.concat([hmacTag, Buffer.from(value, 'utf-8')]);
+
+  // PKCS7 pad to AES block size (16 bytes)
+  const blockSize = 16;
+  const padLen = blockSize - (plaintext.length % blockSize);
+  const padded = Buffer.concat([plaintext, Buffer.alloc(padLen, padLen)]);
+
+  const cipher = crypto.createCipheriv('aes-128-cbc', TEST_KEY, IV);
+  cipher.setAutoPadding(false); // We padded manually
+  const encrypted = Buffer.concat([cipher.update(padded), cipher.final()]);
+
+  // Prefix with "v10"
+  return Buffer.concat([Buffer.from('v10'), encrypted]);
+}
+
+function chromiumEpoch(unixSeconds: number): bigint {
+  return BigInt(unixSeconds) * 1000000n + CHROMIUM_EPOCH_OFFSET;
+}
+
+// ─── Create Fixture Database ────────────────────────────────────
+
+function createFixtureDb() {
+  fs.mkdirSync(FIXTURE_DIR, { recursive: true });
+  if (fs.existsSync(FIXTURE_DB)) fs.unlinkSync(FIXTURE_DB);
+
+  const db = new Database(FIXTURE_DB);
+  db.run(`CREATE TABLE cookies (
+    host_key TEXT NOT NULL,
+    name TEXT NOT NULL,
+    value TEXT NOT NULL DEFAULT '',
+    encrypted_value BLOB NOT NULL DEFAULT x'',
+    path TEXT NOT NULL DEFAULT '/',
+    expires_utc INTEGER NOT NULL DEFAULT 0,
+    is_secure INTEGER NOT NULL DEFAULT 0,
+    is_httponly INTEGER NOT NULL DEFAULT 0,
+    has_expires INTEGER NOT NULL DEFAULT 0,
+    samesite INTEGER NOT NULL DEFAULT 1
+  )`);
+
+  const insert = db.prepare(`INSERT INTO cookies
+    (host_key, name, value, encrypted_value, path, expires_utc, is_secure, is_httponly, has_expires, samesite)
+    VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?, ?)`);
+
+  const futureExpiry = Number(chromiumEpoch(Math.floor(Date.now() / 1000) + 86400 * 365));
+  const pastExpiry = Number(chromiumEpoch(Math.floor(Date.now() / 1000) - 86400));
+
+  // Domain 1: .github.com — 3 encrypted cookies
+  insert.run('.github.com', 'session_id', '', encryptCookieValue('abc123'), '/', futureExpiry, 1, 1, 1, 1);
+  insert.run('.github.com', 'user_token', '', encryptCookieValue('token-xyz'), '/', futureExpiry, 1, 0, 1, 0);
+  insert.run('.github.com', 'theme', '', encryptCookieValue('dark'), '/', futureExpiry, 0, 0, 1, 2);
+
+  // Domain 2: .google.com — 2 cookies
+  insert.run('.google.com', 'NID', '', encryptCookieValue('google-nid-value'), '/', futureExpiry, 1, 1, 1, 0);
+  insert.run('.google.com', 'SID', '', encryptCookieValue('google-sid-value'), '/', futureExpiry, 1, 1, 1, 1);
+
+  // Domain 3: .example.com — 1 unencrypted cookie (value field set, no encrypted_value)
+  insert.run('.example.com', 'plain_cookie', 'hello-world', Buffer.alloc(0), '/', futureExpiry, 0, 0, 1, 1);
+
+  // Domain 4: .expired.com — 1 expired cookie (should be filtered out)
+  insert.run('.expired.com', 'old', '', encryptCookieValue('expired-value'), '/', pastExpiry, 0, 0, 1, 1);
+
+  // Domain 5: .session.com — session cookie (has_expires=0)
+  insert.run('.session.com', 'sess', '', encryptCookieValue('session-value'), '/', 0, 1, 1, 0, 1);
+
+  // Domain 6: .corrupt.com — cookie with garbage encrypted_value
+  insert.run('.corrupt.com', 'bad', '', Buffer.from('v10' + 'not-valid-ciphertext-at-all'), '/', futureExpiry, 0, 0, 1, 1);
+
+  // Domain 7: .mixed.com — one good, one corrupt
+  insert.run('.mixed.com', 'good', '', encryptCookieValue('mixed-good'), '/', futureExpiry, 0, 0, 1, 1);
+  insert.run('.mixed.com', 'bad', '', Buffer.from('v10' + 'garbage-data-here!!!'), '/', futureExpiry, 0, 0, 1, 1);
+
+  db.close();
+}
+
+// ─── Mock Setup ─────────────────────────────────────────────────
+// We need to mock:
+// 1. The Keychain access (getKeychainPassword) to return TEST_PASSWORD
+// 2. The cookie DB path resolution to use our fixture DB
+
+// We'll import the module after setting up the mocks
+let findInstalledBrowsers: any;
+let listDomains: any;
+let importCookies: any;
+let CookieImportError: any;
+
+beforeAll(async () => {
+  createFixtureDb();
+
+  // Mock Bun.spawn to return test password for keychain access
+  const origSpawn = Bun.spawn;
+  // @ts-ignore - monkey-patching for test
+  Bun.spawn = function(cmd: any, opts: any) {
+    // Intercept security find-generic-password calls
+    if (Array.isArray(cmd) && cmd[0] === 'security' && cmd[1] === 'find-generic-password') {
+      const service = cmd[3]; // -s <service>
+      // Return test password for any known test service
+      return {
+        stdout: new ReadableStream({
+          start(controller) {
+            controller.enqueue(new TextEncoder().encode(TEST_PASSWORD + '\n'));
+            controller.close();
+          }
+        }),
+        stderr: new ReadableStream({
+          start(controller) { controller.close(); }
+        }),
+        exited: Promise.resolve(0),
+        kill: () => {},
+      };
+    }
+    // Pass through other spawn calls
+    return origSpawn(cmd, opts);
+  };
+
+  // Import the module (uses our mocked Bun.spawn)
+  const mod = await import('../src/cookie-import-browser');
+  findInstalledBrowsers = mod.findInstalledBrowsers;
+  listDomains = mod.listDomains;
+  importCookies = mod.importCookies;
+  CookieImportError = mod.CookieImportError;
+});
+
+afterAll(() => {
+  // Clean up fixture DB
+  try { fs.unlinkSync(FIXTURE_DB); } catch {}
+  try { fs.rmdirSync(FIXTURE_DIR); } catch {}
+});
+
+// ─── Helper: Override DB path for tests ─────────────────────────
+// The real code resolves paths via ~/Library/Application Support/<browser>/Default/Cookies
+// We need to test against our fixture DB directly. We'll test the pure decryption functions
+// by calling importCookies with a browser that points to our fixture.
+// Since the module uses a hardcoded registry, we test the decryption logic via a different approach:
+// We'll directly call the internal decryption by setting up the DB in the expected location.
+
+// For the unit tests below, we test the decryption pipeline by:
+// 1. Creating encrypted cookies with known values
+// 2. Decrypting them with the module's decryption logic
+// The actual DB path resolution is tested separately.
+
+// ─── Tests ──────────────────────────────────────────────────────
+
+describe('Cookie Import Browser', () => {
+
+  describe('Decryption Pipeline', () => {
+    test('encrypts and decrypts round-trip correctly', () => {
+      // Verify our test helper produces valid ciphertext
+      const encrypted = encryptCookieValue('hello-world');
+      expect(encrypted.slice(0, 3).toString()).toBe('v10');
+
+      // Decrypt manually to verify
+      const ciphertext = encrypted.slice(3);
+      const decipher = crypto.createDecipheriv('aes-128-cbc', TEST_KEY, IV);
+      const plaintext = Buffer.concat([decipher.update(ciphertext), decipher.final()]);
+      // Skip 32-byte HMAC tag
+      const value = plaintext.slice(32).toString('utf-8');
+      expect(value).toBe('hello-world');
+    });
+
+    test('handles empty encrypted_value', () => {
+      const encrypted = encryptCookieValue('');
+      const ciphertext = encrypted.slice(3);
+      const decipher = crypto.createDecipheriv('aes-128-cbc', TEST_KEY, IV);
+      const plaintext = Buffer.concat([decipher.update(ciphertext), decipher.final()]);
+      // 32-byte tag + empty value → slice(32) = empty
+      expect(plaintext.length).toBe(32); // just the HMAC tag, padded to block boundary? Actually 32 + 0 padded = 48
+      // With PKCS7 padding: 32 bytes + 16 bytes of padding = 48 bytes padded → decrypts to 32 bytes + padding removed = 32 bytes
+    });
+
+    test('handles special characters in cookie values', () => {
+      const specialValue = 'a=b&c=d; path=/; expires=Thu, 01 Jan 2099';
+      const encrypted = encryptCookieValue(specialValue);
+      const ciphertext = encrypted.slice(3);
+      const decipher = crypto.createDecipheriv('aes-128-cbc', TEST_KEY, IV);
+      const plaintext = Buffer.concat([decipher.update(ciphertext), decipher.final()]);
+      expect(plaintext.slice(32).toString('utf-8')).toBe(specialValue);
+    });
+  });
+
+  describe('Fixture DB Structure', () => {
+    test('fixture DB has correct domain counts', () => {
+      const db = new Database(FIXTURE_DB, { readonly: true });
+      const rows = db.query(
+        `SELECT host_key, COUNT(*) as count FROM cookies GROUP BY host_key ORDER BY count DESC`
+      ).all() as any[];
+      db.close();
+
+      const counts = Object.fromEntries(rows.map((r: any) => [r.host_key, r.count]));
+      expect(counts['.github.com']).toBe(3);
+      expect(counts['.google.com']).toBe(2);
+      expect(counts['.example.com']).toBe(1);
+      expect(counts['.expired.com']).toBe(1);
+      expect(counts['.session.com']).toBe(1);
+      expect(counts['.corrupt.com']).toBe(1);
+      expect(counts['.mixed.com']).toBe(2);
+    });
+
+    test('encrypted cookies in fixture have v10 prefix', () => {
+      const db = new Database(FIXTURE_DB, { readonly: true });
+      const rows = db.query(
+        `SELECT name, encrypted_value FROM cookies WHERE host_key = '.github.com'`
+      ).all() as any[];
+      db.close();
+
+      for (const row of rows) {
+        const ev = Buffer.from(row.encrypted_value);
+        expect(ev.slice(0, 3).toString()).toBe('v10');
+      }
+    });
+
+    test('decrypts all github.com cookies from fixture DB', () => {
+      const db = new Database(FIXTURE_DB, { readonly: true });
+      const rows = db.query(
+        `SELECT name, value, encrypted_value FROM cookies WHERE host_key = '.github.com'`
+      ).all() as any[];
+      db.close();
+
+      const expected: Record<string, string> = {
+        'session_id': 'abc123',
+        'user_token': 'token-xyz',
+        'theme': 'dark',
+      };
+
+      for (const row of rows) {
+        const ev = Buffer.from(row.encrypted_value);
+        if (ev.length === 0) continue;
+        const ciphertext = ev.slice(3);
+        const decipher = crypto.createDecipheriv('aes-128-cbc', TEST_KEY, IV);
+        const plaintext = Buffer.concat([decipher.update(ciphertext), decipher.final()]);
+        const value = plaintext.slice(32).toString('utf-8');
+        expect(value).toBe(expected[row.name]);
+      }
+    });
+
+    test('unencrypted cookie uses value field directly', () => {
+      const db = new Database(FIXTURE_DB, { readonly: true });
+      const row = db.query(
+        `SELECT value, encrypted_value FROM cookies WHERE host_key = '.example.com'`
+      ).get() as any;
+      db.close();
+
+      expect(row.value).toBe('hello-world');
+      expect(Buffer.from(row.encrypted_value).length).toBe(0);
+    });
+  });
+
+  describe('sameSite Mapping', () => {
+    test('maps sameSite values correctly', () => {
+      // Read from fixture DB and verify mapping
+      const db = new Database(FIXTURE_DB, { readonly: true });
+
+      // samesite=0 → None
+      const none = db.query(`SELECT samesite FROM cookies WHERE name = 'user_token'`).get() as any;
+      expect(none.samesite).toBe(0);
+
+      // samesite=1 → Lax
+      const lax = db.query(`SELECT samesite FROM cookies WHERE name = 'session_id'`).get() as any;
+      expect(lax.samesite).toBe(1);
+
+      // samesite=2 → Strict
+      const strict = db.query(`SELECT samesite FROM cookies WHERE name = 'theme'`).get() as any;
+      expect(strict.samesite).toBe(2);
+
+      db.close();
+    });
+  });
+
+  describe('Chromium Epoch Conversion', () => {
+    test('converts Chromium epoch to Unix timestamp correctly', () => {
+      // Round-trip: pick a known Unix timestamp, convert to Chromium, convert back
+      const knownUnix = 1704067200; // 2024-01-01T00:00:00Z
+      const chromiumTs = BigInt(knownUnix) * 1000000n + CHROMIUM_EPOCH_OFFSET;
+      const unixTs = Number((chromiumTs - CHROMIUM_EPOCH_OFFSET) / 1000000n);
+      expect(unixTs).toBe(knownUnix);
+    });
+
+    test('session cookies (has_expires=0) get expires=-1', () => {
+      const db = new Database(FIXTURE_DB, { readonly: true });
+      const row = db.query(
+        `SELECT has_expires, expires_utc FROM cookies WHERE host_key = '.session.com'`
+      ).get() as any;
+      db.close();
+      expect(row.has_expires).toBe(0);
+      // When has_expires=0, the module should return expires=-1
+    });
+  });
+
+  describe('Error Handling', () => {
+    test('CookieImportError has correct properties', () => {
+      const err = new CookieImportError('test message', 'test_code', 'retry');
+      expect(err.message).toBe('test message');
+      expect(err.code).toBe('test_code');
+      expect(err.action).toBe('retry');
+      expect(err.name).toBe('CookieImportError');
+      expect(err instanceof Error).toBe(true);
+    });
+
+    test('CookieImportError without action', () => {
+      const err = new CookieImportError('no action', 'some_code');
+      expect(err.action).toBeUndefined();
+    });
+  });
+
+  describe('Browser Registry', () => {
+    test('findInstalledBrowsers returns array', () => {
+      const browsers = findInstalledBrowsers();
+      expect(Array.isArray(browsers)).toBe(true);
+      // Each entry should have the right shape
+      for (const b of browsers) {
+        expect(b).toHaveProperty('name');
+        expect(b).toHaveProperty('dataDir');
+        expect(b).toHaveProperty('keychainService');
+        expect(b).toHaveProperty('aliases');
+      }
+    });
+  });
+
+  describe('Corrupt Data Handling', () => {
+    test('garbage ciphertext produces decryption error', () => {
+      const garbage = Buffer.from('v10' + 'this-is-not-valid-ciphertext!!');
+      const ciphertext = garbage.slice(3);
+      expect(() => {
+        const decipher = crypto.createDecipheriv('aes-128-cbc', TEST_KEY, IV);
+        Buffer.concat([decipher.update(ciphertext), decipher.final()]);
+      }).toThrow();
+    });
+  });
+
+  describe('Profile Validation', () => {
+    test('rejects path traversal in profile names', () => {
+      // The validateProfile function should reject profiles with / or ..
+      // We can't call it directly (internal), but we can test via listDomains
+      // which calls validateProfile
+      expect(() => listDomains('chrome', '../etc')).toThrow(/Invalid profile/);
+      expect(() => listDomains('chrome', 'Default/../../etc')).toThrow(/Invalid profile/);
+    });
+
+    test('rejects control characters in profile names', () => {
+      expect(() => listDomains('chrome', 'Default\x00evil')).toThrow(/Invalid profile/);
+    });
+  });
+
+  describe('Unknown Browser', () => {
+    test('throws for unknown browser name', () => {
+      expect(() => listDomains('firefox')).toThrow(/Unknown browser.*firefox/i);
+    });
+
+    test('error includes list of supported browsers', () => {
+      try {
+        listDomains('firefox');
+        throw new Error('Should have thrown');
+      } catch (err: any) {
+        expect(err.code).toBe('unknown_browser');
+        expect(err.message).toContain('comet');
+        expect(err.message).toContain('chrome');
+      }
+    });
+  });
+});