fix: move keccak256 to signRawECDSAMessage

Author: MaxMoskalenko

sdk/src/client/signer.ts

@@ -81,9 +81,9 @@ export class SessionKeyStateSigner implements StateSigner {
     }
 
     async signState(channelId: Hex, state: State): Promise<Hex> {
-        const stateHash = getStateHash(channelId, state);
+        const packedState = getPackedState(channelId, state);
 
-        return signRawECDSAMessage(stateHash, this.sessionKey);
+        return signRawECDSAMessage(packedState, this.sessionKey);
     }
 
     async signRawMessage(message: Hex): Promise<Hex> {

sdk/src/rpc/api.ts

@@ -1,4 +1,4 @@
-import { Address, Hex, keccak256, stringToBytes, WalletClient } from 'viem';
+import { Address, Hex, keccak256, stringToBytes, toHex, WalletClient } from 'viem';
 import { privateKeyToAccount } from 'viem/accounts';
 import {
     MessageSigner,
@@ -643,11 +643,9 @@ export function createEIP712AuthMessageSigner(
 export function createECDSAMessageSigner(privateKey: Hex): MessageSigner {
     return async (payload: RequestData | ResponsePayload): Promise<Hex> => {
         try {
-            const messageBytes = keccak256(
-                stringToBytes(JSON.stringify(payload, (_, v) => (typeof v === 'bigint' ? v.toString() : v))),
-            );
+            const message = toHex(JSON.stringify(payload, (_, v) => (typeof v === 'bigint' ? v.toString() : v)));
 
-            return signRawECDSAMessage(messageBytes, privateKey);
+            return signRawECDSAMessage(message, privateKey);
         } catch (error) {
             console.error('ECDSA signing failed:', error);
             throw new Error(`ECDSA signing failed: ${error}`);

sdk/src/utils/sign.ts

@@ -1,8 +1,9 @@
-import { Hex } from 'viem';
+import { Hex, keccak256 } from 'viem';
 import { privateKeyToAccount } from 'viem/accounts';
 
 export const signRawECDSAMessage = async (message: Hex, privateKey: Hex): Promise<Hex> => {
-    const flatSignature = await privateKeyToAccount(privateKey).sign({ hash: message });
+    const hash = keccak256(message);
+    const flatSignature = await privateKeyToAccount(privateKey).sign({ hash });
 
     return flatSignature;
 };

sdk/test/unit/client/state.test.ts

@@ -25,7 +25,7 @@ describe('_prepareAndSignInitialState', () => {
     const adjudicatorAddress = '0xADJ' as Hex;
     const challengeDuration = BigInt(123);
     const stateSigner = {
-        getAddress: jest.fn(async () => '0xOWNER' as Hex),
+        getAddress: jest.fn(() => '0xOWNER' as Hex),
         signState: jest.fn(async (_1: Hex, _2: State) => 'accSig'),
         signRawMessage: jest.fn(async (_: Hex) => 'accSig'),
     };