diff options
| author | Craig Jennings <c@cjennings.net> | 2026-05-30 22:13:21 -0500 |
|---|---|---|
| committer | Craig Jennings <c@cjennings.net> | 2026-05-30 22:13:21 -0500 |
| commit | e446dab251fb0889c516c491e0d73a6ec9f0b873 (patch) | |
| tree | 681d95d6c75f0b4f5573ec9a55ba3e817ab84d3d /.ai/scripts/cmail-action.py | |
| parent | 143feda0644d2289954b694f3ce4cee2fc74b808 (diff) | |
| download | rulesets-e446dab251fb0889c516c491e0d73a6ec9f0b873.tar.gz rulesets-e446dab251fb0889c516c491e0d73a6ec9f0b873.zip | |
feat(cmail): add --cc/--bcc and threading headers to cmail-action send
cmail-action send couldn't do a proper reply (no Cc/Bcc, no In-Reply-To/References), so an org-drill session that needed to reply to an upstream maintainer hand-rolled a raw MIME message through msmtp instead. I extended build_message (the pure function) with cc, bcc, in_reply_to, and references, wired the matching --cc/--bcc (repeatable), --in-reply-to, and --references flags through cmd_send, and wrote the tests first. send_message derives recipients from the To/Cc/Bcc headers and strips Bcc, so no manual recipient list is needed.
Diffstat (limited to '.ai/scripts/cmail-action.py')
| -rwxr-xr-x | .ai/scripts/cmail-action.py | 32 |
1 files changed, 28 insertions, 4 deletions
diff --git a/.ai/scripts/cmail-action.py b/.ai/scripts/cmail-action.py index 10eb215..0acd82d 100755 --- a/.ai/scripts/cmail-action.py +++ b/.ai/scripts/cmail-action.py @@ -120,17 +120,31 @@ def extract_body(msg): return msg.get_content() -def build_message(from_addr, to_addr, subject, body, attachments=None): +def build_message(from_addr, to_addr, subject, body, attachments=None, + cc=None, bcc=None, in_reply_to=None, references=None): """Construct an EmailMessage from the given fields and attachments. attachments is a list of (filename, maintype, subtype, content_bytes) - tuples — typically the return value of load_attachment per file. Pure - function: no I/O, no SMTP. + tuples — typically the return value of load_attachment per file. + + cc and bcc accept either a list of addresses or a single string; the + Cc/Bcc headers are set when present (smtplib.send_message reads them for + delivery and strips Bcc before sending). in_reply_to and references set + the In-Reply-To and References headers so a reply threads on the + recipient's end. Pure function: no I/O, no SMTP. """ msg = EmailMessage() msg["From"] = from_addr msg["To"] = to_addr + if cc: + msg["Cc"] = ", ".join(cc) if isinstance(cc, (list, tuple)) else cc + if bcc: + msg["Bcc"] = ", ".join(bcc) if isinstance(bcc, (list, tuple)) else bcc msg["Subject"] = subject + if in_reply_to: + msg["In-Reply-To"] = in_reply_to + if references: + msg["References"] = references msg.set_content(body) for filename, maintype, subtype, content in (attachments or []): msg.add_attachment(content, maintype=maintype, subtype=subtype, @@ -310,7 +324,9 @@ def cmd_send(args): body = Path(args.body_file).read_text() else: body = sys.stdin.read() - msg = build_message(USER, args.to, args.subject, body, attachments) + msg = build_message(USER, args.to, args.subject, body, attachments, + cc=args.cc, bcc=args.bcc, + in_reply_to=args.in_reply_to, references=args.references) smtp = smtp_connect() try: smtp.send_message(msg) @@ -377,6 +393,14 @@ def main(): "contents become the body") p_send.add_argument("--attach", action="append", default=[], help="path to attach (repeatable)") + p_send.add_argument("--cc", action="append", default=[], + help="Cc address (repeatable)") + p_send.add_argument("--bcc", action="append", default=[], + help="Bcc address (repeatable)") + p_send.add_argument("--in-reply-to", + help="Message-ID this replies to (threads on the recipient's end)") + p_send.add_argument("--references", + help="References header (space-separated Message-IDs)") p_send.set_defaults(func=cmd_send) args = p.parse_args() |
