Repository navigation
Make the trust-map explainer a verification explainer; remove the review flow - #29
Merged
Merged
Conversation
Remove the components that only served understando's PR review: the start panel, the checklist with the reviewer's calls in local storage and its Markdown export, the verdict, the checks before you approve, the review and chapter progress marks in the rail, the merge verdict with its gates and bug panels, the diff spine with its file cards and Reviewed boxes, the older what-changed and unclaimed-changes levels, the self-check questions, and the PR-body section (trust-map.mjs pr-section). Keep the claim, check and bar rows, the evidence guide, the claims and their checks (now each claim's one detail, with its worked examples), the invariants (now in view), the code refs with line marks, the typed diagrams and the paged shell. The data contract takes a minimal shape: promises, bars, check rows, test counts, examples and the diff are optional. The fixture is now a verification explainer, and its browser tests check the page, the minimal shape and the old map shape. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…ract Rewrite the trust-map card as a short card for a verification explainer: one claim from the data to the page, the minimal data contract, the components, the copy rules and the export checks. Update the component reference, DESIGN.md and the design surface to match. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…no tier row
- A check reads as one sentence from its description and provenance
("Runs the overdraft refusal test (a contract check) from tests/test_core.py."); its ID is only in the trailing ref.
- The export check now finds glued fields: a name said twice in one sentence, and a colon-joined list of fields. The check block no longer skips the copy checks.
- "At the PR head", "Output at the PR head" and "Open these lines at the PR head" become "At <sha>: passed", "Output" and "Open these lines on GitHub" (the branch name for a ref on another branch).
- The "Strongest check" tier row is dropped: the Evidence row names the strongest check and its result.
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
theclaymethod
marked this pull request as ready for review
October 12, 2026 02:58
# Conflicts: # plugins/visual-explainer/assets/runtime.json.gz
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.
The trust-map explainer now explains verification only. The review flow is gone: the checklist, the merge verdict, the approval checks, the review progress marks and the PR-section export.
Changes
trust-map.mjs(the PR-section export); and UnderstandingCheck (the general Quiz stays). About 2,830 lines fewer.verification.tsx.cards/trust-map.mdis now a short "Verification explainer" card. It opens with a worked example, then gives the minimal contract.chapternow links to#<id>.Verification
npm run checkOpen item: DESIGN.md still names the old fonts. That mismatch was there before this change.
🤖 Generated with Claude Code