Skip to content

feat: add install.sh for one-line agent-skills install #56

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
moonming merged 1 commit into from
Jul 13, 2026
Merged

feat: add install.sh for one-line agent-skills install #56

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
85 changes: 85 additions & 0 deletions install.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,85 @@
#!/bin/sh
# Install the Apache APISIX AI agent skills into your AI coding agent.
#
# Each skill is a SKILL.md knowledge pack that teaches an agent (Claude Code,
# Cursor, Copilot, Windsurf, OpenCode, ...) how to configure Apache APISIX
# through the a6 CLI. This script copies them into your agent's skills directory.
#
# Quick start (installs into ~/.claude/skills for Claude Code):
# curl -fsSL https://raw.githubusercontent.com/api7/a6/main/install.sh | sh
#
# Install somewhere else (e.g. a project-local Cursor rules dir):
# curl -fsSL https://raw.githubusercontent.com/api7/a6/main/install.sh | sh -s -- --dir ./.cursor/rules
# SKILLS_DIR=~/.config/opencode/skills sh -c "$(curl -fsSL https://raw.githubusercontent.com/api7/a6/main/install.sh)"
set -eu

REPO="api7/a6"
BRANCH="main"
LABEL="Apache APISIX"

# Target directory. Default: Claude Code personal skills. Override with
# SKILLS_DIR=... or --dir <path>.
SKILLS_DIR="${SKILLS_DIR:-$HOME/.claude/skills}"

while [ $# -gt 0 ]; do
case "$1" in
--dir) SKILLS_DIR="${2:?--dir needs a path}"; shift 2 ;;
--dir=*)
SKILLS_DIR="${1#*=}"
[ -n "$SKILLS_DIR" ] || { echo "install.sh: --dir needs a path" >&2; exit 1; }
shift
;;
-h | --help)
echo "Usage: install.sh [--dir <path>] (default: \$HOME/.claude/skills)"
exit 0
;;
*) echo "install.sh: unknown option '$1'" >&2; exit 1 ;;
esac
done

if command -v curl >/dev/null 2>&1; then
fetch() { curl -fsSL "$1"; }
elif command -v wget >/dev/null 2>&1; then
fetch() { wget -qO- "$1"; }
else
echo "install.sh: need curl or wget on your PATH." >&2
exit 1
fi

echo "Installing ${LABEL} agent skills into ${SKILLS_DIR} ..."

TMP="$(mktemp -d)"
trap 'rm -rf "$TMP"' EXIT INT TERM

# Download the repo tarball (no git required) and extract just the skills.
fetch "https://codeload.github.com/${REPO}/tar.gz/refs/heads/${BRANCH}" >"$TMP/repo.tgz" ||
{ echo "install.sh: download failed." >&2; exit 1; }
tar -xzf "$TMP/repo.tgz" -C "$TMP"

SRC="$(find "$TMP" -maxdepth 2 -type d -name skills | head -n 1)"
if [ -z "$SRC" ] || [ ! -d "$SRC" ]; then
echo "install.sh: could not find a skills/ directory in the download." >&2
exit 1
fi

mkdir -p "$SKILLS_DIR"
count=0
for dir in "$SRC"/*/; do
[ -f "${dir}SKILL.md" ] || continue
name="$(basename "$dir")"
rm -rf "${SKILLS_DIR}/${name}"

@coderabbitai coderabbitai Bot Jul 13, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🩺 Stability & Availability | 🟡 Minor | ⚡ Quick win

Guard against empty SKILLS_DIR in rm -rf to prevent accidental root deletion.

If SKILLS_DIR is set to an empty string (e.g., --dir= or SKILLS_DIR=""), this line expands to rm -rf "/${name}", deleting a top-level directory. While mkdir -p "" on line 59 would likely fail and exit first under set -eu, adding an explicit :? guard is a low-cost safety net.

🛡️ Proposed fix 
-  rm -rf "${SKILLS_DIR}/${name}"
+  rm -rf "${SKILLS_DIR:?}/${name}"
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
rm -rf "${SKILLS_DIR}/${name}"
rm -rf "${SKILLS_DIR:?}/${name}"
🧰 Tools
🪛 Shellcheck (0.11.0)

[warning] 64-64: Use "${var:?}" to ensure this never expands to / .

(SC2115)

🤖 Prompt for AI Agents 
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@install.sh` at line 64, Update the rm command in the skills cleanup flow to
require a non-empty SKILLS_DIR before expansion, using the shell’s explicit
parameter guard so an empty value aborts instead of targeting a root-level path.
Preserve the existing name-based removal behavior when SKILLS_DIR is set.

Source: Linters/SAST tools

cp -R "$dir" "${SKILLS_DIR}/${name}"
count=$((count + 1))
done

if [ "$count" -eq 0 ]; then
echo "install.sh: no SKILL.md packs found to install." >&2
exit 1
fi

echo "Installed ${count} skills to ${SKILLS_DIR}"
echo
echo "Next: ask your AI coding agent to configure ${LABEL} in plain language, e.g."
echo " \"add key-auth to my /orders route and rate-limit it to 100 requests per minute\""
echo
echo "Browse the catalog: https://docs.api7.ai/apisix/ai-agent-skills"
Loading