Skip to content

Conversation

@Teagan42
Copy link
Contributor

@Teagan42 Teagan42 commented Oct 7, 2025

What

  • wrap the loader's Qdrant client usage in a try/finally so it always closes after each run
  • extend the integration test harness to track client shutdowns and assert a single close per run
  • bump the project version to 2.0.7 and sync dependency manifests

Why

  • ensure Qdrant client resources are released even when runs fail, and verify the behavior with automated coverage

Affects

  • loader pipeline orchestration and loader integration tests

Testing

  • uv run pytest tests/test_loader_integration.py

Documentation

  • not required

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

@Teagan42 Teagan42 force-pushed the codex/wrap-client-closure-in-try/finally branch from f5ca040 to 3149c68 Compare October 7, 2025 09:45
@github-actions
Copy link

github-actions bot commented Oct 7, 2025

Coverage

Coverage Report
FileStmtsMissCoverMissing
mcp_plex/common
   cache.py39197%27
   types.py166199%168
   validation.py28293%37–38
mcp_plex/loader
   __init__.py1821592%89–96, 138, 206, 321–323, 325, 412–431
   __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__.py35111767%55–56, 136–137, 185, 202, 213–214, 220–229, 235–242, 248–333, 363, 366, 370, 397–423, 457–467, 478, 491, 509, 515, 533, 549, 570, 632–634, 638
   __main__.py440%3–8
   cli.py62198%73
   config.py821285%58, 61–69, 76, 82, 102, 116
   media.py854053%25–26, 62, 66, 69, 84–85, 95, 105, 107–145
mcp_plex/server/tools
   media_library.py2896578%36, 477, 498–501, 543–559, 597, 618–635, 676–681, 693, 712–713, 727–739, 755, 757–765, 779–791, 805–809
TOTAL270037386% 

Tests Skipped Failures Errors Time
160 0 💤 13 ❌ 0 🔥 1m 29s ⏱️

@Teagan42 Teagan42 merged commit 89f6a1b into main Oct 7, 2025
4 checks passed
@Teagan42 Teagan42 deleted the codex/wrap-client-closure-in-try/finally branch October 7, 2025 09:48
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