Skip to content

Conversation

@Teagan42
Copy link
Contributor

What

  • add a --log-level option to the MCP server CLI that also honors the LOG_LEVEL environment variable
  • cover the new CLI option with a regression test
  • bump the project version to 2.0.17 and sync dependent manifests

Why

  • the server needed parity with the loader so deployments can control verbosity consistently via flags or environment variables

Affects

  • MCP server CLI configuration and packaging metadata

Testing

  • uv run pytest tests/test_server_cli.py -k log_level

Documentation

  • not needed

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

@github-actions
Copy link

Coverage

Coverage Report
FileStmtsMissCoverMissing
mcp_plex/loader
   __init__.py1821194%139, 209, 323–325, 327, 416–435
   qdrant.py198299%338–343
   samples.py41393%55–57
mcp_plex/loader/pipeline
   enrichment.py5207586%234, 255–261, 314, 319, 350, 352, 359, 361, 461, 482, 502, 510–512, 515–519, 522–524, 532–537, 545, 596–597, 621, 679–680, 696, 723, 730, 732, 736, 788, 806, 818, 847, 853, 856–864, 890, 893–895, 912–914, 934–937, 941, 955–959, 981, 983, 986–988
   ingestion.py1551392%82, 133–153, 188, 225, 241, 334, 340, 358
   orchestrator.py95595%60, 136, 165, 188–189
   persistence.py130993%123, 165–166, 174, 185, 189–191, 240
mcp_plex/server
   __init__.py3952195%123, 129–132, 151, 157–159, 255, 317, 335–336, 344, 419, 513, 543, 562, 569, 588, 701
   cli.py75297%44, 96
   config.py83199%63
   media.py2502391%145, 161, 207–213, 222–225, 263–270, 293, 327, 333, 339, 357–358
mcp_plex/server/tools
   media_library.py3401296%206, 579, 677, 696, 747, 798, 869, 959–963
TOTAL299517794% 

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

@Teagan42 Teagan42 merged commit 88875cd into main Oct 13, 2025
4 checks passed
@Teagan42 Teagan42 deleted the codex/add-log-level-support-to-loader-and-server branch October 13, 2025 22: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