review feedback + enhancements from PR 343

Author: patelspratik

bin/install-latest-linux.sh

@@ -2,16 +2,20 @@
 set -eo pipefail
 
 # Fetch release metadata from GitHub API
-API_RESPONSE="$(curl -sf https://api.github.com/repos/brevdev/brev-cli/releases/latest 2>&1)" || {
+API_RESPONSE="$(curl -sf ${GITHUB_TOKEN:+-H "Authorization: token ${GITHUB_TOKEN}"} https://api.github.com/repos/brevdev/brev-cli/releases/latest)" || {
     echo "Error: Failed to fetch release info from GitHub API." >&2
-    echo "This may be caused by rate limiting. Try again later or set a GITHUB_TOKEN." >&2
+    echo "This is often caused by rate limiting when many requests come from the same IP." >&2
+    echo "If you are using a VPN, try turning it off and running this script again." >&2
+    echo "You can also set GITHUB_TOKEN to avoid rate limits." >&2
+    echo "For more details, see: https://docs.github.com/rest/overview/resources-in-the-rest-api#rate-limiting" >&2
     exit 1
 }
 
 # Extract the download URL for linux/amd64
 DOWNLOAD_URL="$(echo "${API_RESPONSE}" | grep "browser_download_url.*linux.*amd64" | cut -d '"' -f 4)"
 if [ -z "${DOWNLOAD_URL}" ]; then
     echo "Error: Could not find release for linux amd64" >&2
+    echo "GitHub API response (truncated): ${API_RESPONSE:0:200}" >&2
     exit 1
 fi
 

bin/install-latest.sh

@@ -10,16 +10,20 @@ case "${ARCH}" in
 esac
 
 # Fetch release metadata from GitHub API
-API_RESPONSE="$(curl -sf https://api.github.com/repos/brevdev/brev-cli/releases/latest 2>&1)" || {
+API_RESPONSE="$(curl -sf ${GITHUB_TOKEN:+-H "Authorization: token ${GITHUB_TOKEN}"} https://api.github.com/repos/brevdev/brev-cli/releases/latest)" || {
     echo "Error: Failed to fetch release info from GitHub API." >&2
-    echo "This may be caused by rate limiting. Try again later or set a GITHUB_TOKEN." >&2
+    echo "This is often caused by rate limiting when many requests come from the same IP." >&2
+    echo "If you are using a VPN, try turning it off and running this script again." >&2
+    echo "You can also set GITHUB_TOKEN to avoid rate limits." >&2
+    echo "For more details, see: https://docs.github.com/rest/overview/resources-in-the-rest-api#rate-limiting" >&2
     exit 1
 }
 
 # Extract the download URL for this platform
 DOWNLOAD_URL="$(echo "${API_RESPONSE}" | grep "browser_download_url.*${OS}.*${ARCH}" | cut -d '"' -f 4)"
 if [ -z "${DOWNLOAD_URL}" ]; then
     echo "Error: Could not find release for ${OS} ${ARCH}" >&2
+    echo "GitHub API response (truncated): ${API_RESPONSE:0:200}" >&2
     exit 1
 fi
 

scripts/install-agent-skill.sh

@@ -116,12 +116,21 @@ done
 rm -rf "$TMPDIR"
 
 # Resolve commit SHA and write .version file
-COMMIT_SHA=$(curl -fsSL "https://api.github.com/repos/$REPO/commits/$BRANCH" 2>/dev/null | grep '"sha"' | head -1 | sed 's/.*"sha": *"\([^"]*\)".*/\1/')
-if [[ -n "$COMMIT_SHA" ]]; then
-    for dir in "${INSTALL_DIRS[@]}"; do
-        printf 'branch=%s\ncommit=%s\n' "$BRANCH" "$COMMIT_SHA" > "$dir/.version"
-    done
-    echo -e "  ${GREEN}✓${NC} .version (${COMMIT_SHA:0:12})"
+VERSION_RESPONSE=$(curl -fsSL "https://api.github.com/repos/$REPO/commits/$BRANCH" 2>&1) || true
+if echo "$VERSION_RESPONSE" | grep -q "API rate limit exceeded"; then
+    echo -e "  ${YELLOW}⚠${NC} .version (skipped — GitHub API rate limit exceeded)"
+    echo -e "  ${YELLOW}If you are using a VPN, try turning it off and running this script again.${NC}"
+    echo -e "  ${YELLOW}See: https://docs.github.com/rest/overview/resources-in-the-rest-api#rate-limiting${NC}"
+else
+    COMMIT_SHA=$(echo "$VERSION_RESPONSE" | grep '"sha"' | head -1 | sed 's/.*"sha": *"\([^"]*\)".*/\1/')
+    if [[ -n "$COMMIT_SHA" ]]; then
+        for dir in "${INSTALL_DIRS[@]}"; do
+            printf 'branch=%s\ncommit=%s\n' "$BRANCH" "$COMMIT_SHA" > "$dir/.version"
+        done
+        echo -e "  ${GREEN}✓${NC} .version (${COMMIT_SHA:0:12})"
+    else
+        echo -e "  ${YELLOW}⚠${NC} .version (could not resolve commit SHA)"
+    fi
 fi
 
 echo ""