Skip to content

Bounded download retries, MCP 1.x/2.x support, geo acceptance - #10

Merged
galjos merged 1 commit into
mainfrom
fix/skill-retry-mcp-geo-20260921
Sep 21, 2026
Merged

galjos merged 1 commit into
mainfrom
fix/skill-retry-mcp-geo-20260921

Conversation

@galjos

@galjos galjos commented Sep 21, 2026

Copy link
Copy Markdown
Owner

Robustness and skill accuracy, no change to CLI output contracts:

  • MIMIT, comuni, and Overpass downloads retry transient failures with backoff via a shared helper, with unit tests.
  • MCP server supports SDK 1.12+ and 2.x through a version shim; mcp<2 pin lifted; CI matrix covers both majors; smoke script tolerates both field namings. Verified 105 tests plus stdio smoke on 1.12.0 and 2.2.0.
  • Skill lists all flags, the multi-fuel cheapest rule, and the CLI/MCP name map; README flag lists corrected.
  • New scripts/geo-acceptance.sh runs in weekly smoke: Bozen/Roma centers, Livo disambiguation plus exit-2 check, charger radius. Upstream outages warn distinctly from location failures.

Verified: pytest 105 passed on both majors, offline evals parse, live geo-acceptance 5/5.

MIMIT, comuni, and Overpass downloads retry transient failures with backoff through a shared helper. The MCP server imports MCPServer/FastMCP and ToolAnnotations through a version shim covering SDK 1.12+ and 2.x, so the mcp<2 pin is lifted; CI covers both majors and the smoke script tolerates both field namings. Skill documents all flags plus the CLI/MCP name map. New weekly geo-acceptance script checks Bozen/Roma centers, Livo disambiguation, and charger radius, reporting outages distinctly from location failures.
@galjos
galjos merged commit 81e4c9f into main Sep 21, 2026
6 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.

1 participant