feat: expose a row's sender to the delegate

4 files changed, 34 insertions, 0 deletions

diff --git a/src/notmuchworker.cpp b/src/notmuchworker.cpp
index 43f4758..28834df 100644
--- a/src/notmuchworker.cpp
+++ b/src/notmuchworker.cpp
@@ -222,6 +222,7 @@ void walkReplies(notmuch_messages_t *messages, int depth,
             QString::fromUtf8(notmuch_message_get_filename(message));
         node.from =
             QString::fromUtf8(notmuch_message_get_header(message, "from"));
+        node.senderAddress = senderAddressOf(message);
         node.subject =
             QString::fromUtf8(notmuch_message_get_header(message, "subject"));
         node.date =
diff --git a/src/threadlistmodel.cpp b/src/threadlistmodel.cpp
index fcd8e4f..a6fada3 100644
--- a/src/threadlistmodel.cpp
+++ b/src/threadlistmodel.cpp
@@ -383,6 +383,13 @@ QVariant ThreadListModel::data(const QModelIndex &index, int role) const
         }
         case AccountLabelRole:
             return QString();
+        case SenderAddressRole:
+            // The REPLY's own sender, exactly as SendersRole serves the node's
+            // `from` and not the thread's summary. The bare address is what
+            // the avatar hashes and the business-senders list matches.
+            return node.senderAddress;
+        case SenderNameRole:
+            return node.from;
         case Qt::DisplayRole:
         case SubjectRole:
             return node.subject;
@@ -678,6 +685,16 @@ QVariant ThreadListModel::data(const QModelIndex &index, int role) const
         if (!thread.recipients.isEmpty())
             return thread.recipients;
         return thread.authors;
+    case SenderAddressRole:
+        // The bare address of the message this card stands for, carried from
+        // the query. This is what the avatar's hash and the business-senders
+        // lookup consume; the display name below is what the card draws.
+        return thread.firstMessageSender;
+    case SenderNameRole:
+        // The same string the card's first line shows: recipients in a flat
+        // view, where `authors` is the user on every row and says nothing.
+        return !thread.recipients.isEmpty() ? thread.recipients
+                                            : thread.authors;
     case DateRole:
         // The QDateTime itself. Formatting belongs to the delegate now: the
         // card decides how much of a date it has room for, and a pre-formatted
diff --git a/src/threadlistmodel.h b/src/threadlistmodel.h
index 2e56328..4af09d8 100644
--- a/src/threadlistmodel.h
+++ b/src/threadlistmodel.h
@@ -52,6 +52,15 @@ public:
         /// Fill colour for that chip.
         AccountColourRole,
 
+        /// The bare address of the message this row stands for, for the
+        /// avatar's hash and for the business-senders lookup. Empty when the
+        /// query did not resolve one, which the delegate handles by falling
+        /// back to the account.
+        SenderAddressRole,
+        /// The display name to take initials from. `authors` for a thread row,
+        /// `recipients` in a flat view, matching what the card already shows.
+        SenderNameRole,
+
         /// Every tag on the thread, for the strip under the message pane.
         TagsRole,
 
diff --git a/src/types.h b/src/types.h
index 47ddb78..a1689ab 100644
--- a/src/types.h
+++ b/src/types.h
@@ -166,6 +166,13 @@ struct MessageNode
     QString messageId;
     QString threadId;  ///< The thread this message belongs to.
     QString from;
+
+    /// The BARE address of the message's From, with any display name
+    /// discarded, like `ThreadSummary::firstMessageSender` is for a thread
+    /// row. `from` above is the raw header and usually carries a display
+    /// name, which is what the avatar's hash and the business-senders
+    /// lookup cannot take initials or match against.
+    QString senderAddress;
     QString subject;
     QDateTime date;
     QStringList tags;