@nathapp/nax 0.28.0 → 0.29.0

package/test/unit/precheck-story-size-gate.test.ts

@@ -1,288 +0,0 @@
-// RE-ARCH: keep
-/**
- * Tests for src/precheck/story-size-gate.ts
- *
- * Tests story size gate detection logic including heuristic signals
- * (AC count, description length, bullet points).
- */
-
-import { describe, expect, test } from "bun:test";
-import type { NaxConfig } from "../../src/config";
-import type { PRD, UserStory } from "../../src/prd/types";
-import { checkStorySizeGate } from "../../src/precheck/story-size-gate";
-
-// ─────────────────────────────────────────────────────────────────────────────
-// Test fixtures
-// ─────────────────────────────────────────────────────────────────────────────
-
-const createMockConfig = (
-  storySizeGateConfig?: NaxConfig["precheck"],
-): NaxConfig =>
-  ({
-    precheck: storySizeGateConfig,
-    version: 1,
-    models: {
-      fast: "haiku",
-      balanced: "sonnet",
-      powerful: "opus",
-    },
-    autoMode: {
-      enabled: false,
-      defaultAgent: "test",
-      fallbackOrder: [],
-      complexityRouting: { simple: "fast", medium: "balanced", complex: "powerful", expert: "powerful" },
-      escalation: { enabled: false, tierOrder: [] },
-    },
-    routing: { strategy: "keyword" },
-    execution: {
-      maxIterations: 10,
-      iterationDelayMs: 1000,
-      costLimit: 10,
-      sessionTimeoutSeconds: 600,
-      verificationTimeoutSeconds: 300,
-      maxStoriesPerFeature: 100,
-      rectification: {
-        enabled: false,
-        maxRetries: 0,
-        fullSuiteTimeoutSeconds: 120,
-        maxFailureSummaryChars: 2000,
-        abortOnIncreasingFailures: true,
-      },
-      regressionGate: { enabled: false, timeoutSeconds: 120 },
-      contextProviderTokenBudget: 2000,
-    },
-    quality: {
-      requireTypecheck: false,
-      requireLint: false,
-      requireTests: false,
-      commands: {},
-      forceExit: false,
-      detectOpenHandles: false,
-      detectOpenHandlesRetries: 1,
-      gracePeriodMs: 5000,
-      drainTimeoutMs: 2000,
-      shell: "/bin/sh",
-      stripEnvVars: [],
-      environmentalEscalationDivisor: 2,
-    },
-    tdd: {
-      maxRetries: 0,
-      autoVerifyIsolation: false,
-      autoApproveVerifier: false,
-      strategy: "off",
-    },
-    constitution: { enabled: false, path: "", maxTokens: 2000 },
-    analyze: { llmEnhanced: false, model: "fast", fallbackToKeywords: true, maxCodebaseSummaryTokens: 5000 },
-    review: { enabled: false, checks: [], commands: {} },
-    plan: { model: "balanced", outputPath: "" },
-    acceptance: { enabled: false, maxRetries: 0, generateTests: false, testPath: "" },
-    context: {
-      testCoverage: {
-        enabled: false,
-        detail: "names-only",
-        maxTokens: 500,
-        testPattern: "**/*.test.ts",
-        scopeToStory: false,
-      },
-      autoDetect: { enabled: false, maxFiles: 5, traceImports: false },
-    },
-  }) as NaxConfig;
-
-const createMockStory = (overrides: Partial<UserStory> = {}): UserStory => ({
-  id: "US-001",
-  title: "Test story",
-  description: "Test description",
-  acceptanceCriteria: ["AC1"],
-  tags: [],
-  dependencies: [],
-  status: "pending",
-  passes: false,
-  escalations: [],
-  attempts: 0,
-  ...overrides,
-});
-
-const createMockPRD = (stories: UserStory[] = []): PRD => ({
-  project: "test-project",
-  feature: "test-feature",
-  branchName: "test-branch",
-  userStories: stories,
-  createdAt: new Date().toISOString(),
-  updatedAt: new Date().toISOString(),
-});
-
-// ─────────────────────────────────────────────────────────────────────────────
-// Tests
-// ─────────────────────────────────────────────────────────────────────────────
-
-describe("checkStorySizeGate", () => {
-  test("passes when gate is disabled", async () => {
-    const config = createMockConfig({ storySizeGate: { enabled: false, maxAcCount: 6, maxDescriptionLength: 2000, maxBulletPoints: 8 } });
-    const story = createMockStory({ acceptanceCriteria: Array(10).fill("AC") });
-    const prd = createMockPRD([story]);
-
-    const result = await checkStorySizeGate(config, prd);
-
-    expect(result.check.passed).toBe(true);
-    expect(result.check.tier).toBe("warning");
-    expect(result.check.message).toContain("disabled");
-    expect(result.flaggedStories).toHaveLength(0);
-  });
-
-  test("passes when no stories exceed thresholds", async () => {
-    const config = createMockConfig({ storySizeGate: { enabled: true, maxAcCount: 6, maxDescriptionLength: 2000, maxBulletPoints: 8 } });
-    const story = createMockStory({
-      acceptanceCriteria: ["AC1", "AC2", "AC3"],
-      description: "Short description",
-    });
-    const prd = createMockPRD([story]);
-
-    const result = await checkStorySizeGate(config, prd);
-
-    expect(result.check.passed).toBe(true);
-    expect(result.check.tier).toBe("warning");
-    expect(result.flaggedStories).toHaveLength(0);
-  });
-
-  test("flags story when AC count exceeds threshold", async () => {
-    const config = createMockConfig({ storySizeGate: { enabled: true, maxAcCount: 6, maxDescriptionLength: 2000, maxBulletPoints: 8 } });
-    const story = createMockStory({
-      id: "US-001",
-      acceptanceCriteria: Array(8).fill("AC"),
-      description: "Short description",
-    });
-    const prd = createMockPRD([story]);
-
-    const result = await checkStorySizeGate(config, prd);
-
-    expect(result.check.passed).toBe(false);
-    expect(result.check.tier).toBe("warning");
-    expect(result.check.message).toContain("US-001");
-    expect(result.flaggedStories).toHaveLength(1);
-    expect(result.flaggedStories[0].storyId).toBe("US-001");
-    expect(result.flaggedStories[0].signals.acCount.flagged).toBe(true);
-    expect(result.flaggedStories[0].signals.acCount.value).toBe(8);
-  });
-
-  test("flags story when description length exceeds threshold", async () => {
-    const config = createMockConfig({ storySizeGate: { enabled: true, maxAcCount: 6, maxDescriptionLength: 100, maxBulletPoints: 8 } });
-    const longDescription = "a".repeat(150);
-    const story = createMockStory({
-      id: "US-002",
-      acceptanceCriteria: ["AC1"],
-      description: longDescription,
-    });
-    const prd = createMockPRD([story]);
-
-    const result = await checkStorySizeGate(config, prd);
-
-    expect(result.check.passed).toBe(false);
-    expect(result.flaggedStories).toHaveLength(1);
-    expect(result.flaggedStories[0].signals.descriptionLength.flagged).toBe(true);
-    expect(result.flaggedStories[0].signals.descriptionLength.value).toBe(150);
-  });
-
-  test("flags story when bullet point count exceeds threshold", async () => {
-    const config = createMockConfig({ storySizeGate: { enabled: true, maxAcCount: 6, maxDescriptionLength: 2000, maxBulletPoints: 5 } });
-    const description = `
-Requirements:
-- Item 1
-- Item 2
-- Item 3
-- Item 4
-- Item 5
-- Item 6
-- Item 7
-`;
-    const story = createMockStory({
-      id: "US-003",
-      acceptanceCriteria: ["AC1"],
-      description,
-    });
-    const prd = createMockPRD([story]);
-
-    const result = await checkStorySizeGate(config, prd);
-
-    expect(result.check.passed).toBe(false);
-    expect(result.flaggedStories).toHaveLength(1);
-    expect(result.flaggedStories[0].signals.bulletPoints.flagged).toBe(true);
-    expect(result.flaggedStories[0].signals.bulletPoints.value).toBeGreaterThan(5);
-  });
-
-  test("flags multiple stories when multiple exceed thresholds", async () => {
-    const config = createMockConfig({ storySizeGate: { enabled: true, maxAcCount: 3, maxDescriptionLength: 50, maxBulletPoints: 5 } });
-    const story1 = createMockStory({
-      id: "US-001",
-      acceptanceCriteria: Array(5).fill("AC"),
-    });
-    const story2 = createMockStory({
-      id: "US-002",
-      description: "a".repeat(100),
-    });
-    const prd = createMockPRD([story1, story2]);
-
-    const result = await checkStorySizeGate(config, prd);
-
-    expect(result.check.passed).toBe(false);
-    expect(result.flaggedStories).toHaveLength(2);
-    expect(result.flaggedStories[0].storyId).toBe("US-001");
-    expect(result.flaggedStories[1].storyId).toBe("US-002");
-  });
-
-  test("only checks pending stories, ignores completed/failed/skipped", async () => {
-    const config = createMockConfig({ storySizeGate: { enabled: true, maxAcCount: 3, maxDescriptionLength: 2000, maxBulletPoints: 8 } });
-    const pendingStory = createMockStory({
-      id: "US-001",
-      status: "pending",
-      acceptanceCriteria: Array(5).fill("AC"),
-    });
-    const passedStory = createMockStory({
-      id: "US-002",
-      status: "passed",
-      acceptanceCriteria: Array(5).fill("AC"),
-    });
-    const failedStory = createMockStory({
-      id: "US-003",
-      status: "failed",
-      acceptanceCriteria: Array(5).fill("AC"),
-    });
-    const prd = createMockPRD([pendingStory, passedStory, failedStory]);
-
-    const result = await checkStorySizeGate(config, prd);
-
-    expect(result.check.passed).toBe(false);
-    expect(result.flaggedStories).toHaveLength(1);
-    expect(result.flaggedStories[0].storyId).toBe("US-001");
-  });
-
-  test("uses default thresholds when config is not provided", async () => {
-    const config = createMockConfig(undefined);
-    const story = createMockStory({
-      id: "US-001",
-      acceptanceCriteria: Array(10).fill("AC"),
-    });
-    const prd = createMockPRD([story]);
-
-    const result = await checkStorySizeGate(config, prd);
-
-    // Default threshold is 6, so 10 AC should be flagged
-    expect(result.check.passed).toBe(false);
-    expect(result.flaggedStories).toHaveLength(1);
-  });
-
-  test("includes recommendation message in flagged story", async () => {
-    const config = createMockConfig({ storySizeGate: { enabled: true, maxAcCount: 3, maxDescriptionLength: 2000, maxBulletPoints: 8 } });
-    const story = createMockStory({
-      id: "US-001",
-      acceptanceCriteria: Array(5).fill("AC"),
-    });
-    const prd = createMockPRD([story]);
-
-    const result = await checkStorySizeGate(config, prd);
-
-    expect(result.flaggedStories[0].recommendation).toContain("US-001");
-    expect(result.flaggedStories[0].recommendation).toContain("too large");
-    expect(result.flaggedStories[0].recommendation).toContain("5 AC");
-    expect(result.flaggedStories[0].recommendation).toContain("max 3");
-  });
-});

package/test/unit/precheck-types.test.ts

@@ -1,143 +0,0 @@
-// RE-ARCH: keep
-/**
- * Tests for src/precheck/types.ts
- *
- * Tests the precheck type definitions including PrecheckResult, CheckStatus, and CheckTier.
- */
-
-import { describe, expect, test } from "bun:test";
-import type { Check, CheckStatus, CheckTier, PrecheckResult } from "../../src/precheck/types";
-
-describe("PrecheckResult type structure", () => {
-  test("PrecheckResult has blockers array", () => {
-    const result: PrecheckResult = {
-      blockers: [],
-      warnings: [],
-    };
-
-    expect(result.blockers).toBeDefined();
-    expect(Array.isArray(result.blockers)).toBe(true);
-  });
-
-  test("PrecheckResult has warnings array", () => {
-    const result: PrecheckResult = {
-      blockers: [],
-      warnings: [],
-    };
-
-    expect(result.warnings).toBeDefined();
-    expect(Array.isArray(result.warnings)).toBe(true);
-  });
-
-  test("PrecheckResult can contain Check objects in blockers", () => {
-    const check: Check = {
-      name: "git-repo-exists",
-      tier: "blocker",
-      passed: false,
-      message: "Not a git repository",
-    };
-
-    const result: PrecheckResult = {
-      blockers: [check],
-      warnings: [],
-    };
-
-    expect(result.blockers).toHaveLength(1);
-    expect(result.blockers[0].name).toBe("git-repo-exists");
-    expect(result.blockers[0].tier).toBe("blocker");
-    expect(result.blockers[0].passed).toBe(false);
-  });
-
-  test("PrecheckResult can contain Check objects in warnings", () => {
-    const check: Check = {
-      name: "claude-md-exists",
-      tier: "warning",
-      passed: false,
-      message: "CLAUDE.md not found",
-    };
-
-    const result: PrecheckResult = {
-      blockers: [],
-      warnings: [check],
-    };
-
-    expect(result.warnings).toHaveLength(1);
-    expect(result.warnings[0].name).toBe("claude-md-exists");
-    expect(result.warnings[0].tier).toBe("warning");
-  });
-});
-
-describe("Check type structure", () => {
-  test("Check has required name field", () => {
-    const check: Check = {
-      name: "test-check",
-      tier: "blocker",
-      passed: true,
-      message: "Check passed",
-    };
-
-    expect(check.name).toBe("test-check");
-  });
-
-  test("Check has required tier field", () => {
-    const check: Check = {
-      name: "test-check",
-      tier: "blocker",
-      passed: true,
-      message: "Check passed",
-    };
-
-    expect(check.tier).toBe("blocker");
-  });
-
-  test("Check has required passed field", () => {
-    const check: Check = {
-      name: "test-check",
-      tier: "blocker",
-      passed: false,
-      message: "Check failed",
-    };
-
-    expect(check.passed).toBe(false);
-  });
-
-  test("Check has required message field", () => {
-    const check: Check = {
-      name: "test-check",
-      tier: "blocker",
-      passed: true,
-      message: "All good",
-    };
-
-    expect(check.message).toBe("All good");
-  });
-});
-
-describe("CheckTier type values", () => {
-  test("CheckTier accepts blocker value", () => {
-    const tier: CheckTier = "blocker";
-    expect(tier).toBe("blocker");
-  });
-
-  test("CheckTier accepts warning value", () => {
-    const tier: CheckTier = "warning";
-    expect(tier).toBe("warning");
-  });
-});
-
-describe("CheckStatus type values", () => {
-  test("CheckStatus accepts passed value", () => {
-    const status: CheckStatus = "passed";
-    expect(status).toBe("passed");
-  });
-
-  test("CheckStatus accepts failed value", () => {
-    const status: CheckStatus = "failed";
-    expect(status).toBe("failed");
-  });
-
-  test("CheckStatus accepts skipped value", () => {
-    const status: CheckStatus = "skipped";
-    expect(status).toBe("skipped");
-  });
-});

package/test/unit/prompts/builder.test.ts

@@ -1,258 +0,0 @@
-/**
- * PromptBuilder unit tests — PB-001
- *
- * Tests verify section ordering, non-overridable sections, and override fallthrough.
- * All tests are expected to FAIL until PromptBuilder is implemented.
- */
-
-import { describe, expect, test } from "bun:test";
-import { mkdtempSync, writeFileSync } from "node:fs";
-import { tmpdir } from "node:os";
-import { join } from "node:path";
-import type { UserStory } from "../../../src/prd";
-import { PromptBuilder } from "../../../src/prompts/builder";
-import type { PromptRole } from "../../../src/prompts/types";
-
-// ---------------------------------------------------------------------------
-// Fixtures
-// ---------------------------------------------------------------------------
-
-function makeStory(overrides: Partial<UserStory> = {}): UserStory {
-  return {
-    id: "US-001",
-    title: "Example story",
-    description: "Do the thing",
-    acceptanceCriteria: ["Criterion 1", "Criterion 2"],
-    tags: [],
-    dependencies: [],
-    status: "pending",
-    passes: false,
-    escalations: [],
-    ...overrides,
-  };
-}
-
-const ROLES: PromptRole[] = ["test-writer", "implementer", "verifier", "single-session"];
-
-// ---------------------------------------------------------------------------
-// 1. Fluent API — builder returns itself for chaining
-// ---------------------------------------------------------------------------
-
-describe("PromptBuilder fluent API", () => {
-  test("PromptBuilder.for() returns a PromptBuilder instance", () => {
-    const builder = PromptBuilder.for("test-writer");
-    expect(builder).toBeInstanceOf(PromptBuilder);
-  });
-
-  test(".story() is chainable", () => {
-    const builder = PromptBuilder.for("implementer").story(makeStory());
-    expect(builder).toBeInstanceOf(PromptBuilder);
-  });
-
-  test(".context() is chainable", () => {
-    const builder = PromptBuilder.for("verifier").story(makeStory()).context("# Context");
-    expect(builder).toBeInstanceOf(PromptBuilder);
-  });
-
-  test(".constitution() is chainable", () => {
-    const builder = PromptBuilder.for("single-session").story(makeStory()).constitution("Be helpful.");
-    expect(builder).toBeInstanceOf(PromptBuilder);
-  });
-
-  test(".override() is chainable", () => {
-    const builder = PromptBuilder.for("test-writer").story(makeStory()).override("/tmp/override.md");
-    expect(builder).toBeInstanceOf(PromptBuilder);
-  });
-
-  test(".build() returns a Promise<string>", async () => {
-    const result = PromptBuilder.for("test-writer").story(makeStory()).build();
-    expect(result).toBeInstanceOf(Promise);
-    const text = await result;
-    expect(typeof text).toBe("string");
-  });
-});
-
-// ---------------------------------------------------------------------------
-// 2. Section ordering
-// ---------------------------------------------------------------------------
-
-describe("PromptBuilder section order", () => {
-  test("constitution appears before role task", async () => {
-    const prompt = await PromptBuilder.for("test-writer")
-      .story(makeStory())
-      .constitution("CONSTITUTION_MARKER")
-      .build();
-
-    const constitutionIdx = prompt.indexOf("CONSTITUTION_MARKER");
-    const roleTaskIdx = prompt.indexOf("YOUR TASK") !== -1 ? prompt.indexOf("YOUR TASK") : prompt.indexOf("Your role");
-
-    expect(constitutionIdx).toBeGreaterThanOrEqual(0);
-    expect(roleTaskIdx).toBeGreaterThanOrEqual(0);
-    expect(constitutionIdx).toBeLessThan(roleTaskIdx);
-  });
-
-  test("story context appears before conventions footer", async () => {
-    const story = makeStory({ title: "STORY_TITLE_MARKER" });
-    const prompt = await PromptBuilder.for("implementer").story(story).build();
-
-    const storyIdx = prompt.indexOf("STORY_TITLE_MARKER");
-    // Conventions footer is always last — it contains "conventions" or appears after story
-    const footerIdx = prompt.lastIndexOf("conventions") !== -1 ? prompt.lastIndexOf("conventions") : prompt.length - 1;
-
-    expect(storyIdx).toBeGreaterThanOrEqual(0);
-    expect(storyIdx).toBeLessThan(footerIdx);
-  });
-
-  test("isolation rules appear after role task body", async () => {
-    const prompt = await PromptBuilder.for("test-writer").story(makeStory()).build();
-
-    // Isolation rules section — comes after the main role task body
-    const isolationIdx = prompt.indexOf("isolation") !== -1 ? prompt.indexOf("isolation") : prompt.indexOf("ISOLATION");
-
-    expect(isolationIdx).toBeGreaterThanOrEqual(0);
-  });
-
-  test("context markdown appears before conventions footer", async () => {
-    const ctxMarker = "CONTEXT_MARKDOWN_MARKER";
-    const prompt = await PromptBuilder.for("verifier").story(makeStory()).context(ctxMarker).build();
-
-    const ctxIdx = prompt.indexOf(ctxMarker);
-    const footerIdx = prompt.lastIndexOf("conventions") !== -1 ? prompt.lastIndexOf("conventions") : prompt.length;
-
-    expect(ctxIdx).toBeGreaterThanOrEqual(0);
-    expect(ctxIdx).toBeLessThan(footerIdx);
-  });
-
-  describe("section order for each role", () => {
-    for (const role of ROLES) {
-      test(`role ${role}: constitution → role task → story context → isolation rules → context md → conventions footer`, async () => {
-        const story = makeStory({ title: `STORY_${role.toUpperCase()}` });
-        const ctxMd = `CTXMD_${role.toUpperCase()}`;
-        const prompt = await PromptBuilder.for(role)
-          .story(story)
-          .constitution("CONSTITUTION_BLOCK")
-          .context(ctxMd)
-          .build();
-
-        const constitutionIdx = prompt.indexOf("CONSTITUTION_BLOCK");
-        const storyIdx = prompt.indexOf(`STORY_${role.toUpperCase()}`);
-        const ctxIdx = prompt.indexOf(ctxMd);
-
-        // All markers present
-        expect(constitutionIdx).toBeGreaterThanOrEqual(0);
-        expect(storyIdx).toBeGreaterThanOrEqual(0);
-        expect(ctxIdx).toBeGreaterThanOrEqual(0);
-
-        // Ordering: constitution < story < ctx
-        expect(constitutionIdx).toBeLessThan(storyIdx);
-        expect(storyIdx).toBeLessThan(ctxIdx);
-      });
-    }
-  });
-});
-
-// ---------------------------------------------------------------------------
-// 3. Non-overridable sections always present
-// ---------------------------------------------------------------------------
-
-describe("PromptBuilder non-overridable sections", () => {
-  test("story context always included even when override is set", async () => {
-    const tmpDir = mkdtempSync(join(tmpdir(), "nax-pb-test-"));
-    const overridePath = join(tmpDir, "override.md");
-    writeFileSync(overridePath, "# Custom override body\nThis replaces the template.");
-
-    const story = makeStory({ title: "NON_OVERRIDABLE_STORY" });
-    const prompt = await PromptBuilder.for("test-writer").story(story).override(overridePath).build();
-
-    expect(prompt).toContain("NON_OVERRIDABLE_STORY");
-  });
-
-  test("conventions footer always last even when override is set", async () => {
-    const tmpDir = mkdtempSync(join(tmpdir(), "nax-pb-test-"));
-    const overridePath = join(tmpDir, "override.md");
-    writeFileSync(overridePath, "# Custom override body");
-
-    const prompt = await PromptBuilder.for("implementer").story(makeStory()).override(overridePath).build();
-
-    // Conventions footer must exist and be after the override content
-    const overrideIdx = prompt.indexOf("Custom override body");
-    const conventionsIdx = prompt.lastIndexOf("conventions");
-    expect(conventionsIdx).toBeGreaterThan(overrideIdx);
-  });
-
-  test("isolation rules always present even when override is set", async () => {
-    const tmpDir = mkdtempSync(join(tmpdir(), "nax-pb-test-"));
-    const overridePath = join(tmpDir, "override.md");
-    writeFileSync(overridePath, "# My custom template");
-
-    const prompt = await PromptBuilder.for("test-writer").story(makeStory()).override(overridePath).build();
-
-    // Isolation rules must appear somewhere in the final prompt
-    const lowerPrompt = prompt.toLowerCase();
-    const hasIsolation = lowerPrompt.includes("isolation") || lowerPrompt.includes("isolat");
-    expect(hasIsolation).toBe(true);
-  });
-
-  test("story context not removable via override for each role", async () => {
-    const tmpDir = mkdtempSync(join(tmpdir(), "nax-pb-test-"));
-    const overridePath = join(tmpDir, "override.md");
-    writeFileSync(overridePath, "Override that attempts to hide story context.");
-
-    for (const role of ROLES) {
-      const story = makeStory({ title: `ROLE_${role}_TITLE` });
-      const prompt = await PromptBuilder.for(role).story(story).override(overridePath).build();
-      expect(prompt).toContain(`ROLE_${role}_TITLE`);
-    }
-  });
-});
-
-// ---------------------------------------------------------------------------
-// 4. Override fallthrough to default template
-// ---------------------------------------------------------------------------
-
-describe("PromptBuilder override fallthrough", () => {
-  test("missing override file falls through to default template", async () => {
-    const prompt = await PromptBuilder.for("test-writer")
-      .story(makeStory({ title: "FALLTHROUGH_STORY" }))
-      .override("/nonexistent/path/override.md")
-      .build();
-
-    // Should still contain story context (non-overridable)
-    expect(prompt).toContain("FALLTHROUGH_STORY");
-    // Should contain default role task content (not crash)
-    expect(typeof prompt).toBe("string");
-    expect(prompt.length).toBeGreaterThan(0);
-  });
-
-  test("no override set uses default template body", async () => {
-    const promptWithout = await PromptBuilder.for("test-writer")
-      .story(makeStory({ title: "DEFAULT_TEMPLATE_STORY" }))
-      .build();
-
-    expect(promptWithout).toContain("DEFAULT_TEMPLATE_STORY");
-    expect(promptWithout.length).toBeGreaterThan(0);
-  });
-
-  test("valid override file replaces default template body", async () => {
-    const tmpDir = mkdtempSync(join(tmpdir(), "nax-pb-test-"));
-    const overridePath = join(tmpDir, "override.md");
-    const overrideBody = "UNIQUE_OVERRIDE_BODY_CONTENT";
-    writeFileSync(overridePath, overrideBody);
-
-    const prompt = await PromptBuilder.for("implementer").story(makeStory()).override(overridePath).build();
-
-    expect(prompt).toContain(overrideBody);
-  });
-});
-
-// ---------------------------------------------------------------------------
-// 5. Types exported correctly
-// ---------------------------------------------------------------------------
-
-describe("src/prompts/types exports", () => {
-  test("PromptRole values are correct literals", () => {
-    // This is a compile-time check — if types.ts exports correctly, import works
-    const roles: PromptRole[] = ["test-writer", "implementer", "verifier", "single-session"];
-    expect(roles).toHaveLength(4);
-  });
-});