Merge branch 'master' of github.com:0xsequence/go-sequence into relayer-front-update

Author: Agusx1211

Makefile

@@ -46,6 +46,14 @@ start-testchain-anvil:
 start-testchain-anvil-verbose:
 	cd ./testutil/testchain && pnpm start:anvil:verbose
 
+# Fork a chain: make start-testchain-fork FORK_URL=https://nodes.sequence.app/polygon-zkevm
+FORK_URL ?= https://nodes.sequence.app/polygon-zkevm
+start-testchain-fork:
+	cd ./testutil/testchain && FORK_URL="$(FORK_URL)" pnpm start:anvil:fork -- "$(FORK_URL)"
+
+start-testchain-fork-verbose:
+	cd ./testutil/testchain && FORK_URL="$(FORK_URL)" pnpm start:anvil:fork:verbose -- "$(FORK_URL)"
+
 check-testchain-running:
 	@curl http://localhost:8545 -H"Content-type: application/json" -X POST -d '{"jsonrpc":"2.0","method":"eth_syncing","params":[],"id":1}' --write-out '%{http_code}' --silent --output /dev/null | grep 200 > /dev/null \
 	|| { echo "*****"; echo "Oops! testchain is not running. Please run 'make start-testchain' in another terminal."; echo "*****"; exit 1; }

README.md

@@ -14,6 +14,8 @@ For documentation on sequence, please see our [docs](https://docs.sequence.xyz)
 2. `make start-testchain` -- starts the test ethereum chain (id 1337)
 3. (in a separate terminal) `make test` -- runs test suite
 
+To run tests against a forked chain instead: `make start-testchain-fork` (default fork URL is Polygon ZK EVM). Override with `make start-testchain-fork FORK_URL=https://your-rpc-url`.
+
 
 ## Testing
 

auth.go

@@ -101,7 +101,7 @@ func EIP6492ValidateSignature() ethauth.ValidatorFunc {
 			return false, "", fmt.Errorf("sig is invalid: %w", err)
 		}
 
-		isValid, err := eip6492.ValidateEIP6492Offchain(ctx, provider, signer, hash, sig, nil)
+		isValid, err := eip6492.ValidateEIP6492(ctx, provider, signer, hash, sig, nil)
 		if err != nil {
 			return false, "", fmt.Errorf("failed to validate: %w", err)
 		}

core/v2/v2.go

@@ -1036,7 +1036,7 @@ func (l *signatureTreeDynamicSignatureLeaf) recover(ctx context.Context, payload
 		signature := l.signature
 
 		if provider != nil {
-			isValid, err := eip6492.ValidateEIP6492Offchain(ctx, provider, l.address, payload.Digest().Hash, signature, nil)
+			isValid, err := eip6492.ValidateEIP6492(ctx, provider, l.address, payload.Digest().Hash, signature, nil)
 			if err != nil {
 				return nil, nil, fmt.Errorf("unable to validate signature for %v: %w", l.address, err)
 			}

core/v3/v3.go

@@ -1122,7 +1122,7 @@ func (l *signatureTreeSignatureERC1271Leaf) recover(ctx context.Context, payload
 	signature := l.Signature
 
 	if provider != nil {
-		isValid, err := eip6492.ValidateEIP6492Offchain(ctx, provider, l.Address, payload.Digest().Hash, signature, nil)
+		isValid, err := eip6492.ValidateEIP6492(ctx, provider, l.Address, payload.Digest().Hash, signature, nil)
 		if err != nil {
 			return nil, nil, fmt.Errorf("unable to validate ERC-1271 signature: %w", err)
 		}

eip6492_live_test.go

@@ -0,0 +1,139 @@
+package sequence_test
+
+import (
+	"context"
+	"strings"
+	"testing"
+
+	"github.com/0xsequence/ethkit/go-ethereum/accounts"
+	"github.com/0xsequence/ethkit/go-ethereum/common"
+	"github.com/0xsequence/go-sequence"
+	"github.com/0xsequence/go-sequence/lib/eip6492"
+	"github.com/stretchr/testify/require"
+)
+
+// These tests require a live testchain (make start-testchain or make start-testchain-fork).
+// They create a Sequence wallet, sign a message, wrap in EIP-6492, and validate via
+// ValidateEIP6492Offchain or ValidateEIP6492Onchain. Onchain tests skip if the validator
+// is not deployed at EIP_6492_ADDRESS on the chain.
+
+func TestEIP6492Live_ValidateSequenceWalletMessage_Offchain(t *testing.T) {
+	// V2 counterfactual wallet on testchain
+	wallet, err := testChain.V2DummySequenceWallet(10, true)
+	require.NoError(t, err)
+
+	message := []byte("hello world!")
+	_, eip191Message := accounts.TextAndHash(message)
+
+	sig, err := wallet.SignMessage([]byte(eip191Message))
+	require.NoError(t, err)
+
+	wrapped, err := sequence.EIP6492Signature(sig, wallet.GetWalletConfig())
+	require.NoError(t, err)
+
+	digest := common.BytesToHash(accounts.TextHash(message))
+	ctx := context.Background()
+
+	valid, err := eip6492.ValidateEIP6492Offchain(ctx, testChain.Provider, wallet.Address(), digest, wrapped, nil)
+	require.NoError(t, err)
+	require.True(t, valid, "EIP-6492 offchain validation should succeed for Sequence wallet message signature")
+}
+
+func TestEIP6492Live_ValidateSequenceWalletMessage_ViaIsValidMessageSignature(t *testing.T) {
+	// Same flow as above but via the public API to ensure full path works
+	wallet, err := testChain.V2DummySequenceWallet(11, true)
+	require.NoError(t, err)
+
+	message := []byte("eip6492 live test")
+	_, eip191Message := accounts.TextAndHash(message)
+
+	sig, err := wallet.SignMessage([]byte(eip191Message))
+	require.NoError(t, err)
+
+	wrapped, err := sequence.EIP6492Signature(sig, wallet.GetWalletConfig())
+	require.NoError(t, err)
+
+	isValid, err := sequence.IsValidMessageSignature(
+		wallet.Address(),
+		message,
+		wrapped,
+		testChain.ChainID(),
+		testChain.Provider,
+		nil,
+	)
+	require.NoError(t, err)
+	require.True(t, isValid, "IsValidMessageSignature should succeed for EIP-6492 wrapped Sequence wallet signature")
+}
+
+func TestEIP6492Live_ValidateDeployedSequenceWallet_Offchain(t *testing.T) {
+	// Deploy the wallet first, then validate (tests both counterfactual deploy path and already-deployed path)
+	wallet, err := testChain.V2DummySequenceWallet(12, false)
+	require.NoError(t, err)
+
+	message := []byte("deployed wallet sign")
+	_, eip191Message := accounts.TextAndHash(message)
+
+	sig, err := wallet.SignMessage([]byte(eip191Message))
+	require.NoError(t, err)
+
+	wrapped, err := sequence.EIP6492Signature(sig, wallet.GetWalletConfig())
+	require.NoError(t, err)
+
+	digest := common.BytesToHash(accounts.TextHash(message))
+	ctx := context.Background()
+
+	valid, err := eip6492.ValidateEIP6492Offchain(ctx, testChain.Provider, wallet.Address(), digest, wrapped, nil)
+	require.NoError(t, err)
+	require.True(t, valid, "EIP-6492 offchain validation should succeed for deployed Sequence wallet")
+}
+
+func TestEIP6492Live_ValidateSequenceWalletMessage_Onchain(t *testing.T) {
+	// Same as offchain test but via pre-deployed validator at EIP_6492_ADDRESS.
+	// Skips if the validator is not deployed on this chain.
+	wallet, err := testChain.V2DummySequenceWallet(13, true)
+	require.NoError(t, err)
+
+	message := []byte("hello world onchain!")
+	_, eip191Message := accounts.TextAndHash(message)
+
+	sig, err := wallet.SignMessage([]byte(eip191Message))
+	require.NoError(t, err)
+
+	wrapped, err := sequence.EIP6492Signature(sig, wallet.GetWalletConfig())
+	require.NoError(t, err)
+
+	digest := common.BytesToHash(accounts.TextHash(message))
+	ctx := context.Background()
+
+	valid, err := eip6492.ValidateEIP6492Onchain(ctx, testChain.Provider, wallet.Address(), digest, wrapped, nil)
+	if err != nil && strings.Contains(err.Error(), "returned no data") {
+		t.Skipf("EIP-6492 validator not deployed on this chain: %v", err)
+	}
+	require.NoError(t, err)
+	require.True(t, valid, "EIP-6492 onchain validation should succeed for Sequence wallet message signature")
+}
+
+func TestEIP6492Live_ValidateDeployedSequenceWallet_Onchain(t *testing.T) {
+	// Deployed wallet, validated via onchain contract. Skips if validator not deployed.
+	wallet, err := testChain.V2DummySequenceWallet(14, false)
+	require.NoError(t, err)
+
+	message := []byte("deployed wallet onchain sign")
+	_, eip191Message := accounts.TextAndHash(message)
+
+	sig, err := wallet.SignMessage([]byte(eip191Message))
+	require.NoError(t, err)
+
+	wrapped, err := sequence.EIP6492Signature(sig, wallet.GetWalletConfig())
+	require.NoError(t, err)
+
+	digest := common.BytesToHash(accounts.TextHash(message))
+	ctx := context.Background()
+
+	valid, err := eip6492.ValidateEIP6492Onchain(ctx, testChain.Provider, wallet.Address(), digest, wrapped, nil)
+	if err != nil && strings.Contains(err.Error(), "returned no data") {
+		t.Skipf("EIP-6492 validator not deployed on this chain: %v", err)
+	}
+	require.NoError(t, err)
+	require.True(t, valid, "EIP-6492 onchain validation should succeed for deployed Sequence wallet")
+}

go.mod

@@ -1,13 +1,11 @@
 module github.com/0xsequence/go-sequence
 
-go 1.24.0
-
-toolchain go1.24.2
+go 1.25.0
 
 // replace github.com/0xsequence/ethkit => ../ethkit
 
 require (
-	github.com/0xsequence/ethkit v1.39.0
+	github.com/0xsequence/ethkit v1.43.0
 	github.com/0xsequence/go-ethauth v0.14.0
 	github.com/BurntSushi/toml v1.2.1
 	github.com/davecgh/go-spew v1.1.1
@@ -19,7 +17,7 @@ require (
 	github.com/spf13/cobra v1.10.1
 	github.com/stretchr/testify v1.11.1
 	go.uber.org/mock v0.6.0
-	golang.org/x/crypto v0.45.0
+	golang.org/x/crypto v0.49.0
 )
 
 require (
@@ -54,9 +52,9 @@ require (
 	github.com/supranational/blst v0.3.16 // indirect
 	github.com/tyler-smith/go-bip39 v1.1.0 // indirect
 	github.com/zeebo/xxh3 v1.0.2 // indirect
-	golang.org/x/net v0.47.0 // indirect
-	golang.org/x/sync v0.18.0 // indirect
-	golang.org/x/sys v0.38.0 // indirect
-	golang.org/x/text v0.31.0 // indirect
+	golang.org/x/net v0.51.0 // indirect
+	golang.org/x/sync v0.20.0 // indirect
+	golang.org/x/sys v0.42.0 // indirect
+	golang.org/x/text v0.35.0 // indirect
 	gopkg.in/yaml.v3 v3.0.1 // indirect
 )

go.sum

@@ -1,6 +1,4 @@
-go 1.24.0
-
-toolchain go1.24.2
+go 1.25.0
 
 use (
 	.