Files
openclaw/test/scripts/plugin-sdk-surface-report.test.ts
Peter Steinberger 7fbe426cda test(core,plugins,scripts): remove low-value tests (batch d023) (#158911)
* test(media): deslop s155 tests

* test(googlechat): deslop s156 tests

* test(scripts): deslop s140 tests

* test(reef): deslop s158 tests

* test(infra): deslop s153 tests

* test(microsoft-foundry): deslop s163 tests

* test(ai): deslop s164 tests

* test(sessions): deslop s160 tests

* test(browser): deslop s165 tests

* test(config): deslop s162 tests

* test: preserve redaction and current release identity coverage

* test: retain helpers used by current main regressions

* test(googlechat): use current config type owner

* test(googlechat): remove unused config type import
2026-09-26 15:29:21 +00:00

195 lines
6.5 KiB
TypeScript

// Plugin Sdk Surface Report tests cover plugin sdk surface report script behavior.
import { spawnSync } from "node:child_process";
import fs from "node:fs";
import { beforeAll, describe, expect, it } from "vitest";
import {
collectPluginSdkSurfaceReport,
evaluatePluginSdkSurfaceReport,
readPluginSdkSurfaceBudgets,
} from "../../scripts/plugin-sdk-surface-report.mts";
const pluginSdkSurfaceBudgetEnvPattern = /^OPENCLAW_PLUGIN_SDK_MAX_/u;
function baseSurfaceReportEnv(): NodeJS.ProcessEnv {
return Object.fromEntries(
Object.entries(process.env).filter(([key]) => !pluginSdkSurfaceBudgetEnvPattern.test(key)),
);
}
function runSurfaceReport(env: Record<string, string>) {
return spawnSync(
process.execPath,
["--import", "tsx", "scripts/plugin-sdk-surface-report.mts", "--check"],
{
cwd: process.cwd(),
encoding: "utf8",
env: {
...baseSurfaceReportEnv(),
...env,
},
},
);
}
type PublicSurfaceCounts = {
callableExports: number;
exports: number;
wildcardReexports: number;
};
function readDefaultPublicSurfaceBudgets(): PublicSurfaceCounts {
const { budgets } = readPluginSdkSurfaceBudgets({});
return {
exports: budgets.publicExports,
callableExports: budgets.publicFunctionExports,
wildcardReexports: budgets.publicWildcardReexports,
};
}
type SurfaceReport = ReturnType<typeof collectPluginSdkSurfaceReport>;
let surfaceReport: SurfaceReport;
function readCurrentPublicSurfaceCounts(): PublicSurfaceCounts {
return {
exports: surfaceReport.publicStats.totals.exports,
callableExports: surfaceReport.publicStats.totals.callableExports,
wildcardReexports: surfaceReport.publicWildcards.count,
};
}
describe("plugin SDK surface report", () => {
beforeAll(() => {
surfaceReport = collectPluginSdkSurfaceReport();
});
it("rejects unknown CLI options before collecting SDK stats", () => {
for (const args of [["--chekc"], ["chekc", "--help"]]) {
const result = spawnSync(
process.execPath,
["--import", "tsx", "scripts/plugin-sdk-surface-report.mts", ...args],
{
cwd: process.cwd(),
encoding: "utf8",
},
);
expect(result.status).toBe(1);
expect(result.stdout).toBe("");
expect(result.stderr.trim()).toBe(`Unknown plugin SDK surface report option: ${args[0]}`);
expect(result.stderr).not.toContain("at ");
}
});
it("prints help before collecting SDK stats", () => {
const result = spawnSync(
process.execPath,
["--import", "tsx", "scripts/plugin-sdk-surface-report.mts", "--help"],
{
cwd: process.cwd(),
encoding: "utf8",
},
);
expect(result.status).toBe(0);
expect(result.stdout).toContain(
"Usage: node --import tsx scripts/plugin-sdk-surface-report.mts",
);
expect(result.stderr).toBe("");
expect(result.stdout).not.toContain("all SDK entrypoints:");
});
it("rejects loose numeric budget env vars before collecting SDK stats", () => {
const result = runSurfaceReport({
OPENCLAW_PLUGIN_SDK_MAX_PUBLIC_EXPORTS: "1e9",
});
expect(result.status).toBe(1);
expect(result.stdout).toBe("");
expect(result.stderr).toContain(
"OPENCLAW_PLUGIN_SDK_MAX_PUBLIC_EXPORTS must be a non-negative integer",
);
expect(result.stderr).not.toContain("at ");
});
it("rejects unsafe budget env vars before collecting SDK stats", () => {
const result = runSurfaceReport({
OPENCLAW_PLUGIN_SDK_MAX_PUBLIC_EXPORTS: "9007199254740992",
});
expect(result.status).toBe(1);
expect(result.stdout).toBe("");
expect(result.stderr).toContain(
"OPENCLAW_PLUGIN_SDK_MAX_PUBLIC_EXPORTS must be a safe non-negative integer",
);
expect(result.stderr).not.toContain("at ");
});
it("accepts exact deprecated export budget overrides by public entrypoint", () => {
const budgetConfig = readPluginSdkSurfaceBudgets({
OPENCLAW_PLUGIN_SDK_MAX_PUBLIC_DEPRECATED_EXPORTS_BY_ENTRYPOINT: JSON.stringify({ core: 3 }),
});
expect(evaluatePluginSdkSurfaceReport(surfaceReport, budgetConfig)).not.toContain(
expect.stringContaining("public deprecated exports in core"),
);
});
it("keeps default public surface budgets pinned to current source counts", () => {
expect(readDefaultPublicSurfaceBudgets()).toEqual(readCurrentPublicSurfaceCounts());
const channelMessage = surfaceReport.publicStats.byEntrypoint.get("channel-message");
expect(channelMessage).toBeDefined();
expect(
readPluginSdkSurfaceBudgets({}).publicDeprecatedExportsByEntrypointBudget["channel-message"],
).toBe(channelMessage?.deprecatedExports);
});
it("accepts frozen named facades while rejecting missing deprecated reexports", () => {
expect(surfaceReport.deprecatedBarrelWithoutReexports).toEqual([]);
const report = {
...surfaceReport,
deprecatedBarrelWithoutReexports: ["channel-message"],
};
expect(evaluatePluginSdkSurfaceReport(report, readPluginSdkSurfaceBudgets({}))).toContain(
"deprecated barrel entrypoints without reexports: channel-message",
);
});
it("keeps approval store internals out of the deprecated infra barrel", () => {
const source = fs.readFileSync("src/plugin-sdk/infra-runtime.ts", "utf8");
expect(source).not.toMatch(/export\s+(?:type\s+)?\*\s+from\s+["'][^"']*exec-approvals/u);
for (const internalName of [
"ensureExecApprovalsSnapshot",
"persistAllowAlwaysDecisionLocked",
"recordAllowlistMatchesUseLocked",
"resolveExecApprovalsLocked",
"restoreExecApprovalsSnapshotLocked",
"updateExecApprovals",
]) {
expect(source).not.toContain(internalName);
}
});
it("rejects callable surface growth from the canonical source graph", () => {
const budget = readDefaultPublicSurfaceBudgets().callableExports;
const budgetConfig = readPluginSdkSurfaceBudgets({
OPENCLAW_PLUGIN_SDK_MAX_PUBLIC_FUNCTION_EXPORTS: String(budget - 1),
});
expect(evaluatePluginSdkSurfaceReport(surfaceReport, budgetConfig)).toContain(
`public callable exports ${budget} > ${budget - 1}`,
);
});
it("rejects deprecated export growth by public entrypoint", () => {
const budgetConfig = readPluginSdkSurfaceBudgets({
OPENCLAW_PLUGIN_SDK_MAX_PUBLIC_DEPRECATED_EXPORTS_BY_ENTRYPOINT: JSON.stringify({ core: 1 }),
});
expect(evaluatePluginSdkSurfaceReport(surfaceReport, budgetConfig)).toContain(
"public deprecated exports in core 3 > 1",
);
});
});