Skip to content

refactor(retrieval): clean up legacy MapNav contracts - #391

Merged
EricNGOntos merged 3 commits into
mainfrom
refactor/wuchengke/cleanup-mapnav-agent-corpus
Sep 8, 2026
Merged

EricNGOntos merged 3 commits into
mainfrom
refactor/wuchengke/cleanup-mapnav-agent-corpus

Conversation

@EricNGOntos

Copy link
Copy Markdown
Contributor

Summary

  • remove unused legacy MapNav configuration and projection fields
  • simplify navigation filters, actions, and trace mapping with focused test updates
  • document the planned agent corpus schema and mark the current MapNav path as legacy

Verification

  • make check
  • uv run --all-packages pytest packages/shared-python/shared/tests/test_nav_bridge_config.py packages/shared-python/shared/tests/test_nav_node_filter.py packages/shared-python/shared/tests/test_nav_node_filter_wire.py packages/shared-python/shared/tests/test_nav_projection_prod.py packages/shared-python/shared/tests/test_nav_scope_filter.py packages/shared-python/shared/tests/test_nav_trace_map.py (25 passed)

…sed parameters

Removed hard-coded parameters from the navigation configuration, including budget modes and projection limits. Updated related functions to streamline the navigation process and improve clarity. Adjusted tests to reflect these changes and ensure functionality remains intact.
…lean up code

Removed the map_mode parameter from the navigation configuration and adjusted related functions to reflect this change. Increased max_dispatch_depth and max_harvest_depth values for improved navigation performance. Updated tests to ensure consistency with the new configuration settings.
@EricNGOntos
EricNGOntos merged commit ac089d3 into main Sep 8, 2026
6 checks passed
@EricNGOntos
EricNGOntos deleted the refactor/wuchengke/cleanup-mapnav-agent-corpus branch September 8, 2026 03:43
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