fix: address CodeRabbit round 4 findings

- Guard onMarkAsRead behind reply handler existence check
- Disable reply composer when session is unavailable
- Use groupId consistently in collapse filtering and T-key toggle
- Align slash-command key handler with rendered list ordering (fuzzyMatchWithScore + sort by score)

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: Felipe Gobbi

src/renderer/components/AgentInbox/FocusModeView.tsx

@@ -751,20 +751,16 @@ export default function FocusModeView({
 
 	const handleQuickReply = useCallback(() => {
 		const text = replyText.trim();
-		if (!text) return;
-		if (onQuickReply) {
-			onQuickReply(item.sessionId, item.tabId, text);
-		}
+		if (!text || !onQuickReply) return;
+		onQuickReply(item.sessionId, item.tabId, text);
 		onMarkAsRead?.(item.sessionId, item.tabId);
 		setReplyText('');
 	}, [replyText, item, onQuickReply, onMarkAsRead]);
 
 	const handleOpenAndReply = useCallback(() => {
 		const text = replyText.trim();
-		if (!text) return;
-		if (onOpenAndReply) {
-			onOpenAndReply(item.sessionId, item.tabId, text);
-		}
+		if (!text || !onOpenAndReply) return;
+		onOpenAndReply(item.sessionId, item.tabId, text);
 		onMarkAsRead?.(item.sessionId, item.tabId);
 	}, [replyText, item, onOpenAndReply, onMarkAsRead]);
 
@@ -1039,6 +1035,7 @@ export default function FocusModeView({
 						<textarea
 							ref={replyInputRef}
 							value={replyText}
+							disabled={!sessionExists}
 							onChange={(e) => setReplyText(e.target.value)}
 							onKeyDown={(e) => {
 								if (e.key === 'Enter') {
@@ -1067,7 +1064,7 @@ export default function FocusModeView({
 								// CRITICAL: Prevent focus-mode keyboard shortcuts from firing while typing
 								e.stopPropagation();
 							}}
-							placeholder="Reply to agent..."
+							placeholder={sessionExists ? 'Reply to agent...' : 'Session unavailable'}
 							rows={1}
 							aria-label="Reply to agent"
 							className="flex-1 resize-none rounded-lg px-3 py-2 text-sm outline-none"
@@ -1088,7 +1085,7 @@ export default function FocusModeView({
 						{/* Quick Reply button (primary) */}
 						<button
 							onClick={handleQuickReply}
-							disabled={!replyText.trim()}
+							disabled={!sessionExists || !replyText.trim()}
 							className="p-2 rounded-lg transition-colors flex-shrink-0"
 							style={{
 								backgroundColor: replyText.trim()
@@ -1104,7 +1101,7 @@ export default function FocusModeView({
 						{/* Open & Reply button (secondary) */}
 						<button
 							onClick={handleOpenAndReply}
-							disabled={!replyText.trim()}
+							disabled={!sessionExists || !replyText.trim()}
 							className="p-1.5 rounded-lg transition-colors flex-shrink-0 text-xs"
 							style={{
 								border: `1px solid ${theme.colors.border}`,

src/renderer/components/AgentInbox/InboxListView.tsx

@@ -560,7 +560,9 @@ export default function InboxListView({
 		return allRows.filter((row) => {
 			if (row.type === 'header') return true;
 			const collapseKey =
-				sortMode === 'byAgent' ? row.item.sessionId : (row.item.groupName ?? 'Ungrouped');
+				sortMode === 'byAgent'
+					? row.item.sessionId
+					: (row.item.groupId ?? row.item.groupName ?? 'Ungrouped');
 			return !collapsedGroups.has(collapseKey);
 		});
 	}, [allRows, collapsedGroups, sortMode]);
@@ -886,7 +888,9 @@ export default function InboxListView({
 						toggleGroup(row.groupKey);
 					} else {
 						const groupKey =
-							sortMode === 'byAgent' ? row.item.sessionId : (row.item.groupName ?? 'Ungrouped');
+							sortMode === 'byAgent'
+								? row.item.sessionId
+								: (row.item.groupId ?? row.item.groupName ?? 'Ungrouped');
 						toggleGroup(groupKey);
 					}
 				}

src/renderer/hooks/input/useInputKeyDown.ts

@@ -12,7 +12,7 @@
 import { useCallback } from 'react';
 import type { TabCompletionSuggestion, TabCompletionFilter } from '../input/useTabCompletion';
 import type { AtMentionSuggestion } from '../input/useAtMentionCompletion';
-import { fuzzyMatch } from '../../utils/search';
+import { fuzzyMatchWithScore } from '../../utils/search';
 import { useInputContext } from '../../contexts/InputContext';
 import { useSessionStore, selectActiveSession } from '../../stores/sessionStore';
 import { useUIStore } from '../../stores/uiStore';
@@ -206,11 +206,21 @@ export function useInputKeyDown(deps: InputKeyDownDeps): InputKeyDownReturn {
 			// Handle slash command autocomplete
 			if (slashCommandOpen) {
 				const isTerminalMode = activeSession?.inputMode === 'terminal';
-				const filteredCommands = allSlashCommands.filter((cmd) => {
-					if ('terminalOnly' in cmd && cmd.terminalOnly && !isTerminalMode) return false;
-					if ('aiOnly' in cmd && cmd.aiOnly && isTerminalMode) return false;
-					return fuzzyMatch(cmd.command, inputValue);
-				});
+				const inputLower = inputValue.toLowerCase();
+				const filteredCommands = allSlashCommands
+					.map((cmd) => {
+						if ('terminalOnly' in cmd && cmd.terminalOnly && !isTerminalMode) return null;
+						if ('aiOnly' in cmd && cmd.aiOnly && isTerminalMode) return null;
+						const result = fuzzyMatchWithScore(cmd.command, inputLower);
+						if (!result.matches) return null;
+						return { cmd, score: result.score };
+					})
+					.filter(
+						(item): item is { cmd: (typeof allSlashCommands)[number]; score: number } =>
+							item !== null
+					)
+					.sort((a, b) => b.score - a.score)
+					.map((item) => item.cmd);
 
 				if (e.key === 'ArrowDown') {
 					e.preventDefault();