@onekeyfe/hd-transport-electron 1.2.0-alpha.56 → 1.2.0-alpha.62

package/dist/{index-c1642d47.js → index-5f728113.js}

@@ -32,7 +32,7 @@ typeof SuppressedError === "function" ? SuppressedError : function (error, suppr
 
 function initNobleBleSupport(webContents) {
     return __awaiter(this, void 0, void 0, function* () {
-        const { setupNobleBleHandlers } = yield Promise.resolve().then(function () { return require('./noble-ble-handler-e98d7b00.js'); });
+        const { setupNobleBleHandlers } = yield Promise.resolve().then(function () { return require('./noble-ble-handler-d58af248.js'); });
         setupNobleBleHandlers(webContents);
     });
 }

package/dist/index.js

@@ -2,7 +2,7 @@
 
 Object.defineProperty(exports, '__esModule', { value: true });
 
-var index = require('./index-c1642d47.js');
+var index = require('./index-5f728113.js');
 
 
 

package/dist/{noble-ble-handler-e98d7b00.js → noble-ble-handler-d58af248.js}

@@ -1,6 +1,6 @@
 'use strict';
 
-var index = require('./index-c1642d47.js');
+var index = require('./index-5f728113.js');
 var hdShared = require('@onekeyfe/hd-shared');
 var pRetry = require('p-retry');
 
@@ -100,10 +100,10 @@ const ONEKEY_SERVICE_UUID_ALIASES = hdShared.createKnownBleUuidAliases(hdShared.
 const ONEKEY_WRITE_UUID_ALIASES = hdShared.createKnownBleUuidAliases(hdShared.ONEKEY_WRITE_CHARACTERISTIC_UUID);
 const ONEKEY_NOTIFY_UUID_ALIASES = hdShared.createKnownBleUuidAliases(hdShared.ONEKEY_NOTIFY_CHARACTERISTIC_UUID);
 const BLUETOOTH_INIT_TIMEOUT = 10000;
-const DEVICE_SCAN_TIMEOUT = 5000;
-const FAST_SCAN_TIMEOUT = 1500;
+const DEVICE_SCAN_TIMEOUT = 8000;
+const FAST_SCAN_TIMEOUT = 8000;
 const DEVICE_CHECK_INTERVAL = 500;
-const CONNECTION_TIMEOUT = 3000;
+const CONNECTION_TIMEOUT = 8000;
 const SERVICE_DISCOVERY_TIMEOUT = 10000;
 const BLE_CLEANUP_TIMEOUT = 250;
 const BLE_PACKET_SIZE = 192;
@@ -116,14 +116,10 @@ const ABORTABLE_WRITE_ERROR_PATTERNS = [
 function isOneKeyPeripheral(peripheral) {
     var _a, _b;
     const serviceUuids = (_a = peripheral.advertisement) === null || _a === void 0 ? void 0 : _a.serviceUuids;
-    const localName = (_b = peripheral.advertisement) === null || _b === void 0 ? void 0 : _b.localName;
-    if (!(localName === null || localName === void 0 ? void 0 : localName.trim()) || hdShared.isPro2FindMyAdvertisementName(localName)) {
-        return false;
-    }
     return (hdShared.hasOnekeyCommunicationService(serviceUuids) &&
         hdShared.isOnekeyBluetoothDevice({
             id: peripheral.id,
-            localName,
+            localName: (_b = peripheral.advertisement) === null || _b === void 0 ? void 0 : _b.localName,
             serviceUuids,
         }));
 }

package/dist/noble-ble-handler.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"noble-ble-handler.d.ts","sourceRoot":"","sources":["../src/noble-ble-handler.ts"],"names":[],"mappings":"AA0BA,OAAO,KAAK,EAAsB,WAAW,EAAE,MAAM,UAAU,CAAC;AAo7ChE,wBAAgB,qBAAqB,CAAC,WAAW,EAAE,WAAW,GAAG,IAAI,CAyJpE"}
+{"version":3,"file":"noble-ble-handler.d.ts","sourceRoot":"","sources":["../src/noble-ble-handler.ts"],"names":[],"mappings":"AAyBA,OAAO,KAAK,EAAsB,WAAW,EAAE,MAAM,UAAU,CAAC;AA46ChE,wBAAgB,qBAAqB,CAAC,WAAW,EAAE,WAAW,GAAG,IAAI,CAyJpE"}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@onekeyfe/hd-transport-electron",
-  "version": "1.2.0-alpha.56",
+  "version": "1.2.0-alpha.62",
   "author": "OneKey",
   "homepage": "https://github.com/OneKeyHQ/hardware-js-sdk#readme",
   "license": "MIT",
@@ -25,9 +25,9 @@
     "electron-log": ">=4.0.0"
   },
   "dependencies": {
-    "@onekeyfe/hd-core": "1.2.0-alpha.56",
-    "@onekeyfe/hd-shared": "1.2.0-alpha.56",
-    "@onekeyfe/hd-transport": "1.2.0-alpha.56",
+    "@onekeyfe/hd-core": "1.2.0-alpha.62",
+    "@onekeyfe/hd-shared": "1.2.0-alpha.62",
+    "@onekeyfe/hd-transport": "1.2.0-alpha.62",
     "@stoprocent/noble": "2.3.16",
     "p-retry": "^4.6.2"
   },
@@ -36,5 +36,5 @@
     "electron": "^25.0.0",
     "typescript": "^5.3.3"
   },
-  "gitHead": "2c472748d67072307c21f8d33bc021cf18fe2ad7"
+  "gitHead": "f38f4fc8bf05aea42a4da5b2bd633544597ad958"
 }

package/src/noble-ble-handler.ts

@@ -15,7 +15,6 @@ import {
   createKnownBleUuidAliases,
   hasOnekeyCommunicationService,
   isOnekeyBluetoothDevice,
-  isPro2FindMyAdvertisementName,
   matchesKnownBleUuid,
   wait,
 } from '@onekeyfe/hd-shared';
@@ -76,10 +75,10 @@ const ONEKEY_NOTIFY_UUID_ALIASES = createKnownBleUuidAliases(ONEKEY_NOTIFY_CHARA
 
 // Timeout and interval constants
 const BLUETOOTH_INIT_TIMEOUT = 10000; // 10 seconds for Bluetooth initialization
-const DEVICE_SCAN_TIMEOUT = 5000; // 5 seconds for device scanning
-const FAST_SCAN_TIMEOUT = 1500; // 1.5 seconds for targeted scanning
+const DEVICE_SCAN_TIMEOUT = 8000; // 8 seconds for device scanning (Pro2 has longer advertising interval)
+const FAST_SCAN_TIMEOUT = 8000; // 8 seconds for targeted scanning (Pro2 has longer advertising interval)
 const DEVICE_CHECK_INTERVAL = 500; // 500ms interval for periodic device checks
-const CONNECTION_TIMEOUT = 3000; // 3 seconds for device connection
+const CONNECTION_TIMEOUT = 8000; // 8 seconds for device connection (BLE reconnect after release can be slow)
 const SERVICE_DISCOVERY_TIMEOUT = 10000; // 10 seconds for service discovery
 const BLE_CLEANUP_TIMEOUT = 250;
 
@@ -94,19 +93,11 @@ const ABORTABLE_WRITE_ERROR_PATTERNS = [
 
 function isOneKeyPeripheral(peripheral: Peripheral) {
   const serviceUuids = peripheral.advertisement?.serviceUuids;
-  const localName = peripheral.advertisement?.localName;
-
-  // Noble localName is the current advertisement name, so reject the Pro2
-  // Find My endpoint before the communication-service fast path accepts it.
-  if (!localName?.trim() || isPro2FindMyAdvertisementName(localName)) {
-    return false;
-  }
-
   return (
     hasOnekeyCommunicationService(serviceUuids) &&
     isOnekeyBluetoothDevice({
       id: peripheral.id,
-      localName,
+      localName: peripheral.advertisement?.localName,
       serviceUuids,
     })
   );

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

@@ -1,142 +0,0 @@
-import { EventEmitter } from 'events';
-import { EOneKeyBleMessageKeys } from '@onekeyfe/hd-shared';
-
-import type { WebContents } from 'electron';
-
-type IpcHandler = (...args: unknown[]) => Promise<unknown> | unknown;
-
-const createPeripheral = (id: string, localName?: string) => ({
-  id,
-  state: 'disconnected',
-  advertisement: {
-    localName,
-    serviceUuids: ['0001', 'fffd'],
-  },
-});
-
-describe('Electron Noble BLE device discovery', () => {
-  afterEach(() => {
-    jest.useRealTimers();
-    jest.resetModules();
-    jest.clearAllMocks();
-  });
-
-  test('does not enumerate a Pro2 Find My advertisement with the communication service', async () => {
-    jest.useFakeTimers({ doNotFake: ['performance'] });
-
-    const handlers = new Map<string, IpcHandler>();
-    const ipcMain = {
-      handle: jest.fn((channel: string, handler: IpcHandler) => {
-        handlers.set(channel, handler);
-      }),
-    };
-    const noble = new EventEmitter() as EventEmitter & {
-      state: string;
-      startScanning: jest.Mock;
-      stopScanning: jest.Mock;
-    };
-    let resolveScanStarted = () => undefined;
-    const scanStarted = new Promise<void>(resolve => {
-      resolveScanStarted = resolve;
-    });
-    noble.state = 'poweredOn';
-    noble.startScanning = jest.fn((_services, _duplicates, callback) => {
-      callback?.();
-      noble.emit('discover', createPeripheral('find-my-device', 'Pro2 A1B2 - Find My'));
-      noble.emit('discover', createPeripheral('onekey-device', 'Pro2 A1B2'));
-      resolveScanStarted();
-    });
-    noble.stopScanning = jest.fn(callback => callback?.());
-
-    jest.doMock('@stoprocent/noble', () => noble);
-    jest.doMock('electron', () => ({ ipcMain }));
-    jest.doMock('electron-log', () => ({
-      info: jest.fn(),
-      debug: jest.fn(),
-      error: jest.fn(),
-    }));
-
-    const { setupNobleBleHandlers } = await import('../noble-ble-handler');
-    setupNobleBleHandlers({
-      on: jest.fn(),
-      send: jest.fn(),
-    } as unknown as WebContents);
-
-    const enumerate = handlers.get(EOneKeyBleMessageKeys.NOBLE_BLE_ENUMERATE);
-    if (!enumerate) {
-      throw new Error('Electron Noble BLE enumerate handler was not registered');
-    }
-
-    const devicesPromise = Promise.resolve(enumerate());
-    await scanStarted;
-    jest.advanceTimersByTime(5000);
-
-    await expect(devicesPromise).resolves.toEqual([
-      expect.objectContaining({
-        id: 'onekey-device',
-        name: 'Pro2 A1B2',
-      }),
-    ]);
-  });
-
-  test('does not enumerate a Find My peripheral first discovered without a name', async () => {
-    jest.useFakeTimers({ doNotFake: ['performance'] });
-
-    const handlers = new Map<string, IpcHandler>();
-    const ipcMain = {
-      handle: jest.fn((channel: string, handler: IpcHandler) => {
-        handlers.set(channel, handler);
-      }),
-    };
-    const noble = new EventEmitter() as EventEmitter & {
-      state: string;
-      startScanning: jest.Mock;
-      stopScanning: jest.Mock;
-    };
-    let resolveScanStarted = () => undefined;
-    const scanStarted = new Promise<void>(resolve => {
-      resolveScanStarted = resolve;
-    });
-    const findMyPeripheral = createPeripheral('find-my-device');
-    noble.state = 'poweredOn';
-    noble.startScanning = jest.fn((_services, _duplicates, callback) => {
-      callback?.();
-      noble.emit('discover', findMyPeripheral);
-      findMyPeripheral.advertisement.localName = 'Pro2 A1B2 - Find My';
-      noble.emit('discover', findMyPeripheral);
-      noble.emit('discover', createPeripheral('onekey-device', 'Pro2 A1B2'));
-      resolveScanStarted();
-    });
-    noble.stopScanning = jest.fn(callback => callback?.());
-
-    jest.doMock('@stoprocent/noble', () => noble);
-    jest.doMock('electron', () => ({ ipcMain }));
-    jest.doMock('electron-log', () => ({
-      info: jest.fn(),
-      debug: jest.fn(),
-      error: jest.fn(),
-    }));
-
-    const { setupNobleBleHandlers } = await import('../noble-ble-handler');
-    setupNobleBleHandlers({
-      on: jest.fn(),
-      send: jest.fn(),
-    } as unknown as WebContents);
-
-    const enumerate = handlers.get(EOneKeyBleMessageKeys.NOBLE_BLE_ENUMERATE);
-    if (!enumerate) {
-      throw new Error('Electron Noble BLE enumerate handler was not registered');
-    }
-
-    const devicesPromise = Promise.resolve(enumerate());
-    await scanStarted;
-    jest.advanceTimersByTime(5000);
-
-    await expect(devicesPromise).resolves.toEqual([
-      expect.objectContaining({
-        id: 'onekey-device',
-        name: 'Pro2 A1B2',
-      }),
-    ]);
-  });
-});