From f16ad3690ebe4119ed0bfed95e29569e5aaf50d7 Mon Sep 17 00:00:00 2001 From: Kjell Tore Guttormsen Date: Thu, 18 Jun 2026 11:10:51 +0200 Subject: [PATCH] test(voyage): retarget phase_models/phase_signals doc-consistency to canonical doc homes The v5.x doc consolidation (67f6dd5) moved phase_models into docs/profiles.md and phase_signals into docs/HANDOVER-CONTRACTS.md, leaving CLAUDE.md to only link them via Reference docs. Two doc-consistency assertions still read CLAUDE.md and went red on main. Retarget both to the canonical doc homes (legacy-alias check preserved on docs/profiles.md). Suite: 578 pass / 0 fail / 2 skipped. Co-Authored-By: Claude Opus 4.8 (1M context) Claude-Session: https://claude.ai/code/session_01LqBYc8Ltrk7LipyJmGxXiB --- tests/lib/doc-consistency.test.mjs | 23 ++++++++++++++++------- 1 file changed, 16 insertions(+), 7 deletions(-) diff --git a/tests/lib/doc-consistency.test.mjs b/tests/lib/doc-consistency.test.mjs index bc96ab4..8806379 100644 --- a/tests/lib/doc-consistency.test.mjs +++ b/tests/lib/doc-consistency.test.mjs @@ -317,17 +317,22 @@ test('CLAUDE.md documents --profile flag', () => { ); }); -test('CLAUDE.md uses canonical name `phase_models`', () => { - const md = read('CLAUDE.md'); +test('canonical name `phase_models` documented in docs/profiles.md', () => { + // v5.x doc consolidation (67f6dd5) moved profile/phase detail OUT of + // CLAUDE.md and into docs/. docs/profiles.md is now the canonical home for + // phase_models; CLAUDE.md only links to it via "Reference docs". Assert the + // canonical name lives in the canonical doc and no legacy alias crept in. + // (was: read('CLAUDE.md') — v4.1 SC #20.) + const md = read('docs/profiles.md'); assert.match( md, /phase_models/, - 'CLAUDE.md must use canonical name "phase_models" (v4.1 SC #20)', + 'docs/profiles.md must use canonical name "phase_models" (v4.1 SC #20)', ); for (const bad of ['model_per_phase', 'phase_to_model', 'profile_phase_models']) { assert.ok( !md.includes(bad), - `CLAUDE.md must NOT use legacy alias "${bad}"`, + `docs/profiles.md must NOT use legacy alias "${bad}"`, ); } }); @@ -574,10 +579,14 @@ test('v5.1 — HANDOVER-CONTRACTS.md schema row includes phase_signals + phase_s 'HANDOVER-CONTRACTS must add a phase_signals_partial row to the Handover 1 schema table'); }); -test('v5.1 — voyage CLAUDE.md mentions phase_signals', () => { - const t = read('CLAUDE.md'); +test('v5.1 — phase_signals documented in docs/HANDOVER-CONTRACTS.md', () => { + // v5.x doc consolidation (67f6dd5) moved phase_signals out of CLAUDE.md. + // phase_signals is a Handover 1 (brief) field, so docs/HANDOVER-CONTRACTS.md + // is its canonical home (the schema rows are pinned separately above). + // CLAUDE.md only links to it. (was: read('CLAUDE.md') — v5.1.) + const t = read('docs/HANDOVER-CONTRACTS.md'); assert.ok(t.includes('phase_signals'), - 'voyage CLAUDE.md must document phase_signals (v5.1)'); + 'docs/HANDOVER-CONTRACTS.md must document phase_signals (v5.1)'); }); test('v5.1 — voyage README.md mentions phase_signals', () => {