Skip to content

test(ce-explain): guard predict-then-reveal parity across SKILL.md and check-in reference#1081

Merged
tmchow merged 1 commit into
EveryInc:mainfrom
LukeTheoJohnson:test-ce-explain-parity-guard
Jul 9, 2026
Merged

test(ce-explain): guard predict-then-reveal parity across SKILL.md and check-in reference#1081
tmchow merged 1 commit into
EveryInc:mainfrom
LukeTheoJohnson:test-ce-explain-parity-guard

Conversation

@LukeTheoJohnson

@LukeTheoJohnson LukeTheoJohnson commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

Closes #1057

Summary

Implements the parity guard requested in #1057. skills/ce-explain/SKILL.md Phase 3 and references/check-in.md both deliberately carry the predict-then-reveal protocol, but nothing stopped the two separate editable copies drifting apart silently: the routing test asserted only two substrings, in SKILL.md alone.

Fix

Extends tests/skills/ce-explain-routing.test.ts with a parity block asserting BOTH files carry the three invariants named in the issue:

  • the prediction question (what does this change do, and why was it made)
  • the turn-end rule ("end the turn")
  • the never-same-message rule (no explanation in the same message as the prediction prompt)

The assertions are structural regexes. The two copies already word the protocol slightly differently, and future wording improvements stay possible as long as each invariant survives in both files. Failure messages name the sibling copy so whoever trips the guard knows the duplication is by design and where the other copy lives.

Verification

  • Red probe: replacing "End the turn." with "Continue." in check-in.md fails exactly one test, naming the file and the missing invariant; restoring the line returns the suite to green. 11/11 tests in the file pass on this branch (5 existing + 6 new).
  • Full bun test baseline diff against main on this machine: zero new failures.

@LukeTheoJohnson LukeTheoJohnson marked this pull request as ready for review July 8, 2026 11:55
@tmchow tmchow merged commit fc0395b into EveryInc:main Jul 9, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

ce-explain: add cross-file parity guard for duplicated predict-then-reveal protocol

2 participants