Skip to content
This repository was archived by the owner on May 15, 2026. It is now read-only.

Commit afa6cf9

Browse files
committed
fix: add max_lines to nativeArgs parsing and update test for removed setting
1 parent f070c84 commit afa6cf9

2 files changed

Lines changed: 2 additions & 1 deletion

File tree

src/core/assistant-message/NativeToolCallParser.ts

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -365,6 +365,7 @@ export class NativeToolCallParser {
365365
? {
366366
anchor_line: this.coerceOptionalNumber(partialArgs.indentation.anchor_line),
367367
max_levels: this.coerceOptionalNumber(partialArgs.indentation.max_levels),
368+
max_lines: this.coerceOptionalNumber(partialArgs.indentation.max_lines),
368369
include_siblings: this.coerceOptionalBoolean(
369370
partialArgs.indentation.include_siblings,
370371
),
@@ -663,6 +664,7 @@ export class NativeToolCallParser {
663664
? {
664665
anchor_line: this.coerceOptionalNumber(args.indentation.anchor_line),
665666
max_levels: this.coerceOptionalNumber(args.indentation.max_levels),
667+
max_lines: this.coerceOptionalNumber(args.indentation.max_lines),
666668
include_siblings: this.coerceOptionalBoolean(
667669
args.indentation.include_siblings,
668670
),

webview-ui/src/components/settings/__tests__/ContextManagementSettings.spec.tsx

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -197,7 +197,6 @@ describe("ContextManagementSettings", () => {
197197
// Check for other sliders
198198
expect(screen.getByTestId("open-tabs-limit-slider")).toBeInTheDocument()
199199
expect(screen.getByTestId("workspace-files-limit-slider")).toBeInTheDocument()
200-
expect(screen.getByTestId("max-concurrent-file-reads-slider")).toBeInTheDocument()
201200

202201
// Check for checkboxes
203202
expect(screen.getByTestId("show-rooignored-files-checkbox")).toBeInTheDocument()

0 commit comments

Comments
 (0)