Repository navigation
Conversation
PR Code Suggestions ✨No code suggestions found for the PR. ⚙️ Agent run details
|
There was a problem hiding this comment.
🟡 Changes recommended
The generated reference omits a required warning and presents a currently unsafe routing example.
1 open finding
What changed in this PR
Updates PR-Agent’s default model chain to GPT-6.1 Sol with GPT-6 Luna fallback.
Changes:
- Updates active model defaults and routing examples.
- Documents pricing, reasoning behavior, compatibility, and collision limitations.
- Makes per-directory fallback tests independent of shipped defaults.
| File | Description |
|---|---|
pr_agent/settings/configuration.toml |
Sets new defaults and examples. |
docs/docs/usage-guide/configuration_reference.md |
Updates generated defaults and routing example. |
docs/docs/usage-guide/changing_a_model.md |
Documents migration and compatibility details. |
tests/unittest/test_per_directory_settings.py |
Adds explicit root fallback fixtures. |
🧠 Review effort: Balanced
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
PeterDaveHello
force-pushed
the
chore/gpt-6-defaults-after-release
branch
from
October 12, 2026 18:26
da6ef00 to
b9bbd72
Compare
naorpeled
reviewed
Oct 16, 2026
| fallback_models = ["..."] | ||
| ``` | ||
|
|
||
| :::note[Default model choice] |
Member
There was a problem hiding this comment.
Can you please drop this change? 🙏
We'll soon upgrade litellm so it won't be necessary
At medium effort, Artificial Analysis reports Sol scoring 48 versus 39 for GPT-5.6 Sol, at $0.21 versus $0.50 per index task. Luna and Terra both score 30, at $0.02 versus $0.18. These benchmark results are not measurements of PR-Agent review quality. OpenAI reports stronger research debugging for Sol, but comparable KernelGen performance. Keep evaluation rationale in this message and the PR description rather than in long-lived usage documentation. Align model examples, clarify effort normalization, and warn about optional Luna retries. Keep the routing prerequisite in the generated rules description and cover warning retention with a regression test. Keep the root-fallback fixture independent of the shipped defaults.
PeterDaveHello
force-pushed
the
chore/gpt-6-defaults-after-release
branch
from
October 16, 2026 20:12
b9bbd72 to
c7016f2
Compare
naorpeled
approved these changes
Oct 16, 2026
IsmaelMartinez
added a commit
to PeterDaveHello/pr-agent
that referenced
this pull request
Oct 18, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.

User description
Summary
Use
gpt-6.1-solas the default primary model andgpt-6-lunaas the fallback. Keep GPT-5.x models selectable; existing explicit model configurations remain supported.Update the commented reasoning/weak-model examples, routing examples, and generated configuration reference together. The second routing rule retains the Sol primary. The per-directory settings test supplies its own root fallback list instead of depending on the shipped default. The generated routing-rules description now retains the collision-repair prerequisite alongside the examples; a regression test prevents that warning from being lost.
Remove the added
Default model choicenote to avoid maintaining model prices and version-specific cost-map advice in the usage guide. Model-selection evidence remains in this description.Only
config.modelandconfig.fallback_modelschange as active runtime settings. Model handlers, registries, dependencies, token limits, and retry implementation remain unchanged.Why these defaults
Independent evaluation at the same reasoning effort
The Artificial Analysis Intelligence Index v4.3.2 provides a capability reference beyond token prices. Every row uses medium, matching the default
config.reasoning_effort; scores and benchmark-task costs come from the same medium-variant page. Values checked October 12, 2026.The index combines 10 evaluations: AA-Briefcase v1.1, GDPval-AA v2.1, AutomationBench-AA, Terminal-Bench 4.0, SciCode, Humanity's Last Exam, GDP.pdf, CritPt, AA-Omniscience, and AA-LCR v1.1. These span work tasks, coding/terminal use, knowledge/reasoning, and long-context reasoning; see the methodology. At that snapshot, SciCode and CritPt were marked "Under review" by the evaluator.
This is a composite index, not a coding-only score or a measurement of PR-Agent review quality. Equal aggregate scores do not imply equal capabilities on individual tasks, and index-point gains are not percentage improvements in review quality. The cited pages do not expose usable constituent scores for these comparisons, so no individual coding-benchmark superiority is claimed.
Task cost is the evaluator's weighted average, accounting for input, cache-hit, cache-write, reasoning, and answer tokens. Percentage reductions above use displayed rounded costs; they are not predictions of cost per pull request. Results can change as evaluations or pricing are updated.
Vendor debugging evaluation
OpenAI's GPT-6.1 Sol system card reports an internal research-debugging mean rubric score of 75.52%, versus 68.32% for GPT-5.6 Sol (+7.20 percentage points). It reports comparable KernelGen 1P performance, not a universal improvement.
These are vendor results, not independent replication or the same medium-normalized comparison as the table above. They do not directly measure PR-Agent review quality or establish that Luna outperforms Terra. Evaluation rationale stays in the commit/PR description rather than requiring ongoing benchmark maintenance in the usage guide.
Published token prices
Standard API prices, USD per million tokens, checked October 12, 2026:
gpt-5.6is the alias for GPT-5.6 Sol, so the migration comparison uses the actual previous defaults. Input/cached-input/output unit prices decrease by 50%/75%/50% for the primary and 95%/95%/approximately 95.83% for the fallback.These are Standard rates for prompts with no more than 272K input tokens. GPT-5.6 Sol's quoted prices include its current promotional pricing, published as available at least through November 21, 2026. Token usage, reasoning effort, cache writes, long-context pricing, processing tiers, and regional premiums affect the final bill; lower unit prices do not guarantee lower total cost.
Compatibility and known limitations
config.model_weakunset to use the normal chain for weak-model calls. Explicitly enabling the commented Luna weak-model example with Luna as fallback creates duplicate Luna attempts; the warnings document this limitation rather than changing retry behavior.minimaltolowand Sol's unsupportednonetolow; Luna supportsnone. The guide clarifies these settings. Usable context remains capped byconfig.max_model_tokens.Validation
PR Type
Enhancement, Documentation, Tests
Description
Default to GPT-6.1 Sol with Luna fallback.
Align model examples and configuration reference.
Document pricing limits, compatibility, and retry caveats.
Decouple fallback preservation tests from shipped defaults.
Diagram Walkthrough
File Walkthrough
configuration.toml
Adopt Sol primary and Luna fallback defaultspr_agent/settings/configuration.toml
config.modeltogpt-6.1-solandconfig.fallback_modelsto["gpt-6-luna"].duplicate Luna retries.
collision repair.
changing_a_model.md
Document new defaults and model migration caveatsdocs/docs/usage-guide/changing_a_model.md
metadata limitations.
normalization, and context limits.
repair requirement.
for the second rule.
configuration_reference.md
Align configuration reference with new model defaultsdocs/docs/usage-guide/configuration_reference.md
configuration.toml.gpt-6-lunaandgpt-6.1-sol.test_per_directory_settings.py
Isolate root fallback preservation from shipped defaultstests/unittest/test_per_directory_settings.py
list.
shipped defaults.
fallbacks.
⚙️ Agent run details