feat(tools): restore squad-cli as a default-installed tool#488
Closed
primetimetank21 wants to merge 3 commits into
Closed
feat(tools): restore squad-cli as a default-installed tool#488primetimetank21 wants to merge 3 commits into
primetimetank21 wants to merge 3 commits into
Conversation
release: 0.9.6 -- Sprint 16 wrap
release: 0.9.7 (Sprint 17)
release: 0.9.8
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Restores
@bradygaster/squad-clias a default-installed tool (#487), reversing the over-scrub from PR #475.squad-cli is a dev tool (engine behind
gosquad='copilot --agent squad --yolo'), not squad coordination infrastructure. Same class of over-scrub as the gosquad removal reversed in PR #478.Closes #487
What Changed
Installers (new)
scripts/linux/tools/squad-cli.sh-- version-aware, sources nvm,timeout 10on version detect,--no-fund --no-audit, soft-fail (log_warn+exit 0) if npm absentscripts/windows/tools/squad-cli.ps1-- version-aware, soft-fail (Write-Warn+return) if npm absent, ASCII-only (PS 5.1 safe)Version pin
.tool-versions:squad-cli 0.10.0(npm latest stable)Orchestrator wiring
scripts/linux/setup.sh:run_tool "squad-cli"added immediately afterrun_tool "copilot-cli"scripts/windows/setup.ps1: dot-source +Install-SquadClicall added immediately afterInstall-CopilotCliinMain()Tests
tests/test_nvm_bootstrap.sh: T7-T9 (nvm sourcing, version pin, idempotency check for squad-cli.sh)tests/test_windows_setup.ps1: Group HH (HH-1 through HH-4) -- function existence, Get-ToolVersion usage, ordering in Main, soft-fail guard.github/workflows/e2e-install.yml:squad --versionassertion added to linux, macos, and windows e2e jobsDocs
README.md: squad-cli row in tools table,squad-cli.shin repo structure diagram, Windows tools count bumped to 10Key Decisions (locked)
0.10.0(npm latest; 0.9.4 was the prior pin)Write-Warn+returnif npm absent (notexit 1).squad/tree, no squad workflows, no spawn scriptsLocal Verification
squad-cli.ps1parses clean (AST), ASCII-only verifiedsquad-cli.shpasses shellcheck (pre-commit hook)Not locally exercisable (no npm/nvm in this environment): actual npm install,
squad --versionlive output, e2e job assertions. Kix to verify in CI.