@remit/web-client 0.0.321 → 0.0.323

package/src/hooks/useWarningsAttention.ts

@@ -0,0 +1,37 @@
+import { meOperationsListQuarantineOptions } from "@remit/api-http-client/@tanstack/react-query.gen.ts";
+import type { RemitImapQuarantineListResponse } from "@remit/api-http-client/types.gen.ts";
+import { useQuery } from "@tanstack/react-query";
+import { isUnauthenticated, softErrorStatuses } from "@/lib/error-classifier";
+
+const WARNINGS_LIST_META = softErrorStatuses(401);
+
+export function useWarningsList() {
+	return useQuery({
+		...meOperationsListQuarantineOptions(),
+		meta: WARNINGS_LIST_META,
+	});
+}
+
+export const WARNINGS_ATTENTION = {
+	listed: "Sync warnings",
+	unread: "Warnings could not be read",
+	signedOut: "Session ended",
+} as const;
+
+export function warningsAttentionOf(
+	data: RemitImapQuarantineListResponse | undefined,
+	error: unknown,
+): string | undefined {
+	if (isUnauthenticated(error)) return WARNINGS_ATTENTION.signedOut;
+	if (error) return WARNINGS_ATTENTION.unread;
+	if (!data) return undefined;
+	if ((data.entries?.length ?? 0) > 0 || (data.warnings?.length ?? 0) > 0) {
+		return WARNINGS_ATTENTION.listed;
+	}
+	return undefined;
+}
+
+export function useWarningsAttention(): string | undefined {
+	const list = useWarningsList();
+	return warningsAttentionOf(list.data, list.error);
+}

package/src/lib/account-order.test.ts

@@ -39,12 +39,6 @@ describe("compareAccounts", () => {
 		assert.ok(compareAccounts(a, b) < 0);
 		assert.ok(compareAccounts(b, a) > 0);
 	});
-
-	test("identical accounts compare equal", () => {
-		const a = account({ accountId: "x", createdAt: 1000 });
-		const b = account({ accountId: "x", createdAt: 1000 });
-		assert.strictEqual(compareAccounts(a, b), 0);
-	});
 });
 
 describe("sortAccountsByCreatedAt", () => {
@@ -77,8 +71,4 @@ describe("sortAccountsByCreatedAt", () => {
 			["a", "m", "z"],
 		);
 	});
-
-	test("empty input returns empty array", () => {
-		assert.deepStrictEqual(sortAccountsByCreatedAt([]), []);
-	});
 });

package/src/lib/adjacent-message.test.ts

@@ -31,13 +31,4 @@ describe("adjacentMessageId", () => {
 	test("returns null when there is no current id", () => {
 		assert.strictEqual(adjacentMessageId(ids, undefined, "next"), null);
 	});
-
-	test("returns null for a single-item list at both ends", () => {
-		assert.strictEqual(adjacentMessageId(["only"], "only", "next"), null);
-		assert.strictEqual(adjacentMessageId(["only"], "only", "previous"), null);
-	});
-
-	test("returns null for an empty list", () => {
-		assert.strictEqual(adjacentMessageId([], "a", "next"), null);
-	});
 });

package/src/lib/app-info.ts

@@ -30,6 +30,3 @@ export const APP_SHA_IS_COMMIT = /^[0-9a-f]{40}$/.test(APP_SHA);
 export const GITHUB_COMMIT_URL: string | undefined = APP_SHA_IS_COMMIT
 	? `https://github.com/remit-mail/reader/commit/${APP_SHA}`
 	: undefined;
-
-export const GITHUB_NEW_ISSUE_URL =
-	"https://github.com/remit-mail/reader/issues/new";

package/src/lib/autodiscovery.test.ts

@@ -8,53 +8,6 @@ import {
 
 // Provider table unit tests for top 5 providers (no network calls)
 describe("autodiscovery provider table", () => {
-	it("resolves Gmail", async () => {
-		const result = await discoverSettings("user@gmail.com");
-		assert.ok(result, "should find Gmail");
-		assert.equal(result.source, "provider-table");
-		assert.equal(result.imap.host, "imap.gmail.com");
-		assert.equal(result.imap.port, 993);
-		assert.equal(result.imap.security, "tls");
-		assert.equal(result.smtp.host, "smtp.gmail.com");
-		assert.equal(result.smtp.port, 587);
-		assert.equal(result.smtp.security, "starttls");
-	});
-
-	it("resolves iCloud", async () => {
-		const result = await discoverSettings("user@icloud.com");
-		assert.ok(result, "should find iCloud");
-		assert.equal(result.source, "provider-table");
-		assert.equal(result.imap.host, "imap.mail.me.com");
-		assert.equal(result.imap.port, 993);
-		assert.equal(result.smtp.security, "starttls");
-	});
-
-	it("resolves Fastmail", async () => {
-		const result = await discoverSettings("user@fastmail.com");
-		assert.ok(result, "should find Fastmail");
-		assert.equal(result.source, "provider-table");
-		assert.equal(result.imap.host, "imap.fastmail.com");
-		assert.equal(result.smtp.host, "smtp.fastmail.com");
-		assert.equal(result.smtp.port, 465);
-		assert.equal(result.smtp.security, "tls");
-	});
-
-	it("resolves Outlook", async () => {
-		const result = await discoverSettings("user@outlook.com");
-		assert.ok(result, "should find Outlook");
-		assert.equal(result.source, "provider-table");
-		assert.equal(result.imap.host, "outlook.office365.com");
-		assert.equal(result.smtp.host, "smtp.office365.com");
-	});
-
-	it("resolves Yahoo", async () => {
-		const result = await discoverSettings("user@yahoo.com");
-		assert.ok(result, "should find Yahoo");
-		assert.equal(result.source, "provider-table");
-		assert.equal(result.imap.host, "imap.mail.yahoo.com");
-		assert.equal(result.smtp.security, "tls");
-	});
-
 	it("returns heuristic for unknown domain", async () => {
 		// Use a clearly fake domain — network calls will fail/timeout
 		// In the test environment there's no network, so it falls to heuristic

package/src/lib/bug-report.test.ts

@@ -1,11 +1,14 @@
 import assert from "node:assert/strict";
 import { describe, it } from "node:test";
 import {
+	formatSyncWarningReport,
 	QUARANTINE_REPORT_DISCLAIMER,
 	type QuarantineEntry,
 	quarantineDemoEntries,
 	quarantineIssueTitle,
 	quarantineReportSections,
+	syncWarningDemoEntries,
+	syncWarningIssueTitle,
 } from "@remit/ui";
 import type { BugReportContext } from "./bug-report";
 import { buildBugReportDetails, buildGitHubIssueUrl } from "./bug-report";
@@ -45,6 +48,20 @@ describe("buildGitHubIssueUrl", () => {
 		);
 	});
 
+	it("points at the issues URL the runtime config names", () => {
+		globalThis.__REMIT_CONFIG__ = {
+			issuesUrl: "https://github.com/remit-mail/reader-cloudflare/issues/new",
+		};
+		const url = buildGitHubIssueUrl(baseCtx);
+		globalThis.__REMIT_CONFIG__ = undefined;
+		assert.ok(
+			url.startsWith(
+				"https://github.com/remit-mail/reader-cloudflare/issues/new?",
+			),
+			`Expected the configured issues URL, got: ${url}`,
+		);
+	});
+
 	it("prefills a title param", () => {
 		const url = buildGitHubIssueUrl(baseCtx);
 		assert.ok(url.includes("title="), "Expected a title param");
@@ -447,3 +464,24 @@ describe("quarantine reports", () => {
 		);
 	});
 });
+
+describe("buildGitHubIssueUrl for a sync warning", () => {
+	const entry = {
+		...syncWarningDemoEntries[1],
+		folderLabel: "Tax return 2026",
+	};
+
+	it("files the stage, state and attempts without the folder name", () => {
+		const decoded = decode(
+			buildGitHubIssueUrl({
+				...baseCtx,
+				title: syncWarningIssueTitle(entry),
+				syncWarning: formatSyncWarningReport(entry),
+			}),
+		);
+		assert.ok(decoded.includes("Message failed to sync: Retrying at Header"));
+		assert.ok(decoded.includes("**Attempts**: 7"));
+		assert.ok(decoded.includes("abcdef1"));
+		assert.ok(!decoded.includes("Tax return 2026"));
+	});
+});

package/src/lib/bug-report.ts

@@ -1,10 +1,6 @@
 import type { QuarantineReportSections } from "@remit/ui";
-import {
-	APP_BUILD_TIME,
-	APP_SHORT_SHA,
-	GITHUB_COMMIT_URL,
-	GITHUB_NEW_ISSUE_URL,
-} from "./app-info";
+import { getRuntimeConfig } from "../runtime-config";
+import { APP_BUILD_TIME, APP_SHORT_SHA, GITHUB_COMMIT_URL } from "./app-info";
 import { getRecentErrors } from "./console-errors";
 import {
 	getFailingRequest,
@@ -53,6 +49,7 @@ export interface BugReportContext {
 	 * @remit/ui so the MIME tree is fenced after truncation, never before.
 	 */
 	quarantine?: QuarantineReportSections;
+	syncWarning?: string;
 	/** Overrides the derived title, e.g. for a quarantine report. */
 	title?: string;
 }
@@ -63,6 +60,7 @@ export interface BugReportSeed {
 	stack?: string;
 	componentStack?: string;
 	quarantine?: QuarantineReportSections;
+	syncWarning?: string;
 	/** Overrides the derived title, e.g. for a quarantine report. */
 	title?: string;
 }
@@ -85,6 +83,7 @@ export function buildBugReportContext(seed?: BugReportSeed): BugReportContext {
 		stack: seed?.stack,
 		componentStack: seed?.componentStack,
 		quarantine: seed?.quarantine,
+		syncWarning: seed?.syncWarning,
 		title: seed?.title,
 	};
 }
@@ -194,6 +193,10 @@ function buildIssueBody(
 		);
 	}
 
+	if (ctx.syncWarning) {
+		lines.push(ctx.syncWarning, "");
+	}
+
 	const resolvedStack = overrides?.stack ?? ctx.stack;
 	if (resolvedStack) {
 		lines.push("## Stacktrace", fencedBlock(resolvedStack), "");
@@ -236,7 +239,7 @@ function issueUrl(ctx: BugReportContext, overrides?: BodyOverrides): string {
 		title: buildIssueTitle(ctx),
 		body: buildIssueBody(ctx, overrides),
 	});
-	return `${GITHUB_NEW_ISSUE_URL}?${params.toString()}`;
+	return `${getRuntimeConfig().issuesUrl}?${params.toString()}`;
 }
 
 /**

package/src/lib/bulk-action-copy.ts

@@ -1,4 +1,4 @@
-import type { ErrorBannerSeverity } from "@/components/ui/error-banners";
+import type { ErrorBannerSeverity } from "@remit/ui";
 import type { BulkRunOutcome } from "@/lib/bulk-actions";
 import { type DeleteOutcome, deleteExpunges, formatNumber } from "@/lib/format";
 

package/src/lib/calendar-route.test.ts

@@ -5,7 +5,6 @@
  */
 import assert from "node:assert/strict";
 import { describe, it } from "node:test";
-import { CALENDAR_DENSITY_KEY } from "./calendar-density.js";
 import {
 	calendarSearchSchema,
 	calendarViewMountsAgenda,
@@ -219,10 +218,6 @@ describe("calendarSearchSchema", () => {
 });
 
 describe("density is not URL state", () => {
-	it("is stored against the device, under a key of its own", () => {
-		assert.equal(CALENDAR_DENSITY_KEY, "remit:calendar-density");
-	});
-
 	it("has no reading in the query", () => {
 		assert.deepEqual(calendarSearchSchema.parse({ density: "compact" }), {});
 	});

package/src/lib/config-import.test.ts

@@ -135,7 +135,7 @@ test("a stopped import splits the write order into landed, failed and never reac
 				{
 					code: "import_write_failed",
 					message:
-						"the filter store refused the write The items above it were written and remain; the import stopped here.",
+						"the filter store refused the write The items above it were written and remain; import the same file again with merge to finish.",
 					details: { section: "filters", key: "Receipts" },
 				},
 			],
@@ -151,38 +151,7 @@ test("a stopped import splits the write order into landed, failed and never reac
 	assert.equal(byId.get("settings")?.state, "not-attempted");
 });
 
-test("an import the server says wrote nothing marks no section landed, whatever ran before the stop", () => {
-	const results = sectionResults(
-		reportOf({
-			applied: false,
-			items: [
-				item("filters", "Receipts", "rejected", "the store refused the write"),
-			],
-			errors: [
-				{
-					code: "import_write_failed",
-					message:
-						"the store refused the write Nothing was written; the import stopped here.",
-					details: { section: "filters", key: "Receipts" },
-				},
-			],
-		}),
-	);
-
-	const byId = new Map(results.map((result) => [result.section, result]));
-	assert.equal(byId.get("accounts")?.state, "not-landed");
-	assert.match(
-		byId.get("accounts")?.detail ?? "",
-		/Nothing from this section was written/,
-	);
-	assert.equal(byId.get("labels")?.state, "not-landed");
-	assert.equal(byId.get("filters")?.state, "failed");
-	assert.equal(byId.get("addressFlags")?.state, "not-attempted");
-	assert.equal(byId.get("settings")?.state, "not-attempted");
-	assert.equal(results.filter((result) => result.state === "landed").length, 0);
-});
-
-test("a first write that fails on a store with no transaction lands nothing either", () => {
+test("a first write that fails lands nothing", () => {
 	const results = sectionResults(
 		reportOf({
 			applied: false,
@@ -193,7 +162,7 @@ test("a first write that fails on a store with no transaction lands nothing eith
 				{
 					code: "import_write_failed",
 					message:
-						"the store refused The items above it were written and remain; the import stopped here.",
+						"the store refused The items above it were written and remain; import the same file again with merge to finish.",
 					details: { section: "accounts", key: "a@example.test" },
 				},
 			],
@@ -239,11 +208,6 @@ test("a pending folder names the setting waiting on it, and repeats are folded",
 					message: "not here yet",
 					details: { folderPath: "Raadsstukken", accountId: "acct-1" },
 				},
-				{
-					code: "anchor_not_embedded",
-					message: "carried as text",
-					details: { filter: "Waternet" },
-				},
 			],
 		}),
 	);

package/src/lib/dedupe-thread-messages.test.ts

@@ -76,8 +76,4 @@ describe("dedupeThreadMessages (#166)", () => {
 
 		assert.deepStrictEqual(got, items);
 	});
-
-	test("returns an empty array for empty input", () => {
-		assert.deepStrictEqual(dedupeThreadMessages([]), []);
-	});
 });

package/src/lib/desktop-breakpoint.test.ts

@@ -73,12 +73,4 @@ describe("the lg variant and the desktop hooks gate on the same condition", () =
 			);
 		});
 	}
-
-	it("excludes a coarse pointer held upright", () => {
-		assert.match(DESKTOP_MEDIA_QUERY, /min-width: 1024px/);
-		assert.match(
-			DESKTOP_MEDIA_QUERY,
-			/not \(\(orientation: portrait\) and \(pointer: coarse\)\)/,
-		);
-	});
 });

package/src/lib/drafts.test.ts

@@ -187,29 +187,6 @@ describe("groupDraftSections", () => {
 		assert.deepStrictEqual(sections, []);
 	});
 
-	test("section labels match spec", () => {
-		const draft = outboxMessage({
-			outboxMessageId: "d1",
-			accountId: "acc-1",
-			status: "draft",
-		});
-		const thread = threadMessage({
-			threadMessageId: "tm1",
-			threadId: "t1",
-			messageId: "m1",
-			accountConfigId: "acc-1",
-		});
-		const sections = groupDraftSections({
-			outboxMessages: [draft],
-			accountId: "acc-1",
-			imapThreads: [thread],
-		});
-		assert.deepStrictEqual(
-			sections.map((s) => s.label),
-			["Not yet sent (Remit)", "On the server"],
-		);
-	});
-
 	test("Remit-drafts section row id equals outboxMessageId", () => {
 		const draft = outboxMessage({
 			outboxMessageId: "outbox-abc",

package/src/lib/fatal-error.test.ts

@@ -6,7 +6,6 @@ import {
 	__resetFatalError,
 	getCurrentFatalError,
 	reportFatalError,
-	setFatalErrorTelemetry,
 	subscribeFatalError,
 } from "./fatal-error";
 
@@ -39,24 +38,6 @@ describe("reportFatalError", () => {
 		assert.equal(getCurrentFatalError(), fatal);
 	});
 
-	it("emits a telemetry fatal-error event through the registered sink", () => {
-		const recorded: Array<{ message: string; ctx?: Record<string, string> }> =
-			[];
-		setFatalErrorTelemetry({
-			recordPageView: () => {},
-			recordError: (error, ctx) =>
-				recorded.push({ message: error.message, ctx }),
-			recordEvent: () => {},
-			recordTiming: () => {},
-		});
-
-		reportFatalError(new Error("telemetry please"));
-
-		assert.equal(recorded.length, 1);
-		assert.equal(recorded[0].message, "telemetry please");
-		assert.equal(recorded[0].ctx?.fatal, "true");
-	});
-
 	it("unsubscribe stops further notifications", () => {
 		let count = 0;
 		const unsubscribe = subscribeFatalError(() => {

package/src/lib/folder-roles.test.ts

@@ -36,10 +36,6 @@ describe("buildMailboxRoleMap", () => {
 		]);
 		assert.equal(map.get("mb-1"), "inbox");
 	});
-
-	test("empty appointments produce an empty map", () => {
-		assert.equal(buildMailboxRoleMap([]).size, 0);
-	});
 });
 
 describe("labelForMailbox", () => {

package/src/lib/hot-sync-window.test.ts

@@ -2,7 +2,6 @@ import assert from "node:assert/strict";
 import { afterEach, describe, test } from "node:test";
 import {
 	__resetHotSyncWindow,
-	HOT_POLL_INTERVAL_MS,
 	HOT_WINDOW_MS,
 	isHotSyncWindowActive,
 	startHotSyncWindow,
@@ -57,8 +56,4 @@ describe("hot sync window", () => {
 		startHotSyncWindow(NOW);
 		assert.equal(presses, 1);
 	});
-
-	test("the hot cadence is thirty seconds", () => {
-		assert.equal(HOT_POLL_INTERVAL_MS, 30_000);
-	});
 });

package/src/lib/list-header-chrome.ts

@@ -28,7 +28,7 @@ export interface ListHeaderChrome {
 	/** The expanded field, which takes the title's place for as long as it is up. */
 	searchField: ReactNode;
 	/**
-	 * The read-only two-engine results panel, when a query is being typed and
+	 * The read-only results panel, when a query is being typed and
 	 * this view answers it with the panel rather than its own rows. The body
 	 * renders it in place of its list; `MailListHeader` cannot swap the body
 	 * out from above, because the header is inside it.

package/src/lib/mail-context.ts

@@ -21,7 +21,7 @@ export interface MailContextValue {
 	/**
 	 * Name indexes `parseSearchTokens` resolves `in:`/`account:` chips against
 	 * (#428 follow-up). Computed once here so every consumer — chip rendering,
-	 * the brief/flagged token filters, `useSemanticSearch` — agrees on the same
+	 * the brief/flagged token filters — agrees on the same
 	 * resolution. See `lib/search-token-index.ts`.
 	 */
 	mailboxNameIndex: ReadonlyMap<string, string>;

package/src/lib/message-attachments.test.ts

@@ -8,6 +8,7 @@ const part = (
 ): RemitImapBodyPartResponse =>
 	({
 		bodyPartId: "part-1",
+		partPath: "2",
 		mediaType: "APPLICATION",
 		mediaSubtype: "PDF",
 		sizeOctets: 2048,
@@ -83,4 +84,80 @@ describe("selectMessageAttachments", () => {
 		]);
 		assert.equal(attachment.typeLabel, "FILE");
 	});
+
+	describe("a forwarded message", () => {
+		const forwarded = part({
+			bodyPartId: "forwarded",
+			partPath: "2",
+			mediaType: "MESSAGE",
+			mediaSubtype: "rfc822",
+			dispositionFilename: "forwarded.eml",
+		});
+		const inner = [
+			part({
+				bodyPartId: "inner-root",
+				partPath: "2.0",
+				mediaType: "MULTIPART",
+				mediaSubtype: "mixed",
+				isMultipart: true,
+				disposition: undefined,
+			}),
+			part({
+				bodyPartId: "inner-text",
+				partPath: "2.1",
+				mediaType: "TEXT",
+				mediaSubtype: "plain",
+				disposition: undefined,
+			}),
+			part({
+				bodyPartId: "inner-pdf",
+				partPath: "2.2",
+				dispositionFilename: "original.pdf",
+			}),
+		];
+
+		it("is listed as a message, and the attachment inside it is not", () => {
+			const attachments = selectMessageAttachments([
+				forwarded,
+				...inner,
+				part({ bodyPartId: "outer-pdf", partPath: "3" }),
+			]);
+			assert.deepEqual(
+				attachments.map((a) => [a.bodyPartId, a.kind, a.filename]),
+				[
+					["forwarded", "message", "forwarded.eml"],
+					["outer-pdf", "file", "report.pdf"],
+				],
+			);
+		});
+
+		it("is listed whatever disposition the sender gave it", () => {
+			const [attachment] = selectMessageAttachments([
+				{ ...forwarded, disposition: undefined },
+			]);
+			assert.equal(attachment?.kind, "message");
+			assert.equal(attachment?.filename, "forwarded.eml");
+			assert.equal(attachment?.typeLabel, "MESSAGE");
+		});
+
+		it("lists its own attachment once it is the message being read", () => {
+			assert.deepEqual(
+				selectMessageAttachments(inner).map((a) => a.bodyPartId),
+				["inner-pdf"],
+			);
+		});
+
+		it("is not confused with a delivery report", () => {
+			assert.deepEqual(
+				selectMessageAttachments([
+					part({
+						mediaType: "MESSAGE",
+						mediaSubtype: "delivery-status",
+						disposition: undefined,
+					}),
+				]),
+				[],
+			);
+		});
+	});
 });

package/src/lib/message-attachments.ts

@@ -1,15 +1,22 @@
 import type { RemitImapBodyPartResponse } from "@remit/api-http-client/types.gen.ts";
+import {
+	isEncapsulatedMessage,
+	ownParts,
+} from "@remit/data-ports/message-parts";
 import { ContentDisposition } from "@remit/domain-enums";
-import { sanitizeAttachmentFilename } from "@remit/ui";
+import { type AttachmentKind, sanitizeAttachmentFilename } from "@remit/ui";
 
 /**
  * The attachment-disposition body parts of a message, in the order the message
  * declares them. `pickRenderablePart` (message-body-source.ts) drops exactly
  * these parts so the renderer never treats a PDF as a body; this is the other
- * half of that split.
+ * half of that split. A forwarded message is listed whatever its disposition,
+ * and the parts inside it are its own, not the carrying message's.
  */
 export interface MessageAttachment {
 	bodyPartId: string;
+	partPath: string;
+	kind: AttachmentKind;
 	/** Display and save name — sanitized once, used for both. */
 	filename: string;
 	typeLabel: string;
@@ -17,7 +24,8 @@ export interface MessageAttachment {
 	contentUrl: string;
 }
 
-const typeLabelFor = (mediaSubtype: string): string => {
+const typeLabelFor = (kind: AttachmentKind, mediaSubtype: string): string => {
+	if (kind === "message") return "MESSAGE";
 	const label = mediaSubtype.toUpperCase();
 	return label === "OCTET-STREAM" ? "FILE" : label;
 };
@@ -35,20 +43,28 @@ const fallbackFilenameFor = (mediaSubtype: string): string => {
 export const selectMessageAttachments = (
 	parts: readonly RemitImapBodyPartResponse[],
 ): MessageAttachment[] =>
-	parts
+	ownParts(parts)
 		.filter(
 			(part) =>
 				!part.isMultipart &&
-				part.disposition === ContentDisposition.Attachment &&
+				(part.disposition === ContentDisposition.Attachment ||
+					isEncapsulatedMessage(part)) &&
 				part.contentUrl.length > 0,
 		)
-		.map((part) => ({
-			bodyPartId: part.bodyPartId,
-			filename: sanitizeAttachmentFilename(
-				part.dispositionFilename ?? "",
-				fallbackFilenameFor(part.mediaSubtype),
-			),
-			typeLabel: typeLabelFor(part.mediaSubtype),
-			sizeOctets: part.sizeOctets,
-			contentUrl: part.contentUrl,
-		}));
+		.map((part) => {
+			const kind: AttachmentKind = isEncapsulatedMessage(part)
+				? "message"
+				: "file";
+			return {
+				bodyPartId: part.bodyPartId,
+				partPath: part.partPath,
+				kind,
+				filename: sanitizeAttachmentFilename(
+					part.dispositionFilename ?? "",
+					fallbackFilenameFor(part.mediaSubtype),
+				),
+				typeLabel: typeLabelFor(kind, part.mediaSubtype),
+				sizeOctets: part.sizeOctets,
+				contentUrl: part.contentUrl,
+			};
+		});