gsd-pi 2.33.1 → 2.34.0-dev.6769913

package/dist/resources/extensions/gsd/tests/worker-registry.test.ts

@@ -1,148 +0,0 @@
-/**
- * Tests for the parallel worker registry used by the dashboard overlay.
- *
- * Verifies worker lifecycle (register → update → cleanup), batch grouping,
- * and the hasActiveWorkers() status check.
- */
-
-import { createTestContext } from './test-helpers.ts';
-import {
-  registerWorker,
-  updateWorker,
-  getActiveWorkers,
-  getWorkerBatches,
-  hasActiveWorkers,
-  resetWorkerRegistry,
-} from '../../subagent/worker-registry.ts';
-
-const { assertEq, assertTrue, report } = createTestContext();
-
-// ─── Setup ────────────────────────────────────────────────────────────────────
-
-resetWorkerRegistry();
-
-// ─── Registration ─────────────────────────────────────────────────────────────
-
-console.log("\n=== Worker Registration ===");
-
-{
-  resetWorkerRegistry();
-  const id = registerWorker("scout", "Explore codebase", 0, 3, "batch-1");
-  assertTrue(id.startsWith("worker-"), "worker ID has correct prefix");
-  const workers = getActiveWorkers();
-  assertEq(workers.length, 1, "one worker registered");
-  assertEq(workers[0].agent, "scout", "worker agent name correct");
-  assertEq(workers[0].task, "Explore codebase", "worker task correct");
-  assertEq(workers[0].status, "running", "worker starts as running");
-  assertEq(workers[0].index, 0, "worker index correct");
-  assertEq(workers[0].batchSize, 3, "worker batch size correct");
-  assertEq(workers[0].batchId, "batch-1", "worker batch ID correct");
-}
-
-// ─── Multiple workers in a batch ──────────────────────────────────────────────
-
-console.log("\n=== Multiple Workers in a Batch ===");
-
-{
-  resetWorkerRegistry();
-  const id1 = registerWorker("scout", "Task A", 0, 3, "batch-2");
-  const id2 = registerWorker("researcher", "Task B", 1, 3, "batch-2");
-  const id3 = registerWorker("worker", "Task C", 2, 3, "batch-2");
-
-  const workers = getActiveWorkers();
-  assertEq(workers.length, 3, "three workers registered");
-  assertTrue(hasActiveWorkers(), "has active workers");
-
-  const batches = getWorkerBatches();
-  assertEq(batches.size, 1, "one batch");
-  const batch = batches.get("batch-2");
-  assertTrue(batch !== undefined, "batch-2 exists");
-  assertEq(batch!.length, 3, "batch has 3 workers");
-}
-
-// ─── Worker status updates ────────────────────────────────────────────────────
-
-console.log("\n=== Worker Status Updates ===");
-
-{
-  resetWorkerRegistry();
-  const id1 = registerWorker("scout", "Task A", 0, 2, "batch-3");
-  const id2 = registerWorker("worker", "Task B", 1, 2, "batch-3");
-
-  updateWorker(id1, "completed");
-  const workers = getActiveWorkers();
-  const w1 = workers.find(w => w.id === id1);
-  assertEq(w1?.status, "completed", "worker 1 marked completed");
-
-  const w2 = workers.find(w => w.id === id2);
-  assertEq(w2?.status, "running", "worker 2 still running");
-  assertTrue(hasActiveWorkers(), "still has active workers (worker 2 running)");
-}
-
-// ─── Failed worker ────────────────────────────────────────────────────────────
-
-console.log("\n=== Failed Worker ===");
-
-{
-  resetWorkerRegistry();
-  const id = registerWorker("scout", "Task A", 0, 1, "batch-4");
-  updateWorker(id, "failed");
-  const workers = getActiveWorkers();
-  assertEq(workers[0].status, "failed", "worker marked failed");
-}
-
-// ─── Multiple batches ─────────────────────────────────────────────────────────
-
-console.log("\n=== Multiple Batches ===");
-
-{
-  resetWorkerRegistry();
-  registerWorker("scout", "Task A", 0, 2, "batch-5");
-  registerWorker("worker", "Task B", 1, 2, "batch-5");
-  registerWorker("researcher", "Task C", 0, 1, "batch-6");
-
-  const batches = getWorkerBatches();
-  assertEq(batches.size, 2, "two batches");
-  assertEq(batches.get("batch-5")!.length, 2, "batch-5 has 2 workers");
-  assertEq(batches.get("batch-6")!.length, 1, "batch-6 has 1 worker");
-}
-
-// ─── hasActiveWorkers with all completed ──────────────────────────────────────
-
-console.log("\n=== hasActiveWorkers — all completed ===");
-
-{
-  resetWorkerRegistry();
-  const id1 = registerWorker("scout", "Task A", 0, 2, "batch-7");
-  const id2 = registerWorker("worker", "Task B", 1, 2, "batch-7");
-  updateWorker(id1, "completed");
-  updateWorker(id2, "completed");
-  assertTrue(!hasActiveWorkers(), "no active workers when all completed");
-}
-
-// ─── Reset clears everything ─────────────────────────────────────────────────
-
-console.log("\n=== Reset ===");
-
-{
-  registerWorker("scout", "Task", 0, 1, "batch-8");
-  assertTrue(getActiveWorkers().length > 0, "workers exist before reset");
-  resetWorkerRegistry();
-  assertEq(getActiveWorkers().length, 0, "no workers after reset");
-  assertTrue(!hasActiveWorkers(), "hasActiveWorkers false after reset");
-}
-
-// ─── Update non-existent worker is no-op ──────────────────────────────────────
-
-console.log("\n=== Update non-existent worker ===");
-
-{
-  resetWorkerRegistry();
-  // Should not throw
-  updateWorker("nonexistent-id", "completed");
-  assertEq(getActiveWorkers().length, 0, "no workers created by updating nonexistent");
-}
-
-// ─── Summary ──────────────────────────────────────────────────────────────────
-
-report();

package/dist/resources/extensions/gsd/tests/workflow-templates.test.ts

@@ -1,173 +0,0 @@
-// GSD Workflow Templates — Unit Tests
-//
-// Tests registry loading, template resolution, auto-detection, and listing.
-
-import { createTestContext } from './test-helpers.ts';
-import {
-  loadRegistry,
-  resolveByName,
-  autoDetect,
-  listTemplates,
-  getTemplateInfo,
-  loadWorkflowTemplate,
-} from '../workflow-templates.ts';
-
-const { assertEq, assertTrue, assertMatch, report } = createTestContext();
-
-// ═══════════════════════════════════════════════════════════════════════════
-// Registry Loading
-// ═══════════════════════════════════════════════════════════════════════════
-
-console.log('\n── Registry Loading ──');
-
-{
-  const registry = loadRegistry();
-  assertTrue(registry !== null, 'Registry should load');
-  assertEq(registry.version, 1, 'Registry version should be 1');
-  assertTrue(Object.keys(registry.templates).length >= 8, 'Should have at least 8 templates');
-
-  // Verify required template keys exist
-  const expectedIds = ['full-project', 'bugfix', 'small-feature', 'refactor', 'spike', 'hotfix', 'security-audit', 'dep-upgrade'];
-  for (const id of expectedIds) {
-    assertTrue(id in registry.templates, `Template "${id}" should exist in registry`);
-  }
-
-  // Verify each template has required fields
-  for (const [id, entry] of Object.entries(registry.templates)) {
-    assertTrue(typeof entry.name === 'string' && entry.name.length > 0, `${id}: name should be non-empty string`);
-    assertTrue(typeof entry.description === 'string' && entry.description.length > 0, `${id}: description should be non-empty`);
-    assertTrue(typeof entry.file === 'string' && entry.file.endsWith('.md'), `${id}: file should be a .md path`);
-    assertTrue(Array.isArray(entry.phases) && entry.phases.length > 0, `${id}: phases should be non-empty array`);
-    assertTrue(Array.isArray(entry.triggers) && entry.triggers.length > 0, `${id}: triggers should be non-empty array`);
-  }
-}
-
-// ═══════════════════════════════════════════════════════════════════════════
-// Resolve by Name
-// ═══════════════════════════════════════════════════════════════════════════
-
-console.log('\n── Resolve by Name ──');
-
-{
-  // Exact match
-  const bugfix = resolveByName('bugfix');
-  assertTrue(bugfix !== null, 'Should resolve "bugfix"');
-  assertEq(bugfix!.id, 'bugfix', 'ID should be "bugfix"');
-  assertEq(bugfix!.confidence, 'exact', 'Exact name should have exact confidence');
-
-  // Case-insensitive name match
-  const spike = resolveByName('Research Spike');
-  assertTrue(spike !== null, 'Should resolve "Research Spike" by name');
-  assertEq(spike!.id, 'spike', 'Should resolve to spike');
-
-  // Alias match
-  const bug = resolveByName('bug');
-  assertTrue(bug !== null, 'Should resolve "bug" alias');
-  assertEq(bug!.id, 'bugfix', 'Alias "bug" should map to bugfix');
-
-  const feat = resolveByName('feat');
-  assertTrue(feat !== null, 'Should resolve "feat" alias');
-  assertEq(feat!.id, 'small-feature', 'Alias "feat" should map to small-feature');
-
-  const deps = resolveByName('deps');
-  assertTrue(deps !== null, 'Should resolve "deps" alias');
-  assertEq(deps!.id, 'dep-upgrade', 'Alias "deps" should map to dep-upgrade');
-
-  // No match
-  const missing = resolveByName('nonexistent-template');
-  assertTrue(missing === null, 'Should return null for unknown template');
-}
-
-// ═══════════════════════════════════════════════════════════════════════════
-// Auto-Detection
-// ═══════════════════════════════════════════════════════════════════════════
-
-console.log('\n── Auto-Detection ──');
-
-{
-  // Should detect bugfix from "fix" keyword
-  const fixMatches = autoDetect('fix the login button');
-  assertTrue(fixMatches.length > 0, 'Should detect matches for "fix the login button"');
-  assertTrue(fixMatches.some(m => m.id === 'bugfix'), 'Should include bugfix in matches');
-
-  // Should detect spike from "research" keyword
-  const researchMatches = autoDetect('research authentication libraries');
-  assertTrue(researchMatches.length > 0, 'Should detect matches for "research"');
-  assertTrue(researchMatches.some(m => m.id === 'spike'), 'Should include spike in matches');
-
-  // Should detect hotfix from "urgent" keyword
-  const urgentMatches = autoDetect('urgent production is down');
-  assertTrue(urgentMatches.length > 0, 'Should detect matches for "urgent"');
-  assertTrue(urgentMatches.some(m => m.id === 'hotfix'), 'Should include hotfix in matches');
-
-  // Should detect dep-upgrade from "upgrade" keyword
-  const upgradeMatches = autoDetect('upgrade react to v19');
-  assertTrue(upgradeMatches.length > 0, 'Should detect matches for "upgrade"');
-  assertTrue(upgradeMatches.some(m => m.id === 'dep-upgrade'), 'Should include dep-upgrade in matches');
-
-  // Multi-word triggers should score higher
-  const projectMatches = autoDetect('create a new project from scratch');
-  const projectMatch = projectMatches.find(m => m.id === 'full-project');
-  assertTrue(projectMatch !== undefined, 'Should detect full-project for "from scratch"');
-
-  // Empty input should return no matches
-  const emptyMatches = autoDetect('');
-  assertEq(emptyMatches.length, 0, 'Empty input should return no matches');
-}
-
-// ═══════════════════════════════════════════════════════════════════════════
-// List Templates
-// ═══════════════════════════════════════════════════════════════════════════
-
-console.log('\n── List Templates ──');
-
-{
-  const output = listTemplates();
-  assertTrue(output.includes('Workflow Templates'), 'Should have header');
-  assertTrue(output.includes('bugfix'), 'Should list bugfix');
-  assertTrue(output.includes('spike'), 'Should list spike');
-  assertTrue(output.includes('hotfix'), 'Should list hotfix');
-  assertTrue(output.includes('/gsd start'), 'Should include usage hint');
-}
-
-// ═══════════════════════════════════════════════════════════════════════════
-// Template Info
-// ═══════════════════════════════════════════════════════════════════════════
-
-console.log('\n── Template Info ──');
-
-{
-  const info = getTemplateInfo('bugfix');
-  assertTrue(info !== null, 'Should return info for bugfix');
-  assertTrue(info!.includes('Bug Fix'), 'Should include template name');
-  assertTrue(info!.includes('triage'), 'Should include phase names');
-  assertTrue(info!.includes('Triggers'), 'Should include triggers section');
-
-  const missing = getTemplateInfo('nonexistent');
-  assertTrue(missing === null, 'Should return null for unknown template');
-}
-
-// ═══════════════════════════════════════════════════════════════════════════
-// Load Workflow Template Content
-// ═══════════════════════════════════════════════════════════════════════════
-
-console.log('\n── Load Workflow Template ──');
-
-{
-  const content = loadWorkflowTemplate('bugfix');
-  assertTrue(content !== null, 'Should load bugfix template');
-  assertTrue(content!.includes('Bugfix Workflow'), 'Should contain workflow title');
-  assertTrue(content!.includes('Phase 1: Triage'), 'Should contain triage phase');
-  assertTrue(content!.includes('Phase 4: Ship'), 'Should contain ship phase');
-
-  const hotfixContent = loadWorkflowTemplate('hotfix');
-  assertTrue(hotfixContent !== null, 'Should load hotfix template');
-  assertTrue(hotfixContent!.includes('Hotfix Workflow'), 'Should contain hotfix title');
-
-  const missingContent = loadWorkflowTemplate('nonexistent');
-  assertTrue(missingContent === null, 'Should return null for unknown template');
-}
-
-// ═══════════════════════════════════════════════════════════════════════════
-
-report();

package/dist/resources/extensions/gsd/tests/workspace-index.test.ts

@@ -1,38 +0,0 @@
-import test from "node:test";
-import assert from "node:assert/strict";
-import { mkdtempSync, mkdirSync, rmSync, writeFileSync } from "node:fs";
-import { join } from "node:path";
-import { tmpdir } from "node:os";
-import { getSuggestedNextCommands, indexWorkspace, listDoctorScopeSuggestions } from "../workspace-index.ts";
-
-test("workspace index: indexes active milestone/slice/task and suggests commands", async () => {
-  const base = mkdtempSync(join(tmpdir(), "gsd-workspace-index-test-"));
-  const gsd = join(base, ".gsd");
-  const mDir = join(gsd, "milestones", "M001");
-  const sDir = join(mDir, "slices", "S01");
-  mkdirSync(join(sDir, "tasks"), { recursive: true });
-
-  writeFileSync(join(mDir, "M001-ROADMAP.md"), `# M001: Demo Milestone\n\n## Slices\n- [ ] **S01: Demo Slice** \`risk:low\` \`depends:[]\`\n  > After this: demo works\n`);
-  writeFileSync(join(sDir, "S01-PLAN.md"), `# S01: Demo Slice\n\n**Goal:** Demo\n**Demo:** Demo\n\n## Must-Haves\n- done\n\n## Tasks\n- [ ] **T01: Implement thing** \`est:10m\`\n  Task is in progress.\n`);
-  writeFileSync(join(sDir, "tasks", "T01-PLAN.md"), `# T01: Implement thing\n\n## Steps\n- do it\n`);
-
-  try {
-    const index = await indexWorkspace(base);
-    assert.equal(index.active.milestoneId, "M001");
-    assert.equal(index.active.sliceId, "S01");
-    assert.equal(index.active.taskId, "T01");
-    assert.ok(index.scopes.some(s => s.scope === "M001/S01"));
-    assert.ok(index.scopes.some(s => s.scope === "M001/S01/T01"));
-
-    const suggestions = await listDoctorScopeSuggestions(base);
-    assert.equal(suggestions[0].value, "M001/S01");
-    assert.ok(suggestions.some(item => item.value === "M001/S01/T01"));
-
-    const commands = await getSuggestedNextCommands(base);
-    assert.ok(commands.includes("/gsd auto"));
-    assert.ok(commands.includes("/gsd doctor M001/S01"));
-    assert.ok(commands.includes("/gsd status"));
-  } finally {
-    rmSync(base, { recursive: true, force: true });
-  }
-});

package/dist/resources/extensions/gsd/tests/worktree-bugfix.test.ts

@@ -1,120 +0,0 @@
-/**
- * Tests for worktree edge-case bugfixes:
- *
- *   1. resolveGitDir() follows gitdir: pointer in worktrees
- *   2. captureIntegrationBranch() is a no-op in worktrees
- *   3. detectWorktreeName() correctly identifies worktree paths
- */
-
-import {
-  mkdtempSync, mkdirSync, writeFileSync, rmSync,
-  existsSync, readFileSync,
-} from "node:fs";
-import { join } from "node:path";
-import { tmpdir } from "node:os";
-import { execSync } from "node:child_process";
-import { describe, it, after } from "node:test";
-
-import { resolveGitDir } from "../worktree-manager.ts";
-import { detectWorktreeName, captureIntegrationBranch } from "../worktree.ts";
-import { createTestContext } from "./test-helpers.ts";
-
-const { assertEq, assertTrue, report } = createTestContext();
-
-// ─── Helpers ──────────────────────────────────────────────────────────────
-
-function run(cmd: string, cwd: string): void {
-  execSync(cmd, { cwd, stdio: "ignore" });
-}
-
-function initRepo(dir: string): void {
-  run("git init", dir);
-  run("git config user.email test@test.com", dir);
-  run("git config user.name Test", dir);
-  run("git commit --allow-empty -m init", dir);
-}
-
-// ─── Tests ────────────────────────────────────────────────────────────────
-
-describe("worktree-bugfix", () => {
-  const dirs: string[] = [];
-  after(() => {
-    for (const d of dirs) rmSync(d, { recursive: true, force: true });
-    report();
-  });
-
-  it("resolveGitDir returns .git directory in normal repo", () => {
-    const repo = mkdtempSync(join(tmpdir(), "gsd-wt-fix-"));
-    dirs.push(repo);
-    initRepo(repo);
-    const gitDir = resolveGitDir(repo);
-    assertTrue(gitDir.endsWith(".git"), "ends with .git");
-    assertTrue(existsSync(gitDir), ".git dir exists");
-  });
-
-  it("resolveGitDir follows gitdir: pointer in worktree", () => {
-    const repo = mkdtempSync(join(tmpdir(), "gsd-wt-fix-"));
-    dirs.push(repo);
-    initRepo(repo);
-
-    // Simulate a worktree .git file (git worktree add creates these)
-    const wtDir = mkdtempSync(join(tmpdir(), "gsd-wt-fix-wt-"));
-    dirs.push(wtDir);
-    const realGitDir = join(repo, ".git", "worktrees", "test-wt");
-    mkdirSync(realGitDir, { recursive: true });
-    writeFileSync(join(wtDir, ".git"), `gitdir: ${realGitDir}\n`);
-
-    const resolved = resolveGitDir(wtDir);
-    assertEq(resolved, realGitDir, "resolves to real git dir");
-  });
-
-  it("resolveGitDir returns default when .git doesn't exist", () => {
-    const noGit = mkdtempSync(join(tmpdir(), "gsd-wt-fix-"));
-    dirs.push(noGit);
-    const gitDir = resolveGitDir(noGit);
-    assertTrue(gitDir.endsWith(".git"), "returns default .git path");
-  });
-
-  it("detectWorktreeName returns name for worktree path", () => {
-    assertEq(
-      detectWorktreeName("/project/.gsd/worktrees/M005"),
-      "M005",
-      "detects worktree name",
-    );
-  });
-
-  it("detectWorktreeName returns null for normal repo", () => {
-    assertEq(
-      detectWorktreeName("/project"),
-      null,
-      "null for non-worktree path",
-    );
-  });
-
-  it("captureIntegrationBranch is a no-op when in a worktree", () => {
-    const repo = mkdtempSync(join(tmpdir(), "gsd-wt-fix-"));
-    dirs.push(repo);
-    initRepo(repo);
-
-    // Create a fake worktree path structure
-    const wtPath = join(repo, ".gsd", "worktrees", "M005");
-    mkdirSync(wtPath, { recursive: true });
-    mkdirSync(join(wtPath, ".gsd", "milestones", "M005"), { recursive: true });
-    // Initialize git in the worktree so getService doesn't fail
-    initRepo(wtPath);
-
-    // captureIntegrationBranch should be a no-op — no META.json written
-    const metaPath = join(wtPath, ".gsd", "milestones", "M005", "M005-META.json");
-    captureIntegrationBranch(wtPath, "M005");
-    assertTrue(!existsSync(metaPath), "no META.json written in worktree");
-  });
-
-  it("detectWorktreeName prevents pull in worktree context", () => {
-    // Verifies the guard pattern: if detectWorktreeName returns non-null,
-    // the caller should skip pull/fetch operations
-    const inWorktree = detectWorktreeName("/project/.gsd/worktrees/M006");
-    const inNormal = detectWorktreeName("/project");
-    assertTrue(inWorktree !== null, "worktree detected → skip pull");
-    assertTrue(inNormal === null, "normal repo → allow pull");
-  });
-});