Skip to content

fix: align mapnav production projection with EXP - #254

Merged
EricNGOntos merged 1 commit into
mainfrom
feat/wuchengke/2026-08-11
Aug 11, 2026
Merged

fix: align mapnav production projection with EXP#254
EricNGOntos merged 1 commit into
mainfrom
feat/wuchengke/2026-08-11

Conversation

@EricNGOntos

Copy link
Copy Markdown
Contributor

Summary

  • Fall back to provider structure.summary/preview for inline map summaries when section_summary_store is absent (Knowhere production).
  • Make map_mode config-authoritative so env cannot override production NavConfig.
  • Remove dead agentic leftovers (llm_adapter, old LLM mocks, MCP use_agentic=None, unused trace record_step).

Test plan

  • pytest packages/shared-python/shared/tests/test_nav_projection_prod.py
  • CI lint / typecheck / contract suite
  • Spot-check DISPATCH child map shows summary: from DB structure

Made with Cursor

…c leftovers

Fall back to provider structure summaries when the section store is absent,
make map_mode config-authoritative, and remove unused llm_adapter/old mocks.

Co-authored-by: Cursor <cursoragent@cursor.com>
text = str(_store_get_summary(sid, doc_id=str(doc_id or "")) or "").strip()
if text:
return text
except Exception:
@EricNGOntos
EricNGOntos merged commit 61b6687 into main Aug 11, 2026
6 checks passed
@EricNGOntos
EricNGOntos deleted the feat/wuchengke/2026-08-11 branch August 11, 2026 08:06
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.

2 participants