perf(plugins): drop verbose JSDoc from discovery? params

RomneyDa · RomneyDa · commit 6b6254db8fd3 · 2026-05-19T15:04:06.000-07:00
diff --git a/src/plugins/config-contracts.ts b/src/plugins/config-contracts.ts
@@ -114,12 +114,6 @@ export function resolvePluginConfigContractsById(params: {
   fallbackToBundledMetadataForResolvedBundled?: boolean;
   fallbackBundledPluginIds?: readonly string[];
   pluginIds: readonly string[];
-  /**
-   * Pre-computed discovery result. When supplied, the bundled-fallback path
-   * skips its internal `discoverOpenClawPlugins` call so callers sharing a
-   * discovery snapshot across registry helpers avoid redundant filesystem
-   * walks.
-   */
   discovery?: PluginDiscoveryResult;
 }): ReadonlyMap<string, PluginConfigContractMetadata> {
   const matches = new Map<string, PluginConfigContractMetadata>();
diff --git a/src/plugins/installed-plugin-index-types.ts b/src/plugins/installed-plugin-index-types.ts
@@ -130,12 +130,6 @@ export type LoadInstalledPluginIndexParams = {
   installRecords?: Record<string, PluginInstallRecord>;
   candidates?: PluginCandidate[];
   diagnostics?: PluginDiagnostic[];
-  /**
-   * Pre-computed discovery result. When supplied (and `candidates` is not),
-   * the internal `discoverOpenClawPlugins` call is skipped. Callers sharing a
-   * discovery snapshot across registry helpers in the same flow should supply
-   * this to avoid redundant filesystem walks.
-   */
   discovery?: PluginDiscoveryResult;
   now?: () => Date;
 };
diff --git a/src/plugins/loader.ts b/src/plugins/loader.ts
@@ -202,13 +202,6 @@ export type PluginLoadOptions = {
   loadModules?: boolean;
   throwOnLoadError?: boolean;
   manifestRegistry?: PluginManifestRegistry;
-  /**
-   * Pre-computed plugin discovery result. When supplied, internal calls to
-   * `discoverOpenClawPlugins` are skipped. Callers in the same startup flow
-   * can compute one discovery result and share it across loader entry points
-   * to eliminate redundant filesystem walks. Ignored when `manifestRegistry`
-   * is also provided (the registry already implies a discovery snapshot).
-   */
   discovery?: PluginDiscoveryResult;
 };
 
diff --git a/src/plugins/manifest-registry.ts b/src/plugins/manifest-registry.ts
@@ -920,12 +920,6 @@ export function loadPluginManifestRegistry(
     diagnostics?: PluginDiagnostic[];
     installRecords?: Record<string, PluginInstallRecord>;
     bundledChannelConfigCollector?: BundledChannelConfigCollector;
-    /**
-     * Pre-computed discovery result. When supplied (and `candidates` is not),
-     * the internal `discoverOpenClawPlugins` call is skipped. Callers sharing
-     * a discovery snapshot across multiple registry helpers in the same flow
-     * should supply this to avoid redundant filesystem walks.
-     */
     discovery?: PluginDiscoveryResult;
   } = {},
 ): PluginManifestRegistry {
