Skip to content

Conversation

@Teagan42
Copy link
Contributor

@Teagan42 Teagan42 commented Oct 7, 2025

What

  • narrow the server request modelling helpers to avoid Any, use explicit parameter metadata, and shrink identifier normalization inputs
  • rebuild the OpenAPI stub generation so REST and prompt routes expose typed payload signatures without depending on Any
  • introduce a typed run configuration helper for CLI transports and exercise the new behaviour in unit tests

Why

  • cleaning up loose typing around server glue helps static analysis, avoids runtime evaluation errors, and clarifies OpenAPI payloads

Affects

  • MCP server REST/OpenAPI plumbing
  • CLI transport configuration
  • accompanying tests and dependency metadata

Testing

  • uv run ruff check .
  • uv run pytest

Documentation

  • not needed; behaviour is covered by tests

https://chatgpt.com/codex/tasks/task_e_68e4a6621b0c8328a12d53cb37fd0e12

@github-actions
Copy link

github-actions bot commented Oct 7, 2025

Coverage

Coverage Report
FileStmtsMissCoverMissing
mcp_plex/common
   validation.py28293%37–38
mcp_plex/loader
   __init__.py3741795%84–91, 165, 255–260, 662, 777–779, 781, 867–886
   __main__.py440%2–8
   imdb_cache.py43295%59, 63
mcp_plex/loader/pipeline
   channels.py64297%134–135
   enrichment.py5167585%237, 259–265, 318, 323, 354, 356, 363, 367, 471, 492, 512, 520–522, 525–527, 530–532, 540–545, 553, 607–608, 632, 692–693, 709, 738, 745, 747, 751, 803, 821, 833, 862, 868, 871–879, 905, 908–910, 927–929, 949–952, 956, 970–976, 998, 1000, 1003–1005
   ingestion.py1551392%81, 130–150, 185, 222, 238, 331, 337, 357
   orchestrator.py95595%63, 141, 170, 193–194
   persistence.py130993%122, 164–165, 175, 186, 190–192, 245
mcp_plex/server
   __init__.py8304295%164–165, 245–246, 265, 282, 332, 354–355, 365, 445, 466–469, 483–486, 494, 568, 571, 609, 627–628, 666, 1094, 1367, 1390–1398, 1435, 1453, 1459, 1477, 1624, 1664
   __main__.py440%3–8
   config.py48785%50, 52–55, 65, 76
TOTAL252418293% 

Tests Skipped Failures Errors Time
153 0 💤 0 ❌ 0 🔥 1m 12s ⏱️

@Teagan42 Teagan42 merged commit 631b325 into main Oct 7, 2025
4 checks passed
@Teagan42 Teagan42 deleted the codex/refactor-request-model-and-schema-dependencies branch October 7, 2025 05:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants