refactor(plugins): centralize npm runtime package planning · openclaw/openclaw@25ceffb

Recent Commits to openclaw:main

vincentkoc · 2026-05-03 · via Recent Commits to openclaw:main

Original file line number	Diff line number	Diff line change
`@@ -5,32 +5,11 @@ import path from "node:path";`
`5`	`5`	`import { pathToFileURL } from "node:url";`
`6`	`6`	`import {`
`7`	`7`	`buildPluginNpmRuntime,`
	`8`	`+listPluginNpmRuntimeBuildOutputs,`
	`9`	`+listPublishablePluginPackageDirs,`
`8`	`10`	`resolvePluginNpmRuntimeBuildPlan,`
`9`	`11`	`} from "./lib/plugin-npm-runtime-build.mjs";`
`10`	`12`
`11`		`-function readJsonFile(filePath) {`
`12`		`-return JSON.parse(fs.readFileSync(filePath, "utf8"));`
`13`		`-}`
`14`		`-`
`15`		`-function isPublishablePluginPackage(packageJson) {`
`16`		`-return packageJson.openclaw?.release?.publishToNpm === true;`
`17`		`-}`
`18`		`-`
`19`		`-function listPublishablePluginPackageDirs(repoRoot) {`
`20`		`-const extensionsRoot = path.join(repoRoot, "extensions");`
`21`		`-return fs`
`22`		`-.readdirSync(extensionsRoot, { withFileTypes: true })`
`23`		`-.filter((entry) => entry.isDirectory())`
`24`		`-.map((entry) => path.join("extensions", entry.name))`
`25`		`-.filter((packageDir) => {`
`26`		`-const packageJsonPath = path.join(repoRoot, packageDir, "package.json");`
`27`		`-return (`
`28`		`-fs.existsSync(packageJsonPath) && isPublishablePluginPackage(readJsonFile(packageJsonPath))`
`29`		`-);`
`30`		`-})`
`31`		`-.toSorted((left, right) => left.localeCompare(right));`
`32`		`-}`
`33`		`-`
`34`	`13`	`function parseArgs(argv) {`
`35`	`14`	`const packageDirs = [];`
`36`	`15`	`for (let index = 0; index < argv.length; index += 1) {`
`@@ -51,18 +30,12 @@ function parseArgs(argv) {`
`51`	`30`	`return { packageDirs };`
`52`	`31`	`}`
`53`	`32`
`54`		`-function listMissingRuntimeOutputs(plan) {`
`55`		`-return Object.keys(plan.entry)`
`56`		-.map((entryKey) => path.join(plan.outDir, `${entryKey}.js`))
`57`		`-.filter((filePath) => !fs.existsSync(filePath));`
`58`		`-}`
`59`		`-`
`60`	`33`	`export async function checkPluginNpmRuntimeBuilds(params = {}) {`
`61`	`34`	`const repoRoot = path.resolve(params.repoRoot ?? ".");`
`62`	`35`	`const packageDirs =`
`63`	`36`	`params.packageDirs?.length > 0`
`64`	`37`	`? params.packageDirs`
`65`		`- : listPublishablePluginPackageDirs(repoRoot);`
	`38`	`+ : listPublishablePluginPackageDirs({ repoRoot });`
`66`	`39`	`const rows = [];`
`67`	`40`	`for (const packageDir of packageDirs) {`
`68`	`41`	`const plan = resolvePluginNpmRuntimeBuildPlan({ repoRoot, packageDir });`
`@@ -74,13 +47,12 @@ export async function checkPluginNpmRuntimeBuilds(params = {}) {`
`74`	`47`	`packageDir,`
`75`	`48`	`logLevel: params.logLevel ?? "warn",`
`76`	`49`	`});`
`77`		`-const missing = listMissingRuntimeOutputs(result);`
	`50`	`+const missing = listPluginNpmRuntimeBuildOutputs(result).filter(`
	`51`	`+(runtimePath) =>`
	`52`	`+!fs.existsSync(path.join(result.packageDir, runtimePath.replace(/^\.\//u, ""))),`
	`53`	`+);`
`78`	`54`	`if (missing.length > 0) {`
`79`		`-throw new Error(`
`80`		-`${packageDir} missing built runtime outputs: ${missing
`81`		`- .map((filePath) => path.relative(repoRoot, filePath))`
`82`		- .join(", ")}`,
`83`		`-);`
	`55`	+throw new Error(`${packageDir} missing built runtime outputs: ${missing.join(", ")}`);
`84`	`56`	`}`
`85`	`57`	`rows.push({`
`86`	`58`	`pluginDir: result.pluginDir,`

Original file line number	Diff line number	Diff line change
`@@ -3,7 +3,10 @@ import fs from "node:fs";`
`3`	`3`	`import path from "node:path";`
`4`	`4`	`import { pathToFileURL } from "node:url";`
`5`	`5`	`import JSON5 from "json5";`
`6`		`-import { resolvePluginNpmRuntimeBuildPlan } from "./plugin-npm-runtime-build.mjs";`
	`6`	`+import {`
	`7`	`+listPluginNpmRuntimeBuildOutputs,`
	`8`	`+resolvePluginNpmRuntimeBuildPlan,`
	`9`	`+} from "./plugin-npm-runtime-build.mjs";`
`7`	`10`
`8`	`11`	`const GENERATED_BUNDLED_CHANNEL_CONFIG_METADATA_PATH =`
`9`	`12`	`"src/config/bundled-channel-config-metadata.generated.ts";`
`@@ -28,34 +31,8 @@ function packageRelativePathExists(packageDir, relativePath) {`
`28`	`31`	`return fs.existsSync(path.join(packageDir, relativePath));`
`29`	`32`	`}`
`30`	`33`
`31`		`-function mergePackageFiles(packageDir, files) {`
`32`		`-const merged = new Set(`
`33`		`-Array.isArray(files) ? files.filter((entry) => typeof entry === "string") : [],`
`34`		`-);`
`35`		`-merged.add("dist/**");`
`36`		`-if (packageRelativePathExists(packageDir, "openclaw.plugin.json")) {`
`37`		`-merged.add("openclaw.plugin.json");`
`38`		`-}`
`39`		`-if (packageRelativePathExists(packageDir, "README.md")) {`
`40`		`-merged.add("README.md");`
`41`		`-}`
`42`		`-if (packageRelativePathExists(packageDir, "SKILL.md")) {`
`43`		`-merged.add("SKILL.md");`
`44`		`-}`
`45`		`-if (packageRelativePathExists(packageDir, "skills")) {`
`46`		`-merged.add("skills/**");`
`47`		`-}`
`48`		`-return [...merged];`
`49`		`-}`
`50`		`-`
`51`		`-function listRuntimeBuildOutputs(plan) {`
`52`		`-return Object.keys(plan.entry)`
`53`		-.map((entryKey) => `./dist/${entryKey}.js`)
`54`		`-.toSorted((left, right) => left.localeCompare(right));`
`55`		`-}`
`56`		`-`
`57`	`34`	`function assertPluginNpmRuntimeBuildExists(plan) {`
`58`		`-const missing = listRuntimeBuildOutputs(plan).filter(`
	`35`	`+const missing = listPluginNpmRuntimeBuildOutputs(plan).filter(`
`59`	`36`	`(runtimePath) => !packageRelativePathExists(plan.packageDir, runtimePath.replace(/^\.\//u, "")),`
`60`	`37`	`);`
`61`	`38`	`if (missing.length > 0) {`
`@@ -98,7 +75,7 @@ export function resolveAugmentedPluginNpmPackageJson(params) {`
`98`	`75`
`99`	`76`	`const packageJson = {`
`100`	`77`	`...plan.packageJson,`
`101`		`-files: mergePackageFiles(packageDir, plan.packageJson.files),`
	`78`	`+files: plan.packageFiles,`
`102`	`79`	`openclaw: {`
`103`	`80`	`...plan.packageJson.openclaw,`
`104`	`81`	`runtimeExtensions: plan.runtimeExtensions,`

Original file line number	Diff line number	Diff line change
`@@ -16,6 +16,10 @@ function readJsonFile(filePath) {`
`16`	`16`	`return JSON.parse(fs.readFileSync(filePath, "utf8"));`
`17`	`17`	`}`
`18`	`18`
	`19`	`+export function isPublishablePluginPackage(packageJson) {`
	`20`	`+return packageJson.openclaw?.release?.publishToNpm === true;`
	`21`	`+}`
	`22`	`+`
`19`	`23`	`function normalizePackageEntry(value) {`
`20`	`24`	`return typeof value === "string" ? value.trim().replaceAll("\\", "/") : "";`
`21`	`25`	`}`
`@@ -64,6 +68,54 @@ function resolvePackageDir(repoRoot, packageDir) {`
`64`	`68`	`return path.isAbsolute(packageDir) ? packageDir : path.resolve(repoRoot, packageDir);`
`65`	`69`	`}`
`66`	`70`
	`71`	`+function packageRelativePathExists(packageDir, relativePath) {`
	`72`	`+return fs.existsSync(path.join(packageDir, relativePath));`
	`73`	`+}`
	`74`	`+`
	`75`	`+export function listPublishablePluginPackageDirs(params = {}) {`
	`76`	`+const repoRoot = path.resolve(params.repoRoot ?? ".");`
	`77`	`+const extensionsRoot = path.join(repoRoot, "extensions");`
	`78`	`+return fs`
	`79`	`+.readdirSync(extensionsRoot, { withFileTypes: true })`
	`80`	`+.filter((entry) => entry.isDirectory())`
	`81`	`+.map((entry) => path.join("extensions", entry.name))`
	`82`	`+.filter((packageDir) => {`
	`83`	`+const packageJsonPath = path.join(repoRoot, packageDir, "package.json");`
	`84`	`+return (`
	`85`	`+fs.existsSync(packageJsonPath) && isPublishablePluginPackage(readJsonFile(packageJsonPath))`
	`86`	`+);`
	`87`	`+})`
	`88`	`+.toSorted((left, right) => left.localeCompare(right));`
	`89`	`+}`
	`90`	`+`
	`91`	`+export function listPluginNpmRuntimeBuildOutputs(plan) {`
	`92`	`+return Object.keys(plan.entry)`
	`93`	+.map((entryKey) => `./dist/${entryKey}.js`)
	`94`	`+.toSorted((left, right) => left.localeCompare(right));`
	`95`	`+}`
	`96`	`+`
	`97`	`+export function resolvePluginNpmRuntimePackageFiles(plan) {`
	`98`	`+const merged = new Set(`
	`99`	`+Array.isArray(plan.packageJson.files)`
	`100`	`+ ? plan.packageJson.files.filter((entry) => typeof entry === "string")`
	`101`	`+ : [],`
	`102`	`+);`
	`103`	`+merged.add("dist/**");`
	`104`	`+if (packageRelativePathExists(plan.packageDir, "openclaw.plugin.json")) {`
	`105`	`+merged.add("openclaw.plugin.json");`
	`106`	`+}`
	`107`	`+if (packageRelativePathExists(plan.packageDir, "README.md")) {`
	`108`	`+merged.add("README.md");`
	`109`	`+}`
	`110`	`+if (packageRelativePathExists(plan.packageDir, "SKILL.md")) {`
	`111`	`+merged.add("SKILL.md");`
	`112`	`+}`
	`113`	`+if (packageRelativePathExists(plan.packageDir, "skills")) {`
	`114`	`+merged.add("skills/**");`
	`115`	`+}`
	`116`	`+return [...merged];`
	`117`	`+}`
	`118`	`+`
`67`	`119`	`export function resolvePluginNpmRuntimeBuildPlan(params) {`
`68`	`120`	`const repoRoot = path.resolve(params.repoRoot ?? ".");`
`69`	`121`	`const packageDir = resolvePackageDir(repoRoot, params.packageDir);`
`@@ -72,7 +124,7 @@ export function resolvePluginNpmRuntimeBuildPlan(params) {`
`72`	`124`	`return null;`
`73`	`125`	`}`
`74`	`126`	`const packageJson = readJsonFile(packageJsonPath);`
`75`		`-if (packageJson.openclaw?.release?.publishToNpm !== true) {`
	`127`	`+if (!isPublishablePluginPackage(packageJson)) {`
`76`	`128`	`return null;`
`77`	`129`	`}`
`78`	`130`
`@@ -96,7 +148,7 @@ export function resolvePluginNpmRuntimeBuildPlan(params) {`
`96`	`148`	`]),`
`97`	`149`	`);`
`98`	`150`
`99`		`-return {`
	`151`	`+const plan = {`
`100`	`152`	`repoRoot,`
`101`	`153`	`packageDir,`
`102`	`154`	`pluginDir,`
`@@ -115,6 +167,11 @@ export function resolvePluginNpmRuntimeBuildPlan(params) {`
`115`	`167`	`? toPackageRuntimeEntry(packageJson.openclaw.setupEntry)`
`116`	`168`	`: undefined,`
`117`	`169`	`};`
	`170`	`+return {`
	`171`	`+ ...plan,`
	`172`	`+runtimeBuildOutputs: listPluginNpmRuntimeBuildOutputs(plan),`
	`173`	`+packageFiles: resolvePluginNpmRuntimePackageFiles(plan),`
	`174`	`+};`
`118`	`175`	`}`
`119`	`176`
`120`	`177`	`export async function buildPluginNpmRuntime(params) {`

Original file line number	Diff line number	Diff line change
`@@ -1,37 +1,15 @@`
`1`		`-import fs from "node:fs";`
`2`	`1`	`import path from "node:path";`
`3`	`2`	`import { describe, expect, it } from "vitest";`
`4`		`-import { resolvePluginNpmRuntimeBuildPlan } from "../scripts/lib/plugin-npm-runtime-build.mjs";`
	`3`	`+import {`
	`4`	`+listPublishablePluginPackageDirs,`
	`5`	`+resolvePluginNpmRuntimeBuildPlan,`
	`6`	`+} from "../scripts/lib/plugin-npm-runtime-build.mjs";`
`5`	`7`
`6`	`8`	`const repoRoot = path.resolve(import.meta.dirname, "..");`
`7`	`9`
`8`		`-function readJsonFile(filePath: string): Record<string, unknown> {`
`9`		`-return JSON.parse(fs.readFileSync(filePath, "utf8")) as Record<string, unknown>;`
`10`		`-}`
`11`		`-`
`12`		`-function isPublishablePluginPackage(packageJson: Record<string, unknown>): boolean {`
`13`		`-const openclaw = packageJson.openclaw as { release?: { publishToNpm?: unknown } } \| undefined;`
`14`		`-return openclaw?.release?.publishToNpm === true;`
`15`		`-}`
`16`		`-`
`17`		`-function listPublishablePluginPackageDirs(): string[] {`
`18`		`-const extensionsRoot = path.join(repoRoot, "extensions");`
`19`		`-return fs`
`20`		`-.readdirSync(extensionsRoot, { withFileTypes: true })`
`21`		`-.filter((dirent) => dirent.isDirectory())`
`22`		`-.map((dirent) => path.join(extensionsRoot, dirent.name))`
`23`		`-.filter((packageDir) => {`
`24`		`-const packageJsonPath = path.join(packageDir, "package.json");`
`25`		`-return (`
`26`		`-fs.existsSync(packageJsonPath) && isPublishablePluginPackage(readJsonFile(packageJsonPath))`
`27`		`-);`
`28`		`-})`
`29`		`-.toSorted((left, right) => left.localeCompare(right));`
`30`		`-}`
`31`		`-`
`32`	`10`	`describe("plugin npm runtime build planning", () => {`
`33`	`11`	`it("plans package-local runtime entries for every publishable plugin package", () => {`
`34`		`-const packageDirs = listPublishablePluginPackageDirs();`
	`12`	`+const packageDirs = listPublishablePluginPackageDirs({ repoRoot });`
`35`	`13`	`expect(packageDirs.length).toBeGreaterThan(0);`
`36`	`14`
`37`	`15`	`const plans = packageDirs.map((packageDir) =>`
`@@ -46,6 +24,8 @@ describe("plugin npm runtime build planning", () => {`
`46`	`24`	`for (const plan of plans) {`
`47`	`25`	`expect(plan?.outDir).toBe(path.join(plan?.packageDir ?? "", "dist"));`
`48`	`26`	`expect(plan?.runtimeExtensions.every((entry) => entry.startsWith("./dist/"))).toBe(true);`
	`27`	`+expect(plan?.runtimeBuildOutputs.every((entry) => entry.startsWith("./dist/"))).toBe(true);`
	`28`	`+expect(plan?.packageFiles).toContain("dist/**");`
`49`	`29`	`}`
`50`	`30`	`});`
`51`	`31`
`@@ -75,5 +55,11 @@ describe("plugin npm runtime build planning", () => {`
`75`	`55`	`"runtime-api": path.join(repoRoot, "extensions", "diffs", "runtime-api.ts"),`
`76`	`56`	`}),`
`77`	`57`	`);`
	`58`	`+expect(diffsPlan?.packageFiles).toEqual([`
	`59`	`+"dist/**",`
	`60`	`+"openclaw.plugin.json",`
	`61`	`+"README.md",`
	`62`	`+"skills/**",`
	`63`	`+]);`
`78`	`64`	`});`
`79`	`65`	`});`

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

推荐订阅源

Recent Commits to openclaw:main