Skip to content

Conversation

@Teagan42
Copy link
Contributor

@Teagan42 Teagan42 commented Oct 7, 2025

What

  • prevent negative loader delays via the CLI and runtime guard
  • add regression tests covering CLI and programmatic usage, including explicit CLI exit-code assertions
  • clarify the runtime error message and bump the project version to 2.0.6

Why

  • negative delays previously crashed in asyncio.sleep; these guards catch the issue earlier with a clear error

Affects

  • loader CLI delay option and the load_media orchestrator

Testing

  • uv run ruff check .
  • uv run pytest

Documentation

  • not needed

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

@github-actions
Copy link

github-actions bot commented Oct 7, 2025

Coverage

Coverage Report
FileStmtsMissCoverMissing
mcp_plex/common
   types.py166199%168
   validation.py28293%37–38
mcp_plex/loader
   __init__.py1791592%89–96, 138, 205, 320–322, 324, 410–429
   __main__.py440%2–8
   imdb_cache.py43295%59, 63
   qdrant.py198299%335–340
   samples.py41393%54–56
mcp_plex/loader/pipeline
   channels.py64297%134–135
   enrichment.py5207586%240, 263–269, 322, 327, 358, 360, 367, 371, 475, 496, 516, 524–526, 529–531, 534–536, 544–549, 557, 611–612, 636, 696–697, 713, 742, 749, 751, 755, 807, 825, 837, 866, 872, 875–883, 909, 912–914, 931–933, 953–956, 960, 974–980, 1002, 1004, 1007–1009
   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__.py3512892%55–56, 136–137, 185, 202, 240, 261–264, 278–281, 289, 363, 366, 404, 422–423, 461, 491, 509, 515, 533, 632–634, 638
   __main__.py440%3–8
   cli.py62198%73
   config.py821285%58, 61–69, 76, 82, 102, 116
   media.py85495%62, 84–85, 95
mcp_plex/server/tools
   media_library.py2891196%477, 760, 783–791
TOTAL269719393% 

Tests Skipped Failures Errors Time
159 0 💤 0 ❌ 0 🔥 1m 19s ⏱️

@Teagan42 Teagan42 merged commit 2089e79 into main Oct 7, 2025
4 checks passed
@Teagan42 Teagan42 deleted the codex/update-delay-validation-in-loader branch October 7, 2025 09:32
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