Flow.Launcher/Storage/QueryHistory.cs (1)

42-45: Fix the off-by-one in history trimming.

With the current > _maxHistory check the list happily grows to _maxHistory + 1 entries before trimming, so the cap is never respected exactly.

- if (LastOpenedHistoryItems.Count > _maxHistory)
+ if (LastOpenedHistoryItems.Count >= _maxHistory)
 {
 LastOpenedHistoryItems.RemoveAt(0);
 }

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

Flow.Launcher/ViewModel/MainViewModel.cs (1)

1317-1384: Consider preserving the original subtitle for Last Opened history.

For HistoryStyle.Query (lines 1324-1339), showing only the timestamp as subtitle makes sense. However, for Last Opened history (lines 1343-1382), line 1350 replaces the subtitle with just the timestamp, discarding h.SubTitle which often contains important metadata like file paths or descriptions. This breaks fuzzy search on subtitle and makes it harder to distinguish items with identical titles.

Apply this diff to preserve the subtitle while still showing the timestamp:

 var result = new Result
 {
 Title = string.IsNullOrEmpty(h.Title) ? // Old migrated history items have no title
 Localize.executeQuery(h.Query) :
 h.Title,
- SubTitle = Localize.lastExecuteTime(h.ExecutedDateTime),
+ SubTitle = string.IsNullOrEmpty(h.SubTitle) 
+ ? Localize.lastExecuteTime(h.ExecutedDateTime)
+ : $"{h.SubTitle} • {Localize.lastExecuteTime(h.ExecutedDateTime)}",
 IcoPath = Constant.HistoryIcon,

This preserves searchability while still surfacing when the item was last opened.

Note: The rest of the Last Opened implementation is solid:

• Line 1355: Correctly re-queries the plugin to get the fresh result and its action
• Lines 1358-1370: Properly handles both sync and async actions
• Lines 1373-1376: Good fallback when the result is no longer available
• Line 1347-1349: Appropriate fallback for migrated legacy items

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

Flow.Launcher/ViewModel/MainViewModel.cs (1)

1353-1353: Preserve original subtitle for searchability and context.

Setting SubTitle = Localize.lastExecuteTime(h.ExecutedDateTime) discards the stored h.SubTitle (e.g., file path, plugin info), making it impossible to search by that metadata or distinguish between identically titled entries. Consider combining both pieces of information.

This is similar to a previously flagged concern. If this design is intentional, please clarify the reasoning.

Apply this diff to preserve the original subtitle:

- SubTitle = Localize.lastExecuteTime(h.ExecutedDateTime),
+ SubTitle = string.IsNullOrEmpty(h.SubTitle) 
+ ? Localize.lastExecuteTime(h.ExecutedDateTime)
+ : $"{h.SubTitle} • {Localize.lastExecuteTime(h.ExecutedDateTime)}",

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro