gsd-pi 2.33.1-dev.29a8268 → 2.33.1-dev.bf822e6

package/dist/resources/extensions/gsd/tests/token-optimization-prefs.test.ts

@@ -1,164 +0,0 @@
-import { describe, it } from "node:test";
-import assert from "node:assert/strict";
-
-// Test the type definitions exist and are correct
-describe("token-optimization: types", () => {
-  it("CompressionStrategy accepts valid values", async () => {
-    const { } = await import("../types.js");
-    // Type-level test — if this compiles, the types exist
-    const truncate: import("../types.js").CompressionStrategy = "truncate";
-    const compress: import("../types.js").CompressionStrategy = "compress";
-    assert.equal(truncate, "truncate");
-    assert.equal(compress, "compress");
-  });
-
-  it("ContextSelectionMode accepts valid values", async () => {
-    const full: import("../types.js").ContextSelectionMode = "full";
-    const smart: import("../types.js").ContextSelectionMode = "smart";
-    assert.equal(full, "full");
-    assert.equal(smart, "smart");
-  });
-});
-
-// Test cache hit rate computation
-describe("token-optimization: cache hit rate", () => {
-  it("computeCacheHitRate returns correct percentage", async () => {
-    const { computeCacheHitRate } = await import("../prompt-cache-optimizer.js");
-    assert.equal(computeCacheHitRate({ cacheRead: 900, cacheWrite: 100, input: 100 }), 90);
-    assert.equal(computeCacheHitRate({ cacheRead: 0, cacheWrite: 0, input: 100 }), 0);
-    assert.equal(computeCacheHitRate({ cacheRead: 0, cacheWrite: 0, input: 0 }), 0);
-    assert.equal(computeCacheHitRate({ cacheRead: 500, cacheWrite: 0, input: 500 }), 50);
-  });
-});
-
-// Test structured data savings
-describe("token-optimization: structured data savings", () => {
-  it("compact decisions format is shorter than markdown table", async () => {
-    const { formatDecisionsCompact, measureSavings } = await import("../structured-data-formatter.js");
-    const decisions = [
-      { id: "D001", when_context: "M001/S01", scope: "architecture", decision: "Use SQLite for storage", choice: "WAL mode", rationale: "Built-in, no external deps", revisable: "yes" },
-      { id: "D002", when_context: "M001/S02", scope: "testing", decision: "Unit test all parsers", choice: "node:test", rationale: "Fast, zero-dependency", revisable: "no" },
-    ];
-    const compact = formatDecisionsCompact(decisions);
-    // A realistic markdown table equivalent
-    const markdown = [
-      "| # | When | Scope | Decision | Choice | Rationale | Revisable? |",
-      "|---|------|-------|----------|--------|-----------|------------|",
-      "| D001 | M001/S01 | architecture | Use SQLite for storage | WAL mode | Built-in, no external deps | yes |",
-      "| D002 | M001/S02 | testing | Unit test all parsers | node:test | Fast, zero-dependency | no |",
-    ].join("\n");
-    const savings = measureSavings(compact, markdown);
-    assert.ok(savings > 10, `Expected >10% savings, got ${savings}%`);
-  });
-
-  it("compact requirements format drops low-value fields", async () => {
-    const { formatRequirementsCompact } = await import("../structured-data-formatter.js");
-    const requirements = [{
-      id: "R001", class: "functional", status: "active",
-      description: "API response time < 200ms",
-      why: "User experience", primary_owner: "S01",
-      validation: "Load test P99 < 200ms",
-    }];
-    const compact = formatRequirementsCompact(requirements);
-    assert.ok(!compact.includes("source"), "Should not include source field");
-    assert.ok(!compact.includes("supporting_slices"), "Should not include supporting_slices");
-    assert.ok(compact.includes("R001"), "Should include requirement ID");
-  });
-});
-
-// Test compression levels
-describe("token-optimization: prompt compression", () => {
-  it("light compression removes extra whitespace", async () => {
-    const { compressPrompt } = await import("../prompt-compressor.js");
-    const input = "Line 1\n\n\n\n\nLine 2\n\n\n\nLine 3";
-    const result = compressPrompt(input, { level: "light" });
-    assert.ok(result.savingsPercent > 0, "Should have positive savings");
-    assert.ok(!result.content.includes("\n\n\n"), "Should collapse multiple blank lines");
-  });
-
-  it("moderate compression abbreviates verbose phrases", async () => {
-    const { compressPrompt } = await import("../prompt-compressor.js");
-    const input = "In order to achieve this, it is important to note that the following steps are required.";
-    const result = compressPrompt(input, { level: "moderate" });
-    assert.ok(result.compressedChars < result.originalChars, "Should be shorter");
-  });
-
-  it("code blocks are preserved during compression", async () => {
-    const { compressPrompt } = await import("../prompt-compressor.js");
-    const input = "In order to do this:\n\n```typescript\nconst x = 1;\n```\n\nIn order to verify:";
-    const result = compressPrompt(input, { level: "aggressive" });
-    assert.ok(result.content.includes("const x = 1;"), "Code block should be preserved");
-  });
-});
-
-// Test summary distillation
-describe("token-optimization: summary distillation", () => {
-  it("distills summaries preserving key fields", async () => {
-    const { distillSummaries } = await import("../summary-distiller.js");
-    const summary = `---
-id: S01
-provides:
-  - Core types
-key_files:
-  - src/types.ts
-key_decisions:
-  - D001
----
-
-# S01: Core Types
-
-Built the foundation type system.
-
-## What Happened
-
-Long prose about implementation details that should be dropped...
-`;
-    const result = distillSummaries([summary], 5000);
-    assert.ok(result.savingsPercent > 0, "Should have savings");
-    assert.ok(result.content.includes("Core types"), "Should preserve provides");
-    assert.ok(result.content.includes("src/types.ts"), "Should preserve key_files");
-  });
-});
-
-// Test semantic chunker
-describe("token-optimization: semantic chunking", () => {
-  it("chunks TypeScript code at function boundaries", async () => {
-    const { splitIntoChunks } = await import("../semantic-chunker.js");
-    const code = `export function alpha() {
-  return 1;
-}
-
-export function beta() {
-  return 2;
-}
-
-export function gamma() {
-  return 3;
-}`;
-    const chunks = splitIntoChunks(code);
-    assert.ok(chunks.length >= 2, `Expected >=2 chunks, got ${chunks.length}`);
-  });
-
-  it("scores chunks by relevance to query", async () => {
-    const { chunkByRelevance } = await import("../semantic-chunker.js");
-    const code = `export function createUser(name: string) {
-  return { name, id: generateId() };
-}
-
-export function deleteDatabase() {
-  dropAllTables();
-  clearCache();
-}
-
-export function updateUser(id: string, name: string) {
-  const user = findUser(id);
-  user.name = name;
-  return user;
-}`;
-    const result = chunkByRelevance(code, "user creation and management", { maxChunks: 2 });
-    // The user-related chunks should score higher
-    const content = result.chunks.map(c => c.content).join("\n");
-    assert.ok(content.includes("createUser") || content.includes("updateUser"),
-      "Should include user-related chunks");
-  });
-});

package/dist/resources/extensions/gsd/tests/token-profile.test.ts

@@ -1,268 +0,0 @@
-/**
- * Token Profile — unit tests for M004/S01.
- *
- * Tests profile resolution, preference merging, phase skip defaults,
- * subagent model routing, default-to-balanced behavior, and dispatch
- * table guard clauses (source-level structural verification).
- *
- * Uses source-level checks (readFileSync + string matching) to avoid
- * @gsd/pi-coding-agent import resolution issues in dev environments.
- */
-
-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));
-
-// ─── Source files for structural checks ───────────────────────────────────
-// After decomposition, code lives across multiple modules. Concatenate them
-// so structural string-matching works regardless of which file holds the code.
-
-const dispatchSrc = readFileSync(join(__dirname, "..", "auto-dispatch.ts"), "utf-8");
-const preferencesSrc = [
-  readFileSync(join(__dirname, "..", "preferences.ts"), "utf-8"),
-  readFileSync(join(__dirname, "..", "preferences-types.ts"), "utf-8"),
-  readFileSync(join(__dirname, "..", "preferences-models.ts"), "utf-8"),
-  readFileSync(join(__dirname, "..", "preferences-validation.ts"), "utf-8"),
-].join("\n");
-const typesSrc = readFileSync(join(__dirname, "..", "types.ts"), "utf-8");
-
-// ═══════════════════════════════════════════════════════════════════════════
-// Type Definitions
-// ═══════════════════════════════════════════════════════════════════════════
-
-test("types: TokenProfile type exported with budget/balanced/quality", () => {
-  assert.ok(typesSrc.includes("export type TokenProfile"), "TokenProfile should be exported");
-  assert.ok(typesSrc.includes("'budget'"), "should include budget");
-  assert.ok(typesSrc.includes("'balanced'"), "should include balanced");
-  assert.ok(typesSrc.includes("'quality'"), "should include quality");
-});
-
-test("types: InlineLevel type exported with full/standard/minimal", () => {
-  assert.ok(typesSrc.includes("export type InlineLevel"), "InlineLevel should be exported");
-  assert.ok(typesSrc.includes("'full'"), "should include full");
-  assert.ok(typesSrc.includes("'standard'"), "should include standard");
-  assert.ok(typesSrc.includes("'minimal'"), "should include minimal");
-});
-
-test("types: PhaseSkipPreferences interface exported", () => {
-  assert.ok(typesSrc.includes("export interface PhaseSkipPreferences"), "PhaseSkipPreferences should be exported");
-  assert.ok(typesSrc.includes("skip_research"), "should include skip_research");
-  assert.ok(typesSrc.includes("skip_reassess"), "should include skip_reassess");
-  assert.ok(typesSrc.includes("skip_slice_research"), "should include skip_slice_research");
-  assert.ok(typesSrc.includes("reassess_after_slice"), "should include reassess_after_slice");
-});
-
-// ═══════════════════════════════════════════════════════════════════════════
-// GSDPreferences Interface
-// ═══════════════════════════════════════════════════════════════════════════
-
-test("preferences: GSDPreferences includes token_profile field", () => {
-  assert.ok(
-    preferencesSrc.includes("token_profile?: TokenProfile"),
-    "GSDPreferences should have token_profile field",
-  );
-});
-
-test("preferences: GSDPreferences includes phases field", () => {
-  assert.ok(
-    preferencesSrc.includes("phases?: PhaseSkipPreferences"),
-    "GSDPreferences should have phases field",
-  );
-});
-
-test("preferences: GSDModelConfig includes subagent field", () => {
-  // Check both v1 and v2 configs
-  const v1Match = preferencesSrc.match(/interface GSDModelConfig\s*\{[^}]*subagent/);
-  assert.ok(v1Match, "GSDModelConfig should have subagent field");
-  const v2Match = preferencesSrc.match(/interface GSDModelConfigV2\s*\{[^}]*subagent/);
-  assert.ok(v2Match, "GSDModelConfigV2 should have subagent field");
-});
-
-test("preferences: KNOWN_PREFERENCE_KEYS includes token_profile and phases", () => {
-  assert.ok(preferencesSrc.includes('"token_profile"'), "KNOWN_PREFERENCE_KEYS should include token_profile");
-  assert.ok(preferencesSrc.includes('"phases"'), "KNOWN_PREFERENCE_KEYS should include phases");
-});
-
-// ═══════════════════════════════════════════════════════════════════════════
-// Profile Resolution
-// ═══════════════════════════════════════════════════════════════════════════
-
-test("profile: resolveProfileDefaults exists and handles all 3 tiers", () => {
-  assert.ok(
-    preferencesSrc.includes("export function resolveProfileDefaults"),
-    "resolveProfileDefaults should be exported",
-  );
-  assert.ok(
-    preferencesSrc.includes('case "budget"') &&
-    preferencesSrc.includes('case "balanced"') &&
-    preferencesSrc.includes('case "quality"'),
-    "resolveProfileDefaults should handle all 3 tiers",
-  );
-});
-
-test("profile: budget profile sets phase skips to true", () => {
-  // Extract the budget case block
-  const budgetIdx = preferencesSrc.indexOf('case "budget":');
-  const balancedIdx = preferencesSrc.indexOf('case "balanced":');
-  const budgetBlock = preferencesSrc.slice(budgetIdx, balancedIdx);
-  assert.ok(budgetBlock.includes("skip_research: true"), "budget should skip research");
-  assert.ok(budgetBlock.includes("skip_reassess: true"), "budget should skip reassess");
-  assert.ok(budgetBlock.includes("skip_slice_research: true"), "budget should skip slice research");
-});
-
-test("profile: balanced profile skips research, reassess, and slice research (ADR-003)", () => {
-  const balancedIdx = preferencesSrc.indexOf('case "balanced":');
-  const qualityIdx = preferencesSrc.indexOf('case "quality":');
-  const balancedBlock = preferencesSrc.slice(balancedIdx, qualityIdx);
-  assert.ok(balancedBlock.includes("skip_slice_research: true"), "balanced should skip slice research");
-  assert.ok(balancedBlock.includes("skip_research: true"), "balanced should skip milestone research");
-  assert.ok(balancedBlock.includes("skip_reassess: true"), "balanced should skip reassess");
-});
-
-test("profile: quality profile skips research, slice research, and reassess (ADR-003)", () => {
-  const qualityIdx = preferencesSrc.indexOf('case "quality":');
-  const qualityBlock = preferencesSrc.slice(qualityIdx, qualityIdx + 300);
-  assert.ok(qualityBlock.includes("skip_research: true"), "quality should skip research");
-  assert.ok(qualityBlock.includes("skip_slice_research: true"), "quality should skip slice research");
-  assert.ok(qualityBlock.includes("skip_reassess: true"), "quality should skip reassess");
-});
-
-// ═══════════════════════════════════════════════════════════════════════════
-// Default Behavior (D046)
-// ═══════════════════════════════════════════════════════════════════════════
-
-test("profile: resolveEffectiveProfile defaults to balanced (D046)", () => {
-  assert.ok(
-    preferencesSrc.includes("export function resolveEffectiveProfile"),
-    "resolveEffectiveProfile should be exported",
-  );
-  assert.ok(
-    preferencesSrc.includes('return "balanced"'),
-    "resolveEffectiveProfile should default to balanced",
-  );
-});
-
-// ═══════════════════════════════════════════════════════════════════════════
-// Inline Level Mapping
-// ═══════════════════════════════════════════════════════════════════════════
-
-test("profile: resolveInlineLevel maps profile to inline level", () => {
-  assert.ok(
-    preferencesSrc.includes("export function resolveInlineLevel"),
-    "resolveInlineLevel should be exported",
-  );
-  assert.ok(preferencesSrc.includes('case "budget": return "minimal"'), "budget → minimal");
-  assert.ok(preferencesSrc.includes('case "balanced": return "standard"'), "balanced → standard");
-  assert.ok(preferencesSrc.includes('case "quality": return "full"'), "quality → full");
-});
-
-// ═══════════════════════════════════════════════════════════════════════════
-// Validation
-// ═══════════════════════════════════════════════════════════════════════════
-
-test("validate: validatePreferences handles token_profile", () => {
-  assert.ok(
-    preferencesSrc.includes("preferences.token_profile") &&
-    preferencesSrc.includes("budget, balanced, quality"),
-    "validatePreferences should validate token_profile enum values",
-  );
-});
-
-test("validate: validatePreferences handles phases object", () => {
-  assert.ok(
-    preferencesSrc.includes("preferences.phases") &&
-    preferencesSrc.includes("skip_research") &&
-    preferencesSrc.includes("skip_reassess") &&
-    preferencesSrc.includes("skip_slice_research"),
-    "validatePreferences should validate phases fields",
-  );
-});
-
-test("validate: phases warns on unknown keys", () => {
-  assert.ok(
-    preferencesSrc.includes("knownPhaseKeys") &&
-    preferencesSrc.includes("unknown phases key"),
-    "validatePreferences should warn on unknown phase keys",
-  );
-});
-
-// ═══════════════════════════════════════════════════════════════════════════
-// Merge
-// ═══════════════════════════════════════════════════════════════════════════
-
-test("merge: mergePreferences handles token_profile with nullish coalescing", () => {
-  assert.ok(
-    preferencesSrc.includes("token_profile: override.token_profile ?? base.token_profile"),
-    "mergePreferences should use nullish coalescing for token_profile",
-  );
-});
-
-test("merge: mergePreferences handles phases with spread", () => {
-  assert.ok(
-    preferencesSrc.includes("...(base.phases") && preferencesSrc.includes("...(override.phases"),
-    "mergePreferences should spread phases objects",
-  );
-});
-
-// ═══════════════════════════════════════════════════════════════════════════
-// Subagent Model Routing
-// ═══════════════════════════════════════════════════════════════════════════
-
-test("subagent: budget profile sets subagent model", () => {
-  const budgetIdx = preferencesSrc.indexOf('case "budget":');
-  const balancedIdx = preferencesSrc.indexOf('case "balanced":');
-  const budgetBlock = preferencesSrc.slice(budgetIdx, balancedIdx);
-  assert.ok(budgetBlock.includes("subagent:"), "budget profile should set subagent model");
-});
-
-test("subagent: resolveModelWithFallbacksForUnit handles subagent unit types", () => {
-  assert.ok(
-    preferencesSrc.includes('"subagent"') && preferencesSrc.includes('startsWith("subagent/")'),
-    "resolveModelWithFallbacksForUnit should handle subagent and subagent/* unit types",
-  );
-});
-
-// ═══════════════════════════════════════════════════════════════════════════
-// Dispatch Table — Phase Skip Guards
-// ═══════════════════════════════════════════════════════════════════════════
-
-test("dispatch: research-milestone rule has skip_research guard", () => {
-  // Find the research-milestone rule and check it has the guard
-  const ruleIdx = dispatchSrc.indexOf("research-milestone");
-  assert.ok(ruleIdx > -1, "should have research-milestone rule");
-  // The guard should appear near this rule
-  assert.ok(
-    dispatchSrc.includes("skip_research") && dispatchSrc.includes("research-milestone"),
-    "research-milestone dispatch rule should check phases.skip_research",
-  );
-});
-
-test("dispatch: research-slice rule has skip guards", () => {
-  const ruleIdx = dispatchSrc.indexOf("research-slice");
-  assert.ok(ruleIdx > -1, "should have research-slice rule");
-  const afterRule = dispatchSrc.slice(ruleIdx);
-  assert.ok(
-    afterRule.includes("skip_research") || afterRule.includes("skip_slice_research"),
-    "research-slice rule should check skip_research or skip_slice_research",
-  );
-});
-
-test("dispatch: reassess-roadmap rule has reassess_after_slice opt-in guard (ADR-003)", () => {
-  assert.ok(
-    dispatchSrc.includes("reassess_after_slice") && dispatchSrc.includes("reassess-roadmap"),
-    "reassess-roadmap dispatch rule should check phases.reassess_after_slice",
-  );
-});
-
-test("dispatch: phase skip guards return null (not stop)", () => {
-  // Verify skip guards use return null pattern
-  const researchGuard = dispatchSrc.match(/skip_research\).*?return null/s);
-  assert.ok(researchGuard, "skip_research guard should return null (fall-through)");
-
-  const reassessGuard = dispatchSrc.match(/reassess_after_slice\).*?return null/s);
-  assert.ok(reassessGuard, "reassess_after_slice guard should return null (fall-through)");
-});