Skip to content

Conversation

@GaryZhous
Copy link
Contributor

@GaryZhous GaryZhous commented Aug 8, 2025

Following #3018, where I noticed name and id are interchangeable for locating a goose session. This pull request introduces enhancements and clarifications to the Goose CLI's session management commands, primarily focusing on improved support for session removal and export by both ID and name. The documentation is updated to reflect these changes, and some outdated information about command history search is removed for clarity.

CLI Improvements

  • Added support for removing sessions by name in addition to ID, including new --name/-n aliases for the remove command in SessionCommand and updated argument definitions in crates/goose-cli/src/cli.rs. [1] [2]

Documentation Updates

  • Updated documentation for the remove command to include instructions and examples for removing sessions by name, and clarified option descriptions in documentation/docs/guides/goose-cli-commands.md. [1] [2]
  • Clarified export command options to explicitly document support for exporting by ID as well as by name and path.

Documentation Cleanup

  • Removed the section describing interactive command history search (Ctrl+R) from the CLI guide, streamlining the documentation and removing references to a feature that is no longer present.

After

image

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.

I think looks ok - if you can run cargo fmt and check clippy is happy (scripts dir can help) I think good to go.

@michaelneale michaelneale self-assigned this Aug 11, 2025
@GaryZhous
Copy link
Contributor Author

@michaelneale thanks for the reminder, I just fixed the formatting issue!

@michaelneale michaelneale merged commit 615d1fc into block:main Aug 11, 2025
11 checks passed
lifeizhou-ap added a commit that referenced this pull request Aug 12, 2025
* main:
  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)
  chore(release): release version 1.3.0 (#3921)
  docs: MCP-ui blog content (#3996)
  feat: Add `GOOSE_TERMINAL` env variable to spawned terminals (#3911)
  add missing dependencies for developer setup (#3930)
zanesq added a commit that referenced this pull request Aug 12, 2025
…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)
katzdave added a commit that referenced this pull request Aug 12, 2025
* '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)
@GaryZhous GaryZhous deleted the GaryZ/cli branch August 15, 2025 03:10
ayax79 pushed a commit to ayax79/goose that referenced this pull request Aug 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants