Added Pharos to list of log index fix (#193)

Author: simsonraj

pkg/client/rpc_client.go

@@ -1353,7 +1353,7 @@ func ToBlockNumArg(number *big.Int) string {
 func (r *RPCClient) makeLogsValid(logs []types.Log) error {
 
 	switch r.chainType {
-	case chaintype.ChainSei, chaintype.ChainHedera, chaintype.ChainRootstock:
+	case chaintype.ChainSei, chaintype.ChainHedera, chaintype.ChainRootstock, chaintype.ChainPharos:
 		// Sei, Rootstock and Hedera does not have unique log index position in the block.
 	default:
 		return nil
@@ -1372,7 +1372,7 @@ func (r *RPCClient) makeLogsValid(logs []types.Log) error {
 
 func (r *RPCClient) makeLogValid(log types.Log) (types.Log, error) {
 	switch r.chainType {
-	case chaintype.ChainSei, chaintype.ChainHedera, chaintype.ChainRootstock:
+	case chaintype.ChainSei, chaintype.ChainHedera, chaintype.ChainRootstock, chaintype.ChainPharos:
 		// Sei, Rootstock and Hedera does not have unique log index position in the block.
 	default:
 		return log, nil

pkg/client/rpc_client_internal_test.go

@@ -34,6 +34,7 @@ func TestRPCClient_MakeLogsValid(t *testing.T) {
 		{Name: "Sei", ChainType: chaintype.ChainSei},
 		{Name: "Hedera", ChainType: chaintype.ChainHedera},
 		{Name: "Rootstock", ChainType: chaintype.ChainRootstock},
+		{Name: "Pharos", ChainType: chaintype.ChainPharos},
 	}
 
 	testCases := []struct {

pkg/client/rpc_client_test.go

@@ -394,6 +394,7 @@ func TestRPCClient_SubscribeFilterLogs(t *testing.T) {
 			{Name: "Sei", ChainType: chaintype.ChainSei},
 			{Name: "Hedera", ChainType: chaintype.ChainHedera},
 			{Name: "Rootstock", ChainType: chaintype.ChainRootstock},
+			{Name: "Pharos", ChainType: chaintype.ChainPharos},
 		}
 
 		testCases := []struct {
@@ -476,6 +477,7 @@ func TestRPCClientFilterLogs(t *testing.T) {
 			{Name: "Sei", ChainType: chaintype.ChainSei},
 			{Name: "Hedera", ChainType: chaintype.ChainHedera},
 			{Name: "Rootstock", ChainType: chaintype.ChainRootstock},
+			{Name: "Pharos", ChainType: chaintype.ChainPharos},
 		}
 
 		testCases := []struct {

pkg/config/chaintype/chaintype.go

@@ -26,6 +26,7 @@ const (
 	ChainZircuit         ChainType = "zircuit"
 	ChainTron            ChainType = "tron"
 	ChainRootstock       ChainType = "rootstock"
+	ChainPharos          ChainType = "pharos"
 )
 
 // IsL2 returns true if this chain is a Layer 2 chain. Notably:
@@ -42,7 +43,7 @@ func (c ChainType) IsL2() bool {
 
 func (c ChainType) IsValid() bool {
 	switch c {
-	case "", ChainArbitrum, ChainAstar, ChainCelo, ChainGnosis, ChainHedera, ChainKroma, ChainMantle, ChainMetis, ChainOptimismBedrock, ChainSei, ChainScroll, ChainWeMix, ChainXLayer, ChainZkEvm, ChainZkSync, ChainZircuit, ChainTron, ChainRootstock:
+	case "", ChainArbitrum, ChainAstar, ChainCelo, ChainGnosis, ChainHedera, ChainKroma, ChainMantle, ChainMetis, ChainOptimismBedrock, ChainSei, ChainScroll, ChainWeMix, ChainXLayer, ChainZkEvm, ChainZkSync, ChainZircuit, ChainTron, ChainRootstock, ChainPharos:
 		return true
 	}
 	return false
@@ -86,6 +87,8 @@ func FromSlug(slug string) ChainType {
 		return ChainTron
 	case "rootstock":
 		return ChainRootstock
+	case "pharos":
+		return ChainPharos
 	default:
 		return ChainType(slug)
 	}
@@ -156,4 +159,5 @@ var ErrInvalid = fmt.Errorf("must be one of %s or omitted", strings.Join([]strin
 	string(ChainZircuit),
 	string(ChainTron),
 	string(ChainRootstock),
+	string(ChainPharos),
 }, ", "))