fix(plugins): add missing discovery.js mock exports for addMissingRequiredPluginDiagnostics

LeonidasLux · LeonidasLux · commit 76e05c877c8f · 2026-07-06T10:56:36.000+08:00
plugin-install.test.ts mocks discovery.js but the mock didn't include the newly exported addMissingRequiredPluginDiagnostics, causing CI failures in tests that exercise loadPluginManifestRegistryForInstalledIndex. Also add discoverFromConfigPaths to the mock (already imported but was only latent because the mock's code path wasn't triggered with load paths). Ref. #99196
diff --git a/src/commands/channel-setup/plugin-install.test.ts b/src/commands/channel-setup/plugin-install.test.ts
@@ -91,6 +91,8 @@ vi.mock("../../plugins/loader.js", () => ({
 const discoverOpenClawPlugins = vi.fn((_args?: unknown) => ({ candidates: [], diagnostics: [] }));
 vi.mock("../../plugins/discovery.js", () => ({
   discoverOpenClawPlugins: (args: unknown) => discoverOpenClawPlugins(args),
+  discoverFromConfigPaths: () => ({ candidates: [], diagnostics: [] }),
+  addMissingRequiredPluginDiagnostics: () => {},
 }));
 
 import fs from "node:fs";
@@ -100,9 +102,7 @@ import { loadOpenClawPlugins } from "../../plugins/loader.js";
 import type { PluginManifestRecord } from "../../plugins/manifest-registry.js";
 import { clearPluginMetadataLifecycleCaches } from "../../plugins/plugin-metadata-lifecycle.js";
 import { createEmptyPluginRegistry } from "../../plugins/registry.js";
-import {
-  setActivePluginRegistry,
-} from "../../plugins/runtime.js";
+import { setActivePluginRegistry } from "../../plugins/runtime.js";
 import type { WizardPrompter } from "../../wizard/prompts.js";
 import { makePrompter, makeRuntime } from "../setup/__tests__/test-utils.js";
 import {
