Skip to content

Conversation

@Teagan42
Copy link
Contributor

@Teagan42 Teagan42 commented Oct 5, 2025

Summary

  • port the movie batch handler and metadata aggregation from the legacy loader into the new pipeline enrichment stage
  • swap global HTTP client usage for the injected factory to bundle external requests safely and emit persistence batches
  • extend enrichment stage tests to cover TMDb/IMDb lookups, missing ID handling, and queue emission while bumping the project version

Why

  • the enrichment stage needed real movie metadata enrichment logic and must avoid the legacy global HTTP client
  • ensuring batching via the injected client keeps external calls rate-limit friendly and unblocks downstream persistence work

Affects

  • loader pipeline enrichment stage implementation and its unit tests
  • project metadata versions (pyproject, docker manifest, uv.lock)

Testing

  • uv run ruff check .
  • uv run pytest

Documentation

  • Updated module docstring; no additional documentation changes required

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

@github-actions
Copy link

github-actions bot commented Oct 5, 2025

Coverage

Coverage Report
FileStmtsMissCoverMissing
mcp_plex/loader
   __init__.py81613184%192, 247, 249–251, 267, 353–358, 960–961, 1005, 1007, 1060–1061, 1072–1074, 1077, 1080–1081, 1083, 1105, 1115, 1126–1158, 1173, 1175, 1182–1196, 1199–1213, 1219, 1234–1265, 1270–1303, 1306–1310, 1315–1323, 1333, 1342, 1346–1348, 1467–1469, 1802
mcp_plex/loader/pipeline
   channels.py70297%19–20
   enrichment.py1093370%87–90, 100–103, 122, 132–139, 147, 154–172, 180, 222–227
   ingestion.py81890%67, 98–108, 129, 157, 179
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
TOTAL195221489% 

Tests Skipped Failures Errors Time
111 0 💤 0 ❌ 0 🔥 55.660s ⏱️

@Teagan42 Teagan42 merged commit 5eb61a6 into main Oct 5, 2025
4 checks passed
@Teagan42 Teagan42 deleted the codex/refactor-movie-enrichment-functions-and-tests branch October 5, 2025 19: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