chore: remove skipAuth repetition

Author: davidturnbull

packages/cli/src/cli/localizer/explicit.ts

@@ -34,17 +34,14 @@ export default function createExplicitLocalizer(
     );
   }
 
-  const skipAuth = provider.id === "ollama";
   try {
     const model = createProviderClient(provider.id as ProviderId, provider.model, {
       baseUrl: provider.baseUrl,
-      skipAuth,
     });
     return createLocalizerFromModel({
       model,
       id: provider.id,
       prompt: provider.prompt,
-      skipAuth,
     });
   } catch (error: unknown) {
     if (error instanceof ProviderKeyMissingError) {
@@ -80,7 +77,6 @@ function createLocalizerFromModel(params: {
   model: LanguageModel;
   id: NonNullable<I18nConfig["provider"]>["id"];
   prompt: string;
-  skipAuth?: boolean;
 }): ILocalizer {
   const { model } = params;
 

packages/cli/src/cli/processor/index.ts

@@ -74,11 +74,9 @@ function getPureModelProvider(provider: I18nConfig["provider"]) {
     throw new Error(createUnsupportedProviderErrorMessage(provider?.id));
   }
 
-  const skipAuth = provider?.id === "ollama";
   try {
     return createProviderClient(provider!.id as ProviderId, provider!.model, {
       baseUrl: provider!.baseUrl,
-      skipAuth,
     });
   } catch (error: unknown) {
     if (error instanceof ProviderKeyMissingError) {

packages/providers/src/factory.ts

@@ -1,6 +1,7 @@
 import { LanguageModel } from "ai";
 import { ProviderId } from "./constants";
 import { resolveProviderApiKey } from "./keys";
+import { PROVIDER_METADATA } from "./metadata";
 import { UnsupportedProviderError } from "./errors";
 import { createOpenAI } from "@ai-sdk/openai";
 import { createAnthropic } from "@ai-sdk/anthropic";
@@ -21,7 +22,8 @@ export function createProviderClient(
   modelId: string,
   options?: ClientOptions,
 ): LanguageModel {
-  const skipAuth = options?.skipAuth === true || providerId === "ollama";
+  const skipAuth =
+    options?.skipAuth === true || !PROVIDER_METADATA[providerId]?.apiKeyEnvVar;
   const apiKey = options?.apiKey ?? resolveProviderApiKey(providerId, { required: !skipAuth });
 
   switch (providerId) {