fix: rename unused onValueChange parameters in test mocks

Author: Sannidhya

webview-ui/src/components/settings/__tests__/SettingsView.change-detection.spec.tsx

@@ -61,7 +61,7 @@ vi.mock("@src/components/ui", () => ({
 	DialogDescription: ({ children }: any) => <div data-testid="dialog-description">{children}</div>,
 	DialogFooter: ({ children }: any) => <div data-testid="dialog-footer">{children}</div>,
 	// Add Select components (used by CreateSkillDialog)
-	Select: ({ children, value, onValueChange }: any) => (
+	Select: ({ children, value, onValueChange: _onValueChange }: any) => (
 		<div data-testid="select" data-value={value}>
 			{children}
 		</div>

webview-ui/src/components/settings/__tests__/SettingsView.unsaved-changes.spec.tsx

@@ -62,7 +62,7 @@ vi.mock("@src/components/ui", () => ({
 	DialogDescription: ({ children }: any) => <div data-testid="dialog-description">{children}</div>,
 	DialogFooter: ({ children }: any) => <div data-testid="dialog-footer">{children}</div>,
 	// Add Select components (used by CreateSkillDialog)
-	Select: ({ children, value, onValueChange }: any) => (
+	Select: ({ children, value, onValueChange: _onValueChange }: any) => (
 		<div data-testid="select" data-value={value}>
 			{children}
 		</div>