Skip to content

fix(cli): clarify session-id resume requirement - #10090

Closed
Osraka wants to merge 1 commit into
aaif-goose:mainfrom
Osraka:osraka/clarify-session-id-resume-help
Closed

fix(cli): clarify session-id resume requirement#10090
Osraka wants to merge 1 commit into
aaif-goose:mainfrom
Osraka:osraka/clarify-session-id-resume-help

Conversation

@Osraka

@Osraka Osraka commented Jun 29, 2026

Copy link
Copy Markdown
Contributor

Summary

  • clarify that --session-id requires --resume for goose session / goose run
  • keep the wording broad enough for session-management subcommands that use a session id directly
  • add a CLI help regression test for the goose session --help output

Fixes #10076.

Validation

  • cargo test -p goose-cli --no-default-features --features rustls-tls session_help_describes_session_id_resume_requirement --quiet
  • cargo fmt -p goose-cli -- --check
  • git diff --check

@Osraka

Osraka commented Jun 29, 2026

Copy link
Copy Markdown
Contributor Author

Closing this as duplicate. #10077 has already landed on main and closes #10076 with the session-id help text update. Thanks.

@Osraka Osraka closed this Jun 29, 2026
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.

Bug: goose session --help suggests --session-id is usable, but errors out without --resume

1 participant