Skip to content

feat: Add codename goose CLI - #139

Merged
koralowiec merged 18 commits into
devcontainers-extra:mainfrom
brandonhawi:main
Jul 23, 2025
Merged

feat: Add codename goose CLI#139
koralowiec merged 18 commits into
devcontainers-extra:mainfrom
brandonhawi:main

Conversation

@brandonhawi

@brandonhawi brandonhawi commented Jul 13, 2025

Copy link
Copy Markdown
Contributor

Adds the CLI from the GH releases. Used nanolayer as recommended.

codename goose Docs

$nanolayer_location \
install \
apt-get \
libxcb1

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

libxcb1 is a dep of the goose CLI, so needed to install this

@koralowiec
koralowiec self-requested a review July 21, 2025 09:29

@koralowiec koralowiec left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Heyo! I left two small request changes

Comment thread src/codename-goose-cli/README.md Outdated
Comment thread src/codename-goose-cli/install.sh Outdated
brandonhawi and others added 2 commits July 22, 2025 14:48
Co-authored-by: Arek Kalandyk <36413794+koralowiec@users.noreply.github.com>
Signed-off-by: Brandon Hawi <brandonhawi1@gmail.com>
Co-authored-by: Arek Kalandyk <36413794+koralowiec@users.noreply.github.com>
Signed-off-by: Brandon Hawi <brandonhawi1@gmail.com>
@brandonhawi

Copy link
Copy Markdown
Contributor Author

Heyo! I left two small request changes

Thank you @koralowiec, committed these changes!

@koralowiec

Copy link
Copy Markdown
Member

Heyo, it should be fine now, but I have one last question. Do you think it makes sense to rename it from codename-goose-cli to goose-cli? I was checking out the project, and I typically see that they refer to the tool with Goose CLI or goose

If the name change makes sense, please rename rename the files, directories and mentions (codename-goose-cli to goose-cli). If you think that codename- prefix is needed, that's also fine, just let me know, and I'll merge the PR in the current state

@brandonhawi

Copy link
Copy Markdown
Contributor Author

@koralowiec, I opted for codename-goose-cli to help differentiate from the already existing goose-cli devcontainer feature: https://github.com/rafaph/devcontainer-features/tree/main/src/goose-cli

(This is a DB Migration tool from what I can tell, and not at all related)

I'm fine to change it, just wanted to explain my thought process. I'll defer to your final judgment; what do you think?

@koralowiec

Copy link
Copy Markdown
Member

Okay, that's reasonable. Let's keep codename-goose-cli then

@koralowiec
koralowiec merged commit 153f88b into devcontainers-extra:main Jul 23, 2025
8 of 9 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