Skip to content

fix(providers): complete OpenCode Go effort alias exposure - #8610

Merged
diegosouzapw merged 2 commits into
diegosouzapw:release/v3.8.49from
excessivechaos:fix/opencode-go-effort-catalog-context
Jul 27, 2026
Merged

diegosouzapw merged 2 commits into
diegosouzapw:release/v3.8.49from
excessivechaos:fix/opencode-go-effort-catalog-context

Conversation

@excessivechaos

@excessivechaos excessivechaos commented Jul 25, 2026 •

Copy link
Copy Markdown
Contributor

Summary

Follow-up to #8353 and #8441.

PR #8441 added the OpenCode Go effort aliases to the registry and executor, but two downstream paths still dropped or excluded them:

  • The Playground catalog suppressed all static registry entries whenever synced models existed, so registered effort aliases were absent from /v1/models.
  • The combo compatibility resolver treated Hunyuan3 aliases as having an unknown context window, so hy3 was removed from large tool-heavy requests even when the request fit its confirmed 256K context.

This PR preserves registered effort aliases beside synced base models and adds the confirmed 256K context metadata to all Hunyuan3 variants.

Regression Coverage

  • ests/unit/models-catalog-route.test.ts
  • synced OpenCode Go base plus registered hy3 aliases appear in the Playground catalog
  • ests/unit/opencode-go-catalog-alignment.test.ts
  • all Hunyuan3 variants resolve a 256K context window for combo filtering

Validation

  • Focused catalog tests: 44 passed
  • OpenCode Go registry/executor tests: 22 passed
  • Changed production files lint: passed
  • Changelog integrity: passed
  • Core typecheck: passed
  • Production compilation and static generation: passed (655/655 pages)
  • Standalone assembly: emitted a non-fatal Windows ERR_FS_CP_EINVAL because the isolated worktree's linked
    ode_modules made a native asset source and destination identical; this is a local worktree packaging condition, not a source compilation error
  • Full repository lint reports 81 pre-existing
    o-explicit-any violations in the existing catalog test file; this PR adds no new instances

@excessivechaos
excessivechaos marked this pull request as ready for review July 25, 2026 19:15
@diegosouzapw

Copy link
Copy Markdown
Owner

Hi @excessivechaos — thanks for the OpenCode Go effort alias exposure. Compact, focused change (+78 / −6).

Approved as merge-ready (★4). The full combo+routing suite will run via /merge-prs to confirm capability-table behavior stays consistent across non-hy3 OpenCode models.

@diegosouzapw
diegosouzapw merged commit 052cab3 into diegosouzapw:release/v3.8.49 Jul 27, 2026
3 checks passed
diegosouzapw added a commit that referenced this pull request Jul 27, 2026
…stryker drift

file-size: nine frozen entries could not absorb the combined result of the
31-PR train. Two distinct causes, kept apart in the baseline note on purpose:

  (1) GENUINE irreducible growth at existing chokepoints —
      providerLimits/auth (#8632), rateLimitManager (#8616),
      models-catalog-route.test (#8610).
  (2) COLLISION with #8585, which banked shrinks measured on the pre-train
      release tip while 30 sibling PRs in the SAME train grew those files
      again — chat/accountFallback (#8628), chatCore (#8613),
      videoGeneration (#8581), imageGeneration.

Ceilings re-pinned to the post-merge tip. #8612 (also in this train) automates
shrink-banking so this self-inflicted drift stops recurring.

stryker: three covering unit tests were missing from tap.testFiles —
isLocalStreamLifecycleError-abort-shape (circuitBreaker.ts, a shared base-red
that was reddening Fast Quality Gates on every open PR),
noauth-autocombo-lockout-7623 (accountFallback.ts) and
kimi-quota-reset-recovery (auth.ts), the latter two landed with this train.
HouMinXi pushed a commit to HouMinXi/OmniRoute that referenced this pull request Aug 2, 2026
…apw#8610)

* fix(providers): complete OpenCode Go effort aliases

* chore: number OpenCode Go changelog fragment
HouMinXi pushed a commit to HouMinXi/OmniRoute that referenced this pull request Aug 2, 2026
…stryker drift

file-size: nine frozen entries could not absorb the combined result of the
31-PR train. Two distinct causes, kept apart in the baseline note on purpose:

  (1) GENUINE irreducible growth at existing chokepoints —
      providerLimits/auth (diegosouzapw#8632), rateLimitManager (diegosouzapw#8616),
      models-catalog-route.test (diegosouzapw#8610).
  (2) COLLISION with diegosouzapw#8585, which banked shrinks measured on the pre-train
      release tip while 30 sibling PRs in the SAME train grew those files
      again — chat/accountFallback (diegosouzapw#8628), chatCore (diegosouzapw#8613),
      videoGeneration (diegosouzapw#8581), imageGeneration.

Ceilings re-pinned to the post-merge tip. diegosouzapw#8612 (also in this train) automates
shrink-banking so this self-inflicted drift stops recurring.

stryker: three covering unit tests were missing from tap.testFiles —
isLocalStreamLifecycleError-abort-shape (circuitBreaker.ts, a shared base-red
that was reddening Fast Quality Gates on every open PR),
noauth-autocombo-lockout-7623 (accountFallback.ts) and
kimi-quota-reset-recovery (auth.ts), the latter two landed with this train.
muhamadgalihsaputra pushed a commit to niyatna/NiyatnaRoute that referenced this pull request Sep 27, 2026
…apw#8610)

* fix(providers): complete OpenCode Go effort aliases

* chore: number OpenCode Go changelog fragment
muhamadgalihsaputra pushed a commit to niyatna/NiyatnaRoute that referenced this pull request Sep 27, 2026
…stryker drift

file-size: nine frozen entries could not absorb the combined result of the
31-PR train. Two distinct causes, kept apart in the baseline note on purpose:

  (1) GENUINE irreducible growth at existing chokepoints —
      providerLimits/auth (diegosouzapw#8632), rateLimitManager (diegosouzapw#8616),
      models-catalog-route.test (diegosouzapw#8610).
  (2) COLLISION with diegosouzapw#8585, which banked shrinks measured on the pre-train
      release tip while 30 sibling PRs in the SAME train grew those files
      again — chat/accountFallback (diegosouzapw#8628), chatCore (diegosouzapw#8613),
      videoGeneration (diegosouzapw#8581), imageGeneration.

Ceilings re-pinned to the post-merge tip. diegosouzapw#8612 (also in this train) automates
shrink-banking so this self-inflicted drift stops recurring.

stryker: three covering unit tests were missing from tap.testFiles —
isLocalStreamLifecycleError-abort-shape (circuitBreaker.ts, a shared base-red
that was reddening Fast Quality Gates on every open PR),
noauth-autocombo-lockout-7623 (accountFallback.ts) and
kimi-quota-reset-recovery (auth.ts), the latter two landed with this train.
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