Skip to content

refactor: extract shared route-result assembly#24

Merged
blackopsrepl merged 1 commit intomainfrom
issue/11-extract-shared-route-assembly
Mar 21, 2026
Merged

refactor: extract shared route-result assembly#24
blackopsrepl merged 1 commit intomainfrom
issue/11-extract-shared-route-assembly

Conversation

@blackopsrepl
Copy link
Copy Markdown
Contributor

Summary

route_snapped and route_with still duplicated the same node-path assembly logic for geometry and totals, and route_edge_snapped had its own copy of the same path traversal work.

This refactor extracts shared helpers for:

  • node-path geometry assembly
  • node-path distance/time aggregation
  • standard route-result construction for time-based node routing

The shared helpers are reused by route_snapped and route_with, and route_edge_snapped now reuses the same geometry and path-metric helpers for its internal graph path portion.

Validation

  • cargo fmt --all -- --check
  • cargo clippy --workspace --all-targets -- -D warnings
  • cargo test

Fixes #11

@blackopsrepl blackopsrepl merged commit e437d21 into main Mar 21, 2026
3 checks passed
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.

refactor: extract shared route-result assembly from path-based routing methods

1 participant