i18ntk 4.1.0 → 4.2.1

package/main/i18ntk-init.js

@@ -24,30 +24,33 @@ const { getFormatAdapter } = require('../utils/format-manager');
 const UIi18n = require('./i18ntk-ui');
 loadTranslations();
 const { getUnifiedConfig, parseCommonArgs, displayHelp } = require('../utils/config-helper');
-const { showFrameworkWarningOnce } = require('../utils/cli-helper');
-const { createPrompt, isInteractive } = require('../utils/prompt-helper');
+const { showFrameworkWarningOnce } = require('../utils/cli-helper');
+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 = {
-  'de': { name: 'German', nativeName: 'Deutsch' },
-  'es': { name: 'Spanish', nativeName: 'Español' },
-  'fr': { name: 'French', nativeName: 'Français' },
-  'ru': { name: 'Russian', nativeName: 'Русский' },
-  'it': { name: 'Italian', nativeName: 'Italiano' },
-  'ja': { name: 'Japanese', nativeName: '日本語' },
-  'ko': { name: 'Korean', nativeName: '한국어' },
-  'zh': { name: 'Chinese', nativeName: '中文' },
-  'ar': { name: 'Arabic', nativeName: 'العربية' },
-  'hi': { name: 'Hindi', nativeName: 'हिन्दी' },
-  '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: 'Čeština' },
-  'hu': { name: 'Hungarian', nativeName: 'Magyar' },
-  'tr': { name: 'Turkish', nativeName: 'Türkçe' }
+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' }
 };
 
 class I18nInitializer {
@@ -76,7 +79,7 @@ class I18nInitializer {
       : path.join(this.sourceDir, this.config.sourceLanguage);
     
     // Ensure defaultLanguages is properly initialized from config
-    this.config.defaultLanguages = this.config.defaultLanguages || ['de', 'es', 'fr', 'ru'];
+    this.config.defaultLanguages = this.config.defaultLanguages || ['en', 'de', 'es', 'fr', 'ru'];
     
     // No longer create readline interface here - use CLI helpers
     this.rl = null;
@@ -153,7 +156,7 @@ class I18nInitializer {
       return true;
     }
     const answer = await this.prompt('\n' + t('init.continueWithoutI18nPrompt'));
-    return answer.toLowerCase() === 'y' || answer.toLowerCase() === 'yes';
+    return parseConfirmation(answer, { language: this.config.uiLanguage || this.config.language || 'en', defaultValue: false });
   }
 
   // Add the missing prompt method
@@ -767,7 +770,7 @@ class I18nInitializer {
     await flushStdout();
     const enableProtection = await ask('\n' + t('adminPin.setup_prompt'));
 
-    if (enableProtection.toLowerCase() === 'y' || enableProtection.toLowerCase() === 'yes') {
+    if (parseConfirmation(enableProtection, { language: this.config.uiLanguage || this.config.language || 'en', defaultValue: false })) {
       try {
         const adminAuth = new AdminAuth();
         await adminAuth.initialize();
@@ -816,16 +819,16 @@ class I18nInitializer {
     }
 
     const { ask } = require('../utils/cli');
-    console.log('\nBackup Settings');
-    console.log('Backups are disabled by default to avoid backup recursion and repo pollution.');
-    const enableAnswer = await ask('Enable automatic backups? (y/N): ');
-    const enabled = ['y', 'yes'].includes(String(enableAnswer || '').trim().toLowerCase());
+    console.log('\n' + t('init.backup.title'));
+    console.log(t('init.backup.description'));
+    const enableAnswer = await ask(t('init.backup.enablePrompt'));
+    const enabled = parseConfirmation(enableAnswer, { language: this.config.uiLanguage || this.config.language || 'en', defaultValue: false });
 
     if (!enabled) {
       return defaultBackupConfig;
     }
 
-    const keepAnswer = await ask('How many backups should be kept automatically (1-3, default 1): ');
+    const keepAnswer = await ask(t('init.backup.keepPrompt'));
     const parsedKeep = parseInt(String(keepAnswer || '').trim(), 10);
     const maxBackups = Number.isInteger(parsedKeep) ? Math.min(Math.max(parsedKeep, 1), 3) : 1;
 
@@ -895,7 +898,7 @@ class I18nInitializer {
     let perLanguage = [];
     if (structure !== 'existing') {
       const duplicateChoice = await ask('\n' + t('init.setup.apply_all_prompt'));
-      duplicateStructure = duplicateChoice.toLowerCase() === 'y' || duplicateChoice.toLowerCase() === 'yes';
+      duplicateStructure = parseConfirmation(duplicateChoice, { language: this.config.uiLanguage || this.config.language || 'en', defaultValue: true });
       if (!duplicateStructure) {
         // Prompt for languages to include/exclude
         console.log(t('init.setup.per_language_intro'));
@@ -1054,138 +1057,131 @@ class I18nInitializer {
     console.log(t('init.nextStep3'));
   }
 
-  // Generate completion summary with proper error handling
-  async generateCompletionSummary(results, targetLanguages) {
-    try {
-      console.log('\n' + '='.repeat(50));
-      console.log(t('init.initializationSummaryTitle'));
-      console.log(t('common.separator'));
-
-      let totalChanges = 0;
-      let languagesProcessed = 0;
-      let missingKeysAdded = 0;
-
-      Object.entries(results || {}).forEach(([lang, data]) => {
-        if (!data || typeof data !== 'object') return;
-
-        const langName = LANGUAGE_CONFIG[lang]?.name || 'Unknown';
-        const stats = data.totalStats || { total: 0, translated: 0, percentage: 0, missing: 0 };
-
-        const statusIcon = stats.percentage === 100 ? '✅' : stats.percentage >= 80 ? '🟡' : '🔴';
-
-        console.log(
-          t('init.languageSummary', {
-            icon: statusIcon,
-            name: langName,
-            code: lang,
-            percentage: stats.percentage || 0,
-          })
-        );
-
-        if (data.files && Array.isArray(data.files)) {
-          console.log(t('init.languageFiles', { count: data.files.length }));
-        }
-
-        console.log(
-          t('init.languageKeys', {
-            translated: stats.translated || 0,
-            total: stats.total || 0,
-          })
-        );
-
-        console.log(t('init.languageMissing', { count: stats.missing || 0 }));
-
-        totalChanges += (stats.translated || 0) + (stats.missing || 0);
-        languagesProcessed += 1;
-        missingKeysAdded += stats.missing || 0;
-      });
-
-      console.log('\n📊 COMPLETION SUMMARY');
-      console.log(t('common.separator'));
-      console.log(`📝 Total changes: ${totalChanges}`);
-      console.log(`🌍 Languages processed: ${languagesProcessed}`);
-      console.log(`➕ Missing keys added: ${missingKeysAdded}`);
-
-      if (process.stdin.isTTY && !this.config?.noPrompt) {
-        const { ask } = require('../utils/cli');
-        const generateReport = await ask('\n🤖 Would you like a report generated? (Y/N): ');
-        if (generateReport.toLowerCase() === 'y' || generateReport.toLowerCase() === 'yes') {
-          await this.generateDetailedReport(results, targetLanguages);
-        }
-      }
-    } catch (error) {
-      console.error('\n❌ Error during completion:', error.message);
-      console.log('📊 COMPLETION SUMMARY (Basic)');
-      console.log(t('common.separator'));
-      console.log(`🌍 Languages processed: ${Object.keys(results || {}).length}`);
-    }
-  }
+  // Generate completion summary with proper error handling
+  async generateCompletionSummary(results, targetLanguages) {
+    return await this.generateLocalizedCompletionSummary(results, targetLanguages);
+  }
 
-  // Generate detailed report
+  async generateLocalizedCompletionSummary(results, targetLanguages) {
+    try {
+      console.log('\n' + '='.repeat(50));
+      console.log(t('init.initializationSummaryTitle'));
+      console.log(t('common.separator'));
+
+      let totalChanges = 0;
+      let languagesProcessed = 0;
+      let missingKeysAdded = 0;
+
+      Object.entries(results || {}).forEach(([lang, data]) => {
+        if (!data || typeof data !== 'object') return;
+
+        const langName = LANGUAGE_CONFIG[lang]?.name || 'Unknown';
+        const stats = data.totalStats || { total: 0, translated: 0, percentage: 0, missing: 0 };
+        const statusIcon = stats.percentage === 100 ? '✅' : stats.percentage >= 80 ? '🟡' : '🔴';
+
+        console.log(t('init.languageSummary', {
+          icon: statusIcon,
+          name: langName,
+          code: lang,
+          percentage: stats.percentage || 0,
+        }));
+
+        if (Array.isArray(data.files)) {
+          console.log(t('init.languageFiles', { count: data.files.length }));
+        }
+        console.log(t('init.languageKeys', { translated: stats.translated || 0, total: stats.total || 0 }));
+        console.log(t('init.languageMissing', { count: stats.missing || 0 }));
+
+        totalChanges += (stats.translated || 0) + (stats.missing || 0);
+        languagesProcessed += 1;
+        missingKeysAdded += stats.missing || 0;
+      });
+
+      console.log('\n' + t('init.completionSummaryTitle'));
+      console.log(t('common.separator'));
+      console.log(t('init.totalChanges', { count: totalChanges }));
+      console.log(t('init.languagesProcessed', { count: languagesProcessed }));
+      console.log(t('init.missingKeysAdded', { count: missingKeysAdded }));
+
+      if (process.stdin.isTTY && !this.config?.noPrompt) {
+        const { ask } = require('../utils/cli');
+        const generateReport = await ask('\n' + t('init.reportPrompt'));
+        if (parseConfirmation(generateReport, { language: this.config.uiLanguage || this.config.language || 'en', defaultValue: true })) {
+          await this.generateDetailedReport(results, targetLanguages);
+        }
+      }
+    } catch (error) {
+      console.error('\n' + t('init.completionError', { error: error.message }));
+      console.log(t('init.completionSummaryBasicTitle'));
+      console.log(t('common.separator'));
+      console.log(t('init.languagesProcessed', { count: Object.keys(results || {}).length }));
+    }
+  }
+
+  // Generate detailed report
   async generateDetailedReport(results, targetLanguages) {
     try {
       const outputDir = this.config.outputDir || path.join(process.cwd(), 'i18ntk-reports');
-      if (!SecurityUtils.safeExistsSync(outputDir)) {
-        fs.mkdirSync(outputDir, { recursive: true });
-      }
-      
-      const reportPath = path.join(outputDir, 'init-report.json');
-      const report = {
-        timestamp: new Date().toISOString(),
-        languages: targetLanguages,
-        results: results,
-        summary: {
-          languagesProcessed: targetLanguages.length,
-          totalFiles: Object.values(results).reduce((sum, data) => sum + (data.files?.length || 0), 0),
-          totalKeys: Object.values(results).reduce((sum, data) => sum + (data.totalStats?.total || 0), 0),
-          totalMissing: Object.values(results).reduce((sum, data) => sum + (data.totalStats?.missing || 0), 0)
-        }
-      };
-
-      await fs.promises.writeFile(reportPath, JSON.stringify(report, null, 2));
-      console.log(`✅ Report generated: ${reportPath}`);
-    } catch (error) {
-      console.error('❌ Failed to generate report:', error.message);
-    }
-  }
+      if (!SecurityUtils.safeExistsSync(outputDir)) {
+        fs.mkdirSync(outputDir, { recursive: true });
+      }
+      
+      const reportPayload = {
+        timestamp: new Date().toISOString(),
+        languages: targetLanguages,
+        results,
+        summary: {
+          languagesProcessed: targetLanguages.length,
+          totalFiles: Object.values(results).reduce((sum, data) => sum + (data.files?.length || 0), 0),
+          totalKeys: Object.values(results).reduce((sum, data) => sum + (data.totalStats?.total || 0), 0),
+          totalMissing: Object.values(results).reduce((sum, data) => sum + (data.totalStats?.missing || 0), 0)
+        }
+      };
+      const format = normalizeReportFormat(this.config.reports?.format || this.config.reportFormat || 'markdown');
+      const writtenPath = await writeReportFile(outputDir, 'init-report', reportPayload, { format, title: 'I18NTK Init Report' });
+      console.log(t('init.reportGenerated', { reportPath: writtenPath }));
+    } catch (error) {
+      console.error(t('init.reportFailed', { error: error.message }));
+    }
+  }
 
   // Offer interactive locale optimization after initialization
-  async offerLocaleOptimization() {
-    try {
-      console.log('\n' + '='.repeat(60));
-      console.log('🎯 **PACKAGE SIZE OPTIMIZATION**');
-      console.log('='.repeat(60));
-      
-      try {
-        // Import locale optimizer directly
-        const LocaleOptimizer = require('../utils/locale-optimizer');
-        
-        // First run dry run to show current state
-        console.log('\n🔍 Running locale optimization preview...');
-        const optimizer = new LocaleOptimizer();
-        await optimizer.run({ dryRun: true });
-
-        console.log('\n💡 You can reduce package size by selecting only the languages you need');
-        
-        const answer = await this.prompt('\n🤖 Would you like to run interactive optimization now? (y/n): ');
-        
-        if (answer.toLowerCase() === 'y' || answer.toLowerCase() === 'yes') {
-          console.log('\n🚀 Starting interactive locale optimization...');
-          await optimizer.run({ interactive: true });
-          console.log('\n✅ Package optimization completed!');
-        } else {
-          console.log('\n💡 You can run locale optimization later with:');
-          console.log('   node utils/locale-optimizer.js --interactive');
-        }
-      } catch (error) {
-        console.log('\n⚠️ Could not offer locale optimization:', error.message);
-        console.log('\n💡 You can run locale optimization later with:');
-        console.log('   node utils/locale-optimizer.js --interactive');
-      }
-    } catch (error) {
-      console.log('\n⚠️ Could not offer locale optimization:', error.message);
-    }
-  }
+  async offerLocaleOptimization() {
+    try {
+      console.log('\n' + '='.repeat(60));
+      console.log(t('init.optimize.title'));
+      console.log('='.repeat(60));
+      
+      try {
+        // Import locale optimizer directly
+        const LocaleOptimizer = require('../utils/locale-optimizer');
+        
+        // First run dry run to show current state
+        console.log('\n' + t('init.optimize.preview'));
+        const optimizer = new LocaleOptimizer();
+        await optimizer.run({ dryRun: true });
+
+        console.log('\n' + t('init.optimize.reduceTip'));
+        
+        const answer = await this.prompt('\n' + t('init.optimize.prompt'));
+        
+        if (parseConfirmation(answer, { language: this.config.uiLanguage || this.config.language || 'en', defaultValue: false })) {
+          console.log('\n' + t('init.optimize.starting'));
+          await optimizer.run({ interactive: true });
+          console.log('\n' + t('init.optimize.completed'));
+        } else {
+          console.log('\n' + t('init.optimize.later'));
+          console.log('   node utils/locale-optimizer.js --interactive');
+        }
+      } catch (error) {
+        console.log('\n' + t('init.optimize.unavailable', { error: error.message }));
+        console.log('\n' + t('init.optimize.later'));
+        console.log('   node utils/locale-optimizer.js --interactive');
+      }
+    } catch (error) {
+      console.log('\n' + t('init.optimize.unavailable', { error: error.message }));
+    }
+  }
 
   // Run the initialization process with admin authentication
   async run(options = {}) {

package/main/i18ntk-setup.js

@@ -15,7 +15,7 @@ const SecurityUtils = require('../utils/security');
 const configManager = require('../utils/config-manager');
 const SetupService = require('./manage/services/SetupService');
 
-class I18nSetupManager {
+class I18nSetupManager {
     constructor() {
         // Use the new SetupService for core business logic
         this.setupService = new SetupService();
@@ -26,18 +26,41 @@ class I18nSetupManager {
         return await this.setupService.setup();
     }
 
-    
-}
-
-// CLI interface
-if (require.main === module) {
-    const setupManager = new I18nSetupManager();
-    setupManager.setup().catch(console.error);
-}
+    
+}
+
+function printHelp() {
+    console.log([
+        '',
+        'i18ntk-setup - foundational i18n toolkit setup',
+        '',
+        'Usage:',
+        '  i18ntk-setup [options]',
+        '  node main/i18ntk-setup.js [options]',
+        '',
+        'Options:',
+        '  -h, --help     Show this help message',
+        '',
+        'The setup command detects the current project, writes i18ntk settings,',
+        'and generates i18ntk-setup-report.json in the current working directory.',
+    ].join('\n'));
+}
+
+// CLI interface
+if (require.main === module) {
+    if (process.argv.slice(2).some(arg => arg === '--help' || arg === '-h')) {
+        printHelp();
+        process.exit(0);
+    }
+
+    const setupManager = new I18nSetupManager();
+    setupManager.setup().catch(console.error);
+}
 
 // Export both the class and a run function for direct usage
 module.exports = I18nSetupManager;
-module.exports.run = async function() {
-    const setupManager = new I18nSetupManager();
-    return await setupManager.setup();
-};
+module.exports.run = async function() {
+    const setupManager = new I18nSetupManager();
+    return await setupManager.setup();
+};
+module.exports.printHelp = printHelp;

package/main/i18ntk-sizing.js

@@ -31,12 +31,13 @@
  *   i18ntk sizing --threshold=30 --output-report
  */
 
-const fs = require('fs');
-const path = require('path');
-const { loadTranslations, t } = require('../utils/i18n-helper');
-const configManager = require('../settings/settings-manager');
-const SecurityUtils = require('../utils/security');
-const { getUnifiedConfig } = require('../utils/config-helper');
+const fs = require('fs');
+const path = require('path');
+const { loadTranslations, t } = require('../utils/i18n-helper');
+const configManager = require('../settings/settings-manager');
+const projectConfigManager = require('../utils/config-manager');
+const SecurityUtils = require('../utils/security');
+const { getUnifiedConfig } = require('../utils/config-helper');
 const { logger } = require('../utils/logger');
 const { getGlobalReadline, closeGlobalReadline } = require('../utils/cli');
 const SetupEnforcer = require('../utils/setup-enforcer');
@@ -1120,8 +1121,10 @@ Generated: ${new Date().toISOString()}
       'l': '',
       'o': true,
       'f': 'table',
-      't': 50,
-      'd': false
+      't': 50,
+      'd': false,
+      sourceDirExplicit: false,
+      outputDirExplicit: false
     };
 
     for (let i = 0; i < args.length; i++) {
@@ -1138,9 +1141,10 @@ Generated: ${new Date().toISOString()}
         const key = keyValueMatch[1];
         const value = keyValueMatch[2];
         
-        if (key === 'source-dir' || key === 's') {
-          options['source-dir'] = value;
-          options.s = value;
+        if (key === 'source-dir' || key === 's') {
+          options['source-dir'] = value;
+          options.s = value;
+          options.sourceDirExplicit = true;
         } else if (key === 'languages' || key === 'l') {
           options.languages = value;
           options.l = value;
@@ -1167,9 +1171,10 @@ Generated: ${new Date().toISOString()}
           options['detailed-keys'] = value.toLowerCase() !== 'false';
         } else if (key === 'predict-expansion') {
           options['predict-expansion'] = value.toLowerCase() !== 'false';
-        } else if (key === 'output-dir') {
-          options['output-dir'] = value;
-        }
+        } else if (key === 'output-dir') {
+          options['output-dir'] = value;
+          options.outputDirExplicit = true;
+        }
         continue;
       }
       
@@ -1179,10 +1184,11 @@ Generated: ${new Date().toISOString()}
         const key = match[1];
         const nextArg = args[i + 1];
         
-        if (key === 'source-dir' || key === 's') {
-          options['source-dir'] = nextArg || options['source-dir'];
-          options.s = options['source-dir'];
-          if (nextArg && !nextArg.startsWith('-')) i++;
+        if (key === 'source-dir' || key === 's') {
+          options['source-dir'] = nextArg || options['source-dir'];
+          options.s = options['source-dir'];
+          options.sourceDirExplicit = true;
+          if (nextArg && !nextArg.startsWith('-')) i++;
         } else if (key === 'languages' || key === 'l') {
           options.languages = nextArg || options.languages;
           options.l = options.languages;
@@ -1238,6 +1244,7 @@ Generated: ${new Date().toISOString()}
           }
         } else if (key === 'output-dir') {
           options['output-dir'] = nextArg || options['output-dir'];
+          options.outputDirExplicit = true;
           if (nextArg && !nextArg.startsWith('-')) i++;
         }
       }
@@ -1268,15 +1275,25 @@ Options:
     return options;
   }
 
-  // Add run method for compatibility with manager
-  async run(options = {}) {
-    const { fromMenu = false } = options;
-    
-    const args = this.parseArgs();
-    const config = await getUnifiedConfig('sizing', args);
-
-    this.sourceDir = path.resolve(config.projectRoot || '.', config.sourceDir || './locales');
-    this.outputDir = path.resolve(config.projectRoot || '.', config.outputDir || './i18ntk-reports');
+  // Add run method for compatibility with manager
+  async run(options = {}) {
+    const { fromMenu = false } = options;
+
+    const args = this.parseArgs();
+    const commonArgs = {};
+    if (args.sourceDirExplicit) commonArgs.sourceDir = args['source-dir'];
+    if (args.outputDirExplicit) commonArgs.outputDir = args['output-dir'];
+    if (args['source-language']) commonArgs.sourceLanguage = args['source-language'];
+    const persistedConfig = projectConfigManager.getConfig();
+    const config = await getUnifiedConfig('sizing', commonArgs);
+    const projectRoot = config.projectRoot || persistedConfig.projectRoot || process.cwd();
+    const configuredSizingDir = persistedConfig.scriptDirectories?.sizing;
+    const translationDir = args.sourceDirExplicit
+      ? args['source-dir']
+      : (configuredSizingDir || persistedConfig.i18nDir || persistedConfig.sourceDir || config.i18nDir || config.sourceDir || './locales');
+
+    this.sourceDir = path.resolve(projectRoot, translationDir);
+    this.outputDir = path.resolve(config.projectRoot || '.', config.outputDir || './i18ntk-reports');
     this.threshold = args.threshold ?? config.processing?.sizingThreshold ?? 50;
     this.languages = args.languages ? args.languages.split(',').map(l => l.trim()) : [];
     this.outputReport = args['output-report'] !== undefined ? args['output-report'] : false;