fix: cleanup logs and unused functions

AleksandrSl · AleksandrSl · commit 196b8f869596 · 2025-12-25T00:47:16.000+01:00
diff --git a/packages/new-compiler/src/plugin/build-translator.ts b/packages/new-compiler/src/plugin/build-translator.ts
@@ -64,10 +64,6 @@ export async function processBuildTranslations(
 
   logger.info(`🌍 Build mode: ${buildMode}`);
 
-  if (metadataFilePath) {
-    logger.info(`📋 Using build metadata file: ${metadataFilePath}`);
-  }
-
   const metadata = await loadMetadata(metadataFilePath);
 
   if (!metadata || Object.keys(metadata.entries).length === 0) {
@@ -172,7 +168,10 @@ export async function processBuildTranslations(
       stats,
     };
   } catch (error) {
-    logger.error("❌ Translation generation failed:", error);
+    logger.error(
+      "❌ Translation generation failed:\n",
+      error instanceof Error ? error.message : error,
+    );
     process.exit(1);
   } finally {
     if (translationServer) {
diff --git a/packages/new-compiler/src/plugin/next.ts b/packages/new-compiler/src/plugin/next.ts
@@ -297,7 +297,6 @@ export async function withLingo(
     }
 
     logger.info("Running post-build translation generation...");
-    logger.info(`Build mode: Using metadata file: ${metadataFilePath}`);
 
     try {
       await processBuildTranslations({
@@ -306,7 +305,10 @@ export async function withLingo(
         metadataFilePath,
       });
     } catch (error) {
-      logger.error("Translation generation failed:", error);
+      logger.error(
+        "Translation generation failed:",
+        error instanceof Error ? error.message : error,
+      );
       throw error;
     }
   };
diff --git a/packages/new-compiler/src/translators/lingo/model-factory.ts b/packages/new-compiler/src/translators/lingo/model-factory.ts
@@ -10,6 +10,7 @@ import { createMistral } from "@ai-sdk/mistral";
 import type { LanguageModel } from "ai";
 import * as dotenv from "dotenv";
 import * as path from "path";
+import { formatNoApiKeysError } from "./provider-details";
 
 export type LocaleModel = {
   provider: string;
@@ -169,7 +170,7 @@ export function validateAndGetApiKeys(
   config: "lingo.dev" | Record<string, string>,
 ): ValidatedApiKeys {
   const keys: ValidatedApiKeys = {};
-  const missingKeys: Array<{ provider: string; envVar: string }> = [];
+  const missingProviders: string[] = [];
 
   // Determine which providers are configured
   let providersToValidate: string[];
@@ -195,7 +196,7 @@ export function validateAndGetApiKeys(
 
     if (!providerConfig) {
       throw new Error(
-        `⚠️  Unknown provider "${provider}". Supported providers: ${Object.keys(providerDetails).join(", ")}`,
+        `⚠️ Unknown provider "${provider}". Supported providers: ${Object.keys(providerDetails).join(", ")}`,
       );
     }
 
@@ -208,21 +209,13 @@ export function validateAndGetApiKeys(
     if (key) {
       keys[provider] = key;
     } else {
-      missingKeys.push({
-        provider: providerConfig.name,
-        envVar: providerConfig.apiKeyEnvVar,
-      });
+      missingProviders.push(provider);
     }
   }
 
   // If any keys are missing, throw with detailed error
-  if (missingKeys.length > 0) {
-    const errorLines = missingKeys.map(
-      ({ provider, envVar }) => `  - ${provider}: ${envVar}`,
-    );
-    throw new Error(
-      `⚠️  Missing API keys for configured providers:\n${errorLines.join("\n")}\n\nPlease set the required environment variables.`,
-    );
+  if (missingProviders.length > 0) {
+    throw new Error(formatNoApiKeysError(missingProviders));
   }
 
   return keys;
@@ -253,6 +246,7 @@ export function createAiModel(
     ? validatedKeys[model.provider]
     : undefined;
 
+  // TODO (AleksandrSl 25/12/2025): Do we really need to make a second check? Maybe creation should be combined with validation.
   // Verify key is present for providers that require it
   if (providerConfig.apiKeyEnvVar && !apiKey) {
     throw new Error(
diff --git a/packages/new-compiler/src/translators/lingo/provider-details.ts b/packages/new-compiler/src/translators/lingo/provider-details.ts
@@ -55,111 +55,6 @@ export const providerDetails: Record<string, ProviderDetails> = {
   },
 };
 
-/**
- * Get provider details by ID
- */
-export function getProviderDetails(providerId: string): ProviderDetails | null {
-  return providerDetails[providerId] || null;
-}
-
-/**
- * Get all providers that require API keys
- */
-export function getProvidersRequiringKeys(): string[] {
-  return Object.keys(providerDetails).filter(
-    (id) => providerDetails[id].apiKeyEnvVar !== undefined,
-  );
-}
-
-/**
- * Format a helpful error message when API key is missing
- */
-export function formatMissingApiKeyError(providerId: string): string {
-  const details = providerDetails[providerId];
-
-  if (!details) {
-    return `Unknown provider: ${providerId}`;
-  }
-
-  if (!details.apiKeyEnvVar) {
-    // Provider doesn't need API key (like Ollama)
-    return `Provider ${details.name} doesn't require an API key. Check connection at ${details.getKeyLink}`;
-  }
-
-  return [
-    `⚠️  ${details.name} API key not found.`,
-    ``,
-    `To use ${details.name} for translations, you need to set ${details.apiKeyEnvVar}.`,
-    ``,
-    `👉 Set the API key:`,
-    `   1. Add to .env file: ${details.apiKeyEnvVar}=<your-api-key>`,
-    `   2. Or export in terminal: export ${details.apiKeyEnvVar}=<your-api-key>`,
-    ``,
-    `⭐️ Get your API key:`,
-    `   ${details.getKeyLink}`,
-    ``,
-    `📚 Documentation:`,
-    `   ${details.docsLink}`,
-  ].join("\n");
-}
-
-/**
- * Format a helpful error message when API call fails
- */
-export function formatApiCallError(
-  providerId: string,
-  targetLocale: string,
-  errorMessage: string,
-): string {
-  const details = providerDetails[providerId];
-
-  if (!details) {
-    return `Translation failed with unknown provider "${providerId}": ${errorMessage}`;
-  }
-
-  // Check for common error patterns
-  const isInvalidApiKey =
-    errorMessage.toLowerCase().includes("invalid api key") ||
-    errorMessage.toLowerCase().includes("unauthorized") ||
-    errorMessage.toLowerCase().includes("authentication failed");
-
-  if (isInvalidApiKey) {
-    return [
-      `⚠️  ${details.name} API key is invalid.`,
-      ``,
-      `Error details: ${errorMessage}`,
-      ``,
-      `👉 Please check your API key:`,
-      details.apiKeyEnvVar
-        ? `   Environment variable: ${details.apiKeyEnvVar}`
-        : "",
-      ``,
-      `⭐️ Get a new API key:`,
-      `   ${details.getKeyLink}`,
-      ``,
-      `📚 Troubleshooting:`,
-      `   ${details.docsLink}`,
-    ]
-      .filter(Boolean)
-      .join("\n");
-  }
-
-  // Generic API error
-  return [
-    `⚠️  Translation to "${targetLocale}" failed via ${details.name}.`,
-    ``,
-    `Error details: ${errorMessage}`,
-    ``,
-    `📚 Check ${details.name} documentation for more details:`,
-    `   ${details.docsLink}`,
-    ``,
-    `💡 Tips:`,
-    `   - Check your API quota/credits`,
-    `   - Verify the model is available for your account`,
-    `   - Check ${details.name} status page for outages`,
-  ].join("\n");
-}
-
 /**
  * Format error message when API keys are missing for configured providers
  * @param missingProviders List of providers that are missing API keys
@@ -179,14 +74,12 @@ export function formatNoApiKeysError(
   // Header
   if (allProviders && allProviders.length > missingProviders.length) {
     lines.push(
-      `⚠️  Missing API keys for ${missingProviders.length} of ${allProviders.length} configured providers.`,
+      `Missing API keys for ${missingProviders.length} of ${allProviders.length} configured providers.`,
     );
   } else {
-    lines.push(`⚠️  Missing API keys for configured translation providers.`);
+    lines.push(`Missing API keys for configured translation providers.`);
   }
 
-  lines.push(``);
-
   // List missing providers with their environment variables and links
   lines.push(`Missing API keys for:`);
   for (const providerId of missingProviders) {
@@ -210,8 +103,6 @@ export function formatNoApiKeysError(
     `   1. Add to .env file (recommended)`,
     `   2. Or export in terminal: export API_KEY_NAME=<your-key>`,
     ``,
-    `💡 In development mode, the app will auto-fallback to pseudotranslator.`,
-    `   In production, all configured providers must have valid API keys.`,
   );
 
   return lines.join("\n");
diff --git a/packages/new-compiler/src/translators/lingo/translator.ts b/packages/new-compiler/src/translators/lingo/translator.ts
@@ -1,20 +1,10 @@
 import { generateText } from "ai";
 import { LingoDotDevEngine } from "lingo.dev/sdk";
-import {
-  dictionaryFrom,
-  type DictionarySchema,
-  type TranslatableEntry,
-  type Translator,
-} from "../api";
+import { dictionaryFrom, type DictionarySchema, type TranslatableEntry, type Translator, } from "../api";
 import { getSystemPrompt } from "./prompt";
 import { obj2xml, parseXmlFromResponseText } from "../parse-xml";
 import { shots } from "./shots";
-import {
-  createAiModel,
-  getLocaleModel,
-  validateAndGetApiKeys,
-  type ValidatedApiKeys,
-} from "./model-factory";
+import { createAiModel, getLocaleModel, validateAndGetApiKeys, type ValidatedApiKeys, } from "./model-factory";
 import { Logger } from "../../utils/logger";
 import { DEFAULT_TIMEOUTS, withTimeout } from "../../utils/timeout";
 import type { LocaleCode } from "lingo.dev/spec";
@@ -51,7 +41,7 @@ export class LingoTranslator implements Translator<LingoTranslatorConfig> {
     entriesMap: Record<string, TranslatableEntry>,
   ): Promise<Record<string, string>> {
     this.logger.debug(
-      `[TRACE-LINGO] translate() called for ${locale} with ${Object.keys(entriesMap).length} entries`,
+      `translate() called for ${locale} with ${Object.keys(entriesMap).length} entries`,
     );
 
     const sourceDictionary: DictionarySchema = dictionaryFrom(
@@ -62,7 +52,7 @@ export class LingoTranslator implements Translator<LingoTranslatorConfig> {
     );
 
     this.logger.debug(
-      `[TRACE-LINGO] Created source dictionary with ${Object.keys(sourceDictionary.entries).length} entries`,
+      `Created source dictionary with ${Object.keys(sourceDictionary.entries).length} entries`,
     );
     const translated = await this.translateDictionary(sourceDictionary, locale);
 
@@ -76,35 +66,33 @@ export class LingoTranslator implements Translator<LingoTranslatorConfig> {
     sourceDictionary: DictionarySchema,
     targetLocale: string,
   ): Promise<DictionarySchema> {
+    const chunks = this.chunkDictionary(sourceDictionary);
     this.logger.debug(
-      `[TRACE-LINGO] Chunking dictionary with ${Object.keys(sourceDictionary.entries).length} entries`,
+      `Split dictionary with ${Object.keys(sourceDictionary.entries).length} into ${chunks.length} chunks`,
     );
-    const chunks = this.chunkDictionary(sourceDictionary);
-    this.logger.debug(`[TRACE-LINGO] Split into ${chunks.length} chunks`);
 
     const translatedChunks: DictionarySchema[] = [];
 
     for (let i = 0; i < chunks.length; i++) {
       const chunk = chunks[i];
       this.logger.debug(
-        `[TRACE-LINGO] Translating chunk ${i + 1}/${chunks.length} with ${Object.keys(chunk.entries).length} entries`,
+        `Translating chunk ${i + 1}/${chunks.length} with ${Object.keys(chunk.entries).length} entries`,
       );
       const chunkStartTime = performance.now();
 
       const translatedChunk = await this.translateChunk(chunk, targetLocale);
 
       const chunkEndTime = performance.now();
       this.logger.debug(
-        `[TRACE-LINGO] Chunk ${i + 1}/${chunks.length} completed in ${(chunkEndTime - chunkStartTime).toFixed(2)}ms`,
+        `Chunk ${i + 1}/${chunks.length} completed in ${(chunkEndTime - chunkStartTime).toFixed(2)}ms`,
       );
 
       translatedChunks.push(translatedChunk);
     }
 
-    this.logger.debug(`[TRACE-LINGO] All chunks translated, merging results`);
     const result = this.mergeDictionaries(translatedChunks);
     this.logger.debug(
-      `[TRACE-LINGO] Merge completed, final dictionary has ${Object.keys(result.entries).length} entries`,
+      `Merge completed, final dictionary has ${Object.keys(result.entries).length} entries`,
     );
 
     return result;
@@ -139,7 +127,7 @@ export class LingoTranslator implements Translator<LingoTranslatorConfig> {
       );
     }
 
-    this.logger.info(
+    this.logger.debug(
       `Using Lingo.dev Engine to localize from "${this.config.sourceLocale}" to "${targetLocale}"`,
     );
 
@@ -184,7 +172,7 @@ export class LingoTranslator implements Translator<LingoTranslatorConfig> {
       );
     }
 
-    this.logger.info(
+    this.logger.debug(
       `Using LLM ("${localeModel.provider}":"${localeModel.name}") to translate from "${this.config.sourceLocale}" to "${targetLocale}"`,
     );
 
diff --git a/packages/new-compiler/src/translators/translation-service.ts b/packages/new-compiler/src/translators/translation-service.ts
@@ -117,11 +117,9 @@ export class TranslationService {
           // Use console.error to ensure visibility in all contexts (loader, server, etc.)
           const errorMsg =
             error instanceof Error ? error.message : String(error);
-          this.logger.warn(`\n⚠️ Translation setup error: ${errorMsg}\n`);
-          this.logger.warn(
-            `⚠️ Auto-fallback to pseudotranslator in development mode.\n` +
-              ` Set the required API keys for real translations.\n`,
-          );
+          this.logger.warn(`⚠️ Translation setup error: \n${errorMsg}\n
+⚠️ Auto-fallback to pseudotranslator in development mode.
+Set the required API keys for real translations.`);
 
           this.translator = new PseudoTranslator(
             { delayMedian: 100 },
