i18ntk 1.10.2 → 2.0.2

package/main/i18ntk-scanner.js

@@ -1,4 +1,5 @@
 #!/usr/bin/env node
+
 /**
  * I18NTK TEXT SCANNER
  *
@@ -32,7 +33,7 @@ const SetupEnforcer = require('../utils/setup-enforcer');
   }
 })();
 
-loadTranslations(process.env.I18NTK_LANG);
+loadTranslations();
 
 class I18nTextScanner {
   constructor(config = {}) {
@@ -46,12 +47,12 @@ class I18nTextScanner {
   }
 
   loadLocale() {
-    const uiLocalesDir = path.join(__dirname, '..', 'ui-locales');
+    const uiLocalesDir = path.join(__dirname, '..', 'resources', 'i18n', 'ui-locales');
     const localeFile = path.join(uiLocalesDir, 'en.json');
-    
+
     try {
-      const localeContent = fs.readFileSync(localeFile, 'utf8');
-      return JSON.parse(localeContent);
+      const localeContent = SecurityUtils.safeReadFileSync(localeFile, uiLocalesDir, 'utf8');
+      return SecurityUtils.safeParseJSON(localeContent);
     } catch (error) {
       return {
         scanner: {
@@ -89,16 +90,16 @@ class I18nTextScanner {
     const keyStr = String(key || '');
     const keys = keyStr.split('.');
     let value = this.locale;
-    
+
     for (const k of keys) {
       value = value?.[k];
       if (value === undefined) break;
     }
-    
+
     if (typeof value !== 'string') {
       return key;
     }
-    
+
     return value.replace(/\{([^}]+)\}/g, (match, param) => {
       return params[param] !== undefined ? params[param] : match;
     });
@@ -107,12 +108,12 @@ class I18nTextScanner {
   parseArgs() {
     const args = process.argv.slice(2);
     const parsed = {};
-    
+
     args.forEach(arg => {
       if (arg.startsWith('--')) {
         const [key, ...valueParts] = arg.substring(2).split('=');
         const value = valueParts.join('=');
-        
+
         switch (key) {
           case 'source-dir':
             parsed.sourceDir = value || '';
@@ -148,38 +149,38 @@ class I18nTextScanner {
         }
       }
     });
-    
+
     return parsed;
   }
 
   detectFramework(projectRoot) {
     const packagePath = path.join(projectRoot, 'package.json');
-    
+
     // Check for Python frameworks
     const requirementsPath = path.join(projectRoot, 'requirements.txt');
     const setupPath = path.join(projectRoot, 'setup.py');
     const pyprojectPath = path.join(projectRoot, 'pyproject.toml');
-    
+
     try {
       // Check Python frameworks first
-      if (fs.existsSync(requirementsPath)) {
-        const requirements = fs.readFileSync(requirementsPath, 'utf8');
+      if (SecurityUtils.safeExistsSync(requirementsPath, projectRoot)) {
+        const requirements = SecurityUtils.safeReadFileSync(requirementsPath, projectRoot, 'utf8');
         if (requirements.includes('Django')) return 'django';
         if (requirements.includes('Flask') || requirements.includes('flask-babel')) return 'flask';
       }
-      
-      if (fs.existsSync(setupPath)) {
-        const setup = fs.readFileSync(setupPath, 'utf8');
+
+      if (SecurityUtils.safeExistsSync(setupPath, projectRoot)) {
+        const setup = SecurityUtils.safeReadFileSync(setupPath, projectRoot, 'utf8');
         if (setup.includes('Django')) return 'django';
         if (setup.includes('Flask')) return 'flask';
       }
-      
-      if (fs.existsSync(pyprojectPath)) {
-        const pyproject = fs.readFileSync(pyprojectPath, 'utf8');
+
+      if (SecurityUtils.safeExistsSync(pyprojectPath, projectRoot)) {
+        const pyproject = SecurityUtils.safeReadFileSync(pyprojectPath, projectRoot, 'utf8');
         if (pyproject.includes('Django')) return 'django';
         if (pyproject.includes('Flask')) return 'flask';
       }
-      
+
       // Check for Python files
       const hasPythonFiles = fs.readdirSync(projectRoot, { recursive: true })
         .some(file => file.endsWith && file.endsWith('.py'));
@@ -187,17 +188,18 @@ class I18nTextScanner {
     } catch (error) {
       // Continue to JS frameworks
     }
-    
+
     try {
-      const packageJson = JSON.parse(fs.readFileSync(packagePath, 'utf8'));
+      const packageJsonContent = SecurityUtils.safeReadFileSync(packagePath, projectRoot, 'utf8');
+      const packageJson = SecurityUtils.safeParseJSON(packageJsonContent);
       const deps = { ...packageJson.dependencies, ...packageJson.devDependencies };
-      
+
       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';
-      
+
       return 'vanilla';
     } catch (error) {
       return 'vanilla';
@@ -294,24 +296,24 @@ class I18nTextScanner {
     // Enhanced text detection for Unicode and multilingual support
     const trimmed = text.trim();
     if (trimmed.length < 3) return false;
-    
+
     // Skip if it's just numbers or special characters
     if (/^\d+$/.test(trimmed)) return false;
     if (/^[!@#$%^&*()_+\-=\[\]{};':"\\|,.<>?]+$/.test(trimmed)) return false;
-    
+
     // Allow Unicode characters including CJK, Cyrillic, etc.
     const validChars = trimmed.match(/[\p{L}\p{N}\s\-,.!?':"()\[\]{}]/gu) || [];
     const validRatio = validChars.length / trimmed.length;
-    
+
     // Must have at least 50% valid characters and some alphabetic characters
     const hasAlpha = /[a-zA-Z\u00C0-\u024F\u1E00-\u1EFF\u0400-\u04FF\u4E00-\u9FFF\uAC00-\uD7AF]/u.test(trimmed);
-    
+
     return validRatio >= 0.5 && hasAlpha;
   }
 
   scanFile(filePath, patterns, minLength, maxLength) {
     try {
-      const content = fs.readFileSync(filePath, 'utf8');
+      const content = SecurityUtils.safeReadFileSync(filePath, path.dirname(filePath), 'utf8');
       const lines = content.split('\n');
       const results = [];
 
@@ -319,20 +321,20 @@ class I18nTextScanner {
         let match;
         while ((match = pattern.exec(content)) !== null) {
           const text = match[1] || match[0];
-          
+
           // Skip translation function calls
           const beforeMatch = content.substring(Math.max(0, match.index - 20), match.index);
-          if (beforeMatch.includes('t(') || beforeMatch.includes('i18next.t(') || 
+          if (beforeMatch.includes('t(') || beforeMatch.includes('i18next.t(') ||
               beforeMatch.includes('$t(') || beforeMatch.includes('translate(')) {
             continue;
           }
-          
-          if (text && this.isEnglishText(text) && 
+
+          if (text && this.isEnglishText(text) &&
               text.length >= minLength && text.length <= maxLength) {
-            
+
             const lineNumber = content.substring(0, match.index).split('\n').length;
             const lineContent = lines[lineNumber - 1] || '';
-            
+
             results.push({
               text: text.trim(),
               line: lineNumber,
@@ -357,7 +359,7 @@ class I18nTextScanner {
       .replace(/[^a-z0-9\s]/g, '')
       .replace(/\s+/g, '_')
       .substring(0, 50);
-    
+
     return {
       key: `ui.${key}`,
       original: text,
@@ -402,28 +404,28 @@ class I18nTextScanner {
   }
 
   async scanDirectory(dir, options = {}) {
-    const { 
-      patterns = [], 
-      exclusions = [], 
-      minLength = 3, 
+    const {
+      patterns = [],
+      exclusions = [],
+      minLength = 3,
       maxLength = 100,
-      includeTests = false 
+      includeTests = false
     } = options;
 
-    if (!fs.existsSync(dir)) {
+    if (!SecurityUtils.safeExistsSync(dir, path.dirname(dir))) {
       throw new Error(`Directory does not exist: ${dir}`);
     }
 
     const allResults = [];
     const extensions = ['.js', '.jsx', '.ts', '.tsx', '.vue', '.html', '.svelte', '.py', '.pyx', '.pyi'];
-    
+
     const scanRecursive = (currentDir) => {
       const items = fs.readdirSync(currentDir);
-      
+
       for (const item of items) {
         const fullPath = path.join(currentDir, item);
         const stat = fs.statSync(fullPath);
-        
+
         if (stat.isDirectory()) {
           if (!item.startsWith('.') && !this.shouldExcludeFile(fullPath, exclusions)) {
             scanRecursive(fullPath);
@@ -434,7 +436,7 @@ class I18nTextScanner {
             if (!includeTests && (item.includes('.test.') || item.includes('.spec.'))) {
               continue;
             }
-            
+
             const results = this.scanFile(fullPath, patterns, minLength, maxLength);
             if (results.length > 0) {
               allResults.push({
@@ -452,7 +454,7 @@ class I18nTextScanner {
   }
 
   async generateReport(results, outputDir) {
-    if (!fs.existsSync(outputDir)) {
+    if (!SecurityUtils.safeExistsSync(outputDir, path.dirname(outputDir))) {
       fs.mkdirSync(outputDir, { recursive: true });
     }
 
@@ -470,11 +472,11 @@ class I18nTextScanner {
     };
 
     // JSON report
-    fs.writeFileSync(reportFile, JSON.stringify(summary, null, 2));
+    SecurityUtils.safeWriteFileSync(reportFile, JSON.stringify(summary, null, 2), outputDir);
 
     // Markdown summary
     const mdContent = this.generateMarkdownReport(summary);
-    fs.writeFileSync(summaryFile, mdContent);
+    SecurityUtils.safeWriteFileSync(summaryFile, mdContent, outputDir);
 
     return { reportFile, summaryFile, summary };
   }
@@ -482,10 +484,10 @@ class I18nTextScanner {
   generateMarkdownReport(summary) {
     let content = `# Text Analysis Report
 
-**Framework:** ${summary.framework}  
-**Total Files Scanned:** ${summary.totalFiles}  
-**Text Instances Found:** ${summary.totalInstances}  
-**Files with Hardcoded Text:** ${summary.filesWithText}  
+**Framework:** ${summary.framework}
+**Total Files Scanned:** ${summary.totalFiles}
+**Text Instances Found:** ${summary.totalInstances}
+**Files with Hardcoded Text:** ${summary.filesWithText}
 **Generated:** ${summary.timestamp}
 
 ## Summary
@@ -552,7 +554,7 @@ class I18nTextScanner {
 
     const baseConfig = await getUnifiedConfig('scanner', args);
     this.config = { ...baseConfig, ...(this.config || {}) };
-    
+
     this.sourceDir = this.config.sourceDir || './src';
 
     // Resolve framework with precedence: CLI arg > config.framework.preference|string > auto-detect > fallback
@@ -574,7 +576,7 @@ class I18nTextScanner {
     }
 
     // Validate source directory
-    if (!fs.existsSync(this.sourceDir)) {
+    if (!SecurityUtils.safeExistsSync(this.sourceDir, path.dirname(this.sourceDir))) {
       console.error(`❌ Source directory does not exist: ${this.sourceDir}`);
       process.exit(1);
     }