From 4507ace9e3c73a68be461a44e1dacfc0d4ef3c62 Mon Sep 17 00:00:00 2001 From: dj-oyu <68707227+dj-oyu@users.noreply.github.com> Date: Tue, 24 Feb 2026 15:45:41 +0900 Subject: [PATCH] perf: eliminate string(paramsJSON) conversion in codex CLI provider Use sb.Write(paramsJSON) to write []byte directly to Builder instead of converting to string first, matching the claude CLI provider optimization. Co-Authored-By: Claude Opus 4.6 --- pkg/providers/codex_cli_provider.go | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/pkg/providers/codex_cli_provider.go b/pkg/providers/codex_cli_provider.go index 4c783ece5..9803fbdee 100644 --- a/pkg/providers/codex_cli_provider.go +++ b/pkg/providers/codex_cli_provider.go @@ -152,7 +152,9 @@ func (p *CodexCliProvider) buildToolsPrompt(tools []ToolDefinition) string { } if len(tool.Function.Parameters) > 0 { paramsJSON, _ := json.Marshal(tool.Function.Parameters) - sb.WriteString(fmt.Sprintf("Parameters:\n```json\n%s\n```\n", string(paramsJSON))) + sb.WriteString("Parameters:\n```json\n") + sb.Write(paramsJSON) + sb.WriteString("\n```\n") } sb.WriteString("\n") }