i18ntk 1.10.2 → 2.0.2

package/main/i18ntk-manage.js

@@ -1,4 +1,4 @@
-#!/usr/bin/env node
+#!/usr/bin/env node
 /**
  * I18NTK MANAGEMENT TOOLKIT - MAIN MANAGER
  * 
@@ -18,11 +18,12 @@
  *   node i18ntk-manage.js --command=init
  */
 
-const fs = require('fs');
 const path = require('path');
 const UIi18n = require('./i18ntk-ui');
 const AdminAuth = require('../utils/admin-auth');
 const SecurityUtils = require('../utils/security');
+
+
 const AdminCLI = require('../utils/admin-cli');
 const configManager = require('../settings/settings-manager');
 const { showFrameworkWarningOnce } = require('../utils/cli-helper');
@@ -36,6 +37,8 @@ const SettingsCLI = require('../settings/settings-cli');
 // const I18nDebugger = require('../scripts/debug/debugger');
 const { createPrompt, isInteractive } = require('../utils/prompt-helper');
 const { loadTranslations, t, refreshLanguageFromSettings} = require('../utils/i18n-helper');
+// Preload translations early to avoid missing key warnings
+loadTranslations();
 const cliHelper = require('../utils/cli-helper');
 const { loadConfig, saveConfig, ensureConfigDefaults } = require('../utils/config');
 const pkg = require('../package.json');
@@ -50,10 +53,6 @@ async function runInitFlow() {
   return { i18nDir: settings.i18nDir, sourceDir: settings.sourceDir };
 }
 
-
-
-
-
 /**
  * Ensures the project is properly initialized or exits the process
  * @param {Object} prompt - Prompt interface for user interaction
@@ -129,7 +128,7 @@ async function ensureInitializedOrExit(prompt) {
       console.log('Framework detection prompt will be suppressed for this version.');
     } else if (action === 'detect') {
       // Run framework detection
-      const { detectedLanguage, detectedFramework } = detectEnvironmentAndFramework();
+      const { detectedLanguage, detectedFramework } = await detectEnvironmentAndFramework();
       
       if (detectedFramework && detectedFramework !== 'generic') {
         console.log(`\nDetected framework: ${detectedFramework}`);
@@ -156,7 +155,90 @@ async function ensureInitializedOrExit(prompt) {
   return { ...config, ...settings };
 }
 
+/**
+ * Custom glob implementation using Node.js built-in modules (zero dependencies)
+ * @param {string[]} patterns - Array of glob patterns
+ * @param {Object} options - Options object with cwd and ignore properties
+ * @returns {Promise<string[]>} Array of matching file paths
+ */
+async function customGlob(patterns, options = {}) {
+  const fs = require('fs');
+  const path = require('path');
+  const cwd = options.cwd || process.cwd();
+  const ignorePatterns = options.ignore || [];
+
+  function matchesPattern(filename, pattern) {
+    // Simple pattern matching for **/*.{js,jsx,ts,tsx} style patterns
+    if (pattern.includes('**/*')) {
+      const extensionPart = pattern.split('*.')[1];
+      if (extensionPart) {
+        const extensions = extensionPart.replace('{', '').replace('}', '').split(',');
+        return extensions.some(ext => filename.endsWith('.' + ext.trim()));
+      }
+    }
+    return filename.includes(pattern.replace('**/', ''));
+  }
+
+  function shouldIgnore(filePath) {
+    return ignorePatterns.some(pattern => {
+      if (pattern.includes('**/')) {
+        const patternEnd = pattern.replace('**/', '');
+        return filePath.includes('/' + patternEnd) || filePath.includes('\\' + patternEnd);
+      }
+      return filePath.includes(pattern);
+    });
+  }
+
+  function findFiles(dir, results = []) {
+    try {
+      const items = fs.readdirSync(dir);
+
+      for (const item of items) {
+        const fullPath = path.join(dir, item);
+        const relativePath = path.relative(cwd, fullPath);
+
+        if (shouldIgnore(relativePath)) {
+          continue;
+        }
+
+        try {
+          const stat = fs.statSync(fullPath);
+
+          if (stat.isDirectory()) {
+            findFiles(fullPath, results);
+          } else if (stat.isFile()) {
+            // Check if file matches any of our patterns
+            for (const pattern of patterns) {
+              if (matchesPattern(item, pattern)) {
+                results.push(relativePath);
+                break;
+              }
+            }
+          }
+        } catch (error) {
+          // Skip files we can't access
+          continue;
+        }
+      }
+    } catch (error) {
+      // Skip directories we can't access
+    }
+
+    return results;
+  }
+
+  return findFiles(cwd);
+}
+
 async function detectEnvironmentAndFramework() {
+  // Defensive check to ensure SecurityUtils is available
+  if (!SecurityUtils) {
+    throw new Error('SecurityUtils is not available. This may indicate a module loading issue.');
+  }
+  const fs = require('fs');
+  const path = require('path');
+  
+  
   const packageJsonPath = path.join(process.cwd(), 'package.json');
   const pyprojectPath = path.join(process.cwd(), 'pyproject.toml');
   const requirementsPath = path.join(process.cwd(), 'requirements.txt');
@@ -167,10 +249,10 @@ async function detectEnvironmentAndFramework() {
   let detectedLanguage = 'generic';
   let detectedFramework = 'generic';
 
-  if (fs.existsSync(packageJsonPath)) {
+  if (SecurityUtils.safeExistsSync(packageJsonPath)) {
     detectedLanguage = 'javascript';
     try {
-      const packageJson = JSON.parse(fs.readFileSync(packageJsonPath, 'utf8'));
+      const packageJson = JSON.parse(SecurityUtils.safeReadFileSync(packageJsonPath, path.dirname(packageJsonPath), 'utf8'));
       const deps = { 
         ...(packageJson.dependencies || {}), 
         ...(packageJson.devDependencies || {}),
@@ -189,7 +271,7 @@ async function detectEnvironmentAndFramework() {
           /require\(['\"]i18ntk[\/\\]runtime['\"]\)/
         ];
         
-        const sourceFiles = await glob(['src/**/*.{js,jsx,ts,tsx}'], {
+        const sourceFiles = await customGlob(['src/**/*.{js,jsx,ts,tsx}'], {
           cwd: process.cwd(),
           ignore: ['**/node_modules/**', '**/dist/**', '**/build/**']
         });
@@ -223,11 +305,11 @@ async function detectEnvironmentAndFramework() {
     } catch (error) {
       detectedFramework = 'generic';
     }
-  } else if (fs.existsSync(pyprojectPath) || fs.existsSync(requirementsPath)) {
+  } else if (SecurityUtils.safeExistsSync(pyprojectPath) || SecurityUtils.safeExistsSync(requirementsPath)) {
     detectedLanguage = 'python';
     try {
-      if (fs.existsSync(requirementsPath)) {
-        const requirements = fs.readFileSync(requirementsPath, 'utf8');
+      if (SecurityUtils.safeExistsSync(requirementsPath)) {
+        const requirements = SecurityUtils.safeReadFileSync(requirementsPath, path.dirname(requirementsPath), 'utf8');
         if (requirements.includes('django')) detectedFramework = 'django';
         else if (requirements.includes('flask')) detectedFramework = 'flask';
         else if (requirements.includes('fastapi')) detectedFramework = 'fastapi';
@@ -236,13 +318,13 @@ async function detectEnvironmentAndFramework() {
     } catch (error) {
       detectedFramework = 'generic';
     }
-  } else if (fs.existsSync(goModPath)) {
+  } else if (SecurityUtils.safeExistsSync(goModPath)) {
     detectedLanguage = 'go';
     detectedFramework = 'generic';
-  } else if (fs.existsSync(pomPath)) {
+  } else if (SecurityUtils.safeExistsSync(pomPath)) {
     detectedLanguage = 'java';
     try {
-      const pomContent = fs.readFileSync(pomPath, 'utf8');
+      const pomContent = SecurityUtils.safeReadFileSync(pomPath, path.dirname(pomPath), 'utf8');
       if (pomContent.includes('spring-boot')) detectedFramework = 'spring-boot';
       else if (pomContent.includes('spring')) detectedFramework = 'spring';
       else if (pomContent.includes('quarkus')) detectedFramework = 'quarkus';
@@ -250,10 +332,10 @@ async function detectEnvironmentAndFramework() {
     } catch (error) {
       detectedFramework = 'generic';
     }
-  } else if (fs.existsSync(composerPath)) {
+  } else if (SecurityUtils.safeExistsSync(composerPath)) {
     detectedLanguage = 'php';
     try {
-      const composer = JSON.parse(fs.readFileSync(composerPath, 'utf8'));
+      const composer = JSON.parse(SecurityUtils.safeReadFileSync(composerPath, path.dirname(composerPath), 'utf8'));
       const deps = composer.require || {};
       
       if (deps['laravel/framework']) detectedFramework = 'laravel';
@@ -461,7 +543,7 @@ class I18nManager {
     // Only auto-detect if no settings are configured
     for (const possiblePath of possibleI18nPaths) {
       const resolvedPath = path.resolve(projectRoot, possiblePath);
-      if (fs.existsSync(resolvedPath)) {
+      if (SecurityUtils.safeExistsSync(resolvedPath)) {
         // Check if it contains language directories
         try {
           const items = fs.readdirSync(resolvedPath);
@@ -487,13 +569,13 @@ class I18nManager {
   async checkI18nDependencies() {
     const packageJsonPath = path.resolve('./package.json');
     
-    if (!fs.existsSync(packageJsonPath)) {
+    if (!SecurityUtils.safeExistsSync(packageJsonPath)) {
       console.log(this.ui ? this.ui.t('errors.noPackageJson') : 'No package.json found');
       return false; // Treat as no framework detected
     }
     
     try {
-      const packageJson = JSON.parse(fs.readFileSync(packageJsonPath, 'utf8'));
+      const packageJson = JSON.parse(SecurityUtils.safeReadFileSync(packageJsonPath, path.dirname(packageJsonPath), 'utf8'));
       // Include peerDependencies in the check
       const dependencies = { 
         ...packageJson.dependencies, 
@@ -610,12 +692,29 @@ class I18nManager {
 
   // Add this run method after the checkI18nDependencies method
   async run() {
+    // Add timeout to prevent hanging
+    const args = this.parseArgs();
+
+    const timeout = setTimeout(() => {
+      console.error('❌ CLI startup timeout - something is hanging');
+      if (args.debug) {
+        console.error('🔍 DEBUG: Last known execution point reached');
+      }
+      process.exit(1);
+    }, 10000); // 10 second timeout
+
+    if (args.debug) {
+      console.log('🔍 DEBUG: Starting i18ntk-manage.js...');
+      console.log('🔍 DEBUG: Process.argv:', process.argv);
+      console.log('🔍 DEBUG: Parsed args:', args);
+      console.log('🔍 DEBUG: About to call SetupEnforcer.checkSetupCompleteAsync()');
+    }
+
     let prompt;
     try {
       // Ensure setup is complete before running any operations
       await SetupEnforcer.checkSetupCompleteAsync();
-      
-      const args = this.parseArgs();
+
       prompt = createPrompt({ noPrompt: args.noPrompt || Boolean(args.adminPin) });
       const interactive = isInteractive({ noPrompt: args.noPrompt || Boolean(args.adminPin) });
 
@@ -1186,7 +1285,7 @@ class I18nManager {
   console.log(t('debug.runningDebugTool', { displayName }));
     try {
       const toolPath = path.join(__dirname, '..', 'scripts', 'debug', toolName);
-      if (fs.existsSync(toolPath)) {
+      if (SecurityUtils.safeExistsSync(toolPath)) {
         console.log(`Debug tool available: ${toolName}`);
         console.log(`To run this tool manually: node "${toolPath}"`);
         console.log(`Working directory: ${path.join(__dirname, '..')}`);
@@ -1209,7 +1308,7 @@ class I18nManager {
     
     try {
       const logsDir = path.join(__dirname, '..', 'scripts', 'debug', 'logs');
-      if (fs.existsSync(logsDir)) {
+      if (SecurityUtils.safeExistsSync(logsDir)) {
         const files = fs.readdirSync(logsDir)
           .filter(file => file.endsWith('.log') || file.endsWith('.txt'))
           .sort((a, b) => {
@@ -1230,7 +1329,7 @@ class I18nManager {
           const fileIndex = parseInt(choice) - 1;
           
           if (fileIndex >= 0 && fileIndex < files.length) {
-            const logContent = fs.readFileSync(path.join(logsDir, files[fileIndex]), 'utf8');
+            const logContent = SecurityUtils.safeReadFileSync(path.join(logsDir, files[fileIndex]), logsDir, 'utf8');
             console.log(`\n${t('debug.contentOf', { filename: files[fileIndex] })}:`);
             console.log('============================================================');
             console.log(logContent.slice(-2000)); // Show last 2000 characters
@@ -1290,7 +1389,7 @@ class I18nManager {
       
       // Check which directories exist and have files
       for (const dir of targetDirs) {
-        if (fs.existsSync(dir.path)) {
+        if (SecurityUtils.safeExistsSync(dir.path)) {
           const files = this.getAllReportFiles(dir.path);
           if (files.length > 0) {
             availableDirs.push({
@@ -1429,7 +1528,7 @@ class I18nManager {
     let files = [];
     
     try {
-      if (!fs.existsSync(dir)) {
+      if (!SecurityUtils.safeExistsSync(dir)) {
         return [];
       }
       
@@ -1561,7 +1660,7 @@ if (require.main === module) {
   if (args.includes('--version') || args.includes('-v')) {
     try {
       const packageJsonPath = path.resolve(__dirname, '../package.json');
-      const packageJson = JSON.parse(fs.readFileSync(packageJsonPath, 'utf8'));
+      const packageJson = JSON.parse(SecurityUtils.safeReadFileSync(packageJsonPath, path.dirname(packageJsonPath), 'utf8'));
       const versionInfo = packageJson.versionInfo || {};
       
       console.log(`\n🌍 i18n Toolkit (i18ntk)`);
@@ -1592,4 +1691,4 @@ if (require.main === module) {
   manager.run();
 }
 
-module.exports = I18nManager;
+module.exports = I18nManager;