From fd42aa4c8fa06413808601771ca2d8f5a3c533c2 Mon Sep 17 00:00:00 2001
From: can1357 <me@can.ac>
Date: Sun, 26 Apr 2026 03:47:48 +0200
Subject: [PATCH] fix(coding-agent): adjusted strict validation settings across
 coding-agent tools

- Updated `CustomToolAdapter` to read `strict` from the wrapped tool instead of hard-coding it.
- Disabled strict validation for `LspTool` and `ResolveTool` by setting their `strict` flags to false.
- Updated `YieldTool` to set `strict` false in its fallback path and remove the local strict accumulator.
---
 .../coding-agent/src/extensibility/custom-tools/wrapper.ts    | 3 ++-
 packages/coding-agent/src/lsp/index.ts                        | 2 +-
 packages/coding-agent/src/tools/resolve.ts                    | 2 +-
 packages/coding-agent/src/tools/yield.ts                      | 4 +---
 4 files changed, 5 insertions(+), 6 deletions(-)

diff --git a/packages/coding-agent/src/extensibility/custom-tools/wrapper.ts b/packages/coding-agent/src/extensibility/custom-tools/wrapper.ts
index e3edeeb0f..3483c6009 100644
--- a/packages/coding-agent/src/extensibility/custom-tools/wrapper.ts
+++ b/packages/coding-agent/src/extensibility/custom-tools/wrapper.ts
@@ -14,13 +14,14 @@ export class CustomToolAdapter<TParams extends TSchema = TSchema, TDetails = any
 	declare label: string;
 	declare description: string;
 	declare parameters: TParams;
-	readonly strict = true;
+	readonly strict: boolean | undefined;
 
 	constructor(
 		private tool: CustomTool<TParams, TDetails>,
 		private getContext: () => CustomToolContext,
 	) {
 		applyToolProxy(tool, this);
+		this.strict = tool.strict;
 	}
 
 	execute(
diff --git a/packages/coding-agent/src/lsp/index.ts b/packages/coding-agent/src/lsp/index.ts
index 68a61bd68..db1021809 100644
--- a/packages/coding-agent/src/lsp/index.ts
+++ b/packages/coding-agent/src/lsp/index.ts
@@ -1115,11 +1115,11 @@ export class LspTool implements AgentTool<typeof lspSchema, LspToolDetails, Them
 	readonly label = "LSP";
 	readonly description: string;
 	readonly parameters = lspSchema;
-	readonly strict = true;
 	readonly renderCall = renderCall;
 	readonly renderResult = renderResult;
 	readonly mergeCallAndResult = true;
 	readonly inline = true;
+	readonly strict = false;
 
 	constructor(private readonly session: ToolSession) {
 		this.description = prompt.render(lspDescription);
diff --git a/packages/coding-agent/src/tools/resolve.ts b/packages/coding-agent/src/tools/resolve.ts
index dbd6b9e5d..78a3e4794 100644
--- a/packages/coding-agent/src/tools/resolve.ts
+++ b/packages/coding-agent/src/tools/resolve.ts
@@ -109,7 +109,7 @@ export class ResolveTool implements AgentTool<typeof resolveSchema, ResolveToolD
 	readonly hidden = true;
 	readonly description: string;
 	readonly parameters = resolveSchema;
-	readonly strict = true;
+	readonly strict = false;
 
 	constructor(private readonly session: ToolSession) {
 		this.description = prompt.render(resolveDescription);
diff --git a/packages/coding-agent/src/tools/yield.ts b/packages/coding-agent/src/tools/yield.ts
index 877dbfde4..bfe9ca0a1 100644
--- a/packages/coding-agent/src/tools/yield.ts
+++ b/packages/coding-agent/src/tools/yield.ts
@@ -74,7 +74,6 @@ export class YieldTool implements AgentTool<TSchema, YieldDetails> {
 		let validate: ValidateFunction | undefined;
 		let dataSchema: TSchema;
 		let parameters: TSchema;
-		let strict = true;
 
 		try {
 			const schemaResult = normalizeSchema(session.outputSchema);
@@ -131,12 +130,11 @@ export class YieldTool implements AgentTool<TSchema, YieldDetails> {
 			});
 			parameters = createParameters(dataSchema);
 			validate = undefined;
-			strict = false;
+			this.strict = false;
 		}
 
 		this.#validate = validate;
 		this.parameters = parameters;
-		this.strict = strict;
 	}
 
 	async execute(