Skip to content

emrg: sync Agent.md Python test count 1001 -> 1003 (fix doc-count guard red CI) - #927

Closed
pm25coder wants to merge 1 commit into
argszero:masterfrom
pm25coder:feature/sync-doc-test-count-1003
Closed

emrg: sync Agent.md Python test count 1001 -> 1003 (fix doc-count guard red CI)#927
pm25coder wants to merge 1 commit into
argszero:masterfrom
pm25coder:feature/sync-doc-test-count-1003

Conversation

@pm25coder

Copy link
Copy Markdown
Contributor

Summary

Fix the red CI on master — the doc-count guard (tests/test_doc_counts.py::test_python_count_matches_docs) fails because the documented Python test count in Agent.md is stale.

Background

PR #926 (emrg stop TypeError fix) added 2 new test methods (TestStopAllSkipGui in tests/test_main.py) but did not sync Agent.md's documented count. As a result:

  • CI is currently red on master: test and test-windows both fail with AssertionError: Agent.md documents 1001 Python tests but 1003 are collected (run 32492754536).
  • Local run reproduces the same failure (937 passed + 1 failed + 65 skipped = 1003 collected).

Change

  • Agent.md: documented count 10011003 (matches --collect-only).

Verification

  • uv run pytest tests/ -q938 passed, 65 skipped (guard now green).
  • Import check from emrg.client.app import run_client OK; emrg --help OK.

@pm25coder
pm25coder force-pushed the feature/sync-doc-test-count-1003 branch from bd35f7d to ecf4108 Compare August 21, 2026 15:03

@argszero argszero left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for fixing the doc-count guard! Heads-up: PR #926 (created 3 minutes earlier) already covers this exact fix as a minimal 1-file diff (Agent.md 1001 → 1003) with CI green. Your branch appears based on an older master (diff shows ~31k additions across many files including a full _stop_all.py rewrite), so this PR would be hard to merge cleanly. Recommend closing this PR in favor of #926 — or if you want to keep it, rebase your branch onto current master so only the Agent.md sync remains.

@argszero

Copy link
Copy Markdown
Owner

Closing as superseded: the canonical fix (PR #926, Agent.md 1001 → 1003) was merged to master as commit cb80506. Your branch also carries ~31k unrelated changes from an older base. If you want to contribute the underlying work (e.g. the _stop_all.py rewrite), please rebase onto current master and open a focused PR.

@argszero argszero closed this Aug 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants