Exclude slackbot-only models from /openrouter/models by default

[kiloconnect]

Summary

The /openrouter/models and /api/organizations/[id]/models endpoints now exclude slackbot-only models (e.g. anthropic/claude-opus-4.6:slackbot, minimax/minimax-m2.1:slackbot) by default. These models are only relevant for Kilo for Slack integrations and should not appear in the standard model selector.

Changes

Backend

• src/lib/providers/openrouter/index.ts: enhancedModelList() and getEnhancedOpenRouterModels() now accept an { includeSlackbotOnly } option. When false (default), models with slackbot_only: true are filtered out.
• src/app/api/openrouter/models/route.ts: Reads ?includeSlackbotOnly=true query parameter and passes it through.
• src/app/api/organizations/[id]/models/route.ts: Reads ?includeSlackbotOnly=true query parameter and passes it through to the tRPC listAvailableModels procedure.
• src/routers/organizations/organization-settings-router.ts: listAvailableModels accepts includeSlackbotOnly in its input and passes it to getEnhancedOpenRouterModels().
• src/app/api/cron/sync-model-stats/route.ts: Explicitly passes { includeSlackbotOnly: true } to preserve complete model set for stats syncing.

Frontend

• src/app/api/openrouter/hooks.ts: useModelSelectorList() accepts an optional second argument { includeSlackbotOnly }. When true, appends the query param to the fetch URL. The query key includes this flag so React Query caches them separately.

Usage

// Default: no slackbot-only models
useModelSelectorList(organizationId)

// Include slackbot-only models
useModelSelectorList(organizationId, { includeSlackbotOnly: true })

GET /api/openrouter/models                          # excludes slackbot-only
GET /api/openrouter/models?includeSlackbotOnly=true  # includes slackbot-only

GET /api/organizations/{id}/models                          # excludes slackbot-only
GET /api/organizations/{id}/models?includeSlackbotOnly=true  # includes slackbot-only

Built for Christiaan by Kilo for Slack

[kiloconnect]

Code Review Summary

Status: 1 Issues Found | Recommendation: Address before merge

Overview

Severity	Count
CRITICAL	0
WARNING	1
SUGGESTION	0

Issue Details (click to expand)

WARNING

File	Line	Issue
src/app/api/openrouter/hooks.ts	120	includeSlackbotOnly option is ignored when organizationId is set

Other Observations (not in diff)

Issues found in unchanged code that cannot receive inline comments:

File	Line	Issue

Files Reviewed (6 files)

• src/app/api/cron/sync-model-stats/route.ts - 0 issues
• src/app/api/openrouter/hooks.ts - 1 issue
• src/app/api/openrouter/models/route.ts - 0 issues
• src/app/api/organizations/[id]/models/route.ts - 0 issues
• src/lib/providers/openrouter/index.ts - 0 issues
• src/routers/organizations/organization-settings-router.ts - 0 issues

Fix these issues in Kilo Cloud

[kiloconnect]

getEnhancedOpenRouterModels consumer audit

Consumer	File	Should filter slackbot_only?	Action taken
/api/openrouter/models	src/app/api/openrouter/models/route.ts	✅ Yes — user-facing model selector	Already filtered (PR #149)
/api/organizations/[id]/models	src/app/api/organizations/[id]/models/route.ts → organization-settings-router.ts	✅ Yes — org model selector, same use case	Updated: filters by default, ?includeSlackbotOnly=true to opt in
/api/organizations/[id]/defaults	src/app/api/organizations/[id]/defaults/route.ts	✅ Yes — picks a default model for the org	No change needed: already uses default getEnhancedOpenRouterModels() which filters out slackbot_only
/api/cron/sync-model-stats	src/app/api/cron/sync-model-stats/route.ts	❌ No — needs complete model set for stats sync	Updated: explicitly passes { includeSlackbotOnly: true } to preserve current behavior

[kiloconnect]

WARNING: includeSlackbotOnly option is ignored when organizationId is set

useModelSelectorList() adds includeSlackbotOnly to the query key and builds a URL for /api/openrouter/models, but when organizationId is present it fetches /api/organizations/:id/models without forwarding the query param. That makes includeSlackbotOnly a no-op for org-scoped requests (likely the common case).

Suggested change

	organizationId ? `/api/organizations/${organizationId}/models` : modelsUrl
	organizationId ? `/api/organizations/${organizationId}/models${includeSlackbotOnly ? '?includeSlackbotOnly=true' : ''}` : modelsUrl