Files
bmad-code-org__bmad-method/test/test-dev-auto-contract.js
Alex Verkhovsky cfee292715 fix(dev-auto): move deferred findings into the spec (#2640)
Record deferred review findings only in spec frontmatter and remove the
deferred-work output. Make updates safe for legacy specs and YAML-special
content, strengthen contract coverage, and synchronize the reference docs.
2026-07-28 18:42:42 -07:00

138 lines
6.1 KiB
JavaScript

/**
* Regression coverage for bmad-dev-auto's deferred-finding contract.
*
* Ensures the canonical source keeps:
* 1. Machine-readable `deferred` frontmatter on the spec template.
* 2. Review-step instructions that persist deferred findings only in the spec.
* 3. Reference docs that tell orchestrators to read deferred findings from the spec.
*/
'use strict';
const fs = require('node:fs');
const path = require('node:path');
const yaml = require('yaml');
const colors = {
reset: '\u001B[0m',
green: '\u001B[32m',
red: '\u001B[31m',
cyan: '\u001B[36m',
};
let totalTests = 0;
let passedTests = 0;
const failures = [];
function test(name, fn) {
totalTests++;
try {
fn();
passedTests++;
console.log(` ${colors.green}\u2713${colors.reset} ${name}`);
} catch (error) {
console.log(` ${colors.red}\u2717${colors.reset} ${name} ${colors.red}${error.message}${colors.reset}`);
failures.push({ name, message: error.message });
}
}
function assert(condition, message) {
if (!condition) throw new Error(message);
}
function read(relativePath) {
return fs.readFileSync(path.join(__dirname, '..', relativePath), 'utf-8');
}
function parseFrontmatter(content, relativePath) {
assert(content.startsWith('---\n'), `${relativePath} must start with a frontmatter delimiter`);
const end = content.indexOf('\n---\n', 4);
assert(end !== -1, `${relativePath} must close its frontmatter delimiter`);
return yaml.parse(content.slice(4, end));
}
function dedent(content) {
const lines = content.split('\n');
const indents = lines.filter((line) => line.trim()).map((line) => line.match(/^ */)[0].length);
const width = Math.min(...indents);
return lines.map((line) => line.slice(width)).join('\n');
}
console.log(`\n${colors.cyan}bmad-dev-auto deferred contract${colors.reset}\n`);
test('spec template exposes machine-readable deferred frontmatter', () => {
const relativePath = 'src/bmm-skills/4-implementation/bmad-dev-auto/spec-template.md';
const frontmatter = parseFrontmatter(read(relativePath), relativePath);
assert(Array.isArray(frontmatter.deferred), 'spec-template.md frontmatter must declare deferred as a list');
assert(frontmatter.deferred.length === 0, 'spec-template.md deferred list must start empty');
});
test('dev-auto steps preserve their frontmatter boundaries', () => {
const root = 'src/bmm-skills/4-implementation/bmad-dev-auto';
const stepOnePath = `${root}/step-01-clarify-and-route.md`;
const stepOneFrontmatter = parseFrontmatter(read(stepOnePath), stepOnePath);
assert(stepOneFrontmatter.spec_file === '', 'step-01 must define spec_file in frontmatter');
assert(stepOneFrontmatter.spec_folder === '', 'step-01 must define spec_folder in frontmatter');
assert(stepOneFrontmatter.story_id === '', 'step-01 must define story_id in frontmatter');
for (const filename of ['step-02-plan.md', 'step-04-review.md']) {
const relativePath = `${root}/${filename}`;
const content = read(relativePath);
if (content.startsWith('---\n')) parseFrontmatter(content, relativePath);
}
});
test('review step safely records deferred findings only in the spec', () => {
const content = read('src/bmm-skills/4-implementation/bmad-dev-auto/step-04-review.md');
assert(content.includes('If the field is absent'), 'step-04-review.md must initialize deferred for legacy specs');
assert(content.includes('never add a second `deferred:` key'), 'step-04-review.md must forbid duplicate deferred keys');
assert(content.includes('parse the complete frontmatter as YAML'), 'step-04-review.md must validate the updated frontmatter');
assert(!content.includes('deferred_work_file'), 'step-04-review.md must not mention a deferred-work ledger path');
assert(!content.includes('deferred-work.md'), 'step-04-review.md must not mention the deferred-work ledger artifact');
const example = content.match(/```yaml\n([\s\S]*?)\n[ \t]*```/);
assert(example, 'step-04-review.md must include the deferred YAML example');
const specialCharacters = dedent(example[1])
.replace('<one sentence>', 'Parser fails: malformed # input')
.replace('<why this is real>', 'Observed: value # remains data\n Second evidence line');
const parsed = yaml.parse(specialCharacters);
assert(parsed.deferred[0].summary === 'Parser fails: malformed # input', 'summary example must preserve YAML-special characters');
assert(
parsed.deferred[0].evidence === 'Observed: value # remains data\nSecond evidence line',
'evidence example must preserve YAML-special characters and line breaks',
);
});
test('reference docs direct orchestrators to the spec deferred list', () => {
const content = read('docs/reference/dev-auto.md');
assert(
content.includes('Read deferred findings from the spec frontmatter `deferred:` list'),
'docs/reference/dev-auto.md must tell orchestrators where to read deferred findings',
);
assert(!content.includes('deferred-work.md'), 'docs/reference/dev-auto.md must not describe a deferred-work ledger artifact');
});
test('Chinese reference documents the same deferred contract', () => {
const content = read('docs/zh-cn/reference/dev-auto.md');
assert(content.includes('spec frontmatter 的 `deferred:` list'), 'Chinese reference must direct orchestrators to the deferred list');
assert(!content.includes('deferred-work.md'), 'Chinese reference must not describe a deferred-work ledger artifact');
});
console.log(`\n${colors.cyan}${'═'.repeat(55)}${colors.reset}`);
console.log(`${colors.cyan}Test Results:${colors.reset}`);
console.log(` Total: ${totalTests}`);
console.log(` Passed: ${colors.green}${passedTests}${colors.reset}`);
console.log(` Failed: ${passedTests === totalTests ? colors.green : colors.red}${totalTests - passedTests}${colors.reset}`);
console.log(`${colors.cyan}${'═'.repeat(55)}${colors.reset}\n`);
if (failures.length > 0) {
console.log(`${colors.red}FAILED TESTS:${colors.reset}\n`);
for (const failure of failures) {
console.log(`${colors.red}\u2717${colors.reset} ${failure.name}`);
console.log(` ${failure.message}\n`);
}
process.exit(1);
}
console.log(`${colors.green}All tests passed!${colors.reset}\n`);