lsd-pi 1.0.0 → 1.1.0

package/src/resources/extensions/gsd/tests/tool-call-loop-guard.test.ts

@@ -1,166 +0,0 @@
-// tool-call-loop-guard — Tests for the tool-call loop detection guard.
-//
-// Verifies that identical consecutive tool calls are detected and blocked
-// after exceeding the threshold, and that the guard resets properly.
-
-import { test } from 'node:test';
-import assert from 'node:assert/strict';
-import {
-  checkToolCallLoop,
-  resetToolCallLoopGuard,
-  disableToolCallLoopGuard,
-  getToolCallLoopCount,
-} from '../bootstrap/tool-call-loop-guard.ts';
-
-
-// ═══════════════════════════════════════════════════════════════════════════
-// Allows first N calls, blocks after threshold
-// ═══════════════════════════════════════════════════════════════════════════
-
-console.log('\n── Loop guard: blocks after threshold ──');
-
-{
-  resetToolCallLoopGuard();
-
-  // First 4 identical calls should be allowed (threshold is 4)
-  for (let i = 1; i <= 4; i++) {
-    const result = checkToolCallLoop('web_search', { query: 'same query' });
-    assert.ok(result.block === false, `Call ${i} should be allowed`);
-    assert.deepStrictEqual(result.count, i, `Count should be ${i} after call ${i}`);
-  }
-
-  // 5th identical call should be blocked
-  const blocked = checkToolCallLoop('web_search', { query: 'same query' });
-  assert.ok(blocked.block === true, '5th identical call should be blocked');
-  assert.ok(blocked.reason!.includes('web_search'), 'Reason should mention tool name');
-  assert.ok(blocked.reason!.includes('5'), 'Reason should mention count');
-}
-
-// ═══════════════════════════════════════════════════════════════════════════
-// Different tool calls reset the streak
-// ═══════════════════════════════════════════════════════════════════════════
-
-console.log('\n── Loop guard: different calls reset streak ──');
-
-{
-  resetToolCallLoopGuard();
-
-  checkToolCallLoop('web_search', { query: 'query A' });
-  checkToolCallLoop('web_search', { query: 'query A' });
-  checkToolCallLoop('web_search', { query: 'query A' });
-  assert.deepStrictEqual(getToolCallLoopCount(), 3, 'Count should be 3 after 3 identical calls');
-
-  // A different call resets the streak
-  const different = checkToolCallLoop('bash', { command: 'ls' });
-  assert.ok(different.block === false, 'Different tool call should be allowed');
-  assert.deepStrictEqual(getToolCallLoopCount(), 1, 'Count should reset to 1 after different call');
-
-  // Same tool but different args also resets
-  checkToolCallLoop('web_search', { query: 'query A' });
-  checkToolCallLoop('web_search', { query: 'query B' }); // different args
-  assert.deepStrictEqual(getToolCallLoopCount(), 1, 'Different args should reset count');
-}
-
-// ═══════════════════════════════════════════════════════════════════════════
-// Reset clears the guard
-// ═══════════════════════════════════════════════════════════════════════════
-
-console.log('\n── Loop guard: reset clears state ──');
-
-{
-  resetToolCallLoopGuard();
-  checkToolCallLoop('web_search', { query: 'q' });
-  checkToolCallLoop('web_search', { query: 'q' });
-  checkToolCallLoop('web_search', { query: 'q' });
-  assert.deepStrictEqual(getToolCallLoopCount(), 3, 'Count should be 3 before reset');
-
-  resetToolCallLoopGuard();
-  assert.deepStrictEqual(getToolCallLoopCount(), 0, 'Count should be 0 after reset');
-
-  // After reset, the same call starts fresh
-  const result = checkToolCallLoop('web_search', { query: 'q' });
-  assert.ok(result.block === false, 'Call after reset should be allowed');
-  assert.deepStrictEqual(getToolCallLoopCount(), 1, 'Count should be 1 after first call post-reset');
-}
-
-// ═══════════════════════════════════════════════════════════════════════════
-// Disable makes guard permissive
-// ═══════════════════════════════════════════════════════════════════════════
-
-console.log('\n── Loop guard: disable allows everything ──');
-
-{
-  disableToolCallLoopGuard();
-
-  for (let i = 0; i < 10; i++) {
-    const result = checkToolCallLoop('web_search', { query: 'same' });
-    assert.ok(result.block === false, `Call ${i + 1} should be allowed when disabled`);
-  }
-
-  // Re-enable via reset
-  resetToolCallLoopGuard();
-  checkToolCallLoop('web_search', { query: 'q' });
-  assert.deepStrictEqual(getToolCallLoopCount(), 1, 'Guard should be active again after reset');
-}
-
-// ═══════════════════════════════════════════════════════════════════════════
-// Arg order doesn't affect hash
-// ═══════════════════════════════════════════════════════════════════════════
-
-console.log('\n── Loop guard: arg order is normalized ──');
-
-{
-  resetToolCallLoopGuard();
-
-  checkToolCallLoop('web_search', { query: 'test', limit: 5 });
-  const result = checkToolCallLoop('web_search', { limit: 5, query: 'test' }); // same args, different order
-  assert.ok(result.block === false, 'Same args in different order should count as consecutive');
-  assert.deepStrictEqual(getToolCallLoopCount(), 2, 'Should detect as same call regardless of key order');
-}
-
-// ═══════════════════════════════════════════════════════════════════════════
-// Nested/array arguments produce distinct hashes
-// ═══════════════════════════════════════════════════════════════════════════
-
-console.log('\n── Loop guard: nested args are not stripped ──');
-
-{
-  resetToolCallLoopGuard();
-
-  // Simulate ask_user_questions-style calls with different nested content
-  for (let i = 1; i <= 5; i++) {
-    const result = checkToolCallLoop('ask_user_questions', {
-      questions: [{ id: `q${i}`, question: `Question ${i}?` }],
-    });
-    assert.ok(result.block === false, `Nested call ${i} with unique content should be allowed`);
-    assert.deepStrictEqual(getToolCallLoopCount(), 1, `Each unique nested call should reset count to 1`);
-  }
-
-  // Truly identical nested calls should still be detected
-  resetToolCallLoopGuard();
-  for (let i = 1; i <= 4; i++) {
-    checkToolCallLoop('ask_user_questions', {
-      questions: [{ id: 'same', question: 'Same?' }],
-    });
-  }
-  const blocked = checkToolCallLoop('ask_user_questions', {
-    questions: [{ id: 'same', question: 'Same?' }],
-  });
-  assert.ok(blocked.block === true, 'Identical nested calls should still be blocked');
-}
-
-// ═══════════════════════════════════════════════════════════════════════════
-// Nested object key order is normalized
-// ═══════════════════════════════════════════════════════════════════════════
-
-console.log('\n── Loop guard: nested key order is normalized ──');
-
-{
-  resetToolCallLoopGuard();
-
-  checkToolCallLoop('tool', { outer: { b: 2, a: 1 } });
-  const result = checkToolCallLoop('tool', { outer: { a: 1, b: 2 } });
-  assert.deepStrictEqual(getToolCallLoopCount(), 2, 'Same nested args in different key order should match');
-}
-
-// ═══════════════════════════════════════════════════════════════════════════

package/src/resources/extensions/gsd/tests/tool-naming.test.ts

@@ -1,125 +0,0 @@
-// tool-naming — Verifies canonical + alias tool registration for GSD DB tools.
-//
-// Each DB tool must register under its canonical gsd_concept_action name
-// AND under a backward-compatible alias name.
-// The alias must share the exact same execute function reference as the canonical tool.
-
-import { test } from 'node:test';
-import assert from 'node:assert/strict';
-import { registerDbTools } from '../bootstrap/db-tools.ts';
-
-
-// ─── Mock PI ──────────────────────────────────────────────────────────────────
-
-function makeMockPi() {
-  const tools: any[] = [];
-  return {
-    registerTool: (tool: any) => tools.push(tool),
-    tools,
-  } as any;
-}
-
-// ─── Rename map ───────────────────────────────────────────────────────────────
-
-const RENAME_MAP: Array<{ canonical: string; alias: string }> = [
-  { canonical: "gsd_decision_save", alias: "gsd_save_decision" },
-  { canonical: "gsd_requirement_update", alias: "gsd_update_requirement" },
-  { canonical: "gsd_requirement_save", alias: "gsd_save_requirement" },
-  { canonical: "gsd_summary_save", alias: "gsd_save_summary" },
-  { canonical: "gsd_milestone_generate_id", alias: "gsd_generate_milestone_id" },
-  { canonical: "gsd_task_complete", alias: "gsd_complete_task" },
-  { canonical: "gsd_slice_complete", alias: "gsd_complete_slice" },
-  { canonical: "gsd_plan_milestone", alias: "gsd_milestone_plan" },
-  { canonical: "gsd_plan_slice", alias: "gsd_slice_plan" },
-  { canonical: "gsd_plan_task", alias: "gsd_task_plan" },
-  { canonical: "gsd_replan_slice", alias: "gsd_slice_replan" },
-  { canonical: "gsd_reassess_roadmap", alias: "gsd_roadmap_reassess" },
-  { canonical: "gsd_complete_milestone", alias: "gsd_milestone_complete" },
-  { canonical: "gsd_validate_milestone", alias: "gsd_milestone_validate" },
-];
-
-// ─── Registration count ──────────────────────────────────────────────────────
-
-console.log('\n── Tool naming: registration count ──');
-
-const pi = makeMockPi();
-registerDbTools(pi);
-
-assert.deepStrictEqual(pi.tools.length, 29, 'Should register exactly 29 tools (14 canonical + 14 aliases + 1 gate tool)');
-
-// ─── Both names exist for each pair ──────────────────────────────────────────
-
-console.log('\n── Tool naming: canonical and alias names exist ──');
-
-for (const { canonical, alias } of RENAME_MAP) {
-  const canonicalTool = pi.tools.find((t: any) => t.name === canonical);
-  const aliasTool = pi.tools.find((t: any) => t.name === alias);
-
-  assert.ok(canonicalTool !== undefined, `Canonical tool "${canonical}" should be registered`);
-  assert.ok(aliasTool !== undefined, `Alias tool "${alias}" should be registered`);
-}
-
-// ─── Execute function identity ───────────────────────────────────────────────
-
-console.log('\n── Tool naming: execute function identity (===) ──');
-
-for (const { canonical, alias } of RENAME_MAP) {
-  const canonicalTool = pi.tools.find((t: any) => t.name === canonical);
-  const aliasTool = pi.tools.find((t: any) => t.name === alias);
-
-  if (canonicalTool && aliasTool) {
-    assert.ok(
-      canonicalTool.execute === aliasTool.execute,
-      `"${canonical}" and "${alias}" should share the same execute function reference`,
-    );
-  }
-}
-
-// ─── Alias descriptions include "(alias for ...)" ───────────────────────────
-
-console.log('\n── Tool naming: alias descriptions ──');
-
-for (const { canonical, alias } of RENAME_MAP) {
-  const aliasTool = pi.tools.find((t: any) => t.name === alias);
-
-  if (aliasTool) {
-    assert.ok(
-      aliasTool.description.includes(`alias for ${canonical}`),
-      `Alias "${alias}" description should include "alias for ${canonical}"`,
-    );
-  }
-}
-
-// ─── Canonical tools have proper promptGuidelines ────────────────────────────
-
-console.log('\n── Tool naming: canonical promptGuidelines use canonical name ──');
-
-for (const { canonical } of RENAME_MAP) {
-  const canonicalTool = pi.tools.find((t: any) => t.name === canonical);
-
-  if (canonicalTool) {
-    const guidelinesText = canonicalTool.promptGuidelines.join(' ');
-    assert.ok(
-      guidelinesText.includes(canonical),
-      `Canonical tool "${canonical}" promptGuidelines should reference its own name`,
-    );
-  }
-}
-
-// ─── Alias promptGuidelines direct to canonical ──────────────────────────────
-
-console.log('\n── Tool naming: alias promptGuidelines redirect to canonical ──');
-
-for (const { canonical, alias } of RENAME_MAP) {
-  const aliasTool = pi.tools.find((t: any) => t.name === alias);
-
-  if (aliasTool) {
-    const guidelinesText = aliasTool.promptGuidelines.join(' ');
-    assert.ok(
-      guidelinesText.includes(`Alias for ${canonical}`),
-      `Alias "${alias}" promptGuidelines should say "Alias for ${canonical}"`,
-    );
-  }
-}
-
-// ═══════════════════════════════════════════════════════════════════════════

package/src/resources/extensions/gsd/tests/triage-dispatch.test.ts

@@ -1,345 +0,0 @@
-/**
- * Triage dispatch ordering contract tests.
- *
- * These tests verify structural invariants of the triage integration
- * by inspecting the actual source code of auto-post-unit.ts, auto.ts,
- * and post-unit-hooks.ts. Full behavioral testing requires the
- * @gsd/pi-coding-agent runtime.
- */
-
-import test from "node:test";
-import assert from "node:assert/strict";
-import { readFileSync } from "node:fs";
-import { join, dirname } from "node:path";
-import { fileURLToPath } from "node:url";
-
-const __dirname = dirname(fileURLToPath(import.meta.url));
-const hooksPath = join(__dirname, "..", "post-unit-hooks.ts");
-const registryPath = join(__dirname, "..", "rule-registry.ts");
-const autoPromptsPath = join(__dirname, "..", "auto-prompts.ts");
-
-// After decomposition, triage/dispatch logic lives in auto-post-unit.ts
-const postUnitSrc = readFileSync(join(__dirname, "..", "auto-post-unit.ts"), "utf-8");
-// auto.ts retains top-level orchestration and imports
-const autoSrc = [
-  readFileSync(join(__dirname, "..", "auto.ts"), "utf-8"),
-  postUnitSrc,
-  readFileSync(join(__dirname, "..", "auto-start.ts"), "utf-8"),
-].join("\n");
-// Hook exclusion logic lives in the rule-registry (facade delegates there)
-const hooksSrc = [
-  readFileSync(hooksPath, "utf-8"),
-  readFileSync(registryPath, "utf-8"),
-].join("\n");
-const autoPromptsSrc = (() => { try { return readFileSync(autoPromptsPath, "utf-8"); } catch { return autoSrc; } })();
-
-// ─── Hook exclusion ──────────────────────────────────────────────────────────
-
-test("dispatch: triage-captures excluded from post-unit hook triggering", () => {
-  assert.ok(
-    hooksSrc.includes('"triage-captures"'),
-    "post-unit-hooks.ts should reference triage-captures",
-  );
-  assert.ok(
-    hooksSrc.includes('completedUnitType === "triage-captures"'),
-    "should check for triage-captures in the hook exclusion guard",
-  );
-});
-
-// ─── Triage check placement ──────────────────────────────────────────────────
-
-test("dispatch: triage check appears after hook section and before stepMode check", () => {
-  const triageCheckIndex = postUnitSrc.indexOf("// ── Triage check");
-  const quickTaskIndex = postUnitSrc.indexOf("// ── Quick-task dispatch");
-  const stepModeIndex = postUnitSrc.indexOf("if (s.stepMode)");
-
-  assert.ok(triageCheckIndex > 0, "triage check block should exist");
-  assert.ok(quickTaskIndex > 0, "quick-task dispatch block should exist");
-  assert.ok(stepModeIndex > 0, "step mode check should exist");
-
-  assert.ok(
-    triageCheckIndex < quickTaskIndex,
-    "triage check should come before quick-task dispatch",
-  );
-  assert.ok(
-    quickTaskIndex < stepModeIndex,
-    "quick-task dispatch should come before stepMode check",
-  );
-});
-
-// ─── Guard conditions ────────────────────────────────────────────────────────
-
-test("dispatch: triage check guards against step mode", () => {
-  const triageBlock = postUnitSrc.slice(
-    postUnitSrc.indexOf("// ── Triage check"),
-    postUnitSrc.indexOf("// ── Quick-task dispatch"),
-  );
-  assert.ok(
-    triageBlock.includes("!s.stepMode"),
-    "triage block should guard against step mode",
-  );
-});
-
-test("dispatch: triage check guards against hook unit types", () => {
-  const triageBlock = postUnitSrc.slice(
-    postUnitSrc.indexOf("// ── Triage check"),
-    postUnitSrc.indexOf("// ── Quick-task dispatch"),
-  );
-  assert.ok(
-    triageBlock.includes('!s.currentUnit.type.startsWith("hook/")'),
-    "triage block should not fire for hook units",
-  );
-});
-
-test("dispatch: triage check guards against triage-on-triage", () => {
-  const triageBlock = postUnitSrc.slice(
-    postUnitSrc.indexOf("// ── Triage check"),
-    postUnitSrc.indexOf("// ── Quick-task dispatch"),
-  );
-  assert.ok(
-    triageBlock.includes('s.currentUnit.type !== "triage-captures"'),
-    "triage block should not fire for triage units",
-  );
-});
-
-test("dispatch: triage check guards against quick-task triggering triage", () => {
-  const triageBlock = postUnitSrc.slice(
-    postUnitSrc.indexOf("// ── Triage check"),
-    postUnitSrc.indexOf("// ── Quick-task dispatch"),
-  );
-  assert.ok(
-    triageBlock.includes('s.currentUnit.type !== "quick-task"'),
-    "triage block should not fire for quick-task units",
-  );
-});
-
-test("dispatch: triage dispatch keeps the loop in continue mode", () => {
-  const triageBlock = postUnitSrc.slice(
-    postUnitSrc.indexOf("// ── Triage check"),
-    postUnitSrc.indexOf("// ── Quick-task dispatch"),
-  );
-  assert.ok(
-    triageBlock.includes('return "continue"') || triageBlock.includes("return enqueueSidecar("),
-    "triage dispatch should return 'continue' after enqueuing sidecar work",
-  );
-});
-
-test("dispatch: triage imports hasPendingCaptures and loadPendingCaptures", () => {
-  assert.ok(
-    autoSrc.includes("hasPendingCaptures") && autoSrc.includes("loadPendingCaptures"),
-    "should import capture functions",
-  );
-  assert.ok(
-    autoSrc.includes('from "./captures.js"'),
-    "should import from captures module",
-  );
-});
-
-// ─── Prompt integration ──────────────────────────────────────────────────────
-
-test("dispatch: replan prompt builder loads capture context", () => {
-  const src = autoPromptsSrc;
-  assert.ok(
-    src.includes("loadReplanCaptures"),
-    "buildReplanSlicePrompt should load replan captures",
-  );
-  assert.ok(
-    src.includes("captureContext"),
-    "buildReplanSlicePrompt should pass captureContext to template",
-  );
-});
-
-test("dispatch: reassess prompt builder loads deferred captures", () => {
-  const src = autoPromptsSrc;
-  assert.ok(
-    src.includes("loadDeferredCaptures"),
-    "buildReassessRoadmapPrompt should load deferred captures",
-  );
-  assert.ok(
-    src.includes("deferredCaptures"),
-    "buildReassessRoadmapPrompt should pass deferredCaptures to template",
-  );
-});
-
-// ─── Prompt templates ────────────────────────────────────────────────────────
-
-test("dispatch: replan prompt template includes captureContext variable", () => {
-  const promptPath = join(__dirname, "..", "prompts", "replan-slice.md");
-  const prompt = readFileSync(promptPath, "utf-8");
-  assert.ok(
-    prompt.includes("{{captureContext}}"),
-    "replan-slice.md should include {{captureContext}}",
-  );
-});
-
-test("dispatch: reassess prompt template includes deferredCaptures variable", () => {
-  const promptPath = join(__dirname, "..", "prompts", "reassess-roadmap.md");
-  const prompt = readFileSync(promptPath, "utf-8");
-  assert.ok(
-    prompt.includes("{{deferredCaptures}}"),
-    "reassess-roadmap.md should include {{deferredCaptures}}",
-  );
-});
-
-test("dispatch: triage prompt template exists and has classification criteria", () => {
-  const promptPath = join(__dirname, "..", "prompts", "triage-captures.md");
-  const prompt = readFileSync(promptPath, "utf-8");
-  assert.ok(prompt.includes("quick-task"), "should have quick-task classification");
-  assert.ok(prompt.includes("inject"), "should have inject classification");
-  assert.ok(prompt.includes("defer"), "should have defer classification");
-  assert.ok(prompt.includes("replan"), "should have replan classification");
-  assert.ok(prompt.includes("note"), "should have note classification");
-  assert.ok(prompt.includes("{{pendingCaptures}}"), "should have pending captures variable");
-});
-
-// ─── Dashboard integration ───────────────────────────────────────────────────
-
-test("dashboard: AutoDashboardData includes pendingCaptureCount field", () => {
-  assert.ok(
-    autoSrc.includes("pendingCaptureCount"),
-    "auto.ts should have pendingCaptureCount in AutoDashboardData",
-  );
-});
-
-test("dashboard: getAutoDashboardData computes pendingCaptureCount", () => {
-  assert.ok(
-    autoSrc.includes("pendingCaptureCount = countPendingCaptures") ||
-    autoSrc.includes("pendingCaptureCount = countPendingCaptures(basePath)"),
-    "getAutoDashboardData should compute pendingCaptureCount from countPendingCaptures (single-read)",
-  );
-});
-
-test("dashboard: overlay renders pending captures badge", () => {
-  const overlayPath = join(__dirname, "..", "dashboard-overlay.ts");
-  const overlaySrc = readFileSync(overlayPath, "utf-8");
-  assert.ok(
-    overlaySrc.includes("pendingCaptureCount"),
-    "dashboard-overlay.ts should reference pendingCaptureCount",
-  );
-  assert.ok(
-    overlaySrc.includes("pending capture"),
-    "dashboard-overlay.ts should show pending captures text",
-  );
-});
-
-test("dashboard: overlay labels triage-captures and quick-task unit types", () => {
-  const overlayPath = join(__dirname, "..", "dashboard-overlay.ts");
-  const overlaySrc = readFileSync(overlayPath, "utf-8");
-  assert.ok(
-    overlaySrc.includes('"triage-captures"'),
-    "unitLabel should handle triage-captures",
-  );
-  assert.ok(
-    overlaySrc.includes('"quick-task"'),
-    "unitLabel should handle quick-task",
-  );
-});
-
-// ─── Post-triage resolution execution ─────────────────────────────────────────
-
-test("dispatch: post-triage resolution executor fires after triage-captures unit", () => {
-  const postTriageBlock = postUnitSrc.slice(
-    postUnitSrc.indexOf("Post-triage: execute actionable resolutions"),
-  );
-  assert.ok(
-    postTriageBlock.includes('s.currentUnit.type === "triage-captures"'),
-    "should check for triage-captures unit completion",
-  );
-  assert.ok(
-    postTriageBlock.includes("executeTriageResolutions"),
-    "should call executeTriageResolutions",
-  );
-});
-
-test("dispatch: post-triage executor handles inject results", () => {
-  const postTriageBlock = postUnitSrc.slice(
-    postUnitSrc.indexOf("Post-triage: execute actionable resolutions"),
-  );
-  assert.ok(
-    postTriageBlock.includes("triageResult.injected"),
-    "should check injected count",
-  );
-});
-
-test("dispatch: post-triage executor handles replan results", () => {
-  const postTriageBlock = postUnitSrc.slice(
-    postUnitSrc.indexOf("Post-triage: execute actionable resolutions"),
-  );
-  assert.ok(
-    postTriageBlock.includes("triageResult.replanned"),
-    "should check replanned count",
-  );
-});
-
-test("dispatch: post-triage executor queues quick-tasks", () => {
-  const postTriageBlock = postUnitSrc.slice(
-    postUnitSrc.indexOf("Post-triage: execute actionable resolutions"),
-  );
-  assert.ok(
-    postTriageBlock.includes("s.pendingQuickTasks"),
-    "should push quick-tasks to s.pendingQuickTasks queue",
-  );
-});
-
-// ─── Quick-task dispatch ──────────────────────────────────────────────────────
-
-test("dispatch: quick-task dispatch block exists after triage check", () => {
-  const quickTaskBlock = postUnitSrc.indexOf("// ── Quick-task dispatch");
-  const triageBlock = postUnitSrc.indexOf("// ── Triage check");
-
-  assert.ok(quickTaskBlock > 0, "quick-task dispatch block should exist");
-  assert.ok(
-    quickTaskBlock > triageBlock,
-    "quick-task dispatch should come after triage check",
-  );
-});
-
-test("dispatch: quick-task dispatch uses buildQuickTaskPrompt", () => {
-  const quickTaskSection = postUnitSrc.slice(
-    postUnitSrc.indexOf("// ── Quick-task dispatch"),
-  );
-  assert.ok(
-    quickTaskSection.includes("buildQuickTaskPrompt"),
-    "should call buildQuickTaskPrompt for quick-task dispatch",
-  );
-});
-
-test("dispatch: quick-task dispatch marks capture as executed", () => {
-  const quickTaskSection = postUnitSrc.slice(
-    postUnitSrc.indexOf("// ── Quick-task dispatch"),
-  );
-  assert.ok(
-    quickTaskSection.includes("markCaptureExecuted"),
-    "should mark capture as executed after dispatch",
-  );
-});
-
-test("dispatch: quick-task dispatch keeps the loop in continue mode", () => {
-  const quickTaskSection = postUnitSrc.slice(
-    postUnitSrc.indexOf("// ── Quick-task dispatch"),
-    postUnitSrc.indexOf("if (s.stepMode)"),
-  );
-  assert.ok(
-    quickTaskSection.includes('return "continue"') || quickTaskSection.includes("return enqueueSidecar("),
-    "quick-task dispatch should return 'continue' after enqueuing sidecar work",
-  );
-});
-
-// ─── Post-unit hook exclusion for quick-task ──────────────────────────────────
-
-test("dispatch: quick-task excluded from post-unit hook triggering", () => {
-  assert.ok(
-    hooksSrc.includes('"quick-task"'),
-    "post-unit-hooks.ts should reference quick-task",
-  );
-});
-
-// ─── pendingQuickTasks queue lifecycle ────────────────────────────────────────
-
-test("dispatch: pendingQuickTasks queue is reset on auto-mode start/stop", () => {
-  const resetMatches = autoSrc.match(/s\.pendingQuickTasks = \[\]/g);
-  assert.ok(
-    resetMatches && resetMatches.length >= 2,
-    "s.pendingQuickTasks should be reset in start and stop paths",
-  );
-});