Support Ledger clear signing

Author: franciszekjob

.github/workflows/checks.yml

@@ -3,9 +3,9 @@ name: Checks
 env:
   CAIRO_LANG_VERSION: "0.13.1"
   DEVNET_VERSION: "0.1.2"
-  DEVNET_SHA: 7e7dbb5
-  LEDGER_APP_SHA: dd58c5c
-  LEDGER_APP_DEV_TOOLS_SHA: a037d42181f4bed9694246256e2c9e2a899e775c302a9c6482c81f87c28e1432
+  DEVNET_SHA: fc5a2753a2eedcc27eed7a4fae3ecac08c2ca1b4 # v0.3.0
+  LEDGER_APP_SHA: 768a7b47b0da681b28112342edd76e2c9b292c4e # v2.3.1
+  LEDGER_APP_DEV_TOOLS_SHA: a845b2ab0b5dd824133f73858f6f373edea85ec1bd828245bf50ce9700f33bcb # v4.5.0
 
 on:
   push:
@@ -197,6 +197,7 @@ jobs:
           run: |
             cd /apps/app-starknet
             git checkout ${{ env.LEDGER_APP_SHA }}
+            cd starknet
             cargo clean
             cargo ledger build nanox
 
@@ -216,6 +217,9 @@ jobs:
       - name: Wait for Speculos to start
         run: sleep 5
 
+      - name: Allow blind signing
+        run: ./starknet_py/tests/unit/signer/allow_ledger_blind_signing.sh
+
       - name: Update automation rules
         working-directory: starknet_py/tests/unit/signer
         run: |

docs/api/signer.rst

@@ -47,3 +47,11 @@ To use LedgerSigner, you need to install starknetpy with `ledger` extra like thi
 .. autoclass:: LedgerSigner
     :members:
     :member-order: groupwise
+
+-----------------
+LedgerSigningMode
+-----------------
+
+.. autoclass:: LedgerSigningMode
+    :members:
+    :member-order: groupwise

docs/guide/signing.rst

@@ -15,7 +15,7 @@ signing algorithm, it is possible to create ``Account`` with custom
 Signing with Ledger
 -------------------
 :ref:`LedgerSigner` allows you to sign transactions using a Ledger device. The device must be unlocked and Starknet app needs to be open.
-Currently used version of Starknet app is ``1.1.1`` and only blind-signing is possible. Clear-signing will be available in the near future.
+Currently used version of `Starknet app <https://github.com/LedgerHQ/app-starknet>`_ is ``2.3.1``.
 
 .. codesnippet:: ../../starknet_py/tests/unit/signer/test_ledger_signer.py
     :language: python

docs/migration_guide.rst

@@ -1,6 +1,14 @@
 Migration guide
 ===============
 
+******************************
+0.25.1 Migration guide
+******************************
+
+.. py:currentmodule:: starknet_py.net.signer.ledger_signer
+
+1. Support for clear signing with :class:`LedgerSigner` has been added. It's now the default signing mode (see :class:`LedgerSigningMode`).
+
 ******************************
 0.25.0 Migration guide
 ******************************

pyproject.toml

@@ -31,7 +31,6 @@ docs = [
 ]
 ledger = [
     "ledgerwallet>=0.5.0,<1.0.0",
-    "bip-utils>=2.9.3,<3.0.0",
 ]
 
 [project.urls]

starknet_py/constants.py

@@ -39,14 +39,6 @@
 
 ROOT_PATH = Path(__file__).parent
 
-# Ledger constants
-STARKNET_CLA = 0x5A
-EIP_2645_PURPOSE = 0x80000A55
-EIP_2645_PATH_LENGTH = 6
-PUBLIC_KEY_RESPONSE_LENGTH = 65
-SIGNATURE_RESPONSE_LENGTH = 65
-VERSION_RESPONSE_LENGTH = 3
-
 # Result of `encode_shortstring("ANY_CALLER")`
 ANY_CALLER = 0x414E595F43414C4C4552
 
@@ -55,3 +47,8 @@
 class OutsideExecutionInterfaceID(IntEnum):
     V1 = 0x68CFD18B92D1907B8BA3CC324900277F5A3622099431EA85DD8089255E4181
     V2 = 0x1D1144BB2138366FF28D8E9AB57456B1D332AC42196230C3A602003C89872
+
+
+ARGENT_V040_CLASS_HASH = (
+    0x036078334509B514626504EDC9FB252328D1A240E4E948BEF8D0C08DFF45927F
+)