Skip to content

emrg: Agent.md — sync Python test count 1183→1186 (parallel-merge drift) - #1084

Merged
argszero merged 1 commit into
masterfrom
feature/sync-agent-doc-count-1186
Aug 31, 2026
Merged

emrg: Agent.md — sync Python test count 1183→1186 (parallel-merge drift)#1084
argszero merged 1 commit into
masterfrom
feature/sync-agent-doc-count-1186

Conversation

@argszero

Copy link
Copy Markdown
Owner

Fix Agent.md Python test count drift after parallel merges of #1082 (+3 LLM tests) and #1083 (+3 session tests).

Problem

Both PRs bumped Agent.md from 1180 → 1183, but the combined master now collects 1186 tests. The doc-count guard (test_doc_counts.py, #511) catches this on the post-merge master CI run — Agent.md and the guard must be synced.

Fix

  • Agent.md Python line: 1183 → 1186
  • Verified: pytest --collect-only = 1186; full local pytest 1185 passed + 1 skipped; import + CLI green.

@argszero argszero left a comment

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

✅ LGTM — cycle (1/3)

Reviewed head e124b3f against master (ad13965):

  • Change is exactly the fix needed: Agent.md Python test count 1183 → 1186.
  • Root cause verified: #1082 (+3 LLM tests) and #1083 (+3 session tests) both bumped Agent.md from 1180 → 1183 in parallel; the merged master collects 1186 (confirmed via pytest --collect-only = 1186), so the doc-count guard (test_doc_counts.py, #511) fails on post-merge CI run 33368344148. This PR syncs the doc to the real count.
  • Verified: PR branch Agent.md contains 1186, matching the collected count — the guard passes on this branch; CI test + test-windows green (run 33368557820); mergeable_state clean.
    No issues found.

@argszero argszero left a comment

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

✅ LGTM — cycle (2/3)

Re-verified head e124b3f against master (ad13965) this cycle:

  • diff unchanged: single line in Agent.md, Python test count 1183 → 1186.
  • PR branch Agent.md = 1186, matching pytest --collect-only = 1186 on the merged master — the doc-count guard (#511) passes with this change, fixing the post-merge CI failure (run 33368344148) caused by parallel merges of #1082 + #1083.
  • CI test + test-windows green (run 33368557820); mergeable_state clean.
    No issues found.

@argszero argszero left a comment

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

✅ LGTM — cycle (3/3)

Re-verified head e124b3f against master (ad13965) this cycle:

  • diff unchanged: single line in Agent.md, Python test count 1183 → 1186.
  • PR branch Agent.md = 1186 matches pytest --collect-only = 1186 on merged master — doc-count guard (#511) passes with this change, fixing post-merge CI failure (run 33368344148) from parallel merges of #1082 + #1083.
  • CI test + test-windows green (run 33368557820); mergeable_state clean.
    No issues found.

@argszero
argszero merged commit 0be7f46 into master Aug 31, 2026
2 checks passed
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.

1 participant