Skip to content

Add skill install and dune auth to the install.sh script#24

Open
ivpusic wants to merge 3 commits intomainfrom
install_improvements
Open

Add skill install and dune auth to the install.sh script#24
ivpusic wants to merge 3 commits intomainfrom
install_improvements

Conversation

@ivpusic
Copy link
Member

@ivpusic ivpusic commented Mar 10, 2026

Currently there are separate steps, we are not consolidating that into single script.

@cursor
Copy link

cursor bot commented Mar 10, 2026

PR Summary

Low Risk
Changes are confined to the installer’s post-install UX and optional commands; main download/verify/install logic is unchanged, with minimal risk beyond potentially surprising interactive prompts.

Overview
Enhances install.sh to run a post-install interactive setup when a TTY is available, prompting the user to (optionally) install the Dune AI coding skill via npx skills add duneanalytics/skills and to authenticate by running dune auth.

For non-interactive installs (no /dev/tty), the script now prints success plus follow-up tips instead of running prompts, and it replaces the previous single “Installed …” log line with this new flow.

Written by Cursor Bugbot for commit add8fb3. Configure here.

}

if key == "" {
if stdin, ok := cmd.InOrStdin().(*os.File); ok && !term.IsTerminal(int(stdin.Fd())) {
Copy link
Member Author

Choose a reason for hiding this comment

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

better handling of envs that don't have interactive terminal

Copy link

Choose a reason for hiding this comment

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

what does cmd.InOrStdin().(*os.File); do?

}

if key == "" {
if stdin, ok := cmd.InOrStdin().(*os.File); ok && !term.IsTerminal(int(stdin.Fd())) {
Copy link

Choose a reason for hiding this comment

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

what does cmd.InOrStdin().(*os.File); do?

}

has_tty() {
[ -t 0 ] || [ -t 1 ] || [ -t 2 ]
Copy link

Choose a reason for hiding this comment

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

😂 😵‍💫

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