jhheider / agentmail-rs / 29476202136

81%

push

github

refactor: modularize into client/ and types/ modules (#12)

* refactor: split lib.rs into client/ and types/ modules (pure move)

Behavior-preserving reorg of the single 1129-line lib.rs along resource
seams. lib.rs keeps the crate docs, Error, Client (struct + Debug + new/
from_env), and constants; the HTTP chokepoint moves to client/mod.rs and each
resource's methods to client/{inboxes,messages,webhooks,drafts}.rs; wire types
move to types/{inboxes,messages,webhooks,drafts,attachments}.rs with Page in
types/mod.rs; urlish and QueryBuilder move to util.rs. Public API is unchanged
(pub use types::* keeps every item at agentmail::X). tests/http.rs splits into
a tests/http/ binary (common + core + messages + drafts). Page::query,
MessageListFilters::query, and urlish become pub(crate) so sibling modules can
call them; no other visibility changes.

Co-Authored-By: DeepSeek <noreply@deepseek.com>
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01CcYsnLASyk9cN6DTktGH7h

* refactor: generic request body and shared execute() seam

Replace the Option<serde_json::Value> body with a generic Option<&B: Serialize>
so call sites pass Some(&value) directly instead of
serde_json::to_value(x).expect(...); bodiless verbs pass None::<&NoBody>. Split
the request builder into a private execute() -> (StatusCode, String) so the
retry loop (added later) wraps exactly one place. No public API or behavior
change; the full wiremock suite passes unchanged.

Co-Authored-By: DeepSeek <noreply@deepseek.com>
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01CcYsnLASyk9cN6DTktGH7h

---------

Co-authored-by: DeepSeek <noreply@deepseek.com>
Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

375 of 420 new or added lines in 8 files covered. (89.29%)

388 of 445 relevant lines covered (87.19%)

6.13 hits per line