fix(doctor): honor runtimeExtensions before flagging entry_unresolved · openclaw/openclaw@0f39636

Recent Commits to openclaw:main

arniesaha · 2026-06-01 · via Recent Commits to openclaw:main

Original file line number	Diff line number	Diff line change
`@@ -109,6 +109,59 @@ describe("runPostUpgradeProbes — plugin.entry_unresolved", () => {`
`109`	`109`	`await fs.rm(root, { recursive: true, force: true });`
`110`	`110`	`}`
`111`	`111`	`});`
	`112`	`+`
	`113`	`+it("does not flag entry_unresolved when runtimeExtensions exists even if source entry is missing", async () => {`
	`114`	`+const root = await makeFixtureRoot("runtime-extensions");`
	`115`	`+try {`
	`116`	`+const pluginDir = path.join(root, "user-plugins", "runtime-only");`
	`117`	`+await fs.mkdir(path.join(pluginDir, "dist"), { recursive: true });`
	`118`	`+await fs.writeFile(path.join(pluginDir, "dist", "index.js"), "export default {};", "utf-8");`
	`119`	`+// Source entry (./src/index.ts) does NOT exist`
	`120`	`+// But runtime entry (./dist/index.js) DOES exist`
	`121`	`+await fs.writeFile(`
	`122`	`+path.join(pluginDir, "package.json"),`
	`123`	`+JSON.stringify({`
	`124`	`+name: "runtime-only",`
	`125`	`+version: "0.0.1",`
	`126`	`+type: "module",`
	`127`	`+openclaw: {`
	`128`	`+extensions: ["./src/index.ts"],`
	`129`	`+runtimeExtensions: ["./dist/index.js"],`
	`130`	`+},`
	`131`	`+}),`
	`132`	`+"utf-8",`
	`133`	`+);`
	`134`	`+await fs.writeFile(`
	`135`	`+path.join(pluginDir, "openclaw.plugin.json"),`
	`136`	`+JSON.stringify({ id: "runtime-only" }),`
	`137`	`+"utf-8",`
	`138`	`+);`
	`139`	`+`
	`140`	`+const installsPath = path.join(root, "plugins", "installs.json");`
	`141`	`+await fs.mkdir(path.dirname(installsPath), { recursive: true });`
	`142`	`+await fs.writeFile(`
	`143`	`+installsPath,`
	`144`	`+JSON.stringify({`
	`145`	`+version: 1,`
	`146`	`+plugins: [`
	`147`	`+{`
	`148`	`+pluginId: "runtime-only",`
	`149`	`+manifestPath: path.join(pluginDir, "openclaw.plugin.json"),`
	`150`	`+rootDir: pluginDir,`
	`151`	`+enabled: true,`
	`152`	`+packageJson: { path: "package.json" },`
	`153`	`+},`
	`154`	`+],`
	`155`	`+}),`
	`156`	`+"utf-8",`
	`157`	`+);`
	`158`	`+`
	`159`	`+const report = await runPostUpgradeProbes({ installsPath });`
	`160`	`+expect(report.findings.filter((f) => f.code === "plugin.entry_unresolved")).toHaveLength(0);`
	`161`	`+} finally {`
	`162`	`+await fs.rm(root, { recursive: true, force: true });`
	`163`	`+}`
	`164`	`+});`
`112`	`165`	`});`
`113`	`166`
`114`	`167`	`describe("runPostUpgradeProbes — plugin.manifest_drift", () => {`

Original file line number	Diff line number	Diff line change
`@@ -1,6 +1,7 @@`
`1`	`1`	`import crypto from "node:crypto";`
`2`	`2`	`import fs from "node:fs/promises";`
`3`	`3`	`import path from "node:path";`
	`4`	`+import { listBuiltRuntimeEntryCandidates } from "../plugins/package-entrypoints.js";`
`4`	`5`	`import type { PostUpgradeFinding, PostUpgradeReport } from "./doctor-post-upgrade.types.js";`
`5`	`6`
`6`	`7`	`type InstalledPluginRecord = {`
`@@ -17,7 +18,7 @@ type InstallsJson = { plugins: InstalledPluginRecord[] };`
`17`	`18`	`async function readInstalledPackageJson(rootDir: string, packageJsonRelPath: string) {`
`18`	`19`	`const absPath = path.join(rootDir, packageJsonRelPath);`
`19`	`20`	`const raw = await fs.readFile(absPath, "utf-8");`
`20`		`-return JSON.parse(raw) as { openclaw?: { extensions?: string[] } };`
	`21`	`+return JSON.parse(raw) as { openclaw?: { extensions?: string[]; runtimeExtensions?: string[] } };`
`21`	`22`	`}`
`22`	`23`
`23`	`24`	`async function fileExists(absPath: string): Promise<boolean> {`
`@@ -48,7 +49,7 @@ export async function runPostUpgradeProbes(params: {`
`48`	`49`	`for (const record of installs.plugins) {`
`49`	`50`	`if (!record.enabled) continue;`
`50`	`51`	`const pkgRelPath = record.packageJson?.path ?? "package.json";`
`51`		`-let pkg: { openclaw?: { extensions?: string[] } };`
	`52`	`+let pkg: { openclaw?: { extensions?: string[]; runtimeExtensions?: string[] } };`
`52`	`53`	`try {`
`53`	`54`	`pkg = await readInstalledPackageJson(record.rootDir, pkgRelPath);`
`54`	`55`	`} catch (err) {`
`@@ -58,9 +59,31 @@ export async function runPostUpgradeProbes(params: {`
`58`	`59`	`continue;`
`59`	`60`	`}`
`60`	`61`	`const entries = pkg.openclaw?.extensions ?? [];`
`61`		`-for (const entry of entries) {`
	`62`	`+const runtimeExtensions = pkg.openclaw?.runtimeExtensions ?? [];`
	`63`	`+for (const [index, entry] of entries.entries()) {`
	`64`	`+// First, check if there's an explicit runtimeExtensions entry at this index`
	`65`	`+const runtimeEntry = runtimeExtensions[index];`
	`66`	`+if (runtimeEntry) {`
	`67`	`+const absRuntimeEntry = path.resolve(record.rootDir, runtimeEntry);`
	`68`	`+if (await fileExists(absRuntimeEntry)) {`
	`69`	`+// Runtime entry exists, so we're good; skip to next entry`
	`70`	`+continue;`
	`71`	`+}`
	`72`	`+// Runtime entry doesn't exist; flag it`
	`73`	`+findings.push({`
	`74`	`+level: "error",`
	`75`	`+code: "plugin.entry_unresolved",`
	`76`	+message: `Plugin ${record.pluginId} declares runtimeExtensions entry ${runtimeEntry} but the file does not exist at ${absRuntimeEntry}.`,
	`77`	`+plugin: record.pluginId,`
	`78`	`+entry: runtimeEntry,`
	`79`	`+});`
	`80`	`+continue;`
	`81`	`+}`
	`82`	`+`
	`83`	`+// No explicit runtime entry; try the source entry`
`62`	`84`	`const absEntry = path.resolve(record.rootDir, entry);`
`63`	`85`	`if (!(await fileExists(absEntry))) {`
	`86`	`+// Source entry doesn't exist either; flag it`
`64`	`87`	`findings.push({`
`65`	`88`	`level: "error",`
`66`	`89`	`code: "plugin.entry_unresolved",`

此内容由惯性聚合(RSS阅读器)自动聚合整理，仅供阅读参考。原文来自 — 版权归原作者所有。

推荐订阅源

Recent Commits to openclaw:main