@onekeyfe/hardware-cli 1.2.0-alpha.11 → 1.2.0-alpha.12

package/dist/transports/nobleBlePlugin.js

@@ -288,9 +288,9 @@ function subscribeNotifications(deviceId, generation, notifyCharacteristic) {
         throw hd_shared_1.ERRORS.TypedError(hd_shared_1.HardwareErrorCode.BleCharacteristicNotifyChangeFailure);
     });
 }
-function writeCharacteristic(characteristic, buffer) {
+function writeCharacteristic(characteristic, buffer, withoutResponse) {
     return new Promise((resolve, reject) => {
-        characteristic.write(buffer, true, (error) => {
+        characteristic.write(buffer, withoutResponse, (error) => {
             if (error) {
                 reject(error);
                 return;
@@ -359,7 +359,7 @@ function createNobleBlePlugin() {
             const buffer = Buffer.from(data, 'hex');
             for (let offset = 0; offset < buffer.length; offset += BLE_PACKET_SIZE) {
                 const chunk = buffer.subarray(offset, Math.min(offset + BLE_PACKET_SIZE, buffer.length));
-                await writeCharacteristic(characteristics.write, chunk);
+                await writeCharacteristic(characteristics.write, chunk, true);
                 if (offset + BLE_PACKET_SIZE < buffer.length) {
                     await (0, hd_shared_1.wait)(BLE_WRITE_DELAY);
                 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@onekeyfe/hardware-cli",
-  "version": "1.2.0-alpha.11",
+  "version": "1.2.0-alpha.12",
   "description": "OneKey hardware wallet CLI for testing device communication",
   "author": "OneKey",
   "license": "Apache-2.0",
@@ -30,12 +30,12 @@
     "test": "jest"
   },
   "dependencies": {
-    "@onekeyfe/hd-common-connect-sdk": "1.2.0-alpha.11",
-    "@onekeyfe/hd-core": "1.2.0-alpha.11",
-    "@onekeyfe/hd-shared": "1.2.0-alpha.11",
-    "@onekeyfe/hd-transport-usb": "1.2.0-alpha.11",
+    "@onekeyfe/hd-common-connect-sdk": "1.2.0-alpha.12",
+    "@onekeyfe/hd-core": "1.2.0-alpha.12",
+    "@onekeyfe/hd-shared": "1.2.0-alpha.12",
+    "@onekeyfe/hd-transport-usb": "1.2.0-alpha.12",
     "@stoprocent/noble": "2.3.16",
     "commander": "^12.0.0"
   },
-  "gitHead": "05d3421ec00e07e99f6665e6436a612503e32dfc"
+  "gitHead": "3601e8003c96a2a4ca7856e865d16f48c0c5b758"
 }

package/src/__tests__/noble-ble-plugin.test.ts

@@ -39,6 +39,7 @@ const createPeripheral = (id: string) => {
       connect: jest.fn(callback => callback()),
       disconnect: jest.fn(callback => callback()),
     },
+    write,
     notify,
   };
 };
@@ -110,4 +111,33 @@ describe('Noble BLE plugin notification routing', () => {
 
     expect(result).toBe('completed');
   });
+
+  test('uses withoutResponse for normal and high-volume writes', async () => {
+    const device = createPeripheral('device-a');
+    const noble = new EventEmitter() as EventEmitter & {
+      state: string;
+      startScanning: jest.Mock;
+      stopScanning: jest.Mock;
+    };
+    noble.state = 'poweredOn';
+    noble.startScanning = jest.fn((_services, _duplicates, callback) => {
+      callback?.();
+      noble.emit('discover', device.peripheral);
+    });
+    noble.stopScanning = jest.fn(callback => callback?.());
+    jest.doMock('@stoprocent/noble', () => noble);
+
+    const { createNobleBlePlugin } = await import('../transports/nobleBlePlugin');
+    const plugin = createNobleBlePlugin();
+    await plugin.init();
+    await plugin.connect('device-a');
+
+    await plugin.send('device-a', 'aa');
+    await plugin.send('device-a', 'bb');
+
+    expect(device.write.write.mock.calls.map(([, withoutResponse]) => withoutResponse)).toEqual([
+      true,
+      true,
+    ]);
+  });
 });

package/src/transports/nobleBlePlugin.ts

@@ -379,9 +379,13 @@ function subscribeNotifications(
     });
 }
 
-function writeCharacteristic(characteristic: Characteristic, buffer: Buffer) {
+function writeCharacteristic(
+  characteristic: Characteristic,
+  buffer: Buffer,
+  withoutResponse: boolean
+) {
   return new Promise<void>((resolve, reject) => {
-    characteristic.write(buffer, true, (error?: Error) => {
+    characteristic.write(buffer, withoutResponse, (error?: Error) => {
       if (error) {
         reject(error);
         return;
@@ -463,7 +467,7 @@ export function createNobleBlePlugin(): LowlevelTransportSharedPlugin {
       const buffer = Buffer.from(data, 'hex');
       for (let offset = 0; offset < buffer.length; offset += BLE_PACKET_SIZE) {
         const chunk = buffer.subarray(offset, Math.min(offset + BLE_PACKET_SIZE, buffer.length));
-        await writeCharacteristic(characteristics.write, chunk);
+        await writeCharacteristic(characteristics.write, chunk, true);
         if (offset + BLE_PACKET_SIZE < buffer.length) {
           await wait(BLE_WRITE_DELAY);
         }