From 1e4f511f0a3fd5975fb4ad3b1194e88e4246d584 Mon Sep 17 00:00:00 2001 From: Peter Steinberger Date: Fri, 1 May 2026 21:39:03 +0100 Subject: [PATCH] chore: clean up plugin dependency wording --- extensions/codex/src/app-server/managed-binary.test.ts | 6 +++--- extensions/qa-lab/src/suite-runtime-agent-process.test.ts | 4 ++-- scripts/test-docker-all.mjs | 2 +- scripts/test-live-build-docker.sh | 2 +- test/scripts/postinstall-bundled-plugins.test.ts | 4 ++-- 5 files changed, 9 insertions(+), 9 deletions(-) diff --git a/extensions/codex/src/app-server/managed-binary.test.ts b/extensions/codex/src/app-server/managed-binary.test.ts index 304ce68c721..4bf95f3e4e3 100644 --- a/extensions/codex/src/app-server/managed-binary.test.ts +++ b/extensions/codex/src/app-server/managed-binary.test.ts @@ -64,8 +64,8 @@ describe("managed Codex app-server binary", () => { ); }); - it("finds Codex in the external runtime-deps install root used by packaged plugins", async () => { - const installRoot = path.join("/tmp", "openclaw-runtime-deps", "codex"); + it("finds Codex in the package install root used by packaged plugins", async () => { + const installRoot = path.join("/tmp", "openclaw-plugin-package", "codex"); const pluginRoot = path.join(installRoot, "dist", "extensions", "codex"); const installedCommand = managedCommandPath(installRoot, "linux"); const pathExists = vi.fn(async (filePath: string) => filePath === installedCommand); @@ -83,7 +83,7 @@ describe("managed Codex app-server binary", () => { }); }); - it("fails clearly when bundled runtime deps did not stage Codex", async () => { + it("fails clearly when the managed Codex binary is missing", async () => { await expect( resolveManagedCodexAppServerStartOptions(startOptions("managed"), { platform: "darwin", diff --git a/extensions/qa-lab/src/suite-runtime-agent-process.test.ts b/extensions/qa-lab/src/suite-runtime-agent-process.test.ts index 133eb73f196..473690ff6eb 100644 --- a/extensions/qa-lab/src/suite-runtime-agent-process.test.ts +++ b/extensions/qa-lab/src/suite-runtime-agent-process.test.ts @@ -194,7 +194,7 @@ describe("qa suite runtime agent process helpers", () => { child.stdout.emit( "data", Buffer.from( - '\u001b[35m[plugins]\u001b[39m \u001b[36mcodex installed bundled runtime deps\u001b[39m\n{"results":[{"text":"ORBIT-10"}]}\n', + '\u001b[35m[plugins]\u001b[39m \u001b[36mcodex loaded plugin package metadata\u001b[39m\n{"results":[{"text":"ORBIT-10"}]}\n', ), ); child.emit("exit", 0); @@ -225,7 +225,7 @@ describe("qa suite runtime agent process helpers", () => { child.stdout.emit( "data", Buffer.from( - '[plugins] memory-core installed bundled runtime deps\n{\n "results": [\n {\n "text": "ORBIT-10"\n }\n ]\n}\n', + '[plugins] memory-core loaded plugin package metadata\n{\n "results": [\n {\n "text": "ORBIT-10"\n }\n ]\n}\n', ), ); child.emit("exit", 0); diff --git a/scripts/test-docker-all.mjs b/scripts/test-docker-all.mjs index 237c2b04b60..3437e3117f2 100644 --- a/scripts/test-docker-all.mjs +++ b/scripts/test-docker-all.mjs @@ -1175,7 +1175,7 @@ async function main() { console.log(`==> Selected lanes: ${selectedLaneNames.join(", ")}`); } console.log(`==> Docker lane timings: ${timingStore.enabled ? timingsFile : "disabled"}`); - console.log(`==> Live-test bundled plugin deps: ${baseEnv.OPENCLAW_DOCKER_BUILD_EXTENSIONS}`); + console.log(`==> Live-test bundled plugins: ${baseEnv.OPENCLAW_DOCKER_BUILD_EXTENSIONS}`); const schedulerOptions = parseSchedulerOptions(process.env, parallelism); const tailSchedulerOptions = parseSchedulerOptions(process.env, tailParallelism); console.log( diff --git a/scripts/test-live-build-docker.sh b/scripts/test-live-build-docker.sh index 35e42403578..c25cc2df9a7 100755 --- a/scripts/test-live-build-docker.sh +++ b/scripts/test-live-build-docker.sh @@ -40,5 +40,5 @@ if [[ "${OPENCLAW_SKIP_DOCKER_BUILD:-}" == "1" ]]; then fi echo "==> Build live-test image: $LIVE_IMAGE_NAME (target=build)" -echo "==> Bundled plugin deps: ${DOCKER_BUILD_EXTENSIONS}" +echo "==> Bundled plugins: ${DOCKER_BUILD_EXTENSIONS}" docker_build_run live-build "${DOCKER_BUILD_ARGS[@]}" --target build -t "$LIVE_IMAGE_NAME" -f "$ROOT_DIR/Dockerfile" "$ROOT_DIR" diff --git a/test/scripts/postinstall-bundled-plugins.test.ts b/test/scripts/postinstall-bundled-plugins.test.ts index 5a5f8eff2ad..bfe4af5a2a9 100644 --- a/test/scripts/postinstall-bundled-plugins.test.ts +++ b/test/scripts/postinstall-bundled-plugins.test.ts @@ -61,7 +61,7 @@ describe("bundled plugin postinstall", () => { ).toBe(true); }); - it("does not install bundled plugin deps outside of source checkouts by default", async () => { + it("does not install bundled plugin package deps outside of source checkouts by default", async () => { const extensionsDir = await createExtensionsDir(); const packageRoot = path.dirname(path.dirname(extensionsDir)); await writePluginPackage(extensionsDir, "acpx", { @@ -530,7 +530,7 @@ describe("bundled plugin postinstall", () => { }); it("ignores staged bundled plugin node_modules when pruning packaged dist", async () => { - const packageRoot = await createTempDirAsync("openclaw-packaged-install-runtime-deps-"); + const packageRoot = await createTempDirAsync("openclaw-packaged-install-dist-prune-"); const staleFile = path.join(packageRoot, "dist", "stale-runtime.js"); const packageJson = path.join(packageRoot, "dist", "extensions", "slack", "package.json"); const binDir = path.join(packageRoot, "dist", "extensions", "slack", "node_modules", ".bin");