From d59f5867a8ea5f5e3f9874c76d773152d490030c Mon Sep 17 00:00:00 2001 From: yusyus Date: Sun, 26 Oct 2025 17:23:40 +0300 Subject: [PATCH] Fix setup_mcp.sh path issues (Issue #157) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Fixed all incorrect path references in setup_mcp.sh script. ## Issue: setup_mcp.sh was using incorrect paths (mcp/ instead of skill_seeker_mcp/), causing: - ERROR: Could not open requirements file: 'mcp/requirements.txt' - Configuration pointing to non-existent mcp/server.py - All path validations failing ## Root Cause: The MCP server was renamed from 'mcp/' to 'skill_seeker_mcp/' but setup_mcp.sh wasn't updated to reflect the new directory structure. ## Fix: Updated all path references throughout setup_mcp.sh: 1. **Line 44**: mcp/requirements.txt → skill_seeker_mcp/requirements.txt 2. **Line 63**: mcp/server.py → skill_seeker_mcp/server.py 3. **Line 113**: $REPO_PATH/mcp/server.py → $REPO_PATH/skill_seeker_mcp/server.py 4. **Line 154**: $REPO_PATH/mcp/server.py → $REPO_PATH/skill_seeker_mcp/server.py 5. **Line 169-170**: Verification paths updated 6. **Line 232**: Test command updated ## Changes: **Before:** ```bash pip3 install -r mcp/requirements.txt # ❌ File not found timeout 3 python3 mcp/server.py # ❌ File not found "$REPO_PATH/mcp/server.py" # ❌ Wrong path python3 mcp/server.py # ❌ Wrong command ``` **After:** ```bash pip3 install -r skill_seeker_mcp/requirements.txt # ✅ Correct timeout 3 python3 skill_seeker_mcp/server.py # ✅ Correct "$REPO_PATH/skill_seeker_mcp/server.py" # ✅ Correct python3 skill_seeker_mcp/server.py # ✅ Correct ``` ## Verification: - ✅ Script syntax validated (bash -n) - ✅ All 6 path references updated - ✅ File exists at skill_seeker_mcp/requirements.txt - ✅ File exists at skill_seeker_mcp/server.py Fixes #157 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude --- setup_mcp.sh | 16 ++++++++-------- 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/setup_mcp.sh b/setup_mcp.sh index 092d5cc..0f91014 100755 --- a/setup_mcp.sh +++ b/setup_mcp.sh @@ -41,7 +41,7 @@ echo "" if [[ $REPLY =~ ^[Yy]$ ]]; then echo "Installing MCP server dependencies..." - pip3 install -r mcp/requirements.txt || { + pip3 install -r skill_seeker_mcp/requirements.txt || { echo -e "${RED}❌ Failed to install MCP dependencies${NC}" exit 1 } @@ -60,7 +60,7 @@ echo "" # Step 4: Test MCP server echo "Step 4: Testing MCP server..." -timeout 3 python3 mcp/server.py 2>/dev/null || { +timeout 3 python3 skill_seeker_mcp/server.py 2>/dev/null || { if [ $? -eq 124 ]; then echo -e "${GREEN}✓${NC} MCP server starts correctly (timeout expected)" else @@ -110,7 +110,7 @@ echo " \"mcpServers\": {" echo " \"skill-seeker\": {" echo " \"command\": \"python3\"," echo " \"args\": [" -echo " \"$REPO_PATH/mcp/server.py\"" +echo " \"$REPO_PATH/skill_seeker_mcp/server.py\"" echo " ]," echo " \"cwd\": \"$REPO_PATH\"" echo " }" @@ -151,7 +151,7 @@ if [[ $REPLY =~ ^[Yy]$ ]]; then "skill-seeker": { "command": "python3", "args": [ - "$REPO_PATH/mcp/server.py" + "$REPO_PATH/skill_seeker_mcp/server.py" ], "cwd": "$REPO_PATH" } @@ -166,10 +166,10 @@ EOF echo "" # Verify the path exists - if [ -f "$REPO_PATH/mcp/server.py" ]; then - echo -e "${GREEN}✓${NC} Verified: MCP server file exists at $REPO_PATH/mcp/server.py" + if [ -f "$REPO_PATH/skill_seeker_mcp/server.py" ]; then + echo -e "${GREEN}✓${NC} Verified: MCP server file exists at $REPO_PATH/skill_seeker_mcp/server.py" else - echo -e "${RED}❌ Warning: MCP server not found at $REPO_PATH/mcp/server.py${NC}" + echo -e "${RED}❌ Warning: MCP server not found at $REPO_PATH/skill_seeker_mcp/server.py${NC}" echo "Please check the path!" fi else @@ -229,7 +229,7 @@ echo " • Full docs: ${YELLOW}README.md${NC}" echo "" echo "Troubleshooting:" echo " • Check logs: ~/Library/Logs/Claude Code/ (macOS)" -echo " • Test server: python3 mcp/server.py" +echo " • Test server: python3 skill_seeker_mcp/server.py" echo " • Run tests: python3 -m pytest tests/test_mcp_server.py -v" echo "" echo "Happy skill creating! 🚀"