Skip to content

Conversation

@Teagan42
Copy link
Contributor

@Teagan42 Teagan42 commented Oct 7, 2025

What

  • remove the maxsize constraint from the loader ingestion queue
  • update the ingestion stage test expectations for the unbounded queue
  • bump the project version to 1.0.17 and refresh the lock file

Why

  • unblock ingestion pipelines by avoiding artificial queue limits in the orchestrator

Affects

  • loader queue orchestration and associated ingestion stage tests

Testing

  • uv run ruff check .
  • uv run pytest

Documentation

  • not required

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

@Teagan42 Teagan42 force-pushed the codex/remove-queue-size-limitations branch from 50f7f7b to c2fa374 Compare October 7, 2025 03:44
@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, 633, 748–750, 752, 838–857, 1222
mcp_plex/loader/pipeline
   __init__.py16756%57–62, 68
   channels.py63297%135–136
   enrichment.py4866686%206, 228–234, 287, 292, 323, 325, 332, 336, 440, 461, 481, 489–491, 498–501, 504–506, 514, 577, 630, 656, 663, 665, 669, 721, 739, 751, 780, 786, 789–797, 823, 826–828, 845–847, 867–870, 874, 888–894, 916, 918, 921–923
   ingestion.py1551392%80, 129–149, 187, 224, 240, 333, 339, 359
   orchestrator.py92595%48, 126, 155, 178–179
   persistence.py129993%113, 155–156, 166, 177, 181–183, 236
mcp_plex/server
   __init__.py6453195%52–53, 133–134, 162, 231, 237–238, 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
TOTAL225616293% 

Tests Skipped Failures Errors Time
144 0 💤 0 ❌ 0 🔥 1m 10s ⏱️

@Teagan42 Teagan42 merged commit 57820f9 into main Oct 7, 2025
4 checks passed
@Teagan42 Teagan42 deleted the codex/remove-queue-size-limitations branch October 7, 2025 03:46
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