Skip to content

fix: disable Tencent Hy3 free model - #5764

Merged
chrarnoldus merged 1 commit into
mainfrom
fix/disable-tencent-free-model
Aug 31, 2026
Merged

fix: disable Tencent Hy3 free model#5764
chrarnoldus merged 1 commit into
mainfrom
fix/disable-tencent-free-model

Conversation

@chrarnoldus

Copy link
Copy Markdown
Contributor

Summary

  • Disable tencent/hy3:free, excluding it from Auto Free routing and preferred models.
  • Mark the free model unavailable for stale clients without disabling paid tencent/hy3.
  • Update regression coverage and use an active free model for the rate-limiting test.

Verification

  • Passed: formatting, targeted oxlint on all six changed files, and git diff --check.
  • Local pnpm --filter web typecheck exceeded the two-minute environment limit during the tRPC build; CI will verify types.
  • Tests intentionally delegated to CI as requested.

Co-authored-by: kiloconnect[bot] <240665456+kiloconnect[bot]@users.noreply.github.com>
@chrarnoldus chrarnoldus self-assigned this Aug 31, 2026
@kilo-code-bot

kilo-code-bot Bot commented Aug 31, 2026

Copy link
Copy Markdown
Contributor

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Executive Summary

Disabling tencent/hy3:free via exclusive status and the unavailable-model list follows existing patterns and leaves paid tencent/hy3 intact.

Files Reviewed (6 files)
  • apps/web/src/app/api/openrouter/[...path]/route.test.ts
  • apps/web/src/lib/ai-gateway/models.test.ts
  • apps/web/src/lib/ai-gateway/providers/tencent.ts
  • apps/web/src/lib/ai-gateway/unavailable-models.test.ts
  • apps/web/src/lib/ai-gateway/unavailable-models.ts
  • apps/web/src/tests/openrouter-models-config.test.ts

Reviewed by grok-4.6 · Input: 189.9K · Output: 9.9K · Cached: 393.2K

Review guidance: REVIEW.md from base branch main

@chrarnoldus
chrarnoldus merged commit 6f490c6 into main Aug 31, 2026
17 checks passed
@chrarnoldus
chrarnoldus deleted the fix/disable-tencent-free-model branch August 31, 2026 08:48
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