@dxv-systems/helpdesk 0.2.0 → 0.3.0

package/README.md

@@ -14,7 +14,7 @@ The app owns the schema and the cron, and mounts the inbox (see Inbox UI). It pa
 
 | Interface | What the app implements |
 |---|---|
-| `HelpdeskStore` | Threads (including the nullable `dismissedBy` staff id), messages, sync state, draft runs, send intents. Methods marked atomic in `types.ts` must be one transaction. Unique keys: message on `(mailbox, id)`, draft run on `(mailbox, inboundMessageId, generation)`, send intent on `id`. |
+| `HelpdeskStore` | Threads (including the nullable `dismissedBy` staff id), messages (with their `direction`, `inbound` or `outbound`), sync state (one link per folder), draft runs, send intents. Methods marked atomic in `types.ts` must be one transaction. Unique keys: message on `(mailbox, id)`, draft run on `(mailbox, inboundMessageId, generation)`, send intent on `id`. `getLatestMessageForThread` returns the newest inbound message only. |
 | `MailboxGateway` | One mailbox: `delta`, `getMessage`, `createReply`, `updateDraftBody`, `sendDraft` (and optionally `listAttachments`, `downloadAttachment`, `addAttachment`). Declared structurally against `@dxv-systems/mail-graph`'s `MailboxClient`, so `createMailboxClient(http, address)` can be passed as is. Use immutable ids (`Prefer: IdType="ImmutableId"`). An expired delta link (410, `syncStateNotFound`, `resyncRequired`) may throw `DeltaLinkExpiredError` or mail-graph's `GraphDeltaExpiredError`; both are recognised by name or signal. |
 | `Drafter` (optional) | Without one, no draft is generated and staff write every reply (see Staff-written replies). `start(request)` starts a run and returns its id. It must treat the same `idempotencyKey` as the same run. The result comes back later through `completeDraft`. |
 | `Notifier` (optional) | `review_needed` and `send_outcome_unknown`. A throwing notifier never breaks the flow. |
@@ -55,7 +55,7 @@ Three switches the app passes in `deps.controls` (a value, or a function that is
 
 The package default is `read_only`, so a mailbox nobody configured is never written to.
 
-**Kill switch.** With `paused: true`, `ingestPage`, `syncMailbox`, `ensureDraft`, `completeDraft`, `regenerateDraft`, `editDraft`, `sweepStuckDrafts`, `unignoreThread`, `approveAndSend`, `markRepliedExternally` and `dismissThread` return `{ status: "paused" }` (`ensureDraft`: `"paused"`) and make no Graph call, no drafter call and no store write. `isPaused(result)` narrows it. A sync that is paused between pages returns the pages already done. A drafter report that arrives while paused is dropped; the run stays in flight and, after the pause, `sweepStuckDrafts` moves it to review as `drafter_failed`. Bookkeeping without mailbox access (`connectMailbox`, `resolveUnknownSend`, `markWaiting`, `closeThread`) is not paused. Use it as the emergency stop: removing the Exchange role can take up to 2 hours to apply.
+**Kill switch.** With `paused: true`, `ingestPage`, `ingestSentPage`, `syncMailbox`, `ensureDraft`, `completeDraft`, `regenerateDraft`, `editDraft`, `sweepStuckDrafts`, `unignoreThread`, `approveAndSend`, `markRepliedExternally` and `dismissThread` return `{ status: "paused" }` (`ensureDraft`: `"paused"`) and make no Graph call, no drafter call and no store write. `isPaused(result)` narrows it. A sync that is paused between pages returns the pages already done. A drafter report that arrives while paused is dropped; the run stays in flight and, after the pause, `sweepStuckDrafts` moves it to review as `drafter_failed`. Bookkeeping without mailbox access (`connectMailbox`, `resolveUnknownSend`, `markWaiting`, `closeThread`) is not paused. Use it as the emergency stop: removing the Exchange role can take up to 2 hours to apply.
 
 **Mailbox scope.** `config.connectedMailboxes` is the helpdesk's scope. Any call for another mailbox (a `mailbox` argument, a thread or a drafter report of another mailbox, or a gateway whose `address` differs from the mailbox asked for) throws `MailboxNotInScopeError` before any Graph call. The same Entra app may reach other mailboxes for other features; this helpdesk never touches them.
 
@@ -65,7 +65,7 @@ Run `syncMailbox` for a mailbox from one worker at a time (a lock or a queue). O
 
 ## Ingestion
 
-`ingestPage(deps, mailbox)` fetches one delta page and processes it; `syncMailbox` loops until the round is complete.
+`ingestPage(deps, mailbox)` fetches one Inbox delta page and processes it; `ingestSentPage(deps, mailbox)` does the same for Sent Items (see Sent Items). `syncMailbox` runs Inbox pages until the round is complete, then Sent Items pages, each folder with its own `maxPages` budget, and returns the results of both in one array (`IngestResult.folder` is `inbox` or `sentitems`). An Inbox round that is not complete yet skips Sent Items for that run.
 
 - **Idempotent on the immutable message id.** A message already stored is never inserted again. A re-read of stored mail only finishes work a crash left open (a draft run that was never created, a draft ready to be routed to review).
 - **The link is stored last.** `nextLink` or `deltaLink` is saved after every item of the page is processed. A crash re-reads the page.
@@ -75,12 +75,25 @@ Run `syncMailbox` for a mailbox from one worker at a time (a lock or a queue). O
 - **Removed items** set `removedAt` on the stored message. The ticket keeps its state. `removedAt` means "gone from the Inbox", nothing more: Graph reports a move to another folder and a delete alike, both as `@removed` with reason `deleted` (seen in a real tenant).
 - Items are processed oldest first, so a reply in the same page finds its parent.
 
+### Sent Items
+
+The ticket shows every reply the mailbox sent to the customer, whether it went out through the helpdesk or from Outlook. `ingestSentPage` reads the mailbox's Sent Items folder (`folderId: "sentitems"`) with the same rules as the Inbox: items first, link stored last, an expired link restarts from `max(connectedAt, sentLastReceivedAt - 48 h)`, and the kill switch returns `{ status: "paused" }`. The folder has its own link and newest time on the sync state (`sentLink`, `sentLastReceivedAt`).
+
+- **Display only.** A sent mail is stored as a message with `direction: "outbound"`, `fromAddress` the mailbox, and `receivedAt` the send time. It never changes a ticket: no state change, no new `latestInboundId`, no draft, and it never opens a ticket.
+- **Matching.** Always inside the same mailbox. An `In-Reply-To` that names a stored mail picks that ticket. Otherwise the ticket is found through the `References` chain and the `conversationId`; a ticket found only by conversation also needs a header hit, or a To or Cc recipient that is one of its participants. The sent mail is stored only when exactly one ticket is left. No match, or more than one: it is skipped and counted in `IngestResult.unmatched`, and never stored.
+- **Replies sent by the helpdesk.** With immutable ids a draft keeps its id after it is sent, so the Sent Items copy of an approved reply has the id the send intent recorded. The inbox uses that to show it once (see Inbox UI).
+- **Skipped:** drafts, mail sent before `connectedAt`, and mail already stored (a re-read changes nothing).
+- **Removed items** set `removedAt`: gone from Sent Items.
+- **Customer replies to our mail.** A customer reply's `In-Reply-To` usually names our sent mail. Once that mail is synced, the reply threads through its headers like any other.
+
+Mail the mailbox sent to itself or that lands in its own Inbox is still skipped by the Inbox sync.
+
 ## Threading
 
 `matchThread` in this order, always inside one mailbox (also checked on the rows a store returns):
 
 1. **`conversationId`.** Accepted when the sender already wrote in that thread, or the mail's In-Reply-To / References point into it. An unknown sender with only a matching conversation id starts a new thread: Exchange groups mail without thread headers by subject, so a bare match can join strangers. If the headers point at a different thread than the conversation id, it also starts a new thread.
-2. **`In-Reply-To` and the whole `References` chain**, against stored `internetMessageId`s (lowercase, no angle brackets). The whole chain matters because the reply a customer answers is our own sent mail, which we never stored; the earlier ids in the chain still hit. Several open threads in one chain: no match. One open thread next to closed or ignored ones: the open one. A forward (`FW:`, `Fwd:`, `WG:`) from someone outside the thread is new business even though it carries old headers.
+2. **`In-Reply-To` and the whole `References` chain**, against stored `internetMessageId`s (lowercase, no angle brackets). The whole chain matters because the reply a customer answers is our own sent mail, which is stored only once Sent Items is synced (and never when it matched no ticket); the earlier ids in the chain still hit. Several open threads in one chain: no match. One open thread next to closed or ignored ones: the open one. A forward (`FW:`, `Fwd:`, `WG:`) from someone outside the thread is new business even though it carries old headers.
 3. **Normalised subject plus the same sender**, within 14 days (`subjectWindowMs`). Prefixes `Re`, `Antw`, `AW`, `SV`, `FW`, `Fwd`, `WG`, `TR`, `VS`, `Re[2]`, and gateway tags like `[EXTERN]` are stripped. An empty subject never matches. More than one open candidate: no match.
 
 ## Ticket states
@@ -101,7 +114,7 @@ Run `syncMailbox` for a mailbox from one worker at a time (a lock or a queue). O
 | new, drafted, review | `dismiss` | ignored |
 | sent, waiting | `bounce` | review |
 
-`transition(state, event)` is the pure function; anything else throws `InvalidTransitionError` (with `from` and `event`). `ignore` belongs to the noise filter and `dismiss` to staff, so the filter can never set aside a ticket that already has a draft. `unignoreThread` is the staff action for mail that was wrongly ignored: ignored to new, `latestInboundId` set to the newest message, `dismissedBy` cleared, and a draft starts.
+`transition(state, event)` is the pure function; anything else throws `InvalidTransitionError` (with `from` and `event`). `ignore` belongs to the noise filter and `dismiss` to staff, so the filter can never set aside a ticket that already has a draft. `unignoreThread` is the staff action for mail that was wrongly ignored: ignored to new, `latestInboundId` set to the newest inbound message (never our own sent mail), `dismissedBy` cleared, and a draft starts.
 
 **No reply needed.** `dismissThread(deps, { threadId, dismissedBy })` is the staff action for mail that needs no answer (a newsletter the filter missed, "thanks, got it", spam): new, drafted or review to ignored, with `Thread.dismissedBy` set to the staff id (null on a ticket the noise filter ignored, so analytics can tell the two apart). It returns `{ status: "paused" }` under the kill switch and refuses a mailbox outside the scope. It throws `SendPendingError` (code `send_in_progress` or `send_outcome_unknown`) while a send is under way or unclear; a `sending` intent older than 2 minutes is turned into `unknown` first, so staff get the resolve buttons. After the state change it stops an in-flight draft run (marked failed, so a late report is a no-op). It touches no mailbox. `approveAndSend` checks the state again after it claims the send, so a dismissal that lands just before the claim stops the send (`StaleThreadError`, the intent becomes `failed`). Ignored stays final: a later reply opens a new ticket. Only the app decides when to call `markWaiting` (sent to waiting) and `closeThread` (waiting to closed, for example after N quiet days).
 
@@ -205,7 +218,7 @@ export default function HelpdeskPage() {
 | `createHelpdeskHandlers` option | |
 |---|---|
 | `deps` | The `HelpdeskDeps` (store, `gatewayFor`, optional drafter, notifier, `controls`, clock, config). The handlers call the domain functions with it and nothing else. |
-| `inbox` | `HelpdeskInboxQueries`: `listThreads(query)` (filters `states`, `mailbox`, `assigneeId`: undefined is anyone, null is unassigned; `cursor` and `limit`; newest activity first; returns `{ rows: [{ thread, assignee }], nextCursor }`), `listMessages(threadId)` (oldest first), `getAssignee(threadId)`, `setAssignee(threadId, { id, name } \| null)`. Assignment is app data; the domain never reads it. |
+| `inbox` | `HelpdeskInboxQueries`: `listThreads(query)` (filters `states`, `mailbox`, `assigneeId`: undefined is anyone, null is unassigned; `cursor` and `limit`; newest activity first; returns `{ rows: [{ thread, assignee }], nextCursor }`), `listMessages(threadId)` (every message of both directions, oldest first), `getAssignee(threadId)`, `setAssignee(threadId, { id, name } \| null)`. Assignment is app data; the domain never reads it. |
 | `getStaff(request)` | `{ id, name, email }` for staff, null for anyone else (every route answers 401). `id` is stored as `approvedBy` on a send and as `dismissedBy` on a dismissal. |
 | `isProduction` | Outside production every route answers 403 `not_production`. |
 | `basePath` | Default `/api/helpdesk`. |
@@ -217,8 +230,8 @@ export default function HelpdeskPage() {
 | Route | Calls |
 |---|---|
 | `GET threads?state=review,new&mailbox=&assignee=me\|none\|<id>&cursor=&limit=` | `inbox.listThreads`; also returns the pilot switches per mailbox |
-| `GET threads/:id` | thread, messages (metadata), latest draft, send status, and `actions` the state machine and pilot controls allow now |
-| `GET threads/:id/messages/:messageId` | `gateway.getMessage` through the scope check; body as plain text, plus its attachments (`gateway.listAttachments`, inline images left out). For a delivery report (`bounce: true` in the detail's messages) also `bounce`: `{ recipient, statusCode, reason, diagnostic }`, read on the server, with `reason` one plain Dutch sentence (e.g. 5.1.1 "Dit adres bestaat niet.") |
+| `GET threads/:id` | thread, messages (metadata; `outbound: true` for mail synced from Sent Items), latest draft, send status (`send.messageId`: the sent mail's id once the status is `sent`), and `actions` the state machine and pilot controls allow now |
+| `GET threads/:id/messages/:messageId` | `gateway.getMessage` through the scope check; body as plain text, plus its attachments (`gateway.listAttachments`, inline images left out). A sent mail's body and attachments load the same way. For a delivery report (`bounce: true` in the detail's messages, inbound only) also `bounce`: `{ recipient, statusCode, reason, diagnostic }`, read on the server, with `reason` one plain Dutch sentence (e.g. 5.1.1 "Dit adres bestaat niet.") |
 | `GET threads/:id/messages/:messageId/attachments/:attachmentId` | download of one file attachment of that message (`gateway.downloadAttachment`) |
 | `PUT threads/:id/draft` `{ text }` | `editDraft` |
 | `POST threads/:id/regenerate` | `regenerateDraft` |
@@ -231,7 +244,7 @@ export default function HelpdeskPage() {
 | `POST threads/:id/dismiss` | `dismissThread`, with the signed-in staff id as `dismissedBy` |
 | `POST threads/:id/resolve-send` `{ outcome: "sent" \| "not_sent" }` | `resolveUnknownSend` |
 
-Errors are `{ error, message }` with a Dutch `message` (`INBOX_ERROR_MESSAGES`): `paused` (423), `write_blocked` (409, with `mode`), `mailbox_not_in_scope` (403), `stale_draft`, `draft_not_ready`, `invalid_transition`, `stale_thread`, `send_in_progress`, `send_outcome_unknown` (409), `empty_reply` (422), `send_failed` (502, Graph refused, nothing sent), `thread_not_found` (404). A send whose outcome is unclear answers 200 with `outcome: "unknown_outcome"`. Responses never carry draft bodies from Outlook, idempotency keys, draft ids or raw error text.
+Errors are `{ error, message }` with a Dutch `message` (`INBOX_ERROR_MESSAGES`): `paused` (423), `write_blocked` (409, with `mode`), `mailbox_not_in_scope` (403), `stale_draft`, `draft_not_ready`, `invalid_transition`, `stale_thread`, `send_in_progress`, `send_outcome_unknown` (409), `empty_reply` (422), `send_failed` (502, Graph refused, nothing sent), `thread_not_found` (404). A send whose outcome is unclear answers 200 with `outcome: "unknown_outcome"`. Responses never carry draft bodies from Outlook, idempotency keys, draft ids or raw error text. The one exception is `send.messageId` of a sent reply: once sent, the draft id is the id of the mail in Sent Items.
 
 **Attachments.** The message body route lists the attachments of a customer mail; inline images (signature logos) are left out. A file up to 4 MB gets a download link on the app itself; a larger file, an attached mail (`item`) or a link to a shared file (`reference`) is listed with a note to open it in Outlook, because a Vercel function answers at most 4.5 MB. The download is checked like the body (the message must belong to the ticket, scope check, paused refuses) and the attachment id must be one Graph lists for that message. It is always served as `application/octet-stream` with `Content-Disposition: attachment`, `X-Content-Type-Options: nosniff` and a sandbox CSP, so customer files never render on the app's origin. Both gateway methods are optional: a gateway without them shows no attachments. mail-graph's `MailboxClient` has them. Attachments are never stored by the helpdesk.
 
@@ -253,15 +266,28 @@ What staff sees: one page with the list and the open ticket side by side; on a p
 - **List.** Status and assignee filters side by side (native, labelled selects with the app's input style; the chosen status shows the count the list already has), a mailbox filter below them when there are several mailboxes, and "Meer laden". Each ticket shows its subject and time, the customer, and one status badge.
 - **One status badge.** State and review reason make one label: a bounced reply shows "Niet afgeleverd" instead of "Te beoordelen"; other reasons appear as small text next to the badge ("Voorstel klaar", "Zelf antwoorden").
 - **Header.** Subject, the status badge, `customer · naar <mailbox>`, the assignment ("Niet toegewezen · Aan mij", "Toegewezen aan jou", "Toegewezen aan <naam> · Aan mij") and a "Meer acties" menu with "Geen antwoord nodig", "Al beantwoord in Outlook" (both ask inline first) and "Toewijzing weghalen".
-- **Conversation.** One chronological list: customer mails on the left with initials, the sent reply on the right ("Jouw antwoord · vanaf <mailbox>", its files, "Verstuurd, staat in Verzonden items"). Attachments are file chips with type and size. A delivery report is an event ("Jouw antwoord is niet afgeleverd", the Dutch reason, and the raw text under "Technische details van Microsoft"), never the raw mail. Only the latest sent reply is shown; the helpdesk keeps no copy of earlier ones.
+- **Conversation.** One chronological list: customer mails on the left with initials, every reply the mailbox sent on the right ("Jouw antwoord · vanaf <mailbox>", the body and files loaded like customer mail, "Verstuurd, staat in Verzonden items"). Attachments are file chips with type and size. A delivery report is an event ("Jouw antwoord is niet afgeleverd", the Dutch reason, and the raw text under "Technische details van Microsoft"), never the raw mail. A reply just sent through the helpdesk shows its approved text until its Sent Items copy is synced (up to one sync interval); from then on only the copy is shown.
 - **Reply.** A "Jouw antwoord" card: "Voorstel van de AI · pas aan waar nodig" while the text is the drafter's, the reply box, files ("Bijlage toevoegen", max. 10 files, 20 MB together), and a footer with the sending mailbox and "Nieuw voorstel", "Opslaan" and "Goedkeuren en versturen". In `read_only` the ticket carries an "Alleen lezen" badge and the send button is disabled with the reason next to it. Without a usable draft the box is empty, and sending saves the typed reply first; a blank reply cannot be sent. Sending always asks for confirmation inline (recipient and mailbox named; edits are saved first), a stale reply asks again with "Toch versturen", and an unclear send shows "Het staat in Verzonden items" / "Het is niet verstuurd".
 - **Bounced ticket.** Instead of the reply card, "Wat nu?" with "Opnieuw verstuurd vanuit Outlook" (`replied-externally`) and "Geen antwoord meer nodig" (`dismiss`).
 
 A dismissed ticket leaves the list (a ticket whose new state falls outside the chosen view always does) and stays open with "Toch behandelen" to undo it.
 
+## Upgrading from 0.2
+
+0.3 syncs Sent Items. The app's store and inbox queries need these changes before the new version runs:
+
+1. **Messages: a `direction` column**, `'inbound'` or `'outbound'`, not null, default `'inbound'`. Every existing row is inbound mail, so the default backfills it. Store it on insert (`createThread`, `appendMessage`) and return it from every read of a message.
+2. **Sync state: `sent_link` and `sent_last_received_at`**, both nullable text (or timestamp for the second). `saveSyncState` must store `sentLink` and `sentLastReceivedAt` and `getSyncState` must return them. If they are not stored, Sent Items is read again from `connectedAt` on every run: nothing is stored twice, but every run costs more.
+3. **`getLatestMessageForThread` filters on `direction = 'inbound'`.** Without the filter, `unignoreThread` could make our own sent mail the message to answer.
+4. **`listMessages` returns both directions**, oldest first. The inbox shows outbound messages as replies. `findMessageThreads` and `findThreadsByConversationId` also look at both directions; no change is needed there if they do not filter on it.
+5. **`syncMailbox` results** now hold Inbox and Sent Items pages in one array. Code that reads them can use `IngestResult.folder` (`inbox` or `sentitems`) and the new `unmatched` count (always 0 for the Inbox).
+6. **`InboxThreadDetail`**: messages have `outbound`, and `send` has `messageId`. Only relevant to apps that read the detail themselves; the React page handles both.
+
+No new permissions: Sent Items is part of the same mailbox access. A mailbox that is already connected starts syncing its Sent Items from `connectedAt` on the first run after the upgrade, so earlier replies are picked up too.
+
 ## Testing
 
-`@dxv-systems/helpdesk/testing` has in-memory fakes (`createMemoryStore`, `createFakeMailbox`, `createFakeDrafter`, `createManualClock`, and `createTestWorld` to wire them), with the uniqueness and compare-and-set rules a real store needs. Apps can run their own store against the same scenarios. `createTestWorld({ controls })` sets the pilot controls (without them the package defaults apply), and each fake mailbox records every gateway call in `calls` and the mailbox writes in `writeCalls`, so a test can assert that read-only or paused made none.
+`@dxv-systems/helpdesk/testing` has in-memory fakes (`createMemoryStore`, `createFakeMailbox`, `createFakeDrafter`, `createManualClock`, and `createTestWorld` to wire them), with the uniqueness and compare-and-set rules a real store needs. Apps can run their own store against the same scenarios. `createTestWorld({ controls })` sets the pilot controls (without them the package defaults apply), and each fake mailbox records every gateway call in `calls` and the mailbox writes in `writeCalls`, so a test can assert that read-only or paused made none. `deliver({ ..., folder: "sentitems", to: [...] })` puts a mail in the fake mailbox's Sent Items.
 
 ## Development
 

package/dist/inbox/handlers.js

@@ -204,11 +204,14 @@ function createHelpdeskHandlers(config) {
                 subject: m.subject,
                 receivedAt: m.receivedAt,
                 noise: m.noise.action === "ignore",
-                bounce: m.noise.reason === "bounce",
+                bounce: m.direction === "inbound" && m.noise.reason === "bounce",
                 removed: m.removedAt !== null,
+                outbound: m.direction === "outbound",
             })),
             draft: draftView(thread, run ?? (latestRun?.status === "failed" ? latestRun : null)),
-            send: intent ? { status: intent.status, at: intent.updatedAt } : null,
+            send: intent
+                ? { status: intent.status, at: intent.updatedAt, messageId: intent.status === "sent" ? intent.draftId : null }
+                : null,
             replyAttachments: await replyAttachmentViews(thread.id, intent),
             actions: allowedActions(thread, run, intent, controls.paused, mode, Boolean(deps.drafter), canAttach(thread.mailbox)),
         };
@@ -303,7 +306,7 @@ function createHelpdeskHandlers(config) {
             throw new RouteError(404, "message_unavailable");
         const { contentType, content } = message.body;
         const text = contentType === "html" ? (0, plain_text_1.htmlToPlainText)(content) : content.replace(/\r\n?/g, "\n").trim();
-        const isBounce = stored.noise.reason === "bounce";
+        const isBounce = stored.direction === "inbound" && stored.noise.reason === "bounce";
         const listed = !isBounce && gateway.listAttachments && message.hasAttachments !== false ? await gateway.listAttachments(messageId) : [];
         const attachments = listed.filter((a) => !a.isInline).map((a) => attachmentView(threadId, messageId, a));
         const body = { id: messageId, text, attachments, bounce: isBounce ? bounceView(text) : null };

package/dist/inbox/types.d.ts

@@ -45,6 +45,7 @@ export interface InboxMessageView {
     noise: boolean;
     bounce: boolean;
     removed: boolean;
+    outbound: boolean;
 }
 export interface InboxBounceView {
     recipient: string | null;
@@ -104,6 +105,7 @@ export interface InboxThreadDetail {
     send: {
         status: SendStatus;
         at: string;
+        messageId: string | null;
     } | null;
     replyAttachments: InboxReplyAttachmentView[];
     actions: InboxAction[];

package/dist/index.d.ts

@@ -5,7 +5,7 @@ export { DEFAULT_WRITE_MODE, isPaused, writeModeFor } from "./controls";
 export { normalizeSubject, isForwardSubject } from "./subject";
 export { classifyNoise, type NoiseInput, type NoiseVerdict } from "./noise";
 export { matchThread, normalizeMessageId, type ThreadCandidate, type ThreadMatch } from "./threading";
-export { connectMailbox, ingestPage, isSyncFailing, syncMailbox, SYNC_FAILING_AFTER, type IngestResult, } from "./ingest";
+export { connectMailbox, ingestPage, ingestSentPage, isSyncFailing, syncMailbox, SYNC_FAILING_AFTER, type IngestResult, type SyncFolder, } from "./ingest";
 export { ensureDraft, completeDraft, regenerateDraft, editDraft, sweepStuckDrafts, unignoreThread, type EnsureDraftOutcome, } from "./drafting";
 export { renderCommentHtml, prependToDraftBody, hasQuotedHistory, findQuoteStart, } from "./draft-body";
 export { approveAndSend, reconcileUnknownSends, resolveUnknownSend, markRepliedExternally, markWaiting, closeThread, dismissThread, classifySendError, type SendOutcome, } from "./sending";

package/dist/index.js

@@ -14,7 +14,7 @@ var __exportStar = (this && this.__exportStar) || function(m, exports) {
     for (var p in m) if (p !== "default" && !Object.prototype.hasOwnProperty.call(exports, p)) __createBinding(exports, m, p);
 };
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.startReplyUpload = exports.removeReplyAttachment = exports.pendingReplyAttachments = exports.addReplyAttachment = exports.MAX_REPLY_ATTACHMENTS = exports.MAX_REPLY_ATTACHMENT_BYTES = exports.classifySendError = exports.dismissThread = exports.closeThread = exports.markWaiting = exports.markRepliedExternally = exports.resolveUnknownSend = exports.reconcileUnknownSends = exports.approveAndSend = exports.findQuoteStart = exports.hasQuotedHistory = exports.prependToDraftBody = exports.renderCommentHtml = exports.unignoreThread = exports.sweepStuckDrafts = exports.editDraft = exports.regenerateDraft = exports.completeDraft = exports.ensureDraft = exports.SYNC_FAILING_AFTER = exports.syncMailbox = exports.isSyncFailing = exports.ingestPage = exports.connectMailbox = exports.normalizeMessageId = exports.matchThread = exports.classifyNoise = exports.isForwardSubject = exports.normalizeSubject = exports.writeModeFor = exports.isPaused = exports.DEFAULT_WRITE_MODE = void 0;
+exports.startReplyUpload = exports.removeReplyAttachment = exports.pendingReplyAttachments = exports.addReplyAttachment = exports.MAX_REPLY_ATTACHMENTS = exports.MAX_REPLY_ATTACHMENT_BYTES = exports.classifySendError = exports.dismissThread = exports.closeThread = exports.markWaiting = exports.markRepliedExternally = exports.resolveUnknownSend = exports.reconcileUnknownSends = exports.approveAndSend = exports.findQuoteStart = exports.hasQuotedHistory = exports.prependToDraftBody = exports.renderCommentHtml = exports.unignoreThread = exports.sweepStuckDrafts = exports.editDraft = exports.regenerateDraft = exports.completeDraft = exports.ensureDraft = exports.SYNC_FAILING_AFTER = exports.syncMailbox = exports.isSyncFailing = exports.ingestSentPage = exports.ingestPage = exports.connectMailbox = exports.normalizeMessageId = exports.matchThread = exports.classifyNoise = exports.isForwardSubject = exports.normalizeSubject = exports.writeModeFor = exports.isPaused = exports.DEFAULT_WRITE_MODE = void 0;
 __exportStar(require("./types"), exports);
 __exportStar(require("./errors"), exports);
 __exportStar(require("./states"), exports);
@@ -33,6 +33,7 @@ Object.defineProperty(exports, "normalizeMessageId", { enumerable: true, get: fu
 var ingest_1 = require("./ingest");
 Object.defineProperty(exports, "connectMailbox", { enumerable: true, get: function () { return ingest_1.connectMailbox; } });
 Object.defineProperty(exports, "ingestPage", { enumerable: true, get: function () { return ingest_1.ingestPage; } });
+Object.defineProperty(exports, "ingestSentPage", { enumerable: true, get: function () { return ingest_1.ingestSentPage; } });
 Object.defineProperty(exports, "isSyncFailing", { enumerable: true, get: function () { return ingest_1.isSyncFailing; } });
 Object.defineProperty(exports, "syncMailbox", { enumerable: true, get: function () { return ingest_1.syncMailbox; } });
 Object.defineProperty(exports, "SYNC_FAILING_AFTER", { enumerable: true, get: function () { return ingest_1.SYNC_FAILING_AFTER; } });

package/dist/ingest.d.ts

@@ -1,5 +1,7 @@
 import type { HelpdeskDeps, Paused, SyncFailing, SyncState } from "./types";
+export type SyncFolder = "inbox" | "sentitems";
 export interface IngestResult {
+    folder: SyncFolder;
     fetched: number;
     created: number;
     appended: number;
@@ -13,11 +15,13 @@ export interface IngestResult {
     gone: number;
     draftsStarted: number;
     deliveryFailures: number;
+    unmatched: number;
     restarted: boolean;
     hasMore: boolean;
 }
 export declare function connectMailbox(deps: HelpdeskDeps, mailbox: string): Promise<SyncState>;
 export declare function ingestPage(deps: HelpdeskDeps, mailbox: string): Promise<IngestResult | Paused>;
+export declare function ingestSentPage(deps: HelpdeskDeps, mailbox: string): Promise<IngestResult | Paused>;
 export declare function syncMailbox(deps: HelpdeskDeps, mailbox: string, options?: {
     maxPages?: number;
 }): Promise<IngestResult[] | Paused | SyncFailing>;

package/dist/ingest.js

@@ -3,6 +3,7 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.SYNC_FAILING_AFTER = void 0;
 exports.connectMailbox = connectMailbox;
 exports.ingestPage = ingestPage;
+exports.ingestSentPage = ingestSentPage;
 exports.syncMailbox = syncMailbox;
 exports.isSyncFailing = isSyncFailing;
 const controls_1 = require("./controls");
@@ -34,32 +35,44 @@ function threadingHeaders(msg) {
     const references = msg.references ?? (raw ? (raw.match(/<[^>]+>/g) ?? []) : []);
     return { inReplyTo, references };
 }
-function restartFrom(state, overlapMs) {
-    if (!state.lastReceivedAt)
-        return state.connectedAt;
-    const back = Date.parse(state.lastReceivedAt) - overlapMs;
-    return new Date(Math.max(Date.parse(state.connectedAt), back)).toISOString();
+function cursorOf(state, folder) {
+    return folder === "inbox"
+        ? { link: state.link, lastReceivedAt: state.lastReceivedAt }
+        : { link: state.sentLink ?? null, lastReceivedAt: state.sentLastReceivedAt ?? null };
 }
-async function fetchPage(deps, gateway, state) {
+function withCursor(state, folder, cursor) {
+    return folder === "inbox"
+        ? { ...state, link: cursor.link, lastReceivedAt: cursor.lastReceivedAt }
+        : { ...state, sentLink: cursor.link, sentLastReceivedAt: cursor.lastReceivedAt };
+}
+function restartFrom(connectedAt, lastReceivedAt, overlapMs) {
+    if (!lastReceivedAt)
+        return connectedAt;
+    const back = Date.parse(lastReceivedAt) - overlapMs;
+    return new Date(Math.max(Date.parse(connectedAt), back)).toISOString();
+}
+async function fetchPage(deps, gateway, state, folder) {
     const pageSize = deps.config.pageSize;
-    const first = (s) => gateway.delta(s.link
-        ? { link: s.link, ...(pageSize ? { pageSize } : {}) }
+    const cursor = cursorOf(state, folder);
+    const first = (link) => gateway.delta(link
+        ? { link, ...(pageSize ? { pageSize } : {}) }
         : {
-            folderId: "inbox",
-            since: restartFrom(s, deps.config.restartOverlapMs ?? DEFAULT_RESTART_OVERLAP_MS),
+            folderId: folder,
+            since: restartFrom(state.connectedAt, cursor.lastReceivedAt, deps.config.restartOverlapMs ?? DEFAULT_RESTART_OVERLAP_MS),
             ...(pageSize ? { pageSize } : {}),
         });
     try {
-        return { page: await first(state), restarted: false };
+        return { page: await first(cursor.link), restarted: false };
     }
     catch (err) {
-        if (!state.link || !(0, errors_1.isDeltaLinkExpired)(err))
+        if (!cursor.link || !(0, errors_1.isDeltaLinkExpired)(err))
             throw err;
-        return { page: await first({ ...state, link: null }), restarted: true };
+        return { page: await first(null), restarted: true };
     }
 }
-function emptyResult() {
+function emptyResult(folder) {
     return {
+        folder,
         fetched: 0,
         created: 0,
         appended: 0,
@@ -73,12 +86,13 @@ function emptyResult() {
         gone: 0,
         draftsStarted: 0,
         deliveryFailures: 0,
+        unmatched: 0,
         restarted: false,
         hasMore: false,
     };
 }
 async function reconcileExisting(deps, existing, result) {
-    if (existing.removedAt)
+    if (existing.removedAt || existing.direction === "outbound")
         return;
     const thread = await deps.store.getThread(existing.threadId);
     if (thread && (thread.state === "new" || thread.state === "drafted")) {
@@ -126,6 +140,7 @@ async function processNew(deps, gateway, mailbox, item, state, result) {
         noise: { action: noise.action, reason: noise.reason },
         suppressAutoResponse: noise.suppressAutoResponse === true,
         removedAt: null,
+        direction: "inbound",
     };
     if (noise.reason === "bounce") {
         const refs = [...new Set([...(0, bounce_1.bouncedReplyRefs)(msg.body?.content ?? ""), ...headerRefs(headers)])];
@@ -260,7 +275,7 @@ async function processItem(deps, gateway, mailbox, item, state, result) {
     }
     await processNew(deps, gateway, mailbox, item, state, result);
 }
-async function ingestPage(deps, mailbox) {
+async function ingestFolderPage(deps, mailbox, folder, process) {
     const controls = await (0, controls_1.loadControls)(deps);
     if (controls.paused)
         return (0, controls_1.paused)();
@@ -268,35 +283,142 @@ async function ingestPage(deps, mailbox) {
     const state = await deps.store.getSyncState(mailbox);
     if (!state)
         throw new errors_1.MailboxNotConnectedError(mailbox);
-    const { page, restarted } = await fetchPage(deps, gateway, state);
+    const { page, restarted } = await fetchPage(deps, gateway, state, folder);
     const link = page.nextLink ?? page.deltaLink;
     if (!link)
         throw new errors_1.InvalidDeltaPageError();
-    const result = emptyResult();
+    const result = emptyResult(folder);
     result.restarted = restarted;
     result.fetched = page.items.length;
     const stamp = (i) => i.receivedDateTime ?? "";
     const live = page.items.filter((i) => !i.removed).sort((a, b) => stamp(a).localeCompare(stamp(b)));
     const gone = page.items.filter((i) => i.removed);
-    let newest = state.lastReceivedAt;
+    let newest = cursorOf(state, folder).lastReceivedAt;
     for (const item of [...live, ...gone]) {
-        await processItem(deps, gateway, mailbox, item, state, result);
+        await process(deps, gateway, mailbox, item, state, result);
         if (item.receivedDateTime && (!newest || item.receivedDateTime > newest)) {
             newest = item.receivedDateTime;
         }
     }
-    await deps.store.saveSyncState(mailbox, { ...state, link, lastReceivedAt: newest });
+    await deps.store.saveSyncState(mailbox, withCursor(state, folder, { link, lastReceivedAt: newest }));
     result.hasMore = page.nextLink !== null;
     return result;
 }
+async function ingestPage(deps, mailbox) {
+    return ingestFolderPage(deps, mailbox, "inbox", processItem);
+}
+async function ingestSentPage(deps, mailbox) {
+    return ingestFolderPage(deps, mailbox, "sentitems", processSentItem);
+}
+async function processSentItem(deps, gateway, mailbox, item, state, result) {
+    if (item.removed) {
+        const stored = await deps.store.getMessage(mailbox, item.id);
+        if (stored)
+            await deps.store.markMessageRemoved(mailbox, item.id, (0, internal_1.nowIso)(deps));
+        result.removed++;
+        return;
+    }
+    if (item.isDraft) {
+        result.skippedDrafts++;
+        return;
+    }
+    if (await deps.store.getMessage(mailbox, item.id)) {
+        result.unchanged++;
+        return;
+    }
+    if (item.receivedDateTime && Date.parse(item.receivedDateTime) < Date.parse(state.connectedAt)) {
+        result.skippedPredates++;
+        return;
+    }
+    const msg = await gateway.getMessage(item.id, { includeHeaders: true, bodyType: "text" });
+    if (!msg) {
+        result.gone++;
+        return;
+    }
+    if (msg.isDraft === true) {
+        result.skippedDrafts++;
+        return;
+    }
+    const conversationId = msg.conversationId ?? item.conversationId ?? null;
+    const recipients = [...msg.toRecipients, ...msg.ccRecipients]
+        .map((a) => a.address.trim().toLowerCase())
+        .filter(Boolean);
+    const message = {
+        id: item.id,
+        mailbox,
+        internetMessageId: (0, threading_1.normalizeMessageId)(msg.internetMessageId ?? item.internetMessageId),
+        conversationId,
+        subject: msg.subject ?? item.subject ?? "",
+        fromAddress: mailbox.trim().toLowerCase(),
+        receivedAt: msg.sentDateTime ?? msg.receivedDateTime ?? item.receivedDateTime ?? (0, internal_1.nowIso)(deps),
+        noise: { action: "process", reason: "none" },
+        suppressAutoResponse: false,
+        removedAt: null,
+        direction: "outbound",
+    };
+    const headers = threadingHeaders(msg);
+    for (let attempt = 0; attempt < MAX_ATTEMPTS; attempt++) {
+        const thread = await matchSent(deps, { mailbox, headers, conversationId, recipients });
+        if (!thread) {
+            result.unmatched++;
+            return;
+        }
+        const outcome = await deps.store.appendMessage({ ...message, threadId: thread.id }, { threadId: thread.id, expectedState: thread.state, patch: {} });
+        if (outcome === "stale")
+            continue;
+        if (outcome === "duplicate")
+            result.duplicates++;
+        else
+            result.appended++;
+        return;
+    }
+    throw new errors_1.StaleThreadError(item.id);
+}
+async function matchSent(deps, input) {
+    const { mailbox, headers, conversationId, recipients } = input;
+    const refs = [...new Set(headerRefs(headers))];
+    const hits = refs.length ? await deps.store.findMessageThreads(mailbox, refs) : [];
+    const parent = (0, threading_1.normalizeMessageId)(headers.inReplyTo);
+    const direct = parent ? hits.find((h) => h.internetMessageId === parent) : undefined;
+    const headerIds = new Set(direct ? [direct.threadId] : hits.map((h) => h.threadId));
+    const candidates = new Map();
+    for (const id of headerIds) {
+        const t = await deps.store.getThread(id);
+        if (t && t.mailbox === mailbox)
+            candidates.set(t.id, t);
+    }
+    if (!direct && conversationId) {
+        for (const t of await deps.store.findThreadsByConversationId(mailbox, conversationId)) {
+            if (t.mailbox !== mailbox)
+                continue;
+            if (headerIds.has(t.id) || recipients.some((r) => t.participants.includes(r)))
+                candidates.set(t.id, t);
+        }
+    }
+    if (candidates.size !== 1)
+        return null;
+    return [...candidates.values()][0] ?? null;
+}
 async function syncMailbox(deps, mailbox, options = {}) {
     const results = [];
+    const maxPages = options.maxPages ?? 20;
     try {
-        for (let i = 0; i < (options.maxPages ?? 20); i++) {
+        let inboxDone = false;
+        for (let i = 0; i < maxPages; i++) {
             const r = await ingestPage(deps, mailbox);
             if ((0, controls_1.isPaused)(r))
                 return results.length ? results : r;
             results.push(r);
+            if (!r.hasMore) {
+                inboxDone = true;
+                break;
+            }
+        }
+        for (let i = 0; inboxDone && i < maxPages; i++) {
+            const r = await ingestSentPage(deps, mailbox);
+            if ((0, controls_1.isPaused)(r))
+                return results;
+            results.push(r);
             if (!r.hasMore)
                 break;
         }

package/dist/react.js

@@ -414,6 +414,7 @@ function ThreadView({ cls, api, fetchImpl, threadId, pollIntervalMs, renderCusto
     const can = (action) => detail.actions.includes(action);
     const bounced = thread.state === "review" && thread.reviewReason === "delivery_failed";
     const sentReply = send?.status === "sent" && draft?.text && !draft.stale ? { text: draft.text, at: send.at } : null;
+    const syncedCopy = send?.messageId ? detail.messages.some((m) => m.id === send.messageId && !m.removed) : false;
     const showComposer = !bounced && !sentReply && (draft !== null || thread.state === "review");
     const menuItems = [];
     if (!bounced && can("dismiss"))
@@ -426,7 +427,7 @@ function ThreadView({ cls, api, fetchImpl, threadId, pollIntervalMs, renderCusto
     return ((0, jsx_runtime_1.jsxs)("article", { "aria-labelledby": `${id}-subject`, className: cls.detailPane, "data-thread-state": thread.state, "data-write-mode": pilot.writeMode, children: [(0, jsx_runtime_1.jsxs)("header", { className: cls.header, children: [(0, jsx_runtime_1.jsx)("button", { type: "button", className: cls.backLink, onClick: onBack, children: "Terug naar tickets" }), (0, jsx_runtime_1.jsxs)("div", { className: cls.headerTop, children: [(0, jsx_runtime_1.jsxs)("div", { className: "flex min-w-0 flex-col gap-1.5", children: [(0, jsx_runtime_1.jsx)("h2", { id: `${id}-subject`, ref: headingRef, tabIndex: -1, className: cls.heading, children: thread.subject || "(geen onderwerp)" }), (0, jsx_runtime_1.jsxs)("div", { className: cls.meta, children: [(0, jsx_runtime_1.jsx)(StatusBadge, { cls: cls, state: thread.state, reason: thread.reviewReason, withNote: false }), pilot.writeMode === "read_only" && (0, jsx_runtime_1.jsx)("span", { className: cls.badge, children: "Alleen lezen" }), (0, jsx_runtime_1.jsx)("span", { children: thread.customerAddress }), (0, jsx_runtime_1.jsx)("span", { "aria-hidden": "true", children: "\u00B7" }), (0, jsx_runtime_1.jsxs)("span", { children: ["naar ", thread.mailbox] })] }), (thread.relatedThreadId || (thread.state === "ignored" && thread.dismissedBy)) && ((0, jsx_runtime_1.jsxs)("div", { className: cls.meta, children: [thread.relatedThreadId && (0, jsx_runtime_1.jsx)("span", { children: "Vervolg op een eerder ticket" }), thread.state === "ignored" && thread.dismissedBy && ((0, jsx_runtime_1.jsxs)("span", { children: ["Geen antwoord nodig, gemarkeerd door ", thread.dismissedBy === detail.viewer.id ? "jou" : "een collega"] }))] }))] }), (0, jsx_runtime_1.jsxs)("div", { className: cls.headerActions, children: [(0, jsx_runtime_1.jsx)(Assignment, { cls: cls, detail: detail, busy: busy, run: run }), menuItems.length > 0 && (0, jsx_runtime_1.jsx)(MoreActions, { cls: cls, busy: busy, items: menuItems })] })] }), confirming && ((0, jsx_runtime_1.jsx)(ConfirmAction, { cls: cls, kind: confirming, busy: busy, onCancel: () => setConfirming(null), onConfirm: () => {
                             setConfirming(null);
                             run(confirming === "dismiss" ? "dismiss" : "replied-externally");
-                        } }))] }), (0, jsx_runtime_1.jsx)("div", { "aria-live": "polite", role: "status", children: notice && notice.kind !== "error" && (0, jsx_runtime_1.jsx)("p", { className: notice.kind === "success" ? cls.success : cls.warning, children: notice.text }) }), notice?.kind === "error" && ((0, jsx_runtime_1.jsx)("p", { role: "alert", className: cls.error, children: notice.text })), renderCustomerContext && (0, jsx_runtime_1.jsx)("section", { className: cls.context, children: renderCustomerContext(thread) }), (0, jsx_runtime_1.jsx)(Conversation, { cls: cls, detail: detail, bodies: bodies.current, sentReply: sentReply }), bounced && (can("replied_externally") || can("dismiss")) && (0, jsx_runtime_1.jsx)(BouncedNextSteps, { cls: cls, busy: busy, can: can, run: run }), showComposer && ((0, jsx_runtime_1.jsx)(Composer, { cls: cls, detail: detail, busy: busy, text: text, dirty: dirty, onText: (value) => {
+                        } }))] }), (0, jsx_runtime_1.jsx)("div", { "aria-live": "polite", role: "status", children: notice && notice.kind !== "error" && (0, jsx_runtime_1.jsx)("p", { className: notice.kind === "success" ? cls.success : cls.warning, children: notice.text }) }), notice?.kind === "error" && ((0, jsx_runtime_1.jsx)("p", { role: "alert", className: cls.error, children: notice.text })), renderCustomerContext && (0, jsx_runtime_1.jsx)("section", { className: cls.context, children: renderCustomerContext(thread) }), (0, jsx_runtime_1.jsx)(Conversation, { cls: cls, detail: detail, bodies: bodies.current, sentReply: syncedCopy ? null : sentReply }), bounced && (can("replied_externally") || can("dismiss")) && (0, jsx_runtime_1.jsx)(BouncedNextSteps, { cls: cls, busy: busy, can: can, run: run }), showComposer && ((0, jsx_runtime_1.jsx)(Composer, { cls: cls, detail: detail, busy: busy, text: text, dirty: dirty, onText: (value) => {
                     setText(value);
                     setDirty(true);
                 }, act: act, run: run, can: can, uploadFile: uploadFile })), showComposer && thread.state === "review" && (can("dismiss") || can("replied_externally")) && ((0, jsx_runtime_1.jsxs)("p", { className: cls.muted, children: ["Geen antwoord nodig of al vanuit Outlook beantwoord? Kies het bij ", (0, jsx_runtime_1.jsx)("strong", { className: "font-medium text-foreground", children: "Meer acties" }), " ", "bovenaan."] })), (0, jsx_runtime_1.jsx)(OtherActions, { cls: cls, busy: busy, can: can, run: run })] }));
@@ -486,17 +487,26 @@ function FileChips({ cls, attachments }) {
         }) }));
 }
 function Conversation({ cls, detail, bodies, sentReply, }) {
-    const entries = detail.messages.map((m) => ({ kind: m.bounce ? "bounce" : "mail", at: m.receivedAt, message: m }));
+    const copyId = sentReply ? detail.send?.messageId : null;
+    const entries = detail.messages.filter((m) => !(copyId && m.id === copyId)).map((m) => ({
+        kind: m.outbound ? "sent" : m.bounce ? "bounce" : "mail",
+        at: m.receivedAt,
+        message: m,
+    }));
     if (sentReply)
         entries.push({ kind: "reply", at: sentReply.at, text: sentReply.text });
     entries.sort((a, b) => a.at.localeCompare(b.at));
     const sentFiles = (detail.replyAttachments ?? []).filter((a) => a.sent);
-    return ((0, jsx_runtime_1.jsx)("ol", { "aria-label": "Gesprek", className: cls.conversation, children: entries.map((e) => e.kind === "reply" ? ((0, jsx_runtime_1.jsxs)("li", { className: cls.entryOurs, children: [(0, jsx_runtime_1.jsx)("span", { className: cls.avatarOurs, children: (0, jsx_runtime_1.jsx)(Icon, { name: "send", className: cls.icon }) }), (0, jsx_runtime_1.jsxs)("article", { className: cls.messageOurs, children: [(0, jsx_runtime_1.jsxs)("div", { className: cls.messageHeader, children: [(0, jsx_runtime_1.jsxs)("span", { children: [(0, jsx_runtime_1.jsx)("strong", { className: "font-semibold", children: "Jouw antwoord" }), " ", (0, jsx_runtime_1.jsxs)("span", { className: "text-muted-foreground", children: ["\u00B7 vanaf ", detail.thread.mailbox] })] }), (0, jsx_runtime_1.jsx)("time", { dateTime: e.at, className: cls.muted, children: formatDate(e.at) })] }), (0, jsx_runtime_1.jsx)("div", { className: cls.messageBody, children: e.text }), sentFiles.length > 0 && ((0, jsx_runtime_1.jsx)("ul", { "aria-label": "Verstuurde bijlagen", className: cls.fileChips, children: sentFiles.map((a) => ((0, jsx_runtime_1.jsxs)("li", { className: cls.replyChip, children: [(0, jsx_runtime_1.jsx)("span", { className: "break-all", children: a.name }), " ", (0, jsx_runtime_1.jsx)("span", { className: cls.fileChipMeta, children: formatSize(a.size) })] }, a.id))) })), (0, jsx_runtime_1.jsxs)("span", { className: cls.sentNote, children: [(0, jsx_runtime_1.jsx)(Icon, { name: "check", className: cls.icon }), "Verstuurd, staat in Verzonden items"] })] })] }, "reply")) : e.kind === "bounce" ? ((0, jsx_runtime_1.jsx)(BounceItem, { cls: cls, message: e.message, body: bodies.get(e.message.id) }, e.message.id)) : ((0, jsx_runtime_1.jsx)(MessageItem, { cls: cls, message: e.message, body: bodies.get(e.message.id) }, e.message.id))) }));
+    return ((0, jsx_runtime_1.jsx)("ol", { "aria-label": "Gesprek", className: cls.conversation, children: entries.map((e) => e.kind === "reply" ? ((0, jsx_runtime_1.jsxs)("li", { className: cls.entryOurs, children: [(0, jsx_runtime_1.jsx)("span", { className: cls.avatarOurs, children: (0, jsx_runtime_1.jsx)(Icon, { name: "send", className: cls.icon }) }), (0, jsx_runtime_1.jsxs)("article", { className: cls.messageOurs, children: [(0, jsx_runtime_1.jsxs)("div", { className: cls.messageHeader, children: [(0, jsx_runtime_1.jsxs)("span", { children: [(0, jsx_runtime_1.jsx)("strong", { className: "font-semibold", children: "Jouw antwoord" }), " ", (0, jsx_runtime_1.jsxs)("span", { className: "text-muted-foreground", children: ["\u00B7 vanaf ", detail.thread.mailbox] })] }), (0, jsx_runtime_1.jsx)("time", { dateTime: e.at, className: cls.muted, children: formatDate(e.at) })] }), (0, jsx_runtime_1.jsx)("div", { className: cls.messageBody, children: e.text }), sentFiles.length > 0 && ((0, jsx_runtime_1.jsx)("ul", { "aria-label": "Verstuurde bijlagen", className: cls.fileChips, children: sentFiles.map((a) => ((0, jsx_runtime_1.jsxs)("li", { className: cls.replyChip, children: [(0, jsx_runtime_1.jsx)("span", { className: "break-all", children: a.name }), " ", (0, jsx_runtime_1.jsx)("span", { className: cls.fileChipMeta, children: formatSize(a.size) })] }, a.id))) })), (0, jsx_runtime_1.jsxs)("span", { className: cls.sentNote, children: [(0, jsx_runtime_1.jsx)(Icon, { name: "check", className: cls.icon }), "Verstuurd, staat in Verzonden items"] })] })] }, "reply")) : e.kind === "bounce" ? ((0, jsx_runtime_1.jsx)(BounceItem, { cls: cls, message: e.message, body: bodies.get(e.message.id) }, e.message.id)) : e.kind === "sent" ? ((0, jsx_runtime_1.jsx)(SentItem, { cls: cls, message: e.message, mailbox: detail.thread.mailbox, body: bodies.get(e.message.id) }, e.message.id)) : ((0, jsx_runtime_1.jsx)(MessageItem, { cls: cls, message: e.message, body: bodies.get(e.message.id) }, e.message.id))) }));
 }
 function MessageItem({ cls, message, body }) {
     return ((0, jsx_runtime_1.jsxs)("li", { className: cls.entry, children: [(0, jsx_runtime_1.jsx)("span", { className: cls.avatar, "aria-hidden": "true", children: initials(message.fromAddress) }), (0, jsx_runtime_1.jsxs)("article", { className: cls.message, children: [(0, jsx_runtime_1.jsxs)("div", { className: cls.messageHeader, children: [(0, jsx_runtime_1.jsxs)("span", { className: "min-w-0 break-words", children: [(0, jsx_runtime_1.jsx)("strong", { className: "font-semibold", children: message.fromAddress }), " ", (0, jsx_runtime_1.jsx)("span", { className: "text-muted-foreground", children: "\u00B7 klant" }), message.noise && (0, jsx_runtime_1.jsx)("span", { className: "text-muted-foreground", children: " \u00B7 automatisch bericht" }), message.removed && (0, jsx_runtime_1.jsx)("span", { className: "text-muted-foreground", children: " \u00B7 niet meer in de Inbox" })] }), (0, jsx_runtime_1.jsx)("time", { dateTime: message.receivedAt, className: cls.muted, children: formatDate(message.receivedAt) })] }), body === undefined && (0, jsx_runtime_1.jsx)("p", { className: cls.muted, children: "Bericht laden\u2026" }), body && "error" in body && (0, jsx_runtime_1.jsx)("p", { className: cls.muted, children: body.error }), body && "text" in body &&
                         (body.text ? ((0, jsx_runtime_1.jsx)("div", { className: cls.messageBody, children: body.text })) : ((0, jsx_runtime_1.jsx)("p", { className: cls.emptyBody, children: body.attachments.length > 0 ? "Geen tekst, alleen een bijlage." : "Geen tekst." }))), body && "text" in body && (0, jsx_runtime_1.jsx)(FileChips, { cls: cls, attachments: body.attachments })] })] }));
 }
+function SentItem({ cls, message, mailbox, body }) {
+    return ((0, jsx_runtime_1.jsxs)("li", { className: cls.entryOurs, "data-outbound": "", children: [(0, jsx_runtime_1.jsx)("span", { className: cls.avatarOurs, children: (0, jsx_runtime_1.jsx)(Icon, { name: "send", className: cls.icon }) }), (0, jsx_runtime_1.jsxs)("article", { className: cls.messageOurs, children: [(0, jsx_runtime_1.jsxs)("div", { className: cls.messageHeader, children: [(0, jsx_runtime_1.jsxs)("span", { className: "min-w-0 break-words", children: [(0, jsx_runtime_1.jsx)("strong", { className: "font-semibold", children: "Jouw antwoord" }), " ", (0, jsx_runtime_1.jsxs)("span", { className: "text-muted-foreground", children: ["\u00B7 vanaf ", message.fromAddress || mailbox] }), message.removed && (0, jsx_runtime_1.jsx)("span", { className: "text-muted-foreground", children: " \u00B7 niet meer in Verzonden items" })] }), (0, jsx_runtime_1.jsx)("time", { dateTime: message.receivedAt, className: cls.muted, children: formatDate(message.receivedAt) })] }), body === undefined && (0, jsx_runtime_1.jsx)("p", { className: cls.muted, children: "Bericht laden\u2026" }), body && "error" in body && (0, jsx_runtime_1.jsx)("p", { className: cls.muted, children: body.error }), body && "text" in body &&
+                        (body.text ? ((0, jsx_runtime_1.jsx)("div", { className: cls.messageBody, children: body.text })) : ((0, jsx_runtime_1.jsx)("p", { className: cls.emptyBody, children: body.attachments.length > 0 ? "Geen tekst, alleen een bijlage." : "Geen tekst." }))), body && "text" in body && (0, jsx_runtime_1.jsx)(FileChips, { cls: cls, attachments: body.attachments }), !message.removed && ((0, jsx_runtime_1.jsxs)("span", { className: cls.sentNote, children: [(0, jsx_runtime_1.jsx)(Icon, { name: "check", className: cls.icon }), "Verstuurd, staat in Verzonden items"] }))] })] }));
+}
 function BounceItem({ cls, message, body }) {
     const report = body && "text" in body ? body.bounce : null;
     const reason = report?.reason ?? "De ontvangende server weigerde het bericht.";

package/dist/testing/index.d.ts

@@ -26,6 +26,10 @@ export interface FakeMail {
     isDraft?: boolean;
     bodyText?: string;
     attachments?: FakeAttachment[];
+    folder?: "inbox" | "sentitems";
+    to?: string[];
+    cc?: string[];
+    sentDateTime?: string;
 }
 export interface FakeAttachment {
     id: string;

package/dist/testing/index.js

@@ -63,7 +63,7 @@ function createMemoryStore() {
         async getLatestMessageForThread(threadId) {
             await tick();
             const last = [...messages.values()]
-                .filter((m) => m.threadId === threadId)
+                .filter((m) => m.threadId === threadId && m.direction === "inbound")
                 .sort((a, b) => a.receivedAt.localeCompare(b.receivedAt))
                 .at(-1);
             return last ? clone(last) : null;
@@ -184,7 +184,8 @@ function createMemoryStore() {
 const WRITE_METHODS = ["createReply", "updateDraftBody", "sendDraft"];
 function createFakeMailbox(address) {
     const mails = new Map();
-    const log = [];
+    const logs = { inbox: [], sentitems: [] };
+    const folderOf = (id) => (mails.get(id)?.folder ?? "inbox");
     const drafts = new Map();
     let expired = false;
     let draftCounter = 0;
@@ -192,7 +193,7 @@ function createFakeMailbox(address) {
         id: m.id,
         removed: false,
         ...(m.subject !== undefined ? { subject: m.subject } : {}),
-        ...(m.from ? { from: { address: m.from } } : {}),
+        ...(m.from ? { from: { address: m.from } } : m.folder === "sentitems" ? { from: { address } } : {}),
         ...(m.receivedDateTime ? { receivedDateTime: m.receivedDateTime } : {}),
         ...(m.internetMessageId ? { internetMessageId: m.internetMessageId } : {}),
         ...(m.conversationId ? { conversationId: m.conversationId } : {}),
@@ -215,16 +216,17 @@ function createFakeMailbox(address) {
         },
         deliver(mail) {
             mails.set(mail.id, mail);
-            log.push(summary(mail));
+            logs[mail.folder ?? "inbox"].push(summary(mail));
         },
         touch(id) {
             const m = mails.get(id);
             if (m)
-                log.push({ ...summary(m), isRead: true, categories: ["seen"] });
+                logs[m.folder ?? "inbox"].push({ ...summary(m), isRead: true, categories: ["seen"] });
         },
         remove(id, reason = "deleted") {
+            const folder = folderOf(id);
             mails.delete(id);
-            log.push({ id, removed: true, removedReason: reason });
+            logs[folder].push({ id, removed: true, removedReason: reason });
         },
         expireLinks() {
             expired = true;
@@ -237,11 +239,15 @@ function createFakeMailbox(address) {
                 throw box.outage;
             const size = options.pageSize ?? 50;
             let start = 0;
+            let folder = options.folderId === "sentitems" ? "sentitems" : "inbox";
             if (options.link) {
                 if (expired)
                     throw new errors_1.DeltaLinkExpiredError();
-                start = Number(options.link.split(":")[1]);
+                folder = options.link.startsWith("s:") ? "sentitems" : "inbox";
+                start = Number(options.link.split(":").at(-1));
             }
+            const log = logs[folder];
+            const prefix = folder === "sentitems" ? "s:" : "";
             const since = options.since ? Date.parse(String(options.since)) : null;
             const visible = (i) => Boolean(options.link) ||
                 since === null ||
@@ -253,8 +259,8 @@ function createFakeMailbox(address) {
             const more = end < log.length;
             return {
                 items: slice.filter(visible),
-                nextLink: more ? `p:${end}` : null,
-                deltaLink: more ? null : `d:${end}`,
+                nextLink: more ? `${prefix}p:${end}` : null,
+                deltaLink: more ? null : `${prefix}d:${end}`,
             };
         },
         async getMessage(id, options) {
@@ -272,8 +278,9 @@ function createFakeMailbox(address) {
             return {
                 ...summary(m),
                 body: { contentType: "text", content: m.bodyText ?? "hello" },
-                toRecipients: [{ address }],
-                ccRecipients: [],
+                ...(m.sentDateTime ? { sentDateTime: m.sentDateTime } : {}),
+                toRecipients: (m.to ?? (m.folder === "sentitems" ? [] : [address])).map((a) => ({ address: a })),
+                ccRecipients: (m.cc ?? []).map((a) => ({ address: a })),
                 replyTo: (m.replyTo ?? []).map((address) => ({ address })),
                 ...(options?.includeHeaders
                     ? {

package/dist/types.d.ts

@@ -161,6 +161,7 @@ export interface ThreadUpdate {
     patch: ThreadPatch;
 }
 export type NoiseReason = "auto_submitted" | "precedence" | "bounce" | "out_of_office" | "mailing_list" | "machine_sender" | "own_address" | "connected_mailbox" | "none";
+export type MessageDirection = "inbound" | "outbound";
 export interface StoredMessage {
     id: string;
     mailbox: string;
@@ -176,6 +177,7 @@ export interface StoredMessage {
     };
     suppressAutoResponse: boolean;
     removedAt: string | null;
+    direction: MessageDirection;
 }
 export interface SyncState {
     link: string | null;
@@ -184,6 +186,8 @@ export interface SyncState {
     failingSince?: string | null;
     failures?: number;
     lastError?: string | null;
+    sentLink?: string | null;
+    sentLastReceivedAt?: string | null;
 }
 export interface SyncFailing {
     status: "failing";

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@dxv-systems/helpdesk",
-  "version": "0.2.0",
+  "version": "0.3.0",
   "description": "Mailbox helpdesk domain logic for DXV apps — idempotent delta ingestion, conservative threading, ticket state machine, noise detection, and replies that a person approves in the helpdesk before they are sent from the mailbox (quoted history kept, retry-safe, never sent twice). Pure logic behind interfaces, plus the staff inbox as a route handler (./server) and a React component (./react). Zero runtime dependencies.",
   "keywords": [
     "dxv",