Skip to content

Commit 87080d6

Browse files
committed
refactor(webview): use canonical provider identifiers
1 parent c29315e commit 87080d6

12 files changed

Lines changed: 185 additions & 120 deletions

File tree

webview-ui/src/components/chat/CodeIndexPopover.tsx

Lines changed: 13 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,12 @@ import {
1212
import * as ProgressPrimitive from "@radix-ui/react-progress"
1313
import { TriangleAlert } from "lucide-react"
1414

15-
import { type IndexingStatus, type EmbedderProvider, CODEBASE_INDEX_DEFAULTS } from "@roo-code/types"
15+
import {
16+
providerIdentifiers,
17+
type IndexingStatus,
18+
type EmbedderProvider,
19+
CODEBASE_INDEX_DEFAULTS,
20+
} from "@roo-code/types"
1621

1722
import { vscode } from "@src/utils/vscode"
1823
import { useExtensionState } from "@src/context/ExtensionStateContext"
@@ -95,15 +100,15 @@ const createValidationSchema = (provider: EmbedderProvider, t: any) => {
95100
})
96101

97102
switch (provider) {
98-
case "openai":
103+
case providerIdentifiers.openai:
99104
return baseSchema.extend({
100105
codeIndexOpenAiKey: z.string().min(1, t("settings:codeIndex.validation.openaiApiKeyRequired")),
101106
codebaseIndexEmbedderModelId: z
102107
.string()
103108
.min(1, t("settings:codeIndex.validation.modelSelectionRequired")),
104109
})
105110

106-
case "ollama":
111+
case providerIdentifiers.ollama:
107112
return baseSchema.extend({
108113
codebaseIndexEmbedderBaseUrl: z
109114
.string()
@@ -131,23 +136,23 @@ const createValidationSchema = (provider: EmbedderProvider, t: any) => {
131136
.min(1, t("settings:codeIndex.validation.modelDimensionRequired")),
132137
})
133138

134-
case "gemini":
139+
case providerIdentifiers.gemini:
135140
return baseSchema.extend({
136141
codebaseIndexGeminiApiKey: z.string().min(1, t("settings:codeIndex.validation.geminiApiKeyRequired")),
137142
codebaseIndexEmbedderModelId: z
138143
.string()
139144
.min(1, t("settings:codeIndex.validation.modelSelectionRequired")),
140145
})
141146

142-
case "mistral":
147+
case providerIdentifiers.mistral:
143148
return baseSchema.extend({
144149
codebaseIndexMistralApiKey: z.string().min(1, t("settings:codeIndex.validation.mistralApiKeyRequired")),
145150
codebaseIndexEmbedderModelId: z
146151
.string()
147152
.min(1, t("settings:codeIndex.validation.modelSelectionRequired")),
148153
})
149154

150-
case "vercel-ai-gateway":
155+
case providerIdentifiers.vercelAiGateway:
151156
return baseSchema.extend({
152157
codebaseIndexVercelAiGatewayApiKey: z
153158
.string()
@@ -157,7 +162,7 @@ const createValidationSchema = (provider: EmbedderProvider, t: any) => {
157162
.min(1, t("settings:codeIndex.validation.modelSelectionRequired")),
158163
})
159164

160-
case "bedrock":
165+
case providerIdentifiers.bedrock:
161166
return baseSchema.extend({
162167
codebaseIndexBedrockRegion: z.string().min(1, t("settings:codeIndex.validation.bedrockRegionRequired")),
163168
codebaseIndexBedrockProfile: z.string().optional(),
@@ -166,7 +171,7 @@ const createValidationSchema = (provider: EmbedderProvider, t: any) => {
166171
.min(1, t("settings:codeIndex.validation.modelSelectionRequired")),
167172
})
168173

169-
case "openrouter":
174+
case providerIdentifiers.openrouter:
170175
return baseSchema.extend({
171176
codebaseIndexOpenRouterApiKey: z
172177
.string()

0 commit comments

Comments
 (0)