lsd-pi 1.0.0 → 1.1.0

package/src/resources/extensions/gsd/tests/workflow-projections.test.ts

@@ -1,173 +0,0 @@
-// GSD Extension — workflow-projections unit tests
-// Tests the pure rendering functions (no DB required).
-
-import test from 'node:test';
-import assert from 'node:assert/strict';
-import { renderPlanContent } from '../workflow-projections.ts';
-import type { SliceRow, TaskRow } from '../gsd-db.ts';
-
-// ─── Test fixtures ────────────────────────────────────────────────────────
-
-function makeSlice(overrides: Partial<SliceRow> = {}): SliceRow {
-  return {
-    id: 'S01',
-    milestone_id: 'M001',
-    title: 'Auth Layer',
-    status: 'active',
-    risk: 'high',
-    depends: [],
-    demo: 'Login flow works end-to-end',
-    goal: 'Implement JWT authentication',
-    full_summary_md: '',
-    full_uat_md: '',
-    success_criteria: '',
-    proof_level: '',
-    integration_closure: '',
-    observability_impact: '',
-    created_at: '2026-01-01T00:00:00Z',
-    completed_at: null,
-    sequence: 1,
-    replan_triggered_at: null,
-    ...overrides,
-  };
-}
-
-function makeTask(overrides: Partial<TaskRow> = {}): TaskRow {
-  return {
-    id: 'T01',
-    slice_id: 'S01',
-    milestone_id: 'M001',
-    title: 'Create JWT middleware',
-    status: 'pending',
-    description: 'Implement JWT validation middleware',
-    estimate: '2h',
-    files: ['src/middleware/auth.ts'],
-    verify: 'npm test src/middleware/auth.test.ts',
-    one_liner: '',
-    narrative: '',
-    verification_result: '',
-    duration: '',
-    completed_at: null,
-    blocker_discovered: false,
-    deviations: '',
-    known_issues: '',
-    key_files: [],
-    key_decisions: [],
-    full_summary_md: '',
-    full_plan_md: '',
-    inputs: [],
-    expected_output: [],
-    observability_impact: '',
-    sequence: 1,
-    ...overrides,
-  };
-}
-
-// ─── renderPlanContent: structure ────────────────────────────────────────
-
-test('workflow-projections: renderPlanContent starts with H1 containing slice id and title', () => {
-  const content = renderPlanContent(makeSlice(), []);
-  assert.ok(content.startsWith('# S01: Auth Layer'), `expected H1, got: ${content.slice(0, 60)}`);
-});
-
-test('workflow-projections: renderPlanContent includes Goal line', () => {
-  const content = renderPlanContent(makeSlice(), []);
-  assert.ok(content.includes('**Goal:** Implement JWT authentication'));
-});
-
-test('workflow-projections: renderPlanContent includes Demo line', () => {
-  const content = renderPlanContent(makeSlice(), []);
-  assert.ok(content.includes('**Demo:** After this: Login flow works end-to-end'));
-});
-
-test('workflow-projections: renderPlanContent falls back to TBD when goal and full_summary_md are empty', () => {
-  const slice = makeSlice({ goal: '', full_summary_md: '' });
-  const content = renderPlanContent(slice, []);
-  assert.ok(content.includes('**Goal:** TBD'));
-});
-
-test('workflow-projections: renderPlanContent falls back to TBD when goal is empty (full_summary_md ignored #2945)', () => {
-  const slice = makeSlice({ goal: '', full_summary_md: 'Fallback goal text' });
-  const content = renderPlanContent(slice, []);
-  // #2945: full_summary_md is no longer used as a fallback — it contains
-  // multi-line rendered markdown that corrupts single-line fields.
-  assert.ok(content.includes('**Goal:** TBD'), `expected TBD fallback, got: ${content}`);
-});
-
-test('workflow-projections: renderPlanContent includes ## Tasks section', () => {
-  const content = renderPlanContent(makeSlice(), []);
-  assert.ok(content.includes('## Tasks'));
-});
-
-// ─── renderPlanContent: task checkboxes ──────────────────────────────────
-
-test('workflow-projections: pending task renders with [ ] checkbox', () => {
-  const task = makeTask({ status: 'pending' });
-  const content = renderPlanContent(makeSlice(), [task]);
-  assert.ok(content.includes('- [ ] **T01:'), `expected unchecked, got: ${content}`);
-});
-
-test('workflow-projections: done task renders with [x] checkbox', () => {
-  const task = makeTask({ status: 'done' });
-  const content = renderPlanContent(makeSlice(), [task]);
-  assert.ok(content.includes('- [x] **T01:'), `expected checked, got: ${content}`);
-});
-
-test('workflow-projections: complete status renders with [x] checkbox', () => {
-  const task = makeTask({ status: 'complete' }); // 'complete' and 'done' both → checked
-  const content = renderPlanContent(makeSlice(), [task]);
-  assert.ok(content.includes('- [x] **T01:'));
-});
-
-// ─── renderPlanContent: task sublines ────────────────────────────────────
-
-test('workflow-projections: task with estimate renders Estimate subline', () => {
-  const task = makeTask({ estimate: '2h' });
-  const content = renderPlanContent(makeSlice(), [task]);
-  assert.ok(content.includes('  - Estimate: 2h'));
-});
-
-test('workflow-projections: task with empty estimate omits Estimate subline', () => {
-  const task = makeTask({ estimate: '' });
-  const content = renderPlanContent(makeSlice(), [task]);
-  assert.ok(!content.includes('  - Estimate:'));
-});
-
-test('workflow-projections: task with files renders Files subline', () => {
-  const task = makeTask({ files: ['src/auth.ts', 'src/auth.test.ts'] });
-  const content = renderPlanContent(makeSlice(), [task]);
-  assert.ok(content.includes('  - Files: src/auth.ts, src/auth.test.ts'));
-});
-
-test('workflow-projections: task with empty files array omits Files subline', () => {
-  const task = makeTask({ files: [] });
-  const content = renderPlanContent(makeSlice(), [task]);
-  assert.ok(!content.includes('  - Files:'));
-});
-
-test('workflow-projections: task with verify renders Verify subline', () => {
-  const task = makeTask({ verify: 'npm test' });
-  const content = renderPlanContent(makeSlice(), [task]);
-  assert.ok(content.includes('  - Verify: npm test'));
-});
-
-test('workflow-projections: task with no verify omits Verify subline', () => {
-  const task = makeTask({ verify: '' });
-  const content = renderPlanContent(makeSlice(), [task]);
-  assert.ok(!content.includes('  - Verify:'));
-});
-
-test('workflow-projections: task with duration renders Duration subline', () => {
-  const task = makeTask({ duration: '45m' });
-  const content = renderPlanContent(makeSlice(), [task]);
-  assert.ok(content.includes('  - Duration: 45m'));
-});
-
-test('workflow-projections: multiple tasks rendered in order', () => {
-  const t1 = makeTask({ id: 'T01', title: 'First task', sequence: 1 });
-  const t2 = makeTask({ id: 'T02', title: 'Second task', sequence: 2 });
-  const content = renderPlanContent(makeSlice(), [t1, t2]);
-  const idxT1 = content.indexOf('**T01:');
-  const idxT2 = content.indexOf('**T02:');
-  assert.ok(idxT1 < idxT2, 'T01 should appear before T02');
-});

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

@@ -1,171 +0,0 @@
-// GSD Workflow Templates — Unit Tests
-//
-// Tests registry loading, template resolution, auto-detection, and listing.
-
-import { test } from 'node:test';
-import assert from 'node:assert/strict';
-import {
-  loadRegistry,
-  resolveByName,
-  autoDetect,
-  listTemplates,
-  getTemplateInfo,
-  loadWorkflowTemplate,
-} from '../workflow-templates.ts';
-
-
-// ═══════════════════════════════════════════════════════════════════════════
-// Registry Loading
-// ═══════════════════════════════════════════════════════════════════════════
-
-console.log('\n── Registry Loading ──');
-
-{
-  const registry = loadRegistry();
-  assert.ok(registry !== null, 'Registry should load');
-  assert.deepStrictEqual(registry.version, 1, 'Registry version should be 1');
-  assert.ok(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) {
-    assert.ok(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)) {
-    assert.ok(typeof entry.name === 'string' && entry.name.length > 0, `${id}: name should be non-empty string`);
-    assert.ok(typeof entry.description === 'string' && entry.description.length > 0, `${id}: description should be non-empty`);
-    assert.ok(typeof entry.file === 'string' && entry.file.endsWith('.md'), `${id}: file should be a .md path`);
-    assert.ok(Array.isArray(entry.phases) && entry.phases.length > 0, `${id}: phases should be non-empty array`);
-    assert.ok(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');
-  assert.ok(bugfix !== null, 'Should resolve "bugfix"');
-  assert.deepStrictEqual(bugfix!.id, 'bugfix', 'ID should be "bugfix"');
-  assert.deepStrictEqual(bugfix!.confidence, 'exact', 'Exact name should have exact confidence');
-
-  // Case-insensitive name match
-  const spike = resolveByName('Research Spike');
-  assert.ok(spike !== null, 'Should resolve "Research Spike" by name');
-  assert.deepStrictEqual(spike!.id, 'spike', 'Should resolve to spike');
-
-  // Alias match
-  const bug = resolveByName('bug');
-  assert.ok(bug !== null, 'Should resolve "bug" alias');
-  assert.deepStrictEqual(bug!.id, 'bugfix', 'Alias "bug" should map to bugfix');
-
-  const feat = resolveByName('feat');
-  assert.ok(feat !== null, 'Should resolve "feat" alias');
-  assert.deepStrictEqual(feat!.id, 'small-feature', 'Alias "feat" should map to small-feature');
-
-  const deps = resolveByName('deps');
-  assert.ok(deps !== null, 'Should resolve "deps" alias');
-  assert.deepStrictEqual(deps!.id, 'dep-upgrade', 'Alias "deps" should map to dep-upgrade');
-
-  // No match
-  const missing = resolveByName('nonexistent-template');
-  assert.ok(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');
-  assert.ok(fixMatches.length > 0, 'Should detect matches for "fix the login button"');
-  assert.ok(fixMatches.some(m => m.id === 'bugfix'), 'Should include bugfix in matches');
-
-  // Should detect spike from "research" keyword
-  const researchMatches = autoDetect('research authentication libraries');
-  assert.ok(researchMatches.length > 0, 'Should detect matches for "research"');
-  assert.ok(researchMatches.some(m => m.id === 'spike'), 'Should include spike in matches');
-
-  // Should detect hotfix from "urgent" keyword
-  const urgentMatches = autoDetect('urgent production is down');
-  assert.ok(urgentMatches.length > 0, 'Should detect matches for "urgent"');
-  assert.ok(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');
-  assert.ok(upgradeMatches.length > 0, 'Should detect matches for "upgrade"');
-  assert.ok(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');
-  assert.ok(projectMatch !== undefined, 'Should detect full-project for "from scratch"');
-
-  // Empty input should return no matches
-  const emptyMatches = autoDetect('');
-  assert.deepStrictEqual(emptyMatches.length, 0, 'Empty input should return no matches');
-}
-
-// ═══════════════════════════════════════════════════════════════════════════
-// List Templates
-// ═══════════════════════════════════════════════════════════════════════════
-
-console.log('\n── List Templates ──');
-
-{
-  const output = listTemplates();
-  assert.ok(output.includes('Workflow Templates'), 'Should have header');
-  assert.ok(output.includes('bugfix'), 'Should list bugfix');
-  assert.ok(output.includes('spike'), 'Should list spike');
-  assert.ok(output.includes('hotfix'), 'Should list hotfix');
-  assert.ok(output.includes('/gsd start'), 'Should include usage hint');
-}
-
-// ═══════════════════════════════════════════════════════════════════════════
-// Template Info
-// ═══════════════════════════════════════════════════════════════════════════
-
-console.log('\n── Template Info ──');
-
-{
-  const info = getTemplateInfo('bugfix');
-  assert.ok(info !== null, 'Should return info for bugfix');
-  assert.ok(info!.includes('Bug Fix'), 'Should include template name');
-  assert.ok(info!.includes('triage'), 'Should include phase names');
-  assert.ok(info!.includes('Triggers'), 'Should include triggers section');
-
-  const missing = getTemplateInfo('nonexistent');
-  assert.ok(missing === null, 'Should return null for unknown template');
-}
-
-// ═══════════════════════════════════════════════════════════════════════════
-// Load Workflow Template Content
-// ═══════════════════════════════════════════════════════════════════════════
-
-console.log('\n── Load Workflow Template ──');
-
-{
-  const content = loadWorkflowTemplate('bugfix');
-  assert.ok(content !== null, 'Should load bugfix template');
-  assert.ok(content!.includes('Bugfix Workflow'), 'Should contain workflow title');
-  assert.ok(content!.includes('Phase 1: Triage'), 'Should contain triage phase');
-  assert.ok(content!.includes('Phase 4: Ship'), 'Should contain ship phase');
-
-  const hotfixContent = loadWorkflowTemplate('hotfix');
-  assert.ok(hotfixContent !== null, 'Should load hotfix template');
-  assert.ok(hotfixContent!.includes('Hotfix Workflow'), 'Should contain hotfix title');
-
-  const missingContent = loadWorkflowTemplate('nonexistent');
-  assert.ok(missingContent === null, 'Should return null for unknown template');
-}
-
-// ═══════════════════════════════════════════════════════════════════════════

package/src/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/src/resources/extensions/gsd/tests/worktree-bugfix.test.ts

@@ -1,117 +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 assert from 'node:assert/strict';
-
-import { resolveGitDir } from "../worktree-manager.ts";
-import { detectWorktreeName, captureIntegrationBranch } from "../worktree.ts";
-
-// ─── 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 });
-  });
-
-  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);
-    assert.ok(gitDir.endsWith(".git"), "ends with .git");
-    assert.ok(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);
-    assert.deepStrictEqual(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);
-    assert.ok(gitDir.endsWith(".git"), "returns default .git path");
-  });
-
-  it("detectWorktreeName returns name for worktree path", () => {
-    assert.deepStrictEqual(
-      detectWorktreeName("/project/.gsd/worktrees/M005"),
-      "M005",
-      "detects worktree name",
-    );
-  });
-
-  it("detectWorktreeName returns null for normal repo", () => {
-    assert.deepStrictEqual(
-      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");
-    assert.ok(!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");
-    assert.ok(inWorktree !== null, "worktree detected → skip pull");
-    assert.ok(inNormal === null, "normal repo → allow pull");
-  });
-});