Skip to content

feat: make gpt-5.6-luna the default OpenAI model - #2007

Merged
ahuang11 merged 1 commit into
mainfrom
fix/2002-gpt-5.6-luna-default
Aug 13, 2026
Merged

feat: make gpt-5.6-luna the default OpenAI model#2007
ahuang11 merged 1 commit into
mainfrom
fix/2002-gpt-5.6-luna-default

Conversation

@ghostiee-11

Copy link
Copy Markdown
Collaborator

Makes gpt-5.6-luna the default OpenAI model, adapting from the API's own 400 rather than hardcoding which models reject temperature or reasoning_effort.

input (cached) output cost
gpt-5.6-luna 8,097 (2,652) 534 0.178 cents
gpt-5.4-mini 6,218 (1,152) 527 0.626 cents

3.5x cheaper on the same question.

Fixes #2002

gpt-5.6-luna is cheaper than gpt-5.4-mini and carries a much larger
context, but chat completions rejects function tools while reasoning is
active, and the model rejects a non-default temperature while reasoning
is on. Lumen sends a temperature on every request and uses function
tools for structured output, so the id swap alone fails on every call.

Rather than hardcode which models need which workaround, adapt from the
API's own 400: it names the offending parameter, so OpenAI.run_client
records the fix per model and retries. New models need no code change,
and providers that accept the parameters are never touched.

Fixes #2002
@ghostiee-11
ghostiee-11 requested a review from ahuang11 August 12, 2026 21:32
@codecov

codecov Bot commented Aug 12, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 99.01961% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 73.06%. Comparing base (6e01d8a) to head (7d92e37).

Files with missing lines Patch % Lines
lumen/ai/llm.py 96.96% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2007      +/-   ##
==========================================
+ Coverage   72.98%   73.06%   +0.07%     
==========================================
  Files         205      205              
  Lines       36865    36967     +102     
==========================================
+ Hits        26907    27011     +104     
+ Misses       9958     9956       -2     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@ahuang11
ahuang11 merged commit 7d587f3 into main Aug 13, 2026
14 checks passed
@ahuang11
ahuang11 deleted the fix/2002-gpt-5.6-luna-default branch August 13, 2026 16:00
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.

Make gpt-5.6-luna default

2 participants