refactor(feed): drop unused aggregate getter, dedupe sender parse

The sender-in-title rendering reads the FeedConfig DTO via the read
model, so the Feed.senderInTitle getter had no consumer — remove it.
In buildFeed, parse the from-address once and reuse it for both the
title prefix and the author; drop the dead `?? email.from` fallback
(parseFromAddress already returns the address as the name).

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
This commit is contained in:
Julien Herr
2026-05-25 15:52:51 +02:00
parent 82a4bd8341
commit 16029460bc
2 changed files with 5 additions and 10 deletions
+5 -5
View File
@@ -74,17 +74,17 @@ function buildFeed(
baseUrl,
EmailAddress.parse(email.from)?.siteBaseUrl() ?? "",
);
const sender = parseFromAddress(email.from);
const subject = htmlToText(email.subject);
const title = feedConfig.sender_in_title
? `[${parseFromAddress(email.from).name ?? email.from}] ${subject}`
: subject;
feed.addItem({
title,
title: feedConfig.sender_in_title
? `[${sender.name}] ${subject}`
: subject,
id: entryUrl,
link: entryUrl,
description: bodyContent,
content: bodyContent,
author: [parseFromAddress(email.from)],
author: [sender],
date: new Date(email.receivedAt),
enclosure: firstAttachment
? {