refactor: remove signer types

execution/types/runtime_decoder.go

@@ -16,13 +16,13 @@ type RuntimeDecoder interface {
 	DecodeCall(buffer *bytes.Buffer) (primitives.Call, error)
 }
 
-type runtimeDecoder[S primitives.PublicKey] struct {
+type runtimeDecoder[P primitives.PublicKey] struct {
 	modules []types.Module
 	extra   primitives.SignedExtra
 }
 
-func NewRuntimeDecoder[S primitives.PublicKey](modules []types.Module, extra primitives.SignedExtra) RuntimeDecoder {
-	return runtimeDecoder[S]{
+func NewRuntimeDecoder[P primitives.PublicKey](modules []types.Module, extra primitives.SignedExtra) RuntimeDecoder {
+	return runtimeDecoder[P]{
 		modules: modules,
 		extra:   extra,
 	}

execution/types/runtime_decoder_test.go

@@ -19,7 +19,7 @@ const (
 	signedExtrinsicVersion = 132
 )
 
-type testKeyType = primitives.Ed25519PublicKey
+type testPublicKeyType = primitives.Ed25519PublicKey
 
 var (
 	signedExtrinsicBytes = []byte{
@@ -55,7 +55,7 @@ var (
 
 func Test_RuntimeDecoder_New(t *testing.T) {
 	target := setupRuntimeDecoder()
-	expect := runtimeDecoder[testKeyType]{
+	expect := runtimeDecoder[testPublicKeyType]{
 		modules: []primitives.Module{mockModuleOne},
 		extra:   mockSignedExtra,
 	}
@@ -357,5 +357,5 @@ func setupRuntimeDecoder() RuntimeDecoder {
 
 	apis := []primitives.Module{mockModuleOne}
 
-	return NewRuntimeDecoder[testKeyType](apis, mockSignedExtra)
+	return NewRuntimeDecoder[testPublicKeyType](apis, mockSignedExtra)
 }

frame/balances/call_force_free.go

@@ -28,7 +28,7 @@ func newCallForceFree(moduleId sc.U8, functionId sc.U8, storedMap primitives.Sto
 }
 
 func (c callForceFree) DecodeArgs(buffer *bytes.Buffer) (primitives.Call, error) {
-	who, err := types.DecodeMultiAddress[testKeyType](buffer)
+	who, err := types.DecodeMultiAddress[testPublicKeyType](buffer)
 	if err != nil {
 		return nil, err
 	}

frame/balances/call_force_transfer.go

@@ -8,7 +8,7 @@ import (
 	primitives "github.com/LimeChain/gosemble/primitives/types"
 )
 
-type testKeyType = primitives.Ed25519PublicKey
+type testPublicKeyType = primitives.Ed25519PublicKey
 
 type callForceTransfer struct {
 	primitives.Callable
@@ -28,11 +28,11 @@ func newCallForceTransfer(moduleId sc.U8, functionId sc.U8, storedMap primitives
 }
 
 func (c callForceTransfer) DecodeArgs(buffer *bytes.Buffer) (primitives.Call, error) {
-	source, err := types.DecodeMultiAddress[testKeyType](buffer)
+	source, err := types.DecodeMultiAddress[testPublicKeyType](buffer)
 	if err != nil {
 		return nil, err
 	}
-	dest, err := types.DecodeMultiAddress[testKeyType](buffer)
+	dest, err := types.DecodeMultiAddress[testPublicKeyType](buffer)
 	if err != nil {
 		return nil, err
 	}

frame/balances/call_set_balance.go

@@ -32,7 +32,7 @@ func newCallSetBalance(moduleId sc.U8, functionId sc.U8, storedMap types.StoredM
 }
 
 func (c callSetBalance) DecodeArgs(buffer *bytes.Buffer) (types.Call, error) {
-	targetAddress, err := types.DecodeMultiAddress[testKeyType](buffer)
+	targetAddress, err := types.DecodeMultiAddress[testPublicKeyType](buffer)
 	if err != nil {
 		return nil, err
 	}

frame/balances/call_transfer.go

@@ -30,7 +30,7 @@ func newCallTransfer(moduleId sc.U8, functionId sc.U8, storedMap primitives.Stor
 }
 
 func (c callTransfer) DecodeArgs(buffer *bytes.Buffer) (primitives.Call, error) {
-	dest, err := types.DecodeMultiAddress[testKeyType](buffer)
+	dest, err := types.DecodeMultiAddress[testPublicKeyType](buffer)
 	if err != nil {
 		return nil, err
 	}

frame/balances/call_transfer_all.go

@@ -27,7 +27,7 @@ func newCallTransferAll(moduleId sc.U8, functionId sc.U8, storedMap primitives.S
 }
 
 func (c callTransferAll) DecodeArgs(buffer *bytes.Buffer) (primitives.Call, error) {
-	dest, err := types.DecodeMultiAddress[testKeyType](buffer)
+	dest, err := types.DecodeMultiAddress[testPublicKeyType](buffer)
 	if err != nil {
 		return nil, err
 	}

frame/balances/call_transfer_keep_alive.go

@@ -27,7 +27,7 @@ func newCallTransferKeepAlive(moduleId sc.U8, functionId sc.U8, storedMap primit
 }
 
 func (c callTransferKeepAlive) DecodeArgs(buffer *bytes.Buffer) (primitives.Call, error) {
-	dest, err := types.DecodeMultiAddress[testKeyType](buffer)
+	dest, err := types.DecodeMultiAddress[testPublicKeyType](buffer)
 	if err != nil {
 		return nil, err
 	}

frame/balances/events.go

@@ -68,7 +68,7 @@ func newEventSlashed(moduleIndex sc.U8, account primitives.AccountId[primitives.
 	return primitives.NewEvent(moduleIndex, EventSlashed, account, amount)
 }
 
-func DecodeEvent[S primitives.PublicKey](moduleIndex sc.U8, buffer *bytes.Buffer) (primitives.Event, error) {
+func DecodeEvent[P primitives.PublicKey](moduleIndex sc.U8, buffer *bytes.Buffer) (primitives.Event, error) {
 	decodedModuleIndex, err := sc.DecodeU8(buffer)
 	if err != nil {
 		return primitives.Event{}, err
@@ -84,7 +84,7 @@ func DecodeEvent[S primitives.PublicKey](moduleIndex sc.U8, buffer *bytes.Buffer
 
 	switch b {
 	case EventEndowed:
-		account, err := primitives.DecodeAccountId[S](buffer)
+		account, err := primitives.DecodeAccountId[P](buffer)
 		if err != nil {
 			return primitives.Event{}, err
 		}
@@ -94,7 +94,7 @@ func DecodeEvent[S primitives.PublicKey](moduleIndex sc.U8, buffer *bytes.Buffer
 		}
 		return newEventEndowed(moduleIndex, account, freeBalance), nil
 	case EventDustLost:
-		account, err := primitives.DecodeAccountId[S](buffer)
+		account, err := primitives.DecodeAccountId[P](buffer)
 		if err != nil {
 			return primitives.Event{}, err
 		}
@@ -104,11 +104,11 @@ func DecodeEvent[S primitives.PublicKey](moduleIndex sc.U8, buffer *bytes.Buffer
 		}
 		return newEventDustLost(moduleIndex, account, amount), nil
 	case EventTransfer:
-		from, err := primitives.DecodeAccountId[S](buffer)
+		from, err := primitives.DecodeAccountId[P](buffer)
 		if err != nil {
 			return primitives.Event{}, err
 		}
-		to, err := primitives.DecodeAccountId[S](buffer)
+		to, err := primitives.DecodeAccountId[P](buffer)
 		if err != nil {
 			return primitives.Event{}, err
 		}
@@ -118,7 +118,7 @@ func DecodeEvent[S primitives.PublicKey](moduleIndex sc.U8, buffer *bytes.Buffer
 		}
 		return newEventTransfer(moduleIndex, from, to, amount), nil
 	case EventBalanceSet:
-		account, err := primitives.DecodeAccountId[S](buffer)
+		account, err := primitives.DecodeAccountId[P](buffer)
 		if err != nil {
 			return primitives.Event{}, err
 		}
@@ -132,7 +132,7 @@ func DecodeEvent[S primitives.PublicKey](moduleIndex sc.U8, buffer *bytes.Buffer
 		}
 		return newEventBalanceSet(moduleIndex, account, free, reserved), nil
 	case EventReserved:
-		account, err := primitives.DecodeAccountId[S](buffer)
+		account, err := primitives.DecodeAccountId[P](buffer)
 		if err != nil {
 			return primitives.Event{}, err
 		}
@@ -142,7 +142,7 @@ func DecodeEvent[S primitives.PublicKey](moduleIndex sc.U8, buffer *bytes.Buffer
 		}
 		return newEventReserved(moduleIndex, account, amount), nil
 	case EventUnreserved:
-		account, err := primitives.DecodeAccountId[S](buffer)
+		account, err := primitives.DecodeAccountId[P](buffer)
 		if err != nil {
 			return primitives.Event{}, err
 		}
@@ -152,11 +152,11 @@ func DecodeEvent[S primitives.PublicKey](moduleIndex sc.U8, buffer *bytes.Buffer
 		}
 		return newEventUnreserved(moduleIndex, account, amount), nil
 	case EventReserveRepatriated:
-		from, err := primitives.DecodeAccountId[S](buffer)
+		from, err := primitives.DecodeAccountId[P](buffer)
 		if err != nil {
 			return primitives.Event{}, err
 		}
-		to, err := primitives.DecodeAccountId[S](buffer)
+		to, err := primitives.DecodeAccountId[P](buffer)
 		if err != nil {
 			return primitives.Event{}, err
 		}
@@ -170,7 +170,7 @@ func DecodeEvent[S primitives.PublicKey](moduleIndex sc.U8, buffer *bytes.Buffer
 		}
 		return newEventReserveRepatriated(moduleIndex, from, to, amount, destinationStatus), nil
 	case EventDeposit:
-		account, err := primitives.DecodeAccountId[S](buffer)
+		account, err := primitives.DecodeAccountId[P](buffer)
 		if err != nil {
 			return primitives.Event{}, err
 		}
@@ -180,7 +180,7 @@ func DecodeEvent[S primitives.PublicKey](moduleIndex sc.U8, buffer *bytes.Buffer
 		}
 		return newEventDeposit(moduleIndex, account, amount), nil
 	case EventWithdraw:
-		account, err := primitives.DecodeAccountId[S](buffer)
+		account, err := primitives.DecodeAccountId[P](buffer)
 		if err != nil {
 			return primitives.Event{}, err
 		}
@@ -190,7 +190,7 @@ func DecodeEvent[S primitives.PublicKey](moduleIndex sc.U8, buffer *bytes.Buffer
 		}
 		return newEventWithdraw(moduleIndex, account, amount), nil
 	case EventSlashed:
-		account, err := primitives.DecodeAccountId[S](buffer)
+		account, err := primitives.DecodeAccountId[P](buffer)
 		if err != nil {
 			return primitives.Event{}, err
 		}

frame/balances/events_test.go

@@ -6,7 +6,6 @@ import (
 
 	sc "github.com/LimeChain/goscale"
 	"github.com/LimeChain/gosemble/frame/balances/types"
-	primitives "github.com/LimeChain/gosemble/primitives/types"
 	"github.com/stretchr/testify/assert"
 )
 
@@ -20,7 +19,7 @@ func Test_Balances_DecodeEvent_Endowed(t *testing.T) {
 	buffer.Write(targetAddressId.Bytes())
 	buffer.Write(targetValue.Bytes())
 
-	result, err := DecodeEvent[primitives.Ed25519PublicKey](moduleId, buffer)
+	result, err := DecodeEvent[testPublicKeyType](moduleId, buffer)
 	assert.Nil(t, err)
 
 	assert.Equal(t,
@@ -38,7 +37,7 @@ func Test_Balances_DecodeEvent_DustLost(t *testing.T) {
 	buffer.Write(targetAddressId.Bytes())
 	buffer.Write(targetValue.Bytes())
 
-	result, err := DecodeEvent[primitives.Ed25519PublicKey](moduleId, buffer)
+	result, err := DecodeEvent[testPublicKeyType](moduleId, buffer)
 	assert.Nil(t, err)
 
 	assert.Equal(t,
@@ -61,7 +60,7 @@ func Test_Balances_DecodeEvent_Transfer(t *testing.T) {
 	buffer.Write(toAddressAccountId.Bytes())
 	buffer.Write(targetValue.Bytes())
 
-	result, _ := DecodeEvent[primitives.Ed25519PublicKey](moduleId, buffer)
+	result, _ := DecodeEvent[testPublicKeyType](moduleId, buffer)
 
 	assert.Equal(t,
 		sc.NewVaryingData(sc.U8(moduleId), EventTransfer, fromAddressId, toAddressAccountId, targetValue),
@@ -80,7 +79,7 @@ func Test_Balances_DecodeEvent_BalanceSet(t *testing.T) {
 	buffer.Write(newFree.Bytes())
 	buffer.Write(newReserved.Bytes())
 
-	result, err := DecodeEvent[primitives.Ed25519PublicKey](moduleId, buffer)
+	result, err := DecodeEvent[testPublicKeyType](moduleId, buffer)
 	assert.Nil(t, err)
 
 	assert.Equal(t,
@@ -99,7 +98,7 @@ func Test_Balances_DecodeEvent_Reserved(t *testing.T) {
 	buffer.Write(targetAddressId.Bytes())
 	buffer.Write(targetValue.Bytes())
 
-	result, err := DecodeEvent[primitives.Ed25519PublicKey](moduleId, buffer)
+	result, err := DecodeEvent[testPublicKeyType](moduleId, buffer)
 	assert.Nil(t, err)
 
 	assert.Equal(t,
@@ -118,7 +117,7 @@ func Test_Balances_DecodeEvent_Unreserved(t *testing.T) {
 	buffer.Write(targetAddressId.Bytes())
 	buffer.Write(targetValue.Bytes())
 
-	result, err := DecodeEvent[primitives.Ed25519PublicKey](moduleId, buffer)
+	result, err := DecodeEvent[testPublicKeyType](moduleId, buffer)
 	assert.Nil(t, err)
 
 	assert.Equal(t,
@@ -140,7 +139,7 @@ func Test_Balances_DecodeEvent_ReserveRepatriated(t *testing.T) {
 	buffer.Write(targetValue.Bytes())
 	buffer.Write(types.BalanceStatusFree.Bytes())
 
-	result, err := DecodeEvent[primitives.Ed25519PublicKey](moduleId, buffer)
+	result, err := DecodeEvent[testPublicKeyType](moduleId, buffer)
 	assert.Nil(t, err)
 
 	assert.Equal(t,
@@ -164,7 +163,7 @@ func Test_Balances_DecodeEvent_Deposit(t *testing.T) {
 	buffer.Write(targetAddressId.Bytes())
 	buffer.Write(targetValue.Bytes())
 
-	result, err := DecodeEvent[primitives.Ed25519PublicKey](moduleId, buffer)
+	result, err := DecodeEvent[testPublicKeyType](moduleId, buffer)
 	assert.Nil(t, err)
 
 	assert.Equal(t,
@@ -183,7 +182,7 @@ func Test_Balances_DecodeEvent_Withdraw(t *testing.T) {
 	buffer.Write(targetAddressId.Bytes())
 	buffer.Write(targetValue.Bytes())
 
-	result, err := DecodeEvent[primitives.Ed25519PublicKey](moduleId, buffer)
+	result, err := DecodeEvent[testPublicKeyType](moduleId, buffer)
 	assert.Nil(t, err)
 
 	assert.Equal(t,
@@ -202,7 +201,7 @@ func Test_Balances_DecodeEvent_Slashed(t *testing.T) {
 	buffer.Write(targetAddressId.Bytes())
 	buffer.Write(targetValue.Bytes())
 
-	result, err := DecodeEvent[primitives.Ed25519PublicKey](moduleId, buffer)
+	result, err := DecodeEvent[testPublicKeyType](moduleId, buffer)
 	assert.Nil(t, err)
 
 	assert.Equal(t,
@@ -216,7 +215,7 @@ func Test_Balances_DecodeEvent_InvalidModule_Panics(t *testing.T) {
 	buffer.WriteByte(0)
 
 	assert.PanicsWithValue(t, errInvalidEventModule, func() {
-		DecodeEvent[primitives.Ed25519PublicKey](moduleId, buffer)
+		DecodeEvent[testPublicKeyType](moduleId, buffer)
 	})
 }
 
@@ -226,6 +225,6 @@ func Test_Balances_DecodeEvent_InvalidType_Panics(t *testing.T) {
 	buffer.WriteByte(255)
 
 	assert.PanicsWithValue(t, errInvalidEventType, func() {
-		DecodeEvent[primitives.Ed25519PublicKey](moduleId, buffer)
+		DecodeEvent[testPublicKeyType](moduleId, buffer)
 	})
 }

frame/grandpa/module.go

@@ -38,7 +38,7 @@ type GrandpaModule interface {
 	Authorities() (sc.Sequence[primitives.Authority], error)
 }
 
-type Module[S primitives.PublicKey] struct {
+type Module[P primitives.PublicKey] struct {
 	primitives.DefaultInherentProvider
 	hooks.DefaultDispatchModule
 	Index   sc.U8

frame/grandpa/storage.go

@@ -13,6 +13,6 @@ type storage struct {
 	Authorities support.StorageValue[primitives.VersionedAuthorityList]
 }
 
-func newStorage[S primitives.PublicKey]() *storage {
-	return &storage{Authorities: support.NewSimpleStorageValue(keyGrandpaAuthorities, primitives.DecodeVersionedAuthorityList[S])}
+func newStorage[P primitives.PublicKey]() *storage {
+	return &storage{Authorities: support.NewSimpleStorageValue(keyGrandpaAuthorities, primitives.DecodeVersionedAuthorityList[P])}
 }