mirror of
https://github.com/openclaw/openclaw.git
synced 2026-05-15 02:10:48 +00:00
* fix(whatsapp): await write stream finish in encryptedStream to fix race-condition ENOENT crash * fix(whatsapp): ship Baileys media hotfix on npm installs * fix(whatsapp): keep Baileys hotfix postinstall best-effort * fix(whatsapp): harden Baileys postinstall temp writes * fix(whatsapp): preserve Baileys hotfix file mode --------- Co-authored-by: termtek <termtek@ubuntu.tail2b72cd.ts.net>
395 lines
12 KiB
TypeScript
395 lines
12 KiB
TypeScript
import fs from "node:fs";
|
|
import path from "node:path";
|
|
import { afterEach, describe, expect, it } from "vitest";
|
|
import { cleanupTrackedTempDirs, makeTrackedTempDir } from "./test-helpers/fs-fixtures.js";
|
|
|
|
type StageRuntimeDepsInstallParams = {
|
|
packageJson: Record<string, unknown>;
|
|
};
|
|
|
|
type StageBundledPluginRuntimeDeps = (params?: {
|
|
cwd?: string;
|
|
repoRoot?: string;
|
|
installAttempts?: number;
|
|
installPluginRuntimeDepsImpl?: (params: StageRuntimeDepsInstallParams) => void;
|
|
}) => void;
|
|
|
|
async function loadStageBundledPluginRuntimeDeps(): Promise<StageBundledPluginRuntimeDeps> {
|
|
const moduleUrl = new URL("../../scripts/stage-bundled-plugin-runtime-deps.mjs", import.meta.url);
|
|
const loaded = (await import(moduleUrl.href)) as {
|
|
stageBundledPluginRuntimeDeps: StageBundledPluginRuntimeDeps;
|
|
};
|
|
return loaded.stageBundledPluginRuntimeDeps;
|
|
}
|
|
|
|
async function loadPostinstallBundledPluginsModule(): Promise<{
|
|
applyBaileysEncryptedStreamFinishHotfix: (params?: {
|
|
chmodSync?: (path: string, mode: number) => void;
|
|
packageRoot?: string;
|
|
createTempPath?: (targetPath: string) => string;
|
|
writeFileSync?: (pathOrFd: string | number, value: string, encoding?: string) => void;
|
|
}) => {
|
|
applied: boolean;
|
|
reason: string;
|
|
targetPath?: string;
|
|
error?: string;
|
|
};
|
|
}> {
|
|
const moduleUrl = new URL("../../scripts/postinstall-bundled-plugins.mjs", import.meta.url);
|
|
return (await import(moduleUrl.href)) as {
|
|
applyBaileysEncryptedStreamFinishHotfix: (params?: {
|
|
chmodSync?: (path: string, mode: number) => void;
|
|
packageRoot?: string;
|
|
createTempPath?: (targetPath: string) => string;
|
|
writeFileSync?: (pathOrFd: string | number, value: string, encoding?: string) => void;
|
|
}) => {
|
|
applied: boolean;
|
|
reason: string;
|
|
targetPath?: string;
|
|
error?: string;
|
|
};
|
|
};
|
|
}
|
|
|
|
const tempDirs: string[] = [];
|
|
|
|
function makeRepoRoot(prefix: string): string {
|
|
return makeTrackedTempDir(prefix, tempDirs);
|
|
}
|
|
|
|
function writeRepoFile(repoRoot: string, relativePath: string, value: string) {
|
|
const fullPath = path.join(repoRoot, relativePath);
|
|
fs.mkdirSync(path.dirname(fullPath), { recursive: true });
|
|
fs.writeFileSync(fullPath, value, "utf8");
|
|
}
|
|
|
|
afterEach(() => {
|
|
cleanupTrackedTempDirs(tempDirs);
|
|
});
|
|
|
|
describe("stageBundledPluginRuntimeDeps", () => {
|
|
it("drops Lark SDK type cargo while keeping runtime entrypoints", () => {
|
|
const repoRoot = makeRepoRoot("openclaw-stage-bundled-runtime-deps-");
|
|
|
|
writeRepoFile(
|
|
repoRoot,
|
|
"dist/extensions/feishu/package.json",
|
|
JSON.stringify(
|
|
{
|
|
name: "@openclaw/feishu",
|
|
version: "2026.4.10",
|
|
dependencies: {
|
|
"@larksuiteoapi/node-sdk": "^1.60.0",
|
|
},
|
|
openclaw: {
|
|
bundle: {
|
|
stageRuntimeDependencies: true,
|
|
},
|
|
},
|
|
},
|
|
null,
|
|
2,
|
|
),
|
|
);
|
|
|
|
writeRepoFile(
|
|
repoRoot,
|
|
"node_modules/@larksuiteoapi/node-sdk/package.json",
|
|
JSON.stringify(
|
|
{
|
|
name: "@larksuiteoapi/node-sdk",
|
|
version: "1.60.0",
|
|
main: "./lib/index.js",
|
|
module: "./es/index.js",
|
|
types: "./types",
|
|
},
|
|
null,
|
|
2,
|
|
),
|
|
);
|
|
writeRepoFile(
|
|
repoRoot,
|
|
"node_modules/@larksuiteoapi/node-sdk/lib/index.js",
|
|
"export const runtime = true;\n",
|
|
);
|
|
writeRepoFile(
|
|
repoRoot,
|
|
"node_modules/@larksuiteoapi/node-sdk/es/index.js",
|
|
"export const moduleRuntime = true;\n",
|
|
);
|
|
writeRepoFile(
|
|
repoRoot,
|
|
"node_modules/@larksuiteoapi/node-sdk/types/index.d.ts",
|
|
"export interface HugeTypeSurface {}\n",
|
|
);
|
|
|
|
return loadStageBundledPluginRuntimeDeps().then((stageBundledPluginRuntimeDeps) => {
|
|
stageBundledPluginRuntimeDeps({ repoRoot });
|
|
|
|
const stagedRoot = path.join(
|
|
repoRoot,
|
|
"dist",
|
|
"extensions",
|
|
"feishu",
|
|
"node_modules",
|
|
"@larksuiteoapi",
|
|
"node-sdk",
|
|
);
|
|
expect(fs.existsSync(path.join(stagedRoot, "lib", "index.js"))).toBe(true);
|
|
expect(fs.existsSync(path.join(stagedRoot, "es", "index.js"))).toBe(true);
|
|
expect(fs.existsSync(path.join(stagedRoot, "types"))).toBe(false);
|
|
});
|
|
});
|
|
|
|
it("strips non-runtime dependency sections before temp npm staging", async () => {
|
|
const repoRoot = makeRepoRoot("openclaw-stage-bundled-runtime-manifest-");
|
|
writeRepoFile(
|
|
repoRoot,
|
|
"dist/extensions/amazon-bedrock/package.json",
|
|
JSON.stringify(
|
|
{
|
|
name: "@openclaw/amazon-bedrock-provider",
|
|
version: "2026.4.10",
|
|
dependencies: {
|
|
"@aws-sdk/client-bedrock": "3.1024.0",
|
|
},
|
|
devDependencies: {
|
|
"@openclaw/plugin-sdk": "workspace:*",
|
|
},
|
|
peerDependencies: {
|
|
openclaw: "^0.0.0",
|
|
},
|
|
peerDependenciesMeta: {
|
|
openclaw: {
|
|
optional: true,
|
|
},
|
|
},
|
|
openclaw: {
|
|
bundle: {
|
|
stageRuntimeDependencies: true,
|
|
},
|
|
},
|
|
},
|
|
null,
|
|
2,
|
|
),
|
|
);
|
|
|
|
const stageBundledPluginRuntimeDeps = await loadStageBundledPluginRuntimeDeps();
|
|
const installs: Array<Record<string, unknown>> = [];
|
|
stageBundledPluginRuntimeDeps({
|
|
repoRoot,
|
|
installAttempts: 1,
|
|
installPluginRuntimeDepsImpl(params: { packageJson: Record<string, unknown> }) {
|
|
installs.push(params.packageJson);
|
|
},
|
|
});
|
|
|
|
expect(installs).toHaveLength(1);
|
|
expect(installs[0]?.dependencies).toEqual({
|
|
"@aws-sdk/client-bedrock": "3.1024.0",
|
|
});
|
|
expect(installs[0]?.devDependencies).toBeUndefined();
|
|
expect(installs[0]?.peerDependencies).toBeUndefined();
|
|
expect(installs[0]?.peerDependenciesMeta).toBeUndefined();
|
|
});
|
|
|
|
it("patches installed Baileys encryptedStream flush ordering for shipped runtime deps", async () => {
|
|
const repoRoot = makeRepoRoot("openclaw-stage-bundled-runtime-hotfix-");
|
|
const targetPath = path.join(
|
|
repoRoot,
|
|
"node_modules",
|
|
"@whiskeysockets",
|
|
"baileys",
|
|
"lib",
|
|
"Utils",
|
|
"messages-media.js",
|
|
);
|
|
writeRepoFile(
|
|
repoRoot,
|
|
"node_modules/@whiskeysockets/baileys/lib/Utils/messages-media.js",
|
|
[
|
|
"import { once } from 'events';",
|
|
"const encryptedStream = async () => {",
|
|
" encFileWriteStream.write(mac);",
|
|
" encFileWriteStream.end();",
|
|
" originalFileStream?.end?.();",
|
|
" stream.destroy();",
|
|
" logger?.debug('encrypted data successfully');",
|
|
"};",
|
|
].join("\n"),
|
|
);
|
|
|
|
const { applyBaileysEncryptedStreamFinishHotfix } = await loadPostinstallBundledPluginsModule();
|
|
const result = applyBaileysEncryptedStreamFinishHotfix({ packageRoot: repoRoot });
|
|
|
|
expect(result).toEqual({
|
|
applied: true,
|
|
reason: "patched",
|
|
targetPath,
|
|
});
|
|
expect(fs.readFileSync(targetPath, "utf8")).toContain(
|
|
"const encFinishPromise = once(encFileWriteStream, 'finish');",
|
|
);
|
|
expect(fs.readFileSync(targetPath, "utf8")).toContain(
|
|
"await Promise.all([encFinishPromise, originalFinishPromise]);",
|
|
);
|
|
});
|
|
|
|
it("preserves the original module read mode when replacing Baileys", async () => {
|
|
const repoRoot = makeRepoRoot("openclaw-stage-bundled-runtime-hotfix-mode-");
|
|
const targetPath = path.join(
|
|
repoRoot,
|
|
"node_modules",
|
|
"@whiskeysockets",
|
|
"baileys",
|
|
"lib",
|
|
"Utils",
|
|
"messages-media.js",
|
|
);
|
|
writeRepoFile(
|
|
repoRoot,
|
|
"node_modules/@whiskeysockets/baileys/lib/Utils/messages-media.js",
|
|
[
|
|
"import { once } from 'events';",
|
|
"const encryptedStream = async () => {",
|
|
" encFileWriteStream.write(mac);",
|
|
" encFileWriteStream.end();",
|
|
" originalFileStream?.end?.();",
|
|
" stream.destroy();",
|
|
" logger?.debug('encrypted data successfully');",
|
|
"};",
|
|
].join("\n"),
|
|
);
|
|
fs.chmodSync(targetPath, 0o644);
|
|
|
|
const { applyBaileysEncryptedStreamFinishHotfix } = await loadPostinstallBundledPluginsModule();
|
|
const result = applyBaileysEncryptedStreamFinishHotfix({ packageRoot: repoRoot });
|
|
|
|
expect(result).toEqual({
|
|
applied: true,
|
|
reason: "patched",
|
|
targetPath,
|
|
});
|
|
expect(fs.statSync(targetPath).mode & 0o777).toBe(0o644);
|
|
});
|
|
|
|
it("refuses symlink targets for the Baileys hotfix", async () => {
|
|
const repoRoot = makeRepoRoot("openclaw-stage-bundled-runtime-hotfix-symlink-");
|
|
const targetPath = path.join(
|
|
repoRoot,
|
|
"node_modules",
|
|
"@whiskeysockets",
|
|
"baileys",
|
|
"lib",
|
|
"Utils",
|
|
"messages-media.js",
|
|
);
|
|
const redirectedTarget = path.join(repoRoot, "redirected-messages-media.js");
|
|
writeRepoFile(repoRoot, "redirected-messages-media.js", "const untouched = true;\n");
|
|
fs.mkdirSync(path.dirname(targetPath), { recursive: true });
|
|
fs.symlinkSync(redirectedTarget, targetPath);
|
|
|
|
const { applyBaileysEncryptedStreamFinishHotfix } = await loadPostinstallBundledPluginsModule();
|
|
const result = applyBaileysEncryptedStreamFinishHotfix({ packageRoot: repoRoot });
|
|
|
|
expect(result).toEqual({
|
|
applied: false,
|
|
reason: "unsafe_target",
|
|
targetPath,
|
|
});
|
|
expect(fs.readFileSync(redirectedTarget, "utf8")).toBe("const untouched = true;\n");
|
|
});
|
|
|
|
it("downgrades Baileys hotfix write failures to a non-fatal result", async () => {
|
|
const repoRoot = makeRepoRoot("openclaw-stage-bundled-runtime-hotfix-write-failure-");
|
|
const targetPath = path.join(
|
|
repoRoot,
|
|
"node_modules",
|
|
"@whiskeysockets",
|
|
"baileys",
|
|
"lib",
|
|
"Utils",
|
|
"messages-media.js",
|
|
);
|
|
writeRepoFile(
|
|
repoRoot,
|
|
"node_modules/@whiskeysockets/baileys/lib/Utils/messages-media.js",
|
|
[
|
|
"import { once } from 'events';",
|
|
"const encryptedStream = async () => {",
|
|
" encFileWriteStream.write(mac);",
|
|
" encFileWriteStream.end();",
|
|
" originalFileStream?.end?.();",
|
|
" stream.destroy();",
|
|
" logger?.debug('encrypted data successfully');",
|
|
"};",
|
|
].join("\n"),
|
|
);
|
|
|
|
const { applyBaileysEncryptedStreamFinishHotfix } = await loadPostinstallBundledPluginsModule();
|
|
const result = applyBaileysEncryptedStreamFinishHotfix({
|
|
packageRoot: repoRoot,
|
|
writeFileSync() {
|
|
throw new Error("read-only filesystem");
|
|
},
|
|
});
|
|
|
|
expect(result).toEqual({
|
|
applied: false,
|
|
reason: "error",
|
|
targetPath,
|
|
error: "read-only filesystem",
|
|
});
|
|
expect(fs.readFileSync(targetPath, "utf8")).toContain("encFileWriteStream.end();");
|
|
});
|
|
|
|
it("refuses pre-created symlink temp paths instead of following them", async () => {
|
|
const repoRoot = makeRepoRoot("openclaw-stage-bundled-runtime-hotfix-temp-symlink-");
|
|
const targetPath = path.join(
|
|
repoRoot,
|
|
"node_modules",
|
|
"@whiskeysockets",
|
|
"baileys",
|
|
"lib",
|
|
"Utils",
|
|
"messages-media.js",
|
|
);
|
|
const redirectedTarget = path.join(repoRoot, "redirected-temp-target.js");
|
|
const attackerTempPath = path.join(
|
|
path.dirname(targetPath),
|
|
".messages-media.js.attacker-temp",
|
|
);
|
|
writeRepoFile(
|
|
repoRoot,
|
|
"node_modules/@whiskeysockets/baileys/lib/Utils/messages-media.js",
|
|
[
|
|
"import { once } from 'events';",
|
|
"const encryptedStream = async () => {",
|
|
" encFileWriteStream.write(mac);",
|
|
" encFileWriteStream.end();",
|
|
" originalFileStream?.end?.();",
|
|
" stream.destroy();",
|
|
" logger?.debug('encrypted data successfully');",
|
|
"};",
|
|
].join("\n"),
|
|
);
|
|
writeRepoFile(repoRoot, "redirected-temp-target.js", "const untouched = true;\n");
|
|
fs.symlinkSync(redirectedTarget, attackerTempPath);
|
|
|
|
const { applyBaileysEncryptedStreamFinishHotfix } = await loadPostinstallBundledPluginsModule();
|
|
const result = applyBaileysEncryptedStreamFinishHotfix({
|
|
packageRoot: repoRoot,
|
|
createTempPath() {
|
|
return attackerTempPath;
|
|
},
|
|
});
|
|
|
|
expect(result.applied).toBe(false);
|
|
expect(result.reason).toBe("error");
|
|
expect(result.error).toContain("EEXIST");
|
|
expect(fs.readFileSync(redirectedTarget, "utf8")).toBe("const untouched = true;\n");
|
|
expect(fs.readFileSync(targetPath, "utf8")).toContain("encFileWriteStream.end();");
|
|
});
|
|
});
|