docs: add plain-language identification-pipeline walkthrough - #295
Merged
Conversation
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
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.
Description
Adds
docs/identification-pipeline.md— the owner-approved, plain-language walkthrough of the printing-identification pipeline (Stage C evidence extraction, Stage D join-key calculator, the human-backed vote gate), as a companion todocs/theory.md(which proves the false-accept bound formally; this one explains the mechanics).docs/identification-pipeline.md, sourced from the owner-approved walkthrough text, preserved verbatim except:docs/reports/2026-07-21-recovery-arc.md), not a vote count — fixed to "12,684 gate-checked cards" rather than softened or left as-is.theory.md's intro now points to it for the plain-language version (one line, in place); the new doc points totheory.md§7 (the deduction-chain composition) and todocs/features/printing-tags.md/docs/features/catalog-completion-plan.md(the systems it feeds into).CLAUDE.md's flat docs/ index anddocs/README.md's "Understanding the system" map, both directly under thetheory.mdentry.86fca804) confirms the 99.7% short-circuit rate and the 1,552 wrongly-vetoed proxy-marker matches; Fix collector-line parser mistaking glued denominator+rarity for set code #260/Fix proxy-marker regex to match plural/passive forms #280/Add playtest family to proxy-marker regex #285/Add review-queue clustering + batch no-match confirmation API (#262) #265/Downgrade AI-Generated tag from sensitive back to standard #292 all resolve to real merged commits with matching descriptions; "Marked as proxy" tag + marker-absence moderation flags (catalog compliance scan) #291 ("Marked as proxy", cited as "planned") is confirmed open, unimplemented, matching that description exactly; the 2/20,677 ambiguous-tiebreak rate matchestheory.md§7a anddocs/reports/2026-07-21-staged-write.mdexactly.Wiki/site placement (recommendation, not unilaterally decided beyond wiki)
Added
docs/identification-pipeline.mdto.github/wiki-publish-map.jsonwithtargets: ["wiki"](wiki page nameIdentification-Pipeline), directly undertheory.md's own entry — this matches every other "Understanding the system" entry's current pattern (wiki-only) exceptoverview.md.Did NOT add a
"site"target.docs/documentation-process.mdstatestheory.mdandself-hosting.md"remain per this proposal's proposed initial mapping" for the site pipeline — i.e. they're named as next-in-line forsitebut that widening hasn't happened yet. Addingsiteto this new companion page ahead oftheory.mditself getting it would jump the owner's own stated sequencing. Recommend: if/whentheory.mdgets widened to["wiki", "site"], add this page in the same pass (e.g.sitePath: "/guide/how-cards-get-identified").Checklist
pre-commitand installed the hooks withpre-commit installbefore creating any commits.python3 .github/scripts/docs_lint.py— clean.python3 .github/scripts/tests/test_publish_wiki_link_rewrite.py— 4 tests, OK.python3 .github/scripts/tests/test_publish_readme.py— 8 tests, OK.python3 .github/scripts/publish_readme.py .regenerate-and-diff against committedreadme.md— no diff (this change doesn't touch a README-REGION source, as expected).npx prettier@2.7.1 --checkon every touched file — clean.CLAUDE.md,docs/README.md,docs/theory.md,.github/wiki-publish-map.json.)Merge-time checklist item
Identification-Pipelineneeds creating on next docs-wiki-publish run (automatic once merged —docs/identification-pipeline.mdis now in.github/wiki-publish-map.json). Owner: consider widening tositetarget together withtheory.mdper the recommendation above.