liuyu/openclaw
1
0
Fork 0
mirror of https://github.com/openclaw/openclaw.git synced 2026-06-06 05:51:15 +08:00
Code Issues Packages Projects Releases Wiki Activity

fix(test): reject malformed boundary prep timeouts

Browse Source
This commit is contained in:
Vincent Koc
2026-06-02 22:26:15 +02:00
parent ac8338bb02
commit 6af047c7f6
2 changed files with 11 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
scripts/prepare-extension-package-boundary-artifacts.mjs
View File

@@ -2,6 +2,7 @@ import { spawn } from "node:child_process";
import fs from "node:fs";
import path, { resolve } from "node:path";
import { isLocalCheckEnabled } from "./lib/local-heavy-check-runtime.mjs";
import { parsePositiveInt } from "./lib/numeric-options.mjs";
const repoRoot = resolve(import.meta.dirname, "..");
const runTsgoScript = path.join(repoRoot, "scripts/run-tsgo.mjs");
@@ -257,12 +258,11 @@ export function parseMode(argv = process.argv.slice(2)) {
}
export function resolveBoundaryRootShimsTimeoutMs(env = process.env) {
const raw = env.OPENCLAW_PLUGIN_SDK_BOUNDARY_ROOT_SHIMS_TIMEOUT_MS;
if (raw === undefined || raw.trim() === "") {
const raw = env.OPENCLAW_PLUGIN_SDK_BOUNDARY_ROOT_SHIMS_TIMEOUT_MS?.trim();
if (!raw) {
return 300_000;
}
const parsed = Number.parseInt(raw, 10);
return Number.isInteger(parsed) && parsed > 0 && String(parsed) === raw.trim() ? parsed : 300_000;
return parsePositiveInt(raw, "OPENCLAW_PLUGIN_SDK_BOUNDARY_ROOT_SHIMS_TIMEOUT_MS");
}
function collectNewestMtime(paths, params = {}) {

9
test/scripts/prepare-extension-package-boundary-artifacts.test.ts
View File

@@ -371,10 +371,15 @@ describe("prepare-extension-package-boundary-artifacts", () => {
OPENCLAW_PLUGIN_SDK_BOUNDARY_ROOT_SHIMS_TIMEOUT_MS: "450000",
}),
).toBe(450_000);
expect(
expect(() =>
resolveBoundaryRootShimsTimeoutMs({
OPENCLAW_PLUGIN_SDK_BOUNDARY_ROOT_SHIMS_TIMEOUT_MS: "120s",
}),
).toBe(300_000);
).toThrow("OPENCLAW_PLUGIN_SDK_BOUNDARY_ROOT_SHIMS_TIMEOUT_MS must be a positive integer");
expect(() =>
resolveBoundaryRootShimsTimeoutMs({
OPENCLAW_PLUGIN_SDK_BOUNDARY_ROOT_SHIMS_TIMEOUT_MS: "0",
}),
).toThrow("OPENCLAW_PLUGIN_SDK_BOUNDARY_ROOT_SHIMS_TIMEOUT_MS must be a positive integer");
});
});