chore: cleanup more stupid defaults

Author: AleksandrSl

cmp/compiler/src/plugin/transform/utils.ts

@@ -55,7 +55,10 @@ export function isReactComponent(
     }
   }
 
-  // TODO (AleksandrSl 25/11/2025): In which order does it traverse? If it's DFS it may find the incorrect return?
+  // Babel traverses with DFS so we get the innermost function return first if any.
+  // But if at least one function returns JSX I guess we should transform the whole function.
+  // It's a weird corner case.
+
   // Check for explicit return statements with JSX
   // We could also check for the first JSX?
   let returnsJSX = false;
@@ -181,7 +184,6 @@ export function constructTranslationCall(
             t.arrowFunctionExpression([], element)
           : // Create: tagName: (chunks) => <Element>{chunks}</Element>
             t.arrowFunctionExpression(
-              // TODO (AleksandrSl 28/11/2025): Use content instead of the chunks later
               [t.identifier("chunks")],
               t.jsxElement(
                 t.jsxOpeningElement(

cmp/compiler/src/react/server-only/translations.ts

@@ -41,7 +41,7 @@ async function readFromFilesystem(
   locale: LocaleCode,
   basePath: string = process.cwd(),
 ): Promise<Record<string, string>> {
-  // TODO (AleksandrSl 02/12/2025): Sanity check. We need to try loading the most up to date translations first. Fo the dev mode they are in lingo. Gor build they are in next.
+  // TODO (AleksandrSl 15/12/2025): What should be the correct way here to load translations on server for other frameworks?
   const possiblePaths = [
     join(basePath, cacheDir, `${locale}.json`),
     join(basePath, ".next", `${locale}.json`),

cmp/compiler/src/translators/index.ts

@@ -12,7 +12,6 @@ export { PseudoTranslator } from "./pseudotranslator";
 export { Service } from "./lingo";
 export type { LingoTranslatorConfig } from "./lingo";
 export { createTranslator } from "./translator-factory";
-export type { TranslatorFactoryConfig } from "./translator-factory";
 
 // Translation Service (orchestrator)
 export { TranslationService } from "./translation-service";

cmp/compiler/src/translators/lingo/service.ts

@@ -25,7 +25,7 @@ import type { LocaleCode } from "lingo.dev/spec";
 export interface LingoTranslatorConfig {
   models: "lingo.dev" | Record<string, string>;
   sourceLocale: LocaleCode;
-  prompt?: string | null;
+  prompt?: string;
 }
 
 /**
@@ -200,7 +200,7 @@ export class Service implements Translator<LingoTranslatorConfig> {
               content: getSystemPrompt({
                 sourceLocale: this.config.sourceLocale,
                 targetLocale,
-                prompt: this.config.prompt ?? undefined,
+                prompt: this.config.prompt,
               }),
             },
             // Add few-shot examples

cmp/compiler/src/translators/translator-factory.ts

@@ -7,12 +7,13 @@ import { PseudoTranslator } from "./pseudotranslator";
 import { Service } from "./lingo";
 import { Logger } from "../utils/logger";
 import type { LocaleCode } from "lingo.dev/spec";
+import type { LingoEnvironment } from "../types";
 
-export interface TranslatorFactoryConfig {
+interface TranslatorFactoryConfig {
   sourceLocale: LocaleCode;
-  models?: "lingo.dev" | Record<string, string>;
+  models: "lingo.dev" | Record<string, string>;
   prompt?: string;
-  environment: "development" | "production";
+  environment: LingoEnvironment;
   dev?: {
     usePseudotranslator?: boolean;
   };
@@ -39,7 +40,7 @@ export interface TranslatorFactoryConfig {
 export function createTranslator(
   config: TranslatorFactoryConfig,
   logger: Logger,
-): Translator<any> {
+): Translator<unknown> {
   const isDev = config.environment === "development";
 
   // 1. Explicit dev override takes precedence
@@ -52,8 +53,7 @@ export function createTranslator(
   // LingoTranslator constructor will validate and fetch API keys
   // If validation fails, it will throw an error with helpful message
   try {
-    // TODO (AleksandrSl 14/12/2025): Get rid of default
-    const models = config.models || "lingo.dev";
+    const models = config.models;
 
     logger.info(
       `Creating Lingo translator with models: ${JSON.stringify(models)}`,
@@ -63,7 +63,7 @@ export function createTranslator(
       {
         models,
         sourceLocale: config.sourceLocale,
-        prompt: config.prompt || null,
+        prompt: config.prompt,
       },
       logger,
     );
@@ -72,8 +72,8 @@ export function createTranslator(
     if (isDev) {
       // Use console.error to ensure visibility in all contexts (loader, server, etc.)
       const errorMsg = error instanceof Error ? error.message : String(error);
-      console.error(`\n❌ [Lingo] Translation setup error: ${errorMsg}\n`);
-      console.warn(
+      logger.error(`\n❌ [Lingo] Translation setup error: ${errorMsg}\n`);
+      logger.warn(
         `⚠️  [Lingo] Auto-fallback to pseudotranslator in development mode.\n` +
           `   Set the required API keys for real translations.\n`,
       );