Preserve model settings when selecting a specific OpenRouter provider (#3915)

Author: cte

.changeset/twelve-pigs-reply.md

@@ -0,0 +1,5 @@
+---
+"roo-cline": patch
+---
+
+Preserve model settings when selecting a specific OpenRouter provider

src/extension.ts

@@ -145,17 +145,19 @@ export async function activate(context: vscode.ExtensionContext) {
 	const socketPath = process.env.ROO_CODE_IPC_SOCKET_PATH
 	const enableLogging = typeof socketPath === "string"
 
-	// Watch the core files and automatically reload the extension host
-	const enableCoreAutoReload = process.env?.NODE_ENV === "development"
-	if (enableCoreAutoReload) {
-		console.log(`♻️♻️♻️ Core auto-reloading is ENABLED!`)
+	// Watch the core files and automatically reload the extension host.
+	if (process.env.NODE_ENV === "development") {
+		console.log(`♻️♻️♻️ Core auto-reloading is ENABLED! Watching for changes in ${context.extensionPath}/**/*.ts`)
+
 		const watcher = vscode.workspace.createFileSystemWatcher(
-			new vscode.RelativePattern(context.extensionPath, "src/**/*.ts"),
+			new vscode.RelativePattern(context.extensionPath, "**/*.ts"),
 		)
+
 		watcher.onDidChange((uri) => {
 			console.log(`♻️ File changed: ${uri.fsPath}. Reloading host…`)
 			vscode.commands.executeCommand("workbench.action.reloadWindow")
 		})
+
 		context.subscriptions.push(watcher)
 	}
 

webview-ui/eslint.config.mjs

@@ -5,7 +5,16 @@ export default [
 	...reactConfig,
 	{
 		rules: {
-			"@typescript-eslint/no-unused-vars": "off",
+			"@typescript-eslint/no-unused-vars": [
+				"error",
+				{
+					args: "all",
+					ignoreRestSiblings: true,
+					varsIgnorePattern: "^_",
+					argsIgnorePattern: "^_",
+					caughtErrorsIgnorePattern: "^_",
+				},
+			],
 			"@typescript-eslint/no-explicit-any": "off",
 			"react/prop-types": "off",
 			"react/display-name": "off",

webview-ui/src/components/common/__tests__/CodeBlock.test.tsx

@@ -33,7 +33,7 @@ jest.mock("lucide-react", () => {
 	return new Proxy(
 		{},
 		{
-			get: function (obj, prop) {
+			get: function (_obj, prop) {
 				// Return a component factory for any icon that's requested
 				if (prop === "__esModule") {
 					return true

webview-ui/src/components/prompts/__tests__/PromptsView.test.tsx

@@ -17,7 +17,7 @@ jest.mock("lucide-react", () => {
 	return new Proxy(
 		{},
 		{
-			get: function (obj, prop) {
+			get: function (_obj, prop) {
 				// Return a component factory for any icon that's requested
 				if (prop === "__esModule") {
 					return true

webview-ui/src/components/settings/CodeIndexSettings.tsx

@@ -17,7 +17,6 @@ import {
 } from "@/components/ui/alert-dialog"
 
 import { vscode } from "@/utils/vscode"
-import { ExtensionStateContextType } from "@/context/ExtensionStateContext"
 import { CodebaseIndexConfig, CodebaseIndexModels, ProviderSettings } from "../../../../src/schemas"
 import { EmbedderProvider } from "../../../../src/shared/embeddingModels"
 import { z } from "zod"

webview-ui/src/components/settings/__tests__/SettingsView.test.tsx

@@ -15,7 +15,7 @@ jest.mock("lucide-react", () => {
 	return new Proxy(
 		{},
 		{
-			get: function (obj, prop) {
+			get: function (_obj, prop) {
 				// Return a component factory for any icon that's requested
 				if (prop === "__esModule") {
 					return true