mirror of
https://github.com/openclaw/openclaw.git
synced 2026-08-15 23:24:03 -06:00
7e42dae611
* refactor(memory-core): split index manager along the inheritance chain * fix(memory): restore the manager lazy boundary * refactor(memory-core): delete dead manager plumbing * chore(lint): ratchet max-lines baseline after memory manager split * fix(memory): keep the manager out of the runtime facade * test(memory): cover transitive manager facade imports
168 lines
5.6 KiB
TypeScript
168 lines
5.6 KiB
TypeScript
// Check Runtime Sidecar Loaders tests cover check runtime sidecar loaders script behavior.
|
|
import { existsSync, readFileSync } from "node:fs";
|
|
import { dirname, relative, resolve } from "node:path";
|
|
import ts from "typescript";
|
|
import { describe, expect, it } from "vitest";
|
|
import {
|
|
collectTsdownEntrySources,
|
|
findRuntimeSidecarLoaderViolations,
|
|
} from "../../scripts/check-runtime-sidecar-loaders.mts";
|
|
|
|
function listRuntimeStaticSpecifiers(sourcePath: string): string[] {
|
|
const source = readFileSync(sourcePath, "utf8");
|
|
const sourceFile = ts.createSourceFile(sourcePath, source, ts.ScriptTarget.Latest, true);
|
|
return sourceFile.statements.flatMap((statement) => {
|
|
if (
|
|
ts.isImportDeclaration(statement) &&
|
|
ts.isStringLiteral(statement.moduleSpecifier) &&
|
|
!statement.importClause?.isTypeOnly
|
|
) {
|
|
return [statement.moduleSpecifier.text];
|
|
}
|
|
if (
|
|
ts.isExportDeclaration(statement) &&
|
|
statement.moduleSpecifier &&
|
|
ts.isStringLiteral(statement.moduleSpecifier) &&
|
|
!statement.isTypeOnly &&
|
|
!(
|
|
statement.exportClause &&
|
|
ts.isNamedExports(statement.exportClause) &&
|
|
statement.exportClause.elements.every((element) => element.isTypeOnly)
|
|
)
|
|
) {
|
|
return [statement.moduleSpecifier.text];
|
|
}
|
|
return [];
|
|
});
|
|
}
|
|
|
|
function resolveLocalSource(importerPath: string, specifier: string): string | null {
|
|
if (!specifier.startsWith(".")) {
|
|
return null;
|
|
}
|
|
const resolved = resolve(dirname(importerPath), specifier);
|
|
const candidates = [resolved, resolved.replace(/\.js$/, ".ts"), resolve(resolved, "index.ts")];
|
|
return candidates.find((candidate) => existsSync(candidate)) ?? null;
|
|
}
|
|
|
|
function collectRuntimeStaticGraph(entryPath: string): Set<string> {
|
|
const pending = [entryPath];
|
|
const visited = new Set<string>();
|
|
for (const sourcePath of pending) {
|
|
if (visited.has(sourcePath)) {
|
|
continue;
|
|
}
|
|
visited.add(sourcePath);
|
|
for (const specifier of listRuntimeStaticSpecifiers(sourcePath)) {
|
|
const resolved = resolveLocalSource(sourcePath, specifier);
|
|
if (resolved && !visited.has(resolved)) {
|
|
pending.push(resolved);
|
|
}
|
|
}
|
|
}
|
|
return visited;
|
|
}
|
|
|
|
describe("check-runtime-sidecar-loaders", () => {
|
|
it("keeps the memory runtime facade out of the manager sidecar graph", () => {
|
|
const sourcePath = new URL("../../extensions/memory-core/runtime-api.ts", import.meta.url);
|
|
const runtimeGraph = [...collectRuntimeStaticGraph(sourcePath.pathname)].map((filePath) =>
|
|
relative(resolve(dirname(sourcePath.pathname), "../.."), filePath),
|
|
);
|
|
|
|
expect(runtimeGraph.filter((filePath) => /(^|\/)manager(?:-|\.)/.test(filePath))).toEqual([]);
|
|
});
|
|
|
|
it("flags hidden createRequire runtime sidecars that are not build entries", () => {
|
|
const source = `
|
|
import { createRequire } from "node:module";
|
|
const require = createRequire(import.meta.url);
|
|
export function loadRuntime() {
|
|
return require("./missing.runtime.js");
|
|
}
|
|
`;
|
|
|
|
expect(
|
|
findRuntimeSidecarLoaderViolations(source, "src/tasks/task-registry.ts", new Set()),
|
|
).toEqual([
|
|
{
|
|
line: 5,
|
|
specifier: "./missing.runtime.js",
|
|
sourcePath: "src/tasks/missing.runtime.ts",
|
|
reason:
|
|
'hidden local runtime loader "./missing.runtime.js" resolves to src/tasks/missing.runtime.ts, but that source is not an explicit tsdown entry',
|
|
},
|
|
]);
|
|
});
|
|
|
|
it("allows hidden createRequire runtime sidecars when the source is an explicit build entry", () => {
|
|
const source = `
|
|
import { createRequire } from "node:module";
|
|
const require = createRequire(import.meta.url);
|
|
export function loadRuntime() {
|
|
return require("./task-registry-control.runtime.js");
|
|
}
|
|
`;
|
|
|
|
expect(
|
|
findRuntimeSidecarLoaderViolations(
|
|
source,
|
|
"src/tasks/task-registry.ts",
|
|
new Set(["src/tasks/task-registry-control.runtime.ts"]),
|
|
),
|
|
).toStrictEqual([]);
|
|
});
|
|
|
|
it("resolves candidate arrays used by source/build fallback loops", () => {
|
|
const source = `
|
|
import { createRequire } from "node:module";
|
|
const require = createRequire(import.meta.url);
|
|
const CANDIDATES = ["./control.runtime.js", "./control.runtime.ts"] as const;
|
|
export function loadRuntime() {
|
|
for (const candidate of CANDIDATES) {
|
|
return require(candidate);
|
|
}
|
|
}
|
|
`;
|
|
|
|
expect(
|
|
findRuntimeSidecarLoaderViolations(source, "src/tasks/task-registry.ts", new Set()),
|
|
).toEqual([
|
|
{
|
|
line: 7,
|
|
specifier: "./control.runtime.js",
|
|
sourcePath: "src/tasks/control.runtime.ts",
|
|
reason:
|
|
'hidden local runtime loader "./control.runtime.js" resolves to src/tasks/control.runtime.ts, but that source is not an explicit tsdown entry',
|
|
},
|
|
]);
|
|
});
|
|
|
|
it("ignores bundler-visible dynamic imports", () => {
|
|
const source = `
|
|
let runtimePromise: Promise<typeof import("./control.runtime.js")> | undefined;
|
|
export function loadRuntime() {
|
|
runtimePromise ??= import("./control.runtime.js");
|
|
return runtimePromise;
|
|
}
|
|
`;
|
|
|
|
expect(
|
|
findRuntimeSidecarLoaderViolations(source, "src/tasks/task-registry.ts", new Set()),
|
|
).toStrictEqual([]);
|
|
});
|
|
|
|
it("collects explicit tsdown entry sources", () => {
|
|
expect(
|
|
collectTsdownEntrySources([
|
|
{
|
|
entry: {
|
|
index: "src/index.ts",
|
|
"task-registry-control.runtime": "src/tasks/task-registry-control.runtime.ts",
|
|
},
|
|
},
|
|
]),
|
|
).toEqual(new Set(["src/index.ts", "src/tasks/task-registry-control.runtime.ts"]));
|
|
});
|
|
});
|