devicely 2.1.9 → 2.2.1

package/lib/logger.js

@@ -1,7 +1,9 @@
 // Centralized Logger for Production
-// Use environment variable to control debug output
+// Use environment variable or --debug flag to control debug output
 
-const DEBUG_MODE = process.env.DEBUG === 'true' || process.env.NODE_ENV === 'development';
+const DEBUG_MODE = process.argv.includes('--debug') || 
+                   process.env.DEBUG === 'true' || 
+                   process.env.NODE_ENV === 'development';
 
 class Logger {
   constructor(module) {
@@ -28,8 +30,18 @@ class Logger {
 
   // Keep critical startup/connection messages
   startup(...args) {
-    console.log(`[STARTUP][${this.module}]`, ...args);
+    console.log(`✅ [${this.module}]`, ...args);
   }
+  
+  // Show execution results (always visible)
+  result(...args) {
+    console.log(`📝 [${this.module}]`, ...args);
+  }
+}
+
+// Show debug mode status on startup
+if (DEBUG_MODE) {
+  console.log('🔧 DEBUG MODE ENABLED - Use devicely without --debug flag for quiet mode\n');
 }
 
 module.exports = Logger;

package/lib/server.js

@@ -9,6 +9,10 @@ const { spawn } = require('child_process');
 const WebSocket = require('ws');
 const fs = require('fs').promises;
 const AIProviderManager = require('./aiProviders');
+const Logger = require('./logger');
+
+// Initialize logger
+const logger = new Logger('Server');
 
 // Import scriptLoader for encrypted scripts
 let scriptLoader;
@@ -643,12 +647,11 @@ function executeCommand(deviceNames, command, wsClient) {
 
 function executeWithScript(scriptPath, deviceNames, command, wsClient) {
   return new Promise((resolve, reject) => {
-    console.log(`\n${'='.repeat(60)}`);
-    console.log(`Script: ${path.basename(scriptPath)}`);
-    console.log(`Devices: [${deviceNames.join(', ')}]`);
-    console.log(`Command: "${command}"`);
-
-    console.log(`${'='.repeat(60)}\n`);
+    logger.debug(`\n${'='.repeat(60)}`);
+    logger.debug(`Script: ${path.basename(scriptPath)}`);
+    logger.debug(`Devices: [${deviceNames.join(', ')}]`);
+    logger.debug(`Command: "${command}"`);
+    logger.debug(`${'='.repeat(60)}\n`);
     
     // Execute command on each device in parallel
     const devicePromises = deviceNames.map((deviceName, index) => {
@@ -657,6 +660,8 @@ function executeWithScript(scriptPath, deviceNames, command, wsClient) {
         // Use -d flag for each device
         const args = ['-d', deviceName, command];
         
+        logger.debug(`[${index + 1}/${deviceNames.length}] Executing on: ${deviceName}`);
+        
         
         const child = spawn(scriptPath, args, {
           cwd: path.dirname(scriptPath)
@@ -669,7 +674,7 @@ function executeWithScript(scriptPath, deviceNames, command, wsClient) {
         child.stdout.on('data', (data) => {
           const text = data.toString();
           output += text;
-          console.log(`[${deviceName}] STDOUT: ${text.trim()}`);
+          logger.debug(`[${deviceName}] STDOUT: ${text.trim()}`);
           if (wsClient) {
             wsClient.send(JSON.stringify({
               type: 'command_output',
@@ -728,15 +733,15 @@ function executeWithScript(scriptPath, deviceNames, command, wsClient) {
 // Execute Android commands (uses device UDID instead of name)
 function executeAndroidDevices(scriptPath, devices, command, wsClient) {
   return new Promise((resolve, reject) => {
-    console.log(`\n${'='.repeat(60)}`);
-    console.log(`Script: ${path.basename(scriptPath)}`);
-    console.log(`Devices: [${devices.map(d => d.name).join(', ')}]`);
-    console.log(`Command: "${command}"`);
-    console.log(`${'='.repeat(60)}\n`);
+    logger.debug(`\n${'='.repeat(60)}`);
+    logger.debug(`Script: ${path.basename(scriptPath)}`);
+    logger.debug(`Devices: [${devices.map(d => d.name).join(', ')}]`);
+    logger.debug(`Command: "${command}"`);
+    logger.debug(`${'='.repeat(60)}\n`);
     
     // Execute command on each device in parallel
     const devicePromises = devices.map((device, index) => {
-      console.log(`[${index + 1}/${devices.length}] Creating execution promise for: ${device.name} (${device.udid})`);
+      logger.debug(`[${index + 1}/${devices.length}] Creating execution promise for: ${device.name} (${device.udid}`);
       
       return new Promise((resolveDevice, rejectDevice) => {
         // Android script expects: <device_serial> <command> [args...]
@@ -754,7 +759,7 @@ function executeAndroidDevices(scriptPath, devices, command, wsClient) {
         child.stdout.on('data', (data) => {
           const text = data.toString();
           output += text;
-          console.log(`[${device.name}] STDOUT: ${text.trim()}`);
+          logger.debug(`[${device.name}] STDOUT: ${text.trim()}`);
           if (wsClient) {
             wsClient.send(JSON.stringify({ 
               type: 'output', 
@@ -767,7 +772,7 @@ function executeAndroidDevices(scriptPath, devices, command, wsClient) {
         child.stderr.on('data', (data) => {
           const text = data.toString();
           error += text;
-          console.log(`[${device.name}] STDERR: ${text.trim()}`);
+          logger.debug(`[${device.name}] STDERR: ${text.trim()}`);
         });
         
         child.on('close', (code) => {