Skip to content

Conversation

@Teagan42
Copy link
Contributor

@Teagan42 Teagan42 commented Oct 5, 2025

What

  • add an enrichment stage scaffold that wires the ingest and persistence queues
  • provide placeholder handlers for movie, episode, and sample batches and expose the injected IMDb retry queue
  • cover the new stage with tests for logging, completion handling, and retry queue wiring

Why

  • downstream loader work needs a concrete enrichment stage to integrate with before the full metadata logic is ported

Affects

  • loader pipeline enrichment stage and associated unit tests

Testing

  • uv run ruff check mcp_plex/loader/pipeline/enrichment.py tests/test_enrichment_stage.py
  • uv run pytest tests/test_enrichment_stage.py

Documentation

  • not needed

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

@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
   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
TOTAL189418190% 

Tests Skipped Failures Errors Time
110 0 💤 0 ❌ 0 🔥 56.761s ⏱️

@Teagan42 Teagan42 merged commit 47a68f4 into main Oct 5, 2025
4 checks passed
@Teagan42 Teagan42 deleted the codex/define-enrichmentstage-class-in-enrichment.py branch October 5, 2025 18:30
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