perf: update codex version

Author: ibuler

Dockerfile

@@ -7,7 +7,7 @@ ARG VERSION=dev
 RUN CGO_ENABLED=0 go build -trimpath -ldflags "-s -w -X main.Version=${VERSION}" -o /opt/kael/kael ./cmd/kael
 
 FROM node:22-trixie-slim AS stage-codex
-RUN npm install --global @openai/codex@0.153.2 \
+RUN npm install --global '@openai/codex@>=0.153.2' \
     && codex --version
 
 FROM node:22-trixie-slim

config_example.yml

@@ -1,5 +1,5 @@
-# Codex harness is the sole Agent runtime; install the exact pinned CLI version.
-# npm install -g @openai/codex@0.153.2
+# Codex harness is the sole Agent runtime; install CLI version 0.153.2 or newer.
+# npm install -g '@openai/codex@>=0.153.2'
 CODEX_BINARY: codex
 # CHAT_AI_BASE_URL in Core must support Responses API. Chat Completions is not supported.
 # 与 Koko 一致，Kael 只使用平铺配置。

docs/adr/0007-codex-harness.md

@@ -13,7 +13,7 @@
 
 ## 协议与执行
 
-固定 `codex-cli 0.153.2`；启动时检查精确版本，Docker 同样固定 npm 包版本。依赖该版本的 experimental dynamicTools 和 environments 字段，升级需要重新生成/验证协议并执行集成测试。
+要求 `codex-cli 0.153.2` 或更高版本；启动时按语义版本比较，Docker 安装满足该下限的最新版本。Codex 尚未达到 1.0，跨 minor 版本仍可能调整 experimental dynamicTools 和 environments 字段，因此升级后需要执行集成测试。
 
 Luna bootstrap 必须收到 `agent_engine=codex`、`agent_protocol_version=1`。现有 `/kael/api/v1` HTTP、PanelDelivery SSE、工具回执和审批端点保持。`model.requested/completed` 的 `scope=agent_turn` 表示一次完整 harness turn，包含工具和审批等待；不再声称是内部单次模型请求。现有 ModelCall/ModelRequestCount 存储字段在本分支记录 harness turn，值通常为 1；Luna 不将该总耗时分摊到某次工具调用。usage 从 Codex 累积计数扣除本 turn 起点，覆盖该 turn 内所有模型调用。
 
@@ -35,7 +35,7 @@ Core TerminalConfig 仍是模型凭据唯一来源；启动及每次 Run 执行
 
 ## 试用
 
-1. 安装 `npm install -g @openai/codex@0.153.2`，用 `codex --version` 核对。
+1. 安装 `npm install -g '@openai/codex@>=0.153.2'`，用 `codex --version` 核对不低于 `0.153.2`。
 2. Core 开启 Chat AI，配置支持 Responses 的模型、base URL 和 API key。保留原组件注册和 Platform Gateway delegation 配置。
 3. 使用 `CODEX_BINARY` 指定可执行路径，或使用默认 PATH 中的 `codex`。运行 `make run`；同时使用 Luna 的 `harness` 分支。
 4. 验证终端只读排查、SQL/脚本 proposal 审批、拒绝审批、停止、多轮追问及断连。真实模型效果和真实 Core/Koko/Chen 联调需要试用环境验收。

internal/runtime/harness.go

@@ -10,6 +10,7 @@ import (
 	"os"
 	"os/exec"
 	"path/filepath"
+	"strconv"
 	"strings"
 	"sync"
 	"time"
@@ -97,8 +98,8 @@ func NewHarness(ctx context.Context, binary, root string, loader model.ConfigLoa
 	versionCtx, cancel := context.WithTimeout(ctx, 10*time.Second)
 	defer cancel()
 	version, err := exec.CommandContext(versionCtx, resolved, "--version").Output()
-	if err != nil || strings.TrimSpace(string(version)) != "codex-cli "+CodexVersion {
-		return nil, fmt.Errorf("Kael requires codex-cli %s", CodexVersion)
+	if err != nil || !compatibleCodexVersion(string(version)) {
+		return nil, fmt.Errorf("Kael requires codex-cli %s or newer", CodexMinimumVersion)
 	}
 	config, err := loader(ctx)
 	if err != nil {
@@ -119,6 +120,51 @@ func NewHarness(ctx context.Context, binary, root string, loader model.ConfigLoa
 	return h, nil
 }
 
+func compatibleCodexVersion(output string) bool {
+	version, ok := strings.CutPrefix(strings.TrimSpace(output), "codex-cli ")
+	if !ok {
+		return false
+	}
+	actual, ok := parseCodexVersion(version)
+	if !ok {
+		return false
+	}
+	minimum, ok := parseCodexVersion(CodexMinimumVersion)
+	if !ok {
+		return false
+	}
+	for i := range actual {
+		if actual[i] != minimum[i] {
+			return actual[i] > minimum[i]
+		}
+	}
+	return true
+}
+
+func parseCodexVersion(version string) ([3]uint64, bool) {
+	var parsed [3]uint64
+	parts := strings.Split(version, ".")
+	if len(parts) != len(parsed) {
+		return parsed, false
+	}
+	for i, part := range parts {
+		if part == "" {
+			return parsed, false
+		}
+		for _, digit := range part {
+			if digit < '0' || digit > '9' {
+				return parsed, false
+			}
+		}
+		value, err := strconv.ParseUint(part, 10, 64)
+		if err != nil {
+			return parsed, false
+		}
+		parsed[i] = value
+	}
+	return parsed, true
+}
+
 func validateConfig(config model.Config) error {
 	parsed, err := url.Parse(config.BaseURL)
 	if err != nil || parsed.Host == "" || (parsed.Scheme != "http" && parsed.Scheme != "https") || parsed.User != nil || parsed.RawQuery != "" || parsed.Fragment != "" || strings.TrimSpace(config.Model) == "" || strings.TrimSpace(config.APIKey) == "" {

internal/runtime/harness_test.go

@@ -17,12 +17,33 @@ import (
 	"github.com/jumpserver/kael/internal/model"
 )
 
-// Runs the actual pinned Codex executable against a local Responses fixture.
+func TestCompatibleCodexVersion(t *testing.T) {
+	tests := map[string]bool{
+		"codex-cli 0.144.4":   false,
+		"codex-cli 0.153.1":   false,
+		"codex-cli 0.153.2":   true,
+		"codex-cli 0.153.999": true,
+		"codex-cli 0.154.0":   true,
+		"codex-cli 1.0.0":     true,
+		"codex-cli 0.153":     false,
+		"codex-cli 0.153.dev": false,
+		"codex 0.153.2":       false,
+	}
+	for output, want := range tests {
+		t.Run(output, func(t *testing.T) {
+			if got := compatibleCodexVersion(output); got != want {
+				t.Fatalf("compatibleCodexVersion(%q) = %t, want %t", output, got, want)
+			}
+		})
+	}
+}
+
+// Runs an actual compatible Codex executable against a local Responses fixture.
 // No real model request or external API credential is used.
 func TestCodexAppServerIntegration(t *testing.T) {
 	binary := os.Getenv("KAEL_TEST_CODEX")
 	if binary == "" {
-		t.Skip("set KAEL_TEST_CODEX to the pinned Codex binary for the protocol integration test")
+		t.Skip("set KAEL_TEST_CODEX to a compatible Codex binary for the protocol integration test")
 	}
 	var mu sync.Mutex
 	requests := 0

internal/runtime/protocol.go

@@ -11,7 +11,7 @@ import (
 	"sync"
 )
 
-const CodexVersion = "0.153.2"
+const CodexMinimumVersion = "0.153.2"
 const maxProtocolBytes = 8 * 1024 * 1024
 
 type rpcMessage struct {