feat: fix default and free models - #16922
Conversation
|
👋 This PR needs a couple of things fixed before OpenHands can review it:
Push an update once this is addressed and this check re-runs automatically. This is an automated check - no AI was used to generate this comment. |
Make Kimi K3 and GLM 5.2 billable while retaining DeepSeek V4 Flash as the free OpenHands default. Co-authored-by: openhands <openhands@all-hands.dev>
6a5defc to
7e4d62d
Compare
✅ Mock-LLM Docker E2E Test Results66/66 passed Commit: Details
Posted by the Mock-LLM E2E workflow · results are deterministic (scripted LLM responses) |
❌ Mock-LLM E2E Tests63/66 passed · 1 failed · 2 skipped Commit: Details
🔍 Failure details (1)❌ automations/mock-llm-automation.spec.ts › mock-LLM automation lifecycle › step 2: create automation and dispatch run via the UIPosted by the Mock-LLM E2E workflow · results are deterministic (scripted LLM responses) |
Co-authored-by: openhands <openhands@all-hands.dev>
Co-authored-by: openhands <openhands@all-hands.dev>
Co-authored-by: openhands <openhands@all-hands.dev>
|
🚦 CI is currently failing on this PR's latest commit. Please fix the failing checks before OpenHands reviews it - this is re-checked automatically once you push a new commit. (A maintainer can also request This is an automated check - no AI was used to generate this comment. |
|
@OpenHands please create an openhands/saas-deploy feature branch for this |
|
I'm on it! juanmichelini can track my progress at all-hands.dev |
|
@juanmichelini Done — created the It registers the One dev-catalog note surfaced in the PR body: This comment was created by an AI agent (OpenHands) on behalf of the user. |
|
Handled the PR comment. juanmichelini asked for an What I did:
No changes were needed in the checked-out frontend workspace; the work lived entirely in the saas-deploy repository. |
Co-authored-by: openhands <openhands@all-hands.dev>
|
Updated the onboarding regression assertion to expect |
|
needs human approval. Please replace the HUMAN: section placeholder with at least 20 characters describing what you tested; repository policy reserves this section for human contributors. |
Co-authored-by: openhands <openhands@all-hands.dev>
|
Fixed the remaining onboarding regression assertions to expect the OpenAI provider, GPT-5.6 Sol, and no OpenHands account helper. Pushed as |
|
PR Artifacts Cleaned Up The |
| @@ -17,15 +17,15 @@ interface SetupLlmStepProps { | |||
|
|
|||
| /** | |||
| * Pre-fills the LLM form with the OpenHands provider's free default model | |||
There was a problem hiding this comment.
This comment will no longer be correct when the model changes to openai/gpt-5.6-sol since it will not be pre-filled to a free default model.
There was a problem hiding this comment.
Fixed in commit a3062b0: corrected the JSDoc to describe the OpenAI GPT-5.6 Sol default and removed the stale OpenHands/free-model wording. This reply was created by an AI agent (OpenHands) on behalf of the user.
HUMAN:
I tested the onboarding LLM settings and verified the OpenAI / GPT-5.6 Sol default and Free model badge behavior.
AGENT:
Why
The OpenHands source default should use the OpenAI provider with GPT-5.6 Sol, matching the requested product decision. DeepSeek V4 Flash remains available as the sole free OpenHands-routed model.
Summary
openai/gpt-5.6-solthe Canvas and onboarding default.openhands/deepseek-v4-flashin the frontend free-model labels.Issue Number
Fixes #16914
How to Test
npm run make-i18nnpx vitest run __tests__/api/settings-service.test.ts __tests__/components/settings/llm-profiles/llm-settings-local-view.test.tsx __tests__/utils/format-model-name.test.tsnpm run typecheckVideo/Screenshots
Verification screenshots
gpt-5.6-solselected on first opening.deepseek-v4-flashis marked Free;glm-5.2is not.Validation
git diff --checkpassed.The SaaS pricing PR remains OpenHands/saas-deploy#801. The enterprise PR remains OpenHands/enterprise#253 and intentionally keeps its DeepSeek migration.
This pull request was created by an AI agent (OpenHands) on behalf of the user.
🐳 Docker images for this PR
• GHCR package: https://github.com/OpenHands/OpenHands/pkgs/container/agent-canvas
ghcr.io/openhands/agent-canvasghcr.io/openhands/agent-server:1.42.1-pythonopenhands-automation==1.8.076d5404bfcdff804abf239d0b523d3f78942747ePull (multi-arch manifest)
# Multi-arch manifest — Docker automatically pulls the correct architecture docker pull ghcr.io/openhands/agent-canvas:sha-76d5404Run
All tags pushed for this build
About Multi-Architecture Support
sha-76d5404) is a multi-arch manifest supporting both amd64 and arm64sha-76d5404-amd64) are also available if needed