Skip to content

fix: harden hook fallback and MCP node resolution#1301

Open
Shockang wants to merge 4 commits intothedotmack:mainfrom
Shockang:fix/openclaw-troubleshooting
Open

fix: harden hook fallback and MCP node resolution#1301
Shockang wants to merge 4 commits intothedotmack:mainfrom
Shockang:fix/openclaw-troubleshooting

Conversation

@Shockang
Copy link

@Shockang Shockang commented Mar 8, 2026

Summary

  • prefer installed cache plugin paths in plugin/hooks/hooks.json before falling back to the marketplace source path
  • resolve the Node.js executable for MCP startup from CLAUDE_MEM_NODE_PATH, common install locations, or PATH instead of hardcoding node
  • add regression tests for the hook fallback and runtime resolver changes

Issue Mapping

Test Plan

  • bun test tests/infrastructure/plugin-distribution.test.ts tests/infrastructure/process-manager.test.ts tests/worker-spawn.test.ts tests/hook-lifecycle.test.ts tests/server/server.test.ts
  • npm run build
  • shell-validated the new hooks.json fallback snippet against both cache-present and cache-missing HOME layouts

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fix(#1215): PR #1229 fallback path points to marketplace source, not installed cache — hooks still fail when CLAUDE_PLUGIN_ROOT unset

1 participant