Skip to content

Update documented reasoning_effort configuration values#2370

Open
PeterDaveHello wants to merge 1 commit intoThe-PR-Agent:mainfrom
PeterDaveHelloKitchen:align-reasoning-effort-values
Open

Update documented reasoning_effort configuration values#2370
PeterDaveHello wants to merge 1 commit intoThe-PR-Agent:mainfrom
PeterDaveHelloKitchen:align-reasoning-effort-values

Conversation

@PeterDaveHello
Copy link
Copy Markdown
Contributor

Document the full set of values already accepted by ReasoningEffort. This keeps the settings comment and usage guide aligned with runtime validation.

GitHub Copilot Pull Request Summary

This pull request updates the configuration options for reasoning effort in both the documentation and the configuration file. The main change is the expansion of the allowed values for the reasoning_effort setting, providing users with finer control over model reasoning levels.

Configuration and documentation updates:

  • Expanded the list of possible values for reasoning_effort from "low", "medium", "high" to include "none", "minimal", "low", "medium", "high", "xhigh" in both pr_agent/settings/configuration.toml and docs/docs/usage-guide/changing_a_model.md [1] [2].
  • Updated documentation to clarify that available reasoning_effort values depend on the model and provider, and removed the implication that only "low", "medium", or "high" are supported.

Document the full set of values already accepted by ReasoningEffort.
This keeps the settings comment and usage guide aligned with runtime validation.
@qodo-free-for-open-source-projects
Copy link
Copy Markdown
Contributor

Review Summary by Qodo

Document all supported reasoning_effort configuration values

📝 Documentation

Grey Divider

Walkthroughs

Description
• Expanded documented reasoning_effort values from three to six options
• Updated configuration file comments to reflect all supported values
• Clarified that available values depend on model and provider
• Aligned documentation with runtime validation capabilities
Diagram
flowchart LR
  A["reasoning_effort config"] -->|"old: low, medium, high"| B["Limited documentation"]
  A -->|"new: none, minimal, low, medium, high, xhigh"| C["Complete documentation"]
  C -->|"model/provider dependent"| D["Updated guidance"]
Loading

Grey Divider

File Changes

1. pr_agent/settings/configuration.toml 📝 Documentation +1/-1

Expand reasoning_effort configuration values documentation

• Updated reasoning_effort comment to list all six supported values
• Changed from "low", "medium", "high" to "none", "minimal", "low", "medium", "high", "xhigh"
• Maintains default value of "medium"

pr_agent/settings/configuration.toml


2. docs/docs/usage-guide/changing_a_model.md 📝 Documentation +2/-2

Update reasoning_effort documentation with all values

• Updated TOML example to show all six supported reasoning_effort values
• Clarified that available values depend on model and provider
• Changed guidance from listing specific values to referencing "any supported value"
• Improved user understanding of configuration flexibility

docs/docs/usage-guide/changing_a_model.md


Grey Divider

Qodo Logo

@qodo-free-for-open-source-projects
Copy link
Copy Markdown
Contributor

qodo-free-for-open-source-projects Bot commented May 3, 2026

Code Review by Qodo

🐞 Bugs (0) 📘 Rule violations (0) 📎 Requirement gaps (0)

Grey Divider

Great, no issues found!

Qodo reviewed your code and found no material issues that require review

Grey Divider

Qodo Logo

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.

1 participant