Skip to content

install.sh: banner, step-by-step output and highlighted next steps - #29

Merged
lavicosa merged 2 commits into
mainfrom
devin/1788686843-pretty-install-output
Sep 7, 2026
Merged

install.sh: banner, step-by-step output and highlighted next steps#29
lavicosa merged 2 commits into
mainfrom
devin/1788686843-pretty-install-output

Conversation

@devin-ai-integration

Copy link
Copy Markdown
Contributor

Summary

Makes the curl ... | sh installer output friendlier. Behaviour (download, checksum, PATH handling, receipt, marker file, flags) is unchanged; only presentation changed.

  ____                            _
 |  _ \ ___  __ _ _   _  ___  ___| |_ _   _
 | |_) / _ \/ _` | | | |/ _ \/ __| __| | | |
 |  _ <  __/ (_| | |_| |  __/\__ \ |_| |_| |
 |_| \_\___|\__, |\__,_|\___||___/\__|\__, |
               |_|                    |___/

  Requesty CLI installer

  ✓ Detected darwin/arm64
  ✓ Resolved latest release: v0.1.10
  Installing requesty v0.1.10
  ✓ Downloaded requesty_darwin_arm64.tar.gz
  ✓ Verified checksum
  ✓ Installed requesty to /Users/daniel/.requesty/bin/requesty
  ✓ Added /Users/daniel/.requesty/bin to PATH in /Users/daniel/.zshrc

  ✓ requesty v0.1.10 is ready

  │  Next steps
  │
  │  1. Reload your shell (or open a new terminal):
  │     source ~/.zshrc
  │  2. Start the CLI:
  │     requesty

Details worth knowing:

  • setup_style enables ANSI colors only when [ -t 1 ], NO_COLOR is unset and TERM != dumb; ✓ ✗ │ fall back to * x | when the locale isn't UTF-8. Still plain POSIX sh.
  • The "Next steps" block adapts: shows source <profile> (with $HOME shortened to ~) when a profile was modified, shows the export PATH=... line to add when the profile file doesn't exist (PATH_LINE), and only requesty when PATH already contains the install dir.
  • The already-installed branch now prints the same summary block instead of a bare line.
  • fail output is prefixed with a red ✗ error: and still goes to stderr.

Tested locally with sh, dash, in a pty (colors) and piped (no colors), fresh install / upgrade / already-installed / missing profile / 404 version.

Link to Devin session: https://app.devin.ai/sessions/aaf2a185ba7346f583e9c636d89c06fe
Open in Devin Desktop: https://app.devin.ai/desktop/session/aaf2a185ba7346f583e9c636d89c06fe?variant=devin
Requested by: @requesty-ditti

Co-Authored-By: Daniel Trugman <daniel@requesty.ai>
@devin-ai-integration

Copy link
Copy Markdown
Contributor Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment, CI, and merge conflict monitoring

@github-actions

github-actions Bot commented Sep 6, 2026

Copy link
Copy Markdown

AI Cost

No spend detected for branch devin/1788686843-pretty-install-output in the last 30 days.

Updated 2026-09-06T09:55Z · last 30 days · data from Requesty

@github-actions github-actions Bot added the cost:$0.00 AI cost for this branch label Sep 6, 2026
Comment thread install.sh Outdated
…citly

Co-Authored-By: Daniel Trugman <daniel@requesty.ai>
@lavicosa
lavicosa merged commit 88c38ac into main Sep 7, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cost:$0.00 AI cost for this branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants