Skip to content

fix(recipe): read prompted file parameters#8504

Open
vonbai wants to merge 1 commit intoaaif-goose:mainfrom
vonbai:goalx/recipe-file-params
Open

fix(recipe): read prompted file parameters#8504
vonbai wants to merge 1 commit intoaaif-goose:mainfrom
vonbai:goalx/recipe-file-params

Conversation

@vonbai
Copy link
Copy Markdown

@vonbai vonbai commented Apr 13, 2026

Summary

  • Read file contents for prompted recipe parameters declared with input_type: file.
  • Preserve the existing prompt-building flow for other parameter input types.
  • Add regression coverage so a file parameter is inserted as contents instead of a literal path string.

Validation

  • cargo fmt --check
  • cargo test -p goose --no-default-features --features rustls-tls recipe::build_recipe::tests::file_parameter_tests::test_build_recipe_user_prompt_file_parameter_reads_file_content

Note: the focused test run emitted the existing providers test dead-code warning for clear_env; validation still passed.

Signed-off-by: vonbai <nswanqi@gmail.com>
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