From a136cea5493f106b99dcb44e1ea02a09eeaf4fc2 Mon Sep 17 00:00:00 2001 From: Cursor Agent Date: Sun, 4 Oct 2026 20:57:19 +0000 Subject: [PATCH 1/2] fix(check-versions): watch execution-log and deck-review stamps PR #9 stamped both examples at v2.8.0 and left them out of FILES. Plan 009 already shipped in 5b4af3f; the index still said TODO. Co-authored-by: Todd Dailey --- plans/README.md | 2 +- scripts/check-versions.mjs | 2 ++ tests/test_linters.py | 10 ++++++++++ 3 files changed, 13 insertions(+), 1 deletion(-) diff --git a/plans/README.md b/plans/README.md index 6983924..4d56713 100644 --- a/plans/README.md +++ b/plans/README.md @@ -16,7 +16,7 @@ All quality gates were green at planning time: `uv run --with pytest pytest test | 006 | Tighten feature-floor vocabulary, dead-control scan, hex matching in the QA gates | P2 | M | 005 | DONE (4b65691; gallery pages reclassified to --reference — they had zero real interactivity) | | 007 | SKILL.md §Design system progressive disclosure → references/design.md | P3 | M | 005 | DONE (70dfe42; 1,004-tok reduction — plan projected ≥1.2k, in-scope content only weighed ~1,022; accepted) | | 008 | render-podcast ↔ canonical-example parity enforcement + VERSION constant | P3 | M | 001, 004 | DONE (859da1c; dark-block swap was dead code — deleted, replaced with a loud drift assertion) | -| 009 | Single-source the HTML-check vocabulary (shared JS lib + JS/Python parity test) | P3 | M | 002, 006 | TODO | +| 009 | Single-source the HTML-check vocabulary (shared JS lib + JS/Python parity test) | P3 | M | 002, 006 | DONE (5b4af3f) | Status values: TODO | IN PROGRESS | DONE | BLOCKED (with one-line reason) | REJECTED (with one-line rationale — finding fixed independently or approach abandoned) diff --git a/scripts/check-versions.mjs b/scripts/check-versions.mjs index d6286b9..e58b504 100644 --- a/scripts/check-versions.mjs +++ b/scripts/check-versions.mjs @@ -36,6 +36,8 @@ const FILES = [ "examples/primitives.html", "examples/podcast.html", "examples/podcast-transcript.html", + "examples/execution-log.html", + "examples/deck-review.html", "bin/render-podcast", ]; diff --git a/tests/test_linters.py b/tests/test_linters.py index 84916cb..db6d805 100644 --- a/tests/test_linters.py +++ b/tests/test_linters.py @@ -153,6 +153,16 @@ def test_check_versions_clean(): assert r.returncode == 0, r.stdout + r.stderr +def test_check_versions_covers_newest_examples(): + """PR #9 stamped execution-log and deck-review at v2.8.0 but left them + out of FILES. The checker must print OK for both. Omitting either path + from FILES drops that line and this test fails.""" + r = _run("scripts/check-versions.mjs") + assert r.returncode == 0, r.stdout + r.stderr + assert "OK examples/execution-log.html" in r.stdout + assert "OK examples/deck-review.html" in r.stdout + + # --- review-contracts.mjs ----------------------------------------------------- def test_review_contracts_clean(): From 7f69dfb26fe5887856575ede9a0323008da7b0f9 Mon Sep 17 00:00:00 2001 From: Cursor Agent Date: Sun, 4 Oct 2026 21:08:11 +0000 Subject: [PATCH 2/2] chore: drop history sermon from newest-examples test Comment Sicko: the docstring restated PR #9 and the two OK asserts. Co-authored-by: Todd Dailey --- tests/test_linters.py | 3 --- 1 file changed, 3 deletions(-) diff --git a/tests/test_linters.py b/tests/test_linters.py index db6d805..1e6d720 100644 --- a/tests/test_linters.py +++ b/tests/test_linters.py @@ -154,9 +154,6 @@ def test_check_versions_clean(): def test_check_versions_covers_newest_examples(): - """PR #9 stamped execution-log and deck-review at v2.8.0 but left them - out of FILES. The checker must print OK for both. Omitting either path - from FILES drops that line and this test fails.""" r = _run("scripts/check-versions.mjs") assert r.returncode == 0, r.stdout + r.stderr assert "OK examples/execution-log.html" in r.stdout