Compare commits

...

3 Commits

Author SHA1 Message Date
cexll
0fc5eaaa2d fix: update version tests to match 5.6.3
Generated with SWE-Agent.ai

Co-Authored-By: SWE-Agent.ai <noreply@swe-agent.ai>
2026-01-14 17:26:21 +08:00
cexll
420eb857ff chore: bump codeagent-wrapper version to 5.6.3
Generated with SWE-Agent.ai

Co-Authored-By: SWE-Agent.ai <noreply@swe-agent.ai>
2026-01-14 17:14:06 +08:00
cexll
661656c587 fix(codeagent-wrapper): use config override for codex reasoning effort
Replace invalid `--reasoning-effort` CLI flag with `-c model_reasoning_effort=<value>`
config override, as codex does not support the former.

Generated with SWE-Agent.ai

Co-Authored-By: SWE-Agent.ai <noreply@swe-agent.ai>
2026-01-14 17:04:21 +08:00
3 changed files with 8 additions and 8 deletions

View File

@@ -766,7 +766,7 @@ func buildCodexArgs(cfg *Config, targetArg string) []string {
} }
if reasoningEffort := strings.TrimSpace(cfg.ReasoningEffort); reasoningEffort != "" { if reasoningEffort := strings.TrimSpace(cfg.ReasoningEffort); reasoningEffort != "" {
args = append(args, "--reasoning-effort", reasoningEffort) args = append(args, "-c", "model_reasoning_effort="+reasoningEffort)
} }
args = append(args, "--skip-git-repo-check") args = append(args, "--skip-git-repo-check")

View File

@@ -14,7 +14,7 @@ import (
) )
const ( const (
version = "5.5.0" version = "5.6.3"
defaultWorkdir = "." defaultWorkdir = "."
defaultTimeout = 7200 // seconds (2 hours) defaultTimeout = 7200 // seconds (2 hours)
defaultCoverageTarget = 90.0 defaultCoverageTarget = 90.0

View File

@@ -1970,7 +1970,7 @@ func TestRunBuildCodexArgs_NewMode_WithReasoningEffort(t *testing.T) {
args := buildCodexArgs(cfg, "my task") args := buildCodexArgs(cfg, "my task")
expected := []string{ expected := []string{
"e", "e",
"--reasoning-effort", "high", "-c", "model_reasoning_effort=high",
"--skip-git-repo-check", "--skip-git-repo-check",
"-C", "/test/dir", "-C", "/test/dir",
"--json", "--json",
@@ -2010,13 +2010,13 @@ func TestRunCodexTaskWithContext_CodexReasoningEffort(t *testing.T) {
found := false found := false
for i := 0; i+1 < len(gotArgs); i++ { for i := 0; i+1 < len(gotArgs); i++ {
if gotArgs[i] == "--reasoning-effort" && gotArgs[i+1] == "high" { if gotArgs[i] == "-c" && gotArgs[i+1] == "model_reasoning_effort=high" {
found = true found = true
break break
} }
} }
if !found { if !found {
t.Fatalf("expected --reasoning-effort high in args, got %v", gotArgs) t.Fatalf("expected -c model_reasoning_effort=high in args, got %v", gotArgs)
} }
} }
@@ -3736,7 +3736,7 @@ func TestVersionFlag(t *testing.T) {
} }
}) })
want := "codeagent-wrapper version 5.5.0\n" want := "codeagent-wrapper version 5.6.3\n"
if output != want { if output != want {
t.Fatalf("output = %q, want %q", output, want) t.Fatalf("output = %q, want %q", output, want)
@@ -3752,7 +3752,7 @@ func TestVersionShortFlag(t *testing.T) {
} }
}) })
want := "codeagent-wrapper version 5.5.0\n" want := "codeagent-wrapper version 5.6.3\n"
if output != want { if output != want {
t.Fatalf("output = %q, want %q", output, want) t.Fatalf("output = %q, want %q", output, want)
@@ -3768,7 +3768,7 @@ func TestVersionLegacyAlias(t *testing.T) {
} }
}) })
want := "codex-wrapper version 5.5.0\n" want := "codex-wrapper version 5.6.3\n"
if output != want { if output != want {
t.Fatalf("output = %q, want %q", output, want) t.Fatalf("output = %q, want %q", output, want)