emrg: sync Agent.md Python test count to 1003 (doc-count guard fix for commit 463c2bb) - #926
Conversation
argszero
left a comment
There was a problem hiding this comment.
✅ LGTM — cycle. Doc-only fix: Agent.md Python test count 1001 → 1003 (syncs the post-463c2bb +2 tests); verified locally 1002 passed + 1 skipped (1003 collected); CI test + test-windows PASS (run 32495085112). Restores master CI to green after the direct-push commit 463c2bb skipped the doc sync.
argszero
left a comment
There was a problem hiding this comment.
✅ LGTM — cycle 1229. Re-verified: 1-file doc-only diff (Agent.md 1001 → 1003), head unchanged; CI test + test-windows PASS (run 32495085112). This is the canonical minimal fix for the post-463c2bb master CI-red (doc-count guard). 2/3 LGTMs.
argszero
left a comment
There was a problem hiding this comment.
✅ LGTM — cycle 1230. Head 8b4e95b unchanged; doc-only Agent.md count sync 1001 → 1003; CI test + test-windows PASS (run 32495085112). 3rd consecutive LGTM (cycles 1228 → 1229 → 1230), no ❌ in between — merge condition satisfied.
Uh oh!
There was an error while loading. Please reload this page.
…rgszero#926) `emrg stop` crashed with `TypeError: _stop_all() got an unexpected keyword argument 'skip_gui'`: the --skip-gui flag (rant 2026-08-21T12:44:34) was plumbed into the call site and emrg/_stop_all.py supports it, but the emrg.__main__._stop_all wrapper signature was never updated. - _stop_all(skip_gui=False) now forwards to stop_all(skip_gui=skip_gui) - tests: mocked forwarding regression tests in test_main.py + definition assertion in test_installer_stop.py (would have caught this at CI)
Fix the post-463c2bb CI failure: the doc-count guard (tests/test_doc_counts.py::test_python_count_matches_docs) fails on master because commit 463c2bb added 2 tests (TestStopAllSkipGui in tests/test_main.py) but Agent.md still documents 1001.
(1001)→(1003)