Skip to content

Conversation

@Teagan42
Copy link
Contributor

Summary

  • add a regression test that ensures the README covers live Plex loader credentials
  • document how to supply Plex and TMDb credentials when ingesting a live library
  • refactor the README documentation tests to share a helper for reading the file

Testing

  • uv run ruff check .
  • uv run pytest

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

@github-actions
Copy link

Coverage

Coverage Report
FileStmtsMissCoverMissing
mcp_plex/loader
   __init__.py1941194%148, 227, 341–343, 345, 434–453
   qdrant.py202399%343–349
   samples.py41393%55–57
mcp_plex/loader/pipeline
   enrichment.py5217386%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, 848, 854, 857–865, 891, 894–896, 913–915, 935–938, 942, 956–960, 982, 984, 987–989
   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__.py7329587%147, 153–156, 175, 181–183, 279, 338, 349, 367–368, 380, 447, 456, 460, 507, 513, 550–551, 570–571, 583–584, 647, 769–774, 785–786, 798, 801–802, 810–818, 832–833, 848–849, 851–864, 876, 883–886, 906–908, 918–919, 922, 924, 926, 933, 936–937, 944, 961, 964, 983, 986, 991, 999, 1008, 1027, 1033–1038, 1049–1050, 1079, 1098, 1105, 1124, 1237
   cli.py75297%44, 96
   config.py83199%63
   media.py3063987%93–98, 119, 123, 125, 129, 131, 136–139, 141, 206, 237, 253, 294–300, 309–312, 350–357, 380, 414, 420, 426, 444–445
mcp_plex/server/tools
   media_library.py3421296%213, 586, 688, 707, 758, 809, 884, 974–978
TOTAL340726692% 

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

@Teagan42 Teagan42 closed this Nov 2, 2025
@Teagan42 Teagan42 deleted the codex/identify-and-fill-documentation-gaps-7p7ve7 branch November 2, 2025 22:10
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