fix(rubric): the reconstruction now announces itself wherever its output is read - #50
Merged
Conversation
…put is read Second-pass review of the article found two residual defects, both inside the very artifact the article offers as proof of provenance: 1. The v1 archive reintroduced the dead `docs/skill/playbooks/...` pointer that the previous commit had just removed from the live rubric — because the archive was reconstructed from pre-fix content. It was the only remaining occurrence in the tracked tree, sitting in the file readers are sent to. Now gone everywhere. 2. `AUDIT.v1.md` published a bare "23/23 (100%) gate PASS" with nothing saying it came from a rubric rebuilt after the fact. A generated file gets read on its own, far from the header that carries the caveat — so the number travelled without it. That is the same defect the archive exists to correct, one file over. Fixed as DATA, not as a hand-edit: the rubric declares `provenance:`, and the renderer prints it as the third line of any report. A reconstruction cannot now produce output that looks like a clean run, because the label is generated from the same source as the score. Live rubric output is unchanged (31 items, 25/28, gate FAIL) — it declares no provenance, so no banner appears. 606 tests pass, ruff clean. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Second-pass review found two residual defects, both inside the artifact the article offers as proof.
docs/skill/playbooks/...pointer the previous commit had just removed from the live rubric — it was reconstructed from pre-fix content. It was the only remaining occurrence in the tracked tree. Gone everywhere now.AUDIT.v1.mdpublished a bare 23/23 (100%) gate PASS with nothing saying it came from a rubric rebuilt after the fact. A generated file is read on its own, far from the header carrying the caveat — so the number travelled without it. Same defect the archive exists to correct, one file over.Fixed as data, not a hand-edit: the rubric declares
provenance:, and the renderer prints it as line 3 of any report. A reconstruction can no longer produce output that looks like a clean run, because the label is generated from the same source as the score.Live rubric output unchanged (31 items, 25/28, gate FAIL) — it declares no provenance, so no banner.
606 tests, ruff clean.
🤖 Generated with Claude Code