chore: remove docs and scripts contamination

Author: Zoo (VP)

apps/vscode-e2e/src/suite/tools/terminal-profile.test.ts

@@ -66,7 +66,7 @@ suite("Terminal Profile", function () {
 			"linux",
 			{
 				...originalProfiles,
-				[PROFILE_NAME]: { path: "/bin/bash", args: ["--login"] },
+				[PROFILE_NAME]: { path: "/bin/bash", args: ["--noprofile", "--norc"] },
 			},
 			vscode.ConfigurationTarget.Global,
 		)
@@ -172,7 +172,8 @@ suite("Terminal Profile", function () {
 					options.name === "Zoo Code" &&
 					options.shellPath === "/bin/bash" &&
 					Array.isArray(options.shellArgs) &&
-					options.shellArgs.includes("--login")
+					options.shellArgs.includes("--noprofile") &&
+					options.shellArgs.includes("--norc")
 				)
 			})
 			assert.ok(profileTerminal, "Expected a Zoo Code terminal created with the configured Bash profile")

knip.json

@@ -1,16 +1,7 @@
 {
 	"$schema": "https://unpkg.com/knip@5/schema.json",
-	"ignore": [
-		"**/__tests__/**",
-		"apps/vscode-e2e/**",
-		"scripts/**",
-		"apps/cli/scripts/**",
-		"src/integrations/terminal/CommandScheduler.ts",
-		"src/integrations/terminal/CommandTrace.ts",
-		"src/integrations/terminal/shell/types.ts"
-	],
+	"ignore": ["**/__tests__/**", "apps/vscode-e2e/**", "scripts/**", "apps/cli/scripts/**"],
 	"ignoreDependencies": ["lint-staged"],
-	"ignoreBinaries": ["playwright"],
 	"ignoreExportsUsedInFile": true,
 	"playwright": false,
 	"playwright-ct": false,
@@ -22,7 +13,6 @@
 				"@roo-code/config-typescript",
 				"@types/node-cache",
 				"@types/vscode",
-				"@types/shell-quote",
 				"@vscode/ripgrep",
 				"esbuild-wasm",
 				"tree-sitter-wasms",

packages/types/src/__tests__/provider-settings.test.ts

@@ -166,3 +166,75 @@ describe("getApiProtocol", () => {
 		})
 	})
 })
+
+describe("openAiToolStrictMode", () => {
+	it("should be optional and absent by default", () => {
+		const result = providerSettingsSchemaDiscriminated.parse({
+			apiProvider: "openai",
+			openAiModelId: "test-model",
+		})
+		expect(result.apiProvider).toBe("openai")
+		if (result.apiProvider === "openai") {
+			expect(result.openAiToolStrictMode).toBeUndefined()
+		}
+	})
+
+	it("should accept true when provided", () => {
+		const result = providerSettingsSchemaDiscriminated.parse({
+			apiProvider: "openai",
+			openAiModelId: "test-model",
+			openAiToolStrictMode: true,
+		})
+		expect(result.apiProvider).toBe("openai")
+		if (result.apiProvider === "openai") {
+			expect(result.openAiToolStrictMode).toBe(true)
+		}
+	})
+
+	it("should accept false when provided", () => {
+		const result = providerSettingsSchemaDiscriminated.parse({
+			apiProvider: "openai",
+			openAiModelId: "test-model",
+			openAiToolStrictMode: false,
+		})
+		expect(result.apiProvider).toBe("openai")
+		if (result.apiProvider === "openai") {
+			expect(result.openAiToolStrictMode).toBe(false)
+		}
+	})
+
+	it("should not break existing profile deserialization when absent", () => {
+		const existingProfile = {
+			apiProvider: "openai" as const,
+			openAiBaseUrl: "https://api.example.com/v1",
+			openAiApiKey: "sk-test",
+			openAiModelId: "gpt-4",
+			openAiStreamingEnabled: true,
+		}
+		const result = providerSettingsSchemaDiscriminated.parse(existingProfile)
+		expect(result.apiProvider).toBe("openai")
+		if (result.apiProvider === "openai") {
+			expect(result.openAiModelId).toBe("gpt-4")
+			expect(result.openAiToolStrictMode).toBeUndefined()
+		}
+	})
+
+	it("should only exist on the openai (OpenAI Compatible) provider profile", () => {
+		const openAiResult = providerSettingsSchemaDiscriminated.parse({
+			apiProvider: "openai",
+			openAiToolStrictMode: true,
+		})
+		expect(openAiResult.apiProvider).toBe("openai")
+		if (openAiResult.apiProvider === "openai") {
+			expect(openAiResult.openAiToolStrictMode).toBe(true)
+		}
+
+		// Anthropic provider should not have this field
+		const anthropicResult = providerSettingsSchemaDiscriminated.parse({
+			apiProvider: "anthropic",
+			apiKey: "sk-test",
+		})
+		expect(anthropicResult.apiProvider).toBe("anthropic")
+		expect((anthropicResult as Record<string, unknown>).openAiToolStrictMode).toBeUndefined()
+	})
+})