skillpull 0.3.3 → 0.4.0

package/README.md

@@ -115,6 +115,12 @@ description: What this skill does
 Skill content here...
 ```
 
+## Requirements
+
+- macOS, Linux, or Windows (via [WSL](https://learn.microsoft.com/en-us/windows/wsl/) or [Git Bash](https://gitforwindows.org/))
+- Git
+- Bash 3+
+
 ## License
 
 MIT

package/package.json

@@ -1,10 +1,14 @@
 {
   "name": "skillpull",
-  "version": "0.3.3",
+  "version": "0.4.0",
   "description": "Sync AI agent skills from Git repositories to Claude, Codex, Kiro, and Cursor",
   "bin": {
     "skillpull": "./skillpull"
   },
+  "scripts": {
+    "postinstall": "echo '✓ skillpull installed. Run: skillpull init'",
+    "preuninstall": "echo '✓ skillpull uninstalled.'"
+  },
   "keywords": [
     "ai",
     "skills",

package/skillpull

@@ -1,7 +1,7 @@
 #!/usr/bin/env bash
 set -euo pipefail
 
-VERSION="0.3.3"
+VERSION="0.4.0"
 MANIFEST_FILE=".skillpull.json"
 TMPDIR_PREFIX="skillpull"
 CONFIG_DIR="$HOME/.config/skillpull"
@@ -46,6 +46,118 @@ warn()  { printf "  ${YELLOW}!${RESET} %s\n" "$*" >&2; }
 err()   { printf "  ${RED}✗${RESET} %s\n" "$*" >&2; }
 dim()   { [[ "${QUIET:-0}" == "1" ]] || printf "  ${DIM}%s${RESET}\n" "$*"; }
 
+# ── Cross-platform sed -i ──
+sedi() {
+  if [[ "$(uname)" == "Darwin" ]]; then
+    sed -i '' "$@"
+  else
+    sed -i "$@"
+  fi
+}
+
+# ── Interactive multi-select menu (bash 3 compatible) ──
+# Usage: select_menu "prompt" item1 item2 item3 ...
+# Sets SELECTED_ITEMS=("item1" "item3") with user's choices
+SELECTED_ITEMS=()
+
+select_menu() {
+  local prompt="$1"; shift
+  local items=("$@")
+  local count=${#items[@]}
+  local cursor=0
+  local i
+
+  # Track selected state with a simple string: "0" = unselected, "1" = selected
+  local selected=""
+  for ((i=0; i<count; i++)); do
+    selected="${selected}0"
+  done
+
+  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
+
+  # Draw initial list
+  for ((i=0; i<count; i++)); do
+    local marker="[ ]"
+    if [[ "${selected:$i:1}" == "1" ]]; then marker="[x]"; fi
+    if [[ $i -eq $cursor ]]; then
+      printf "  ${CYAN}> %s %s${RESET}\n" "$marker" "${items[$i]}" >&2
+    else
+      printf "    %s %s\n" "$marker" "${items[$i]}" >&2
+    fi
+  done
+
+  # 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
+
+    case "$key" in
+      $'\033[A'|k) # Up
+        [[ $cursor -gt 0 ]] && cursor=$((cursor - 1))
+        ;;
+      $'\033[B'|j) # Down
+        [[ $cursor -lt $((count - 1)) ]] && cursor=$((cursor + 1))
+        ;;
+      ' ') # Space - toggle selection
+        if [[ "${selected:$cursor:1}" == "0" ]]; then
+          selected="${selected:0:$cursor}1${selected:$((cursor+1))}"
+        else
+          selected="${selected:0:$cursor}0${selected:$((cursor+1))}"
+        fi
+        ;;
+      '') # Enter - confirm
+        break
+        ;;
+    esac
+
+    # Redraw: move cursor up by $count lines
+    printf '\033[%dA' "$count" >&2
+
+    for ((i=0; i<count; i++)); do
+      printf '\033[2K' >&2  # Clear line
+      local marker="[ ]"
+      if [[ "${selected:$i:1}" == "1" ]]; then marker="[x]"; fi
+      if [[ $i -eq $cursor ]]; then
+        printf "  ${CYAN}> %s %s${RESET}\n" "$marker" "${items[$i]}" >&2
+      else
+        printf "    %s %s\n" "$marker" "${items[$i]}" >&2
+      fi
+    done
+  done
+
+  # 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
+      SELECTED_ITEMS+=("${items[$i]}")
+    fi
+  done
+}
+
 # ── Cleanup ──
 _TMPDIR=""
 cleanup() { [[ -n "${_TMPDIR:-}" && -d "${_TMPDIR:-}" ]] && rm -rf "$_TMPDIR" || true; }
@@ -83,12 +195,12 @@ write_alias() {
   ensure_config
   local esc_url; esc_url="$(_json_escape "$url")"
   if grep -q "\"${name}\"" "$CONFIG_FILE" 2>/dev/null; then
-    sed -i "s|\"${name}\":\"[^\"]*\"|\"${name}\":\"${esc_url}\"|" "$CONFIG_FILE"
+    sedi "s|\"${name}\":\"[^\"]*\"|\"${name}\":\"${esc_url}\"|" "$CONFIG_FILE"
   else
     # Insert into aliases object
-    sed -i "s|\"aliases\":{|\"aliases\":{\"${name}\":\"${esc_url}\",|" "$CONFIG_FILE"
+    sedi "s|\"aliases\":{|\"aliases\":{\"${name}\":\"${esc_url}\",|" "$CONFIG_FILE"
     # Clean trailing comma before }
-    sed -i 's/,}/}/g' "$CONFIG_FILE"
+    sedi 's/,}/}/g' "$CONFIG_FILE"
   fi
 }
 
@@ -100,9 +212,9 @@ remove_alias() {
     return 1
   fi
   # Remove the alias entry
-  sed -i "s|\"${name}\":\"[^\"]*\",\?||" "$CONFIG_FILE"
+  sedi "s|\"${name}\":\"[^\"]*\",\?||" "$CONFIG_FILE"
   # Clean up double commas and trailing commas
-  sed -i 's/,,/,/g; s/,}/}/g; s/{,/{/g' "$CONFIG_FILE"
+  sedi 's/,,/,/g; s/,}/}/g; s/{,/{/g' "$CONFIG_FILE"
   info "Alias '$name' removed"
 }
 
@@ -131,7 +243,7 @@ set_registry() {
   local url="$1"
   ensure_config
   local esc_url; esc_url="$(_json_escape "$url")"
-  sed -i "s|\"registry\":\"[^\"]*\"|\"registry\":\"${esc_url}\"|" "$CONFIG_FILE"
+  sedi "s|\"registry\":\"[^\"]*\"|\"registry\":\"${esc_url}\"|" "$CONFIG_FILE"
   info "Default registry set to: $url"
 }
 
@@ -292,11 +404,11 @@ write_manifest_entry() {
     mv "$tmp" "$manifest"
   else
     # Append before closing braces
-    sed -i '/"skills":/,$ {
+    sedi '/"skills":/,$ {
       /^  }/ i\'"$entry"',
     }' "$manifest"
     # Fix trailing comma before closing brace
-    sed -i ':a;N;$!ba;s/,\n  }/\n  }/g' "$manifest"
+    sedi ':a;N;$!ba;s/,\n  }/\n  }/g' "$manifest"
   fi
 }
 
@@ -367,6 +479,28 @@ 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"
@@ -491,9 +625,38 @@ cmd_installed() {
 }
 
 cmd_remove() {
-  local skill_name="$1" target_dir="$2" agent="${3:-claude}"
+  local skill_name="${1:-}" target_dir="$2" agent="${3:-claude}"
   local fmt; fmt="$(agent_format "$agent")"
 
+  # No skill name provided -> interactive select
+  if [[ -z "$skill_name" && -t 0 ]]; then
+    if [[ ! -d "$target_dir" ]]; then
+      warn "No skills directory found at $target_dir"
+      return 0
+    fi
+    local installed_skills=()
+    local installed_names=()
+    while IFS= read -r d; do
+      [[ -n "$d" ]] && installed_skills+=("$d") && installed_names+=("$(skill_display_name "$d")")
+    done < <(discover_skills "$target_dir")
+
+    if [[ ${#installed_skills[@]} -eq 0 ]]; then
+      warn "No skills installed in $target_dir"
+      return 0
+    fi
+
+    select_menu "Select skills to remove:" "${installed_names[@]}"
+    if [[ ${#SELECTED_ITEMS[@]} -eq 0 ]]; then
+      warn "No skills selected"
+      return 0
+    fi
+
+    for sel in "${SELECTED_ITEMS[@]}"; do
+      cmd_remove "$sel" "$target_dir" "$agent"
+    done
+    return 0
+  fi
+
   if [[ "$fmt" == "mdc" ]]; then
     local dest="$target_dir/${skill_name}.mdc"
     if [[ ! -f "$dest" ]]; then
@@ -527,7 +690,7 @@ cmd_remove() {
     local tmp; tmp="$(mktemp)"
     grep -v "\"${skill_name}\"" "$manifest" > "$tmp" || true
     mv "$tmp" "$manifest"
-    sed -i ':a;N;$!ba;s/,\n  }/\n  }/g' "$manifest"
+    sedi ':a;N;$!ba;s/,\n  }/\n  }/g' "$manifest"
   fi
 
   info "Removed: $skill_name from $target_dir"
@@ -672,13 +835,32 @@ cmd_push() {
 cmd_uninstall() {
   local self; self="$(realpath "$0" 2>/dev/null || echo "$0")"
   local local_bin="${SKILLPULL_INSTALL_DIR:-$HOME/.local/bin}/skillpull"
+  local removed=0
 
   printf "\n  ${CYAN}Uninstall skillpull${RESET}\n\n"
 
-  # Remove local binary if exists
+  # Remove local binary (~/.local/bin)
   if [[ -f "$local_bin" ]]; then
     rm -f "$local_bin"
     info "Removed $local_bin"
+    removed=1
+  fi
+
+  # Remove npm global package (this is likely where the command lives)
+  if command -v npm &>/dev/null; then
+    local npm_pkg; npm_pkg="$(npm ls -g skillpull --parseable 2>/dev/null)" || true
+    if [[ -n "$npm_pkg" ]]; then
+      npm uninstall -g skillpull 2>/dev/null
+      info "Removed npm global package"
+      removed=1
+    fi
+  fi
+
+  # Remove the running binary itself if still exists and wasn't already cleaned
+  if [[ -f "$self" && "$self" != "$local_bin" ]]; then
+    rm -f "$self" 2>/dev/null || true
+    info "Removed $self"
+    removed=1
   fi
 
   # Remove config
@@ -693,21 +875,12 @@ cmd_uninstall() {
     fi
   fi
 
-  # npm global uninstall
-  if command -v npm &>/dev/null; then
-    local npm_global; npm_global="$(npm root -g 2>/dev/null)/../bin/skillpull"
-    if [[ -f "$npm_global" ]] || npm ls -g skillpull &>/dev/null 2>&1; then
-      printf "  Remove npm global package? (y/n) "
-      read -r yn
-      if [[ "$yn" == "y" || "$yn" == "Y" ]]; then
-        npm uninstall -g skillpull 2>/dev/null
-        info "npm global package removed"
-      fi
-    fi
-  fi
-
   echo ""
-  info "skillpull uninstalled."
+  if [[ "$removed" == "1" ]]; then
+    info "skillpull uninstalled."
+  else
+    warn "skillpull not found. Already uninstalled?"
+  fi
 }
 
 cmd_init() {
@@ -887,17 +1060,21 @@ main() {
     local global_bin="${SKILLPULL_INSTALL_DIR:-$HOME/.local/bin}/skillpull"
 
     if [[ "$self" != "$global_bin" && ! -f "$global_bin" ]]; then
+      info "Installing skillpull v${VERSION} ..."
       mkdir -p "$(dirname "$global_bin")"
       cp "$self" "$global_bin"
       chmod +x "$global_bin"
-      info "Installed skillpull to $global_bin"
+      info "Installed to $global_bin"
       if ! echo "$PATH" | tr ':' '\n' | grep -qx "$(dirname "$global_bin")"; then
         warn "Add to PATH: export PATH=\"$(dirname "$global_bin"):\$PATH\""
       fi
-    else
-      usage
+      echo ""
+      info "Run 'skillpull init' to set up your default skill repo."
+      exit 0
     fi
-    exit 0
+
+    # Already installed, no args -> treat as pull (will use registry + interactive select)
+    cmd="pull"
   fi
 
   while [[ $# -gt 0 ]]; do
@@ -960,7 +1137,16 @@ main() {
 
   case "${cmd:-}" in
     pull|"")
-      [[ -z "$repo_url" ]] && { err "Missing source"; usage; exit 1; }
+      if [[ -z "$repo_url" ]]; then
+        # No source given, try registry
+        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
       # If resolve_repo_url returned registry URL for bare name, use original as skill filter
       if [[ "$resolved" != "$repo_url" && "$repo_url" != @* && "$repo_url" != */* && "$repo_url" != *"://"* && "$repo_url" != git@* ]]; then
@@ -1012,7 +1198,6 @@ main() {
       done
       ;;
     remove)
-      [[ -z "$skill_name" ]] && { err "Missing skill name"; usage; exit 1; }
       for agent in "${agents[@]}"; do
         local target_dir
         target_dir="$(resolve_target_dir "$agent" "$is_global" "$custom_path")" || continue