tools: remove jade_capture_image_data.py since jade_cli can do this now
What changed, and why it matters
This commit simply deletes a helper Python script used to capture image data from a Blockstream Jade hardware wallet. The commit message says the same task can now be done with jade_cli, so the old standalone script is no longer needed. There is no code change to the wallet firmware, no bug fix, and no security-related content in the diff or message.
No security action needed. Treat as routine maintenance. If downstream users relied on this script, they should migrate to jade_cli for equivalent functionality.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The removed script (jade_capture_image_data.py) was a thin wrapper around JadeAPI that connected over BLE or serial, called get_version_info(), then called jade.capture_image_data(check_qr=True) and wrote the result to capture.dat. Deleting it is a repository housekeeping change; functionality is reportedly retained in jade_cli. No cryptographic, authentication, or firmware logic is modified.
Changed components
tools/jade_capture_image_data.py (deleted)Inspect captured patch +0 / −49
diff --git a/jade_capture_image_data.py b/jade_capture_image_data.py
deleted file mode 100755
index c4163d6..0000000
--- a/jade_capture_image_data.py
+++ /dev/null
@@ -1,49 +0,0 @@
-#!/usr/bin/env python
-
-import sys
-import time
-import logging
-from jadepy import JadeAPI
-
-LOGGING = logging.INFO
-QR = True
-
-# Enable jade logging
-if LOGGING:
- jadehandler = logging.StreamHandler()
- jadehandler.setLevel(LOGGING)
-
- logger = logging.getLogger('jadepy.jade')
- logger.setLevel(LOGGING)
- logger.addHandler(jadehandler)
-
- device_logger = logging.getLogger('jadepy.jade-device')
- device_logger.setLevel(LOGGING)
- device_logger.addHandler(jadehandler)
-
-
-if len(sys.argv) > 1 and sys.argv[1] == 'ble':
- print('Fetching jade version info over BLE')
- serial_number = sys.argv[2] if len(sys.argv) > 2 else None
- create_jade_fn = JadeAPI.create_ble
- kwargs = {'serial_number': serial_number}
-else:
- print('Fetching jade version info over serial')
- serial_device = sys.argv[1] if len(sys.argv) > 1 else None
- create_jade_fn = JadeAPI.create_serial
- kwargs = {'device': serial_device, 'timeout': 120}
-
-print("Connecting...")
-with create_jade_fn(**kwargs) as jade:
- try:
- verinfo = jade.get_version_info()
- print(f'Connected: {verinfo}')
-
- image_data = jade.capture_image_data(check_qr=QR)
- assert image_data
-
- with open('capture.dat', 'wb') as f:
- f.write(image_data)
-
- except Exception as e:
- print("ERROR:", repr(e))
Why this scored 15/100
Community notes
Notes can correct, qualify, or add evidence to the AI analysis. Every note shown here has been validated by a human moderator.
The AI analysis stands alone for now. Submit a note if you can add evidence or important context.