refactor(config): dedupe plugin metadata test snapshots ·...

Recent Commits to openclaw:main

vincentkoc · 2026-06-18 · via Recent Commits to openclaw:main

Original file line number	Diff line number	Diff line change
`@@ -4,16 +4,14 @@ import path from "node:path";`
`4`	`4`	`import { afterAll, afterEach, describe, expect, it, vi } from "vitest";`
`5`	`5`	`import { setCurrentPluginMetadataSnapshot } from "../plugins/current-plugin-metadata-snapshot.js";`
`6`	`6`	`import type { PluginCandidate, PluginDiscoveryResult } from "../plugins/discovery.js";`
`7`		`-import { resolveInstalledPluginIndexPolicyHash } from "../plugins/installed-plugin-index-policy.js";`
`8`		`-import type { PluginManifestRegistry } from "../plugins/manifest-registry.js";`
`9`		`-import type { PluginMetadataSnapshot } from "../plugins/plugin-metadata-snapshot.js";`
`10`	`7`	`import {`
`11`	`8`	`applyPluginAutoEnable,`
`12`	`9`	`detectPluginAutoEnableCandidates,`
`13`	`10`	`materializePluginAutoEnableCandidates,`
`14`	`11`	`resolvePluginAutoEnableCandidateReason,`
`15`	`12`	`} from "./plugin-auto-enable.js";`
`16`	`13`	`import {`
	`14`	`+createPluginMetadataSnapshot,`
`17`	`15`	`makeIsolatedEnv,`
`18`	`16`	`makeRegistry,`
`19`	`17`	`resetPluginAutoEnableTestState,`
`@@ -81,53 +79,6 @@ function makeBundledChannelCandidate(params: {`
`81`	`79`	`};`
`82`	`80`	`}`
`83`	`81`
`84`		`-function createPluginMetadataSnapshot(params: {`
`85`		`-config?: OpenClawConfig;`
`86`		`-manifestRegistry: PluginManifestRegistry;`
`87`		`-workspaceDir?: string;`
`88`		`-}): PluginMetadataSnapshot {`
`89`		`-const policyHash = resolveInstalledPluginIndexPolicyHash(params.config);`
`90`		`-return {`
`91`		`- policyHash,`
`92`		`- ...(params.workspaceDir ? { workspaceDir: params.workspaceDir } : {}),`
`93`		`-index: {`
`94`		`-version: 1,`
`95`		`-hostContractVersion: "test",`
`96`		`-compatRegistryVersion: "test",`
`97`		`-migrationVersion: 1,`
`98`		`- policyHash,`
`99`		`-generatedAtMs: 1,`
`100`		`-installRecords: {},`
`101`		`-plugins: [],`
`102`		`-diagnostics: [],`
`103`		`-},`
`104`		`-registryDiagnostics: [],`
`105`		`-manifestRegistry: params.manifestRegistry,`
`106`		`-plugins: params.manifestRegistry.plugins,`
`107`		`-diagnostics: params.manifestRegistry.diagnostics,`
`108`		`-byPluginId: new Map(params.manifestRegistry.plugins.map((plugin) => [plugin.id, plugin])),`
`109`		`-normalizePluginId: (pluginId) => pluginId,`
`110`		`-owners: {`
`111`		`-channels: new Map(),`
`112`		`-channelConfigs: new Map(),`
`113`		`-providers: new Map(),`
`114`		`-modelCatalogProviders: new Map(),`
`115`		`-cliBackends: new Map(),`
`116`		`-setupProviders: new Map(),`
`117`		`-commandAliases: new Map(),`
`118`		`-contracts: new Map(),`
`119`		`-},`
`120`		`-metrics: {`
`121`		`-registrySnapshotMs: 0,`
`122`		`-manifestRegistryMs: 0,`
`123`		`-ownerMapsMs: 0,`
`124`		`-totalMs: 0,`
`125`		`-indexPluginCount: 0,`
`126`		`-manifestPluginCount: params.manifestRegistry.plugins.length,`
`127`		`-},`
`128`		`-};`
`129`		`-}`
`130`		`-`
`131`	`82`	`afterAll(() => {`
`132`	`83`	`resetPluginAutoEnableTestState();`
`133`	`84`	`});`

Original file line number	Diff line number	Diff line change
`@@ -1,10 +1,13 @@`
`1`	`1`	`// Provides fixtures for plugin auto-enable config tests.`
`2`	`2`	`import path from "node:path";`
`3`	`3`	`import { clearCurrentPluginMetadataSnapshot } from "../plugins/current-plugin-metadata-snapshot.js";`
	`4`	`+import { resolveInstalledPluginIndexPolicyHash } from "../plugins/installed-plugin-index-policy.js";`
`4`	`5`	`import type { PluginManifestRegistry } from "../plugins/manifest-registry.js";`
	`6`	`+import type { PluginMetadataSnapshot } from "../plugins/plugin-metadata-snapshot.js";`
`5`	`7`	`import type { PluginOrigin } from "../plugins/plugin-origin.types.js";`
`6`	`8`	`import { clearPluginSetupRegistryCache } from "../plugins/setup-registry.js";`
`7`	`9`	`import { cleanupTrackedTempDirs, makeTrackedTempDir } from "../plugins/test-helpers/fs-fixtures.js";`
	`10`	`+import type { OpenClawConfig } from "./types.openclaw.js";`
`8`	`11`
`9`	`12`	`const tempDirs: string[] = [];`
`10`	`13`
`@@ -71,6 +74,53 @@ export function makeRegistry(`
`71`	`74`	`};`
`72`	`75`	`}`
`73`	`76`
	`77`	`+export function createPluginMetadataSnapshot(params: {`
	`78`	`+config?: OpenClawConfig;`
	`79`	`+manifestRegistry: PluginManifestRegistry;`
	`80`	`+workspaceDir?: string;`
	`81`	`+}): PluginMetadataSnapshot {`
	`82`	`+const policyHash = resolveInstalledPluginIndexPolicyHash(params.config);`
	`83`	`+return {`
	`84`	`+ policyHash,`
	`85`	`+ ...(params.workspaceDir ? { workspaceDir: params.workspaceDir } : {}),`
	`86`	`+index: {`
	`87`	`+version: 1,`
	`88`	`+hostContractVersion: "test",`
	`89`	`+compatRegistryVersion: "test",`
	`90`	`+migrationVersion: 1,`
	`91`	`+ policyHash,`
	`92`	`+generatedAtMs: 1,`
	`93`	`+installRecords: {},`
	`94`	`+plugins: [],`
	`95`	`+diagnostics: [],`
	`96`	`+},`
	`97`	`+registryDiagnostics: [],`
	`98`	`+manifestRegistry: params.manifestRegistry,`
	`99`	`+plugins: params.manifestRegistry.plugins,`
	`100`	`+diagnostics: params.manifestRegistry.diagnostics,`
	`101`	`+byPluginId: new Map(params.manifestRegistry.plugins.map((plugin) => [plugin.id, plugin])),`
	`102`	`+normalizePluginId: (pluginId) => pluginId,`
	`103`	`+owners: {`
	`104`	`+channels: new Map(),`
	`105`	`+channelConfigs: new Map(),`
	`106`	`+providers: new Map(),`
	`107`	`+modelCatalogProviders: new Map(),`
	`108`	`+cliBackends: new Map(),`
	`109`	`+setupProviders: new Map(),`
	`110`	`+commandAliases: new Map(),`
	`111`	`+contracts: new Map(),`
	`112`	`+},`
	`113`	`+metrics: {`
	`114`	`+registrySnapshotMs: 0,`
	`115`	`+manifestRegistryMs: 0,`
	`116`	`+ownerMapsMs: 0,`
	`117`	`+totalMs: 0,`
	`118`	`+indexPluginCount: 0,`
	`119`	`+manifestPluginCount: params.manifestRegistry.plugins.length,`
	`120`	`+},`
	`121`	`+};`
	`122`	`+}`
	`123`	`+`
`74`	`124`	`export function makeApnChannelConfig() {`
`75`	`125`	`return { channels: { apn: { someKey: "value" } } };`
`76`	`126`	`}`

Original file line number	Diff line number	Diff line change
`@@ -1,14 +1,14 @@`
`1`	`1`	`// Covers plugin activation context construction and lazy boundaries.`
`2`	`2`	`import { afterEach, describe, expect, it, vi } from "vitest";`
`3`		`-import { makeRegistry } from "../config/plugin-auto-enable.test-helpers.js";`
	`3`	`+import {`
	`4`	`+createPluginMetadataSnapshot,`
	`5`	`+makeRegistry,`
	`6`	`+} from "../config/plugin-auto-enable.test-helpers.js";`
`4`	`7`	`import type { OpenClawConfig } from "../config/types.openclaw.js";`
`5`	`8`	`import {`
`6`	`9`	`clearCurrentPluginMetadataSnapshot,`
`7`	`10`	`setCurrentPluginMetadataSnapshot,`
`8`	`11`	`} from "./current-plugin-metadata-snapshot.js";`
`9`		`-import { resolveInstalledPluginIndexPolicyHash } from "./installed-plugin-index-policy.js";`
`10`		`-import type { PluginManifestRegistry } from "./manifest-registry.js";`
`11`		`-import type { PluginMetadataSnapshot } from "./plugin-metadata-snapshot.js";`
`12`	`12`
`13`	`13`	`const applyPluginAutoEnableMock = vi.hoisted(() =>`
`14`	`14`	`vi.fn((params: { config?: OpenClawConfig }) => ({`
`@@ -24,53 +24,6 @@ vi.mock("../config/plugin-auto-enable.js", () => ({`
`24`	`24`
`25`	`25`	`import { resolveBundledPluginCompatibleActivationInputs } from "./activation-context.js";`
`26`	`26`
`27`		`-function createPluginMetadataSnapshot(params: {`
`28`		`-config?: OpenClawConfig;`
`29`		`-manifestRegistry: PluginManifestRegistry;`
`30`		`-workspaceDir?: string;`
`31`		`-}): PluginMetadataSnapshot {`
`32`		`-const policyHash = resolveInstalledPluginIndexPolicyHash(params.config);`
`33`		`-return {`
`34`		`- policyHash,`
`35`		`- ...(params.workspaceDir ? { workspaceDir: params.workspaceDir } : {}),`
`36`		`-index: {`
`37`		`-version: 1,`
`38`		`-hostContractVersion: "test",`
`39`		`-compatRegistryVersion: "test",`
`40`		`-migrationVersion: 1,`
`41`		`- policyHash,`
`42`		`-generatedAtMs: 1,`
`43`		`-installRecords: {},`
`44`		`-plugins: [],`
`45`		`-diagnostics: [],`
`46`		`-},`
`47`		`-registryDiagnostics: [],`
`48`		`-manifestRegistry: params.manifestRegistry,`
`49`		`-plugins: params.manifestRegistry.plugins,`
`50`		`-diagnostics: params.manifestRegistry.diagnostics,`
`51`		`-byPluginId: new Map(params.manifestRegistry.plugins.map((plugin) => [plugin.id, plugin])),`
`52`		`-normalizePluginId: (pluginId) => pluginId,`
`53`		`-owners: {`
`54`		`-channels: new Map(),`
`55`		`-channelConfigs: new Map(),`
`56`		`-providers: new Map(),`
`57`		`-modelCatalogProviders: new Map(),`
`58`		`-cliBackends: new Map(),`
`59`		`-setupProviders: new Map(),`
`60`		`-commandAliases: new Map(),`
`61`		`-contracts: new Map(),`
`62`		`-},`
`63`		`-metrics: {`
`64`		`-registrySnapshotMs: 0,`
`65`		`-manifestRegistryMs: 0,`
`66`		`-ownerMapsMs: 0,`
`67`		`-totalMs: 0,`
`68`		`-indexPluginCount: 0,`
`69`		`-manifestPluginCount: params.manifestRegistry.plugins.length,`
`70`		`-},`
`71`		`-};`
`72`		`-}`
`73`		`-`
`74`	`27`	`afterEach(() => {`
`75`	`28`	`clearCurrentPluginMetadataSnapshot();`
`76`	`29`	`applyPluginAutoEnableMock.mockClear();`

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

推荐订阅源

Recent Commits to openclaw:main