@leonardojc/capacitor-ioboard 2.0.5 → 2.0.8

package/android/src/main/java/com/leonardojc/capacitor/ioboard/CapacitorIoboardPlugin.java

@@ -14,6 +14,11 @@ import java.util.ArrayList;
 public class CapacitorIoboardPlugin extends Plugin implements SerialConnectionManager.SerialDataListener {
 
     private static final String TAG = "CapacitorIoboard";
+    
+    // Debug mode control
+    // Set to false for production to minimize logs
+    private static final boolean DEBUG_MODE = BuildConfig.DEBUG;
+    
     private SerialConnectionManager serialManager;
     private IOBoardManager ioBoardManager;
     private boolean isConnected = false;
@@ -23,6 +28,33 @@ public class CapacitorIoboardPlugin extends Plugin implements SerialConnectionMa
     // For response waiting in getStatus()
     private volatile byte[] pendingResponse = null;
     private final Object responseLock = new Object();
+    
+    // Helper methods for conditional logging
+    private void logDebug(String message) {
+        if (DEBUG_MODE) {
+            Log.d(TAG, message);
+        }
+    }
+    
+    private void logInfo(String message) {
+        // Always log INFO level (important events)
+        Log.i(TAG, message);
+    }
+    
+    private void logWarning(String message) {
+        // Always log warnings
+        Log.w(TAG, message);
+    }
+    
+    private void logError(String message) {
+        // Always log errors
+        Log.e(TAG, message);
+    }
+    
+    private void logError(String message, Throwable throwable) {
+        // Always log errors with exceptions
+        Log.e(TAG, message, throwable);
+    }
 
     // Helper method to convert bytes to hex string for debugging
     private String bytesToHex(byte[] bytes) {
@@ -122,14 +154,16 @@ public class CapacitorIoboardPlugin extends Plugin implements SerialConnectionMa
 
         @PluginMethod
     public void getStatus(PluginCall call) {
-        Log.d(TAG, "📊 getStatus called");
+        logDebug("📊 getStatus called");
         
         if (serialManager == null) {
+            logError("Serial connection not initialized");
             call.reject("Serial connection not initialized");
             return;
         }
         
         if (!serialManager.isConnected()) {
+            logError("Not connected to serial port");
             call.reject("Not connected to serial port");
             return;
         }
@@ -206,13 +240,13 @@ public class CapacitorIoboardPlugin extends Plugin implements SerialConnectionMa
             }
             result.put("doorStatusText", doorStatusText.toString());
             
-            Log.i(TAG, String.format("✅ IOBoard Status - Firmware: %s, Door Status: 0x%02X (%s)", 
+            logInfo(String.format("✅ IOBoard Status - Firmware: %s, Door Status: 0x%02X (%s)", 
                 parsedResponse.firmware, parsedResponse.doorStatus, doorStatusText.toString()));
             
             call.resolve(result);
             
         } catch (InterruptedException e) {
-            Log.e(TAG, "Thread interrupted while waiting for status: " + e.getMessage(), e);
+            logError("Thread interrupted while waiting for status: " + e.getMessage(), e);
             call.reject("Status query interrupted: " + e.getMessage());
         } catch (Exception e) {
             Log.e(TAG, "Error in getStatus: " + e.getMessage(), e);

package/android/src/main/java/com/leonardojc/capacitor/ioboard/SerialConnectionManager.java

@@ -19,6 +19,9 @@ import java.util.concurrent.Executors;
 public class SerialConnectionManager {
     private static final String TAG = "SerialConnectionManager";
     
+    // Debug mode control - matches CapacitorIoboardPlugin
+    private static final boolean DEBUG_MODE = BuildConfig.DEBUG;
+    
     public interface SerialDataListener {
         void onDataReceived(byte[] data);
         void onConnectionStateChanged(boolean connected);
@@ -37,7 +40,30 @@ public class SerialConnectionManager {
     public SerialConnectionManager() {
         mainHandler = new Handler(Looper.getMainLooper());
         executorService = Executors.newSingleThreadExecutor();
-        Log.d(TAG, "SerialConnectionManager initialized");
+        logDebug("SerialConnectionManager initialized");
+    }
+    
+    // Helper methods for conditional logging
+    private void logDebug(String message) {
+        if (DEBUG_MODE) {
+            Log.d(TAG, message);
+        }
+    }
+    
+    private void logInfo(String message) {
+        Log.i(TAG, message);
+    }
+    
+    private void logWarning(String message) {
+        Log.w(TAG, message);
+    }
+    
+    private void logError(String message) {
+        Log.e(TAG, message);
+    }
+    
+    private void logError(String message, Throwable throwable) {
+        Log.e(TAG, message, throwable);
     }
     
     public void setDataListener(SerialDataListener listener) {
@@ -166,32 +192,35 @@ public class SerialConnectionManager {
         }
         
         try {
-            // Enhanced logging with hex dump
-            Log.d(TAG, "=== WRITING TO SERIAL PORT ===");
-            Log.d(TAG, "Port: " + currentPortPath);
-            Log.d(TAG, "Data length: " + data.length + " bytes");
-            Log.d(TAG, "HEX data: " + bytesToHex(data));
-            Log.d(TAG, "Raw bytes: " + java.util.Arrays.toString(data));
+            // Enhanced logging with hex dump (DEBUG only)
+            logDebug("=== WRITING TO SERIAL PORT ===");
+            logDebug("Port: " + currentPortPath);
+            logDebug("Data length: " + data.length + " bytes");
+            logDebug("HEX data: " + bytesToHex(data));
+            logDebug("Raw bytes: " + java.util.Arrays.toString(data));
             
             // Clear any pending input first
             if (serialInputStream != null && serialInputStream.available() > 0) {
                 int available = serialInputStream.available();
                 byte[] discarded = new byte[available];
                 serialInputStream.read(discarded);
-                Log.d(TAG, "Discarded " + available + " pending bytes before write");
+                logDebug("Discarded " + available + " pending bytes before write");
             }
             
             long startTime = System.currentTimeMillis();
             
             // Write entire frame at once (IOBoard expects complete frame)
-            Log.d(TAG, "Sending complete frame as single write operation...");
+            logDebug("Sending complete frame as single write operation...");
             serialOutputStream.write(data);
             serialOutputStream.flush();
             
             long endTime = System.currentTimeMillis();
             
-            Log.i(TAG, "✅ Successfully wrote " + data.length + " bytes in " + (endTime - startTime) + "ms");
-            Log.i(TAG, "HEX written: " + bytesToHex(data));
+            // Only log summary in production
+            if (DEBUG_MODE) {
+                logInfo("✅ Successfully wrote " + data.length + " bytes in " + (endTime - startTime) + "ms");
+                logInfo("HEX written: " + bytesToHex(data));
+            }
             
             // Wait longer for IOBoard response (some devices need more processing time)
             Thread.sleep(200); // Increased from 50ms to 200ms
@@ -320,38 +349,42 @@ public class SerialConnectionManager {
                                 byte[] data = new byte[bytesRead];
                                 System.arraycopy(buffer, 0, data, 0, bytesRead);
                                 
-                                // Enhanced logging for received data
-                                Log.d(TAG, "=== 📥 RECEIVED FROM IOBOARD ===");
-                                Log.d(TAG, "Port: " + currentPortPath);
-                                Log.d(TAG, "Received " + bytesRead + " bytes");
-                                Log.d(TAG, "HEX received: " + bytesToHex(data));
-                                Log.d(TAG, "Raw bytes: " + java.util.Arrays.toString(data));
+                                // Enhanced logging for received data (DEBUG only for detailed hex)
+                                logDebug("=== 📥 RECEIVED FROM IOBOARD ===");
+                                logDebug("Port: " + currentPortPath);
+                                logDebug("Received " + bytesRead + " bytes");
+                                logDebug("HEX received: " + bytesToHex(data));
+                                logDebug("Raw bytes: " + java.util.Arrays.toString(data));
                                 
                                 // Parser automático de respuesta del IOBoard
                                 try {
                                     CapacitorIoboardPlugin.IOBoardResponse response = CapacitorIoboardPlugin.parseIOBoardResponse(data);
                                     if (response.isValid) {
-                                        Log.i(TAG, String.format("🎯 IOBoard Response Parsed Successfully: %s", response.toString()));
+                                        logDebug(String.format("🎯 IOBoard Response Parsed Successfully: %s", response.toString()));
                                         
                                         // Logging específico según tipo de respuesta
                                         if (response.type == 0x00) { // Status Response
-                                            Log.i(TAG, String.format("📊 IOBoard Status - Firmware: %s, Door Status: 0x%02X", 
+                                            logInfo(String.format("📊 IOBoard Status - Firmware: %s, Door Status: 0x%02X", 
                                                 response.firmware, response.doorStatus));
                                             
-                                            // Interpretar door status
-                                            String doorStatusDesc = interpretDoorStatus(response.doorStatus);
-                                            Log.i(TAG, String.format("🚪 Door Status: %s", doorStatusDesc));
+                                            // Interpretar door status (DEBUG only)
+                                            if (DEBUG_MODE) {
+                                                String doorStatusDesc = interpretDoorStatus(response.doorStatus);
+                                                logInfo(String.format("🚪 Door Status: %s", doorStatusDesc));
+                                            }
                                         }
                                     } else {
-                                        Log.w(TAG, "⚠️ Received invalid IOBoard response - trying ASCII interpretation");
+                                        logWarning("⚠️ Received invalid IOBoard response - trying ASCII interpretation");
                                         
-                                        // Try to interpret as ASCII if printable
-                                        String ascii = new String(data);
-                                        boolean isPrintable = ascii.chars().allMatch(c -> c >= 32 && c <= 126);
-                                        if (isPrintable) {
-                                            Log.d(TAG, "ASCII: '" + ascii + "'");
-                                        } else {
-                                            Log.d(TAG, "Binary data (non-ASCII)");
+                                        // Try to interpret as ASCII if printable (DEBUG only)
+                                        if (DEBUG_MODE) {
+                                            String ascii = new String(data);
+                                            boolean isPrintable = ascii.chars().allMatch(c -> c >= 32 && c <= 126);
+                                            if (isPrintable) {
+                                                logDebug("ASCII: '" + ascii + "'");
+                                            } else {
+                                                logDebug("Binary data (non-ASCII)");
+                                            }
                                         }
                                     }
                                 } catch (Exception e) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@leonardojc/capacitor-ioboard",
-  "version": "2.0.5",
+  "version": "2.0.8",
   "description": "A comprehensive Capacitor plugin for IOBoard devices with integrated serial communication and complete MTC3P08L protocol support including OTA updates",
   "main": "dist/plugin.cjs.js",
   "module": "dist/esm/index.js",