Files
kilocode/packages/kilo-vscode/tests/unit/model-selector-utils.test.ts
T
kiloconnect[bot] 0918bb4216 fix(vscode): preserve 'Free' in model names like 'Kilo Auto Free'
Only strip the parenthesized '(free)' suffix; bare trailing 'Free' is
part of the name and must be kept. A separate Free label is rendered
elsewhere.
2026-04-28 15:19:59 +00:00

165 lines
6.5 KiB
TypeScript

import { describe, it, expect } from "bun:test"
import {
providerSortKey,
buildTriggerLabel,
stripSubProviderPrefix,
sanitizeName,
KILO_GATEWAY_ID,
PROVIDER_ORDER,
} from "../../webview-ui/src/components/shared/model-selector-utils"
const labels = { select: "Select model", noProviders: "No providers", notSet: "Not set" }
describe("providerSortKey", () => {
it("returns 0 for kilo gateway", () => {
expect(providerSortKey(KILO_GATEWAY_ID)).toBe(0)
})
it("returns correct index for known providers", () => {
expect(providerSortKey("anthropic")).toBe(1)
expect(providerSortKey("openai")).toBe(3)
expect(providerSortKey("google")).toBe(4)
})
it("returns order length for unknown provider", () => {
expect(providerSortKey("unknown-provider")).toBe(PROVIDER_ORDER.length)
})
it("is case-insensitive", () => {
expect(providerSortKey("Anthropic")).toBe(providerSortKey("anthropic"))
expect(providerSortKey("OpenAI")).toBe(providerSortKey("openai"))
})
it("respects custom order array", () => {
const order = ["z-provider", "a-provider"]
expect(providerSortKey("z-provider", order)).toBe(0)
expect(providerSortKey("a-provider", order)).toBe(1)
expect(providerSortKey("other", order)).toBe(2)
})
it("sorts providers correctly when used with sort", () => {
const ids = ["google", "anthropic", "kilo", "openai", "github-copilot"]
const sorted = ids.slice().sort((a, b) => providerSortKey(a) - providerSortKey(b))
expect(sorted).toEqual(["kilo", "anthropic", "github-copilot", "openai", "google"])
})
})
describe("stripSubProviderPrefix", () => {
it("strips prefix before ': '", () => {
expect(stripSubProviderPrefix("Anthropic: Claude Sonnet")).toBe("Claude Sonnet")
expect(stripSubProviderPrefix("OpenAI: GPT-4o")).toBe("GPT-4o")
})
it("leaves names without ': ' unchanged", () => {
expect(stripSubProviderPrefix("GPT-4o")).toBe("GPT-4o")
expect(stripSubProviderPrefix("claude-3-5-sonnet")).toBe("claude-3-5-sonnet")
})
it("does not strip 'Kilo: ' prefix", () => {
expect(stripSubProviderPrefix("Kilo: Auto")).toBe("Kilo: Auto")
expect(stripSubProviderPrefix("kilo: Auto")).toBe("kilo: Auto")
})
})
describe("sanitizeName", () => {
it("strips trailing (free) suffix", () => {
expect(sanitizeName("Llama 3 (free)")).toBe("Llama 3")
})
it("is case-insensitive for parenthesized suffix", () => {
expect(sanitizeName("Model (Free)")).toBe("Model")
expect(sanitizeName("Model (FREE)")).toBe("Model")
})
it("preserves bare trailing Free in names like 'Kilo Auto Free'", () => {
expect(sanitizeName("Kilo Auto Free")).toBe("Kilo Auto Free")
expect(sanitizeName("Mixtral free")).toBe("Mixtral free")
expect(sanitizeName("Mistral:free")).toBe("Mistral:free")
expect(sanitizeName("Gemma-free")).toBe("Gemma-free")
expect(sanitizeName("Model FREE")).toBe("Model FREE")
})
it("leaves names without (free) suffix unchanged", () => {
expect(sanitizeName("GPT-4o")).toBe("GPT-4o")
expect(sanitizeName("Claude Sonnet")).toBe("Claude Sonnet")
})
it("does not strip 'free' from the middle of a name", () => {
expect(sanitizeName("FreeAgent Pro")).toBe("FreeAgent Pro")
})
it("handles extra whitespace around (free) suffix", () => {
expect(sanitizeName("Llama 3 (free) ")).toBe("Llama 3")
expect(sanitizeName("Model (free) ")).toBe("Model")
})
})
describe("buildTriggerLabel", () => {
it("returns resolved model name for non-kilo provider unchanged", () => {
expect(buildTriggerLabel("GPT-4o", "openai", undefined, null, false, "", true, labels)).toBe("GPT-4o")
})
it("strips sub-provider prefix from resolved name for kilo gateway models", () => {
expect(
buildTriggerLabel("Anthropic: Claude Sonnet", KILO_GATEWAY_ID, undefined, null, false, "", true, labels),
).toBe("Claude Sonnet")
})
it("does not strip prefix for non-kilo provider even if name contains ': '", () => {
expect(buildTriggerLabel("Anthropic: Claude Sonnet", "anthropic", undefined, null, false, "", true, labels)).toBe(
"Anthropic: Claude Sonnet",
)
})
it("returns resolved name as-is when providerID is undefined", () => {
expect(buildTriggerLabel("GPT-4o", undefined, undefined, null, false, "", true, labels)).toBe("GPT-4o")
})
it("returns providerName / resolvedName for non-kilo provider with providerName", () => {
expect(buildTriggerLabel("GPT-4o", "openai", "OpenAI", null, false, "", true, labels)).toBe("OpenAI / GPT-4o")
})
it("returns modelID for kilo gateway raw selection", () => {
const raw = { providerID: "kilo", modelID: "kilo-auto/frontier" }
expect(buildTriggerLabel(undefined, undefined, undefined, raw, false, "", true, labels)).toBe("kilo-auto/frontier")
})
it("returns providerID / modelID for non-kilo raw selection", () => {
const raw = { providerID: "anthropic", modelID: "claude-3-5-sonnet" }
expect(buildTriggerLabel(undefined, undefined, undefined, raw, false, "", true, labels)).toBe(
"anthropic / claude-3-5-sonnet",
)
})
it("returns clearLabel when allowClear and no selection", () => {
expect(buildTriggerLabel(undefined, undefined, undefined, null, true, "None", true, labels)).toBe("None")
})
it("falls back to labels.notSet when allowClear and clearLabel is empty", () => {
expect(buildTriggerLabel(undefined, undefined, undefined, null, true, "", true, labels)).toBe("Not set")
})
it("returns labels.select when providers exist and no selection", () => {
expect(buildTriggerLabel(undefined, undefined, undefined, null, false, "", true, labels)).toBe("Select model")
})
it("returns labels.noProviders when no providers available", () => {
expect(buildTriggerLabel(undefined, undefined, undefined, null, false, "", false, labels)).toBe("No providers")
})
it("prefers resolvedName over raw selection", () => {
const raw = { providerID: "anthropic", modelID: "claude-3-5-sonnet" }
expect(buildTriggerLabel("Claude Sonnet", undefined, undefined, raw, false, "", true, labels)).toBe("Claude Sonnet")
})
it("ignores partial raw selection (only providerID)", () => {
const raw = { providerID: "anthropic", modelID: "" }
expect(buildTriggerLabel(undefined, undefined, undefined, raw, false, "", true, labels)).toBe("Select model")
})
it("ignores partial raw selection (only modelID)", () => {
const raw = { providerID: "", modelID: "claude-3-5-sonnet" }
expect(buildTriggerLabel(undefined, undefined, undefined, raw, false, "", true, labels)).toBe("Select model")
})
})