Fix hardcoded context menu index for File option selection (RooCodeInc#4065)

* Fix hardcoded context menu index for File option selection

The ChatTextArea component was using a hardcoded index 3 to select the "File" option by default in the context menu, but this was incorrect - the File option is actually at index 5 in the menu options array. This caused the wrong option (Git) to be selected by default when pressing Escape or when no query is provided.

Additionally, the hardcoded approach was fragile and would break if the context menu order changed in the future.

Key changes:

- Added `DEFAULT_CONTEXT_MENU_OPTIONS` array in context-mentions.ts to define the canonical menu order
- Added helper function `getDefaultContextMenuOptionIndex()` dynamically finds the correct index for any option type
- Updated ChatTextArea to use `DEFAULT_CONTEXT_MENU_OPTION` constant instead of hardcoded 3
- Updated `getContextMenuOptions()` to use the new centralized array

* changeset

Author: abeatrix

.changeset/rare-ducks-give.md

@@ -0,0 +1,5 @@
+---
+"claude-dev": minor
+---
+
+Context menu is default to File option on start up

webview-ui/src/components/chat/ChatTextArea.tsx

@@ -10,6 +10,7 @@ import { FileServiceClient, StateServiceClient } from "@/services/grpc-client"
 import {
 	ContextMenuOptionType,
 	getContextMenuOptions,
+	getContextMenuOptionIndex,
 	insertMention,
 	insertMentionDirectly,
 	removeMention,
@@ -60,6 +61,9 @@ const getImageDimensions = (dataUrl: string): Promise<{ width: number; height: n
 	})
 }
 
+// Set to "File" option by default
+const DEFAULT_CONTEXT_MENU_OPTION = getContextMenuOptionIndex(ContextMenuOptionType.File)
+
 interface ChatTextAreaProps {
 	inputValue: string
 	activeQuote: string | null
@@ -530,7 +534,7 @@ const ChatTextArea = forwardRef<HTMLTextAreaElement, ChatTextAreaProps>(
 					if (event.key === "Escape") {
 						// event.preventDefault()
 						setSelectedType(null)
-						setSelectedMenuIndex(3) // File by default
+						setSelectedMenuIndex(DEFAULT_CONTEXT_MENU_OPTION)
 						return
 					}
 
@@ -770,7 +774,7 @@ const ChatTextArea = forwardRef<HTMLTextAreaElement, ChatTextAreaProps>(
 								})
 						}, 200) // 200ms debounce
 					} else {
-						setSelectedMenuIndex(3) // Set to "File" option by default
+						setSelectedMenuIndex(DEFAULT_CONTEXT_MENU_OPTION)
 					}
 				} else {
 					setSearchQuery("")

webview-ui/src/utils/context-mentions.ts

@@ -75,6 +75,19 @@ export interface ContextMenuQueryItem {
 	description?: string
 }
 
+const DEFAULT_CONTEXT_MENU_OPTIONS = [
+	ContextMenuOptionType.URL,
+	ContextMenuOptionType.Problems,
+	ContextMenuOptionType.Terminal,
+	ContextMenuOptionType.Git,
+	ContextMenuOptionType.Folder,
+	ContextMenuOptionType.File,
+]
+
+export function getContextMenuOptionIndex(option: ContextMenuOptionType) {
+	return DEFAULT_CONTEXT_MENU_OPTIONS.findIndex((item) => item === option)
+}
+
 export function getContextMenuOptions(
 	query: string,
 	selectedType: ContextMenuOptionType | null = null,
@@ -114,14 +127,7 @@ export function getContextMenuOptions(
 			return commits.length > 0 ? [workingChanges, ...commits] : [workingChanges]
 		}
 
-		return [
-			{ type: ContextMenuOptionType.URL },
-			{ type: ContextMenuOptionType.Problems },
-			{ type: ContextMenuOptionType.Terminal },
-			{ type: ContextMenuOptionType.Git },
-			{ type: ContextMenuOptionType.Folder },
-			{ type: ContextMenuOptionType.File },
-		]
+		return DEFAULT_CONTEXT_MENU_OPTIONS.map((type) => ({ type }))
 	}
 
 	const lowerQuery = query.toLowerCase()