- FR-11 rules editor: plain-text area at /rules, saved and versioned
(SaveRules appends a version; restore re-saves an old version), parse
errors surfaced, one-click re-classify. The reply model now reads the
rules document too (FR-11).
- FR-7 unsubscribe: parse and store List-Unsubscribe / List-Unsubscribe-Post
(migration v6), detect email and web links (internal/unsubscribe), a
domain-level ignore list, and an explicit record of what was marked. No
code path contacts an unsubscribe link; acting on one needs Sam's approval.
- Migration v6 is additive; unsubscribe gains url/updated_at.
- Tests: detector, parser, store migration/records/candidates, rules editor,
unsubscribe handlers, message-detail rendering.
Co-Authored-By: Paperclip <noreply@paperclip.ing>
The approved real test reply was sent and the thread updated (same
X-GM-THRID as the original). The live test exposed one bug: the thread
check hardcoded Gmail's sent mailbox as "[Gmail]/Sent Mail", but this
account exposes it as "[Google Mail]/Sent Mail".
- internal/mailfetch: LIST parser, ListMailboxes, and SentMailbox that
finds the mailbox flagged \Sent (RFC 6154) with name fallbacks.
- internal/mailops: VerifyThreadUpdated selects the discovered \Sent
mailbox instead of a hardcoded name.
- cmd/email-agent: -check-thread <id> exercises the check against real
Gmail without sending mail.
- Tests for LIST parsing, sent-mailbox discovery, and the discovery
error path.
- docs/phase5-replies.md: record send-as verification, the sent reply's
thread id, the app confirmation (sent_reply_in_thread=true), and the
fix.
Co-Authored-By: Paperclip <noreply@paperclip.ing>
Implements Phase 5 of AGENTS.md §11 (FR-8, FR-15):
- Message detail pane (GET /partials/message) with body, recommended
action, reply identity, template picker, editable draft, and history.
- internal/reply: OmniRoute (OpenAI-compatible) chat client and a
template-based draft generator. All non-JEV models go through
OmniRoute (§10).
- internal/mailops: SMTP send over smtp.gmail.com:587 with STARTTLS and
the app password, forcing IPv4 (§9); FR-15 reply identity (send as the
arrived-at X-Pm-Original-To address, else "Sam Rolfe"); threading via
In-Reply-To/References; -check-smtp safe verification; and a
[Gmail]/Sent Mail thread check.
- Migration v5 stores messages.message_id_header and drafts.sent_at;
mailfetch parses Message-ID.
- Send is user-triggered only: CSRF + UI confirm + confirm=yes guard,
no auto-send, duplicate-send refused. A sent reply files the original
under Replied-to/<address> and logs every step (FR-13).
- Tests for reply, mailops SMTP/identity/thread check, store drafts and
migration v5, and the server draft/edit/send guards.
- docs/phase5-replies.md, README, .env.example, docker-compose updated.
Co-Authored-By: Paperclip <noreply@paperclip.ing>
- internal/mailfetch: stdlib-only read-only IMAP client (LOGIN, SELECT,
UID SEARCH, UID FETCH) that always dials tcp4; IPv6 to Gmail is broken
from .13 and caused a socket EOF.
- MIME parsing to text, RFC2047 header decoding, FR-2 source address:
X-Pm-Original-To -> To -> Received 'for <addr>' -> Return-Path, never
Delivered-To. The matched header is recorded for audit.
- store: Message upsert/list/count + LogAction; schema migration v2 adds
imap_uid and imap_uidvalidity.
- server: dashboard and /partials/inbox render real mail grouped by
category with a recommended action; POST /actions/fetch runs a scan
(CSRF-checked); nav counts; 'Check mail' button.
- config: GMAIL_USER, GMAIL_APP_PASSWORD, IMAP host/port/lookback/max.
- cmd: -fetch runs one scan and exits (n8n/CLI entry point, FR-1).
- compose/.env.example expose the Gmail + IMAP settings.
- tests: FR-2 precedence, MIME bodies, IMAP literal/parser, store
upsert/migration, inbox rendering.
Co-Authored-By: Paperclip <noreply@paperclip.ing>
