skillpull 0.4.0 → 0.4.4

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "skillpull",
-  "version": "0.4.0",
+  "version": "0.4.4",
   "description": "Sync AI agent skills from Git repositories to Claude, Codex, Kiro, and Cursor",
   "bin": {
     "skillpull": "./skillpull"

package/skillpull

@@ -1,7 +1,7 @@
 #!/usr/bin/env bash
 set -euo pipefail
 
-VERSION="0.4.0"
+VERSION="0.4.4"
 MANIFEST_FILE=".skillpull.json"
 TMPDIR_PREFIX="skillpull"
 CONFIG_DIR="$HOME/.config/skillpull"
@@ -60,6 +60,20 @@ sedi() {
 # Sets SELECTED_ITEMS=("item1" "item3") with user's choices
 SELECTED_ITEMS=()
 
+_read_key() {
+  local old_settings; old_settings="$(stty -g 2>/dev/null)" || true
+  stty -icanon -echo min 1 time 0 2>/dev/null || true
+  local char; char="$(dd bs=1 count=1 2>/dev/null)"
+  # Check for escape sequence
+  if [[ "$char" == $'\033' ]]; then
+    stty -icanon -echo min 0 time 1 2>/dev/null || true
+    local seq; seq="$(dd bs=2 count=1 2>/dev/null)"
+    char="${char}${seq}"
+  fi
+  [[ -n "$old_settings" ]] && stty "$old_settings" 2>/dev/null || true
+  printf '%s' "$char"
+}
+
 select_menu() {
   local prompt="$1"; shift
   local items=("$@")
@@ -75,15 +89,12 @@ select_menu() {
 
   SELECTED_ITEMS=()
 
-  # Save terminal state
-  local old_stty; old_stty="$(stty -g 2>/dev/null)" || true
-
   # Hide cursor
   printf '\033[?25l' >&2
 
   # Print prompt
   printf "\n  ${CYAN}%s${RESET}\n" "$prompt" >&2
-  printf "  ${DIM}↑↓ move  Space select  Enter confirm${RESET}\n\n" >&2
+  printf "  ${DIM}j/k move  Space select  Enter confirm${RESET}\n\n" >&2
 
   # Draw initial list
   for ((i=0; i<count; i++)); do
@@ -98,17 +109,7 @@ select_menu() {
 
   # Input loop
   while true; do
-    # Read single key
-    local key=""
-    IFS= read -rsn1 key 2>/dev/null || true
-
-    # Handle escape sequences (arrow keys)
-    if [[ "$key" == $'\033' ]]; then
-      local seq1="" seq2=""
-      IFS= read -rsn1 -t 0.1 seq1 2>/dev/null || true
-      IFS= read -rsn1 -t 0.1 seq2 2>/dev/null || true
-      key="${key}${seq1}${seq2}"
-    fi
+    local key; key="$(_read_key)"
 
     case "$key" in
       $'\033[A'|k) # Up
@@ -124,7 +125,7 @@ select_menu() {
           selected="${selected:0:$cursor}0${selected:$((cursor+1))}"
         fi
         ;;
-      '') # Enter - confirm
+      ''|$'\n'|$'\r') # Enter - confirm
         break
         ;;
     esac
@@ -147,9 +148,6 @@ select_menu() {
   # Show cursor
   printf '\033[?25h' >&2
 
-  # Restore terminal
-  [[ -n "$old_stty" ]] && stty "$old_stty" 2>/dev/null || true
-
   # Collect selected items
   for ((i=0; i<count; i++)); do
     if [[ "${selected:$i:1}" == "1" ]]; then
@@ -479,28 +477,6 @@ cmd_pull() {
       return 1
     fi
     skills=("${matched[@]}")
-  elif [[ ${#skills[@]} -gt 1 && -t 0 ]]; then
-    # Multiple skills, no filter, interactive terminal -> let user choose
-    local names=()
-    for sd in "${skills[@]}"; do
-      names+=("$(skill_display_name "$sd")")
-    done
-    select_menu "Select skills to install:" "${names[@]}"
-    if [[ ${#SELECTED_ITEMS[@]} -eq 0 ]]; then
-      warn "No skills selected"
-      return 0
-    fi
-    local matched=()
-    for sd in "${skills[@]}"; do
-      local sn; sn="$(skill_display_name "$sd")"
-      for sel in "${SELECTED_ITEMS[@]}"; do
-        if [[ "$sn" == "$sel" ]]; then
-          matched+=("$sd")
-          break
-        fi
-      done
-    done
-    skills=("${matched[@]}")
   fi
 
   mkdir -p "$target_dir"
@@ -951,7 +927,11 @@ cmd_alias() {
 }
 
 cmd_search() {
-  local keyword="$1"
+  local keyword="${1:-}"
+  if [[ -z "$keyword" && -t 0 ]]; then
+    printf "  Search keyword: "
+    read -r keyword
+  fi
   [[ -z "$keyword" ]] && { err "Usage: skillpull search <keyword>"; return 1; }
 
   dim "Searching GitHub for '$keyword' ..."
@@ -1049,7 +1029,7 @@ HELP
 # ── Argument parsing & dispatch ──
 main() {
   local cmd="" repo_url="" skill_name="" custom_path=""
-  local force=0 dry_run=0 is_global=0 use_all=0
+  local force=0 dry_run=0 is_global=0 use_all=0 agent_explicit=0
   local agents=()
   local alias_args=()
   QUIET=0; BRANCH=""
@@ -1087,11 +1067,11 @@ main() {
       --force|-f)   force=1; shift ;;
       --dry-run)    dry_run=1; shift ;;
       --quiet|-q)   QUIET=1; shift ;;
-      --claude)     agents+=("claude"); shift ;;
-      --codex)      agents+=("codex"); shift ;;
-      --kiro)       agents+=("kiro"); shift ;;
-      --cursor)     agents+=("cursor"); shift ;;
-      --all)        use_all=1; shift ;;
+      --claude)     agents+=("claude"); agent_explicit=1; shift ;;
+      --codex)      agents+=("codex"); agent_explicit=1; shift ;;
+      --kiro)       agents+=("kiro"); agent_explicit=1; shift ;;
+      --cursor)     agents+=("cursor"); agent_explicit=1; shift ;;
+      --all)        use_all=1; agent_explicit=1; shift ;;
       list)         cmd="list"; shift ;;
       installed)    cmd="installed"; shift ;;
       remove)       cmd="remove"; shift ;;
@@ -1132,7 +1112,20 @@ main() {
   if [[ "$use_all" == "1" ]]; then
     agents=("claude" "codex" "kiro" "cursor")
   elif [[ ${#agents[@]} -eq 0 ]]; then
-    agents=("$DEFAULT_AGENT")
+    if [[ "$agent_explicit" == "0" && -t 0 && "$cmd" != "installed" && "$cmd" != "update" ]]; then
+      # Interactive terminal, no agent flag -> let user choose
+      select_menu "Install to which tools?" "claude (.claude/skills)" "codex (.codex/skills)" "kiro (.kiro/skills)" "cursor (.cursor/rules)"
+      if [[ ${#SELECTED_ITEMS[@]} -eq 0 ]]; then
+        warn "No tools selected"
+        exit 0
+      fi
+      for sel in "${SELECTED_ITEMS[@]}"; do
+        local agent_name; agent_name="$(echo "$sel" | awk '{print $1}')"
+        agents+=("$agent_name")
+      done
+    else
+      agents=("$DEFAULT_AGENT")
+    fi
   fi
 
   case "${cmd:-}" in
@@ -1160,7 +1153,15 @@ main() {
       done
       ;;
     list)
-      [[ -z "$repo_url" ]] && { err "Missing source"; usage; exit 1; }
+      if [[ -z "$repo_url" ]]; then
+        local reg; reg="$(read_config_key "registry")"
+        if [[ -z "$reg" ]]; then
+          err "No source specified and no registry set."
+          err "Run 'skillpull init' to set a default skill repo."
+          exit 1
+        fi
+        repo_url="$reg"
+      fi
       local resolved; resolved="$(resolve_repo_url "$repo_url")" || exit 1
       cmd_list "$resolved"
       ;;