Skip to content
Closed
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
10 changes: 9 additions & 1 deletion .github/workflows/opencode-review-dispatch.yml
Original file line number Diff line number Diff line change
Expand Up @@ -3361,6 +3361,14 @@ jobs:
"baseURL": "https://opencode.ai/zen/v1"
},
"models": {
"deepseek-v4-flash-free": {
"name": "DeepSeek V4 Flash Free",
"tool_call": true,
"limit": {
"context": 256000,
"output": 64000
}
},
"north-mini-code-free": {
"name": "North Mini Code Free",
"tool_call": true,
Expand Down Expand Up @@ -3707,7 +3715,7 @@ jobs:
# cost-efficient tier, cheaper than the legacy gpt-5 it replaced
# ($1/$6 vs $1.25/$10 per 1M tokens) so the org OpenAI budget
# stretches further between top-ups.
OPENCODE_MODEL_CANDIDATES: "${{ needs.validate-pr-metadata.outputs.is_private == 'false' && 'opencode-free/north-mini-code-free ' || '' }}github-models/deepseek/deepseek-v3-0324 openai/gpt-5.6-luna openrouter/deepseek/deepseek-v3.2 openrouter/qwen/qwen3-coder github-models/openai/gpt-4.1 github-models/openai/gpt-5 github-models/openai/gpt-5-chat github-models/openai/o3 github-models/deepseek/deepseek-r1-0528 github-models/deepseek/deepseek-r1"
OPENCODE_MODEL_CANDIDATES: "${{ needs.validate-pr-metadata.outputs.is_private == 'false' && 'opencode-free/deepseek-v4-flash-free opencode-free/north-mini-code-free ' || '' }}github-models/deepseek/deepseek-v3-0324 openai/gpt-5.6-luna openrouter/deepseek/deepseek-v3.2 openrouter/qwen/qwen3-coder github-models/openai/gpt-4.1 github-models/openai/gpt-5 github-models/openai/gpt-5-chat github-models/openai/o3 github-models/deepseek/deepseek-r1-0528 github-models/deepseek/deepseek-r1"
# One attempt per model, then fall through to the next model. Retrying
# the SAME model 5x let a rate-limited/hung leader consume the whole
# step, so the pool never reached a healthy fallback model.
Expand Down
22 changes: 20 additions & 2 deletions tests/test_opencode_agent_contract.py
Original file line number Diff line number Diff line change
Expand Up @@ -92,11 +92,13 @@ def test_opencode_model_pool_sets_high_effort_for_capable_candidates():
assert candidates_match is not None
conditional_public_candidate = (
"${{ needs.validate-pr-metadata.outputs.is_private == 'false' "
"&& 'opencode-free/north-mini-code-free ' || '' }}"
"&& 'opencode-free/deepseek-v4-flash-free "
"opencode-free/north-mini-code-free ' || '' }}"
)
candidates_text = candidates_match.group(1)
assert candidates_text.startswith(conditional_public_candidate)
candidates = [
"opencode-free/deepseek-v4-flash-free",
"opencode-free/north-mini-code-free",
*candidates_text.removeprefix(conditional_public_candidate).split(),
]
Expand All @@ -115,6 +117,7 @@ def test_opencode_model_pool_sets_high_effort_for_capable_candidates():

assert candidate_pairs
assert candidate_pairs == [
["opencode-free", "deepseek-v4-flash-free"],
["opencode-free", "north-mini-code-free"],
["github-models", "deepseek/deepseek-v3-0324"],
["openai", "gpt-5.6-luna"],
Expand All @@ -135,6 +138,20 @@ def test_opencode_model_pool_sets_high_effort_for_capable_candidates():
assert set(github_candidate_models).issubset(set(github_models))
assert '"context": 256000' in workflow
assert '"output": 64000' in workflow
generated_config_match = re.search(
r"jq -n '(\{.*?\})' >\"\$\{OPENCODE_REVIEW_WORKDIR\}/opencode\.jsonc\"",
workflow,
re.DOTALL,
)
assert generated_config_match is not None
generated_config = json.loads(generated_config_match.group(1))
free_models = generated_config["provider"]["opencode-free"]["models"]
deepseek_model = free_models["deepseek-v4-flash-free"]
north_model = free_models["north-mini-code-free"]
assert deepseek_model["tool_call"] is True
assert "response_format" not in deepseek_model.get("options", {})
assert north_model["tool_call"] is True
assert "response_format" not in north_model["options"]
assert github_candidate_models == [
"deepseek/deepseek-v3-0324",
"openai/gpt-4.1",
Expand Down Expand Up @@ -1212,7 +1229,8 @@ def test_workflow_provisions_sandbox_tool_and_reviewer_agent():
)
assert (
"needs.validate-pr-metadata.outputs.is_private == 'false' && "
"'opencode-free/north-mini-code-free ' || ''"
"'opencode-free/deepseek-v4-flash-free "
"opencode-free/north-mini-code-free ' || ''"
) in workflow
assert (
"github-models/deepseek/deepseek-v3-0324 "
Expand Down
Loading