aaif-goose / aaif-goose/goose

Set request params per model settings in recipe

未关闭
#9,610 2 条评论 0 个 reaction 已指派 1 人 已被 @filipkujawa 认领 在 GitHub 查看
主要语言
Rust
星标
54.2k
派生
6.2k
平均合并
3 天 4 小时
30 天内合并 PR
240

描述

**Please explain the motivation behind the feature request.**
Right now one can only customize [temperature](https://github.com/aaif-goose/goose/blob/main/crates/goose/src/recipe/mod.rs#L107) for model responsible for driving recipe. There is no way to set however `reasoning_effort` or other (like `top_k` or `top_p` for open weights models) params. There is benefit in tweaking those per-recipe instead of going with global `GOOSE_PREDEFINED_MODELS`, as various recipes might benefit from different model tuning.

**Describe the solution you'd like**
Ideally model settings in recipe accept `request_params`.
From what I see that could be plugged in goose-clii [there](https://github.com/aaif-goose/goose/blob/main/crates/goose-cli/src/session/builder.rs#L232-L292).

**Describe alternatives you've considered**
One can workaround current limitation with having proxy between goose and provider(s) to provide model aliases so that via `GOOSE_PREDEFINED_MODELS` (or proxy alone) one could map those tweaks uniquely. That is however cumbersome and as far as understand will not work with ACP providers.

- [x] I have verified this does not duplicate an existing feature request

贡献指南

打开贡献指南

调研方向

The issue points to crates/goose/src/recipe/mod.rs for the current temperature setting and crates/goose-cli/src/session/builder.rs for where request_params could be integrated. Start by examining the ModelSettings struct in the recipe module to add a request_params field. Then, follow the builder pattern in session/builder.rs to propagate these params to the model request. Check for existing tests around model configuration to ensure compatibility.

由索引模型根据 Issue 内容生成。

评估

技术栈
rust
领域
ai-infra-agents, backend-api-design
Issue 类型
功能
难度
3/5
预计耗时
1-2 天
活跃度
活跃
描述清晰度
描述清楚
新手友好度
65/100

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。