Skip to content

docs: retire the knowledge-graph plan into the rule graph - #755

Merged
wboayue merged 1 commit into
mainfrom
docs/prune-knowledge-graph-plan
Aug 9, 2026
Merged

docs: retire the knowledge-graph plan into the rule graph#755
wboayue merged 1 commit into
mainfrom
docs/prune-knowledge-graph-plan

Conversation

@wboayue

@wboayue wboayue commented Aug 8, 2026

Copy link
Copy Markdown
Owner

Reviewed plans/ and removed what was no longer a work item.

plans/claude-md-knowledge-graph.md — deleted

It 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 the CLAUDE.md of 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 in CLAUDE.md's Maintaining the rule graph, and the four nodes below absorbed the node-specific reasoning.

References updated: CLAUDE.md now points at docs/rules/README.md#retired-rule-numbers; plans/code-consistency-followups.md cites docs/rules/ directly.

Four nodes absorbed the reasoning worth keeping

plans/code-consistency-followups.md — pruned, 1,190 → ~580 words

Deleted the closed tracks: the # Examples section (#751, with its count table and per-method list) and the wsh_event_data_* write-up (#752).

Kept and re-verified against main today:

  • the two public param-budget deferrals — option_chain and historical_news signatures unchanged;
  • the four internal 4+-param sites — all present;
  • both market_data realtime-seam items — src/market_data/builder/ still sits a directory above its three siblings, and pub fn market_data in realtime/sync.rs is still pub where realtime_bars / market_depth / tick_by_tick are pub(crate).

Their file.rs:NNN citations became symbol names, per the same decay argument the validator enforces inside nodes.

plans/tick-by-tick-reconnect-decode-desync.md is untouched — open investigation, no fix in the log or CHANGELOG.md.

Checks

Docs-only; no .rs files touched. just rules-check passes — 33 nodes, all links and edges resolve.

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
wboayue force-pushed the docs/prune-knowledge-graph-plan branch from d18ec4d to 6540a5d Compare August 9, 2026 04:08
@wboayue wboayue changed the title docs: prune the completed knowledge-graph plan into a record docs: retire the knowledge-graph plan into the rule graph Aug 9, 2026
@wboayue
wboayue merged commit 732d1f9 into main Aug 9, 2026
4 checks passed
@wboayue
wboayue deleted the docs/prune-knowledge-graph-plan branch August 9, 2026 04:09
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.

1 participant