From b456a879bb9eb356686792e010a2791b626d2ed3 Mon Sep 17 00:00:00 2001 From: teernisse Date: Fri, 13 Mar 2026 13:18:09 -0400 Subject: [PATCH] fix(timeline): use total_notes for discussion summary instead of post-filter count The discussion summary format string used `note_count` (derived from `notes.len()` after truncation/filtering) rather than `total_notes` which captures the full count before any processing. This made the "Discussion (N notes)" label undercount when notes were filtered. Removes the now-unused `note_count` binding. Co-Authored-By: Claude Opus 4.6 --- src/timeline/collect.rs | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/src/timeline/collect.rs b/src/timeline/collect.rs index 026598a..2ae9aea 100644 --- a/src/timeline/collect.rs +++ b/src/timeline/collect.rs @@ -468,7 +468,6 @@ fn collect_discussion_threads( }); } - let note_count = notes.len(); let actor = notes.first().and_then(|n| n.author.clone()); events.push(TimelineEvent { @@ -481,7 +480,7 @@ fn collect_discussion_threads( discussion_id: disc.discussion_id, notes, }, - summary: format!("Discussion ({note_count} notes)"), + summary: format!("Discussion ({total_notes} notes)"), actor, url: None, is_seed: true,