Fall back to execa if VSCode terminal integration fails (RooCodeInc#3049)

Author: cte

.changeset/many-boats-hunt.md

@@ -0,0 +1,5 @@
+---
+"roo-cline": patch
+---
+
+Use a fallback terminal if VSCode shell integration fails

.roomodes

@@ -2,7 +2,7 @@
   "customModes": [
     {
       "slug": "test",
-      "name": "Test",
+      "name": "🧪 Test",
       "roleDefinition": "You are Roo, a Jest testing specialist with deep expertise in:\n- Writing and maintaining Jest test suites\n- Test-driven development (TDD) practices\n- Mocking and stubbing with Jest\n- Integration testing strategies\n- TypeScript testing patterns\n- Code coverage analysis\n- Test performance optimization\n\nYour focus is on maintaining high test quality and coverage across the codebase, working primarily with:\n- Test files in __tests__ directories\n- Mock implementations in __mocks__\n- Test utilities and helpers\n- Jest configuration and setup\n\nYou ensure tests are:\n- Well-structured and maintainable\n- Following Jest best practices\n- Properly typed with TypeScript\n- Providing meaningful coverage\n- Using appropriate mocking strategies",
       "groups": [
         "read",
@@ -20,7 +20,7 @@
     },
     {
       "slug": "translate",
-      "name": "Translate",
+      "name": "🌐 Translate",
       "roleDefinition": "You are Roo, a linguistic specialist focused on translating and managing localization files. Your responsibility is to help maintain and update translation files for the application, ensuring consistency and accuracy across all language resources.",
       "groups": [
         "read",

scripts/run-tests.js

@@ -2,7 +2,7 @@
 const { execSync } = require("child_process")
 
 if (process.platform === "win32") {
-	execSync("npm-run-all test:* lint:*", { stdio: "inherit" })
+	execSync("npm-run-all test:*", { stdio: "inherit" })
 } else {
-	execSync("npm-run-all -p test:* lint:*", { stdio: "inherit" })
+	execSync("npm-run-all -p test:*", { stdio: "inherit" })
 }

src/__mocks__/jest.setup.ts

@@ -1,3 +1,15 @@
+import nock from "nock"
+
+nock.disableNetConnect()
+
+export function allowNetConnect(host?: string | RegExp) {
+	if (host) {
+		nock.enableNetConnect(host)
+	} else {
+		nock.enableNetConnect()
+	}
+}
+
 // Mock the logger globally for all tests
 jest.mock("../utils/logging", () => ({
 	logger: {

src/api/providers/__tests__/glama.test.ts

@@ -5,6 +5,36 @@ import { Anthropic } from "@anthropic-ai/sdk"
 import { GlamaHandler } from "../glama"
 import { ApiHandlerOptions } from "../../../shared/api"
 
+// Mock dependencies
+jest.mock("../fetchers/cache", () => ({
+	getModels: jest.fn().mockImplementation(() => {
+		return Promise.resolve({
+			"anthropic/claude-3-7-sonnet": {
+				maxTokens: 8192,
+				contextWindow: 200000,
+				supportsImages: true,
+				supportsPromptCache: true,
+				inputPrice: 3,
+				outputPrice: 15,
+				cacheWritesPrice: 3.75,
+				cacheReadsPrice: 0.3,
+				description: "Claude 3.7 Sonnet",
+				thinking: false,
+				supportsComputerUse: true,
+			},
+			"openai/gpt-4o": {
+				maxTokens: 4096,
+				contextWindow: 128000,
+				supportsImages: true,
+				supportsPromptCache: false,
+				inputPrice: 5,
+				outputPrice: 15,
+				description: "GPT-4o",
+			},
+		})
+	}),
+}))
+
 // Mock OpenAI client
 const mockCreate = jest.fn()
 const mockWithResponse = jest.fn()

src/api/providers/__tests__/openrouter.test.ts

@@ -9,6 +9,38 @@ import { ApiHandlerOptions } from "../../../shared/api"
 // Mock dependencies
 jest.mock("openai")
 jest.mock("delay", () => jest.fn(() => Promise.resolve()))
+jest.mock("../fetchers/cache", () => ({
+	getModels: jest.fn().mockImplementation(() => {
+		return Promise.resolve({
+			"anthropic/claude-3.7-sonnet": {
+				maxTokens: 8192,
+				contextWindow: 200000,
+				supportsImages: true,
+				supportsPromptCache: true,
+				inputPrice: 3,
+				outputPrice: 15,
+				cacheWritesPrice: 3.75,
+				cacheReadsPrice: 0.3,
+				description: "Claude 3.7 Sonnet",
+				thinking: false,
+				supportsComputerUse: true,
+			},
+			"anthropic/claude-3.7-sonnet:thinking": {
+				maxTokens: 128000,
+				contextWindow: 200000,
+				supportsImages: true,
+				supportsPromptCache: true,
+				inputPrice: 3,
+				outputPrice: 15,
+				cacheWritesPrice: 3.75,
+				cacheReadsPrice: 0.3,
+				description: "Claude 3.7 Sonnet with thinking",
+				thinking: true,
+				supportsComputerUse: true,
+			},
+		})
+	}),
+}))
 
 describe("OpenRouterHandler", () => {
 	const mockOptions: ApiHandlerOptions = {

src/api/providers/__tests__/requesty.test.ts

@@ -1,3 +1,5 @@
+// npx jest src/api/providers/__tests__/requesty.test.ts
+
 import { Anthropic } from "@anthropic-ai/sdk"
 import OpenAI from "openai"
 import { ApiHandlerOptions, ModelInfo } from "../../../shared/api"
@@ -9,6 +11,22 @@ import { convertToR1Format } from "../../transform/r1-format"
 jest.mock("openai")
 jest.mock("../../transform/openai-format")
 jest.mock("../../transform/r1-format")
+jest.mock("../fetchers/cache", () => ({
+	getModels: jest.fn().mockResolvedValue({
+		"test-model": {
+			maxTokens: 8192,
+			contextWindow: 200_000,
+			supportsImages: true,
+			supportsComputerUse: true,
+			supportsPromptCache: true,
+			inputPrice: 3.0,
+			outputPrice: 15.0,
+			cacheWritesPrice: 3.75,
+			cacheReadsPrice: 0.3,
+			description: "Test model description",
+		},
+	}),
+}))
 
 describe("RequestyHandler", () => {
 	let handler: RequestyHandler

src/api/providers/__tests__/unbound.test.ts

@@ -6,6 +6,58 @@ import { ApiHandlerOptions } from "../../../shared/api"
 
 import { UnboundHandler } from "../unbound"
 
+// Mock dependencies
+jest.mock("../fetchers/cache", () => ({
+	getModels: jest.fn().mockImplementation(() => {
+		return Promise.resolve({
+			"anthropic/claude-3-5-sonnet-20241022": {
+				maxTokens: 8192,
+				contextWindow: 200000,
+				supportsImages: true,
+				supportsPromptCache: true,
+				inputPrice: 3,
+				outputPrice: 15,
+				cacheWritesPrice: 3.75,
+				cacheReadsPrice: 0.3,
+				description: "Claude 3.5 Sonnet",
+				thinking: false,
+				supportsComputerUse: true,
+			},
+			"anthropic/claude-3-7-sonnet-20250219": {
+				maxTokens: 8192,
+				contextWindow: 200000,
+				supportsImages: true,
+				supportsPromptCache: true,
+				inputPrice: 3,
+				outputPrice: 15,
+				cacheWritesPrice: 3.75,
+				cacheReadsPrice: 0.3,
+				description: "Claude 3.7 Sonnet",
+				thinking: false,
+				supportsComputerUse: true,
+			},
+			"openai/gpt-4o": {
+				maxTokens: 4096,
+				contextWindow: 128000,
+				supportsImages: true,
+				supportsPromptCache: false,
+				inputPrice: 5,
+				outputPrice: 15,
+				description: "GPT-4o",
+			},
+			"openai/o3-mini": {
+				maxTokens: 4096,
+				contextWindow: 128000,
+				supportsImages: true,
+				supportsPromptCache: false,
+				inputPrice: 1,
+				outputPrice: 3,
+				description: "O3 Mini",
+			},
+		})
+	}),
+}))
+
 // Mock OpenAI client
 const mockCreate = jest.fn()
 const mockWithResponse = jest.fn()

src/api/providers/fetchers/__tests__/openrouter.test.ts

@@ -13,7 +13,7 @@ nockBack.setMode("lockdown")
 
 describe("OpenRouter API", () => {
 	describe("getOpenRouterModels", () => {
-		it("fetches models and validates schema", async () => {
+		it.skip("fetches models and validates schema", async () => {
 			const { nockDone } = await nockBack("openrouter-models.json")
 
 			const models = await getOpenRouterModels()

src/api/providers/glama.ts

@@ -19,7 +19,7 @@ export class GlamaHandler extends RouterProvider implements SingleCompletionHand
 	constructor(options: ApiHandlerOptions) {
 		super({
 			options,
-			name: "unbound",
+			name: "glama",
 			baseURL: "https://glama.ai/api/gateway/openai/v1",
 			apiKey: options.glamaApiKey,
 			modelId: options.glamaModelId,