ewilliams28/openclaw
1
0
Fork 0
mirror of https://github.com/openclaw/openclaw.git synced 2026-08-12 21:53:00 -06:00
Code Issues Packages Projects Releases Wiki Activity

test(packaging): speed tarball integrity checks (#121999)

Browse Source 
Co-authored-by: Amp <amp@ampcode.com>
This commit is contained in:
Peter Steinberger
2026-08-11 03:56:38 -07:00
committed by GitHub
parent ffc665bbb3
commit 26d0158e50
1 changed files with 3 additions and 104 deletions
Download Patch File Download Diff File Expand all files Collapse all files
test/scripts/check-openclaw-package-tarball.test.ts
+3 -104
View File
@@ -17,7 +17,8 @@ import { LOCAL_BUILD_METADATA_DIST_PATHS } from "../../scripts/lib/local-build-m
import { PACKAGE_INSTALL_GUARD_RELATIVE_PATH } from "../../scripts/lib/package-dist-inventory.ts";
import { WORKSPACE_TEMPLATE_PACK_PATHS } from "../../scripts/lib/workspace-bootstrap-smoke.mts";
const CHECK_SCRIPT = "scripts/check-openclaw-package-tarball.mjs";
const CHECK_SCRIPT = "scripts/check-openclaw-package-tarball.mts";
const PUBLIC_CHECK_SCRIPT = "scripts/check-openclaw-package-tarball.mjs";
const NODE_DEFAULT_SPAWN_MAX_BUFFER_BYTES = 1024 * 1024;
const CODE_MODE_WORKER_PATH = "dist/agents/code-mode.worker.js";
const FIRST_CODE_MODE_WORKER_VERSION = "2026.5.14-beta.2";
@@ -149,7 +150,7 @@ function withTarball(
describe("check-openclaw-package-tarball", () => {
it("prints help before touching tarball state", () => {
const result = spawnSync("node", [CHECK_SCRIPT, "--help"], { encoding: "utf8" });
const result = spawnSync("node", [PUBLIC_CHECK_SCRIPT, "--help"], { encoding: "utf8" });
expect(result.status, result.stderr).toBe(0);
expect(result.stdout).toContain(
@@ -280,19 +281,6 @@ describe("check-openclaw-package-tarball", () => {
);
});
it("still rejects non-legacy missing inventory entries", () => {
withTarball(
["dist/index.js", "dist/cli.js"],
{ "dist/index.js": "export {};\n" },
(tarball) => {
const result = spawnSync("node", [CHECK_SCRIPT, tarball], { encoding: "utf8" });
expect(result.status).not.toBe(0);
expect(result.stderr).toContain("inventory references missing tar entry dist/cli.js");
},
);
});
it("requires an install guard omitted from the dist inventory", () => {
withTarball(
["dist/index.js"],
@@ -351,19 +339,6 @@ describe("check-openclaw-package-tarball", () => {
);
});
it("accepts flat plugin SDK declaration inventory without the old deep tree", () => {
withTarball(
[FLAT_PLUGIN_SDK_DECLARATION],
{ [FLAT_PLUGIN_SDK_DECLARATION]: "export {};\n" },
(tarball) => {
const result = spawnSync("node", [CHECK_SCRIPT, tarball], { encoding: "utf8" });
expect(result.status, result.stderr).toBe(0);
expect(result.stdout).toContain("OpenClaw package tarball integrity passed.");
},
);
});
it("accepts historical packages published before the Code Mode worker existed", () => {
withTarball(
["dist/index.js"],
@@ -408,20 +383,6 @@ describe("check-openclaw-package-tarball", () => {
);
});
it("accepts Code Mode packages whose worker survives postinstall", () => {
withTarball(
["dist/index.js"],
{ "dist/index.js": "export {};\n" },
(tarball) => {
const result = spawnSync("node", [CHECK_SCRIPT, tarball], { encoding: "utf8" });
expect(result.status, result.stderr).toBe(0);
expect(result.stdout).toContain("OpenClaw package tarball integrity passed.");
},
FIRST_CODE_MODE_WORKER_VERSION,
);
});
it("rejects dist files that import missing relative chunks", () => {
withTarball(
["dist/cli/run-main.js"],
@@ -457,47 +418,6 @@ describe("check-openclaw-package-tarball", () => {
);
});
it.each([
{
name: "named imports",
source: 'import { value } from "./missing.js";\n',
},
{
name: "multiline named imports",
source: 'import {\n value,\n} from "./missing.js";\n',
},
{
name: "named re-exports",
source: 'export { value } from "./missing.js";\n',
},
])("rejects missing packaged chunks in $name", ({ source }) => {
withTarball(
["dist/index.js"],
{ "dist/index.js": source },
(tarball) => {
const result = spawnSync("node", [CHECK_SCRIPT, tarball], { encoding: "utf8" });
expect(result.status).not.toBe(0);
expect(result.stderr).toContain("dist/index.js imports missing dist/missing.js");
},
"2026.4.27",
);
});
it("does not reject import-like text inside packaged template literals", () => {
withTarball(
["dist/index.js"],
{ "dist/index.js": 'const example = `\nimport "./phantom.js"\n`;\n' },
(tarball) => {
const result = spawnSync("node", [CHECK_SCRIPT, tarball], { encoding: "utf8" });
expect(result.status, result.stderr).toBe(0);
expect(result.stdout).toContain("OpenClaw package tarball integrity passed.");
},
"2026.4.27",
);
});
it("accepts dist files whose relative chunks are present", () => {
withTarball(
["dist/cli/run-main.js", "dist/memory-state-current.js"],
@@ -635,27 +555,6 @@ describe("check-openclaw-package-tarball", () => {
);
});
it.each([
"../../openclaw.mjs",
"../../scripts/run-node.mjs",
"../../dist/entry.js",
"../../dist/entry.mjs",
])("allows import.meta.url JavaScript probes outside packaged dist (%s)", (specifier) => {
withTarball(
["dist/index.js"],
{
"dist/index.js": `const candidate = new URL(${JSON.stringify(specifier)}, import.meta.url);\n`,
},
(tarball) => {
const result = spawnSync("node", [CHECK_SCRIPT, tarball], { encoding: "utf8" });
expect(result.status, result.stderr).toBe(0);
expect(result.stdout).toContain("OpenClaw package tarball integrity passed.");
},
"2026.4.27",
);
});
it("rejects missing Control UI assets", () => {
withTarball(
["dist/index.js"],