@onekeyfe/hd-transport-electron 1.2.8-alpha.3 → 1.2.8-alpha.5

package/dist/{index-98d6f724.js → index-2231e562.js}

@@ -55,13 +55,13 @@ function invokeNobleBleIpc(request) {
 
 function initNobleBleSupport(webContents) {
     return __awaiter(this, void 0, void 0, function* () {
-        const { setupNobleBleHandlers } = yield Promise.resolve().then(function () { return require('./noble-ble-handler-688faeb6.js'); });
+        const { setupNobleBleHandlers } = yield Promise.resolve().then(function () { return require('./noble-ble-handler-804f0a41.js'); });
         setupNobleBleHandlers(webContents);
     });
 }
 function disposeNobleBleSupport(releaseNoble) {
     return __awaiter(this, void 0, void 0, function* () {
-        const handler = yield Promise.resolve().then(function () { return require('./noble-ble-handler-688faeb6.js'); });
+        const handler = yield Promise.resolve().then(function () { return require('./noble-ble-handler-804f0a41.js'); });
         yield handler.disposeNobleBleSupport(releaseNoble);
     });
 }

package/dist/index.js

@@ -2,7 +2,7 @@
 
 Object.defineProperty(exports, '__esModule', { value: true });
 
-var index = require('./index-98d6f724.js');
+var index = require('./index-2231e562.js');
 
 
 

package/dist/{noble-ble-handler-688faeb6.js → noble-ble-handler-804f0a41.js}

@@ -1,6 +1,6 @@
 'use strict';
 
-var index = require('./index-98d6f724.js');
+var index = require('./index-2231e562.js');
 var hdShared = require('@onekeyfe/hd-shared');
 var pRetry = require('p-retry');
 
@@ -376,8 +376,18 @@ function updateBluetoothState(state) {
 function initializeNoble() {
     return index.__awaiter(this, void 0, void 0, function* () {
         assertBleActive();
-        if (noble)
+        if (noble) {
+            if (noble.state === 'poweredOff') {
+                throw hdShared.ERRORS.TypedError(hdShared.HardwareErrorCode.BlePoweredOff);
+            }
+            if (noble.state === 'unsupported') {
+                throw hdShared.ERRORS.TypedError(hdShared.HardwareErrorCode.BleUnsupported);
+            }
+            if (noble.state === 'unauthorized') {
+                throw hdShared.ERRORS.TypedError(hdShared.HardwareErrorCode.BleLocationError);
+            }
             return;
+        }
         try {
             noble = require('@stoprocent/noble');
             logger === null || logger === void 0 ? void 0 : logger.info('[NobleBLE] Noble library loaded');
@@ -869,9 +879,7 @@ function performTargetedScan(targetDeviceId, timeoutMs = NOBLE_BLE_TARGETED_SCAN
 }
 function enumerateDevices(isWindowDestroyed) {
     return index.__awaiter(this, void 0, void 0, function* () {
-        if (!noble) {
-            yield initializeNoble();
-        }
+        yield initializeNoble();
         if (!noble) {
             throw hdShared.ERRORS.TypedError(hdShared.HardwareErrorCode.RuntimeError, 'Noble not available');
         }
@@ -1268,6 +1276,7 @@ function tryDirectConnectById(deviceId) {
 function connectDevice(deviceId, webContents) {
     var _a, _b, _c;
     return index.__awaiter(this, void 0, void 0, function* () {
+        yield initializeNoble();
         logger === null || logger === void 0 ? void 0 : logger.info('[NobleBLE] Connect device request:', {
             deviceId,
             hasDiscovered: discoveredDevices.has(deviceId),
@@ -1281,9 +1290,6 @@ function connectDevice(deviceId, webContents) {
             peripheral = undefined;
         }
         if (!peripheral) {
-            if (!noble) {
-                yield initializeNoble();
-            }
             if (!noble) {
                 throw hdShared.ERRORS.TypedError(hdShared.HardwareErrorCode.RuntimeError, 'Noble not available');
             }

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":"AAiCA,OAAO,KAAK,EAA+B,WAAW,EAAE,MAAM,UAAU,CAAC;AAEzE,OAAO,KAAK,EAAE,wBAAwB,EAAE,oBAAoB,EAAE,MAAM,qBAAqB,CAAC;AAiC1F,KAAK,mBAAmB,GAAG,KAAK,GAAG;IACjC,eAAe,CAAC,EAAE,MAAM,CAAC;IACzB,iBAAiB,CAAC,EAAE,MAAM,CAAC;CAC5B,CAAC;AAEF,wBAAgB,6BAA6B,CAAC,KAAK,EAAE,mBAAmB,EAAE,aAAa,SAAK,+CAG3F;AAED,wBAAgB,8BAA8B,CAAC,KAAK,EAAE,OAAO,GAAG,wBAAwB,CAgCvF;AA+KD,wBAAgB,+BAA+B,CAAC,OAAO,CAAC,EAAE,oBAAoB,UAI7E;AAizDD,wBAAgB,qBAAqB,CAAC,WAAW,EAAE,WAAW,GAAG,IAAI,CAkOpE;AAOD,wBAAgB,sBAAsB,CACpC,YAAY,GAAE,CAAC,QAAQ,EAAE;IAAE,IAAI,IAAI,IAAI,CAAA;CAAE,KAAK,IAAkC,GAC/E,OAAO,CAAC,IAAI,CAAC,CAiEf"}
+{"version":3,"file":"noble-ble-handler.d.ts","sourceRoot":"","sources":["../src/noble-ble-handler.ts"],"names":[],"mappings":"AAiCA,OAAO,KAAK,EAA+B,WAAW,EAAE,MAAM,UAAU,CAAC;AAEzE,OAAO,KAAK,EAAE,wBAAwB,EAAE,oBAAoB,EAAE,MAAM,qBAAqB,CAAC;AAiC1F,KAAK,mBAAmB,GAAG,KAAK,GAAG;IACjC,eAAe,CAAC,EAAE,MAAM,CAAC;IACzB,iBAAiB,CAAC,EAAE,MAAM,CAAC;CAC5B,CAAC;AAEF,wBAAgB,6BAA6B,CAAC,KAAK,EAAE,mBAAmB,EAAE,aAAa,SAAK,+CAG3F;AAED,wBAAgB,8BAA8B,CAAC,KAAK,EAAE,OAAO,GAAG,wBAAwB,CAgCvF;AA+KD,wBAAgB,+BAA+B,CAAC,OAAO,CAAC,EAAE,oBAAoB,UAI7E;AAszDD,wBAAgB,qBAAqB,CAAC,WAAW,EAAE,WAAW,GAAG,IAAI,CAkOpE;AAOD,wBAAgB,sBAAsB,CACpC,YAAY,GAAE,CAAC,QAAQ,EAAE;IAAE,IAAI,IAAI,IAAI,CAAA;CAAE,KAAK,IAAkC,GAC/E,OAAO,CAAC,IAAI,CAAC,CAiEf"}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@onekeyfe/hd-transport-electron",
-  "version": "1.2.8-alpha.3",
+  "version": "1.2.8-alpha.5",
   "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.8-alpha.3",
-    "@onekeyfe/hd-shared": "1.2.8-alpha.3",
-    "@onekeyfe/hd-transport": "1.2.8-alpha.3",
+    "@onekeyfe/hd-core": "1.2.8-alpha.5",
+    "@onekeyfe/hd-shared": "1.2.8-alpha.5",
+    "@onekeyfe/hd-transport": "1.2.8-alpha.5",
     "@stoprocent/noble": "2.3.16",
     "p-retry": "^4.6.2"
   },
@@ -36,5 +36,5 @@
     "electron": "^25.0.0",
     "typescript": "^5.3.3"
   },
-  "gitHead": "18c57239359413a079fffa730b36149c8516aa85"
+  "gitHead": "c1f24dc0a59d0c02fe05594a9c4ae7652ec9cc32"
 }

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

@@ -923,16 +923,21 @@ describe('Noble BLE process shutdown', () => {
     ['unauthorized', HardwareErrorCode.BleLocationError],
     ['unsupported', HardwareErrorCode.BleUnsupported],
   ])(
-    'classifies the initial adapter state %s without waiting for another event',
+    'preserves adapter state %s across repeated scans and connections',
     async (state, errorCode) => {
       jest.useFakeTimers({ doNotFake: ['performance'] });
       const { sdk, native, handlers } = await setup(state);
-      await expect(
-        handlers.get(EOneKeyBleMessageKeys.NOBLE_BLE_ENUMERATE)?.({})
-      ).resolves.toMatchObject({
-        success: false,
-        error: { errorCode },
-      });
+      for (let attempt = 0; attempt < 2; attempt += 1) {
+        for (const channel of [
+          EOneKeyBleMessageKeys.NOBLE_BLE_ENUMERATE,
+          EOneKeyBleMessageKeys.NOBLE_BLE_CONNECT,
+        ]) {
+          await expect(handlers.get(channel)?.({}, 'device')).resolves.toMatchObject({
+            success: false,
+            error: { errorCode },
+          });
+        }
+      }
       expect(native.startScanning).not.toHaveBeenCalled();
       expect(native.listenerCount('stateChange')).toBe(1);
       expect(jest.getTimerCount()).toBe(0);
@@ -940,6 +945,32 @@ describe('Noble BLE process shutdown', () => {
     }
   );
 
+  test.each([
+    ['poweredOff', HardwareErrorCode.BlePoweredOff],
+    ['unauthorized', HardwareErrorCode.BleLocationError],
+    ['unsupported', HardwareErrorCode.BleUnsupported],
+  ])('checks adapter state %s again after initialization', async (state, errorCode) => {
+    const { sdk, native, handlers } = await setup();
+    const checkAvailability = handlers.get(EOneKeyBleMessageKeys.BLE_AVAILABILITY_CHECK);
+    await expect(checkAvailability?.({})).resolves.toMatchObject({ available: true });
+    native.state = state;
+    native.emit('stateChange', state);
+    for (const channel of [
+      EOneKeyBleMessageKeys.NOBLE_BLE_ENUMERATE,
+      EOneKeyBleMessageKeys.NOBLE_BLE_CONNECT,
+    ]) {
+      await expect(handlers.get(channel)?.({}, 'device')).resolves.toMatchObject({
+        success: false,
+        error: { errorCode },
+      });
+    }
+    expect(native.startScanning).not.toHaveBeenCalled();
+    native.state = 'poweredOn';
+    native.emit('stateChange', native.state);
+    await expect(checkAvailability?.({})).resolves.toMatchObject({ available: true });
+    await sdk.disposeNobleBleSupport();
+  });
+
   test('cancels an active scan, ignores its late callback and releases native once', async () => {
     jest.useFakeTimers({ doNotFake: ['performance'] });
     const { sdk, native, handlers } = await setup();

package/src/noble-ble-handler.ts

@@ -429,7 +429,18 @@ function updateBluetoothState(state: string): void {
 // Initialize Noble
 async function initializeNoble(): Promise<void> {
   assertBleActive();
-  if (noble) return;
+  if (noble) {
+    if (noble.state === 'poweredOff') {
+      throw ERRORS.TypedError(HardwareErrorCode.BlePoweredOff);
+    }
+    if (noble.state === 'unsupported') {
+      throw ERRORS.TypedError(HardwareErrorCode.BleUnsupported);
+    }
+    if (noble.state === 'unauthorized') {
+      throw ERRORS.TypedError(HardwareErrorCode.BleLocationError);
+    }
+    return;
+  }
 
   try {
     // eslint-disable-next-line @typescript-eslint/no-var-requires, global-require
@@ -1138,9 +1149,7 @@ async function performTargetedScan(
 
 // Enumerate devices
 async function enumerateDevices(isWindowDestroyed: () => boolean): Promise<DeviceInfo[]> {
-  if (!noble) {
-    await initializeNoble();
-  }
+  await initializeNoble();
 
   if (!noble) {
     throw ERRORS.TypedError(HardwareErrorCode.RuntimeError, 'Noble not available');
@@ -1745,6 +1754,7 @@ async function tryDirectConnectById(deviceId: string): Promise<Peripheral | unde
 
 // Connect to device - supports both discovered and direct connection modes
 async function connectDevice(deviceId: string, webContents: WebContents): Promise<void> {
+  await initializeNoble();
   logger?.info('[NobleBLE] Connect device request:', {
     deviceId,
     hasDiscovered: discoveredDevices.has(deviceId),
@@ -1762,11 +1772,6 @@ async function connectDevice(deviceId: string, webContents: WebContents): Promis
   }
 
   if (!peripheral) {
-    // Initialize Noble if not already done
-    if (!noble) {
-      await initializeNoble();
-    }
-
     if (!noble) {
       throw ERRORS.TypedError(HardwareErrorCode.RuntimeError, 'Noble not available');
     }