Skip to content

Remove unused exclusive model feature restriction#4581

Merged
chrarnoldus merged 5 commits into
mainfrom
remove-exclusive-model-feature-restriction
Jul 16, 2026
Merged

Remove unused exclusive model feature restriction#4581
chrarnoldus merged 5 commits into
mainfrom
remove-exclusive-model-feature-restriction

Conversation

@chrarnoldus

Copy link
Copy Markdown
Contributor

Summary

  • remove the unused KiloExclusiveModel.exclusive_to field from model definitions
  • delete the unreachable feature-specific catalog filtering and request rejection paths
  • remove the obsolete proxy error type and feature-filtering tests

Verification

  • git diff --check
  • confirmed no remaining exclusive_to, filterByFeature, isExcludedForFeature, featureExclusiveModelResponse, or feature_exclusive_model references under apps/web/src
  • targeted route tests attempted, but the local PostgreSQL test database was unavailable

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

kilo-code-bot Bot commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Executive Summary

The latest commit is a pure formatting change reflowing the GET handler's return type annotation onto multiple lines; no behavior change and no new issues found in the incremental diff.

Files Reviewed (1 file)
  • apps/web/src/app/api/openrouter/models/route.ts
Previous Review Summaries (4 snapshots, latest commit f084348)

Current summary above is authoritative. Previous snapshots are kept for context only.

Previous review (commit f084348)

Status: No Issues Found | Recommendation: Merge

Executive Summary

The latest commit only renames the unused request parameter to _request and reformats the return type annotation in the GET handler; no behavior change and no new issues found in the incremental diff.

Files Reviewed (1 file)
  • apps/web/src/app/api/openrouter/models/route.ts

Previous review (commit 6dabe5a)

Status: No Issues Found | Recommendation: Merge

Executive Summary

The latest commit only adds a missing findKiloExclusiveModel import in llm-proxy-helpers.ts, fixing a build/runtime bug left over from the previous commit; no new issues found in the incremental diff.

Files Reviewed (1 file)
  • apps/web/src/lib/ai-gateway/llm-proxy-helpers.ts

Previous review (commit dd7ee49)

Status: No Issues Found | Recommendation: Merge

Executive Summary

The latest commit only adds a missing findKiloExclusiveModel import in llm-proxy-helpers.ts, fixing a build/runtime bug left over from the previous commit; no new issues found in the incremental diff.

Files Reviewed (1 file)
  • apps/web/src/lib/ai-gateway/llm-proxy-helpers.ts

Previous review (commit 10ce4ce)

Status: No Issues Found | Recommendation: Merge

Executive Summary

This is a clean, mechanical removal of the unused exclusive_to model field and the feature-filtering/rejection code path, with no dangling references or logic errors found in the changed code.

Files Reviewed (16 files)
  • apps/web/src/app/admin/api/auto-routing/benchmark-config/route.test.ts
  • apps/web/src/app/api/openrouter/[...path]/route.ts
  • apps/web/src/app/api/openrouter/models/route.test.ts
  • apps/web/src/app/api/openrouter/models/route.ts
  • apps/web/src/app/api/openrouter/models/validate/route.ts
  • apps/web/src/app/api/organizations/[id]/models/route.test.ts
  • apps/web/src/app/api/organizations/[id]/models/route.ts
  • apps/web/src/app/api/organizations/[id]/models/validate/route.ts
  • apps/web/src/lib/ai-gateway/llm-proxy-helpers.ts
  • apps/web/src/lib/ai-gateway/model-api-kinds.test.ts
  • apps/web/src/lib/ai-gateway/models.ts
  • apps/web/src/lib/ai-gateway/providers/anthropic.constants.ts
  • apps/web/src/lib/ai-gateway/providers/deepseek.ts
  • apps/web/src/lib/ai-gateway/providers/google.ts
  • apps/web/src/lib/ai-gateway/providers/kilo-exclusive-model.test.ts
  • apps/web/src/lib/ai-gateway/providers/kilo-exclusive-model.ts
  • apps/web/src/lib/ai-gateway/providers/meta.ts
  • apps/web/src/lib/ai-gateway/providers/openai-exclusive.ts
  • apps/web/src/lib/ai-gateway/providers/qwen.ts
  • apps/web/src/lib/ai-gateway/providers/seed.ts
  • apps/web/src/lib/ai-gateway/providers/stepfun.ts
  • apps/web/src/lib/ai-gateway/providers/streamlake.ts
  • apps/web/src/lib/proxy-error-types.ts

Reviewed by claude-sonnet-5 · Input: 22 · Output: 3K · Cached: 466.9K

Review guidance: REVIEW.md from base branch main

Co-authored-by: kiloconnect[bot] <240665456+kiloconnect[bot]@users.noreply.github.com>
chrarnoldus and others added 3 commits July 16, 2026 10:19
Co-authored-by: kiloconnect[bot] <240665456+kiloconnect[bot]@users.noreply.github.com>
Co-authored-by: kiloconnect[bot] <240665456+kiloconnect[bot]@users.noreply.github.com>
Co-authored-by: kiloconnect[bot] <240665456+kiloconnect[bot]@users.noreply.github.com>
@chrarnoldus chrarnoldus merged commit 05683b8 into main Jul 16, 2026
15 checks passed
@chrarnoldus chrarnoldus deleted the remove-exclusive-model-feature-restriction branch July 16, 2026 11:23
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