Files
msd-core/tests/prompt-budget-cli.test.cjs
Tom Boucher 6a5fa59129 feat(3081): auto-trim review prompts for small-context model reviewers (#3708)
* feat(3081): auto-trim review prompts for small-context model reviewers

Adds review.max_prompt_tokens and review.max_prompt_tokens_per_reviewer
config keys. When configured, the /gsd-review workflow deterministically
trims the assembled prompt before sending to each reviewer (drop CONTEXT
→ RESEARCH → REQUIREMENTS; head-shrink PROJECT.md; tail-truncate PLANs
proportionally; reserve disclosure-note tokens upfront). Trim metadata
is recorded in REVIEWS.md frontmatter. Reviewer is skipped with a
warning if even the minimum review set exceeds the budget.

Closes #3081

* fix(3081): register prompt-budget in SDK query registry and update inventory manifest

review.md references `gsd-sdk query prompt-budget` at three call sites, but the
command had no handler in the SDK registry — failing the registry-integration
drift-guard test on all 6 CI matrix legs. Added a native TypeScript SDK handler
(sdk/src/query/prompt-budget.ts) that ports the applyBudget logic from the CJS
module, registered it in DOMAIN_STATIC_CATALOG, and regenerated
docs/INVENTORY-MANIFEST.json to include the new cli_modules/prompt-budget.cjs entry.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

* fix(3081): bump ws to 8.20.1 and allowlist prompt-budget sibling pair

Two additional CI failures after the registry fix:

1. ws moderate CVE (GHSA-58qx-3vcg-4xpx, uninitialized memory disclosure):
   The advisory covers ws >=8.0.0 <8.20.1. Both root and sdk/package.json
   pinned ^8.20.0 which resolved to 8.20.0. Bumped both to 8.20.1 to clear
   the npm audit drift-guard test (bug-3588-npm-audit-clean.test.cjs).

2. lint-shared-module-handsync detected the new prompt-budget.ts / prompt-budget.cjs
   sibling pair without an allowlist entry. Added a cooperatingSiblings entry
   to scripts/shared-module-handsync-allowlist.json with classification and
   justification matching the established pattern.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

* fix(3081): align prompt-budget skip semantics across CJS and SDK dispatch paths

Replace brittle `[ $EXIT -eq 2 ]` guards with `[ $EXIT -ne 0 ]` in all three
local-reviewer blocks (Ollama, LM Studio, llama.cpp) in workflows/review.md.
Any non-zero exit from prompt-budget now triggers a skip with a descriptive
warning — exit 2/11 prints "budget too small", any other non-zero prints
"unexpected exit code". This ensures the SDK bridge dispatch path (exit 11
via GSDError(Blocked)) triggers the same skip as the CJS path (exit 2).

The SDK handler (sdk/src/query/prompt-budget.ts) already writes both metadata
and prompt files before throwing, so no change needed there.

The Ollama block also gains the missing OLLAMA_SKIP guard so the reviewer
invocation is actually skipped (previously the block only suppressed the
OLLAMA_PROMPT_FILE update but still ran the curl invocation).

SDK integration path (hardFailed via GSDError(Blocked) → exit 11) is covered
by handler unit tests in tests/prompt-budget.test.cjs; no gsd-sdk-*.test.cjs
exercising the full bridge dispatch for this command exists yet — that gap
remains and is documented here.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

* fix prompt-budget trim ordering and review guard follow-ups

* perf: optimize prompt-budget and dedup reviewer trim workflow

* fix(3708): drop source-grep theater tests to satisfy lint-no-source-grep

All four test files added in commit 2df566ed were pure source-grep theater:
they read .cjs / .ts / .md source files and asserted that specific string
literals were present or absent. None exercised runtime behaviour.

Deleted:
- tests/gsd-tools-memory-optimizer.test.cjs   — 7 includes() on gsd-tools.cjs
- tests/prompt-budget-hotpath-optimizer.test.cjs — includes() on prompt-budget.cjs + .ts
- tests/prompt-budget-io-optimizer.test.cjs   — includes() on prompt-budget.ts + gsd-tools.cjs
- tests/review-workflow-budget-dedup.test.cjs — includes() on review.md

Behavioural coverage for the prompt-budget feature already exists in
tests/prompt-budget.test.cjs and tests/prompt-budget-cli.test.cjs (also
added by this PR). No replacement tests needed.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

* fix(3708): correct budget-pressure threshold and minSet accounting

Two bugs in applyBudget caused premature trimming and false hard-fails:

1. UNNEEDED_TRIM: budgetUnderPressure compared baseTokens against
   effectiveBudget - NOTE_RESERVE_TOKENS, triggering trim pressure 80
   tokens before the budget was actually exceeded. Fix: compare against
   effectiveBudget directly; NOTE_RESERVE_TOKENS are still reserved in
   contentBudget once real pressure is confirmed.

2. FALSE_HARDFAIL: minSet included NOTE_RESERVE_TOKENS unconditionally,
   treating the note as mandatory even when no trim would occur and no
   note would be injected. Fix: exclude NOTE_RESERVE_TOKENS from minSet;
   a prompt that fits untrimmed needs no note and must not hard-fail.

Both fixes applied in CJS and TypeScript implementations. Two regression
tests added (cycles 11 and 12) that reproduce each case behaviorally.

---------

Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
2026-05-18 23:13:09 -04:00

270 lines
10 KiB
JavaScript

'use strict';
// allow-test-rule: prompt-content-is-the-product
// The prompt-budget CLI writes an assembled, trimmed prompt string to disk.
// Testing that the prompt omits a dropped section (research) requires a
// content assertion on the output file — the file content IS the product.
// Structured metadata (omitted[], hardFailed, etc.) is always the primary
// assertion; text content checks are secondary and only used to verify the
// trim policy was applied correctly to the assembled output.
/**
* prompt-budget-cli.test.cjs
*
* Integration tests for the `gsd-tools prompt-budget` CLI subcommand.
* Covers the 5 specified scenarios:
* 1. Happy path: budget forces trim (research dropped)
* 2. No-trim path: huge budget, metadata shows no omissions
* 3. Hard-fail path: tiny budget, exit 2, metadata written, prompt empty
* 4. Missing required arg: exit 1, stderr has error message
* 5. Missing input file: exit 1
*/
const { describe, test } = require('node:test');
const assert = require('node:assert/strict');
const fs = require('node:fs');
const path = require('node:path');
const { createTempDir, cleanup, runGsdTools } = require('./helpers.cjs');
const TEST_INSTRUCTIONS = [
'# Cross-AI Plan Review Request',
'',
'You are reviewing implementation plans for a software project phase.',
'Provide structured feedback on plan quality, completeness, and risks.',
].join('\n');
const TEST_ROADMAP = [
'## Phase 3: Implement token budgeting',
'',
'### Goal',
'Add deterministic prompt trimming for small-context local model servers.',
].join('\n');
const TEST_PLAN = [
'## PLAN-01: Add prompt-budget module',
'',
'### Tasks',
'- [ ] Write estimateTokens()',
'- [ ] Write applyBudget()',
'- [ ] Write CLI wrapper',
].join('\n');
const TEST_RESEARCH = 'a'.repeat(8000); // ~2000 tokens — large enough to force trimming
/**
* Run gsd-tools with args as an array (safe for paths with spaces/dollars).
* Returns { exitCode, stdout, stderr }.
*/
function runCli(args, cwd) {
const result = runGsdTools(args, cwd);
return {
exitCode: result.exitCode ?? (result.success ? 0 : 1),
stdout: result.output ?? '',
stderr: result.error ?? '',
};
}
describe('prompt-budget CLI', () => {
// ── Cycle 1: happy path — small budget that trims research ────────────────
test('happy path: trims research when budget is small, exit 0', () => {
const dir = createTempDir('pb-cli-happy-');
try {
// Write input files
const instrFile = path.join(dir, 'instructions.md');
const roadmapFile = path.join(dir, 'roadmap.md');
const planFile = path.join(dir, 'plan-01.md');
const researchFile = path.join(dir, 'research.md');
const outPrompt = path.join(dir, 'out-prompt.md');
const outMeta = path.join(dir, 'out-meta.json');
fs.writeFileSync(instrFile, TEST_INSTRUCTIONS);
fs.writeFileSync(roadmapFile, TEST_ROADMAP);
fs.writeFileSync(planFile, TEST_PLAN);
fs.writeFileSync(researchFile, TEST_RESEARCH);
// Budget of 800 tokens: enough for instructions+roadmap+plan (min set ~389 tokens
// with 10% safety margin) but not the ~2000-token research blob.
// effectiveBudget = floor(800 * 0.9) = 720.
// contentBudget = 720 - 80 (note reserve) = 640. Research (~2000 tokens) won't fit.
const { exitCode, stderr } = runCli([
'prompt-budget',
'--budget', '800',
'--instructions-file', instrFile,
'--roadmap-file', roadmapFile,
'--plan-file', planFile,
'--research-file', researchFile,
'--output-prompt', outPrompt,
'--output-metadata', outMeta,
], dir);
assert.equal(exitCode, 0, `Expected exit 0, got ${exitCode}. stderr: ${stderr}`);
// Output files must exist
assert.ok(fs.existsSync(outPrompt), 'output prompt file should exist');
assert.ok(fs.existsSync(outMeta), 'output metadata file should exist');
// Metadata must parse and have expected shape
const meta = JSON.parse(fs.readFileSync(outMeta, 'utf8'));
assert.equal(typeof meta.budget, 'number');
assert.ok(Array.isArray(meta.omitted));
assert.ok(meta.omitted.includes('research'), `Expected research in omitted, got: ${JSON.stringify(meta.omitted)}`);
assert.equal(meta.hardFailed, false);
// Prompt must not contain research content
const promptText = fs.readFileSync(outPrompt, 'utf8');
assert.ok(promptText.length > 0, 'prompt file must not be empty');
// Research was dropped so the 'aaa...' block should be absent from the prompt
assert.ok(!promptText.includes('a'.repeat(100)), 'dropped research should not appear in prompt');
} finally {
cleanup(dir);
}
});
// ── Cycle 2: no-trim path — huge budget, nothing dropped ──────────────────
test('no-trim path: huge budget returns all sections, exit 0', () => {
const dir = createTempDir('pb-cli-notrim-');
try {
const instrFile = path.join(dir, 'instructions.md');
const roadmapFile = path.join(dir, 'roadmap.md');
const planFile = path.join(dir, 'plan-01.md');
const researchFile = path.join(dir, 'research.md');
const outPrompt = path.join(dir, 'out-prompt.md');
const outMeta = path.join(dir, 'out-meta.json');
fs.writeFileSync(instrFile, TEST_INSTRUCTIONS);
fs.writeFileSync(roadmapFile, TEST_ROADMAP);
fs.writeFileSync(planFile, TEST_PLAN);
fs.writeFileSync(researchFile, 'Some research findings.');
const { exitCode } = runCli([
'prompt-budget',
'--budget', '1000000',
'--instructions-file', instrFile,
'--roadmap-file', roadmapFile,
'--plan-file', planFile,
'--research-file', researchFile,
'--output-prompt', outPrompt,
'--output-metadata', outMeta,
], dir);
assert.equal(exitCode, 0);
const meta = JSON.parse(fs.readFileSync(outMeta, 'utf8'));
assert.deepEqual(meta.omitted, []);
assert.equal(meta.hardFailed, false);
assert.equal(meta.projectMdShrunk, false);
assert.equal(meta.planTruncationPct, 0);
assert.equal(meta.noteInjected, false);
// All sections must appear in the prompt
const promptText = fs.readFileSync(outPrompt, 'utf8');
assert.ok(promptText.includes('Some research findings.'));
} finally {
cleanup(dir);
}
});
// ── Cycle 3: hard-fail path — budget is impossibly small ──────────────────
test('hard-fail path: exit 2 when minimum set exceeds budget, metadata written, prompt empty', () => {
const dir = createTempDir('pb-cli-hardfail-');
try {
const instrFile = path.join(dir, 'instructions.md');
const roadmapFile = path.join(dir, 'roadmap.md');
const planFile = path.join(dir, 'plan-01.md');
const outPrompt = path.join(dir, 'out-prompt.md');
const outMeta = path.join(dir, 'out-meta.json');
// Large instructions to ensure minimum set exceeds the tiny budget
fs.writeFileSync(instrFile, 'a'.repeat(4000)); // ~1000 tokens
fs.writeFileSync(roadmapFile, TEST_ROADMAP);
fs.writeFileSync(planFile, TEST_PLAN);
// Budget of 5 tokens is far below the minimum set
const { exitCode, stderr } = runCli([
'prompt-budget',
'--budget', '5',
'--instructions-file', instrFile,
'--roadmap-file', roadmapFile,
'--plan-file', planFile,
'--output-prompt', outPrompt,
'--output-metadata', outMeta,
], dir);
assert.equal(exitCode, 2, `Expected exit 2, got ${exitCode}. stderr: ${stderr}`);
// Metadata must still be written
assert.ok(fs.existsSync(outMeta), 'metadata file must be written even on hard fail');
const meta = JSON.parse(fs.readFileSync(outMeta, 'utf8'));
assert.equal(meta.hardFailed, true);
// Prompt file must be written but empty
assert.ok(fs.existsSync(outPrompt), 'prompt file must exist even on hard fail');
const promptText = fs.readFileSync(outPrompt, 'utf8');
assert.equal(promptText, '', 'prompt file must be empty on hard fail');
} finally {
cleanup(dir);
}
});
// ── Cycle 4: missing required arg — exit 1, stderr has error ──────────────
test('missing required arg: exit 1, stderr contains error message', () => {
const dir = createTempDir('pb-cli-missingarg-');
try {
const instrFile = path.join(dir, 'instructions.md');
const roadmapFile = path.join(dir, 'roadmap.md');
const planFile = path.join(dir, 'plan-01.md');
const outPrompt = path.join(dir, 'out-prompt.md');
const outMeta = path.join(dir, 'out-meta.json');
fs.writeFileSync(instrFile, TEST_INSTRUCTIONS);
fs.writeFileSync(roadmapFile, TEST_ROADMAP);
fs.writeFileSync(planFile, TEST_PLAN);
// Omit --budget (required)
const { exitCode, stderr } = runCli([
'prompt-budget',
'--instructions-file', instrFile,
'--roadmap-file', roadmapFile,
'--plan-file', planFile,
'--output-prompt', outPrompt,
'--output-metadata', outMeta,
], dir);
assert.equal(exitCode, 1, `Expected exit 1, got ${exitCode}`);
assert.ok(stderr.includes('--budget'), `Expected stderr to mention --budget, got: ${stderr}`);
} finally {
cleanup(dir);
}
});
// ── Cycle 5: missing input file — exit 1 ──────────────────────────────────
test('missing input file: exit 1', () => {
const dir = createTempDir('pb-cli-missingfile-');
try {
const roadmapFile = path.join(dir, 'roadmap.md');
const planFile = path.join(dir, 'plan-01.md');
const outPrompt = path.join(dir, 'out-prompt.md');
const outMeta = path.join(dir, 'out-meta.json');
fs.writeFileSync(roadmapFile, TEST_ROADMAP);
fs.writeFileSync(planFile, TEST_PLAN);
// Instructions file doesn't exist
const { exitCode } = runCli([
'prompt-budget',
'--budget', '10000',
'--instructions-file', path.join(dir, 'nonexistent.md'),
'--roadmap-file', roadmapFile,
'--plan-file', planFile,
'--output-prompt', outPrompt,
'--output-metadata', outMeta,
], dir);
assert.equal(exitCode, 1, 'Expected exit 1 when instructions file is missing');
} finally {
cleanup(dir);
}
});
});