Skip to content

Conversation

@Teagan42
Copy link
Contributor

@Teagan42 Teagan42 commented Oct 5, 2025

What

  • batch sample-mode ingestion in the pipeline stage and expose observable counters
  • add async coverage for empty, partial, and backpressure scenarios in the ingestion stage tests
  • bump the package version to 0.26.63 (and docker manifest) to reflect the new behavior

Why

  • moves batching logic from the legacy loader into the pipeline stage so the queue receives sized batches and observability counters
  • exercises the new batching and backpressure logic to prevent regressions

Affects

  • loader pipeline ingestion stage and its unit tests
  • project version metadata and lockfile

Testing

  • uv run pytest

Documentation

  • not needed

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

@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.py51688%60, 91–95, 105
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
TOTAL181317990% 

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

@Teagan42 Teagan42 merged commit 3ae0ebc into main Oct 5, 2025
4 checks passed
@Teagan42 Teagan42 deleted the codex/refactor-ingestion-operations-into-separate-method branch October 5, 2025 17:47
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