Skip to content

Conversation

@Teagan42
Copy link
Contributor

@Teagan42 Teagan42 commented Oct 5, 2025

What

  • update mcp_plex.loader to document the orchestrator-first design and import LoaderOrchestrator
  • expose LegacyLoaderPipeline for the CLI, provide a deprecated LoaderPipeline alias via __getattr__, and update codepaths to use the legacy name
  • refresh tests to consume the legacy class explicitly and add coverage ensuring the deprecated import warns while returning the orchestrator
  • bump the project version metadata and regenerate uv.lock

Why

  • align the loader package’s public API with the new orchestrator-centric implementation without breaking existing imports

Affects

  • loader package exports, CLI wiring, backwards-compatibility test coverage, and version metadata

Testing

  • uv run ruff check .
  • uv run pytest

Documentation

  • Updated the loader module docstring; no separate docs required

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

@github-actions
Copy link

github-actions bot commented Oct 5, 2025

Coverage

Coverage Report
FileStmtsMissCoverMissing
mcp_plex/loader
   __init__.py75512983%182–189, 238, 324–329, 931–932, 976, 978, 1043–1044, 1055–1057, 1060, 1063–1064, 1066, 1088, 1098, 1109–1141, 1156, 1158, 1165–1179, 1182–1196, 1202, 1217–1248, 1253–1286, 1289–1293, 1298–1306, 1316, 1407–1409, 1470–1472, 1830
mcp_plex/loader/pipeline
   channels.py73297%19–20
   enrichment.py3344487%78, 80, 87, 91, 134–137, 170, 191, 211, 219–221, 228–231, 234–236, 244, 301, 357, 380, 384, 386, 412, 430, 465, 471, 474–482, 508, 532, 603, 606–608
   ingestion.py81890%67, 98–108, 129, 157, 179
   orchestrator.py85594%48, 112, 141, 164–165
   persistence.py117992%109, 151–152, 162, 167, 171–173, 201
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
TOTAL232123790% 

Tests Skipped Failures Errors Time
128 0 💤 0 ❌ 0 🔥 52.679s ⏱️

@Teagan42 Teagan42 merged commit ac03c00 into main Oct 5, 2025
4 checks passed
@Teagan42 Teagan42 deleted the codex/update-loaderpipeline-import-and-documentation branch October 5, 2025 23:04
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