Fix two pre-existing stale governing-spec-ID constants#799
Merged
Conversation
enricopiovesan
force-pushed
the
claude/fix-stale-spec-ids-graph-federation
branch
from
July 21, 2026 21:24
ded7ece to
7f902d1
Compare
graph.rs's METADATA_GRAPH_GOVERNING_SPEC and federation.rs's test fixtures referenced spec IDs (015-metadata-graph, 026-federation- registry-routing) that don't exist in approved-specs.json — both slots were reassigned to unrelated specs (015-capability-discovery- mcp, 026-event-broker) during the v0.2.0 batch, so approved_spec_registry_contains() has silently returned false for both since that batch landed, independent of any later work. Investigation found real spec content had been written for both but never merged into approved-specs.json before their slots were taken. Rather than repoint the constants at an existing-but- unrelated approved spec (the same category of error), land the two specs under fresh IDs: - 077-metadata-graph: written retroactively against graph.rs's shipped behavior (no committed draft of the original existed on any branch). - 078-federation-registry-routing: the original spec document, recovered from an abandoned branch (b81a17b), reviewed against federation.rs/federation_operator.rs and found still accurate. See docs/decision-log.md Decision 33 for full rationale and alternatives considered. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
enricopiovesan
force-pushed
the
claude/fix-stale-spec-ids-graph-federation
branch
from
July 22, 2026 18:39
7f902d1 to
363aab6
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.
Summary
graph.rs'sMETADATA_GRAPH_GOVERNING_SPECandfederation.rs/federation_operator.rs's test fixtures referenced spec IDs (015-metadata-graph,026-federation-registry-routing) that don't exist inapproved-specs.json— both numeric slots were reassigned to unrelated specs (015-capability-discovery-mcp,026-event-broker) during the v0.2.0 governance batch, soapproved_spec_registry_contains()has silently returnedfalsefor both since that batch landed, independent of any later work (originally surfaced while auditingtraverse-registryahead of thetraverse-framework/registryextraction — see that repo'sspecs/014-extraction-compatibilitydecision-log entry 33).origin/022-mcp-wasm-servercommitb81a17b) found real, complete spec content had been written for both features but never merged intoapproved-specs.jsonbefore their numeric slots were taken by different work.077-metadata-graph: written retroactively againstgraph.rs's shipped behavior (no committed draft of the original015-metadata-graphexists on any branch).078-federation-registry-routing: the original spec document, recovered intact fromb81a17b, reviewed againstfederation.rs/federation_operator.rsand found still accurate — landed largely verbatim, with its own stale cross-references to two other still-unapproved draft specs removed.docs/decision-log.mdDecision 33.Governing Spec
077-metadata-graph078-federation-registry-routing070-runtime-event-sink-boundaryProject Item
N/A — ad hoc governance-hygiene fix (not tracked against a Project board item), authorized directly by the repo owner as a one-off cross-repo exception from
traverse-framework/registry's normal registry-only scope.Validation
cargo build -p traverse-registry -p traverse-cli-rscargo test -p traverse-registry -p traverse-cli-rs— 586 tests, 0 failurescargo clippy -p traverse-registry -p traverse-cli-rs --all-targets— cleancargo fmt -p traverse-registry -p traverse-cli-rs -- --check— cleanBASE_SHA=<merge-base> bash scripts/ci/spec_alignment_check.sh— passed locallybash scripts/ci/pr_body_check.sh— passed locallyid/pathvalues and allapproved-specs.jsonpaths resolve to existing files