Skip to content

fix(ai-gateway): report exclusive provider data collection - #5346

Merged
chrarnoldus merged 2 commits into
mainfrom
inference-provider-data-collection
Aug 18, 2026
Merged

fix(ai-gateway): report exclusive provider data collection#5346
chrarnoldus merged 2 commits into
mainfrom
inference-provider-data-collection

Conversation

@chrarnoldus

Copy link
Copy Markdown
Contributor

Summary

  • resolve a Kilo-exclusive model with one inference-provider restriction before falling back to its gateway
  • derive the provider training indicator from requires-data-collection for both restricted providers and concrete gateway providers
  • share the provider metadata return path after resolving the provider slug

Verification

  • pnpm --filter web test -- --runInBand src/lib/ai-gateway/providers/kilo-exclusive-model.test.ts src/lib/ai-gateway/models.test.ts
  • pnpm --filter web typecheck
  • pnpm exec oxlint --config .oxlintrc.json apps/web/src/lib/ai-gateway/providers/kilo-exclusive-model.ts apps/web/src/lib/ai-gateway/providers/kilo-exclusive-model.test.ts
  • git diff --check

@chrarnoldus chrarnoldus self-assigned this Aug 18, 2026
@kilo-code-bot

kilo-code-bot Bot commented Aug 18, 2026

Copy link
Copy Markdown
Contributor

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Executive Summary

Reviewed restriction-first getInferenceProvider resolution and requires-data-collection training mapping; no correctness or security issues in the two-file change.

Files Reviewed (2 files)
  • apps/web/src/lib/ai-gateway/providers/kilo-exclusive-model.ts
  • apps/web/src/lib/ai-gateway/providers/kilo-exclusive-model.test.ts
Previous Review Summary

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

Previous review

Status: No Issues Found | Recommendation: Merge

Executive Summary

Reviewed getInferenceProvider restriction-first resolution and requires-data-collection training mapping; no correctness or security issues in the two-file change.

Files Reviewed (2 files)
  • apps/web/src/lib/ai-gateway/providers/kilo-exclusive-model.ts
  • apps/web/src/lib/ai-gateway/providers/kilo-exclusive-model.test.ts

Reviewed by grok-4.6 · Input: 123.3K · Output: 12.6K · Cached: 336.1K

Review guidance: REVIEW.md from base branch main

@chrarnoldus
chrarnoldus merged commit caea0f7 into main Aug 18, 2026
13 checks passed
@chrarnoldus
chrarnoldus deleted the inference-provider-data-collection branch August 18, 2026 15:04
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