@posx/capacitor-usb-printer 0.0.5 → 0.0.6

package/android/src/main/java/ai/posx/capacitor/usbprinter/UsbPrinter.java

@@ -10,6 +10,7 @@ import android.os.Build;
 import android.util.Log;
 
 import java.util.ArrayList;
+import java.util.Collections;
 import java.util.List;
 import java.util.Map;
 import java.util.concurrent.ConcurrentHashMap;
@@ -24,9 +25,11 @@ public class UsbPrinter {
 
     public UsbPrinter(Context context) {
         usbManager = (UsbManager) context.getSystemService(Context.USB_SERVICE);
+        if (usbManager == null) Log.e(TAG, "USB service unavailable on this device");
     }
 
     public List<UsbDevice> listDevices() {
+        if (usbManager == null) return Collections.emptyList();
         return new ArrayList<>(usbManager.getDeviceList().values());
     }
 
@@ -39,6 +42,7 @@ public class UsbPrinter {
     }
 
     public void requestPermission(Context context, UsbDevice device, PermissionCallback callback) {
+        if (usbManager == null) { callback.onResult(false); return; }
         if (usbManager.hasPermission(device)) {
             callback.onResult(true);
             return;
@@ -70,6 +74,7 @@ public class UsbPrinter {
     }
 
     public boolean connect(UsbDevice device) {
+        if (usbManager == null) return false;
         int id = device.getDeviceId();
         if (connections.containsKey(id)) return true;
 
@@ -101,7 +106,11 @@ public class UsbPrinter {
         }
 
         try {
-            conn.claimInterface(usbInterface, true);
+            if (!conn.claimInterface(usbInterface, true)) {
+                conn.close();
+                Log.e(TAG, "Failed to claim interface for device " + id);
+                return false;
+            }
             String name = device.getProductName() != null ? device.getProductName() : device.getDeviceName();
             String serial = getSerialNumber(device);
             connections.put(id, new DeviceConnection(conn, bulkOut, usbInterface, name, serial));
@@ -129,8 +138,8 @@ public class UsbPrinter {
         DeviceConnection dc = connections.get(deviceId);
         if (dc == null) return false;
         int result = dc.connection.bulkTransfer(dc.bulkOut, data, data.length, 10000);
-        if (result < 0) Log.e(TAG, "Transfer failed for device " + deviceId + ": " + result);
-        return result >= 0;
+        if (result <= 0) Log.e(TAG, "Transfer failed for device " + deviceId + ": " + result);
+        return result > 0;
     }
 
     public DeviceConnection getConnection(int deviceId) {
@@ -138,6 +147,7 @@ public class UsbPrinter {
     }
 
     public UsbDevice findDevice(int deviceId) {
+        if (usbManager == null) return null;
         for (UsbDevice device : usbManager.getDeviceList().values()) {
             if (device.getDeviceId() == deviceId) return device;
         }
@@ -148,8 +158,12 @@ public class UsbPrinter {
         if (hex.length() % 2 != 0) throw new IllegalArgumentException("Hex string length must be even");
         int len = hex.length();
         byte[] data = new byte[len / 2];
-        for (int i = 0; i < len; i += 2)
-            data[i / 2] = (byte) ((Character.digit(hex.charAt(i), 16) << 4) + Character.digit(hex.charAt(i + 1), 16));
+        for (int i = 0; i < len; i += 2) {
+            int hi = Character.digit(hex.charAt(i), 16);
+            int lo = Character.digit(hex.charAt(i + 1), 16);
+            if (hi < 0 || lo < 0) throw new IllegalArgumentException("Invalid hex char at position " + i);
+            data[i / 2] = (byte) ((hi << 4) + lo);
+        }
         return data;
     }
 

package/android/src/main/java/ai/posx/capacitor/usbprinter/UsbPrinterPlugin.java

@@ -39,7 +39,7 @@ public class UsbPrinterPlugin extends Plugin {
                     notifyListeners("deviceDetached", data);
                 } else {
                     // request permission first so serialNumber is readable
-                    implementation.requestPermission(ctx, device, granted -> {
+                    implementation.requestPermission(getContext(), device, granted -> {
                         data.put("serialNumber", implementation.getSerialNumber(device));
                         notifyListeners("deviceAttached", data);
                     });
@@ -173,6 +173,8 @@ public class UsbPrinterPlugin extends Plugin {
             boolean success = implementation.print(deviceId, bytes);
             if (success) call.resolve();
             else call.reject("Not connected or transfer failed");
+        } catch (IllegalArgumentException e) {
+            call.reject("Invalid hex data: " + e.getMessage());
         } catch (Exception e) {
             call.reject("Failed to print", e);
         }

package/package.json

@@ -1,63 +1,63 @@
-{
-  "name": "@posx/capacitor-usb-printer",
-  "version": "0.0.5",
-  "description": "Capacitor plugin for USB ESC/POS thermal printer via Android USB Host API",
-  "main": "dist/plugin.cjs.js",
-  "module": "dist/esm/index.js",
-  "types": "dist/esm/index.d.ts",
-  "unpkg": "dist/plugin.js",
-  "files": [
-    "android/src/main/",
-    "android/build.gradle",
-    "dist/"
-  ],
-  "author": "posx team",
-  "license": "MIT",
-  "keywords": [
-    "capacitor",
-    "plugin",
-    "native"
-  ],
-  "scripts": {
-    "verify": "npm run verify:android && npm run verify:web",
-    "verify:android": "cd android && ./gradlew clean build test && cd ..",
-    "verify:web": "npm run build",
-    "lint": "npm run eslint && npm run prettier -- --check",
-    "fmt": "npm run eslint -- --fix && npm run prettier -- --write",
-    "eslint": "eslint . --ext ts",
-    "prettier": "prettier \"**/*.{css,html,ts,js,java}\" --plugin=prettier-plugin-java",
-    "docgen": "docgen --api UsbPrinterPlugin --output-readme README.md --output-json dist/docs.json",
-    "build": "npm run clean && npm run docgen && tsc && rollup -c rollup.config.mjs",
-    "clean": "rimraf ./dist",
-    "p": "npm run build && npm publish",
-    "bump": "bumpp patch --commit --tag --push --yes",
-    "watch": "tsc --watch",
-    "prepublishOnly": "npm run build"
-  },
-  "devDependencies": {
-    "@capacitor/android": "^5.7.8",
-    "@capacitor/core": "^5.7.8",
-    "@capacitor/docgen": "^0.3.1",
-    "@ionic/eslint-config": "^0.4.0",
-    "@ionic/prettier-config": "^4.0.0",
-    "bumpp": "10.4.1",
-    "eslint": "^8.57.1",
-    "prettier": "^3.6.2",
-    "prettier-plugin-java": "^2.7.7",
-    "rimraf": "^6.1.0",
-    "rollup": "^4.53.2",
-    "typescript": "^5.9.3"
-  },
-  "peerDependencies": {
-    "@capacitor/core": ">=5.0.0"
-  },
-  "prettier": "@ionic/prettier-config",
-  "eslintConfig": {
-    "extends": "@ionic/eslint-config/recommended"
-  },
-  "capacitor": {
-    "android": {
-      "src": "android"
-    }
-  }
-}
+{
+  "name": "@posx/capacitor-usb-printer",
+  "version": "0.0.6",
+  "description": "Capacitor plugin for USB ESC/POS thermal printer via Android USB Host API",
+  "main": "dist/plugin.cjs.js",
+  "module": "dist/esm/index.js",
+  "types": "dist/esm/index.d.ts",
+  "unpkg": "dist/plugin.js",
+  "files": [
+    "android/src/main/",
+    "android/build.gradle",
+    "dist/"
+  ],
+  "author": "posx team",
+  "license": "MIT",
+  "keywords": [
+    "capacitor",
+    "plugin",
+    "native"
+  ],
+  "scripts": {
+    "verify": "npm run verify:android && npm run verify:web",
+    "verify:android": "cd android && ./gradlew clean build test && cd ..",
+    "verify:web": "npm run build",
+    "lint": "npm run eslint && npm run prettier -- --check",
+    "fmt": "npm run eslint -- --fix && npm run prettier -- --write",
+    "eslint": "eslint . --ext ts",
+    "prettier": "prettier \"**/*.{css,html,ts,js,java}\" --plugin=prettier-plugin-java",
+    "docgen": "docgen --api UsbPrinterPlugin --output-readme README.md --output-json dist/docs.json",
+    "build": "npm run clean && npm run docgen && tsc && rollup -c rollup.config.mjs",
+    "clean": "rimraf ./dist",
+    "p": "npm run build && npm publish",
+    "bump": "bumpp patch --commit --tag --push --yes",
+    "watch": "tsc --watch",
+    "prepublishOnly": "npm run build"
+  },
+  "devDependencies": {
+    "@capacitor/android": "^5.7.8",
+    "@capacitor/core": "^5.7.8",
+    "@capacitor/docgen": "^0.3.1",
+    "@ionic/eslint-config": "^0.4.0",
+    "@ionic/prettier-config": "^4.0.0",
+    "bumpp": "10.4.1",
+    "eslint": "^8.57.1",
+    "prettier": "^3.6.2",
+    "prettier-plugin-java": "^2.7.7",
+    "rimraf": "^6.1.0",
+    "rollup": "^4.53.2",
+    "typescript": "^5.9.3"
+  },
+  "peerDependencies": {
+    "@capacitor/core": ">=5.0.0"
+  },
+  "prettier": "@ionic/prettier-config",
+  "eslintConfig": {
+    "extends": "@ionic/eslint-config/recommended"
+  },
+  "capacitor": {
+    "android": {
+      "src": "android"
+    }
+  }
+}