Skip to content

Commit 4e135c2

Browse files
committed
fix: update default settings for inline terminal and codebase indexing
- Change terminalShellIntegrationDisabled default from false to true (so "Use Inline Terminal (recommended)" is checked by default) - Change codebaseIndexEnabled default from true to false (so "Enable Codebase Indexing" is unchecked by default) - Update related tests to reflect new defaults
1 parent a8a4451 commit 4e135c2

File tree

3 files changed

+10
-10
lines changed

3 files changed

+10
-10
lines changed

src/core/webview/ClineProvider.ts

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -2023,7 +2023,7 @@ export class ClineProvider
20232023
terminalOutputLineLimit: terminalOutputLineLimit ?? 500,
20242024
terminalOutputCharacterLimit: terminalOutputCharacterLimit ?? DEFAULT_TERMINAL_OUTPUT_CHARACTER_LIMIT,
20252025
terminalShellIntegrationTimeout: terminalShellIntegrationTimeout ?? Terminal.defaultShellIntegrationTimeout,
2026-
terminalShellIntegrationDisabled: terminalShellIntegrationDisabled ?? false,
2026+
terminalShellIntegrationDisabled: terminalShellIntegrationDisabled ?? true,
20272027
terminalCommandDelay: terminalCommandDelay ?? 0,
20282028
terminalPowershellCounter: terminalPowershellCounter ?? false,
20292029
terminalZshClearEolMark: terminalZshClearEolMark ?? true,
@@ -2075,7 +2075,7 @@ export class ClineProvider
20752075
customCondensingPrompt,
20762076
codebaseIndexModels: codebaseIndexModels ?? EMBEDDING_MODEL_PROFILES,
20772077
codebaseIndexConfig: {
2078-
codebaseIndexEnabled: codebaseIndexConfig?.codebaseIndexEnabled ?? true,
2078+
codebaseIndexEnabled: codebaseIndexConfig?.codebaseIndexEnabled ?? false,
20792079
codebaseIndexQdrantUrl: codebaseIndexConfig?.codebaseIndexQdrantUrl ?? "http://localhost:6333",
20802080
codebaseIndexEmbedderProvider: codebaseIndexConfig?.codebaseIndexEmbedderProvider ?? "openai",
20812081
codebaseIndexEmbedderBaseUrl: codebaseIndexConfig?.codebaseIndexEmbedderBaseUrl ?? "",
@@ -2255,7 +2255,7 @@ export class ClineProvider
22552255
stateValues.terminalOutputCharacterLimit ?? DEFAULT_TERMINAL_OUTPUT_CHARACTER_LIMIT,
22562256
terminalShellIntegrationTimeout:
22572257
stateValues.terminalShellIntegrationTimeout ?? Terminal.defaultShellIntegrationTimeout,
2258-
terminalShellIntegrationDisabled: stateValues.terminalShellIntegrationDisabled ?? false,
2258+
terminalShellIntegrationDisabled: stateValues.terminalShellIntegrationDisabled ?? true,
22592259
terminalCommandDelay: stateValues.terminalCommandDelay ?? 0,
22602260
terminalPowershellCounter: stateValues.terminalPowershellCounter ?? false,
22612261
terminalZshClearEolMark: stateValues.terminalZshClearEolMark ?? true,
@@ -2301,7 +2301,7 @@ export class ClineProvider
23012301
customCondensingPrompt: stateValues.customCondensingPrompt,
23022302
codebaseIndexModels: stateValues.codebaseIndexModels ?? EMBEDDING_MODEL_PROFILES,
23032303
codebaseIndexConfig: {
2304-
codebaseIndexEnabled: stateValues.codebaseIndexConfig?.codebaseIndexEnabled ?? true,
2304+
codebaseIndexEnabled: stateValues.codebaseIndexConfig?.codebaseIndexEnabled ?? false,
23052305
codebaseIndexQdrantUrl:
23062306
stateValues.codebaseIndexConfig?.codebaseIndexQdrantUrl ?? "http://localhost:6333",
23072307
codebaseIndexEmbedderProvider:

src/services/code-index/__tests__/config-manager.spec.ts

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,7 @@ describe("CodeIndexConfigManager", () => {
5353
describe("constructor", () => {
5454
it("should initialize with ContextProxy", () => {
5555
expect(configManager).toBeDefined()
56-
expect(configManager.isFeatureEnabled).toBe(true)
56+
expect(configManager.isFeatureEnabled).toBe(false)
5757
expect(configManager.currentEmbedderProvider).toBe("openai")
5858
})
5959
})
@@ -81,13 +81,13 @@ describe("CodeIndexConfigManager", () => {
8181
expect(configManager.isFeatureEnabled).toBe(true)
8282
})
8383

84-
it("should default to true when codebaseIndexEnabled is not set", async () => {
84+
it("should default to false when codebaseIndexEnabled is not set", async () => {
8585
mockContextProxy.getGlobalState.mockReturnValue({})
8686
mockContextProxy.getSecret.mockReturnValue(undefined)
8787

8888
// Re-create instance to load the configuration
8989
configManager = new CodeIndexConfigManager(mockContextProxy)
90-
expect(configManager.isFeatureEnabled).toBe(true)
90+
expect(configManager.isFeatureEnabled).toBe(false)
9191
})
9292
})
9393

src/services/code-index/config-manager.ts

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@ import { getDefaultModelId, getModelDimension, getModelScoreThreshold } from "..
1010
* Handles loading, validating, and providing access to configuration values.
1111
*/
1212
export class CodeIndexConfigManager {
13-
private codebaseIndexEnabled: boolean = true
13+
private codebaseIndexEnabled: boolean = false
1414
private embedderProvider: EmbedderProvider = "openai"
1515
private modelId?: string
1616
private modelDimension?: number
@@ -46,7 +46,7 @@ export class CodeIndexConfigManager {
4646
private _loadAndSetConfiguration(): void {
4747
// Load configuration from storage
4848
const codebaseIndexConfig = this.contextProxy?.getGlobalState("codebaseIndexConfig") ?? {
49-
codebaseIndexEnabled: true,
49+
codebaseIndexEnabled: false,
5050
codebaseIndexQdrantUrl: "http://localhost:6333",
5151
codebaseIndexEmbedderProvider: "openai",
5252
codebaseIndexEmbedderBaseUrl: "",
@@ -80,7 +80,7 @@ export class CodeIndexConfigManager {
8080
const openRouterApiKey = this.contextProxy?.getSecret("codebaseIndexOpenRouterApiKey") ?? ""
8181

8282
// Update instance variables with configuration
83-
this.codebaseIndexEnabled = codebaseIndexEnabled ?? true
83+
this.codebaseIndexEnabled = codebaseIndexEnabled ?? false
8484
this.qdrantUrl = codebaseIndexQdrantUrl
8585
this.qdrantApiKey = qdrantApiKey ?? ""
8686
this.searchMinScore = codebaseIndexSearchMinScore

0 commit comments

Comments
 (0)