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

package/dist/transports/nobleBlePlugin.js

@@ -16,7 +16,6 @@ const CONNECTION_TIMEOUT = 8000;
 const SERVICE_DISCOVERY_TIMEOUT = 10000;
 const BLE_CLEANUP_TIMEOUT = 100;
 const BLE_PACKET_SIZE = 192;
-const BLE_WRITE_DELAY = 5;
 const BLE_ENCRYPTION_ERROR_PATTERNS = [/encryption is insufficient/i, /insufficient encryption/i];
 let noble = null;
 let nobleReadyPromise = null;
@@ -360,9 +359,6 @@ function createNobleBlePlugin() {
             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, true);
-                if (offset + BLE_PACKET_SIZE < buffer.length) {
-                    await (0, hd_shared_1.wait)(BLE_WRITE_DELAY);
-                }
             }
         },
         async receive(uuid) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@onekeyfe/hardware-cli",
-  "version": "1.2.0-alpha.12",
+  "version": "1.2.0-alpha.14",
   "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.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",
+    "@onekeyfe/hd-common-connect-sdk": "1.2.0-alpha.14",
+    "@onekeyfe/hd-core": "1.2.0-alpha.14",
+    "@onekeyfe/hd-shared": "1.2.0-alpha.14",
+    "@onekeyfe/hd-transport-usb": "1.2.0-alpha.14",
     "@stoprocent/noble": "2.3.16",
     "commander": "^12.0.0"
   },
-  "gitHead": "3601e8003c96a2a4ca7856e865d16f48c0c5b758"
+  "gitHead": "c1edaed6435fb6049c7516c64b298ee62519e797"
 }

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

@@ -140,4 +140,35 @@ describe('Noble BLE plugin notification routing', () => {
       true,
     ]);
   });
+
+  test('does not add a fixed delay between 192-byte writes', async () => {
+    const device = createPeripheral('device-a');
+    const noble = new EventEmitter() as EventEmitter & {
+      state: string;
+      startScanning: jest.Mock;
+      stopScanning: jest.Mock;
+    };
+    const wait = jest.fn(() => Promise.resolve());
+    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);
+    jest.doMock('@onekeyfe/hd-shared', () => ({
+      ...jest.requireActual('@onekeyfe/hd-shared'),
+      wait,
+    }));
+
+    const { createNobleBlePlugin } = await import('../transports/nobleBlePlugin');
+    const plugin = createNobleBlePlugin();
+    await plugin.init();
+    await plugin.connect('device-a');
+
+    await plugin.send('device-a', 'aa'.repeat(193));
+
+    expect(device.write.write).toHaveBeenCalledTimes(2);
+    expect(wait).not.toHaveBeenCalled();
+  });
 });

package/src/transports/nobleBlePlugin.ts

@@ -3,7 +3,6 @@ import {
   HardwareErrorCode,
   ONEKEY_SERVICE_UUID,
   isOnekeyDevice,
-  wait,
 } from '@onekeyfe/hd-shared';
 
 import type { LowLevelDevice, LowlevelTransportSharedPlugin } from '@onekeyfe/hd-transport';
@@ -54,7 +53,6 @@ const CONNECTION_TIMEOUT = 8_000;
 const SERVICE_DISCOVERY_TIMEOUT = 10_000;
 const BLE_CLEANUP_TIMEOUT = 100;
 const BLE_PACKET_SIZE = 192;
-const BLE_WRITE_DELAY = 5;
 const BLE_ENCRYPTION_ERROR_PATTERNS = [/encryption is insufficient/i, /insufficient encryption/i];
 
 let noble: NobleModule | null = null;
@@ -468,9 +466,6 @@ export function createNobleBlePlugin(): LowlevelTransportSharedPlugin {
       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, true);
-        if (offset + BLE_PACKET_SIZE < buffer.length) {
-          await wait(BLE_WRITE_DELAY);
-        }
       }
     },