From 0377592f0b0450e8c5c0b760f5dcc9d8b0879322 Mon Sep 17 00:00:00 2001 From: Hannes Rudolph Date: Thu, 16 Oct 2025 18:25:29 -0600 Subject: [PATCH] test(read-file): remove redundant inline notes per review by @daniel-lxs --- apps/vscode-e2e/src/suite/tools/read-file.test.ts | 6 ------ 1 file changed, 6 deletions(-) diff --git a/apps/vscode-e2e/src/suite/tools/read-file.test.ts b/apps/vscode-e2e/src/suite/tools/read-file.test.ts index 13bdf6b65e..8028a8f7a6 100644 --- a/apps/vscode-e2e/src/suite/tools/read-file.test.ts +++ b/apps/vscode-e2e/src/suite/tools/read-file.test.ts @@ -235,8 +235,6 @@ suite("Roo Code read_file Tool", function () { // Check that no errors occurred assert.strictEqual(errorOccurred, null, "No errors should have occurred") - // Note: read_file tool result content is redacted from clineMessages; verify via AI response instead. - // Also verify the AI mentioned the content in its response const hasContent = messages.some( (m) => @@ -335,8 +333,6 @@ suite("Roo Code read_file Tool", function () { // Verify the read_file tool was executed assert.ok(toolExecuted, "The read_file tool should have been executed") - // Note: read_file tool result content is redacted from clineMessages; verify via AI response instead. - // Also verify the AI mentioned the correct number of lines const hasLineCount = messages.some( (m) => @@ -433,8 +429,6 @@ suite("Roo Code read_file Tool", function () { // Verify tool was executed assert.ok(toolExecuted, "The read_file tool should have been executed") - // Note: read_file tool result content is redacted from clineMessages; verify via AI response instead. - // Also verify the AI mentioned the specific lines const hasLines = messages.some( (m) =>