From 146c830189279491a66b79823ec79049fc7bc6d7 Mon Sep 17 00:00:00 2001
From: Steven Tan <cheeyutcy@gmail.com>
Date: Mon, 9 Mar 2026 19:34:04 +0800
Subject: [PATCH] Fix install.sh skill dirs: use append (+=) for Claude like
 other tools

The Claude case used `dirs=(...)` (assignment) instead of `dirs+=(...)`
(append), which would wipe previously added directories if Claude wasn't
the first tool processed. For example, `--tools copilot,claude` would
lose the copilot skills directory.

All other tools (cursor, copilot, codex, gemini) already use `+=`.
---
 install.sh | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/install.sh b/install.sh
index 7ed46779..e04be37b 100755
--- a/install.sh
+++ b/install.sh
@@ -686,7 +686,7 @@ install_skills() {
     # Determine target directories (array so paths with spaces work)
     for tool in $TOOLS; do
         case $tool in
-            claude) dirs=("$base_dir/.claude/skills") ;;
+            claude) dirs+=("$base_dir/.claude/skills") ;;
             cursor) echo "$TOOLS" | grep -q claude || dirs+=("$base_dir/.cursor/skills") ;;
             copilot) dirs+=("$base_dir/.github/skills") ;;
             codex) dirs+=("$base_dir/.agents/skills") ;;