From ba1dee45536ca38866f3668d6380c85433b44ded Mon Sep 17 00:00:00 2001 From: lmiranda Date: Mon, 2 Feb 2026 21:56:43 -0500 Subject: [PATCH] fix(hooks): correct venv path in startup-check scripts The startup hooks were looking for MCP venvs relative to the plugin directory instead of the marketplace root, causing false "venv missing" errors on every session start. Co-Authored-By: Claude Opus 4.5 --- plugins/data-platform/hooks/startup-check.sh | 4 +++- plugins/pr-review/hooks/startup-check.sh | 4 +++- 2 files changed, 6 insertions(+), 2 deletions(-) diff --git a/plugins/data-platform/hooks/startup-check.sh b/plugins/data-platform/hooks/startup-check.sh index 0a7dbf3..a0ee74c 100755 --- a/plugins/data-platform/hooks/startup-check.sh +++ b/plugins/data-platform/hooks/startup-check.sh @@ -7,7 +7,9 @@ PREFIX="[data-platform]" # Check if MCP venv exists PLUGIN_ROOT="${CLAUDE_PLUGIN_ROOT:-$(dirname "$(dirname "$(realpath "$0")")")}" -VENV_PATH="$PLUGIN_ROOT/mcp-servers/data-platform/.venv/bin/python" +# MCP servers are at marketplace root, not inside plugin +MARKETPLACE_ROOT="$(dirname "$(dirname "$PLUGIN_ROOT")")" +VENV_PATH="$MARKETPLACE_ROOT/mcp-servers/data-platform/.venv/bin/python" if [[ ! -f "$VENV_PATH" ]]; then echo "$PREFIX MCP venv missing - run /initial-setup or setup.sh" diff --git a/plugins/pr-review/hooks/startup-check.sh b/plugins/pr-review/hooks/startup-check.sh index 89e184d..c53fb16 100755 --- a/plugins/pr-review/hooks/startup-check.sh +++ b/plugins/pr-review/hooks/startup-check.sh @@ -7,7 +7,9 @@ PREFIX="[pr-review]" # Check if MCP venv exists PLUGIN_ROOT="${CLAUDE_PLUGIN_ROOT:-$(dirname "$(dirname "$(realpath "$0")")")}" -VENV_PATH="$PLUGIN_ROOT/mcp-servers/gitea/.venv/bin/python" +# MCP servers are at marketplace root, not inside plugin +MARKETPLACE_ROOT="$(dirname "$(dirname "$PLUGIN_ROOT")")" +VENV_PATH="$MARKETPLACE_ROOT/mcp-servers/gitea/.venv/bin/python" if [[ ! -f "$VENV_PATH" ]]; then echo "$PREFIX MCP venvs missing - run setup.sh from installed marketplace"