惯性聚合 高效追踪和阅读你感兴趣的博客、新闻、科技资讯
阅读原文 在惯性聚合中打开

推荐订阅源

Microsoft Azure Blog
Microsoft Azure Blog
GbyAI
GbyAI
P
Proofpoint News Feed
Engineering at Meta
Engineering at Meta
Recent Announcements
Recent Announcements
L
LangChain Blog
B
Blog
阮一峰的网络日志
阮一峰的网络日志
Microsoft Security Blog
Microsoft Security Blog
博客园 - 【当耐特】
M
MIT News - Artificial intelligence
D
Docker
WordPress大学
WordPress大学
J
Java Code Geeks
奇客Solidot–传递最新科技情报
奇客Solidot–传递最新科技情报
The GitHub Blog
The GitHub Blog
博客园 - 叶小钗
Last Week in AI
Last Week in AI
Stack Overflow Blog
Stack Overflow Blog
有赞技术团队
有赞技术团队
MyScale Blog
MyScale Blog
H
Hackread – Cybersecurity News, Data Breaches, AI and More
MongoDB | Blog
MongoDB | Blog
博客园 - Franky

Recent Commits to openclaw:main

test: merge chat side-result checks · openclaw/openclaw@ddd2c2a test: merge cron history checks · openclaw/openclaw@f7eb746 test: merge responsive navigation shell checks · openclaw/openclaw@c2e4b47 docs(changelog): add codex oauth fixes · openclaw/openclaw@628e6cd test: merge navigation routing cases · openclaw/openclaw@5d8cecb Tests: mock channel registry bundled fallback · openclaw/openclaw@2b08233 Secrets: avoid broad web search discovery for single plugin config · openclaw/openclaw@a464f59 test: merge config view browser checks · openclaw/openclaw@20cf511 fix(status): align oauth health with runtime · openclaw/openclaw@eed7116 feat: add macOS screen snapshots for monitor preview (#67954) thanks … · openclaw/openclaw@f377db1 fix: report shared auth scopes in hello-ok (#67810) thanks @BunsDev · openclaw/openclaw@0b6c39b Auto-reply: avoid eager bundled route fallback · openclaw/openclaw@3ea1bf4 Tests: narrow session binding contract setup · openclaw/openclaw@54e4e16 fix(macOS): enable undo/redo in webchat composer text input (#34962) · openclaw/openclaw@00951dc Tests: speed up channel setup promotion · openclaw/openclaw@82b529a Docs: refresh agent instructions · openclaw/openclaw@5775fe2 fix(auth): serialize OAuth refresh across agents to fix #26322 (#67876) · openclaw/openclaw@8e79080 test: allow ollama public surface boundary test · openclaw/openclaw@7d4f1a6 Docs: add test performance guardrails · openclaw/openclaw@89706d3 Tests: restore context-engine usage proof · openclaw/openclaw@e4c4f95 Tests: slim context engine runtime coverage · openclaw/openclaw@74c198f ci: retry failed custom checkouts · openclaw/openclaw@0ee5baf test: trim duplicate provider auth onboarding cases · openclaw/openclaw@1ffc02e matrix: fix sessions_spawn --thread subagent session spawning (#67643) · openclaw/openclaw@1ce2596 test: reduce auth choice fixture churn · openclaw/openclaw@857b9cd test: mock health status config boundaries · openclaw/openclaw@9d5ab4a test: mock onboard config io boundary · openclaw/openclaw@299694d test: mock legacy state plugin boundaries · openclaw/openclaw@2713089 test: mock channel install boundaries · openclaw/openclaw@b945248 test: mock doctor preview channel boundaries · openclaw/openclaw@b1a3ad4
fix(codex): send legacy dynamic tool start specs · opencl...
vincentkoc · 2026-06-17 · via Recent Commits to openclaw:main

File tree

  • extensions/codex/src/app-server

    • lib/codex-npm-plugin-live

Original file line numberDiff line numberDiff line change

@@ -84,6 +84,18 @@ export type CodexDynamicToolNamespaceSpec = JsonObject & {

8484
8585

export type CodexDynamicToolSpec = CodexDynamicToolFunctionSpec | CodexDynamicToolNamespaceSpec;

8686
87+

export type CodexLegacyDynamicToolFunctionSpec = JsonObject & {

88+

name: string;

89+

description: string;

90+

inputSchema: JsonValue;

91+

deferLoading?: boolean;

92+

namespace?: string;

93+

};

94+
95+

export type CodexThreadStartDynamicToolSpec =

96+

| CodexDynamicToolSpec

97+

| CodexLegacyDynamicToolFunctionSpec;

98+
8799

export function flattenCodexDynamicToolFunctions(

88100

tools: readonly CodexDynamicToolSpec[] | undefined,

89101

): CodexDynamicToolFunctionSpec[] {

@@ -105,7 +117,7 @@ export type CodexThreadStartParams = JsonObject & {

105117

approvalsReviewer?: string | null;

106118

sandbox?: string;

107119

serviceTier?: CodexServiceTier | null;

108-

dynamicTools?: CodexDynamicToolSpec[] | null;

120+

dynamicTools?: CodexThreadStartDynamicToolSpec[] | null;

109121

developerInstructions?: string;

110122

experimentalRawEvents?: boolean;

111123

environments?: CodexTurnEnvironmentParams[] | null;

Original file line numberDiff line numberDiff line change

@@ -97,7 +97,7 @@ describe("Codex app-server dynamic tool schema boundary contract", () => {

9797

vi.restoreAllMocks();

9898

});

9999
100-

it("passes prepared executable dynamic tool schemas through thread start unchanged", async () => {

100+

it("passes prepared executable dynamic tool schemas through legacy thread start specs", async () => {

101101

const sessionFile = path.join(tempDir, "session.jsonl");

102102

const workspaceDir = path.join(tempDir, "workspace");

103103

const parameterFreeTool = createParameterFreeTool("message");

@@ -128,7 +128,13 @@ describe("Codex app-server dynamic tool schema boundary contract", () => {

128128

throw new Error(`expected thread/start request, got ${method}`);

129129

}

130130

const startPayload = payload as CodexThreadStartParams | undefined;

131-

expect(startPayload?.dynamicTools).toStrictEqual([dynamicTool]);

131+

expect(startPayload?.dynamicTools).toStrictEqual([

132+

{

133+

name: dynamicTool.name,

134+

description: dynamicTool.description,

135+

inputSchema: dynamicTool.inputSchema,

136+

},

137+

]);

132138

expect(startPayload?.cwd).toBe(workspaceDir);

133139

expect(startPayload?.model).toBe("gpt-5.4");

134140

expect(startPayload?.modelProvider).toBeUndefined();

Original file line numberDiff line numberDiff line change

@@ -290,6 +290,47 @@ describe("Codex app-server thread lifecycle bindings", () => {

290290

expect(request.mock.calls.map(([method]) => method)).toEqual(["thread/start", "thread/resume"]);

291291

});

292292
293+

it("sends legacy flat dynamic tools on thread start", async () => {

294+

const sessionFile = path.join(tempDir, "session.jsonl");

295+

const workspaceDir = path.join(tempDir, "workspace");

296+

const params = createParams(sessionFile, workspaceDir);

297+

const appServer = createThreadLifecycleAppServerOptions();

298+

const request = vi.fn(async (method: string, _requestParams?: unknown) => {

299+

if (method === "thread/start") {

300+

return threadStartResult("thread-flat-tools");

301+

}

302+

throw new Error(`unexpected method: ${method}`);

303+

});

304+
305+

await startOrResumeThread({

306+

client: { request } as never,

307+

params,

308+

cwd: workspaceDir,

309+

dynamicTools: [

310+

createMessageDynamicTool("Send a message."),

311+

createDeferredNamedDynamicTool("web_search"),

312+

],

313+

appServer,

314+

});

315+
316+

const startParams = request.mock.calls.find(([method]) => method === "thread/start")?.[1] as

317+

| { dynamicTools?: unknown[] }

318+

| undefined;

319+

expect(startParams?.dynamicTools).toEqual([

320+

expect.objectContaining({

321+

name: "message",

322+

description: "Send a message.",

323+

}),

324+

expect.objectContaining({

325+

name: "web_search",

326+

namespace: "openclaw",

327+

deferLoading: true,

328+

}),

329+

]);

330+

expect(startParams?.dynamicTools?.[0]).not.toHaveProperty("type");

331+

expect(startParams?.dynamicTools?.[1]).not.toHaveProperty("type");

332+

});

333+
293334

it("keeps the bound local provider when recoverable resume failure starts a fresh thread", async () => {

294335

const sessionFile = path.join(tempDir, "session.jsonl");

295336

const workspaceDir = path.join(tempDir, "workspace");

Original file line numberDiff line numberDiff line change

@@ -39,7 +39,9 @@ import {

3939

import {

4040

flattenCodexDynamicToolFunctions,

4141

isJsonObject,

42+

type CodexDynamicToolFunctionSpec,

4243

type CodexDynamicToolSpec,

44+

type CodexLegacyDynamicToolFunctionSpec,

4345

type CodexSandboxPolicy,

4446

type CodexThreadResumeParams,

4547

type CodexThreadStartParams,

@@ -1066,12 +1068,33 @@ export function buildThreadStartParams(

10661068

developerInstructions:

10671069

options.developerInstructions ??

10681070

buildDeveloperInstructions(params, { dynamicTools: options.dynamicTools }),

1069-

dynamicTools: options.dynamicTools,

1071+

dynamicTools: toCodexThreadStartDynamicTools(options.dynamicTools),

10701072

experimentalRawEvents: true,

10711073

persistExtendedHistory: true,

10721074

};

10731075

}

10741076
1077+

function toCodexThreadStartDynamicTools(

1078+

dynamicTools: readonly CodexDynamicToolSpec[],

1079+

): CodexLegacyDynamicToolFunctionSpec[] {

1080+

// Managed stable Codex still accepts the legacy flat start payload. Keep

1081+

// OpenClaw namespaces internally, but omit `type` on the wire so Codex does

1082+

// not reject a mixed canonical/legacy shape before thread creation.

1083+

return dynamicTools.flatMap((tool) =>

1084+

tool.type === "namespace"

1085+

? tool.tools.map((child) => toCodexLegacyDynamicTool(child, tool.name))

1086+

: [toCodexLegacyDynamicTool(tool)],

1087+

);

1088+

}

1089+
1090+

function toCodexLegacyDynamicTool(

1091+

tool: CodexDynamicToolFunctionSpec,

1092+

namespace?: string,

1093+

): CodexLegacyDynamicToolFunctionSpec {

1094+

const { type: _type, ...legacyTool } = tool;

1095+

return namespace ? { ...legacyTool, namespace } : legacyTool;

1096+

}

1097+
10751098

export function buildThreadResumeParams(

10761099

params: EmbeddedRunAttemptParams,

10771100

options: {

Original file line numberDiff line numberDiff line change

@@ -1,7 +1,7 @@

11

#!/usr/bin/env bash

22

# Installs OpenClaw from a prepared package tarball, installs @openclaw/codex

33

# from a registry/git/tarball spec, and verifies a live Codex app-server turn.

4-

set -euo pipefail

4+

set -Eeuo pipefail

55
66

SCRIPT_ROOT_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")/../.." && pwd)"

77

TRUSTED_HARNESS_DIR="${OPENCLAW_LIVE_DOCKER_TRUSTED_HARNESS_DIR:-$SCRIPT_ROOT_DIR}"

@@ -131,7 +131,7 @@ if ! docker_e2e_run_with_harness \

131131

"${CODEX_PLUGIN_MOUNT[@]}" \

132132

"${PROFILE_MOUNT[@]}" \

133133

-i "$IMAGE_NAME" bash -s >"$run_log" 2>&1 <<'EOF'; then

134-

set -euo pipefail

134+

set -Eeuo pipefail

135135
136136

source scripts/lib/openclaw-e2e-instance.sh

137137

openclaw_e2e_eval_test_state_from_b64 "${OPENCLAW_TEST_STATE_SCRIPT_B64:?missing OPENCLAW_TEST_STATE_SCRIPT_B64}"

@@ -147,7 +147,7 @@ for profile_path in "$HOME/.profile" /home/appuser/.profile; do

147147

if [ -f "$profile_path" ] && [ -r "$profile_path" ]; then

148148

set +e +u

149149

source "$profile_path"

150-

set -euo pipefail

150+

set -Eeuo pipefail

151151

break

152152

fi

153153

done

@@ -242,17 +242,30 @@ run_agent_turn() {

242242

local message="$3"

243243

local out="$4"

244244

local err="$5"

245+

local status

245246
246247

echo "${label}_prompt: $message"

247-

openclaw agent --local \

248+

if openclaw agent --local \

248249

--agent main \

249250

--session-id "$SESSION_ID" \

250251

--model "$MODEL_REF" \

251252

--message "$message" \

252253

--thinking low \

253254

--timeout 420 \

254-

--json >"$out" 2>"$err" </dev/null

255-

print_agent_reply "$out" "$marker" "${label}_reply"

255+

--json >"$out" 2>"$err" </dev/null; then

256+

status=0

257+

else

258+

status=$?

259+

fi

260+

echo "${label}_agent_status: $status stdout_bytes=$(wc -c <"$out" 2>/dev/null || printf 0) stderr_bytes=$(wc -c <"$err" 2>/dev/null || printf 0)"

261+

if [ "$status" -ne 0 ]; then

262+

dump_debug_logs "$status"

263+

exit "$status"

264+

fi

265+

if ! print_agent_reply "$out" "$marker" "${label}_reply"; then

266+

dump_debug_logs 1

267+

exit 1

268+

fi

256269

}

257270
258271

echo "TRANSCRIPT_BEGIN"

Original file line numberDiff line numberDiff line change

@@ -417,7 +417,7 @@ function assertAgentTurn() {

417417
418418

const bindingPath = `${entry.sessionFile}.codex-app-server.json`;

419419

const binding = readJson(bindingPath);

420-

if (binding.schemaVersion !== 1 || typeof binding.threadId !== "string") {

420+

if (![1, 2].includes(binding.schemaVersion) || typeof binding.threadId !== "string") {

421421

throw new Error(`invalid Codex app-server binding: ${JSON.stringify(binding)}`);

422422

}

423423

if (binding.model !== modelRef.split("/").slice(1).join("/")) {