Skip to content

fix(auto-model-change-log): drop removed providerDenyList field#3092

Merged
chrarnoldus merged 2 commits intomainfrom
fix/auto-model-change-log-provider-deny-list
May 7, 2026
Merged

fix(auto-model-change-log): drop removed providerDenyList field#3092
chrarnoldus merged 2 commits intomainfrom
fix/auto-model-change-log-provider-deny-list

Conversation

@kilo-code-bot
Copy link
Copy Markdown
Contributor

@kilo-code-bot kilo-code-bot Bot commented May 7, 2026

Summary

Build was failing with Property 'providerDenyList' does not exist on type 'ModelRestrictions' in apps/web/src/lib/organizations/auto-model-change-log.ts. The providerDenyList field was removed from ModelRestrictions in #2861, but this file still referenced it.

Removes the providerDenySet from the precomputed restrictions and simplifies checkAvailability: when no providerAllowList is configured, every provider that offers the model now admits it (deterministically picking the first alphabetically as the reason provider).

Verification

N/A — build error fix; CI typecheck will confirm.

Visual Changes

N/A

Reviewer Notes

The behavior change for orgs without a providerAllowList matches the new semantics established by #2861 (no provider deny list anymore, so any provider offering the model is acceptable).

@kilo-code-bot
Copy link
Copy Markdown
Contributor Author

kilo-code-bot Bot commented May 7, 2026

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Files Reviewed (2 files)
  • apps/web/src/lib/organizations/auto-model-change-log.ts
  • apps/web/src/lib/organizations/auto-model-change-log.test.ts

Reviewed by gpt-5.5-20260423 · 216,310 tokens

@chrarnoldus chrarnoldus enabled auto-merge (squash) May 7, 2026 09:14
@chrarnoldus chrarnoldus merged commit 5714774 into main May 7, 2026
13 checks passed
@chrarnoldus chrarnoldus deleted the fix/auto-model-change-log-provider-deny-list branch May 7, 2026 09:18
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.

3 participants