i18ntk 4.5.4 → 4.6.1

package/main/i18ntk-init.js

@@ -29,29 +29,56 @@ const { createPrompt, isInteractive } = require('../utils/prompt-helper');
 const { parseConfirmation } = require('../utils/localized-confirm');
 const { normalizeReportFormat, writeReportFile } = require('../utils/report-writer');
 
-// Language configurations with native names
-const LANGUAGE_CONFIG = {
-  'en': { name: 'English', nativeName: 'English' },
-  'de': { name: 'German', nativeName: 'Deutsch' },
-  'es': { name: 'Spanish', nativeName: 'Espa\u00f1ol' },
-  'fr': { name: 'French', nativeName: 'Fran\u00e7ais' },
-  'ru': { name: 'Russian', nativeName: '\u0420\u0443\u0441\u0441\u043a\u0438\u0439' },
-  'it': { name: 'Italian', nativeName: 'Italiano' },
-  'ja': { name: 'Japanese', nativeName: '\u65e5\u672c\u8a9e' },
-  'ko': { name: 'Korean', nativeName: '\ud55c\uad6d\uc5b4' },
-  'zh': { name: 'Chinese', nativeName: '\u4e2d\u6587' },
-  'ar': { name: 'Arabic', nativeName: '\u0627\u0644\u0639\u0631\u0628\u064a\u0629' },
-  'hi': { name: 'Hindi', nativeName: '\u0939\u093f\u0928\u094d\u0926\u0940' },
-  'nl': { name: 'Dutch', nativeName: 'Nederlands' },
-  'sv': { name: 'Swedish', nativeName: 'Svenska' },
-  'da': { name: 'Danish', nativeName: 'Dansk' },
-  'no': { name: 'Norwegian', nativeName: 'Norsk' },
-  'fi': { name: 'Finnish', nativeName: 'Suomi' },
-  'pl': { name: 'Polish', nativeName: 'Polski' },
-  'cs': { name: 'Czech', nativeName: '\u010ce\u0161tina' },
-  'hu': { name: 'Hungarian', nativeName: 'Magyar' },
-  'tr': { name: 'Turkish', nativeName: 'T\u00fcrk\u00e7e' }
-};
+// Language configurations with native names — comprehensive ISO 639-1 coverage
+const LANGUAGE_CONFIG = {
+  'af': { name: 'Afrikaans', nativeName: 'Afrikaans' },  'sq': { name: 'Albanian', nativeName: 'Shqip' },
+  'ar': { name: 'Arabic', nativeName: '\u0627\u0644\u0639\u0631\u0628\u064a\u0629' },  'hy': { name: 'Armenian', nativeName: '\u0540\u0561\u0575\u0565\u0580\u0565\u0576' },
+  'az': { name: 'Azerbaijani', nativeName: 'Az\u0259rbaycan' },  'eu': { name: 'Basque', nativeName: 'Euskara' },
+  'be': { name: 'Belarusian', nativeName: '\u0411\u0435\u043b\u0430\u0440\u0443\u0441\u043a\u0430\u044f' },
+  'bn': { name: 'Bengali', nativeName: '\u09ac\u09be\u0982\u09b2\u09be' },
+  'bs': { name: 'Bosnian', nativeName: 'Bosanski' },  'bg': { name: 'Bulgarian', nativeName: '\u0411\u044a\u043b\u0433\u0430\u0440\u0441\u043a\u0438' },
+  'ca': { name: 'Catalan', nativeName: 'Catal\u00e0' },  'zh': { name: 'Chinese', nativeName: '\u4e2d\u6587' },
+  'hr': { name: 'Croatian', nativeName: 'Hrvatski' },  'cs': { name: 'Czech', nativeName: '\u010ce\u0161tina' },
+  'da': { name: 'Danish', nativeName: 'Dansk' },  'nl': { name: 'Dutch', nativeName: 'Nederlands' },
+  'en': { name: 'English', nativeName: 'English' },  'et': { name: 'Estonian', nativeName: 'Eesti' },
+  'fi': { name: 'Finnish', nativeName: 'Suomi' },  'fr': { name: 'French', nativeName: 'Fran\u00e7ais' },
+  'ka': { name: 'Georgian', nativeName: '\u10e5\u10d0\u10e0\u10d7\u10e3\u10da\u10d8' },
+  'de': { name: 'German', nativeName: 'Deutsch' },  'el': { name: 'Greek', nativeName: '\u0395\u03bb\u03bb\u03b7\u03bd\u03b9\u03ba\u03ac' },
+  'gu': { name: 'Gujarati', nativeName: '\u0a97\u0ac1\u0a9c\u0ab0\u0abe\u0aa4\u0ac0' },
+  'he': { name: 'Hebrew', nativeName: '\u05e2\u05d1\u05e8\u05d9\u05ea' },
+  'hi': { name: 'Hindi', nativeName: '\u0939\u093f\u0928\u094d\u0926\u0940' },
+  'hu': { name: 'Hungarian', nativeName: 'Magyar' },  'is': { name: 'Icelandic', nativeName: '\u00cdslenska' },
+  'id': { name: 'Indonesian', nativeName: 'Bahasa Indonesia' },
+  'ga': { name: 'Irish', nativeName: 'Gaeilge' },  'it': { name: 'Italian', nativeName: 'Italiano' },
+  'ja': { name: 'Japanese', nativeName: '\u65e5\u672c\u8a9e' },
+  'kk': { name: 'Kazakh', nativeName: '\u049a\u0430\u0437\u0430\u049b' },
+  'ko': { name: 'Korean', nativeName: '\ud55c\uad6d\uc5b4' },
+  'lv': { name: 'Latvian', nativeName: 'Latvie\u0161u' },  'lt': { name: 'Lithuanian', nativeName: 'Lietuvi\u0173' },
+  'mk': { name: 'Macedonian', nativeName: '\u041c\u0430\u043a\u0435\u0434\u043e\u043d\u0441\u043a\u0438' },
+  'ms': { name: 'Malay', nativeName: 'Bahasa Melayu' },
+  'ml': { name: 'Malayalam', nativeName: '\u0d2e\u0d32\u0d2f\u0d3e\u0d33\u0d02' },
+  'mr': { name: 'Marathi', nativeName: '\u092e\u0930\u093e\u0920\u0940' },
+  'mn': { name: 'Mongolian', nativeName: '\u041c\u043e\u043d\u0433\u043e\u043b' },
+  'no': { name: 'Norwegian', nativeName: 'Norsk' },
+  'fa': { name: 'Persian', nativeName: '\u0641\u0627\u0631\u0633\u06cc' },
+  'pl': { name: 'Polish', nativeName: 'Polski' },  'pt': { name: 'Portuguese', nativeName: 'Portugu\u00eas' },
+  'pa': { name: 'Punjabi', nativeName: '\u0a2a\u0a70\u0a1c\u0a3e\u0a2c\u0a40' },
+  'ro': { name: 'Romanian', nativeName: 'Rom\u00e2n\u0103' },
+  'ru': { name: 'Russian', nativeName: '\u0420\u0443\u0441\u0441\u043a\u0438\u0439' },
+  'sr': { name: 'Serbian', nativeName: '\u0421\u0440\u043f\u0441\u043a\u0438' },
+  'sk': { name: 'Slovak', nativeName: 'Sloven\u010dina' },
+  'sl': { name: 'Slovenian', nativeName: 'Sloven\u0161\u010dina' },
+  'es': { name: 'Spanish', nativeName: 'Espa\u00f1ol' },
+  'sw': { name: 'Swahili', nativeName: 'Kiswahili' },  'sv': { name: 'Swedish', nativeName: 'Svenska' },
+  'tl': { name: 'Filipino', nativeName: 'Filipino' },
+  'ta': { name: 'Tamil', nativeName: '\u0ba4\u0bae\u0bbf\u0bb4\u0bcd' },
+  'te': { name: 'Telugu', nativeName: '\u0c24\u0c46\u0c32\u0c41\u0c17\u0c41' },
+  'th': { name: 'Thai', nativeName: '\u0e44\u0e17\u0e22' },
+  'tr': { name: 'Turkish', nativeName: 'T\u00fcrk\u00e7e' },
+  'uk': { name: 'Ukrainian', nativeName: '\u0423\u043a\u0440\u0430\u0457\u043d\u0441\u044c\u043a\u0430' },
+  'ur': { name: 'Urdu', nativeName: '\u0627\u0631\u062f\u0648' },
+  'vi': { name: 'Vietnamese', nativeName: 'Ti\u1ebfng Vi\u1ec7t' },
+};
 
 class I18nInitializer {
   constructor(config = {}) {
@@ -67,7 +94,9 @@ class I18nInitializer {
       ...config
     };
         this.format = getFormatAdapter(this.config.format);
-    this.config.supportedExtensions = [this.format.extension];
+    if (!this.config.supportedExtensions || !this.config.supportedExtensions.length) {
+      this.config.supportedExtensions = [this.format.extension];
+    }
     this.detectedFramework = detectFramework(process.cwd());
     if (this.detectedFramework && !this.config.translationPatterns) {
       this.config.translationPatterns = this.detectedFramework.patterns;
@@ -79,7 +108,9 @@ class I18nInitializer {
       : path.join(this.sourceDir, this.config.sourceLanguage);
     
     // Ensure defaultLanguages is properly initialized from config
-    this.config.defaultLanguages = this.config.defaultLanguages || ['en', 'de', 'es', 'fr', 'ru'];
+    if (!this.config.defaultLanguages || !Array.isArray(this.config.defaultLanguages) || this.config.defaultLanguages.length === 0) {
+      this.config.defaultLanguages = ['en', 'de', 'es', 'fr', 'ru'];
+    }
     
     // No longer create readline interface here - use CLI helpers
     this.rl = null;

package/main/i18ntk-scanner.js

@@ -22,6 +22,7 @@ const { getUnifiedConfig, displayHelp } = require('../utils/config-helper');
 const { loadTranslations } = require('../utils/i18n-helper');
 const SecurityUtils = require('../utils/security');
 const SetupEnforcer = require('../utils/setup-enforcer');
+const { detectProjectFramework, getFrameworkPatterns, getFrameworkSuggestions, SCANNER_EXTENSIONS, WRAPPER_SKIP_PATTERNS, getExcludeDirs } = require('../utils/framework-detector');
 
 // Ensure setup is complete before running
 (async () => {
@@ -225,11 +226,26 @@ class I18nTextScanner {
       const packageJson = SecurityUtils.safeParseJSON(packageJsonContent);
       const deps = { ...packageJson.dependencies, ...packageJson.devDependencies };
 
+      if (deps.next || deps['next-intl'] || deps['next-i18next']) return 'next';
       if (deps.react || deps['react-dom']) return 'react';
       if (deps.vue || deps['vue-router']) return 'vue';
       if (deps['@angular/core'] || deps.angular) return 'angular';
-      if (deps.next) return 'next';
       if (deps.svelte) return 'svelte';
+      if (deps.astro || deps['astro-i18next'] || deps['@astrojs/i18n']) return 'astro';
+      if (deps['@remix-run/react'] || deps['remix-i18next']) return 'remix';
+      if (deps.gatsby || deps['gatsby-plugin-react-i18next']) return 'gatsby';
+      if (deps['@builder.io/qwik'] || deps['qwik-speak']) return 'qwik';
+      if (deps['solid-js'] || deps['@solid-primitives/i18n']) return 'solid';
+      if (deps['ember-source'] || deps['ember-intl']) return 'ember';
+      if (deps.nuxt || deps['@nuxtjs/i18n']) return 'nuxt';
+
+      // Check for Rust project
+      const cargoTomlPath = path.join(projectRoot, 'Cargo.toml');
+      if (SecurityUtils.safeExistsSync(cargoTomlPath, projectRoot)) return 'rust';
+
+      // Check for Go project
+      const goModPath = path.join(projectRoot, 'go.mod');
+      if (SecurityUtils.safeExistsSync(goModPath, projectRoot)) return 'go';
 
       return 'vanilla';
     } catch (error) {
@@ -255,57 +271,110 @@ class I18nTextScanner {
 
     const frameworkSpecific = {
       react: [
-        // React specific patterns - enhanced for i18next detection
         /children:\s*["']([^"']{2,99})["']/g,
         /dangerouslySetInnerHTML={{\s*__html:\s*["']([^"']{2,99})["']/g,
-        // JSX text content without translation
         />([^<{][^<>{]*[^}>])</g,
-        // Button text
         /<button[^>]*>([^<]{2,99})<\/button>/g,
-        // Span text
-        /<span[^>]*>([^<]{2,99})<\/span>/g
+        /<span[^>]*>([^<]{2,99})<\/span>/g,
+        /<(?:FormattedMessage|Trans)[\s\S]*?\b(?:id|defaultMessage|i18nKey)\s*=\s*(?:\{|)(['"`])([^'"`}]+)\1[^>]*>/g,
+        /<(?:FormattedMessage|Trans)[\s\S]*?\b(?:id|defaultMessage|i18nKey)\s*=\s*\{[']\s*([^'}]+\.[^'}]+)\s*[']/g
       ],
       vue: [
-        // Vue specific patterns - enhanced for vue-i18n detection
         /v-text=["']([^"']{2,99})["']/g,
         /v-html=["']([^"']{2,99})["']/g,
-        // Vue template text
         />([^<{][^<>{]*[^}>])</g,
-        // Button text
         /<button[^>]*>([^<]{2,99})<\/button>/g,
-        // Span text
-        /<span[^>]*>([^<]{2,99})<\/span>/g
+        /<span[^>]*>([^<]{2,99})<\/span>/g,
+        /\$t\(["']([^"']{2,99})["']\)/g,
+        /v-t=["']([^"']{2,99})["']/g
       ],
       angular: [
-        // Angular specific patterns - enhanced for ngx-translate detection
         /\[innerHTML\]=["']([^"']{2,99})["']/g,
         /\[textContent\]=["']([^"']{2,99})["']/g,
-        // Angular template text
         />([^<{][^<>{]*[^}>])</g,
-        // Button text
         /<button[^>]*>([^<]{2,99})<\/button>/g,
-        // Span text
-        /<span[^>]*>([^<]{2,99})<\/span>/g
+        /<span[^>]*>([^<]{2,99})<\/span>/g,
+        /i18n=["']([^"']{2,99})["']/g,
+        /\[attr\.title\]=["']([^"']{2,99})["']/g
+      ],
+      next: [
+        /children:\s*["']([^"']{2,99})["']/g,
+        />([^<{][^<>{]*[^}>])</g,
+        /<button[^>]*>([^<]{2,99})<\/button>/g,
+        /<(?:FormattedMessage|Trans)[\s\S]*?\b(?:id|defaultMessage|i18nKey)\s*=\s*(?:\{|)(['"`])([^'"`}]+)\1[^>]*>/g
+      ],
+      svelte: [
+        />([^<{][^<>{]*[^}>])</g,
+        /<button[^>]*>([^<]{2,99})<\/button>/g,
+        /\$_\(["']([^"']{2,99})["']\)/g,
+        /t\.set\(["']([^"']{2,99})["']/g,
+        /\{#if[^}]*\}([^<]{2,99})\{\/if\}/g
+      ],
+      astro: [
+        />([^<{][^<>{]*[^}>])</g,
+        /<button[^>]*>([^<]{2,99})<\/button>/g,
+        /t\(["']([^"']{2,99})["']\)/g
+      ],
+      remix: [
+        /children:\s*["']([^"']{2,99})["']/g,
+        />([^<{][^<>{]*[^}>])</g,
+        /<button[^>]*>([^<]{2,99})<\/button>/g,
+        /<(?:FormattedMessage|Trans)[\s\S]*?\b(?:id|defaultMessage|i18nKey)\s*=\s*(?:\{|)(['"`])([^'"`}]+)\1[^>]*>/g
+      ],
+      qwik: [
+        />([^<{][^<>{]*[^}>])</g,
+        /<button[^>]*>([^<]{2,99})<\/button>/g,
+        /t\(["']([^"']{2,99})["']\)/g,
+        /useTranslate\(\)/g
+      ],
+      solid: [
+        />([^<{][^<>{]*[^}>])</g,
+        /<button[^>]*>([^<]{2,99})<\/button>/g,
+        /t\(["']([^"']{2,99})["']\)/g,
+        /useI18n\(\)\.t\(["']([^"']{2,99})["']\)/g
+      ],
+      ember: [
+        />([^<{][^<>{]*[^}>])</g,
+        /\{\{t\s+["']([^"']{2,99})["']\s*\}\}/g,
+        /\{\{intl\.t\s+["']([^"']{2,99})["']\s*\}\}/g
+      ],
+      gatsby: [
+        /children:\s*["']([^"']{2,99})["']/g,
+        />([^<{][^<>{]*[^}>])</g,
+        /<(?:Trans)[\s\S]*?\bi18nKey\s*=\s*(?:\{|)(['"`])([^'"`}]+)\1[^>]*>/g
       ],
       django: [
-        // Django template patterns
         /\{\%\s*trans\s+["']([^"']{2,99})["']\s*%\}/g,
         /\{\%\s*blocktrans\s*%\}([^%]{2,99})\{\%\s*endblocktrans\s*%\}/g,
         /{{\s*_["']([^"']{2,99})["']\s*}}/g,
         /{{\s*gettext\(["']([^"']{2,99})["']\)\s*}}/g
       ],
       flask: [
-        // Flask/Jinja2 template patterns
         /\{\{\s*_["']([^"']{2,99})["']\s*}}/g,
         /\{\{\s*gettext\(["']([^"']{2,99})["']\)\s*}}/g,
         /\{\{\s*lazy_gettext\(["']([^"']{2,99})["']\)\s*}}/g
       ],
       python: [
-        // Python source patterns
         /gettext\(["']([^"']{2,99})["']\)/g,
         /_\(["']([^"']{2,99})["']\)/g,
         /gettext_lazy\(["']([^"']{2,99})["']\)/g,
         /lazy_gettext\(["']([^"']{2,99})["']\)/g
+      ],
+      rust: [
+        /bundle\.get_message\(\s*["']([^"']{2,99})["']\)/g,
+        /\.get_message\s*\(\s*["']([^"']{2,99})["']\)/g,
+        /ts!\s*\(\s*["']([^"']{2,99})["']/g,
+        /fluent!\s*\(\s*["']([^"']{2,99})["']/g
+      ],
+      go: [
+        /i18n\.Translate\([^,]+,\s*["']([^"']{2,99})["']\)/g,
+        /i18n\.NewMessage\([^,]+,\s*["']([^"']{2,99})["']\)/g,
+        /t\.Get\([^,]+,\s*["']([^"']{2,99})["']\)/g
+      ],
+      vanilla: [
+        /t\(["']([^"']{2,99})["']\)/g,
+        /i18n\.t\(["']([^"']{2,99})["']\)/g,
+        /translate\(["']([^"']{2,99})["']\)/g
       ]
     };
 
@@ -518,7 +587,7 @@ class I18nTextScanner {
       key: `ui.${key}`,
       original: text,
       translationKey: `t('ui.${key}')`,
-      frameworkSpecific: this.getFrameworkSpecific(text)
+      frameworkSpecific: getFrameworkSuggestions(this.framework, text)
     };
   }
 
@@ -526,16 +595,51 @@ class I18nTextScanner {
     const frameworks = {
       react: {
         hook: `const { t } = useTranslation();`,
-        usage: `{t('ui.${text.toLowerCase().replace(/[^a-z0-9\s]/g, '').replace(/\s+/g, '_')}')}`,
-        component: `<Trans i18nKey="ui.${text.toLowerCase().replace(/[^a-z0-9\s]/g, '').replace(/\s+/g, '_')}">${text}</Trans>`
+        usage: `{t('ui.${this._keySnippet(text)}')}`,
+        component: `<Trans i18nKey="ui.${this._keySnippet(text)}">${text}</Trans>`
+      },
+      next: {
+        hook: `const t = useTranslations();`,
+        usage: `{t('ui.${this._keySnippet(text)}')}`,
+        component: `<Trans i18nKey="ui.${this._keySnippet(text)}">${text}</Trans>`
       },
       vue: {
-        directive: `{{ $t('ui.${text.toLowerCase().replace(/[^a-z0-9\s]/g, '').replace(/\s+/g, '_')}') }}`,
-        method: `this.$t('ui.${text.toLowerCase().replace(/[^a-z0-9\s]/g, '').replace(/\s+/g, '_')}')`
+        directive: `{{ $t('ui.${this._keySnippet(text)}') }}`,
+        method: `this.$t('ui.${this._keySnippet(text)}')`
       },
       angular: {
         pipe: `{{ '${text}' | translate }}`,
-        service: `this.translateService.instant('ui.${text.toLowerCase().replace(/[^a-z0-9\s]/g, '').replace(/\s+/g, '_')}')`
+        service: `this.translateService.instant('ui.${this._keySnippet(text)}')`
+      },
+      svelte: {
+        store: `$_(('ui.${this._keySnippet(text)}'))`,
+        method: `t.set('ui.${this._keySnippet(text)}', '${text}')`
+      },
+      astro: {
+        import: `import { t } from 'astro-i18next';`,
+        usage: `{t('ui.${this._keySnippet(text)}')}`
+      },
+      remix: {
+        hook: `const { t } = useTranslation();`,
+        usage: `{t('ui.${this._keySnippet(text)}')}`,
+        server: `export const handle = i18next.handle;`
+      },
+      qwik: {
+        hook: `const t = useTranslate();`,
+        usage: `{t('ui.${this._keySnippet(text)}')}`
+      },
+      solid: {
+        hook: `const [t] = useI18n();`,
+        usage: `{t('ui.${this._keySnippet(text)}')}`
+      },
+      ember: {
+        template: `{{t 'ui.${this._keySnippet(text)}'}}`,
+        helper: `this.intl.t('ui.${this._keySnippet(text)}')`
+      },
+      gatsby: {
+        hook: `const { t } = useTranslation();`,
+        usage: `{t('ui.${this._keySnippet(text)}')}`,
+        plugin: `'gatsby-plugin-react-i18next'`
       },
       django: {
         template: `{% trans '${text}' %}`,
@@ -552,14 +656,29 @@ class I18nTextScanner {
         underscore: `from gettext import gettext as _\n_('${text}')`,
         lazy: `from gettext import gettext_lazy as _\n_('${text}')`
       },
+      rust: {
+        fluent: `bundle.get_message("ui_${this._keySnippet(text)}")`,
+        gettext: `gettext("ui_${this._keySnippet(text)}")`
+      },
+      go: {
+        translate: `i18n.Translate("ui_${this._keySnippet(text)}")`,
+        message: `i18n.NewMessage("ui_${this._keySnippet(text)}")`
+      },
       vanilla: {
-        generic: `t('ui.${text.toLowerCase().replace(/[^a-z0-9\s]/g, '').replace(/\s+/g, '_')}')`
+        generic: `t('ui.${this._keySnippet(text)}')`
       }
     };
 
     return frameworks[this.framework] || frameworks.vanilla;
   }
 
+  _keySnippet(text) {
+    return text.toLowerCase()
+      .replace(/[^a-z0-9\s]/g, '')
+      .replace(/\s+/g, '_')
+      .substring(0, 40);
+  }
+
   async scanDirectory(dir, options = {}) {
     const {
       patterns = [],
@@ -574,7 +693,7 @@ class I18nTextScanner {
     }
 
     const allResults = [];
-    const extensions = ['.js', '.jsx', '.ts', '.tsx', '.vue', '.html', '.svelte', '.py', '.pyx', '.pyi'];
+    const extensions = [...SCANNER_EXTENSIONS];
 
     const scanRecursive = (currentDir) => {
       const items = SecurityUtils.safeReaddirSync(currentDir, path.dirname(currentDir), { withFileTypes: true });
@@ -736,7 +855,7 @@ class I18nTextScanner {
     } else if (fwPref && fwPref !== 'auto') {
       this.framework = fwPref;
     } else if (fwDetectEnabled) {
-      const detected = this.detectFramework(process.cwd());
+      const detected = detectProjectFramework(process.cwd());
       this.framework = detected || fwFallback;
     } else {
       this.framework = fwFallback;
@@ -762,7 +881,7 @@ class I18nTextScanner {
     console.log(this.t('scanner.starting', { framework: this.framework }));
     console.log(this.t('scanner.sourceDirectory', { sourceDir: this.sourceDir }));
 
-    const patterns = this.getFrameworkPatterns(this.framework);
+    const patterns = getFrameworkPatterns(this.framework);
     const exclusions = this.config.exclude || ['node_modules', '.git', 'dist', 'build'];
     const minLength = this.config.minLength || 3;
     const maxLength = this.config.maxLength || 100;

package/main/i18ntk-translate.js

@@ -1131,10 +1131,13 @@ async function run(args) {
     }
   }
 
+  const hadRealErrors = grandTranslated === 0 && grandTotal > 0;
+  const hasResiduals = allResidualUntranslated.length > 0;
+
   return {
-    success: allResidualUntranslated.length === 0,
-    exitCode: allResidualUntranslated.length === 0 ? ExitCodes.SUCCESS : ExitCodes.VALIDATION_FAILED,
-    error: allResidualUntranslated.length === 0 ? undefined : 'Auto Translate left untranslated placeholder values',
+    success: !hadRealErrors,
+    exitCode: hadRealErrors ? ExitCodes.VALIDATION_FAILED : ExitCodes.SUCCESS,
+    error: hadRealErrors ? 'Auto Translate failed to translate any values' : undefined,
     total: grandTotal,
     translated: grandTranslated,
     skipped: grandSkipped,