docs: retire the knowledge-graph plan into the rule graph - #755
Merged
Conversation
The plan was a record, not a work item. Its two load-bearing pieces move to docs/rules/README.md, which loads on demand: the rule-number-to-node mapping and the retrieval-check probes. Its remaining reasoning was already folded into feature-matrix, subscription-consumer-idiom, exercise-production-code, and fixture-builders; the rest is in git log. Also prunes plans/code-consistency-followups.md to what is still open — the # Examples track closed in #751 and the wsh_event_data builders in #752. The four internal param-budget sites and both market_data realtime-seam items were re-verified against main; their stale file.rs:NNN citations are now symbol names.
wboayue
force-pushed
the
docs/prune-knowledge-graph-plan
branch
from
August 9, 2026 04:08
d18ec4d to
6540a5d
Compare
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.
Reviewed
plans/and removed what was no longer a work item.plans/claude-md-knowledge-graph.md— deletedIt described itself as complete: "Every follow-up is closed; nothing here is a work item." Two parts of it were still referenced, so they move to
docs/rules/README.md, which loads on demand rather than every session:## Retired rule numbers— the 27-row rule→node mapping, plus the caveat that numbering shifted mid-use, so an old "rule N" citation resolves against theCLAUDE.mdof its own date.## Retrieval check— the three probes, the fail criterion, the 2026-08-06 result, and the warning that the file listing the probes also lists their answers.The rest — the audit narrative, the prose→gates table, the PR-by-PR log — is history recoverable from
git log. Its durable lessons already exist as obligations inCLAUDE.md's Maintaining the rule graph, and the four nodes below absorbed the node-specific reasoning.References updated:
CLAUDE.mdnow points atdocs/rules/README.md#retired-rule-numbers;plans/code-consistency-followups.mdcitesdocs/rules/directly.Four nodes absorbed the reasoning worth keeping
parity/feature-matrix— three green legs don't mean the two sides agree; two sibling tests with the same name and contradictory assertions are a drift report (fix(historical): async retried the wrong case; both sides share the retry helper #744).parity/subscription-consumer-idiom— aClientmethod collecting withwhile let Some(Ok(..))turns a routed error into an emptyVec; no shared layer answers this (fix(contracts): propagate routed errors from blocking matching_symbols #735).testing/exercise-production-code— read a dead arm before deleting it; a mock hosting a copy of the code under test (refactor(subscriptions): drop dispatcher-intercepted RESPONSE_MESSAGE_IDS entries #734, fix(contracts): propagate routed errors from blocking matching_symbols #735, test(orders): move the submit family off its mock-client shadows #750).testing/fixture-builders— a fixture field no assertion reads is unverified input;with_connection_resets(test(stubs): inject Error::ConnectionReset so retry wiring is testable #743, test(wire): one assertion for every rejects-text-framing test #747).plans/code-consistency-followups.md— pruned, 1,190 → ~580 wordsDeleted the closed tracks: the
# Examplessection (#751, with its count table and per-method list) and thewsh_event_data_*write-up (#752).Kept and re-verified against
maintoday:option_chainandhistorical_newssignatures unchanged;market_datarealtime-seam items —src/market_data/builder/still sits a directory above its three siblings, andpub fn market_datainrealtime/sync.rsis stillpubwhererealtime_bars/market_depth/tick_by_tickarepub(crate).Their
file.rs:NNNcitations became symbol names, per the same decay argument the validator enforces inside nodes.plans/tick-by-tick-reconnect-decode-desync.mdis untouched — open investigation, no fix in the log orCHANGELOG.md.Checks
Docs-only; no
.rsfiles touched.just rules-checkpasses — 33 nodes, all links and edges resolve.