Skip to content

Conversation

@Teagan42
Copy link
Contributor

@Teagan42 Teagan42 commented Oct 6, 2025

What

  • import plexapi.base.PlexPartialObject without exposing a PartialPlexObject alias in the loader package and update enrichment helpers to use the canonical name
  • tighten the loader import test to ensure the alias is absent and refresh documentation references
  • bump the project version to 1.0.5 and refresh the dependency manifests

Why

  • ensures downstream code relies on the real Plex API class name and avoids reviving the deprecated alias

Affects

  • loader module exports, enrichment type hints, loader unit test, and loader pipeline plan documentation

Testing

  • uv run ruff check .
  • uv run pytest

Documentation

  • updated loader multiworker plan to reference PlexPartialObject

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

@github-actions
Copy link

github-actions bot commented Oct 6, 2025

Coverage

Coverage Report
FileStmtsMissCoverMissing
mcp_plex/loader
   __init__.py3851895%81–88, 162, 252–257, 618, 733–735, 737, 823–842, 1195
mcp_plex/loader/pipeline
   __init__.py16756%57–62, 68
   enrichment.py4386086%244, 249, 280, 282, 289, 293, 397, 418, 438, 446–448, 455–458, 461–463, 471, 534, 590, 613, 617, 619, 649, 667, 679, 708, 714, 717–725, 751, 754–756, 773–775, 795–798, 802, 816–822, 844, 846, 849–851
   ingestion.py1051190%78, 127–147, 174, 203, 209, 240
   orchestrator.py92595%48, 126, 155, 178–179
   persistence.py129993%112, 154–155, 165, 176, 180–182, 231
mcp_plex/server
   __init__.py6142995%43–44, 119–120, 148, 252, 256, 277–280, 297, 362, 365, 402, 420–421, 458, 1109, 1131–1137, 1173, 1191, 1196, 1214, 1338, 1375
   __main__.py440%3–8
   config.py48785%50, 52–55, 65, 76
TOTAL210015093% 

Tests Skipped Failures Errors Time
130 0 💤 0 ❌ 0 🔥 52.819s ⏱️

@Teagan42 Teagan42 merged commit 17471a1 into main Oct 6, 2025
4 checks passed
@Teagan42 Teagan42 deleted the codex/refactor-plexapi-imports-and-typing branch October 6, 2025 23:27
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