Conversation
PR SummaryLow Risk Overview In the interactive post-install step, the installer now runs Written by Cursor Bugbot for commit d021e8d. Configure here. |
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
Bugbot Autofix prepared a fix for the issue found in the latest run.
- ✅ Fixed: Retry/tip messages omit the new
--skill '*'flag- Updated all four retry/tip messages (lines 92, 111, 115, 119) to include the --skill '*' flag, ensuring users get auto-selection behavior when following suggested commands.
Or push these changes by commenting:
@cursor push 0e1fec6944
Preview (0e1fec6944)
diff --git a/install.sh b/install.sh
--- a/install.sh
+++ b/install.sh
@@ -89,7 +89,7 @@ main() {
echo "" >&2
log "Dune CLI ${version} installed successfully!"
echo "" >&2
- log "Tip: Run 'npx skills add duneanalytics/skills' to install Dune AI skills."
+ log "Tip: Run 'npx skills add duneanalytics/skills --skill '\''*'\''' to install Dune AI skills."
log "Tip: Run 'dune auth' to authenticate with your Dune account."
log "Tip: Run 'dune sim auth' to access real-time blockchain data (balances, tokens, NFTs)."
fi
@@ -89,7 +89,7 @@ main() {
echo "" >&2
log "Dune CLI ${version} installed successfully!"
echo "" >&2
- log "Tip: Run 'npx skills add duneanalytics/skills' to install Dune AI skills."
+ log "Tip: Run 'npx skills add duneanalytics/skills --skill '\''*'\''' to install Dune AI skills."
log "Tip: Run 'dune auth' to authenticate with your Dune account."
log "Tip: Run 'dune sim auth' to access real-time blockchain data (balances, tokens, NFTs)."
fi
@@ -108,15 +108,15 @@ post_install() {
read -r answer < /dev/tty || answer=""
case "$answer" in
[nN]*)
- log "Skipped. You can install them later with: npx skills add duneanalytics/skills"
+ log "Skipped. You can install them later with: npx skills add duneanalytics/skills --skill '\''*'\''"
;;
*)
log "Installing Dune skills..."
- npx skills add duneanalytics/skills --skill '*' < /dev/tty || log "Skill installation failed. You can retry with: npx skills add duneanalytics/skills"
+ npx skills add duneanalytics/skills --skill '*' < /dev/tty || log "Skill installation failed. You can retry with: npx skills add duneanalytics/skills --skill '\''*'\''"
;;
esac
else
- log "Tip: Run 'npx skills add duneanalytics/skills' to install Dune AI skills."
+ log "Tip: Run 'npx skills add duneanalytics/skills --skill '\''*'\''' to install Dune AI skills."
log "(requires Node.js / npx)"
fi
@@ -108,15 +108,15 @@ post_install() {
read -r answer < /dev/tty || answer=""
case "$answer" in
[nN]*)
- log "Skipped. You can install them later with: npx skills add duneanalytics/skills"
+ log "Skipped. You can install them later with: npx skills add duneanalytics/skills --skill '\''*'\''"
;;
*)
log "Installing Dune skills..."
- npx skills add duneanalytics/skills --skill '*' < /dev/tty || log "Skill installation failed. You can retry with: npx skills add duneanalytics/skills"
+ npx skills add duneanalytics/skills --skill '*' < /dev/tty || log "Skill installation failed. You can retry with: npx skills add duneanalytics/skills --skill '\''*'\''"
;;
esac
else
- log "Tip: Run 'npx skills add duneanalytics/skills' to install Dune AI skills."
+ log "Tip: Run 'npx skills add duneanalytics/skills --skill '\''*'\''' to install Dune AI skills."
log "(requires Node.js / npx)"
fiComment @cursor review or bugbot run to trigger another review on this PR


Currently user has to manually select them. This change makes pre-selection automatic.