-
Notifications
You must be signed in to change notification settings - Fork 2.4k
fix: enable 'goose://' handler for debian systems #3952
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Signed-off-by: Jason Cochard <[email protected]>
95a48f5 to
90ab7ff
Compare
| Name=Goose AI Agent | ||
| Comment=Local AI agent for development tasks | ||
| Exec=/path/to/goose/ui/desktop/out/Goose-linux-x64/Goose | ||
| Exec=/path/to/goose/ui/desktop/out/Goose-linux-x64/Goose %U |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is this deliberate?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, the %U becomes deeplink arguments etc when invoked that way, but it is ignored otherwise.
michaelneale
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nice thanks @eyelight works well
…ndow * 'main' of github.com:block/goose: sanitize message content on deserialization (#3966) Move summarize button inside of context view (#4015) blog: post on lead/worker model (#3994) Actually send cancellation to MCP servers (#3865) fix: enable 'goose://' handler for debian systems (#3952) fit: default ollama port (#4001) Remove cognitive complexity clippy lint (#4010) feat: add @-mention file reference expansion to .goosehints (#3873) feat(cli): Add --name/-n to session remove and --id/-i alias for session export (#3941) Docs: provider and model run options (#4013) To-Do Tools (#3902) ci: correctly match doc only changes (#4009) Remove PR trigger for Linux build workflow (#4008) docs: update release docs with an additional step needed + adjust list formatting (#4005)
* 'main' of github.com:block/goose: Move summarize button inside of context view (#4015) blog: post on lead/worker model (#3994) Actually send cancellation to MCP servers (#3865) fix: enable 'goose://' handler for debian systems (#3952) fit: default ollama port (#4001) Remove cognitive complexity clippy lint (#4010) feat: add @-mention file reference expansion to .goosehints (#3873) feat(cli): Add --name/-n to session remove and --id/-i alias for session export (#3941) Docs: provider and model run options (#4013) To-Do Tools (#3902) ci: correctly match doc only changes (#4009) Remove PR trigger for Linux build workflow (#4008)
Signed-off-by: Jason Cochard <[email protected]> Signed-off-by: Jack Wright <[email protected]>
This PR fixes a bug in linux (debian) where
goose://URLs would not know which application to open.After making the changes in the commit, the
.debresulting fromnpm run make -- --targets=@electron-forge/maker-debcontains the required line in thegoose.desktopfile.When the deb is installed using
dpkg, the installed Goose application is successfully invoked upon clicking agoose://web link from within Chromium on Pop!_OS 22.04.