golem15/msd-core
3
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

fix(3560): ignore archived phases for W007 warnings

Browse Source
This commit is contained in:
Tom Boucher
2026-05-15 12:51:51 -04:00
parent 3edbf7f357
commit 21ae65f433
2 changed files with 26 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

20
sdk/src/query/validate.test.ts
View File

@@ -643,6 +643,26 @@ describe('validateHealth', () => {
expect(w006s.some(w => String(w.message).includes('Phase 7'))).toBe(false);
});
it('does not emit W007 for archived milestone-only phase dirs (#3560)', async () => {
await createHealthyPlanning();
await writeFile(join(tmpDir, '.planning', 'ROADMAP.md'), [
'# Roadmap',
'',
'## v1.1: Current',
'',
'### Phase 21: Active',
'',
].join('\n'));
await mkdir(join(tmpDir, '.planning', 'phases', '21-active'), { recursive: true });
await mkdir(join(tmpDir, '.planning', 'milestones', 'v1.0-phases', '02-old-shipped-phase'), { recursive: true });
const result = await validateHealth([], tmpDir);
const data = result.data as Record<string, unknown>;
const warnings = data.warnings as Array<Record<string, unknown>>;
const w007s = warnings.filter(w => w.code === 'W007');
expect(w007s.some(w => String(w.message).includes('Phase 02'))).toBe(false);
});
it('does not emit W006 for unchecked future phases with no directory (#3559)', async () => {
await createHealthyPlanning();
await writeFile(join(tmpDir, '.planning', 'ROADMAP.md'), [

8
sdk/src/query/validate.ts
View File

@@ -621,12 +621,16 @@ export const validateHealth: QueryHandler = async (args, projectDir, workstream)
}
const diskPhases = new Set<string>();
const activeDiskPhases = new Set<string>();
try {
const entries = await readdir(phasesDir, { withFileTypes: true });
for (const e of entries) {
if (e.isDirectory()) {
const dm = e.name.match(/^(\d+[A-Z]?(?:\.\d+)*)/i);
if (dm) diskPhases.add(dm[1]);
if (dm) {
diskPhases.add(dm[1]);
activeDiskPhases.add(dm[1]);
}
}
}
} catch { /* intentionally empty */ }
@@ -655,7 +659,7 @@ export const validateHealth: QueryHandler = async (args, projectDir, workstream)
}
}
for (const p of diskPhases) {
for (const p of activeDiskPhases) {
const variants = phaseVariants(p);
if (![...variants].some((variant) => roadmapPhaseVariants.has(variant))) {
addIssue('warning', 'W007', `Phase ${p} exists on disk but not in ROADMAP.md`, 'Add to roadmap or remove directory');