Repository navigation
fix(models): default to astra medium and fable 5.1 medium - #11347
Merged
maria-rcks merged 5 commits intoSep 12, 2026
Merged
MacroscopeApp / Macroscope - Correctness Check
succeeded
Sep 12, 2026 in 1m 39s
No issues identified (2 code objects reviewed).
• Reviewed files modified since
4d9db1b; other PR files not modified since then were skipped.
• Merge Base:36668db
• Head:ae8fa3b
Details
| ✅ | File Path | U3 Bytes | Comments Posted | Reason |
|---|---|---|---|---|
| ✅ | apps/server/src/provider/Layers/CodexProvider.ts |
439 | 0 | |
| ✅ | apps/server/src/provider/model-manifest.json |
276 | 0 |
Billed Total: 10.00KB of diff | $0.50 (This review was charged at our per-review byte minimum of 10.00KB. Learn more here)
Filtered Issues Details
apps/server/src/provider/Layers/CodexProvider.ts
- line 150: For an Astra catalog that does not advertise
medium(for example, an older catalog with supported effortslowandhighanddefaultReasoningEffort: "low"), this hard-coded comparison marks no option as default. ConsequentlydefaultReasoningisundefinedand the descriptor omitscurrentValue, instead of retaining the provider's supportedlowdefault. The earlier guarded implementation explicitly fell back tomodel.defaultReasoningEffortin this case. [ Out of scope (triage) ]
Loading