Skip to content

Conversation

@dianed-square
Copy link
Contributor

@dianed-square dianed-square commented Dec 19, 2025

Summary

This PR documents the --param flag for recipe deeplink and recipe open and removes the outdated -p alias for --path.

Documentation updates:

  • documentation/docs/guides/goose-cli-commands.md:
    • Add -p, --param <KEY=VALUE> option
    • Remove outdated -p alias for --path
  • documentation/docs/guides/recipes/session-recipes.md:
    • Add --param example

Type of Change

  • Feature
  • Bug fix
  • Refactor / Code quality
  • Performance improvement
  • Documentation
  • Tests
  • Security fix
  • Build / Release
  • Other (specify below)

AI Assistance

  • This PR was created or reviewed with AI assistance

Testing

Manual testing


Copilot AI review requested due to automatic review settings December 19, 2025 21:44
@dianed-square dianed-square requested a review from a team as a code owner December 19, 2025 21:44
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the CLI documentation to accurately reflect the current implementation of recipe commands. The documentation now correctly shows that -p is the short alias for --param (not --path), and documents the --param flag for both recipe deeplink and recipe open commands.

Key Changes

  • Removed the incorrect -p short alias from --path documentation (the code never had this alias)
  • Added -p, --param documentation for recipe deeplink and recipe open commands
  • Added usage examples demonstrating the --param flag

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
documentation/docs/guides/goose-cli-commands.md Updated flag reference table and recipe command documentation to reflect -p as alias for --param, added usage examples
documentation/docs/guides/recipes/session-recipes.md Added example showing how to use --param with recipe deeplink command

The documentation changes are accurate and consistent with the actual CLI implementation in crates/goose-cli/src/cli.rs. All examples are technically correct and the changes improve documentation clarity.

@github-actions
Copy link
Contributor

github-actions bot commented Dec 19, 2025

PR Preview Action v1.6.3
Preview removed because the pull request was closed.
2025-12-19 22:08 UTC

@dianed-square dianed-square merged commit e84d7dc into main Dec 19, 2025
27 checks passed
@dianed-square dianed-square deleted the docs/cli-recipe-param-option branch December 19, 2025 22:06
wpfleger96 added a commit that referenced this pull request Dec 23, 2025
* main: (155 commits)
  remove Tool Selection Strategy preview (#6250)
  fix(cli): correct bash syntax in terminal integration functions (#6181)
  fix : opening a session to view it modifies session history order in desktop (#6156)
  test: fix recipe and audio tests to avoid side effects (#6231)
  chore: Update gemini versions in test_providers.sh (#6246)
  feat: option to stream json - jsonl really (#6228)
  feat: add mcp app renderer (#6095)
  docs: update skills extension to support .agents/skills directories (#6199)
  Add YouTube short to Chrome DevTools MCP tutorial (#6244)
  docs: Caveats for privacy information in logs documentation (#6218)
  move goose issue solver to opus (#6233)
  feat: improved UX for tool calls via execute_code (#6205)
  Blog: Code Mode Doesn't Replace MCP (#6227)
  fix: prevent keychain requests during cargo test (#6219)
  test: fix test_max_turns_limit slow execution and wrong message type (#6221)
  Skills vs MCP blog (#6220)
  Add blog post: Does Your AI Agent Need a Plan? (#6209)
  fix(ui): enable MCP UI to send a prompt message when an element is clicked (#6207)
  docs: param option for recipe deeplink/open (#6206)
  docs: edit in place or fork session (#6203)
  ...
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