From 3a1dceb83a0d8a677f771f19c3e30f281bafa2b6 Mon Sep 17 00:00:00 2001 From: Brummel Date: Sun, 14 Jun 2026 00:20:23 +0200 Subject: [PATCH] =?UTF-8?q?audit:=20cycle=200038=20=E2=80=94=20drift-clean?= =?UTF-8?q?=20(shared=20SMA-cross=20test=20fixtures)?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Close-audit for cycle 0038 (commit range c939673..1f01bad). Architect drift review against the design ledger + CLAUDE.md domain invariants: no contract drift, no debt. No regression scripts are configured (the architect is the gate); build + test + clippy verified green independently by the orchestrator (aura-engine 114 passed, count unchanged; the two load-bearing consumers green; clippy --all-targets -D warnings exit 0). cycle 0038 (clean): - C16 preserved: the new shared fixtures live in a #[cfg(test)] mod test_fixtures, which compiles into no shipped artifact — the engine still ships no sample. - C1 / C11 / C12 strengthened, not weakened: the determinism witness (composite_sma_cross_runs_bit_identical_to_hand_wired) and the sweep-equivalence witness (sweep_equals_n_independent_runs) now consume one shared fixture instead of two hand-kept copies, so the harness topology can no longer silently diverge between the two test modules — the exact hazard #53 named. - Test-organisation convention held: a crate-root #[cfg(test)] module of pub(crate) free fns, sibling to the existing reexport_tests precedent, declared in lib.rs. - graph_model.rs's minimal golden copy (sma_cross / sample_root) was correctly left independent and out of scope, by design. - Stale-cross-ref sweep clean: docs/design/INDEX.md and docs/glossary.md name no fixture site that the move invalidated; the fixture-name hits in docs/ are historical specs/plans pinned to their own cycles, not live contracts. In-flight plan correction (recorded for history): the plan's literal import list over-imported sma_cross, which neither consumer calls directly (reached only transitively via composite_sma_cross_harness); the implementer trimmed it to satisfy the plan's own clippy -D warnings gate — within the task's import-cleanup scope.