Skip to content

Auto select dune and sim skills#45

Merged
ivpusic merged 1 commit intomainfrom
auto_select_sim_skill
Mar 24, 2026
Merged

Auto select dune and sim skills#45
ivpusic merged 1 commit intomainfrom
auto_select_sim_skill

Conversation

@ivpusic
Copy link
Copy Markdown
Member

@ivpusic ivpusic commented Mar 24, 2026

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

@cursor
Copy link
Copy Markdown

cursor bot commented Mar 24, 2026

PR Summary

Low Risk
Low risk shell-script change limited to post-install messaging and the npx skills add invocation; main risk is behavioral change in which skills get installed by default.

Overview
Updates install.sh to treat the AI integration as multiple Dune skills (copy changes in both interactive and non-interactive flows).

In the interactive post-install step, the installer now runs npx skills add duneanalytics/skills --skill '*' so skills are auto-selected rather than requiring manual selection, and adjusts the skip/install messaging accordingly.

Written by Cursor Bugbot for commit d021e8d. Configure here.

@ivpusic ivpusic requested review from charisra and va3093 March 24, 2026 10:11
Copy link
Copy Markdown

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

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.

Create PR

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)"
     fi

Comment @cursor review or bugbot run to trigger another review on this PR

@ivpusic ivpusic merged commit 2534228 into main Mar 24, 2026
3 checks passed
@ivpusic ivpusic deleted the auto_select_sim_skill branch March 24, 2026 10:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants