Skip to content

refactor: route planner type lookup through resolver - #830

Merged
ktjn merged 1 commit into
mainfrom
roadmap/phase2-planner-resolution
Sep 4, 2026
Merged

refactor: route planner type lookup through resolver#830
ktjn merged 1 commit into
mainfrom
roadmap/phase2-planner-resolution

Conversation

@ktjn

@ktjnktjn commented Sep 4, 2026

Copy link
Copy Markdown
Owner

Summary

  • route planner enum and named-type lookup through the common declaration resolver
  • preserve exact-version handling, semantic-only filtering, model fallback, and plan output

Validation

  • uv run ruff format .
  • uv run ruff check .
  • uv run python ../.github/scripts/check_mypy_baseline.py --baseline mypy-baseline.txt -- uv run mypy src/modelable --no-error-summary --show-error-codes
  • planner/browser/semantic focused tests (18 passed)
  • full suite serially: 3155 passed, 67 skipped

Reviewed by subagent: no findings.

@ktjn
ktjn merged commit 7affe73 into mainSep 4, 2026
9 checks passed
@ktjn
ktjn deleted the roadmap/phase2-planner-resolution branch September 4, 2026 18:33
Sign up for freeto 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

@ktjn