Skip to content

Simplify chaitin-cli skill install and improve robustness#34

Merged
innomentats merged 2 commits intomainfrom
skill/chaitin-cli-install
Apr 24, 2026
Merged

Simplify chaitin-cli skill install and improve robustness#34
innomentats merged 2 commits intomainfrom
skill/chaitin-cli-install

Conversation

@mmfu123
Copy link
Copy Markdown
Collaborator

@mmfu123 mmfu123 commented Apr 24, 2026

Summary

  • Remove Windows PowerShell installer (install-chaitin-cli.ps1); Windows users are now guided to manually download from GitHub Releases instead of attempting automated installation
  • Add robust GitHub release tag resolution with 3-level fallback: API → redirect → HTML scraping, with GITHUB_TOKEN/GH_TOKEN auth support
  • Improve install flow with 3-pass strategy: (1) non-sudo user dirs, (2) sudo system dirs, (3) guaranteed fallback to $HOME/.local/bin
  • Add "No-Argument Behavior" section to SKILL.md so /chaitin-cli without args greets, checks/installs, and offers next steps
  • Prefer cp+chmod over coreutils install for better portability across sandboxed and minimal environments
  • Fix set -u compatibility with empty arrays using ${arr[@]+"${arr[@]}"} pattern

Test plan

  • Run /chaitin-cli without arguments and verify greeting + install check flow
  • Test on macOS/Linux: run bash scripts/install-chaitin-cli.sh and confirm binary installs correctly
  • Test on a machine without chaitin-cli: invoke any chaitin-cli command and verify auto-install + retry
  • Verify GitHub API rate-limit scenario triggers redirect/HTML fallback for tag resolution
  • Confirm Windows environment shows manual download instructions instead of attempting PowerShell install

@innomentats innomentats merged commit 77fbed1 into main Apr 24, 2026
3 checks passed
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