diff --git a/src/core/environment/getEnvironmentDetails.ts b/src/core/environment/getEnvironmentDetails.ts index e829df0050..99b3951cd1 100644 --- a/src/core/environment/getEnvironmentDetails.ts +++ b/src/core/environment/getEnvironmentDetails.ts @@ -226,35 +226,6 @@ export async function getEnvironmentDetails(cline: Task, includeFileDetails: boo details += `${modeDetails.name}\n` details += `${modelId}\n` - // Add browser session status - Only show when active to prevent cluttering context - const isBrowserActive = cline.browserSession.isSessionActive() - - if (isBrowserActive) { - // Build viewport info for status (prefer actual viewport if available, else fallback to configured setting) - const configuredViewport = (state?.browserViewportSize as string | undefined) ?? "900x600" - let configuredWidth: number | undefined - let configuredHeight: number | undefined - if (configuredViewport.includes("x")) { - const parts = configuredViewport.split("x").map((v) => Number(v)) - configuredWidth = parts[0] - configuredHeight = parts[1] - } - - let actualWidth: number | undefined - let actualHeight: number | undefined - const vp = cline.browserSession.getViewportSize?.() - if (vp) { - actualWidth = vp.width - actualHeight = vp.height - } - - const width = actualWidth ?? configuredWidth - const height = actualHeight ?? configuredHeight - const viewportInfo = width && height ? `\nCurrent viewport size: ${width}x${height} pixels.` : "" - - details += `\n# Browser Session Status\nActive - A browser session is currently open and ready for browser_action commands${viewportInfo}\n` - } - if (includeFileDetails) { details += `\n\n# Current Workspace Directory (${cline.cwd.toPosix()}) Files\n` const isDesktop = arePathsEqual(cline.cwd, path.join(os.homedir(), "Desktop")) @@ -270,8 +241,7 @@ export async function getEnvironmentDetails(cline: Task, includeFileDetails: boo if (maxFiles === 0) { details += "(Workspace files context disabled. Use list_files to explore if needed.)" } else { - const respectGitIgnore = state?.codebaseIndexConfig?.codebaseIndexRespectGitIgnore ?? true - const [files, didHitLimit] = await listFiles(cline.cwd, true, maxFiles, respectGitIgnore) + const [files, didHitLimit] = await listFiles(cline.cwd, true, maxFiles) const { showRooIgnoredFiles = false } = state ?? {} const result = formatResponse.formatFilesList( diff --git a/src/core/tools/ListFilesTool.ts b/src/core/tools/ListFilesTool.ts index 7d92734ba4..716d7ed784 100644 --- a/src/core/tools/ListFilesTool.ts +++ b/src/core/tools/ListFilesTool.ts @@ -37,10 +37,8 @@ export class ListFilesTool extends BaseTool<"list_files"> { const absolutePath = path.resolve(task.cwd, relDirPath) const isOutsideWorkspace = isPathOutsideWorkspace(absolutePath) - const state = await task.providerRef.deref()?.getState() - const respectGitIgnore = state?.codebaseIndexConfig?.codebaseIndexRespectGitIgnore ?? true - const [files, didHitLimit] = await listFiles(absolutePath, recursive || false, 200, respectGitIgnore) - const { showRooIgnoredFiles = false } = state ?? {} + const [files, didHitLimit] = await listFiles(absolutePath, recursive || false, 200) + const { showRooIgnoredFiles = false } = (await task.providerRef.deref()?.getState()) ?? {} const result = formatResponse.formatFilesList( absolutePath, diff --git a/src/integrations/workspace/WorkspaceTracker.ts b/src/integrations/workspace/WorkspaceTracker.ts index 46475efd24..546cd97cd1 100644 --- a/src/integrations/workspace/WorkspaceTracker.ts +++ b/src/integrations/workspace/WorkspaceTracker.ts @@ -30,14 +30,7 @@ class WorkspaceTracker { return } const tempCwd = this.cwd - let respectGitIgnore = true - try { - const state = await this.providerRef.deref()?.getState() - respectGitIgnore = state?.codebaseIndexConfig?.codebaseIndexRespectGitIgnore ?? true - } catch { - // Fall back to default (respect .gitignore) if state is not available - } - const [files, _] = await listFiles(tempCwd, true, MAX_INITIAL_FILES, respectGitIgnore) + const [files, _] = await listFiles(tempCwd, true, MAX_INITIAL_FILES) if (this.prevWorkSpacePath !== tempCwd) { return } diff --git a/src/integrations/workspace/__tests__/WorkspaceTracker.spec.ts b/src/integrations/workspace/__tests__/WorkspaceTracker.spec.ts index a329ef1ab1..b0a617d970 100644 --- a/src/integrations/workspace/__tests__/WorkspaceTracker.spec.ts +++ b/src/integrations/workspace/__tests__/WorkspaceTracker.spec.ts @@ -240,7 +240,7 @@ describe("WorkspaceTracker", () => { vitest.runAllTimers() // Should initialize file paths for new workspace - expect(listFiles).toHaveBeenCalledWith("/test/new-workspace", true, 1000, true) + expect(listFiles).toHaveBeenCalledWith("/test/new-workspace", true, 1000) vitest.runAllTimers() })