Skip to content

Conversation

@Teagan42
Copy link
Contributor

@Teagan42 Teagan42 commented Oct 5, 2025

What

  • add shared ingestion sentinel and queue type aliases in the loader pipeline helpers
  • update LoaderPipeline to rely on the shared queue aliases and accept the new sentinel alongside existing None tokens
  • document the sentinel usage for upcoming integration coverage

Why

  • centralizing the queue types and sentinel improves consistency between pipeline stages and prepares later stages for sentinel-based coordination without breaking current behavior

Affects

  • loader pipeline queue typing and completion signaling

Testing

  • uv run pytest tests/test_loader_unit.py
  • uv run ruff check mcp_plex/loader/pipeline/channels.py mcp_plex/loader/__init__.py

Documentation

  • updated inline module documentation for the loader pipeline queues

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

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

Tests Skipped Failures Errors Time
101 0 💤 0 ❌ 0 🔥 56.296s ⏱️

@Teagan42 Teagan42 merged commit 591d658 into main Oct 5, 2025
4 checks passed
@Teagan42 Teagan42 deleted the codex/extend-channels.py-with-type-aliases-and-sentinel branch October 5, 2025 17:28
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