Skip to content

Conversation

@Teagan42
Copy link
Contributor

@Teagan42 Teagan42 commented Oct 7, 2025

Summary

  • add reusable JSONValue aliases for shared JSON payload structures and expose them from the common package
  • refactor MediaCache, server helpers, and loader cache integrations to consume typed payload dictionaries and update validation/tests for stricter Plex tag coercion
  • document the new typing expectations and bump the project version metadata

Testing

  • uv run ruff check .
  • uv run pytest

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

@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__.py4071896%85–92, 166, 256–261, 663, 778–780, 782, 868–887, 1252
   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__.py6523295%53–54, 134–135, 163, 213, 239, 245–246, 315, 336–339, 356, 421, 424, 461, 479–480, 517, 1191, 1213–1219, 1255, 1273, 1278, 1296, 1420, 1457
   __main__.py440%3–8
   config.py48785%50, 52–55, 65, 76
TOTAL234116993% 

Tests Skipped Failures Errors Time
149 0 💤 0 ❌ 0 🔥 1m 8s ⏱️

@Teagan42 Teagan42 merged commit a888df4 into main Oct 7, 2025
4 checks passed
@Teagan42 Teagan42 deleted the codex/introduce-reusable-jsonvalue-type-alias branch October 7, 2025 04:34
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