release/v5.0.2

CHANGELOG.md

@@ -1,6 +1,15 @@
 # Changelog
 
-## [v5.0.1] - 2026-01-05
+## [v5.0.2] - 2026-01-09
+
+### Changed
+
+- Minor improvements to file read tool
+- Settings UI Cleanup
+
+---
+
+## [v5.0.1] - 2026-01-09
 
 ### Changed
 

apps/storybook/.storybook/theme-variables.css

@@ -70,7 +70,7 @@ classes at build time based on the @theme */
 --card-foreground: var(--vscode-editor-foreground);
 --popover: var(--vscode-menu-background, var(--vscode-editor-background));
 --popover-foreground: var(--vscode-menu-foreground, var(--vscode-editor-foreground));
---primary: var(--vscode-button-background);
+--primary: var(--color-matterai-green-dark);
 --primary-foreground: var(--vscode-button-foreground);
 --secondary: var(--vscode-button-secondaryBackground);
 --secondary-foreground: var(--vscode-button-secondaryForeground);

jetbrains/plugin/src/main/resources/themes/theme-variables.css

@@ -70,7 +70,7 @@ classes at build time based on the @theme */
 --card-foreground: var(--vscode-editor-foreground);
 --popover: var(--vscode-menu-background, var(--vscode-editor-background));
 --popover-foreground: var(--vscode-menu-foreground, var(--vscode-editor-foreground));
---primary: var(--vscode-button-background);
+--primary: var(--color-matterai-green-dark);
 --primary-foreground: var(--vscode-button-foreground);
 --secondary: var(--vscode-button-secondaryBackground);
 --secondary-foreground: var(--vscode-button-secondaryForeground);

src/core/assistant-message/presentAssistantMessage.ts

@@ -283,15 +283,15 @@ export async function presentAssistantMessage(cline: Task) {
 				break
 			}
 
-			if (cline.didAlreadyUseTool) {
-				// Ignore any content after a tool has already been used.
-				pushToolResult_withToolUseId_kilocode({
-					type: "text",
-					text: `Tool [${block.name}] was not executed because a tool has already been used in this message. Only one tool may be used per message. You must assess the first tool's result before proceeding to use the next tool.`,
-				})
-
-				break
-			}
+			// if (cline.didAlreadyUseTool) {
+			// 	// Ignore any content after a tool has already been used.
+			// 	pushToolResult_withToolUseId_kilocode({
+			// 		type: "text",
+			// 		text: `Tool [${block.name}] was not executed because a tool has already been used in this message. Only one tool may be used per message. You must assess the first tool's result before proceeding to use the next tool.`,
+			// 	})
+
+			// 	break
+			// }
 
 			const pushToolResult = (content: ToolResponse) => {
 				// kilocode_change start

src/core/prompts/tools/native-tools/read_file.ts

@@ -5,7 +5,7 @@ export const read_file_multi = {
 	function: {
 		name: "read_file",
 		description:
-			"Read one or more files and return their contents with line numbers. Use offset and limit to read specific portions of files efficiently. By default reads from the beginning with a reasonable limit.",
+			"Read one or more files and return their contents with line numbers. Use offset and limit to read specific portions of files efficiently. Default and maximum limit is 1000 lines to prevent context overflow.",
 		strict: true,
 		parameters: {
 			type: "object",
@@ -28,7 +28,7 @@ export const read_file_multi = {
 							limit: {
 								type: ["number", "null"],
 								description:
-									"Maximum number of lines to read from offset. If not specified, reads the complete file from offset. Use smaller values for targeted reads.",
+									"Maximum number of lines to read from offset. Default and maximum limit is 1000 lines. Use smaller values for targeted reads.",
 							},
 						},
 						required: ["file_path"],
@@ -48,7 +48,7 @@ export const read_file_single = {
 	function: {
 		name: "read_file",
 		description:
-			"Read a file and return its contents with line numbers. Use offset and limit to read specific portions efficiently.",
+			"Read a file and return its contents with line numbers. Use offset and limit to read specific portions efficiently. Default and maximum limit is 1000 lines to prevent context overflow.",
 		strict: true,
 		parameters: {
 			type: "object",
@@ -64,7 +64,7 @@ export const read_file_single = {
 				limit: {
 					type: ["number", "null"],
 					description:
-						"Maximum number of lines to read from offset. If not specified, reads the complete file from offset.",
+						"Maximum number of lines to read from offset. Default and maximum limit is 1000 lines.",
 				},
 			},
 			required: ["file_path"],

src/core/tools/kilocode.ts

@@ -56,18 +56,28 @@ type FileEntry = {
 }
 
 export function parseNativeFiles(
-	nativeFiles: { file_path?: string; path?: string; offset?: number; limit?: number; line_ranges?: string[] }[],
+	nativeFiles: {
+		file_path?: string
+		path?: string
+		offset?: number | string
+		limit?: number | string
+		line_ranges?: string[]
+	}[],
 ) {
 	const fileEntries = new Array<FileEntry>()
 	for (const file of nativeFiles) {
 		// Support both file_path (new) and path (legacy)
 		const filePath = file.file_path || file.path
 		if (!filePath) continue
 
+		// Parse offset and limit as integers - LLM may send them as strings
+		const parsedOffset = file.offset !== undefined ? parseInt(String(file.offset), 10) : undefined
+		const parsedLimit = file.limit !== undefined ? parseInt(String(file.limit), 10) : undefined
+
 		const fileEntry: FileEntry = {
 			path: filePath,
-			offset: file.offset ?? 1,
-			limit: file.limit, // undefined means read complete file
+			offset: !isNaN(parsedOffset as number) ? parsedOffset : 1,
+			limit: !isNaN(parsedLimit as number) ? parsedLimit : undefined, // undefined means read complete file
 		}
 
 		// Legacy support: convert line_ranges to offset+limit if provided

src/core/tools/readFileTool.ts

@@ -24,6 +24,9 @@ import {
 	ImageMemoryTracker,
 } from "./helpers/imageHelpers"
 
+// Maximum number of lines to read when limit is not specified - prevents context window overflow
+const MAX_READ_FILE_LINES = 1000
+
 export function getReadFileToolDescription(blockName: string, blockParams: any): string {
 	// Handle both single file_path and multiple files via args
 	// kilocode_change start
@@ -95,8 +98,11 @@ export async function readFileTool(
 	const newFilePath: string | undefined = (block.params as any).file_path // New: support file_path directly
 	const legacyStartLineStr: string | undefined = block.params.start_line
 	const legacyEndLineStr: string | undefined = block.params.end_line
-	const offsetParam: number | undefined = (block.params as any).offset
-	const limitParam: number | undefined = (block.params as any).limit
+	// Parse offset and limit as integers - LLM may send them as strings causing string concatenation bugs
+	const rawOffset = (block.params as any).offset
+	const rawLimit = (block.params as any).limit
+	const offsetParam: number | undefined = rawOffset !== undefined ? parseInt(String(rawOffset), 10) : undefined
+	const limitParam: number | undefined = rawLimit !== undefined ? parseInt(String(rawLimit), 10) : undefined
 
 	const nativeFiles: any[] | undefined = (block.params as any).files // kilocode_change: Native JSON format from OpenAI-style tool calls
 
@@ -159,10 +165,14 @@ export async function readFileTool(
 				const filePath = file.file_path || file.path
 				if (!filePath) continue // Skip if no path in a file entry
 
+				// Parse offset and limit as integers - XML parsing may produce strings
+				const parsedOffset = file.offset !== undefined ? parseInt(String(file.offset), 10) : undefined
+				const parsedLimit = file.limit !== undefined ? parseInt(String(file.limit), 10) : undefined
+
 				const fileEntry: FileEntry = {
 					path: filePath,
-					offset: file.offset ?? 1,
-					limit: file.limit, // undefined means read complete file
+					offset: !isNaN(parsedOffset as number) ? parsedOffset : 1,
+					limit: !isNaN(parsedLimit as number) ? parsedLimit : undefined, // undefined means read complete file
 				}
 
 				// Legacy support: convert line_range to offset+limit
@@ -350,10 +360,11 @@ export async function readFileTool(
 		const imageMemoryTracker = new ImageMemoryTracker()
 		const state = await cline.providerRef.deref()?.getState()
 		const {
-			maxReadFileLine = -1,
 			maxImageFileSize = DEFAULT_MAX_IMAGE_FILE_SIZE_MB,
 			maxTotalImageSize = DEFAULT_MAX_TOTAL_IMAGE_SIZE_MB,
 		} = state ?? {}
+		// Always use MAX_READ_FILE_LINES - setting will be removed later
+		const maxReadFileLine = MAX_READ_FILE_LINES
 
 		// Then process only approved files
 		for (const fileResult of fileResults) {
@@ -444,33 +455,37 @@ export async function readFileTool(
 				// Handle offset/limit reads (if limit is specified)
 				if (fileResult.limit !== undefined) {
 					const startLine = fileResult.offset ?? 1
-					const endLine = startLine + fileResult.limit - 1
+					// Cap limit to MAX_READ_FILE_LINES to prevent context window overflow
+					const effectiveLimit = Math.min(fileResult.limit, MAX_READ_FILE_LINES)
+					const endLine = startLine + effectiveLimit - 1
 					const content = addLineNumbers(await readLines(fullPath, endLine - 1, startLine - 1), startLine)
+					let xmlContent = content
+					if (fileResult.limit > MAX_READ_FILE_LINES) {
+						xmlContent = `[showing ${effectiveLimit} lines, capped from requested ${fileResult.limit} lines to prevent context overflow]\n${content}`
+					}
 					updateFileResult(relPath, {
-						xmlContent: content,
+						xmlContent,
 					})
 					continue
 				}
 
-				// Handle definitions-only mode
-				if (maxReadFileLine === 0) {
-					try {
-						const defResult = await parseSourceCodeDefinitionsForFile(fullPath, cline.rooIgnoreController)
-						if (defResult) {
-							// kilocode_change: Return raw definitions without path header
-							updateFileResult(relPath, {
-								xmlContent: `[definitions only, ${totalLines} total lines]\n${defResult}`,
-							})
-						}
-					} catch (error) {
-						if (error instanceof Error && error.message.startsWith("Unsupported language:")) {
-							console.warn(`[read_file] Warning: ${error.message}`)
-						} else {
-							console.error(
-								`[read_file] Unhandled error: ${error instanceof Error ? error.message : String(error)}`,
-							)
-						}
+				// Handle offset-only reads (no limit specified) - cap to MAX_READ_FILE_LINES
+				if (fileResult.offset !== undefined && fileResult.offset > 1) {
+					const startLine = fileResult.offset
+					const endLine = startLine + MAX_READ_FILE_LINES - 1
+					const actualEndLine = Math.min(endLine, totalLines)
+					const linesRead = actualEndLine - startLine + 1
+					const content = addLineNumbers(
+						await readLines(fullPath, actualEndLine - 1, startLine - 1),
+						startLine,
+					)
+					let xmlContent = content
+					if (totalLines > actualEndLine) {
+						xmlContent = `[showing ${linesRead} lines from offset ${startLine}, capped at ${MAX_READ_FILE_LINES} lines. Total file length: ${totalLines} lines]\n${content}`
 					}
+					updateFileResult(relPath, {
+						xmlContent,
+					})
 					continue
 				}
 

src/integrations/theme/default-themes/theme-variables.css

@@ -70,7 +70,7 @@ classes at build time based on the @theme */
 --card-foreground: var(--vscode-editor-foreground);
 --popover: var(--vscode-menu-background, var(--vscode-editor-background));
 --popover-foreground: var(--vscode-menu-foreground, var(--vscode-editor-foreground));
---primary: var(--vscode-button-background);
+--primary: var(--color-matterai-green-dark);
 --primary-foreground: var(--vscode-button-foreground);
 --secondary: var(--vscode-button-secondaryBackground);
 --secondary-foreground: var(--vscode-button-secondaryForeground);

src/package.json

@@ -3,7 +3,7 @@
 	"displayName": "%extension.displayName%",
 	"description": "%extension.description%",
 	"publisher": "matterai",
-	"version": "5.0.1",
+	"version": "5.0.2",
 	"icon": "assets/icons/matterai-ic.png",
 	"galleryBanner": {
 		"color": "#FFFFFF",

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

@@ -1,21 +1,18 @@
+import { CheckCheck, X } from "lucide-react"
 import { HTMLAttributes, useState } from "react"
-import { X, CheckCheck } from "lucide-react"
-import { Trans } from "react-i18next"
-import { Package } from "@roo/package"
 
+import { Button, Input } from "@/components/ui"
 import { useAppTranslation } from "@/i18n/TranslationContext"
-import { VSCodeCheckbox } from "@vscode/webview-ui-toolkit/react"
 import { vscode } from "@/utils/vscode"
-import { Button, Input, Slider } from "@/components/ui"
+import { VSCodeCheckbox } from "@vscode/webview-ui-toolkit/react"
 
-import { SetCachedStateField } from "./types"
-import { SectionHeader } from "./SectionHeader"
-import { Section } from "./Section"
-import { AutoApproveToggle } from "./AutoApproveToggle"
-import { MaxLimitInputs } from "./MaxLimitInputs"
 import { useExtensionState } from "@/context/ExtensionStateContext"
 import { useAutoApprovalState } from "@/hooks/useAutoApprovalState"
 import { useAutoApprovalToggles } from "@/hooks/useAutoApprovalToggles"
+import { AutoApproveToggle } from "./AutoApproveToggle"
+import { Section } from "./Section"
+import { SectionHeader } from "./SectionHeader"
+import { SetCachedStateField } from "./types"
 
 type AutoApproveSettingsProps = HTMLAttributes<HTMLDivElement> & {
 	alwaysAllowReadOnly?: boolean
@@ -32,7 +29,7 @@ type AutoApproveSettingsProps = HTMLAttributes<HTMLDivElement> & {
 	alwaysAllowExecute?: boolean
 	alwaysAllowFollowupQuestions?: boolean
 	alwaysAllowUpdateTodoList?: boolean
-	followupAutoApproveTimeoutMs?: number
+	// followupAutoApproveTimeoutMs?: number
 	allowedCommands?: string[]
 	allowedMaxRequests?: number | undefined
 	allowedMaxCost?: number | undefined
@@ -78,7 +75,7 @@ export const AutoApproveSettings = ({
 	alwaysAllowSubtasks,
 	alwaysAllowExecute,
 	alwaysAllowFollowupQuestions,
-	followupAutoApproveTimeoutMs = 60000,
+	// followupAutoApproveTimeoutMs = 60000,
 	alwaysAllowUpdateTodoList,
 	allowedCommands,
 	allowedMaxRequests,
@@ -130,7 +127,7 @@ export const AutoApproveSettings = ({
 			</SectionHeader>
 
 			{/* kilocode_change start */}
-			<Section>
+			{/* <Section>
 				<div>
 					<VSCodeCheckbox
 						checked={showAutoApproveMenu}
@@ -142,10 +139,10 @@ export const AutoApproveSettings = ({
 						{t("settings:autoApprove.showMenu.description")}
 					</div>
 				</div>
-			</Section>
+			</Section> */}
 
 			{/* YOLO MODE SECTION */}
-			{process.env.NODE_ENV === "development" && (
+			{/* {process.env.NODE_ENV === "development" && (
 				<Section>
 					<div className="border-2 border-yellow-500 rounded-md p-4 bg-yellow-500/10">
 						<div className="flex items-center gap-2 mb-3">
@@ -170,9 +167,9 @@ export const AutoApproveSettings = ({
 						</div>
 					</div>
 				</Section>
-			)}
+			)} */}
 
-			{process.env.NODE_ENV === "development" && yoloMode && (
+			{/* {process.env.NODE_ENV === "development" && yoloMode && (
 				<Section>
 					<div className="bg-yellow-500/10 border border-yellow-500/30 rounded p-3 flex items-center gap-2">
 						<span className="text-lg">⚡</span>
@@ -181,7 +178,7 @@ export const AutoApproveSettings = ({
 						</span>
 					</div>
 				</Section>
-			)}
+			)} */}
 			{/* kilocode_change end */}
 
 			<Section>
@@ -196,7 +193,7 @@ export const AutoApproveSettings = ({
 						}}>
 						<span className="font-medium">{t("settings:autoApprove.enabled")}</span>
 					</VSCodeCheckbox>
-					<div className="text-vscode-descriptionForeground text-sm mt-1">
+					{/* <div className="text-vscode-descriptionForeground text-sm mt-1">
 						<p>{t("settings:autoApprove.description")}</p>
 						<p>
 							<Trans
@@ -219,7 +216,7 @@ export const AutoApproveSettings = ({
 								}}
 							/>
 						</p>
-					</div>
+					</div> */}
 
 					<AutoApproveToggle
 						alwaysAllowReadOnly={alwaysAllowReadOnly}
@@ -235,17 +232,17 @@ export const AutoApproveSettings = ({
 						onToggle={(key, value) => setCachedStateField(key, value)}
 					/>
 
-					<MaxLimitInputs
+					{/* <MaxLimitInputs
 						allowedMaxRequests={allowedMaxRequests}
 						allowedMaxCost={allowedMaxCost}
 						onMaxRequestsChange={(value) => setCachedStateField("allowedMaxRequests", value)}
 						onMaxCostChange={(value) => setCachedStateField("allowedMaxCost", value)}
-					/>
+					/> */}
 				</div>
 
 				{/* ADDITIONAL SETTINGS */}
 
-				{alwaysAllowReadOnly && (
+				{/* {alwaysAllowReadOnly && (
 					<div className="flex flex-col gap-3 pl-3 border-l-2 border-vscode-button-background">
 						<div className="flex items-center gap-4 font-bold">
 							<span className="codicon codicon-eye" />
@@ -356,10 +353,10 @@ export const AutoApproveSettings = ({
 							</div>
 						</div>
 					</div>
-				)}
+				)} */}
 
 				{alwaysAllowExecute && (
-					<div className="flex flex-col gap-3 pl-3 border-l-2 border-vscode-button-background">
+					<div className="flex flex-col gap-3 pl-3 border-vscode-button-background">
 						<div className="flex items-center gap-4 font-bold">
 							<span className="codicon codicon-terminal" />
 							<div>{t("settings:autoApprove.execute.label")}</div>