Merge pull request 'fix(plugins): remove broken mcpServers references that broke plugin loading' (#325) from fix/remove-broken-mcp-references into development

Reviewed-on: #325

.claude-plugin/marketplace.json

@@ -20,7 +20,6 @@
       },
       "homepage": "https://gitea.hotserv.cloud/personal-projects/leo-claude-mktplace/src/branch/main/plugins/projman/README.md",
       "repository": "https://gitea.hotserv.cloud/personal-projects/leo-claude-mktplace.git",
-      "mcpServers": ["./.mcp.json"],
       "category": "development",
       "tags": ["sprint", "agile", "gitea", "project-management"],
       "license": "MIT"
@@ -84,7 +83,6 @@
       },
       "homepage": "https://gitea.hotserv.cloud/personal-projects/leo-claude-mktplace/src/branch/main/plugins/cmdb-assistant/README.md",
       "repository": "https://gitea.hotserv.cloud/personal-projects/leo-claude-mktplace.git",
-      "mcpServers": ["./.mcp.json"],
       "category": "infrastructure",
       "tags": ["cmdb", "netbox", "dcim", "ipam", "data-quality", "validation"],
       "license": "MIT"
@@ -145,7 +143,6 @@
       },
       "homepage": "https://gitea.hotserv.cloud/personal-projects/leo-claude-mktplace/src/branch/main/plugins/pr-review/README.md",
       "repository": "https://gitea.hotserv.cloud/personal-projects/leo-claude-mktplace.git",
-      "mcpServers": ["./.mcp.json"],
       "category": "development",
       "tags": ["code-review", "pull-requests", "security", "quality"],
       "license": "MIT"
@@ -161,7 +158,6 @@
       },
       "homepage": "https://gitea.hotserv.cloud/personal-projects/leo-claude-mktplace/src/branch/main/plugins/data-platform/README.md",
       "repository": "https://gitea.hotserv.cloud/personal-projects/leo-claude-mktplace.git",
-      "mcpServers": ["./.mcp.json"],
       "category": "data",
       "tags": ["pandas", "postgresql", "postgis", "dbt", "data-engineering", "etl"],
       "license": "MIT"
@@ -177,7 +173,6 @@
       },
       "homepage": "https://gitea.hotserv.cloud/personal-projects/leo-claude-mktplace/src/branch/main/plugins/viz-platform/README.md",
       "repository": "https://gitea.hotserv.cloud/personal-projects/leo-claude-mktplace.git",
-      "mcpServers": ["./.mcp.json"],
       "category": "visualization",
       "tags": ["dash", "plotly", "mantine", "charts", "dashboards", "theming", "dmc"],
       "license": "MIT"
@@ -193,7 +188,6 @@
       },
       "homepage": "https://gitea.hotserv.cloud/personal-projects/leo-claude-mktplace/src/branch/main/plugins/contract-validator/README.md",
       "repository": "https://gitea.hotserv.cloud/personal-projects/leo-claude-mktplace.git",
-      "mcpServers": ["./.mcp.json"],
       "category": "development",
       "tags": ["validation", "contracts", "compatibility", "agents", "interfaces", "cross-plugin"],
       "license": "MIT"

.mcp.json

@@ -0,0 +1,24 @@
+{
+  "mcpServers": {
+    "gitea": {
+      "command": "/home/lmiranda/.claude/plugins/marketplaces/leo-claude-mktplace/mcp-servers/gitea/run.sh",
+      "args": []
+    },
+    "netbox": {
+      "command": "/home/lmiranda/.claude/plugins/marketplaces/leo-claude-mktplace/mcp-servers/netbox/run.sh",
+      "args": []
+    },
+    "viz-platform": {
+      "command": "/home/lmiranda/.claude/plugins/marketplaces/leo-claude-mktplace/mcp-servers/viz-platform/run.sh",
+      "args": []
+    },
+    "data-platform": {
+      "command": "/home/lmiranda/.claude/plugins/marketplaces/leo-claude-mktplace/mcp-servers/data-platform/run.sh",
+      "args": []
+    },
+    "contract-validator": {
+      "command": "/home/lmiranda/.claude/plugins/marketplaces/leo-claude-mktplace/mcp-servers/contract-validator/run.sh",
+      "args": []
+    }
+  }
+}

CLAUDE.md

@@ -108,7 +108,7 @@ leo-claude-mktplace/
 ├── plugins/
 │   ├── projman/                  # Sprint management
 │   │   ├── .claude-plugin/plugin.json
-│   │   ├── .mcp.json
+│   │   # .mcp.json removed - now at marketplace root
 │   │   ├── mcp-servers/gitea -> ../../../mcp-servers/gitea  # SYMLINK
 │   │   ├── commands/             # 14 commands (incl. setup, debug, suggest-version)
 │   │   ├── hooks/                # SessionStart: mismatch detection + sprint suggestions
@@ -120,7 +120,7 @@ leo-claude-mktplace/
 │   │   └── agents/
 │   ├── pr-review/                # Multi-agent PR review
 │   │   ├── .claude-plugin/plugin.json
-│   │   ├── .mcp.json
+│   │   # .mcp.json removed - now at marketplace root
 │   │   ├── mcp-servers/gitea -> ../../../mcp-servers/gitea  # SYMLINK
 │   │   ├── commands/             # 6 commands (incl. setup)
 │   │   ├── hooks/                # SessionStart mismatch detection
@@ -131,14 +131,14 @@ leo-claude-mktplace/
 │   │   └── agents/
 │   ├── data-platform/            # Data engineering (NEW v4.0.0)
 │   │   ├── .claude-plugin/plugin.json
-│   │   ├── .mcp.json
+│   │   # .mcp.json removed - now at marketplace root
 │   │   ├── mcp-servers/          # pandas, postgresql, dbt MCPs
 │   │   ├── commands/             # 7 commands
 │   │   ├── hooks/                # SessionStart PostgreSQL check
 │   │   └── agents/               # 2 agents
 │   ├── viz-platform/             # Visualization (NEW v4.0.0)
 │   │   ├── .claude-plugin/plugin.json
-│   │   ├── .mcp.json
+│   │   # .mcp.json removed - now at marketplace root
 │   │   ├── mcp-servers/          # viz-platform MCP
 │   │   ├── commands/             # 7 commands
 │   │   ├── hooks/                # SessionStart DMC check

mcp-servers/gitea/mcp_server/server.py

@@ -144,7 +144,7 @@ class GiteaMCPServer:
                         "type": "object",
                         "properties": {
                             "issue_number": {
-                                "type": "integer",
+                                "type": ["integer", "string"],
                                 "description": "Issue number"
                             },
                             "repo": {
@@ -189,7 +189,7 @@ class GiteaMCPServer:
                         "type": "object",
                         "properties": {
                             "issue_number": {
-                                "type": "integer",
+                                "type": ["integer", "string"],
                                 "description": "Issue number"
                             },
                             "title": {
@@ -211,7 +211,7 @@ class GiteaMCPServer:
                                 "description": "New labels"
                             },
                             "milestone": {
-                                "type": "integer",
+                                "type": ["integer", "string"],
                                 "description": "Milestone ID to assign"
                             },
                             "repo": {
@@ -229,7 +229,7 @@ class GiteaMCPServer:
                         "type": "object",
                         "properties": {
                             "issue_number": {
-                                "type": "integer",
+                                "type": ["integer", "string"],
                                 "description": "Issue number"
                             },
                             "comment": {
@@ -424,7 +424,7 @@ class GiteaMCPServer:
                                 "description": "Tags to filter by (optional)"
                             },
                             "limit": {
-                                "type": "integer",
+                                "type": ["integer", "string"],
                                 "default": 20,
                                 "description": "Maximum results"
                             },
@@ -462,7 +462,7 @@ class GiteaMCPServer:
                         "type": "object",
                         "properties": {
                             "milestone_id": {
-                                "type": "integer",
+                                "type": ["integer", "string"],
                                 "description": "Milestone ID"
                             },
                             "repo": {
@@ -506,7 +506,7 @@ class GiteaMCPServer:
                         "type": "object",
                         "properties": {
                             "milestone_id": {
-                                "type": "integer",
+                                "type": ["integer", "string"],
                                 "description": "Milestone ID"
                             },
                             "title": {
@@ -541,7 +541,7 @@ class GiteaMCPServer:
                         "type": "object",
                         "properties": {
                             "milestone_id": {
-                                "type": "integer",
+                                "type": ["integer", "string"],
                                 "description": "Milestone ID"
                             },
                             "repo": {
@@ -560,7 +560,7 @@ class GiteaMCPServer:
                         "type": "object",
                         "properties": {
                             "issue_number": {
-                                "type": "integer",
+                                "type": ["integer", "string"],
                                 "description": "Issue number"
                             },
                             "repo": {
@@ -578,11 +578,11 @@ class GiteaMCPServer:
                         "type": "object",
                         "properties": {
                             "issue_number": {
-                                "type": "integer",
+                                "type": ["integer", "string"],
                                 "description": "Issue that will depend on another"
                             },
                             "depends_on": {
-                                "type": "integer",
+                                "type": ["integer", "string"],
                                 "description": "Issue that blocks issue_number"
                             },
                             "repo": {
@@ -600,11 +600,11 @@ class GiteaMCPServer:
                         "type": "object",
                         "properties": {
                             "issue_number": {
-                                "type": "integer",
+                                "type": ["integer", "string"],
                                 "description": "Issue that depends on another"
                             },
                             "depends_on": {
-                                "type": "integer",
+                                "type": ["integer", "string"],
                                 "description": "Issue being depended on"
                             },
                             "repo": {
@@ -782,7 +782,7 @@ class GiteaMCPServer:
                         "type": "object",
                         "properties": {
                             "pr_number": {
-                                "type": "integer",
+                                "type": ["integer", "string"],
                                 "description": "Pull request number"
                             },
                             "repo": {
@@ -800,7 +800,7 @@ class GiteaMCPServer:
                         "type": "object",
                         "properties": {
                             "pr_number": {
-                                "type": "integer",
+                                "type": ["integer", "string"],
                                 "description": "Pull request number"
                             },
                             "repo": {
@@ -818,7 +818,7 @@ class GiteaMCPServer:
                         "type": "object",
                         "properties": {
                             "pr_number": {
-                                "type": "integer",
+                                "type": ["integer", "string"],
                                 "description": "Pull request number"
                             },
                             "repo": {
@@ -836,7 +836,7 @@ class GiteaMCPServer:
                         "type": "object",
                         "properties": {
                             "pr_number": {
-                                "type": "integer",
+                                "type": ["integer", "string"],
                                 "description": "Pull request number"
                             },
                             "body": {
@@ -855,7 +855,7 @@ class GiteaMCPServer:
                                     "type": "object",
                                     "properties": {
                                         "path": {"type": "string"},
-                                        "position": {"type": "integer"},
+                                        "position": {"type": ["integer", "string"]},
                                         "body": {"type": "string"}
                                     }
                                 },
@@ -876,7 +876,7 @@ class GiteaMCPServer:
                         "type": "object",
                         "properties": {
                             "pr_number": {
-                                "type": "integer",
+                                "type": ["integer", "string"],
                                 "description": "Pull request number"
                             },
                             "body": {

plugins/cmdb-assistant/.claude-plugin/plugin.json

@@ -19,6 +19,5 @@
     "data-quality",
     "validation"
   ],
-  "commands": ["./commands/"],
-  "mcpServers": ["./.mcp.json"]
+  "commands": ["./commands/"]
 }

plugins/cmdb-assistant/.mcp.json

@@ -1,8 +0,0 @@
-{
-  "mcpServers": {
-    "netbox": {
-      "command": "${CLAUDE_PLUGIN_ROOT}/mcp-servers/netbox/run.sh",
-      "args": []
-    }
-  }
-}

plugins/contract-validator/.claude-plugin/plugin.json

@@ -18,6 +18,5 @@
     "interfaces",
     "cross-plugin"
   ],
-  "commands": ["./commands/"],
-  "mcpServers": ["./.mcp.json"]
+  "commands": ["./commands/"]
 }

plugins/contract-validator/.mcp.json

@@ -1,9 +0,0 @@
-{
-  "mcpServers": {
-    "contract-validator": {
-      "type": "stdio",
-      "command": "${CLAUDE_PLUGIN_ROOT}/mcp-servers/contract-validator/run.sh",
-      "args": []
-    }
-  }
-}

plugins/data-platform/.claude-plugin/plugin.json

@@ -19,6 +19,5 @@
     "etl",
     "dataframe"
   ],
-  "commands": ["./commands/"],
-  "mcpServers": ["./.mcp.json"]
+  "commands": ["./commands/"]
 }

plugins/data-platform/.mcp.json

@@ -1,9 +0,0 @@
-{
-  "mcpServers": {
-    "data-platform": {
-      "type": "stdio",
-      "command": "${CLAUDE_PLUGIN_ROOT}/mcp-servers/data-platform/run.sh",
-      "args": []
-    }
-  }
-}

plugins/doc-guardian/hooks/notify.sh

@@ -1,11 +1,12 @@
 #!/bin/bash
 # doc-guardian notification hook
 # Tracks documentation dependencies and queues updates
-# This is a command hook - guaranteed not to block workflow
 #
-# IMPORTANT: Output is purely informational - uses passive language
-# to avoid triggering Claude to seek user confirmation.
-# Run /doc-sync to process the queue when ready.
+# SILENT BY DEFAULT - No output to avoid interrupting Claude's workflow.
+# Changes are queued to .doc-guardian-queue for later processing.
+# Run /doc-sync or /doc-audit to see and process pending updates.
+#
+# Set DOC_GUARDIAN_VERBOSE=1 to enable notification output.
 
 # Read tool input from stdin (JSON with file_path)
 INPUT=$(cat)
@@ -48,40 +49,26 @@ DEPENDENT_DOCS="${DOC_DEPS[$MODIFIED_TYPE]}"
 # Queue file for tracking pending updates
 QUEUE_FILE="${CLAUDE_PROJECT_ROOT:-.}/.doc-guardian-queue"
 
-# Debounce: skip notification if same type was logged in last 5 seconds
-# This prevents 4+ rapid notifications during batch edits
-DEBOUNCE_SECONDS=5
+# Add to queue (always, for deduplication we check file+type combo)
+# Format: timestamp | type | file_path | dependent_docs
+QUEUE_ENTRY="$(date +%Y-%m-%dT%H:%M:%S) | $MODIFIED_TYPE | $FILE_PATH | $DEPENDENT_DOCS"
+
+# Check if this exact file+type combo already exists in queue (dedup)
 if [ -f "$QUEUE_FILE" ]; then
-    LAST_ENTRY=$(tail -1 "$QUEUE_FILE" 2>/dev/null || true)
-    LAST_TYPE=$(echo "$LAST_ENTRY" | cut -d'|' -f2 | tr -d ' ')
-    LAST_TIME=$(echo "$LAST_ENTRY" | cut -d'|' -f1 | tr -d ' ')
-
-    if [ "$LAST_TYPE" = "$MODIFIED_TYPE" ] && [ -n "$LAST_TIME" ]; then
-        # Convert timestamps to seconds for comparison
-        LAST_EPOCH=$(date -d "$LAST_TIME" +%s 2>/dev/null || echo "0")
-        NOW_EPOCH=$(date +%s)
-        DIFF=$((NOW_EPOCH - LAST_EPOCH))
-
-        if [ "$DIFF" -lt "$DEBOUNCE_SECONDS" ]; then
-            # Still add to queue, but skip notification
-            {
-                echo "$(date +%Y-%m-%dT%H:%M:%S) | $MODIFIED_TYPE | $FILE_PATH | $DEPENDENT_DOCS"
-            } >> "$QUEUE_FILE" 2>/dev/null || true
-            exit 0
-        fi
+    if grep -qF "| $MODIFIED_TYPE | $FILE_PATH |" "$QUEUE_FILE" 2>/dev/null; then
+        # Already queued, skip silently
+        exit 0
     fi
 fi
 
-# Add to queue (create if doesn't exist, append if does)
-{
-    echo "$(date +%Y-%m-%dT%H:%M:%S) | $MODIFIED_TYPE | $FILE_PATH | $DEPENDENT_DOCS"
-} >> "$QUEUE_FILE" 2>/dev/null || true
+# Add to queue
+echo "$QUEUE_ENTRY" >> "$QUEUE_FILE" 2>/dev/null || true
 
-# Count pending updates
-PENDING_COUNT=$(wc -l < "$QUEUE_FILE" 2>/dev/null | tr -d ' ' || echo "1")
-
-# Output passive notification (no action implied)
-# Uses "queued" instead of "update needed" to avoid triggering Claude to ask about it
-echo "[doc-guardian] drift queued: $MODIFIED_TYPE → $DEPENDENT_DOCS ($PENDING_COUNT total)"
+# SILENT by default - only output if DOC_GUARDIAN_VERBOSE is set
+# This prevents Claude from stopping to ask about documentation updates
+if [ "${DOC_GUARDIAN_VERBOSE:-0}" = "1" ]; then
+    PENDING_COUNT=$(wc -l < "$QUEUE_FILE" 2>/dev/null | tr -d ' ' || echo "1")
+    echo "[doc-guardian] queued: $MODIFIED_TYPE ($PENDING_COUNT pending)"
+fi
 
 exit 0

plugins/pr-review/.claude-plugin/plugin.json

@@ -17,6 +17,5 @@
     "performance",
     "multi-agent"
   ],
-  "commands": ["./commands/"],
-  "mcpServers": ["./.mcp.json"]
+  "commands": ["./commands/"]
 }

plugins/pr-review/.mcp.json

@@ -1,8 +0,0 @@
-{
-  "mcpServers": {
-    "gitea": {
-      "command": "${CLAUDE_PLUGIN_ROOT}/mcp-servers/gitea/run.sh",
-      "args": []
-    }
-  }
-}

plugins/projman/.claude-plugin/plugin.json

@@ -17,6 +17,5 @@
     "agile",
     "lessons-learned"
   ],
-  "commands": ["./commands/"],
-  "mcpServers": ["./.mcp.json"]
+  "commands": ["./commands/"]
 }

plugins/projman/.mcp.json

@@ -1,8 +0,0 @@
-{
-  "mcpServers": {
-    "gitea": {
-      "command": "${CLAUDE_PLUGIN_ROOT}/mcp-servers/gitea/run.sh",
-      "args": []
-    }
-  }
-}

plugins/projman/commands/clear-cache.md

@@ -0,0 +1,24 @@
+# /clear-cache
+
+Clear plugin cache to force fresh configuration reload. Run this after marketplace updates.
+
+## When to Use
+
+- After updating the marketplace (`git pull` or reinstall)
+- When MCP servers show stale configuration
+- When plugin changes don't take effect
+
+## What It Does
+
+1. Clears `~/.claude/plugins/cache/leo-claude-mktplace/`
+2. Forces Claude Code to re-read `.mcp.json` files on next session
+
+## Instructions
+
+Run this command, then **restart your Claude Code session** for changes to take effect.
+
+```bash
+rm -rf ~/.claude/plugins/cache/leo-claude-mktplace/
+```
+
+After clearing, inform the user: "Cache cleared. Restart Claude Code for changes to take effect."

plugins/projman/hooks/startup-check.sh

@@ -10,19 +10,6 @@ PLUGIN_ROOT="${CLAUDE_PLUGIN_ROOT:-$(dirname "$(dirname "$(realpath "$0")")")}"
 # Marketplace root is 2 levels up from plugin root (plugins/projman -> .)
 MARKETPLACE_ROOT="$(dirname "$(dirname "$PLUGIN_ROOT")")"
 VENV_REPAIR_SCRIPT="$MARKETPLACE_ROOT/scripts/venv-repair.sh"
-PLUGIN_CACHE="$HOME/.claude/plugins/cache/leo-claude-mktplace"
-
-# ============================================================================
-# Clear stale plugin cache (MUST run before MCP servers load)
-# ============================================================================
-# The cache at ~/.claude/plugins/cache/ holds versioned .mcp.json files.
-# After marketplace updates, cached configs may point to old paths.
-# Clearing forces Claude to read fresh configs from installed marketplace.
-
-if [[ -d "$PLUGIN_CACHE" ]]; then
-    rm -rf "$PLUGIN_CACHE"
-    # Don't output anything - this should be silent and automatic
-fi
 
 # ============================================================================
 # Auto-repair MCP venvs (runs before other checks)

plugins/viz-platform/.claude-plugin/plugin.json

@@ -20,6 +20,5 @@
     "visualization",
     "dmc"
   ],
-  "commands": ["./commands/"],
-  "mcpServers": ["./.mcp.json"]
+  "commands": ["./commands/"]
 }

plugins/viz-platform/.mcp.json

@@ -1,9 +0,0 @@
-{
-  "mcpServers": {
-    "viz-platform": {
-      "type": "stdio",
-      "command": "${CLAUDE_PLUGIN_ROOT}/mcp-servers/viz-platform/run.sh",
-      "args": []
-    }
-  }
-}

scripts/validate-marketplace.sh

@@ -158,9 +158,106 @@ for plugin_dir in "$PLUGINS_DIR"/*/; do
         echo "WARNING: Missing README.md in $plugin_name/"
     fi
 
+    # CRITICAL: Validate file references exist (mcpServers, hooks, commands)
+    # This prevents broken references that silently break plugin loading
+
+    # Check mcpServers references
+    mcp_servers=$(jq -r '.mcpServers // [] | .[]' "$plugin_json" 2>/dev/null)
+    for mcp_ref in $mcp_servers; do
+        mcp_path="$plugin_dir/$mcp_ref"
+        if [[ ! -f "$mcp_path" ]]; then
+            echo "ERROR: BROKEN REFERENCE in $plugin_name/plugin.json"
+            echo "       mcpServers references '$mcp_ref' but file does not exist at:"
+            echo "       $mcp_path"
+            echo ""
+            echo "       FIX: Either create the file or remove the mcpServers entry"
+            exit 1
+        fi
+        echo "  ✓ mcpServers reference: $mcp_ref exists"
+    done
+
+    # Check hooks references (can be array of file paths OR object with handlers)
+    hooks_type=$(jq -r '.hooks | type' "$plugin_json" 2>/dev/null)
+    if [[ "$hooks_type" == "array" ]]; then
+        # Array format: ["./hooks/hooks.json"]
+        hooks=$(jq -r '.hooks[]' "$plugin_json" 2>/dev/null)
+        for hook_ref in $hooks; do
+            hook_path="$plugin_dir/$hook_ref"
+            if [[ ! -f "$hook_path" ]]; then
+                echo "ERROR: BROKEN REFERENCE in $plugin_name/plugin.json"
+                echo "       hooks references '$hook_ref' but file does not exist at:"
+                echo "       $hook_path"
+                echo ""
+                echo "       FIX: Either create the file or remove the hooks entry"
+                exit 1
+            fi
+            echo "  ✓ hooks reference: $hook_ref exists"
+        done
+    elif [[ "$hooks_type" == "object" ]]; then
+        # Object format: { "PostToolUse": [...] } - inline hooks, no file reference to validate
+        echo "  ✓ hooks: inline object format (no file references)"
+    fi
+
+    # Check commands directory references
+    commands=$(jq -r '.commands // [] | .[]' "$plugin_json" 2>/dev/null)
+    for cmd_ref in $commands; do
+        cmd_path="$plugin_dir/$cmd_ref"
+        if [[ ! -d "$cmd_path" ]] && [[ ! -f "$cmd_path" ]]; then
+            echo "ERROR: BROKEN REFERENCE in $plugin_name/plugin.json"
+            echo "       commands references '$cmd_ref' but path does not exist at:"
+            echo "       $cmd_path"
+            echo ""
+            echo "       FIX: Either create the path or remove the commands entry"
+            exit 1
+        fi
+        echo "  ✓ commands reference: $cmd_ref exists"
+    done
+
     echo "✓ $plugin_name valid"
 done
 
+# CRITICAL: Validate marketplace.json file references
+echo ""
+echo "=== Validating Marketplace File References (CRITICAL) ==="
+
+for i in $(seq 0 $((PLUGIN_COUNT - 1))); do
+    PLUGIN_NAME=$(jq -r ".plugins[$i].name" "$MARKETPLACE_JSON")
+    PLUGIN_SOURCE=$(jq -r ".plugins[$i].source" "$MARKETPLACE_JSON")
+    PLUGIN_DIR="$ROOT_DIR/$PLUGIN_SOURCE"
+
+    # Check mcpServers in marketplace.json
+    mcp_servers=$(jq -r ".plugins[$i].mcpServers // [] | .[]" "$MARKETPLACE_JSON" 2>/dev/null)
+    for mcp_ref in $mcp_servers; do
+        mcp_path="$PLUGIN_DIR/$mcp_ref"
+        if [[ ! -f "$mcp_path" ]]; then
+            echo "ERROR: BROKEN REFERENCE in marketplace.json for $PLUGIN_NAME"
+            echo "       mcpServers references '$mcp_ref' but file does not exist at:"
+            echo "       $mcp_path"
+            echo ""
+            echo "       FIX: Either create the file or remove the mcpServers entry from marketplace.json"
+            exit 1
+        fi
+        echo "✓ $PLUGIN_NAME: mcpServers reference $mcp_ref exists"
+    done
+
+    # Check hooks in marketplace.json
+    hooks=$(jq -r ".plugins[$i].hooks // [] | .[]" "$MARKETPLACE_JSON" 2>/dev/null)
+    for hook_ref in $hooks; do
+        hook_path="$PLUGIN_DIR/$hook_ref"
+        if [[ ! -f "$hook_path" ]]; then
+            echo "ERROR: BROKEN REFERENCE in marketplace.json for $PLUGIN_NAME"
+            echo "       hooks references '$hook_ref' but file does not exist at:"
+            echo "       $hook_path"
+            echo ""
+            echo "       FIX: Either create the file or remove the hooks entry from marketplace.json"
+            exit 1
+        fi
+        echo "✓ $PLUGIN_NAME: hooks reference $hook_ref exists"
+    done
+done
+
+echo "✓ All file references validated"
+
 # v5.4.0: Validate agent model fields
 echo ""
 echo "=== Validating Agent Model Fields (v5.4.0+) ==="