Skip to content

Conversation

@Teagan42
Copy link
Contributor

@Teagan42 Teagan42 commented Oct 7, 2025

What

  • store media payloads in the cache under the lookup identifier plus Plex, IMDb, TMDb, and TVDb IDs when available
  • add coverage to ensure repeated lookups by external identifiers reuse the cache instead of calling _find_records
  • bump the project version to 1.0.9 and refresh the uv lock file

Why

  • prevent redundant Qdrant lookups when resolving the same media via different identifiers

Affects

  • mcp_plex/server/__init__.py
  • tests/test_server.py
  • project metadata (pyproject.toml, docker/pyproject.deps.toml, uv.lock)

Testing

  • uv run pytest

Documentation

  • none required

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

@github-actions
Copy link

github-actions bot commented Oct 7, 2025

Coverage

Coverage Report
FileStmtsMissCoverMissing
mcp_plex/loader
   __init__.py3861895%80–87, 161, 251–256, 617, 732–734, 736, 822–841, 1206
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__.py6453295%52–53, 133–134, 162, 231, 237–238, 300, 304, 325–328, 345, 410, 413, 450, 468–469, 506, 1157, 1179–1185, 1221, 1239, 1244, 1262, 1386, 1423
   __main__.py440%3–8
   config.py48785%50, 52–55, 65, 76
TOTAL213215393% 

Tests Skipped Failures Errors Time
132 0 💤 0 ❌ 0 🔥 1m 4s ⏱️

@Teagan42 Teagan42 merged commit 79ecf85 into main Oct 7, 2025
4 checks passed
@Teagan42 Teagan42 deleted the codex/update-media-data-storage-and-caching-tests branch October 7, 2025 00:05
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