fix(release): keep private test helpers out of npm pack ·...

Recent Commits to openclaw:main

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

Original file line number	Diff line number	Diff line change
`@@ -11,7 +11,7 @@`
`11`	`11`	`import { readFileSync, existsSync } from "node:fs";`
`12`	`12`	`import { resolve, dirname } from "node:path";`
`13`	`13`	`import { fileURLToPath, pathToFileURL } from "node:url";`
`14`		`-import { pluginSdkSubpaths } from "./lib/plugin-sdk-entries.mjs";`
	`14`	`+import { publicPluginSdkSubpaths } from "./lib/plugin-sdk-entries.mjs";`
`15`	`15`
`16`	`16`	`const scriptDir = dirname(fileURLToPath(import.meta.url));`
`17`	`17`	`const distFile = resolve(scriptDir, "..", "dist", "plugin-sdk", "index.js");`
`@@ -70,7 +70,7 @@ for (const name of requiredExports) {`
`70`	`70`	`}`
`71`	`71`	`}`
`72`	`72`
`73`		`-for (const entry of pluginSdkSubpaths) {`
	`73`	`+for (const entry of publicPluginSdkSubpaths) {`
`74`	`74`	const jsPath = resolve(scriptDir, "..", "dist", "plugin-sdk", `${entry}.js`);
`75`	`75`	const dtsPath = resolve(scriptDir, "..", "dist", "plugin-sdk", `${entry}.d.ts`);
`76`	`76`	`if (!existsSync(jsPath)) {`

Original file line number	Diff line number	Diff line change
`@@ -12,6 +12,8 @@ const TOP_LEVEL_PUBLIC_SURFACE_EXTENSIONS = new Set([".ts", ".js", ".mts", ".cts`
`12`	`12`	`export const NON_PACKAGED_BUNDLED_PLUGIN_DIRS = new Set(["qa-channel", "qa-lab", "qa-matrix"]);`
`13`	`13`	`const EXCLUDED_CORE_BUNDLED_PLUGIN_DIRS = new Set(["qqbot", "whatsapp"]);`
`14`	`14`	`const BUNDLED_PLUGIN_BUILD_IDS_ENV = "OPENCLAW_BUNDLED_PLUGIN_BUILD_IDS";`
	`15`	`+const TOP_LEVEL_PRIVATE_TEST_SURFACE_RE =`
	`16`	`+/(?:^\|[._-])(?:test\|spec\|test-support\|test-helpers\|test-fixtures\|test-harness\|mock-setup)(?:[._-]\|$)/u;`
`15`	`17`	`const toPosixPath = (value) => value.replaceAll("\\", "/");`
`16`	`18`
`17`	`19`	`function parseBundledPluginBuildIdFilter(env = process.env) {`
`@@ -88,8 +90,7 @@ export function collectTopLevelPublicSurfaceEntries(pluginDir) {`
`88`	`90`	`if (`
`89`	`91`	`normalizedName.endsWith(".d.ts") \|\|`
`90`	`92`	`/^config-api\.(?:[cm]?[jt]s)$/u.test(normalizedName) \|\|`
`91`		`-normalizedName.includes(".test.") \|\|`
`92`		`-normalizedName.includes(".spec.") \|\|`
	`93`	`+TOP_LEVEL_PRIVATE_TEST_SURFACE_RE.test(normalizedName) \|\|`
`93`	`94`	`normalizedName.includes(".fixture.") \|\|`
`94`	`95`	`normalizedName.includes(".snap")`
`95`	`96`	`) {`
`@@ -117,8 +118,7 @@ function collectTopLevelPublicSurfaceEntriesFromFiles(relativeFiles) {`
`117`	`118`	`if (`
`118`	`119`	`normalizedName.endsWith(".d.ts") \|\|`
`119`	`120`	`/^config-api\.(?:[cm]?[jt]s)$/u.test(normalizedName) \|\|`
`120`		`-normalizedName.includes(".test.") \|\|`
`121`		`-normalizedName.includes(".spec.") \|\|`
	`121`	`+TOP_LEVEL_PRIVATE_TEST_SURFACE_RE.test(normalizedName) \|\|`
`122`	`122`	`normalizedName.includes(".fixture.") \|\|`
`123`	`123`	`normalizedName.includes(".snap")`
`124`	`124`	`) {`

Original file line number	Diff line number	Diff line change
`@@ -6,7 +6,7 @@ export const publicPluginSdkSubpaths: string[];`
`6`	`6`	`export const deprecatedPublicPluginSdkEntrypoints: string[];`
`7`	`7`	`export const deprecatedBarrelPluginSdkEntrypoints: string[];`
`8`	`8`
`9`		`-export function buildPluginSdkEntrySources(): Record<string, string>;`
	`9`	`+export function buildPluginSdkEntrySources(entries?: readonly string[]): Record<string, string>;`
`10`	`10`	`export function buildPluginSdkPackageExports(): Record<`
`11`	`11`	`string,`
`12`	`12`	`{`

Original file line number	Diff line number	Diff line change
`@@ -33,10 +33,8 @@ export const deprecatedBarrelPluginSdkEntrypoints = pluginSdkSubpaths.filter((en`
`33`	`33`	`deprecatedBarrelPluginSdkSubpathList.includes(entry),`
`34`	`34`	`);`
`35`	`35`
`36`		`-export function buildPluginSdkEntrySources() {`
`37`		`-return Object.fromEntries(`
`38`		-pluginSdkEntrypoints.map((entry) => [entry, `src/plugin-sdk/${entry}.ts`]),
`39`		`-);`
	`36`	`+export function buildPluginSdkEntrySources(entries = pluginSdkEntrypoints) {`
	`37`	+return Object.fromEntries(entries.map((entry) => [entry, `src/plugin-sdk/${entry}.ts`]));
`40`	`38`	`}`
`41`	`39`
`42`	`40`	`export function buildPluginSdkPackageExports() {`

Original file line number	Diff line number	Diff line change
`@@ -2,7 +2,11 @@ import fs from "node:fs";`
`2`	`2`	`import os from "node:os";`
`3`	`3`	`import path from "node:path";`
`4`	`4`	`import { build } from "tsdown";`
`5`		`-import { buildPluginSdkEntrySources, pluginSdkEntrypoints } from "./lib/plugin-sdk-entries.mjs";`
	`5`	`+import {`
	`6`	`+buildPluginSdkEntrySources,`
	`7`	`+pluginSdkEntrypoints,`
	`8`	`+publicPluginSdkEntrypoints,`
	`9`	`+} from "./lib/plugin-sdk-entries.mjs";`
`6`	`10`
`7`	`11`	`const RUNTIME_SHIMS: Partial<Record<string, string>> = {`
`8`	`12`	`"webhook-path": [`
`@@ -67,14 +71,19 @@ function copyFlatDeclarations(fromDir: string, toDir: string): void {`
`67`	`71`
`68`	`72`	`const distPluginSdkDir = path.join(process.cwd(), "dist/plugin-sdk");`
`69`	`73`	`const flatDeclarationTempDir = fs.mkdtempSync(path.join(os.tmpdir(), "openclaw-plugin-sdk-dts-"));`
	`74`	`+const shouldBuildPrivateQaEntries = process.env.OPENCLAW_BUILD_PRIVATE_QA === "1";`
	`75`	`+const flatDeclarationEntrypoints = shouldBuildPrivateQaEntries`
	`76`	`+ ? pluginSdkEntrypoints`
	`77`	`+ : publicPluginSdkEntrypoints;`
	`78`	`+const flatDeclarationEntrypointSet = new Set(flatDeclarationEntrypoints);`
`70`	`79`
`71`	`80`	`try {`
`72`	`81`	`await build({`
`73`	`82`	`clean: true,`
`74`	`83`	`config: false,`
`75`	`84`	`deps: { neverBundle: (id) => isBareImportSpecifier(id) },`
`76`	`85`	`dts: true,`
`77`		`-entry: buildPluginSdkEntrySources(),`
	`86`	`+entry: buildPluginSdkEntrySources(flatDeclarationEntrypoints),`
`78`	`87`	`failOnWarn: false,`
`79`	`88`	`fixedExtension: false,`
`80`	`89`	`format: "esm",`
`@@ -96,6 +105,10 @@ try {`
`96`	`105`	`// The private workspace package keeps source-shaped declaration paths for local`
`97`	`106`	`// package-boundary projects, so bridge them back to the packaged flat entries.`
`98`	`107`	`for (const entry of pluginSdkEntrypoints) {`
	`108`	`+if (!flatDeclarationEntrypointSet.has(entry)) {`
	`109`	`+continue;`
	`110`	`+}`
	`111`	`+`
`99`	`112`	`const packageTypeOut = path.join(`
`100`	`113`	`process.cwd(),`
`101`	`114`	`packages/plugin-sdk/dist/src/plugin-sdk/${entry}.d.ts`,

Original file line number	Diff line number	Diff line change
`@@ -103,6 +103,12 @@ describe("bundled plugin build entries", () => {`
`103`	`103`	`expect(entries["extensions/telegram/telegram-ingress-worker.runtime"]).toBeUndefined();`
`104`	`104`	`});`
`105`	`105`
	`106`	`+it("keeps top-level bundled plugin test helpers out of public-surface entries", () => {`
	`107`	`+const entries = listBundledPluginBuildEntries();`
	`108`	`+`
	`109`	`+expect(entries["extensions/browser/test-support"]).toBeUndefined();`
	`110`	`+});`
	`111`	`+`
`106`	`112`	`it("discovers repo plugin build entries without directory scans", () => {`
`107`	`113`	`const payload = expectNoNodeFsScans<{`
`108`	`114`	`artifacts: number;`

Original file line number	Diff line number	Diff line change
`@@ -5,7 +5,11 @@ import {`
`5`	`5`	`collectBundledPluginBuildEntries,`
`6`	`6`	`NON_PACKAGED_BUNDLED_PLUGIN_DIRS,`
`7`	`7`	`} from "./scripts/lib/bundled-plugin-build-entries.mjs";`
`8`		`-import { buildPluginSdkEntrySources } from "./scripts/lib/plugin-sdk-entries.mjs";`
	`8`	`+import {`
	`9`	`+buildPluginSdkEntrySources,`
	`10`	`+pluginSdkEntrypoints,`
	`11`	`+publicPluginSdkEntrypoints,`
	`12`	`+} from "./scripts/lib/plugin-sdk-entries.mjs";`
`9`	`13`
`10`	`14`	`type InputOptionsFactory = Extract<NonNullable<UserConfig["inputOptions"]>, Function>;`
`11`	`15`	`type InputOptionsArg = InputOptionsFactory extends (`
`@@ -140,6 +144,9 @@ function nodeBuildConfig(config: UserConfig): UserConfig {`
`140`	`144`
`141`	`145`	`const bundledPluginBuildEntries = collectBundledPluginBuildEntries();`
`142`	`146`	`const shouldBuildPrivateQaEntries = process.env.OPENCLAW_BUILD_PRIVATE_QA === "1";`
	`147`	`+const productionPluginSdkEntrypoints = shouldBuildPrivateQaEntries`
	`148`	`+ ? pluginSdkEntrypoints`
	`149`	`+ : publicPluginSdkEntrypoints;`
`143`	`150`
`144`	`151`	`function buildBundledHookEntries(): Record<string, string> {`
`145`	`152`	`const hooksRoot = path.join(process.cwd(), "src", "hooks", "bundled");`
`@@ -357,10 +364,9 @@ function buildUnifiedDistEntries(): Record<string, string> {`
`357`	`364`	`// Private bundled Codex helper for app-server user MCP config projection.`
`358`	`365`	`"plugin-sdk/codex-mcp-projection": "src/plugin-sdk/codex-mcp-projection.ts",`
`359`	`366`	`...Object.fromEntries(`
`360`		`-Object.entries(buildPluginSdkEntrySources()).map(([entry, source]) => [`
`361`		-`plugin-sdk/${entry}`,
`362`		`-source,`
`363`		`-]),`
	`367`	`+Object.entries(buildPluginSdkEntrySources(productionPluginSdkEntrypoints)).map(`
	`368`	+([entry, source]) => [`plugin-sdk/${entry}`, source],
	`369`	`+),`
`364`	`370`	`),`
`365`	`371`	`...(shouldBuildPrivateQaEntries`
`366`	`372`	`? {`

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

推荐订阅源

Recent Commits to openclaw:main