Greptile Summary

This PR fixes a process-global plugin registry corruption that could occur in subagent flows (issue #52250). By passing activate: false and cache: false to loadOpenClawPlugins inside resolvePluginTools, the tool-resolution path is now a side-effect-free "snapshot" load — it no longer calls setActivePluginRegistry, so it cannot overwrite the registry that was previously established by a real plugin activation (e.g., the one that registered the feishu channel).

• Root cause fix (src/plugins/tools.ts): resolvePluginTools now uses activate: false, cache: false, consistent with the same pattern already used in provider-runtime.ts, contracts/registry.ts, and web-search-providers.runtime.ts.
• Test update (src/plugins/tools.optional.test.ts): Two existing assertions are tightened to verify that both new options are forwarded to loadOpenClawPlugins.
• The loader itself enforces at runtime that activate: false must be paired with cache: false (throws otherwise), so the two options are correctly paired here.

Confidence Score: 5/5

• Safe to merge — minimal, well-targeted change with matching test coverage and no observable regressions.
• The fix is a two-line addition that aligns exactly with the existing activate: false / cache: false snapshot-load pattern already present in multiple other plugin subsystems. The loader enforces this pairing at runtime, so the options cannot drift. Tests are updated to assert the new contract, and the PR description includes a reproduction fixture confirming the bug is resolved.
• No files require special attention.

_{Reviews (1): Last reviewed commit: "fix(plugins): avoid activating registry ..." | Re-trigger Greptile}

Merged via squash.

• Prepared head SHA: 55982a6be69664b3783857fcf9b2447e15114e28
• Merge commit: dc21e3bb1e10224f8398e30f5b9d8e566e5a24b6

Thanks @PerfectPan!