Skip to content

Conversation

@Teagan42
Copy link
Contributor

@Teagan42 Teagan42 commented Oct 6, 2025

What

  • replace mutable loader globals with runtime configuration objects and updated helpers
  • drop the private channel aliases and teach ingestion to use the shared types directly
  • update loader tests and helpers to pass explicit configuration and bump the project version

Why

  • keep configuration near its usage so IMDb and Qdrant helpers can be reused without global mutation

Affects

  • loader runtime configuration, pipeline channels/ingestion, loader unit/integration tests

Testing

  • uv run ruff check .
  • uv run pytest

Documentation

  • not needed

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

@github-actions
Copy link

github-actions bot commented Oct 6, 2025

Coverage

Coverage Report
FileStmtsMissCoverMissing
mcp_plex/loader
   __init__.py3871895%84–91, 165, 255–260, 621, 736–738, 740, 826–845, 1198
mcp_plex/loader/pipeline
   __init__.py15753%57–62, 68
   channels.py62297%19–20
   enrichment.py4306086%248, 253, 284, 286, 293, 297, 376, 397, 417, 425–427, 434–437, 440–442, 450, 507, 563, 586, 590, 592, 618, 636, 648, 677, 683, 686–694, 720, 723–725, 742–744, 764–767, 771, 785–791, 813, 815, 818–820
   ingestion.py81890%66, 97–107, 128, 156, 178
   orchestrator.py85594%48, 112, 141, 164–165
   persistence.py124993%112, 154–155, 165, 170, 174–176, 214
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
TOTAL205214993% 

Tests Skipped Failures Errors Time
120 0 💤 0 ❌ 0 🔥 47.807s ⏱️

@Teagan42 Teagan42 merged commit 2590a85 into main Oct 6, 2025
4 checks passed
@Teagan42 Teagan42 deleted the codex/refactor-loader-globals-and-remove-aliases branch October 6, 2025 05:56
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