Skip to content

fix(release): Added download_cli.ps1 to upload install script#4478

Closed
tiensi wants to merge 2 commits intoblock:mainfrom
tiensi:tiensi/add-download_cli_ps1-to-yml
Closed

fix(release): Added download_cli.ps1 to upload install script#4478
tiensi wants to merge 2 commits intoblock:mainfrom
tiensi:tiensi/add-download_cli_ps1-to-yml

Conversation

@tiensi
Copy link
Contributor

@tiensi tiensi commented Sep 2, 2025

Pull Request Description

As per this issue the powershell install guidance is likely incorrect and doesn't use the existing download_cli.ps1. When investigating, it seems that there is no download_cli.ps1 artifact available through powershell Invoke-WebRequest.

Changes

  • Added download_cli.ps1 to Upload Install Scripts in build yml files. This should enable access to download_cli.ps1 through powershell endpoint:

Invoke-WebRequest -Uri "https://github.com/block/goose/releases/download/stable/download_cli.ps1" -OutFile "download_cli.ps1";

Notes

  • If this works properly I will create a follow up pr to update the readme.
  • I may need help testing as I don't have permissions to trigger a build and verify the artifact.

@tiensi tiensi changed the title feat(release): Added ps1 download_cli to upload install script fix(release): Added ps1 download_cli to upload install script Sep 2, 2025
@tiensi tiensi marked this pull request as ready for review September 2, 2025 05:42
@tiensi tiensi changed the title fix(release): Added ps1 download_cli to upload install script fix(release): Added download_cli.ps1 to upload install script Sep 2, 2025
Copy link
Collaborator

@The-Best-Codes The-Best-Codes left a comment

Choose a reason for hiding this comment

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

LGTM, not sure how to test this exactly though.

@zanesq zanesq requested a review from michaelneale September 2, 2025 16:50
Copy link
Collaborator

@michaelneale michaelneale left a comment

Choose a reason for hiding this comment

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

looks ok, but no way to test

* main: (43 commits)
  feat: add auto-compact threshold configuration UI (block#4178)
  Add container detection to developer extension (block#4559)
  docs: add ampersand to link (block#4560)
  Add video link to README for user guidance (block#4553)
  docs: social channels (block#4552)
  feat: simplify navigation, make reload work (block#4498)
  docs: new recipe warning (block#4545)
  Add AGENTS.md for AI coding assistant support (block#4539)
  docs: non-interactive compact now (block#4543)
  fixed css classes and added some accessibility fixes (block#4492)
  feat(acp): Read files (block#4531)
  Add YouTube Short to Auto Visualiser Tutorial (block#4536)
  Fix/settings page (block#4520)
  update to RMCP 0.6.2 (block#4523)
  docs: nested goosehints (block#4528)
  feat: Agent Client Protocol implementation of goose (block#4511)
  feat: make tests for rmcp based developer server consistent with former implementation (block#4519)
  worlds simplest logging to see where things are blocked (block#3888)
  docs: update quickstart and install topics (block#4378)
  feat: Add configurable Bedrock retry parameters (block#4316)
  ...
@michaelneale
Copy link
Collaborator

FYI - we don't need it in releases, we can adjust scripts to download it from here:

https://raw.githubusercontent.com/block/goose/main/download_cli.ps1

that way will always be the latest - can we adjust the quickstart docs to point to that link instead?

@tiensi
Copy link
Contributor Author

tiensi commented Sep 9, 2025

oh I didn't know that was possible. I'll close this and see if I can test it out. Thanks!

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.

3 participants