Skip to content

feat: add L7 graph sync validator#84

Merged
ChBrain merged 2 commits into
mainfrom
feat/L7-graph-sync
May 6, 2026
Merged

feat: add L7 graph sync validator#84
ChBrain merged 2 commits into
mainfrom
feat/L7-graph-sync

Conversation

@ChBrain

@ChBrain ChBrain commented May 6, 2026

Copy link
Copy Markdown
Owner

L7 - Graph Sync Validator

Implements bidirectional edge consistency checks for the road network graph.

Changes

New validator: scripts/validate_sync.py

  • Parses all road indexes (Holds km chain)
  • Parses all node Owner blocks (Place: road in state)
  • Checks both directions:
    • node claims road Rroad R lists this node
    • node claims state Sroad R lists node under state S subsection
  • Runs independently, re-derives graph from scratch each time

CI Integration: L7-graph-sync job in validate.yml

  • Depends on L3, L4, L5 (state, road, node validators)
  • Only runs when place files change

Content Fixes:

  • A25: regex now accepts [km TBD] placeholders (5 nodes already in indexes)
  • A7 elbtunnel:
    • Index entry format fixed: * [km 156](piece_the_kilometerstein.md):...
    • Holds reformatted with standard km marker
    • Distance corrected (0.5 km to Hamburg-Othmarschen)
  • A111: tunnel added to index with km TBD
  • A40: stretch added to index with km TBD
  • A1: two bridges (Volme, Hengstey) added to index with km TBD

Documentation: tests/README.md

  • Renumbered layers: L6 = deploy bundles, L7 = graph test, L8 = content samples
  • Graph test description updated to reflect implementation

Graph Validation Results

✓ 306 nodes checked, all consistent

Next Steps

  • Merge when L7 passes CI
  • Consider L8 extension (content samples beyond Withheld section)

@ChBrain
ChBrain merged commit 214bde8 into main May 6, 2026
11 checks passed
@ChBrain
ChBrain deleted the feat/L7-graph-sync branch May 6, 2026 18:51
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