From c97e4a5b692642e8c918f4bf0fa0e3c3a9b626d6 Mon Sep 17 00:00:00 2001 From: Defame1297 Date: Sun, 28 Jun 2026 15:32:07 +0000 Subject: [PATCH] fix(check-manifests): skip remote-sourced plugins in local path check MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit ## Why The hook was treating all source values as local directory paths. Remote sources (github/git/npm objects) have no local directory — the jq -r of a JSON object produced garbage, causing the hook to fail on push after adding the mattpocock-skills remote plugin. Co-authored-by: Claude Sonnet 4.6 --- scripts/check-manifests.sh | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/scripts/check-manifests.sh b/scripts/check-manifests.sh index 131f75f..bdea23c 100755 --- a/scripts/check-manifests.sh +++ b/scripts/check-manifests.sh @@ -23,6 +23,13 @@ plugin_count=$(jq '.plugins | length' "$MARKETPLACE") for ((i = 0; i < plugin_count; i++)); do name=$(jq -r ".plugins[$i].name" "$MARKETPLACE") + source_type=$(jq -r ".plugins[$i].source | type" "$MARKETPLACE") + + # Remote sources (github, git, npm objects) have no local directory to check + if [[ "$source_type" != "string" ]]; then + continue + fi + source=$(jq -r ".plugins[$i].source" "$MARKETPLACE") source="${source#./}" plugin_dir="$REPO_ROOT/$source"