Some checks failed
ClawSweeper Dispatch / dispatch (push) Has been cancelled
CodeQL / Security High (actions) (push) Has been cancelled
CodeQL / Security High (channel-runtime-boundary) (push) Has been cancelled
CodeQL / Security High (core-auth-secrets) (push) Has been cancelled
CodeQL / Security High (mcp-process-tool-boundary) (push) Has been cancelled
CodeQL / Security High (network-ssrf-boundary) (push) Has been cancelled
CodeQL / Security High (plugin-trust-boundary) (push) Has been cancelled
CodeQL / Security High (process-exec-boundary) (push) Has been cancelled
Docs Sync Publish Repo / sync-publish-repo (push) Has been cancelled
Docs / docs (push) Has been cancelled
OpenClaw Stable Main Closeout / Resolve stable release closeout inputs (push) Has been cancelled
OpenClaw Stable Main Closeout / Verify stable main closeout (push) Has been cancelled
Workflow Sanity / no-tabs (push) Has been cancelled
Workflow Sanity / actionlint (push) Has been cancelled
Workflow Sanity / generated-doc-baselines (push) Has been cancelled
CI / runner-admission (push) Has been cancelled
CI / preflight (push) Has been cancelled
CI / security-fast (push) Has been cancelled
CI / pnpm-store-warmup (push) Has been cancelled
CI / build-artifacts (push) Has been cancelled
CI / native-i18n (push) Has been cancelled
CI / ${{ matrix.check_name }} (push) Has been cancelled
CI / ${{ matrix.checkName }} (push) Has been cancelled
CI / checks-node-compat-node22 (push) Has been cancelled
CI / check-bundled-channel-config-metadata (push) Has been cancelled
CI / check-dependencies (push) Has been cancelled
CI / check-guards (push) Has been cancelled
CI / check-lint (push) Has been cancelled
CI / check-prod-types (push) Has been cancelled
CI / check-shrinkwrap (push) Has been cancelled
CI / check-test-types (push) Has been cancelled
CI / check-additional-boundaries-a (push) Has been cancelled
CI / check-additional-boundaries-bcd (push) Has been cancelled
CI / check-additional-extension-bundled (push) Has been cancelled
CI / check-additional-extension-channels (push) Has been cancelled
CI / check-additional-extension-package-boundary (push) Has been cancelled
CI / check-additional-runtime-topology-architecture (push) Has been cancelled
CI / check-session-accessor-boundary (push) Has been cancelled
CI / check-session-transcript-reader-boundary (push) Has been cancelled
CI / check-docs (push) Has been cancelled
CI / skills-python (push) Has been cancelled
CI / macos-swift (push) Has been cancelled
CI / ios-build (push) Has been cancelled
CI / ci-timings-summary (push) Has been cancelled
Native App Locale Refresh / Refresh native fa (push) Has been cancelled
Native App Locale Refresh / Refresh native fr (push) Has been cancelled
Native App Locale Refresh / Refresh native hi (push) Has been cancelled
Native App Locale Refresh / Refresh native id (push) Has been cancelled
Native App Locale Refresh / Refresh native it (push) Has been cancelled
Native App Locale Refresh / Refresh native ja-JP (push) Has been cancelled
Control UI Locale Refresh / plan (push) Has been cancelled
Control UI Locale Refresh / Refresh ${{ matrix.locale }} (push) Has been cancelled
Control UI Locale Refresh / Commit control UI locale refresh (push) Has been cancelled
Live Media Runner Image / Build live media runner image (push) Has been cancelled
Native App Locale Refresh / Refresh native ar (push) Has been cancelled
Native App Locale Refresh / Refresh native de (push) Has been cancelled
Native App Locale Refresh / Refresh native es (push) Has been cancelled
Native App Locale Refresh / Refresh native ko (push) Has been cancelled
Native App Locale Refresh / Refresh native nl (push) Has been cancelled
Native App Locale Refresh / Refresh native pl (push) Has been cancelled
Native App Locale Refresh / Refresh native pt-BR (push) Has been cancelled
Native App Locale Refresh / Refresh native ru (push) Has been cancelled
Native App Locale Refresh / Refresh native sv (push) Has been cancelled
Native App Locale Refresh / Refresh native th (push) Has been cancelled
Native App Locale Refresh / Refresh native tr (push) Has been cancelled
Native App Locale Refresh / Refresh native uk (push) Has been cancelled
Native App Locale Refresh / Refresh native vi (push) Has been cancelled
Native App Locale Refresh / Refresh native zh-CN (push) Has been cancelled
Native App Locale Refresh / Refresh native zh-TW (push) Has been cancelled
Native App Locale Refresh / Commit native locale refresh (push) Has been cancelled
Plugin Init Scaffold Validation / Validate provider scaffold (push) Has been cancelled
Plugin NPM Release / preview_plugins_npm (push) Has been cancelled
Plugin NPM Release / Validate release publish approval (push) Has been cancelled
Plugin NPM Release / preview_plugin_pack (push) Has been cancelled
Plugin NPM Release / publish_plugins_npm (push) Has been cancelled
Sandbox Common Smoke / sandbox-common-smoke (push) Has been cancelled
Website Installer Sync / static (push) Has been cancelled
Website Installer Sync / linux-docker (push) Has been cancelled
Website Installer Sync / macos-installer (push) Has been cancelled
Website Installer Sync / windows-installer (push) Has been cancelled
Website Installer Sync / sync-website (push) Has been cancelled
Adolf is a fork/vendored clone of github.com/openclaw/openclaw (v2026.6.11), free to diverge. Tree copied sans upstream .git; upstream remote added for future syncs. Node pinned to 24 (.nvmrc); engines already require >=22.19. Preserves docs/ARCHITECTURE.md. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01LeqyaxJF2nbRXJtae2kNB2
241 lines
7.7 KiB
TypeScript
241 lines
7.7 KiB
TypeScript
// Plugin Lifecycle Probe tests cover QA Lab plugin lifecycle evidence.
|
|
import { EventEmitter } from "node:events";
|
|
import { existsSync, mkdirSync, readFileSync, writeFileSync } from "node:fs";
|
|
import path from "node:path";
|
|
import { afterEach, describe, expect, it, vi } from "vitest";
|
|
import { resolveWindowsTaskkillPath } from "../../../../scripts/lib/windows-taskkill.mjs";
|
|
import { createTempDirTracker } from "../../../helpers/temp-dir.js";
|
|
import {
|
|
assertInspectLoaded,
|
|
assertUninstalled,
|
|
parseDurationMs,
|
|
testing as probeTesting,
|
|
} from "./plugin-lifecycle-probe-runtime.js";
|
|
|
|
const tempDirs = createTempDirTracker();
|
|
|
|
function expectedTaskkillPath(): string {
|
|
return resolveWindowsTaskkillPath();
|
|
}
|
|
|
|
function makeTempDir(): string {
|
|
return tempDirs.make("openclaw-plugin-lifecycle-probe-");
|
|
}
|
|
|
|
function isProcessRunning(pid: number): boolean {
|
|
try {
|
|
process.kill(pid, 0);
|
|
return true;
|
|
} catch {
|
|
return false;
|
|
}
|
|
}
|
|
|
|
async function sleep(ms: number): Promise<void> {
|
|
await new Promise((resolve) => {
|
|
setTimeout(resolve, ms);
|
|
});
|
|
}
|
|
|
|
async function waitForFile(pathToCheck: string, timeoutMs: number): Promise<void> {
|
|
const deadline = Date.now() + timeoutMs;
|
|
while (Date.now() < deadline) {
|
|
if (existsSync(pathToCheck)) {
|
|
return;
|
|
}
|
|
await sleep(25);
|
|
}
|
|
throw new Error(`Timed out waiting for ${pathToCheck}`);
|
|
}
|
|
|
|
class FakeCommandChild extends EventEmitter {
|
|
readonly signals: string[] = [];
|
|
|
|
kill(signal?: NodeJS.Signals | number): boolean {
|
|
this.signals.push(String(signal));
|
|
if (signal === "SIGTERM") {
|
|
queueMicrotask(() => this.emit("exit", 0, null));
|
|
}
|
|
return true;
|
|
}
|
|
}
|
|
|
|
afterEach(tempDirs.cleanup);
|
|
|
|
describe("plugin lifecycle matrix probe", () => {
|
|
it("accepts inspect JSON for an enabled loaded plugin", async () => {
|
|
const dir = makeTempDir();
|
|
const inspectPath = path.join(dir, "inspect.json");
|
|
writeFileSync(
|
|
inspectPath,
|
|
`${JSON.stringify({ plugin: { enabled: true, id: "lifecycle-claw", status: "loaded" } })}\n`,
|
|
"utf8",
|
|
);
|
|
|
|
expect(() => assertInspectLoaded("lifecycle-claw", inspectPath)).not.toThrow();
|
|
});
|
|
|
|
it("rejects inspect JSON that does not prove the runtime loaded", async () => {
|
|
const dir = makeTempDir();
|
|
const inspectPath = path.join(dir, "inspect.json");
|
|
writeFileSync(
|
|
inspectPath,
|
|
`${JSON.stringify({ plugin: { enabled: true, id: "lifecycle-claw", status: "pending" } })}\n`,
|
|
"utf8",
|
|
);
|
|
|
|
expect(() => assertInspectLoaded("lifecycle-claw", inspectPath)).toThrow(
|
|
"expected lifecycle-claw inspect status loaded, got pending",
|
|
);
|
|
});
|
|
|
|
it("rejects missing inspect JSON instead of treating it as an empty object", async () => {
|
|
const dir = makeTempDir();
|
|
const inspectPath = path.join(dir, "missing.json");
|
|
|
|
expect(() => assertInspectLoaded("lifecycle-claw", inspectPath)).toThrow(
|
|
`failed to read JSON from ${inspectPath}`,
|
|
);
|
|
});
|
|
|
|
it("rejects unreadable config during uninstall proof", async () => {
|
|
const dir = makeTempDir();
|
|
const configFile = path.join(dir, ".openclaw", "openclaw.json");
|
|
mkdirSync(path.dirname(configFile), { recursive: true });
|
|
writeFileSync(configFile, "{ malformed\n", "utf8");
|
|
|
|
expect(() =>
|
|
assertUninstalled("lifecycle-claw", {
|
|
HOME: dir,
|
|
OPENCLAW_CONFIG_PATH: configFile,
|
|
}),
|
|
).toThrow(`failed to read JSON from ${configFile}`);
|
|
});
|
|
|
|
it("preserves disabled npm install timeout semantics", () => {
|
|
expect(parseDurationMs("0", "600s")).toBeUndefined();
|
|
});
|
|
|
|
it("rejects timed commands that exit cleanly during kill grace", async () => {
|
|
vi.useFakeTimers();
|
|
try {
|
|
const child = new FakeCommandChild();
|
|
const runPromise = probeTesting.runCommand("fake-command", ["install"], {
|
|
spawnImpl: (() => child) as unknown as typeof import("node:child_process").spawn,
|
|
timeoutKillGraceMs: 100,
|
|
timeoutMs: 10,
|
|
});
|
|
const runError = runPromise.catch((error: unknown) => error);
|
|
|
|
await vi.advanceTimersByTimeAsync(10);
|
|
|
|
const error = await runError;
|
|
expect(error).toBeInstanceOf(Error);
|
|
expect((error as Error).message).toBe("fake-command install timed out after 10ms");
|
|
expect(child.signals).toEqual(["SIGTERM"]);
|
|
|
|
await vi.advanceTimersByTimeAsync(100);
|
|
expect(child.signals).toEqual(["SIGTERM"]);
|
|
} finally {
|
|
vi.useRealTimers();
|
|
}
|
|
});
|
|
|
|
it("force-kills timed Windows commands with taskkill when graceful taskkill fails", async () => {
|
|
vi.useFakeTimers();
|
|
const platformDescriptor = Object.getOwnPropertyDescriptor(process, "platform");
|
|
Object.defineProperty(process, "platform", { value: "win32", configurable: true });
|
|
try {
|
|
const child = Object.assign(new FakeCommandChild(), { pid: 12345 });
|
|
const taskkillImpl = vi
|
|
.fn()
|
|
.mockReturnValueOnce({ status: 1 })
|
|
.mockImplementationOnce(() => {
|
|
queueMicrotask(() => child.emit("exit", null, "SIGTERM"));
|
|
return { status: 0 };
|
|
});
|
|
const runPromise = probeTesting.runCommand("fake-command", ["install"], {
|
|
spawnImpl: (() => child) as unknown as typeof import("node:child_process").spawn,
|
|
taskkillImpl,
|
|
timeoutKillGraceMs: 100,
|
|
timeoutMs: 10,
|
|
});
|
|
const runError = runPromise.catch((error: unknown) => error);
|
|
|
|
await vi.advanceTimersByTimeAsync(10);
|
|
|
|
expect(taskkillImpl).toHaveBeenNthCalledWith(
|
|
1,
|
|
expectedTaskkillPath(),
|
|
["/PID", "12345", "/T"],
|
|
{
|
|
stdio: "ignore",
|
|
windowsHide: true,
|
|
},
|
|
);
|
|
expect(taskkillImpl).toHaveBeenNthCalledWith(
|
|
2,
|
|
expectedTaskkillPath(),
|
|
["/PID", "12345", "/T", "/F"],
|
|
{
|
|
stdio: "ignore",
|
|
windowsHide: true,
|
|
},
|
|
);
|
|
expect(child.signals).toEqual([]);
|
|
|
|
const error = await runError;
|
|
expect(error).toBeInstanceOf(Error);
|
|
expect((error as Error).message).toBe("fake-command install timed out after 10ms");
|
|
} finally {
|
|
if (platformDescriptor) {
|
|
Object.defineProperty(process, "platform", platformDescriptor);
|
|
}
|
|
vi.useRealTimers();
|
|
}
|
|
});
|
|
|
|
it("keeps fallback SIGKILL armed for ignored-stdio descendants", async () => {
|
|
if (process.platform === "win32") {
|
|
return;
|
|
}
|
|
|
|
const dir = makeTempDir();
|
|
const descendantPidPath = path.join(dir, "descendant.pid");
|
|
let descendantPid: number | undefined;
|
|
try {
|
|
const childScript = "process.on('SIGTERM', () => {}); setInterval(() => {}, 1000);";
|
|
const parentScript = [
|
|
"import { spawn } from 'node:child_process';",
|
|
"import { writeFileSync } from 'node:fs';",
|
|
`const child = spawn(process.execPath, ['-e', ${JSON.stringify(childScript)}], { stdio: 'ignore' });`,
|
|
"child.unref();",
|
|
"writeFileSync(process.env.OPENCLAW_TEST_DESCENDANT_PID, String(child.pid));",
|
|
"process.on('SIGTERM', () => process.exit(0));",
|
|
"setInterval(() => {}, 1000);",
|
|
].join("\n");
|
|
|
|
const run = probeTesting.runCommand(
|
|
process.execPath,
|
|
["--input-type=module", "-e", parentScript],
|
|
{
|
|
env: { ...process.env, OPENCLAW_TEST_DESCENDANT_PID: descendantPidPath },
|
|
timeoutKillGraceMs: 250,
|
|
timeoutMs: 500,
|
|
},
|
|
);
|
|
await waitForFile(descendantPidPath, 2_000);
|
|
await sleep(300);
|
|
|
|
await expect(run).rejects.toThrow(/timed out after 500ms/u);
|
|
|
|
descendantPid = Number(readFileSync(descendantPidPath, "utf8"));
|
|
expect(isProcessRunning(descendantPid)).toBe(false);
|
|
} finally {
|
|
if (descendantPid && isProcessRunning(descendantPid)) {
|
|
process.kill(descendantPid, "SIGKILL");
|
|
}
|
|
}
|
|
});
|
|
});
|