christophevg / yoker-assistant / 29756320903

90%

push

github

refactor: descope P1-003 to errata only — drop Mailbox class per owner feedback

The Mailbox wrapper class added indirection without benefit over using
simple_email_gw directly. Per owner feedback during PR review, every send
is a reply (always reply_email, no send_email fallback, no in_reply_to
branch), so the wrapper's seam abstraction was unnecessary for a
demo/tutorial codebase.

Changes:
- Drop src/yoker_assistant/mailbox.py and tests/test_mailbox.py
- The loop (P2-005) will use IMAPClient/SMTPClient from simple_email_gw
  directly; every send is a reply via reply_email(html_body=..., in_reply_to=...)
- P2-007 (Mailbox.fetch loop scaffolding) folded into P2-005
- P3-003 (dedicated Mailbox tests) dropped
- P3-002 covers the html_body= routing regression
- functional.md: §2.2, §2.4, §4.3, §4.4 corrected — seam is the loop
  module, not a wrapper class; no send_email fallback
- TODO.md: P1-003 descoped to errata-only; P2-005 owns gateway calls
  directly; P2-006 stale Mailbox.fetch() reference fixed
- consensus.md: descope update section added at top
- __init__.py / __main__.py docstrings: removed stale mailbox references

🤖 Implemented together with a coding agent.

0 of 1 branches covered (0.0%)

Branch coverage included in aggregate %.

4 of 10 relevant lines covered (40.0%)

0.4 hits per line