Skip to content

Conversation

@Teagan42
Copy link
Contributor

@Teagan42 Teagan42 commented Oct 6, 2025

What

  • add detailed lifecycle logging across the loader orchestrator and ingestion, enrichment, and persistence stages
  • extend loader logging tests to assert the new instrumentation and cover additional ingestion stage messages
  • bump the project version to 1.0.2 to capture the new logging capabilities

Why

  • richer pipeline visibility makes it easier to diagnose why staged loads finish without producing items

Affects

  • loader pipeline logging, associated unit tests, and version metadata for release tracking

Testing

  • uv run ruff check .
  • uv run pytest

Documentation

  • not required for logging-only changes

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

@github-actions
Copy link

github-actions bot commented Oct 6, 2025

Coverage

Coverage Report
FileStmtsMissCoverMissing
mcp_plex/loader
   __init__.py3881895%84–91, 165, 255–260, 621, 736–738, 740, 826–845, 1198
mcp_plex/loader/pipeline
   __init__.py16756%57–62, 68
   channels.py59297%20–21
   enrichment.py4396086%248, 253, 284, 286, 293, 297, 401, 422, 442, 450–452, 459–462, 465–467, 475, 538, 594, 617, 621, 623, 653, 671, 683, 712, 718, 721–729, 755, 758–760, 777–779, 799–802, 806, 820–826, 848, 850, 853–855
   ingestion.py1031189%74, 123–143, 169, 200, 206, 237
   orchestrator.py92595%48, 126, 155, 178–179
   persistence.py129993%112, 154–155, 165, 176, 180–182, 231
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
TOTAL210315293% 

Tests Skipped Failures Errors Time
130 0 💤 0 ❌ 0 🔥 48.618s ⏱️

@Teagan42 Teagan42 merged commit 3b29182 into main Oct 6, 2025
4 checks passed
@Teagan42 Teagan42 deleted the codex/add-detailed-logging-to-pipeline branch October 6, 2025 08:31
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