perf: prefer package-local bundled plugin artifacts · ope...

Recent Commits to openclaw:main

steipete · 2026-05-30 · via Recent Commits to openclaw:main

Original file line number	Diff line number	Diff line change
`@@ -6166,6 +6166,77 @@ module.exports = {`
`6166`	`6166`	`);`
`6167`	`6167`	`});`
`6168`	`6168`
	`6169`	`+it("prefers package-local dist artifacts for bundled source checkout plugins", () => {`
	`6170`	`+const repoRoot = makeTempDir();`
	`6171`	`+const sourceDir = path.join(repoRoot, "extensions", "startup-package-artifact-test");`
	`6172`	`+const runtimeDir = path.join(sourceDir, "dist");`
	`6173`	`+mkdirSafe(sourceDir);`
	`6174`	`+mkdirSafe(runtimeDir);`
	`6175`	`+fs.writeFileSync(`
	`6176`	`+path.join(sourceDir, "openclaw.plugin.json"),`
	`6177`	`+JSON.stringify(`
	`6178`	`+{`
	`6179`	`+id: "startup-package-artifact-test",`
	`6180`	`+configSchema: EMPTY_PLUGIN_SCHEMA,`
	`6181`	`+},`
	`6182`	`+null,`
	`6183`	`+2,`
	`6184`	`+),`
	`6185`	`+"utf-8",`
	`6186`	`+);`
	`6187`	`+fs.writeFileSync(`
	`6188`	`+path.join(sourceDir, "package.json"),`
	`6189`	`+JSON.stringify(`
	`6190`	`+{`
	`6191`	`+openclaw: {`
	`6192`	`+extensions: ["./index.ts"],`
	`6193`	`+},`
	`6194`	`+},`
	`6195`	`+null,`
	`6196`	`+2,`
	`6197`	`+),`
	`6198`	`+"utf-8",`
	`6199`	`+);`
	`6200`	`+fs.writeFileSync(`
	`6201`	`+path.join(sourceDir, "index.ts"),`
	`6202`	`+'throw new Error("source TS should not load during gateway startup");\n',`
	`6203`	`+"utf-8",`
	`6204`	`+);`
	`6205`	`+fs.writeFileSync(`
	`6206`	`+path.join(runtimeDir, "index.js"),`
	`6207`	`+'module.exports = { id: "startup-package-artifact-test", register() {} };\n',`
	`6208`	`+"utf-8",`
	`6209`	`+);`
	`6210`	`+`
	`6211`	`+const registry = withEnv(`
	`6212`	`+{`
	`6213`	`+OPENCLAW_BUNDLED_PLUGINS_DIR: path.join(repoRoot, "extensions"),`
	`6214`	`+OPENCLAW_TEST_TRUST_BUNDLED_PLUGINS_DIR: "1",`
	`6215`	`+OPENCLAW_DISABLE_BUNDLED_PLUGINS: undefined,`
	`6216`	`+},`
	`6217`	`+() =>`
	`6218`	`+loadOpenClawPlugins({`
	`6219`	`+cache: false,`
	`6220`	`+preferBuiltPluginArtifacts: true,`
	`6221`	`+onlyPluginIds: ["startup-package-artifact-test"],`
	`6222`	`+config: {`
	`6223`	`+plugins: {`
	`6224`	`+allow: ["startup-package-artifact-test"],`
	`6225`	`+entries: {`
	`6226`	`+"startup-package-artifact-test": {`
	`6227`	`+enabled: true,`
	`6228`	`+},`
	`6229`	`+},`
	`6230`	`+},`
	`6231`	`+},`
	`6232`	`+}),`
	`6233`	`+);`
	`6234`	`+`
	`6235`	`+expect(`
	`6236`	`+registry.plugins.find((entry) => entry.id === "startup-package-artifact-test")?.status,`
	`6237`	`+).toBe("loaded");`
	`6238`	`+});`
	`6239`	`+`
`6169`	`6240`	`it("prefers package-local dist artifacts over workspace source TS when requested", () => {`
`6170`	`6241`	`useNoBundledPlugins();`
`6171`	`6242`	`const pluginDir = makeTempDir();`

Original file line number	Diff line number	Diff line change
`@@ -626,6 +626,31 @@ function shouldPreferPackageLocalDistRuntimeArtifact(source: string): boolean {`
`626`	`626`	`}`
`627`	`627`	`}`
`628`	`628`
	`629`	`+function resolvePackageLocalDistRuntimeArtifact(params: {`
	`630`	`+source: string;`
	`631`	`+rootDir: string;`
	`632`	`+}): string \| null {`
	`633`	`+const relativeSource = path.relative(params.rootDir, params.source);`
	`634`	`+if (`
	`635`	`+!shouldPreferPackageLocalDistRuntimeArtifact(relativeSource) \|\|`
	`636`	`+relativeSource === "" \|\|`
	`637`	`+relativeSource.startsWith("..") \|\|`
	`638`	`+path.isAbsolute(relativeSource)`
	`639`	`+) {`
	`640`	`+return null;`
	`641`	`+}`
	`642`	`+const artifactRoot = path.join(params.rootDir, "dist");`
	`643`	`+for (const artifactRelativePath of listPackageLocalDistRuntimeArtifactRelativePaths(`
	`644`	`+relativeSource,`
	`645`	`+)) {`
	`646`	`+const artifactSource = path.join(artifactRoot, artifactRelativePath);`
	`647`	`+if (fs.existsSync(artifactSource)) {`
	`648`	`+return safeRealpathOrResolve(artifactSource);`
	`649`	`+}`
	`650`	`+}`
	`651`	`+return null;`
	`652`	`+}`
	`653`	`+`
`629`	`654`	`function resolvePreferredBuiltRuntimeArtifact(params: {`
`630`	`655`	`source: string;`
`631`	`656`	`rootDir: string;`
`@@ -638,28 +663,16 @@ function resolvePreferredBuiltRuntimeArtifact(params: {`
`638`	`663`	`return { source, rootDir };`
`639`	`664`	`}`
`640`	`665`	`if (params.origin !== "bundled") {`
`641`		`-const relativeSource = path.relative(rootDir, source);`
`642`		`-if (`
`643`		`-shouldPreferPackageLocalDistRuntimeArtifact(relativeSource) &&`
`644`		`-relativeSource !== "" &&`
`645`		`-!relativeSource.startsWith("..") &&`
`646`		`-!path.isAbsolute(relativeSource)`
`647`		`-) {`
`648`		`-const artifactRoot = path.join(rootDir, "dist");`
`649`		`-for (const artifactRelativePath of listPackageLocalDistRuntimeArtifactRelativePaths(`
`650`		`-relativeSource,`
`651`		`-)) {`
`652`		`-const artifactSource = path.join(artifactRoot, artifactRelativePath);`
`653`		`-if (fs.existsSync(artifactSource)) {`
`654`		`-return {`
`655`		`-source: safeRealpathOrResolve(artifactSource),`
`656`		`- rootDir,`
`657`		`-};`
`658`		`-}`
`659`		`-}`
	`666`	`+const artifactSource = resolvePackageLocalDistRuntimeArtifact({ source, rootDir });`
	`667`	`+if (artifactSource) {`
	`668`	`+return { source: artifactSource, rootDir };`
`660`	`669`	`}`
`661`	`670`	`return { source, rootDir };`
`662`	`671`	`}`
	`672`	`+const packageLocalArtifactSource = resolvePackageLocalDistRuntimeArtifact({ source, rootDir });`
	`673`	`+if (packageLocalArtifactSource) {`
	`674`	`+return { source: packageLocalArtifactSource, rootDir };`
	`675`	`+}`
`663`	`676`	`const extensionsDir = path.dirname(rootDir);`
`664`	`677`	`if (path.basename(extensionsDir) !== "extensions") {`
`665`	`678`	`return { source, rootDir };`

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

推荐订阅源

Recent Commits to openclaw:main