Skip to content

revert(models): partial revert of PR #1034, keep Vercel routing change#1043

Closed
kilo-code-bot[bot] wants to merge 1 commit intomainfrom
revert-pr-1034-partial
Closed

revert(models): partial revert of PR #1034, keep Vercel routing change#1043
kilo-code-bot[bot] wants to merge 1 commit intomainfrom
revert-pr-1034-partial

Conversation

@kilo-code-bot
Copy link
Contributor

@kilo-code-bot kilo-code-bot bot commented Mar 11, 2026

Summary

  • Partial revert of PR feat(models): add NVIDIA Nemotron 3 Super free model and launch notification #1034 (feat(models): add NVIDIA Nemotron 3 Super free model and launch notification)
  • Removes the nvidia/nemotron-3-super-120b-a12b:free entry from preferredModels in src/lib/models.ts
  • Removes the NVIDIA Nemotron 3 Super launch notification from src/lib/notifications.ts
  • Restores preferredIndex values in src/tests/openrouter-models-sorting.approved.json that were shifted by the model insertion
  • Intentionally keeps the Vercel routing exclusion for NVIDIA models in src/lib/providers/vercel/index.ts

Verification

Visual Changes

N/A

Reviewer Notes

The change to src/lib/providers/vercel/index.ts from PR #1034 (NVIDIA model Vercel routing exclusion) is preserved because that routing guard is still useful regardless of whether the model is in the preferred list.

…notification

Partial revert of PR #1034. Removes the Nemotron model from
preferredModels, removes the launch notification, and restores
preferredIndex values in the approved sorting snapshot.

The Vercel routing exclusion for NVIDIA models is intentionally kept.
@kilo-code-bot
Copy link
Contributor Author

kilo-code-bot bot commented Mar 11, 2026

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Files Reviewed (3 files)
  • src/lib/models.ts
  • src/lib/notifications.ts
  • src/tests/openrouter-models-sorting.approved.json

Reviewed by gpt-5.4-20260305 · 499,719 tokens

@chrarnoldus chrarnoldus deleted the revert-pr-1034-partial branch March 11, 2026 22:35
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.

1 participant