[felipesartori]

Problem

Adding a provider connection auto-imports all of the provider's models. Users who only want the provider's free models had no way to opt in, and the model list had no way to sort/filter by free vs paid.

What this adds

"Import only free models" toggle

• Shown in Add API Key and Edit Connection, only for providers that have documented free models (derived from FREE_MODEL_BUDGETS).
• Persists as providerSpecificData.importFreeModelsOnly.
• Editing an existing connection and changing the flag auto-triggers a replace re-sync, so the available model list reflects the new choice immediately.

Model sync

• sync-models filters fetched models to free-only when the flag is set, and returns freeFilterEmpty so the UI shows a clear "no free models found" message instead of a silent empty import.

Model list controls (Compatible / Passthrough / base sections)

• Free first sort and All / Free only / Paid only view filters in the toolbar (view-only — no routing/persistence change).
• The Free badge now reflects catalog-detected free models.

Free detection — src/shared/utils/freeModels.ts

• isFreeModel: :free suffix, zero prompt+completion price, or membership in the curated free-tier catalog. Resolves provider alias → canonical id (e.g. ollamacloud → ollama-cloud) so catalog lookups match.
• sortModelsFreeFirst: deterministic free-first ordering (alphabetical tie-break) so order stays stable across re-renders/refetches (e.g. while Test all runs).
• providerHasFreeModels / selectModelsForImport helpers.

Known limitation (by design)

"Free" is not machine-readable for every provider. For example, NVIDIA's "Free Endpoint" set is not exposed by any API — it's only knowable by probing each model (HTTP 200/429 = hosted). Those providers rely on the existing Test all models + Auto-hide failed flow. Documented in code comments.

Tests

• tests/unit/free-models.test.ts — detection (:free / zero-price / catalog), alias resolution, import filter (incl. freeFilterEmpty), and free-first sort (stability + no mutation).
• tests/unit/add-api-key-modal-free-models.test.tsx — toggle visibility + payload.
• tests/unit/edit-connection-modal-free-models.test.tsx — toggle visibility, persisted-state reflection, explicit-false on untoggle (regression for the PUT merge keeping stale true), and auto re-sync.
• tests/unit/model-visibility-toolbar-free.test.tsx — sort + free/paid filter controls.

Validation (Hard Rule #18)

TDD: each behavior added as a failing test first, then implemented. All green:

• node --test tests/unit/free-models.test.ts → 16 passed
• vitest component tests → 20 passed (incl. existing connModals regression)
• typecheck:core clean, ESLint 0 errors, i18n UI coverage ≥65 passes.

Also manually verified locally against live OpenRouter and ollama-cloud connections.

Notes

• No DB schema/migration changes — uses the existing providerSpecificData JSON column.
• i18n keys added to en / pt-BR / pt; toolbar labels use providerText fallback for other locales (consistent with existing toolbar keys).

🤖 Generated with Claude Code

[gemini-code-assist]

Code Review

This pull request introduces a feature to filter, sort, and import only free models for supported providers. It adds a new utility file freeModels.ts, integrates free-model toggles and filters across several UI components (such as AddApiKeyModal, EditConnectionModal, and ModelVisibilityToolbar), updates the model synchronization API route, and includes comprehensive unit tests. The code reviewer identified a potential bug in EditConnectionModal.tsx where the importFreeModelsOnly setting might fail to save if updates.providerSpecificData is undefined, and suggested robustness improvements in freeModels.ts to handle nullable provider parameters safely.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

[gemini-code-assist]

If connection.providerSpecificData is initially undefined or null, updates.providerSpecificData may also be undefined. In that case, the condition showFreeModelsToggle && updates.providerSpecificData will evaluate to false, and the importFreeModelsOnly setting will silently fail to save. To prevent this, we should ensure updates.providerSpecificData is initialized as an empty object if it does not exist.

      if (showFreeModelsToggle) {
        if (!updates.providerSpecificData) {
          updates.providerSpecificData = {};
        }
        // Store an explicit boolean (not undefined): the PUT route merges
        // { ...existing, ...incoming }, so an undefined/omitted key would keep the
        // previously-saved 'true' and unchecking would never take effect.
        updates.providerSpecificData.importFreeModelsOnly = formData.importFreeModelsOnly === true;
      }

[gemini-code-assist]

To prevent potential runtime errors or TypeScript compilation issues when provider is undefined or null (e.g., before a connection is fully loaded or resolved in the parent components), we should make isFreeModel more robust by allowing provider to be nullable and checking its type before performing lookups.

Suggested change

	export function isFreeModel(provider: string, model: FreeModelCandidate): boolean {
	if (typeof model.id === "string" && model.id.endsWith(":free")) return true;
	if (isZeroPrice(model.pricing?.prompt) && isZeroPrice(model.pricing?.completion)) return true;
	if (typeof model.id === "string") {
	const canonical = resolveProviderId(provider);
	if (
	FREE_MODEL_IDS_BY_PROVIDER.get(provider)?.has(model.id) ||
	FREE_MODEL_IDS_BY_PROVIDER.get(canonical)?.has(model.id)
	) {
	return true;
	}
	}
	return false;
	}
	export function isFreeModel(provider: string | undefined | null, model: FreeModelCandidate): boolean {
	if (typeof model.id === "string" && model.id.endsWith(":free")) return true;
	if (isZeroPrice(model.pricing?.prompt) && isZeroPrice(model.pricing?.completion)) return true;
	if (typeof model.id === "string" && typeof provider === "string") {
	const canonical = resolveProviderId(provider);
	if (
	FREE_MODEL_IDS_BY_PROVIDER.get(provider)?.has(model.id) ||
	FREE_MODEL_IDS_BY_PROVIDER.get(canonical)?.has(model.id)
	) {
	return true;
	}
	}
	return false;
	}

[gemini-code-assist]

Update selectModelsForImport to accept a nullable provider parameter to align with the updated isFreeModel signature and ensure safe model filtering.

Suggested change

	export function selectModelsForImport<T extends FreeModelCandidate>(
	provider: string,
	fetchedModels: T[],
	importFreeOnly: boolean
	): SelectModelsForImportResult<T> {
	if (!importFreeOnly) {
	return { models: fetchedModels, freeFilterEmpty: false };
	}
	const models = fetchedModels.filter((m) => isFreeModel(provider, m));
	const freeFilterEmpty = fetchedModels.length > 0 && models.length === 0;
	return { models, freeFilterEmpty };
	}
	export function selectModelsForImport<T extends FreeModelCandidate>(
	provider: string | undefined | null,
	fetchedModels: T[],
	importFreeOnly: boolean
	): SelectModelsForImportResult<T> {
	if (!importFreeOnly) {
	return { models: fetchedModels, freeFilterEmpty: false };
	}
	const models = fetchedModels.filter((m) => isFreeModel(provider, m));
	const freeFilterEmpty = fetchedModels.length > 0 && models.length === 0;
	return { models, freeFilterEmpty };
	}

[diegosouzapw]

Merged into release/v3.8.29 🎉 Thanks for this, @felipesartori — the free-models import option + free/paid filters + free-first sort are a great UX addition, and the selectModelsForImport/freeFilterEmpty round-trip is cleanly tested.

On review we made two small adjustments on your branch (you stay the author):

1. Moved the 3 React-component tests from tests/unit/*.test.tsx to tests/unit/ui/ — top-level .tsx files aren't collected by any runner (test:unit globs *.test.ts, test:vitest:ui filters to tests/unit/ui), so check:test-discovery had flagged them as orphans that never run. They pass 12/12 at the new path.
2. Bumped the file-size baseline for the two connection modals (the toggle is cohesive UI, mirroring the prior fix: stabilize reasoning streams and request logs #3879/[Feature] Per-connection 'disable-cooling' opt-out #2997 toggle bumps) and synced your branch with the current release.

Shipping in the next release. 🙏