i18ntk 2.3.8 → 2.5.0

package/utils/config-manager.js

@@ -1,9 +1,10 @@
 const fs = require('fs');
 const path = require('path');
-const os = require('os');
-const crypto = require('crypto');
-const SecurityUtils = require('./security');
-const { logger } = require('./logger');
+const os = require('os');
+const crypto = require('crypto');
+const SecurityUtils = require('./security');
+const { logger } = require('./logger');
+const { envManager } = require('./env-manager');
 
 // Determine package directory and user project root
 const packageDir = path.resolve(__dirname, '..');
@@ -20,55 +21,55 @@ const CONFIG_LOCK_PATH = `${PROJECT_CONFIG_PATH}.lock`;
 const CONFIG_LOCK_TIMEOUT_MS = 5000;
 const CONFIG_LOCK_STALE_MS = 15000;
 const CONFIG_LOCK_RETRY_MS = 50;
-let autosaveDisabledWarned = false;
-let defaultConfigNoticeShown = false;
-let configFallbackNoticeShown = false;
-
-function logInfo(message, details) {
-  logger.info(message, details);
-}
-
-function logWarn(message, details) {
-  logger.warn(message, details);
-}
-
-function logError(message, details) {
-  logger.error(message, details);
-}
-
-function notifyDefaultConfig(reason, error) {
-  if (defaultConfigNoticeShown) {
-    return;
-  }
-  defaultConfigNoticeShown = true;
-
-  logger.info('Using default configuration (reason: configuration error)', { reason });
-  if (logger.isDebugMode() && error) {
-    logger.debug(`Default configuration reason details: ${error.message}`, {
-      stack: error.stack
-    });
-  }
-}
-
-function notifyConfigFallback(error) {
-  if (!configFallbackNoticeShown) {
-    configFallbackNoticeShown = true;
-    logger.info('Using default configuration (reason: configuration error)', {
-      reason: 'configuration error'
-    });
-  }
-
-  logger.recordFirstError('config:load-fallback', {
-    message: error && error.message ? error.message : 'Unknown configuration error',
-    stack: error && error.stack ? error.stack : null
-  });
-
-  if (logger.isDebugMode() && error) {
-    logger.debug(`Configuration load fallback details: ${error.message}`, {
-      stack: error.stack
-    });
-  }
-}
+let autosaveDisabledWarned = false;
+let defaultConfigNoticeShown = false;
+let configFallbackNoticeShown = false;
+
+function logInfo(message, details) {
+  logger.info(message, details);
+}
+
+function logWarn(message, details) {
+  logger.warn(message, details);
+}
+
+function logError(message, details) {
+  logger.error(message, details);
+}
+
+function notifyDefaultConfig(reason, error) {
+  if (defaultConfigNoticeShown) {
+    return;
+  }
+  defaultConfigNoticeShown = true;
+
+  logger.info('Using default configuration (reason: configuration error)', { reason });
+  if (logger.isDebugMode() && error) {
+    logger.debug(`Default configuration reason details: ${error.message}`, {
+      stack: error.stack
+    });
+  }
+}
+
+function notifyConfigFallback(error) {
+  if (!configFallbackNoticeShown) {
+    configFallbackNoticeShown = true;
+    logger.info('Using default configuration (reason: configuration error)', {
+      reason: 'configuration error'
+    });
+  }
+
+  logger.recordFirstError('config:load-fallback', {
+    message: error && error.message ? error.message : 'Unknown configuration error',
+    stack: error && error.stack ? error.stack : null
+  });
+
+  if (logger.isDebugMode() && error) {
+    logger.debug(`Configuration load fallback details: ${error.message}`, {
+      stack: error.stack
+    });
+  }
+}
 
 // Setup tracking file
 const SETUP_COMPLETED_FILE = path.join(PROJECT_SETTINGS_DIR, 'setup.json');
@@ -362,8 +363,7 @@ async function acquireConfigLock(timeoutMs = CONFIG_LOCK_TIMEOUT_MS) {
 }
 
 function isAutosaveDisabled() {
-  const flag = String(process.env.I18NTK_DISABLE_AUTOSAVE || '').trim().toLowerCase();
-  return flag === '1' || flag === 'true' || flag === 'yes';
+  return envManager.getBoolean('I18NTK_DISABLE_AUTOSAVE');
 }
 
 function clone(obj) {
@@ -530,8 +530,8 @@ function loadConfig() {
      currentConfig = cfg;
      return currentConfig;
    } catch (error) {
-     logError('[i18ntk] Error in loadConfig', { error: error.message });
-     notifyConfigFallback(error);
+     logError('[i18ntk] Error in loadConfig', { error: error.message });
+     notifyConfigFallback(error);
      currentConfig = clone(DEFAULT_CONFIG);
      return currentConfig;
    } finally {
@@ -590,7 +590,7 @@ async function saveConfig(cfg = currentConfig) {
       currentConfig = cfg;
       return true;
     } catch (error) {
-      logError('[i18ntk] Error saving configuration', { error: error.message });
+      logError('[i18ntk] Error saving configuration', { error: error.message });
       return false;
     } finally {
       if (releaseLock) {
@@ -643,7 +643,7 @@ function getConfig() {
 
      // Check for legacy config for migration
      if (SecurityUtils.safeExistsSync(LEGACY_CONFIG_PATH)) {
-       logInfo('Migrating legacy configuration');
+       logInfo('Migrating legacy configuration');
        const legacyRaw = SecurityUtils.safeReadFileSync(LEGACY_CONFIG_PATH, path.dirname(LEGACY_CONFIG_PATH), 'utf8');
        const legacyConfig = SecurityUtils.safeParseJSON(legacyRaw);
        if (!legacyConfig || typeof legacyConfig !== 'object') {
@@ -651,7 +651,7 @@ function getConfig() {
        }
        const migratedConfig = { ...DEFAULT_CONFIG, ...legacyConfig };
        saveConfig(migratedConfig).catch((err) => {
-         logWarn('[i18ntk] Warning: failed to persist migrated configuration', { error: err.message });
+         logWarn('[i18ntk] Warning: failed to persist migrated configuration', { error: err.message });
        });
        currentConfig = migratedConfig;
 
@@ -669,18 +669,18 @@ function getConfig() {
      }
 
      // Use package defaults for new installation
-     notifyDefaultConfig('default initialization');
-     saveConfig(DEFAULT_CONFIG).catch((err) => {
-       logWarn('[i18ntk] Warning: failed to persist default configuration', { error: err.message });
-     });
-     currentConfig = DEFAULT_CONFIG;
-     return resolvePaths(DEFAULT_CONFIG);
-
-   } catch (error) {
-     notifyConfigFallback(error);
-     currentConfig = DEFAULT_CONFIG;
-     return resolvePaths(DEFAULT_CONFIG);
-   }
+     notifyDefaultConfig('default initialization');
+     saveConfig(DEFAULT_CONFIG).catch((err) => {
+       logWarn('[i18ntk] Warning: failed to persist default configuration', { error: err.message });
+     });
+     currentConfig = DEFAULT_CONFIG;
+     return resolvePaths(DEFAULT_CONFIG);
+
+   } catch (error) {
+     notifyConfigFallback(error);
+     currentConfig = DEFAULT_CONFIG;
+     return resolvePaths(DEFAULT_CONFIG);
+   }
 }
 
 async function setConfig(cfg) {

package/utils/env-manager.js

@@ -13,6 +13,54 @@ const ALLOWED_ENV_VARS = {
     validate: (value) => ['error', 'warn', 'info', 'debug', 'silent'].includes(value.toLowerCase()),
     transform: (value) => value.toLowerCase()
   },
+
+  'DEBUG_MODE': {
+    default: 'false',
+    validate: (value) => ['true', 'false', '1', '0', 'yes', 'no'].includes(value.toLowerCase()),
+    transform: normalizeBooleanString
+  },
+
+  'JSON_LOG': {
+    default: 'false',
+    validate: (value) => ['true', 'false', '1', '0', 'yes', 'no'].includes(value.toLowerCase()),
+    transform: normalizeBooleanString
+  },
+
+  'NODE_ENV': {
+    default: 'development',
+    validate: (value) => ['development', 'production', 'test'].includes(value.toLowerCase()),
+    transform: (value) => value.toLowerCase()
+  },
+
+  'CI': {
+    default: 'false',
+    validate: (value) => ['true', 'false', '1', '0', 'yes', 'no'].includes(value.toLowerCase()),
+    transform: normalizeBooleanString
+  },
+
+  'CONTINUOUS_INTEGRATION': {
+    default: 'false',
+    validate: (value) => ['true', 'false', '1', '0', 'yes', 'no'].includes(value.toLowerCase()),
+    transform: normalizeBooleanString
+  },
+
+  'NO_INTERACTIVE': {
+    default: 'false',
+    validate: (value) => ['true', 'false', '1', '0', 'yes', 'no'].includes(value.toLowerCase()),
+    transform: normalizeBooleanString
+  },
+
+  'npm_lifecycle_event': {
+    default: '',
+    validate: (value) => typeof value === 'string',
+    transform: (value) => value.trim()
+  },
+
+  'npm_config_loglevel': {
+    default: '',
+    validate: (value) => typeof value === 'string',
+    transform: (value) => value.trim().toLowerCase()
+  },
   
   'I18NTK_OUTDIR': {
     default: './i18ntk-reports',
@@ -21,29 +69,23 @@ const ALLOWED_ENV_VARS = {
   },
   
   // UI and interaction
-  'I18NTK_UI_LANGUAGE': {
-    default: 'en',
-    validate: (value) => ['en', 'de', 'es', 'fr', 'ru', 'ja', 'zh'].includes(value.toLowerCase()),
-    transform: (value) => value.toLowerCase()
-  },
+  'I18NTK_UI_LANGUAGE': {
+    default: 'en',
+    validate: (value) => ['en', 'de', 'es', 'fr', 'ru', 'ja', 'zh'].includes(value.toLowerCase()),
+    transform: (value) => value.toLowerCase()
+  },
   
   'I18NTK_SILENT': {
     default: 'false',
     validate: (value) => ['true', 'false', '1', '0', 'yes', 'no'].includes(value.toLowerCase()),
-    transform: (value) => {
-      const lower = value.toLowerCase();
-      return lower === 'true' || lower === '1' || lower === 'yes' ? 'true' : 'false';
-    }
+    transform: normalizeBooleanString
   },
   
   // Debug and development
   'I18NTK_DEBUG_LOCALES': {
-    default: '0',
+    default: 'false',
     validate: (value) => ['0', '1', 'true', 'false'].includes(value.toLowerCase()),
-    transform: (value) => {
-      const lower = value.toLowerCase();
-      return lower === '1' || lower === 'true' ? '1' : '0';
-    }
+    transform: normalizeBooleanString
   },
   
   // Runtime configuration
@@ -54,19 +96,19 @@ const ALLOWED_ENV_VARS = {
   },
   
   'I18NTK_I18N_DIR': {
-    default: './locales',
+    default: null,
     validate: (value) => typeof value === 'string' && value.length > 0,
     transform: (value) => value.trim()
   },
   
   'I18NTK_SOURCE_DIR': {
-    default: './locales',
+    default: null,
     validate: (value) => typeof value === 'string' && value.length > 0,
     transform: (value) => value.trim()
   },
   
   'I18NTK_PROJECT_ROOT': {
-    default: '.',
+    default: null,
     validate: (value) => typeof value === 'string' && value.length > 0,
     transform: (value) => value.trim()
   },
@@ -87,13 +129,57 @@ const ALLOWED_ENV_VARS = {
   'I18NTK_FRAMEWORK_DETECT': {
     default: 'true',
     validate: (value) => ['true', 'false', '1', '0', 'yes', 'no'].includes(value.toLowerCase()),
-    transform: (value) => {
-      const lower = value.toLowerCase();
-      return lower === 'true' || lower === '1' || lower === 'yes' ? 'true' : 'false';
-    }
+    transform: normalizeBooleanString
+  },
+
+  'I18NTK_DISABLE_AUTOSAVE': {
+    default: 'false',
+    validate: (value) => ['true', 'false', '1', '0', 'yes', 'no'].includes(value.toLowerCase()),
+    transform: normalizeBooleanString
+  },
+
+  'I18NTK_INTERNAL_PATH_PREFIXES': {
+    default: '',
+    validate: (value) => typeof value === 'string',
+    transform: (value) => value.trim()
+  },
+
+  'I18NTK_ENABLE_SECURITY_LOGS': {
+    default: 'false',
+    validate: (value) => ['true', 'false', '1', '0', 'yes', 'no'].includes(value.toLowerCase()),
+    transform: normalizeBooleanString
+  },
+
+  'TERM_PROGRAM': {
+    default: '',
+    validate: (value) => typeof value === 'string',
+    transform: (value) => value.trim()
+  },
+
+  'WT_SESSION': {
+    default: '',
+    validate: (value) => typeof value === 'string',
+    transform: (value) => value.trim()
+  },
+
+  'PSModulePath': {
+    default: '',
+    validate: (value) => typeof value === 'string',
+    transform: (value) => value.trim()
+  },
+
+  'POWERSHELL_EDITION': {
+    default: '',
+    validate: (value) => typeof value === 'string',
+    transform: (value) => value.trim()
   }
 };
 
+function normalizeBooleanString(value) {
+  const lower = String(value || '').toLowerCase();
+  return lower === 'true' || lower === '1' || lower === 'yes' ? 'true' : 'false';
+}
+
 // Security: Block access to sensitive environment variables
 const BLOCKED_PATTERNS = [
   /^SECRET/i,
@@ -170,6 +256,11 @@ class EnvironmentManager {
     }
   }
 
+  getBoolean(name) {
+    const value = this.get(name);
+    return value === true || value === 'true' || value === '1';
+  }
+
   /**
    * Check if an environment variable is allowed
    * @param {string} name - Environment variable name
@@ -229,7 +320,7 @@ class EnvironmentManager {
     const descriptions = {
       'I18NTK_LOG_LEVEL': 'Logging level (error, warn, info, debug, silent)',
       'I18NTK_OUTDIR': 'Output directory for reports and generated files',
-      'I18NTK_UI_LANGUAGE': 'UI language (en, de, es, fr, ru, ja, zh)',
+      'I18NTK_UI_LANGUAGE': 'UI language (en, de, es, fr, ru, ja, zh)',
       'I18NTK_SILENT': 'Run in silent mode without interactive prompts',
       'I18NTK_DEBUG_LOCALES': 'Enable debug logging for locale loading',
       'I18NTK_RUNTIME_DIR': 'Custom runtime directory path',
@@ -248,9 +339,9 @@ class EnvironmentManager {
 // Create singleton instance
 const envManager = new EnvironmentManager();
 
-module.exports = {
-  EnvironmentManager,
-  envManager,
+module.exports = {
+  EnvironmentManager,
+  envManager,
   ALLOWED_ENV_VARS,
   BLOCKED_PATTERNS
-};
+};

package/utils/i18n-helper.js

@@ -1,7 +1,8 @@
 // utils/i18n-helper.js
-const path = require('path');
-const fs = require('fs');
-const { logger } = require('./logger');
+const path = require('path');
+const fs = require('fs');
+const { logger } = require('./logger');
+const { envManager } = require('./env-manager');
 
 // Lazy load SecurityUtils to prevent circular dependencies
 let securityUtils;
@@ -101,15 +102,15 @@ function pkgUiLocalesDirViaThisFile() {
   return path.resolve(__dirname, '..', 'ui-locales');
 }
 
-function pkgUiLocalesDirViaResolve() {
-  try {
-    // Resolve using the current exported package path.
-    const enJson = require.resolve('i18ntk/ui-locales/en.json');
-    return path.dirname(enJson);
-  } catch {
-    return null;
-  }
-}
+function pkgUiLocalesDirViaResolve() {
+  try {
+    // Resolve using the current exported package path.
+    const enJson = require.resolve('i18ntk/ui-locales/en.json');
+    return path.dirname(enJson);
+  } catch {
+    return null;
+  }
+}
 
 // Removed legacyResourcesUiLocalesDir as resources/i18n/ui-locales is deprecated
 
@@ -162,7 +163,7 @@ function findLocaleFilesAllDirs(lang, preferredDir) {
   const SecurityUtils = getSecurityUtils();
   const dirs = resolveLocalesDirs(preferredDir);
 
-  if (process.env.I18NTK_DEBUG_LOCALES === '1') {
+  if (envManager.getBoolean('I18NTK_DEBUG_LOCALES')) {
     console.log('🔎 i18ntk locale search dirs:', dirs);
   }
 
@@ -194,28 +195,28 @@ function findLocaleFilesAllDirs(lang, preferredDir) {
     }
   }
 
-  if (process.env.I18NTK_DEBUG_LOCALES === '1' && errors.length > 0) {
+  if (envManager.getBoolean('I18NTK_DEBUG_LOCALES') && errors.length > 0) {
     console.warn(`⚠️ Locale resolution errors for ${lang}:`, errors);
   }
 
   return files;
 }
 
-let translations = {};
-let currentLanguage = 'en';
-let isInitialized = false;
-const missingKeyCache = new Map();
-const missingKeyTtlMs = 5 * 60 * 1000;
-
-function shouldReportMissingKey(key) {
-  const now = Date.now();
-  const expiresAt = missingKeyCache.get(key) || 0;
-  if (expiresAt > now) {
-    return false;
-  }
-  missingKeyCache.set(key, now + missingKeyTtlMs);
-  return true;
-}
+let translations = {};
+let currentLanguage = 'en';
+let isInitialized = false;
+const missingKeyCache = new Map();
+const missingKeyTtlMs = 5 * 60 * 1000;
+
+function shouldReportMissingKey(key) {
+  const now = Date.now();
+  const expiresAt = missingKeyCache.get(key) || 0;
+  if (expiresAt > now) {
+    return false;
+  }
+  missingKeyCache.set(key, now + missingKeyTtlMs);
+  return true;
+}
 
 function loadTranslations(language) {
   const cfg = safeRequireConfig();
@@ -243,7 +244,7 @@ function loadTranslations(language) {
         currentLanguage = lang;
         isInitialized = true;
 
-        if (process.env.I18NTK_DEBUG_LOCALES === '1') {
+        if (envManager.getBoolean('I18NTK_DEBUG_LOCALES')) {
           console.log(`🗂 Loaded UI locale → ${file} (${lang})`);
         }
 
@@ -255,7 +256,7 @@ function loadTranslations(language) {
         }
       } catch (e) {
         loadErrors.push({ file, error: e.message });
-        if (process.env.I18NTK_DEBUG_LOCALES === '1') {
+        if (envManager.getBoolean('I18NTK_DEBUG_LOCALES')) {
           console.warn(`⚠️ Failed to parse ${file}: ${e.message}`);
         }
       }
@@ -263,7 +264,7 @@ function loadTranslations(language) {
   }
 
   // Log comprehensive error summary if debugging
-  if (process.env.I18NTK_DEBUG_LOCALES === '1' && loadErrors.length > 0) {
+  if (envManager.getBoolean('I18NTK_DEBUG_LOCALES') && loadErrors.length > 0) {
     console.warn(`📊 Locale loading errors summary:`, {
       requested: requested,
       triedLanguages: tryOrder,
@@ -298,11 +299,11 @@ function loadTranslations(language) {
   currentLanguage = 'en';
   isInitialized = true;
 
-  if (loadErrors.length > 0) {
-    logger.warn('No valid UI locale files found. Using built-in English strings.', {
-      errorCount: loadErrors.length
-    });
-  }
+  if (loadErrors.length > 0) {
+    logger.warn('No valid UI locale files found. Using built-in English strings.', {
+      errorCount: loadErrors.length
+    });
+  }
 
   return translations;
 }
@@ -351,24 +352,24 @@ function t(key, params = {}) {
     }
   }
 
-  if (typeof value === 'undefined') {
-    if (shouldReportMissingKey(key)) {
-      logger.logMissingTranslationKey(key, 'Configuration error');
-    }
-    return key;
-  }
+  if (typeof value === 'undefined') {
+    if (shouldReportMissingKey(key)) {
+      logger.logMissingTranslationKey(key, 'Configuration error');
+    }
+    return key;
+  }
 
   // If we found a string, interpolate parameters
   if (typeof value === 'string') {
     return interpolateParams(value, params);
   }
 
-  // Return the key if the final value is not a string
-  if (shouldReportMissingKey(`${key}:non-string`)) {
-    logger.warn(`Translation key does not resolve to a string: ${key}`);
-  }
-  return key;
-}
+  // Return the key if the final value is not a string
+  if (shouldReportMissingKey(`${key}:non-string`)) {
+    logger.warn(`Translation key does not resolve to a string: ${key}`);
+  }
+  return key;
+}
 
 /**
  * Interpolate parameters into a translation string

package/utils/json-output.js

@@ -1,8 +1,9 @@
-const packageJson = require('../package.json');
-
-/**
- * JSON Output Utility for i18ntk commands
- * Provides consistent machine-readable output format for CI/CD integration
+const packageJson = require('../package.json');
+const { envManager } = require('./env-manager');
+
+/**
+ * JSON Output Utility for i18ntk commands
+ * Provides consistent machine-readable output format for CI/CD integration
  */
 
 class JsonOutput {
@@ -23,11 +24,11 @@ class JsonOutput {
     };
   }
 
-  getPackageVersion() {
-    try {
-      return packageJson.version;
-    } catch (error) {
-      return '1.8.3';
+  getPackageVersion() {
+    try {
+      return packageJson.version;
+    } catch (error) {
+      return '1.8.3';
     }
   }
 
@@ -75,7 +76,7 @@ class JsonOutput {
   output() {
     this.data.metadata.duration = Date.now() - this.startTime;
     
-    if (process.env.NODE_ENV !== 'test') {
+    if (envManager.get('NODE_ENV') !== 'test') {
       console.log(JSON.stringify(this.data, null, 2));
     }
     
@@ -97,4 +98,4 @@ class JsonOutput {
   }
 }
 
-module.exports = JsonOutput;
+module.exports = JsonOutput;

package/utils/logger.js

@@ -1,4 +1,5 @@
 const colors = require('./colors-new');
+const { envManager } = require('./env-manager');
 
 const LEVELS = { error: 0, warn: 1, info: 2, debug: 3 };
 const DEFAULT_INTERNAL_PREFIXES = ['[BUILD]', '[WORKERS]', '[I18N]', '[SECURITY]', '[SUCCESS]', '[WARN]', '[ERROR]', '[INFO]', '[DEBUG]'];
@@ -9,8 +10,8 @@ function asBoolean(value) {
 }
 
 function resolveLevel() {
-  const debugMode = asBoolean(process.env.DEBUG_MODE);
-  const configured = String(process.env.I18NTK_LOG_LEVEL || '').trim().toLowerCase();
+  const debugMode = envManager.getBoolean('DEBUG_MODE');
+  const configured = String(envManager.get('I18NTK_LOG_LEVEL') || '').trim().toLowerCase();
   if (configured && Object.prototype.hasOwnProperty.call(LEVELS, configured)) {
     return configured;
   }
@@ -20,7 +21,7 @@ function resolveLevel() {
   }
 
   // Silent-by-default in production-like builds.
-  if (process.env.NODE_ENV === 'production' || asBoolean(process.env.CI)) {
+  if (envManager.get('NODE_ENV') === 'production' || envManager.getBoolean('CI')) {
     return 'error';
   }
 
@@ -43,7 +44,7 @@ function normalizePrefix(prefix, fallback) {
 function write(level, message, options = {}) {
   if (!shouldLog(level)) return;
 
-  const jsonMode = asBoolean(process.env.JSON_LOG);
+  const jsonMode = envManager.getBoolean('JSON_LOG');
   const prefix = normalizePrefix(options.prefix, `[${level.toUpperCase()}]`);
   const details = options.details && typeof options.details === 'object' ? options.details : undefined;
   const text = String(message || '').trim();
@@ -116,7 +117,7 @@ function flushErrorContexts() {
 function emitErrorContextSummary({ force = false } = {}) {
   const contexts = Array.from(FIRST_ERROR_CONTEXT.entries());
   if (!contexts.length) return;
-  if (!force && !asBoolean(process.env.DEBUG_MODE)) return;
+  if (!force && !envManager.getBoolean('DEBUG_MODE')) return;
 
   for (const [key, context] of contexts) {
     write('debug', `First error context (${key})`, { prefix: '[ERROR]', details: context });
@@ -204,7 +205,7 @@ const logger = {
     write('info', String(message), { prefix: '[BUILD]', details });
   },
   isDebugMode() {
-    return asBoolean(process.env.DEBUG_MODE) || resolveLevel() === 'debug';
+    return envManager.getBoolean('DEBUG_MODE') || resolveLevel() === 'debug';
   },
   shouldLog,
   formatDuration,