i18ntk 2.2.0 → 2.3.1

package/main/manage/index.js

@@ -13,16 +13,21 @@
  *   npm run i18ntk:manage -- --help
  */
 
-const path = require('path');
-const AdminAuth = require('../../utils/admin-auth');
-const SecurityUtils = require('../../utils/security');
-const configManager = require('../../settings/settings-manager');
-const { showFrameworkWarningOnce } = require('../../utils/cli-helper');
-const { createPrompt, isInteractive } = require('../../utils/prompt-helper');
-const { loadTranslations, t, refreshLanguageFromSettings} = require('../../utils/i18n-helper');
-const cliHelper = require('../../utils/cli-helper');
-const { loadConfig, saveConfig, ensureConfigDefaults } = require('../../utils/config');
-const pkg = require('../../package.json');
+const path = require('path');
+const fs = require('fs');
+const AdminAuth = require('../../utils/admin-auth');
+const SecurityUtils = require('../../utils/security');
+const configManager = require('../../settings/settings-manager');
+const { validateSourceDir } = require('../../utils/config-helper');
+const { checkInitialized } = require('../../utils/init-helper');
+const { showFrameworkWarningOnce } = require('../../utils/cli-helper');
+const { createPrompt, isInteractive } = require('../../utils/prompt-helper');
+const { loadTranslations, t, refreshLanguageFromSettings} = require('../../utils/i18n-helper');
+const cliHelper = require('../../utils/cli-helper');
+const { blue } = require('../../utils/colors-new');
+const { loadConfig, saveConfig, ensureConfigDefaults } = require('../../utils/config');
+const SettingsCLI = require('../../settings/settings-cli');
+const pkg = require('../../package.json');
 const SetupEnforcer = require('../../utils/setup-enforcer');
 const CommandRouter = require('./commands/CommandRouter');
 
@@ -73,9 +78,8 @@ class I18nManager {
             loadTranslations(uiLanguage);
 
             // Validate source directory exists
-            const {validateSourceDir, displayPaths} = require('../../utils/config-helper');
-            try {
-                validateSourceDir(this.config.sourceDir, 'i18ntk-manage');
+            try {
+                validateSourceDir(this.config.sourceDir, 'i18ntk-manage');
             } catch (err) {
                 console.log(t('init.requiredTitle'));
                 console.log(t('init.requiredBody'));
@@ -132,10 +136,10 @@ class I18nManager {
             if (SecurityUtils.safeExistsSync(resolvedPath)) {
                 // Check if it contains language directories
                 try {
-                    const items = require('fs').readdirSync(resolvedPath);
+                    const items = fs.readdirSync(resolvedPath);
                     const hasLanguageDirs = items.some(item => {
                         const itemPath = path.join(resolvedPath, item);
-                        return require('fs').statSync(itemPath).isDirectory() &&
+                        return fs.statSync(itemPath).isDirectory() &&
                                ['en', 'de', 'es', 'fr', 'ru', 'ja', 'zh'].includes(item);
                     });
 
@@ -170,7 +174,6 @@ class I18nManager {
         };
 
         try {
-            const { checkInitialized } = require('../../utils/init-helper');
             const initStatus = await checkInitialized();
             if (initStatus?.initialized || initStatus?.config?.setup?.completed) {
                 markInternalFrameworkDetected();
@@ -335,8 +338,8 @@ class I18nManager {
             }
             clearStartupTimeout();
 
-            prompt = createPrompt({ noPrompt: args.noPrompt || Boolean(args.adminPin) });
-            const interactive = isInteractive({ noPrompt: args.noPrompt || Boolean(args.adminPin) });
+            prompt = createPrompt({ noPrompt: args.noPrompt });
+            const interactive = isInteractive({ noPrompt: args.noPrompt });
 
             // Load settings and UI language
             const settings = configManager.loadSettings ? configManager.loadSettings() : (configManager.getConfig ? configManager.getConfig() : {});
@@ -346,15 +349,10 @@ class I18nManager {
             // Initialize CommandRouter
             this.commandRouter = new CommandRouter(this.config, this.ui, this.adminAuth);
 
-            if (args.adminPin) {
-                this.adminAuth.verifyPin = async () => true;
-                this.prompt = async () => '';
-            }
-
-            if (args.help) {
-                this.showHelp();
-                return;
-            }
+            if (args.help) {
+                this.showHelp();
+                return;
+            }
 
             let cfgAfterInitCheck = {};
             if (interactive && !shouldSkipInitCheck) {
@@ -366,7 +364,7 @@ class I18nManager {
             }
 
             this.config = { ...this.config, ...cfgAfterInitCheck };
-            await this.initialize({ interactive, noPrompt: args.noPrompt || Boolean(args.adminPin) });
+            await this.initialize({ interactive, noPrompt: args.noPrompt });
             clearStartupTimeout();
             let commandToExecute = requestedCommand;
 
@@ -380,8 +378,7 @@ class I18nManager {
             // Handle debug flag
             if (args.debug) {
                 // Enable debug mode for this session
-                const { blue } = require('../../utils/colors-new');
-                console.log(blue('Debug mode enabled'));
+                console.log(blue('Debug mode enabled'));
             }
 
             if (commandToExecute) {
@@ -489,11 +486,7 @@ class I18nManager {
     // ... existing code for ensureInitializedOrExit, maybePromptFramework, showInteractiveMenu, etc. ...
 
     async ensureInitializedOrExit(prompt) {
-        const { checkInitialized } = require('../../utils/init-helper');
-        const cliHelper = require('../../utils/cli-helper');
-        const pkg = require('../../package.json');
-
-        const { initialized, config } = await checkInitialized();
+        const { initialized, config } = await checkInitialized();
 
         if (!initialized) {
             console.log('\nThis project is not yet initialized with i18ntk.');
@@ -608,10 +601,7 @@ class I18nManager {
         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 packageJsonPath = path.join(process.cwd(), 'package.json');
         const pyprojectPath = path.join(process.cwd(), 'pyproject.toml');
         const requirementsPath = path.join(process.cwd(), 'requirements.txt');
         const goModPath = path.join(process.cwd(), 'go.mod');
@@ -622,7 +612,6 @@ class I18nManager {
         let detectedFramework = 'generic';
 
         try {
-            const { checkInitialized } = require('../../utils/init-helper');
             const initStatus = await checkInitialized();
             if (initStatus?.initialized || initStatus?.config?.setup?.completed) {
                 return { detectedLanguage: 'javascript', detectedFramework: 'i18ntk' };
@@ -738,9 +727,7 @@ class I18nManager {
     }
 
     async customGlob(patterns, options = {}) {
-        const fs = require('fs');
-        const path = require('path');
-        const cwd = options.cwd || process.cwd();
+        const cwd = options.cwd || process.cwd();
         const ignorePatterns = options.ignore || [];
 
         function matchesPattern(filename, pattern) {
@@ -944,8 +931,7 @@ class I18nManager {
                 const authRequired = await this.adminAuth.isAuthRequiredForScript('summaryReports');
                 if (authRequired) {
                     console.log(`\n${t('adminCli.protectedAccess')}`);
-                    const cliHelper = require('../../utils/cli-helper');
-                    const pin = await cliHelper.promptPin(t('adminCli.enterPin') + ': ');
+                    const pin = await cliHelper.promptPin(t('adminCli.enterPin') + ': ');
                     const isValid = await this.adminAuth.verifyPin(pin);
 
                     if (!isValid) {
@@ -1061,8 +1047,7 @@ class I18nManager {
             console.log(t('language.changed', { language: selectedLang.name }));
 
             // Force reload translations for the entire system
-            const { loadTranslations } = require('../../utils/i18n-helper');
-            loadTranslations(selectedLang.code);
+            loadTranslations(selectedLang.code);
 
             // Return to main menu with new language
             await this.prompt('\n' + t('language.pressEnterToContinue'));
@@ -1079,8 +1064,7 @@ class I18nManager {
         const authRequired = await this.adminAuth.isAuthRequiredForScript('debugMenu');
         if (authRequired) {
             console.log(`\n${t('adminPin.protectedAccess')}`);
-            const cliHelper = require('../../utils/cli-helper');
-            const pin = await cliHelper.promptPin(t('adminPin.enterPin') + ': ');
+            const pin = await cliHelper.promptPin(t('adminPin.enterPin') + ': ');
             const isValid = await this.adminAuth.verifyPin(pin);
 
             if (!isValid) {
@@ -1140,35 +1124,46 @@ class I18nManager {
         console.log(`\n${t('debug.recentDebugLogs')}`);
         console.log('============================================================');
 
-        try {
-            const logsDir = path.join(__dirname, '..', '..', 'scripts', 'debug', 'logs');
-            if (SecurityUtils.safeExistsSync(logsDir)) {
-                const files = require('fs').readdirSync(logsDir)
-                    .filter(file => file.endsWith('.log') || file.endsWith('.txt'))
-                    .sort((a, b) => {
-                        const statA = require('fs').statSync(path.join(logsDir, a));
-                        const statB = require('fs').statSync(path.join(logsDir, b));
-                        return statB.mtime - statA.mtime;
-                    })
-                    .slice(0, 5);
-
-                if (files.length > 0) {
-                    files.forEach((file, index) => {
-                        const filePath = path.join(logsDir, file);
-                        const stats = require('fs').statSync(filePath);
-                        console.log(`${index + 1}. ${file} (${stats.mtime.toLocaleString()})`);
-                    });
-
-                    const choice = await this.prompt('\n' + t('debug.selectLogPrompt', { count: files.length }));
-                    const fileIndex = parseInt(choice) - 1;
-
-                    if (fileIndex >= 0 && fileIndex < files.length) {
-                        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
-                        console.log('============================================================');
-                    }
+        try {
+            const logsDir = path.join(__dirname, '..', '..', 'scripts', 'debug', 'logs');
+            const validatedLogsDir = SecurityUtils.validatePath(logsDir, process.cwd());
+            if (validatedLogsDir && SecurityUtils.safeExistsSync(validatedLogsDir, process.cwd())) {
+                const files = fs.readdirSync(validatedLogsDir)
+                    .filter(file => file.endsWith('.log') || file.endsWith('.txt'))
+                    .sort((a, b) => {
+                        const statA = fs.statSync(path.join(validatedLogsDir, a));
+                        const statB = fs.statSync(path.join(validatedLogsDir, b));
+                        return statB.mtime - statA.mtime;
+                    })
+                    .slice(0, 5);
+
+                if (files.length > 0) {
+                    files.forEach((file, index) => {
+                        const filePath = path.join(validatedLogsDir, file);
+                        const validatedFilePath = SecurityUtils.validatePath(filePath, validatedLogsDir);
+                        if (!validatedFilePath) {
+                            return;
+                        }
+                        const stats = fs.statSync(validatedFilePath);
+                        console.log(`${index + 1}. ${file} (${stats.mtime.toLocaleString()})`);
+                    });
+
+                    const choice = await this.prompt('\n' + t('debug.selectLogPrompt', { count: files.length }));
+                    const fileIndex = parseInt(choice) - 1;
+
+                    if (fileIndex >= 0 && fileIndex < files.length) {
+                        const logPath = SecurityUtils.validatePath(path.join(validatedLogsDir, files[fileIndex]), validatedLogsDir);
+                        if (!logPath) {
+                            console.log(t('debug.debugLogsDirectoryNotFound'));
+                            return;
+                        }
+                        const logContent = SecurityUtils.safeReadFileSync(logPath, validatedLogsDir, 'utf8');
+                        console.log(`\n${t('debug.contentOf', { filename: files[fileIndex] })}:`);
+                        console.log('============================================================');
+                        const tailLines = String(logContent || '').split(/\r?\n/).slice(-40).join('\n');
+                        console.log(tailLines);
+                        console.log('============================================================');
+                    }
                 } else {
                     console.log(t('debug.noDebugLogsFound'));
                 }
@@ -1188,8 +1183,7 @@ class I18nManager {
         const authRequired = await this.adminAuth.isAuthRequiredForScript('deleteReports');
         if (authRequired) {
             console.log(`\n${t('adminPin.protectedAccess')}`);
-            const cliHelper = require('../../utils/cli-helper');
-            const pin = await cliHelper.promptPin(t('adminPin.enterPin') + ': ');
+            const pin = await cliHelper.promptPin(t('adminPin.enterPin') + ': ');
             const isValid = await this.adminAuth.verifyPin(pin);
 
             if (!isValid) {
@@ -1215,22 +1209,25 @@ class I18nManager {
             { path: path.join(process.cwd(), 'utils', 'i18ntk-reports'), name: 'Utils Reports', type: 'reports' }
         ].filter(dir => dir.path && typeof dir.path === 'string');
 
-        try {
-            console.log(t('operations.scanningForFiles'));
-
-            let availableDirs = [];
-
-            // Check which directories exist and have files
-            for (const dir of targetDirs) {
-                if (SecurityUtils.safeExistsSync(dir.path)) {
-                    const files = this.getAllReportFiles(dir.path);
-                    if (files.length > 0) {
-                        availableDirs.push({
-                            ...dir,
-                            files: files.map(file => ({ path: file, dir: dir.path })),
-                            count: files.length
-                        });
-                    }
+        try {
+            console.log(t('operations.scanningForFiles'));
+
+            let availableDirs = [];
+            const projectRoot = process.cwd();
+
+            // Check which directories exist and have files
+            for (const dir of targetDirs) {
+                const validatedDirPath = SecurityUtils.validatePath(dir.path, projectRoot);
+                if (validatedDirPath && SecurityUtils.safeExistsSync(validatedDirPath, projectRoot)) {
+                    const files = this.getAllReportFiles(validatedDirPath, validatedDirPath);
+                    if (files.length > 0) {
+                        availableDirs.push({
+                            ...dir,
+                            path: validatedDirPath,
+                            files: files.map(file => ({ path: file, dir: validatedDirPath })),
+                            count: files.length
+                        });
+                    }
                 }
             }
 
@@ -1329,15 +1326,20 @@ class I18nManager {
             if (confirm.toLowerCase() === 'y' || confirm.toLowerCase() === 'yes') {
                 let deletedCount = 0;
 
-                for (const fileInfo of filesToDelete) {
-                    try {
-                        require('fs').unlinkSync(fileInfo.path);
-                        console.log(t('operations.deletedFile', { filename: path.basename(fileInfo.path) }));
-                        deletedCount++;
-                    } catch (error) {
-                        console.log(t('operations.failedToDeleteFile', { filename: path.basename(fileInfo.path), error: error.message }));
-                    }
-                }
+                for (const fileInfo of filesToDelete) {
+                    try {
+                        const validatedDeletePath = SecurityUtils.validatePath(fileInfo.path, process.cwd());
+                        if (!validatedDeletePath) {
+                            console.log(t('operations.failedToDeleteFile', { filename: path.basename(fileInfo.path), error: 'Invalid path' }));
+                            continue;
+                        }
+                        fs.unlinkSync(validatedDeletePath);
+                        console.log(t('operations.deletedFile', { filename: path.basename(validatedDeletePath) }));
+                        deletedCount++;
+                    } catch (error) {
+                        console.log(t('operations.failedToDeleteFile', { filename: path.basename(fileInfo.path), error: error.message }));
+                    }
+                }
 
                 console.log(`\n🎉 Successfully deleted ${deletedCount} files!`);
             } else {
@@ -1352,30 +1354,41 @@ class I18nManager {
         await this.showInteractiveMenu();
     }
 
-    getAllReportFiles(dir) {
-        if (!dir || typeof dir !== 'string') {
-            return [];
-        }
-
-        let files = [];
-
-        try {
-            if (!SecurityUtils.safeExistsSync(dir)) {
-                return [];
-            }
-
-            const items = require('fs').readdirSync(dir);
-            for (const item of items) {
-                const fullPath = path.join(dir, item);
-
-                try {
-                    const stat = require('fs').statSync(fullPath);
-
-                    if (stat.isDirectory()) {
-                        files.push(...this.getAllReportFiles(fullPath));
-                    } else if (
-                        // Common report file extensions
-                        item.endsWith('.json') ||
+    getAllReportFiles(dir, rootDir = dir) {
+        if (!dir || typeof dir !== 'string') {
+            return [];
+        }
+
+        let files = [];
+
+        try {
+            const validatedDir = SecurityUtils.validatePath(dir, rootDir);
+            if (!validatedDir || !SecurityUtils.safeExistsSync(validatedDir, rootDir)) {
+                return [];
+            }
+
+            const items = fs.readdirSync(validatedDir);
+            for (const item of items) {
+                const fullPath = path.join(validatedDir, item);
+
+                try {
+                    const linkStat = fs.lstatSync(fullPath);
+                    if (linkStat.isSymbolicLink()) {
+                        continue;
+                    }
+
+                    const safeFullPath = SecurityUtils.validatePath(fullPath, rootDir);
+                    if (!safeFullPath) {
+                        continue;
+                    }
+
+                    const stat = fs.statSync(safeFullPath);
+
+                    if (stat.isDirectory()) {
+                        files.push(...this.getAllReportFiles(safeFullPath, rootDir));
+                    } else if (
+                        // Common report file extensions
+                        item.endsWith('.json') ||
                         item.endsWith('.html') ||
                         item.endsWith('.txt') ||
                         item.endsWith('.log') ||
@@ -1386,13 +1399,13 @@ class I18nManager {
                         item.includes('_report.') ||
                         item.includes('report-') ||
                         item.includes('report_') ||
-                        item.includes('analysis-') ||
-                        item.includes('validation-')
-                    ) {
-                        files.push(fullPath);
-                    }
-                } catch (error) {
-                    // Skip individual files that can't be accessed
+                        item.includes('analysis-') ||
+                        item.includes('validation-')
+                    ) {
+                        files.push(safeFullPath);
+                    }
+                } catch (error) {
+                    // Skip individual files that can't be accessed
                     continue;
                 }
             }
@@ -1404,12 +1417,12 @@ class I18nManager {
         return files;
     }
 
-    getFilesToDeleteKeepLast(allFiles, keepCount = 3) {
-        // Sort files by modification time (newest first)
-        const sortedFiles = allFiles.sort((a, b) => {
-            try {
-                const statA = require('fs').statSync(a.path || a);
-                const statB = require('fs').statSync(b.path || b);
+    getFilesToDeleteKeepLast(allFiles, keepCount = 3) {
+        // Sort files by modification time (newest first)
+        const sortedFiles = [...allFiles].sort((a, b) => {
+            try {
+                const statA = fs.statSync(a.path || a);
+                const statB = fs.statSync(b.path || b);
                 return statB.mtime.getTime() - statA.mtime.getTime();
             } catch (error) {
                 // If stat fails, sort by filename as fallback
@@ -1429,8 +1442,7 @@ class I18nManager {
             const authRequired = await this.adminAuth.isAuthRequiredForScript('settingsMenu');
             if (authRequired) {
                 console.log(`\n${t('adminPin.protectedAccess')}`);
-                const cliHelper = require('../../utils/cli-helper');
-                const pin = await cliHelper.promptPin(t('adminPin.enterPin') + ': ');
+                const pin = await cliHelper.promptPin(t('adminPin.enterPin') + ': ');
                 const isValid = await this.adminAuth.verifyPin(pin);
 
                 if (!isValid) {
@@ -1443,8 +1455,7 @@ class I18nManager {
                 console.log(t('adminPin.accessGranted'));
             }
 
-            const SettingsCLI = require('../../settings/settings-cli');
-            const settingsCLI = new SettingsCLI();
+            const settingsCLI = new SettingsCLI();
             await settingsCLI.run();
         } catch (error) {
             console.error('❌ Error opening settings:', error.message);
@@ -1454,8 +1465,7 @@ class I18nManager {
     }
 
     prompt(question) {
-        const cliHelper = require('../../utils/cli-helper');
-        // If interactive not available, return empty string to avoid hangs
+        // If interactive not available, return empty string to avoid hangs
         if (!process.stdin.isTTY || process.stdin.destroyed) {
             console.log('\n⚠️ Interactive input not available, using default response.');
             return Promise.resolve('');

package/main/manage/managers/DebugMenu.js

@@ -3,9 +3,10 @@
  * @module managers/DebugMenu
  */
 
-const path = require('path');
-const fs = require('fs');
-const { t } = require('../../../utils/i18n-helper');
+const path = require('path');
+const fs = require('fs');
+const { t } = require('../../../utils/i18n-helper');
+const cliHelper = require('../../../utils/cli-helper');
 
 module.exports = class DebugMenu {
   constructor(manager) {
@@ -21,8 +22,7 @@ module.exports = class DebugMenu {
     const authRequired = await this.adminAuth.isAuthRequiredForScript('debugMenu');
     if (authRequired) {
       console.log(`\n${t('adminPin.protectedAccess')}`);
-      const cliHelper = require('../../../utils/cli-helper');
-      const pin = await cliHelper.promptPin(t('adminPin.enterPin') + ': ');
+      const pin = await cliHelper.promptPin(t('adminPin.enterPin') + ': ');
       const isValid = await this.adminAuth.verifyPin(pin);
 
       if (!isValid) {
@@ -137,4 +137,4 @@ module.exports = class DebugMenu {
   async show() {
     return this.showDebugMenu();
   }
-};
+};

package/main/manage/managers/InteractiveMenu.js

@@ -3,7 +3,9 @@
  * @module managers/InteractiveMenu
  */
 
-const { t } = require('../../../utils/i18n-helper');
+const { t } = require('../../../utils/i18n-helper');
+const cliHelper = require('../../../utils/cli-helper');
+const summaryTool = require('../../i18ntk-summary');
 
 module.exports = class InteractiveMenu {
   constructor(manager) {
@@ -89,8 +91,7 @@ module.exports = class InteractiveMenu {
         const authRequired = await this.adminAuth.isAuthRequiredForScript('summaryReports');
         if (authRequired) {
           console.log(`\n${t('adminCli.protectedAccess')}`);
-          const cliHelper = require('../../../utils/cli-helper');
-          const pin = await cliHelper.promptPin(t('adminCli.enterPin') + ': ');
+          const pin = await cliHelper.promptPin(t('adminCli.enterPin') + ': ');
           const isValid = await this.adminAuth.verifyPin(pin);
 
           if (!isValid) {
@@ -105,8 +106,7 @@ module.exports = class InteractiveMenu {
 
         console.log(t('summary.status.generating'));
         try {
-          const summaryTool = require('../../i18ntk-summary');
-          const summary = new summaryTool();
+          const summary = new summaryTool();
           await summary.run({ fromMenu: true });
           console.log(t('summary.status.completed'));
 
@@ -174,4 +174,4 @@ module.exports = class InteractiveMenu {
   async show() {
     return this.showInteractiveMenu();
   }
-};
+};

package/main/manage/managers/LanguageMenu.js

@@ -3,7 +3,9 @@
  * @module managers/LanguageMenu
  */
 
-const { t } = require('../../../utils/i18n-helper');
+const { t } = require('../../../utils/i18n-helper');
+const { loadTranslations } = require('../../../utils/i18n-helper');
+const SecurityUtils = require('../../../utils/security');
 
 module.exports = class LanguageMenu {
   constructor(manager) {
@@ -45,8 +47,7 @@ module.exports = class LanguageMenu {
       }
 
       // Force reload translations for the entire system
-      const { loadTranslations } = require('../../../utils/i18n-helper');
-      loadTranslations(selectedLang);
+      loadTranslations(selectedLang);
 
       // Return to main menu with new language
       await this.manager.prompt('\n' + t('language.pressEnterToContinue'));
@@ -64,4 +65,4 @@ module.exports = class LanguageMenu {
   async show() {
     return this.showLanguageMenu();
   }
-};
+};

package/main/manage/managers/SettingsMenu.js

@@ -3,7 +3,9 @@
  * @module managers/SettingsMenu
  */
 
-const { t } = require('../../../utils/i18n-helper');
+const { t } = require('../../../utils/i18n-helper');
+const cliHelper = require('../../../utils/cli-helper');
+const SettingsCLI = require('../../../settings/settings-cli');
 
 module.exports = class SettingsMenu {
   constructor(manager) {
@@ -20,8 +22,7 @@ module.exports = class SettingsMenu {
       const authRequired = await this.adminAuth.isAuthRequiredForScript('settingsMenu');
       if (authRequired) {
         console.log(`\n${t('adminPin.protectedAccess')}`);
-        const cliHelper = require('../../../utils/cli-helper');
-        const pin = await cliHelper.promptPin(t('adminPin.enterPin') + ': ');
+        const pin = await cliHelper.promptPin(t('adminPin.enterPin') + ': ');
         const isValid = await this.adminAuth.verifyPin(pin);
 
         if (!isValid) {
@@ -34,8 +35,7 @@ module.exports = class SettingsMenu {
         console.log(t('adminPin.accessGranted'));
       }
 
-      const SettingsCLI = require('../../../settings/settings-cli');
-      const settingsCLI = new SettingsCLI();
+      const settingsCLI = new SettingsCLI();
       await settingsCLI.run();
     } catch (error) {
       console.error('❌ Error opening settings:', error.message);
@@ -50,4 +50,4 @@ module.exports = class SettingsMenu {
   async show() {
     return this.showSettingsMenu();
   }
-};
+};

package/main/manage/services/AuthenticationService.js

@@ -4,7 +4,8 @@
  * @module services/AuthenticationService
  */
 
-const AdminAuth = require('../../../utils/admin-auth');
+const AdminAuth = require('../../../utils/admin-auth');
+const cliHelper = require('../../../utils/cli-helper');
 
 module.exports = class AuthenticationService {
   constructor(config = {}) {
@@ -74,8 +75,7 @@ module.exports = class AuthenticationService {
     }
 
     console.log(ui ? ui.t('adminCli.authRequired') : 'Admin authentication required');
-    const cliHelper = require('../../../utils/cli-helper');
-    const pin = await cliHelper.promptPin(ui ? ui.t('adminCli.enterPin') : 'Enter PIN: ');
+    const pin = await cliHelper.promptPin(ui ? ui.t('adminCli.enterPin') : 'Enter PIN: ');
     const isValid = await this.verifyPin(pin);
 
     if (!isValid) {
@@ -131,8 +131,7 @@ module.exports = class AuthenticationService {
    * @returns {Promise<string>} User input or empty string
    */
   async prompt(question) {
-    const cliHelper = require('../../../utils/cli-helper');
-    // If interactive not available, return empty string to avoid hangs
+    // If interactive not available, return empty string to avoid hangs
     if (!process.stdin.isTTY || process.stdin.destroyed) {
       console.log('\n⚠️ Interactive input not available, using default response.');
       return Promise.resolve('');
@@ -260,4 +259,4 @@ module.exports = class AuthenticationService {
       process.exit(1);
     }
   }
-};
+};