@winspan/claude-forge 8.53.2 → 8.54.4

package/tests/unit/skills/upgrade-engine.test.ts

@@ -1,401 +0,0 @@
-/**
- * Tests for upgrade-engine.ts — matchToOfficial, evaluateWithAI,
- * generateReport, applyDecisions, and pullCandidates (partial).
- */
-
-import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest';
-import path from 'node:path';
-import { promises as fs } from 'node:fs';
-import os from 'node:os';
-import {
-  matchToOfficial,
-  evaluateWithAI,
-  generateReport,
-  applyDecisions,
-} from '../../../src/skills/upgrade-engine.js';
-import type { CandidateSkill, ReportEntry } from '../../../src/skills/upgrade-engine.js';
-import type { OfficialSkill } from '../../../src/skills/official-skills.js';
-import type { ClaudeProvider } from '../../../src/core/ai/provider.js';
-
-// ── Fixtures ────────────────────────────────────────────────────────────────
-
-const officialSkills: OfficialSkill[] = [
-  {
-    name: 'official-db-schema-design',
-    version: '1.0.0',
-    description: '数据库设计方法论',
-    keywords: ['schema', 'database', 'sql', 'migration'],
-    content: '---\nname: official-db-schema-design\nversion: 1.0.0\n---\n# DB Schema Design\n\nOfficial content.',
-  },
-  {
-    name: 'official-debug',
-    version: '1.2.0',
-    description: '系统化调试工作流',
-    keywords: ['debug', 'bug', 'troubleshooting'],
-    content: '---\nname: official-debug\nversion: 1.2.0\n---\n# Debug\n\nOfficial debug content.',
-  },
-];
-
-function makeCandidate(overrides: Partial<CandidateSkill> = {}): CandidateSkill {
-  return {
-    id: 'database-design',
-    source: 'agent-skills',
-    filePath: '/tmp/agent-skills/database-design/SKILL.md',
-    name: 'database-design',
-    description: 'Database schema design patterns',
-    keywords: ['database', 'schema', 'sql'],
-    content: '---\nname: database-design\n---\n# Database Design\n\nContent here.',
-    ...overrides,
-  };
-}
-
-function makeMockProvider(returnValue: string): ClaudeProvider {
-  return {
-    complete: vi.fn().mockResolvedValue(returnValue),
-  } as unknown as ClaudeProvider;
-}
-
-// ── matchToOfficial ──────────────────────────────────────────────────────────
-
-describe('matchToOfficial', () => {
-  it('returns a match for a candidate with overlapping keywords', () => {
-    const candidate = makeCandidate();
-    const result = matchToOfficial(candidate, officialSkills);
-    expect(result).not.toBeNull();
-    expect(result?.officialId).toBe('official-db-schema-design');
-    expect(result?.score).toBeGreaterThanOrEqual(30);
-  });
-
-  it('returns null when score is below threshold (30)', () => {
-    const candidate = makeCandidate({
-      id: 'cooking-recipes',
-      name: 'cooking',
-      description: 'Culinary arts and baking',
-      keywords: ['cooking', 'baking', 'recipes', 'food'],
-    });
-    const result = matchToOfficial(candidate, officialSkills);
-    expect(result).toBeNull();
-  });
-
-  it('matches debug candidate to official-debug', () => {
-    const candidate = makeCandidate({
-      id: 'debug-helper',
-      name: 'debug-helper',
-      description: 'Debugging workflow',
-      keywords: ['debug', 'bug', 'troubleshoot'],
-    });
-    const result = matchToOfficial(candidate, officialSkills);
-    expect(result?.officialId).toBe('official-debug');
-  });
-
-  it('score is within 0-100 range', () => {
-    const candidate = makeCandidate();
-    const result = matchToOfficial(candidate, officialSkills);
-    if (result) {
-      expect(result.score).toBeGreaterThanOrEqual(0);
-      expect(result.score).toBeLessThanOrEqual(100);
-    }
-  });
-
-  it('returns null for empty official skills list', () => {
-    const candidate = makeCandidate();
-    const result = matchToOfficial(candidate, []);
-    expect(result).toBeNull();
-  });
-});
-
-// ── evaluateWithAI — three action paths ─────────────────────────────────────
-
-describe('evaluateWithAI', () => {
-  const official = officialSkills[0];
-  const candidate = makeCandidate();
-
-  it('returns upgrade decision when AI returns action=upgrade', async () => {
-    const json = JSON.stringify({
-      action: 'upgrade',
-      confidence: 85,
-      reasoning: '候选内容更全面',
-      merged_content: null,
-    });
-    const provider = makeMockProvider(json);
-    const result = await evaluateWithAI(candidate, official, provider);
-    expect(result.action).toBe('upgrade');
-    expect(result.confidence).toBe(85);
-    expect(result.reasoning).toBe('候选内容更全面');
-    expect(result.merged_content).toBeNull();
-  });
-
-  it('returns merge decision with merged_content', async () => {
-    const mergedMd = '---\nname: merged\n---\n# Merged Content';
-    const json = JSON.stringify({
-      action: 'merge',
-      confidence: 72,
-      reasoning: '各有独特章节',
-      merged_content: mergedMd,
-    });
-    const provider = makeMockProvider(json);
-    const result = await evaluateWithAI(candidate, official, provider);
-    expect(result.action).toBe('merge');
-    expect(result.merged_content).toBe(mergedMd);
-  });
-
-  it('returns skip decision when AI returns action=skip', async () => {
-    const json = JSON.stringify({
-      action: 'skip',
-      confidence: 30,
-      reasoning: '主题不重叠',
-      merged_content: null,
-    });
-    const provider = makeMockProvider(json);
-    const result = await evaluateWithAI(candidate, official, provider);
-    expect(result.action).toBe('skip');
-  });
-
-  it('returns needs_review when AI response is not valid JSON', async () => {
-    const provider = makeMockProvider('这不是JSON内容，AI出错了');
-    const result = await evaluateWithAI(candidate, official, provider);
-    expect(result.needs_review).toBe(true);
-    expect(result.action).toBe('skip');
-    expect(result.confidence).toBe(0);
-  });
-
-  it('strips markdown code fences before parsing', async () => {
-    const json = JSON.stringify({ action: 'skip', confidence: 50, reasoning: 'ok', merged_content: null });
-    const provider = makeMockProvider('```json\n' + json + '\n```');
-    const result = await evaluateWithAI(candidate, official, provider);
-    expect(result.action).toBe('skip');
-    expect(result.needs_review).toBeUndefined();
-  });
-
-  it('clamps confidence to 0-100 range', async () => {
-    const json = JSON.stringify({ action: 'upgrade', confidence: 150, reasoning: 'ok', merged_content: null });
-    const provider = makeMockProvider(json);
-    const result = await evaluateWithAI(candidate, official, provider);
-    expect(result.confidence).toBe(100);
-  });
-
-  it('returns needs_review when AI provider throws', async () => {
-    const provider = {
-      complete: vi.fn().mockRejectedValue(new Error('API timeout')),
-    } as unknown as ClaudeProvider;
-    const result = await evaluateWithAI(candidate, official, provider);
-    expect(result.needs_review).toBe(true);
-    expect(result.action).toBe('skip');
-    expect(result.reasoning).toContain('API timeout');
-  });
-});
-
-// ── generateReport ────────────────────────────────────────────────────────────
-
-describe('generateReport', () => {
-  let tmpDir: string;
-
-  beforeEach(async () => {
-    tmpDir = await fs.mkdtemp(path.join(os.tmpdir(), 'upgrade-test-'));
-  });
-
-  afterEach(async () => {
-    await fs.rm(tmpDir, { recursive: true, force: true });
-  });
-
-  const entries: ReportEntry[] = [
-    {
-      officialId: 'official-db-schema-design',
-      candidateId: 'database-design',
-      candidateSource: 'agent-skills',
-      action: 'upgrade',
-      confidence: 85,
-      reasoning: '候选更全面',
-      candidateFilePath: '/tmp/agent-skills/database-design/SKILL.md',
-      merged_content: null,
-    },
-    {
-      officialId: 'official-debug',
-      candidateId: 'debug-helper',
-      candidateSource: 'superpowers',
-      action: 'merge',
-      confidence: 70,
-      reasoning: '各有特色',
-      candidateFilePath: '/tmp/superpowers/debug-helper.md',
-      merged_content: '---\nname: merged-debug\n---\n# Merged Debug Content',
-    },
-    {
-      officialId: 'official-tdd',
-      candidateId: 'tdd-guide',
-      candidateSource: 'agent-skills',
-      action: 'skip',
-      confidence: 20,
-      reasoning: '不够好',
-      candidateFilePath: '/tmp/agent-skills/tdd-guide/SKILL.md',
-      merged_content: null,
-    },
-  ];
-
-  const stats = { total: 10, matched: 3, unmatched: 7 };
-
-  it('creates report file at the specified path', async () => {
-    const reportPath = path.join(tmpDir, 'report.md');
-    await generateReport(entries, reportPath, stats);
-    const exists = await fs.access(reportPath).then(() => true).catch(() => false);
-    expect(exists).toBe(true);
-  });
-
-  it('report contains summary table with correct counts', async () => {
-    const reportPath = path.join(tmpDir, 'report.md');
-    await generateReport(entries, reportPath, stats);
-    const content = await fs.readFile(reportPath, 'utf-8');
-    expect(content).toContain('| Candidates scanned | 10 |');
-    expect(content).toContain('| Matched to official | 3 |');
-    expect(content).toContain('| Action: upgrade | 1 |');
-    expect(content).toContain('| Action: merge | 1 |');
-    expect(content).toContain('| Action: skip | 1 |');
-  });
-
-  it('report contains HTML comment machine markers for upgrade', async () => {
-    const reportPath = path.join(tmpDir, 'report.md');
-    await generateReport(entries, reportPath, stats);
-    const content = await fs.readFile(reportPath, 'utf-8');
-    expect(content).toContain(
-      '<!-- upgrade-entry: official-db-schema-design | /tmp/agent-skills/database-design/SKILL.md | upgrade -->',
-    );
-  });
-
-  it('report contains merged-content-begin/end block for merge entries', async () => {
-    const reportPath = path.join(tmpDir, 'report.md');
-    await generateReport(entries, reportPath, stats);
-    const content = await fs.readFile(reportPath, 'utf-8');
-    expect(content).toContain('<!-- merged-content-begin -->');
-    expect(content).toContain('<!-- merged-content-end -->');
-    expect(content).toContain('# Merged Debug Content');
-  });
-
-  it('report contains per-skill sections for all entries', async () => {
-    const reportPath = path.join(tmpDir, 'report.md');
-    await generateReport(entries, reportPath, stats);
-    const content = await fs.readFile(reportPath, 'utf-8');
-    expect(content).toContain('### official-db-schema-design');
-    expect(content).toContain('### official-debug');
-    expect(content).toContain('### official-tdd');
-  });
-});
-
-// ── applyDecisions ────────────────────────────────────────────────────────────
-
-describe('applyDecisions', () => {
-  let tmpDir: string;
-  let officialDir: string;
-  let backupBaseDir: string;
-  let reportPath: string;
-  let candidatePath: string;
-
-  beforeEach(async () => {
-    tmpDir = await fs.mkdtemp(path.join(os.tmpdir(), 'apply-test-'));
-    officialDir = path.join(tmpDir, 'official');
-    backupBaseDir = path.join(tmpDir, 'backups');
-    reportPath = path.join(tmpDir, 'report.md');
-    candidatePath = path.join(tmpDir, 'candidate.md');
-
-    await fs.mkdir(officialDir, { recursive: true });
-    await fs.mkdir(backupBaseDir, { recursive: true });
-
-    // Create a fake official skill file
-    await fs.writeFile(
-      path.join(officialDir, 'official-db-schema-design.md'),
-      '---\nname: official-db-schema-design\n---\n# Original Content',
-      'utf-8',
-    );
-
-    // Create a fake candidate file
-    await fs.writeFile(
-      candidatePath,
-      '---\nname: database-design\n---\n# Better Content from candidate',
-      'utf-8',
-    );
-  });
-
-  afterEach(async () => {
-    await fs.rm(tmpDir, { recursive: true, force: true });
-  });
-
-  it('creates a backup directory before applying', async () => {
-    const report = `# Report\n\n<!-- upgrade-entry: official-db-schema-design | ${candidatePath} | upgrade -->\n`;
-    await fs.writeFile(reportPath, report, 'utf-8');
-
-    const { backupPath } = await applyDecisions(reportPath, officialDir, backupBaseDir);
-    const backupExists = await fs.access(backupPath).then(() => true).catch(() => false);
-    expect(backupExists).toBe(true);
-  });
-
-  it('copies existing official skills to backup before applying', async () => {
-    const report = `# Report\n\n<!-- upgrade-entry: official-db-schema-design | ${candidatePath} | upgrade -->\n`;
-    await fs.writeFile(reportPath, report, 'utf-8');
-
-    const { backupPath } = await applyDecisions(reportPath, officialDir, backupBaseDir);
-    const backupFile = path.join(backupPath, 'official-db-schema-design.md');
-    const backupExists = await fs.access(backupFile).then(() => true).catch(() => false);
-    expect(backupExists).toBe(true);
-  });
-
-  it('applies upgrade action by writing candidate content to official dir', async () => {
-    const report = `# Report\n\n<!-- upgrade-entry: official-db-schema-design | ${candidatePath} | upgrade -->\n`;
-    await fs.writeFile(reportPath, report, 'utf-8');
-
-    const { applied } = await applyDecisions(reportPath, officialDir, backupBaseDir);
-    expect(applied).toBe(1);
-
-    const updatedContent = await fs.readFile(
-      path.join(officialDir, 'official-db-schema-design.md'),
-      'utf-8',
-    );
-    expect(updatedContent).toContain('Better Content from candidate');
-  });
-
-  it('skips entries with action=skip', async () => {
-    const report = `# Report\n\n<!-- upgrade-entry: official-db-schema-design | ${candidatePath} | skip -->\n`;
-    await fs.writeFile(reportPath, report, 'utf-8');
-
-    const { applied, skipped } = await applyDecisions(reportPath, officialDir, backupBaseDir);
-    expect(applied).toBe(0);
-    expect(skipped).toBe(1);
-  });
-
-  it('applies merge action using merged-content-begin/end block', async () => {
-    const mergedContent = '---\nname: merged\n---\n# Merged Result';
-    const report =
-      `# Report\n\n` +
-      `<!-- upgrade-entry: official-db-schema-design | ${candidatePath} | merge -->\n\n` +
-      `<!-- merged-content-begin -->\n${mergedContent}\n<!-- merged-content-end -->\n`;
-    await fs.writeFile(reportPath, report, 'utf-8');
-
-    const { applied } = await applyDecisions(reportPath, officialDir, backupBaseDir);
-    expect(applied).toBe(1);
-
-    const updatedContent = await fs.readFile(
-      path.join(officialDir, 'official-db-schema-design.md'),
-      'utf-8',
-    );
-    expect(updatedContent).toContain('# Merged Result');
-  });
-
-  it('uses -2 suffix for backup when timestamp conflicts', async () => {
-    const report = `# Report\n\n<!-- upgrade-entry: official-db-schema-design | ${candidatePath} | upgrade -->\n`;
-    await fs.writeFile(reportPath, report, 'utf-8');
-
-    // Run twice — second run should get -2 suffix backup
-    const { backupPath: bp1 } = await applyDecisions(reportPath, officialDir, backupBaseDir);
-    // Re-create official file (it was overwritten)
-    await fs.writeFile(
-      path.join(officialDir, 'official-db-schema-design.md'),
-      '---\nname: official-db-schema-design\n---\n# Original',
-      'utf-8',
-    );
-
-    // Manually create a directory at the same timestamp that would be used
-    const dirs1 = await fs.readdir(backupBaseDir);
-    expect(dirs1.length).toBeGreaterThanOrEqual(1);
-
-    // The first backup path should exist
-    const bp1Exists = await fs.access(bp1).then(() => true).catch(() => false);
-    expect(bp1Exists).toBe(true);
-  });
-});

package/tests/unit/skills/upgrade-prompt.test.ts

@@ -1,89 +0,0 @@
-import { describe, it, expect } from 'vitest';
-import { buildEvaluationPrompt } from '../../../src/skills/upgrade-prompt.js';
-import type { CandidateSkill } from '../../../src/skills/upgrade-engine.js';
-import type { OfficialSkill } from '../../../src/skills/official-skills.js';
-
-const candidateMock: CandidateSkill = {
-  id: 'database-design',
-  source: 'agent-skills',
-  filePath: '/tmp/agent-skills/database-design/SKILL.md',
-  name: 'database-design',
-  description: 'Comprehensive database design guide with sharding strategies',
-  keywords: ['database', 'schema', 'migration', 'sharding'],
-  content: '---\nname: database-design\n---\n# Database Design\n\nContent here about schemas and migrations.',
-};
-
-const officialMock: OfficialSkill = {
-  name: 'official-db-schema-design',
-  version: '1.0.0',
-  description: '数据库设计方法论',
-  keywords: ['schema', 'database', 'sql'],
-  content: '---\nname: official-db-schema-design\n---\n# DB Schema Design\n\nOfficial content here.',
-};
-
-describe('buildEvaluationPrompt', () => {
-  it('returns an object with system and user keys', () => {
-    const result = buildEvaluationPrompt(candidateMock, officialMock);
-    expect(result).toHaveProperty('system');
-    expect(result).toHaveProperty('user');
-  });
-
-  it('system prompt contains upgrade keyword', () => {
-    const { system } = buildEvaluationPrompt(candidateMock, officialMock);
-    expect(system.toLowerCase()).toContain('upgrade');
-  });
-
-  it('system prompt contains merge keyword', () => {
-    const { system } = buildEvaluationPrompt(candidateMock, officialMock);
-    expect(system.toLowerCase()).toContain('merge');
-  });
-
-  it('system prompt contains skip keyword', () => {
-    const { system } = buildEvaluationPrompt(candidateMock, officialMock);
-    expect(system.toLowerCase()).toContain('skip');
-  });
-
-  it('system prompt contains JSON format requirement', () => {
-    const { system } = buildEvaluationPrompt(candidateMock, officialMock);
-    // Should mention JSON format with the required fields
-    expect(system).toContain('action');
-    expect(system).toContain('confidence');
-    expect(system).toContain('reasoning');
-    expect(system).toContain('merged_content');
-  });
-
-  it('user prompt contains official skill name', () => {
-    const { user } = buildEvaluationPrompt(candidateMock, officialMock);
-    expect(user).toContain('official-db-schema-design');
-  });
-
-  it('user prompt contains candidate skill source', () => {
-    const { user } = buildEvaluationPrompt(candidateMock, officialMock);
-    expect(user).toContain('agent-skills');
-  });
-
-  it('user prompt contains candidate skill id', () => {
-    const { user } = buildEvaluationPrompt(candidateMock, officialMock);
-    expect(user).toContain('database-design');
-  });
-
-  it('user prompt truncates long content to ~2000 chars', () => {
-    const longContent = 'x'.repeat(5000);
-    const candidate: CandidateSkill = { ...candidateMock, content: longContent };
-    const { user } = buildEvaluationPrompt(candidate, officialMock);
-    // Content should be truncated - user prompt should not contain 5000 x's
-    expect(user).not.toContain('x'.repeat(3000));
-    // But should contain truncation marker
-    expect(user).toContain('[content truncated]');
-  });
-
-  it('user prompt includes official skill keywords', () => {
-    const { user } = buildEvaluationPrompt(candidateMock, officialMock);
-    expect(user).toContain('schema');
-  });
-
-  it('user prompt includes JSON instruction', () => {
-    const { user } = buildEvaluationPrompt(candidateMock, officialMock);
-    expect(user.toLowerCase()).toContain('json');
-  });
-});

package/tests/unit/socket-server.test.ts

@@ -1,183 +0,0 @@
-/**
- * SocketServer behavior tests
- *
- * 重点验证 L3 性能修复：buffer 分片到达时，仅在遇到换行符后才解析，
- * 避免大事件每个 chunk 都跑一次完整 JSON.parse 的 N² 行为。
- */
-import { describe, it, expect, afterEach, vi } from 'vitest';
-import net from 'node:net';
-import os from 'node:os';
-import path from 'node:path';
-import fs from 'node:fs';
-import { SocketServer } from '../../src/daemon/server.js';
-import { EventParser } from '../../src/daemon/event-parser.js';
-
-function makeSockPath(): string {
-  const name = `forge-test-${process.pid}-${Date.now()}-${Math.random().toString(36).slice(2, 8)}.sock`;
-  return path.join(os.tmpdir(), name);
-}
-
-function makeEvent(overrides: Record<string, unknown> = {}): Record<string, unknown> {
-  return {
-    hook_type: 'PostToolUse',
-    timestamp: new Date().toISOString(),
-    session_id: 'test-session',
-    project_path: '/tmp/test',
-    tool_name: 'Read',
-    ...overrides,
-  };
-}
-
-function sendOnSocket(sockPath: string, chunks: string[], delayMs = 5): Promise<string> {
-  return new Promise((resolve, reject) => {
-    const client = net.createConnection(sockPath);
-    let response = '';
-    client.on('connect', async () => {
-      for (const chunk of chunks) {
-        client.write(chunk);
-        if (delayMs > 0) await new Promise((r) => setTimeout(r, delayMs));
-      }
-    });
-    client.on('data', (data) => {
-      response += data.toString();
-    });
-    client.on('end', () => resolve(response));
-    client.on('close', () => resolve(response));
-    client.on('error', reject);
-  });
-}
-
-describe('SocketServer — L3 chunked-parse behavior', () => {
-  const servers: SocketServer[] = [];
-  const sockPaths: string[] = [];
-
-  afterEach(async () => {
-    for (const s of servers) await s.close();
-    servers.length = 0;
-    for (const p of sockPaths) {
-      try { fs.unlinkSync(p); } catch { /* ignore */ }
-    }
-    sockPaths.length = 0;
-  });
-
-  it('parses a complete single-chunk message (with trailing newline)', async () => {
-    const sockPath = makeSockPath();
-    sockPaths.push(sockPath);
-
-    const handler = vi.fn().mockReturnValue({ allow: true });
-    const server = new SocketServer(sockPath, handler);
-    servers.push(server);
-    await new Promise((r) => setTimeout(r, 30));
-
-    const event = makeEvent();
-    const payload = JSON.stringify(event) + '\n';
-    await sendOnSocket(sockPath, [payload]);
-
-    expect(handler).toHaveBeenCalledTimes(1);
-    const passed = handler.mock.calls[0][0];
-    expect(passed.hook_type).toBe('PostToolUse');
-    expect(passed.session_id).toBe('test-session');
-  });
-
-  it('parses a large message split across many chunks WITHOUT per-chunk JSON.parse', async () => {
-    const sockPath = makeSockPath();
-    sockPaths.push(sockPath);
-
-    // 监视 JSON.parse —— 用于断言不再 N²（不能为每个 chunk 都 parse 一次）
-    const parseSpy = vi.spyOn(JSON, 'parse');
-
-    const handler = vi.fn().mockReturnValue({ allow: true });
-    const server = new SocketServer(sockPath, handler);
-    servers.push(server);
-    await new Promise((r) => setTimeout(r, 30));
-
-    // 构造大事件（约 50KB tool_output），分成 ~20 个 chunk 喂入
-    const bigOutput = 'x'.repeat(50_000);
-    const event = makeEvent({ tool_output: { data: bigOutput } });
-    const payload = JSON.stringify(event) + '\n';
-    const chunkSize = Math.ceil(payload.length / 20);
-    const chunks: string[] = [];
-    for (let i = 0; i < payload.length; i += chunkSize) {
-      chunks.push(payload.slice(i, i + chunkSize));
-    }
-    expect(chunks.length).toBeGreaterThan(10);
-
-    const callsBefore = parseSpy.mock.calls.length;
-    await sendOnSocket(sockPath, chunks, 2);
-    const callsAfter = parseSpy.mock.calls.length;
-
-    // 关键断言：handler 必须收到完整事件
-    expect(handler).toHaveBeenCalledTimes(1);
-
-    // 关键断言：JSON.parse 调用次数远少于 chunk 数（旧代码会 ≈ chunks.length 次）
-    // 现在应该只有：可能 1 次（无 auth 时）或 2 次（有 auth 时——auth 检查 + EventParser）
-    const parseDelta = callsAfter - callsBefore;
-    expect(parseDelta).toBeLessThanOrEqual(3);
-    expect(parseDelta).toBeLessThan(chunks.length); // 强对比旧 N² 行为
-
-    parseSpy.mockRestore();
-  });
-
-  it('rejects oversized buffer that never sees a newline', async () => {
-    const sockPath = makeSockPath();
-    sockPaths.push(sockPath);
-
-    const handler = vi.fn();
-    const server = new SocketServer(sockPath, handler);
-    servers.push(server);
-    await new Promise((r) => setTimeout(r, 30));
-
-    // 600KB 无换行 —— 必须超过 MAX_BUFFER_SIZE (512KB) 并被丢弃
-    // 注意：服务端 destroy 后客户端继续 write 会触发 EPIPE，要容错
-    const oversized = 'a'.repeat(600 * 1024);
-    await new Promise<void>((resolve) => {
-      const client = net.createConnection(sockPath);
-      client.on('error', () => { /* EPIPE 预期 */ });
-      client.on('close', () => resolve());
-      client.on('connect', () => {
-        client.write(oversized, () => {
-          // 给服务端处理时间
-          setTimeout(() => client.destroy(), 80);
-        });
-      });
-    });
-
-    expect(handler).not.toHaveBeenCalled();
-  });
-
-  it('uses fallback parse on socket end() for messages without trailing newline', async () => {
-    const sockPath = makeSockPath();
-    sockPaths.push(sockPath);
-
-    const handler = vi.fn().mockReturnValue({ allow: true });
-    const server = new SocketServer(sockPath, handler);
-    servers.push(server);
-    await new Promise((r) => setTimeout(r, 30));
-
-    // 无换行结尾 —— 走 socket.end() 兜底
-    const event = makeEvent();
-    const payload = JSON.stringify(event);
-    await new Promise<void>((resolve, reject) => {
-      const client = net.createConnection(sockPath);
-      client.on('connect', () => {
-        client.end(payload); // 写完即关闭：触发 server 的 'end' 事件
-      });
-      client.on('close', () => resolve());
-      client.on('error', reject);
-    });
-    // 等 server 端处理完
-    await new Promise((r) => setTimeout(r, 50));
-
-    expect(handler).toHaveBeenCalledTimes(1);
-  });
-});
-
-describe('EventParser — sanity', () => {
-  it('parses a valid event JSON', () => {
-    const parser = new EventParser();
-    const raw = JSON.stringify(makeEvent());
-    const out = parser.parse(raw);
-    expect(out.session_id).toBe('test-session');
-    expect(out.hook_type).toBe('PostToolUse');
-  });
-});