Skip to content

fix(registry): restore models[0] default + guards; note muse-spark overlay (#11051/#11049) - #11133

Merged
diegosouzapw merged 1 commit into
cheaper-inference:release/v3.8.50from
maxmad64bis:fix/11051-11049-registry
Aug 22, 2026
Merged

diegosouzapw merged 1 commit into
cheaper-inference:release/v3.8.50from
maxmad64bis:fix/11051-11049-registry

Conversation

@maxmad64bis

Copy link
Copy Markdown
Contributor

Summary

Follow-up to the already-merged #11051 and #11049.

Related Issues

Validation

Change type: provider / other (registry + tests)

  • Focused tests rerun: opencode-zen-go-shared-models.test.ts, check:provider-consistency (267/0), opencode-muse-spark-responses-10867, opencode-zen-muse-spark-targetformat-11046, opencode-go-catalog-alignment
  • npm run lint (pre-commit hooks ran on commit)
  • Reconciled with current active release base upstream/release/v3.8.50
  • Production-code change (registry) includes an automated test in this PR

Tests Added Or Updated

  • tests/unit/opencode-zen-go-shared-models.test.ts — two new narrow guards (non-shared id presence + models[0] default)

Coverage Notes

  • Registry open-sse/config/providers/registry/opencode/{go,zen}/index.ts changes are covered by the new opencode-zen-go-shared-models guards plus the existing check:provider-consistency (267 entries).

Reviewer Notes

…erlay (cheaper-inference#11051/cheaper-inference#11049)

cheaper-inference#11051 placed the SHARED model spread at index 0, silently changing the
dashboard default to kimi-k2.7-code for both opencode-go and opencode-zen.

- Move the spread after glm-5.2 (go) / big-pickle (zen) to restore the
  original models[0] (the dashboard default; ~14 call sites take models[0]).
  Exact pre-cheaper-inference#11051 interleaving not restorable with one spread and not required.
- Narrow guards: assert referenced non-shared ids (minimax-m3, glm-5.1)
  remain present, and models[0].id is glm-5.2 / big-pickle. No full-catalog
  snapshot (passthroughModels: true, live-regenerated).
- cheaper-inference#11049: one-line comment marking zen muse-spark-1.2-contributor-free as an
  explicit wire-format overlay of the base provider (avoid catalog-sync drift).

Why: owner flagged the silent default-model change (cheaper-inference#11051) + overlay-drift
risk (cheaper-inference#11049) on already-merged PRs.

Decision: restore models[0] only (not original interleaving); runtime-narrow
guards instead of a frozen golden list.

Co-Authored-By: opencode <noreply@opencode.ai>
@diegosouzapw
diegosouzapw merged commit efc7134 into cheaper-inference:release/v3.8.50 Aug 22, 2026
9 of 16 checks passed
@maxmad64bis
maxmad64bis deleted the fix/11051-11049-registry branch September 24, 2026 21:12
muhamadgalihsaputra pushed a commit to niyatna/NiyatnaRoute that referenced this pull request Sep 27, 2026
…erlay (cheaper-inference#11051/cheaper-inference#11049) (cheaper-inference#11133)

Validated on the combined batch board over release/v3.8.50 tip 0f43f0f: static gates clean, typecheck:core clean, focused tests green.

Restores the models[0] dashboard default silently changed by cheaper-inference#11051, with narrow guards instead of a brittle full snapshot. Thank you @maxmad64bis!
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.

2 participants