read_file: restore per-file XML for FILE_NOT_FOUND to feed model context; keep unified UI error emission; update tests

Author: daniel-lxs

src/core/tools/__tests__/readFileTool.spec.ts

@@ -1745,11 +1745,11 @@ describe("read_file tool - FILE_NOT_FOUND unified error", () => {
 		expect(Array.isArray(payload.filePaths)).toBe(true)
 		expect(payload.filePaths).toContain(relPath)
 
-		// Assert per-file XML error was suppressed for ENOENT (no <error> block for this file)
+		// Assert per-file XML error is now included for AI model feedback
 		expect(typeof pushed).toBe("string")
 		const xml = String(pushed)
 		expect(xml).toContain("<files>")
-		// no per-file error tags for the not-found case
-		expect(xml).not.toMatch(/<error>.*File not found.*<\/error>/i)
+		// Should now include per-file error tags for the not-found case (for AI model feedback)
+		expect(xml).toContain(`<file><path>${relPath}</path><error>File not found</error></file>`)
 	})
 })

src/core/tools/readFileTool.ts

@@ -630,8 +630,8 @@ export async function readFileTool(
 					updateFileResult(relPath, {
 						status: "error",
 						error: "File not found",
-						// Suppress per-file XML for not-found to avoid duplicate UI signals
-						xmlContent: undefined,
+						// Include XML content for AI model feedback
+						xmlContent: `<file><path>${relPath}</path><error>File not found</error></file>`,
 					})
 					// Don't call handleError for file not found - we'll emit a unified message later
 				} else {
@@ -747,8 +747,8 @@ export async function readFileTool(
 				updateFileResult(relPath, {
 					status: "error",
 					error: "File not found",
-					// Suppress per-file XML for not-found to avoid duplicate UI signals
-					xmlContent: undefined,
+					// Include XML content for AI model feedback
+					xmlContent: `<file><path>${relPath}</path><error>File not found</error></file>`,
 				})
 			}
 			// Don't call handleError for file not found - we'll emit a unified message later