fix: use log block number for refresh session iD (#111)

<!--- Provide a general summary of your changes in the Title above -->

## Description
<!--- Describe your changes in detail -->

## Related Issue Or Context
<!--- If suggesting a new feature or change, please discuss it in an
issue first -->
<!--- If fixing a bug, there should be an issue describing it with steps
to reproduce -->
<!--- Otherwise, describe context and motivation for change herre -->

Closes: #<issue>

## How Has This Been Tested? Testing details.
<!--- Please describe in detail how you tested your changes. -->
<!--- Include details of your testing environment, and the tests you ran
to -->
<!--- see how your change affects other areas of the code, etc. -->

## Types of changes
<!--- What types of changes does your code introduce? Put an `x` in all
the boxes that apply: -->
- [ ] Bug fix (non-breaking change which fixes an issue)
- [ ] New feature (non-breaking change which adds functionality)
- [ ] Breaking change (fix or feature that would cause existing
functionality to change)
- [ ] Documentation

## Checklist:
<!--- Go over all the following points, and put an `x` in all the boxes
that apply. -->
<!--- If you're unsure about any of these, don't hesitate to ask. We're
here to help! -->
- [ ] I have commented my code, particularly in hard-to-understand
areas.
- [ ] I have ensured that all acceptance criteria (or expected behavior)
from issue are met
- [ ] I have updated the documentation locally and in docs.
- [ ] I have added tests to cover my changes.
- [ ] I have ensured that all the checks are passing and green, I've
signed the CLA bot

Author: mpetrun5

.github/workflows/mocks.yml

@@ -22,7 +22,7 @@ jobs:
         with:
           go-version: "^1.23"
 
-      - run: go install go.uber.org/mock/[email protected].0
+      - run: go install go.uber.org/mock/[email protected]
 
       - run: make genmocks
 

chains/evm/calls/events/listener.go

@@ -45,12 +45,11 @@ func (l *Listener) FetchKeygenEvents(ctx context.Context, contractAddress common
 	return logs, nil
 }
 
-func (l *Listener) FetchRefreshEvents(ctx context.Context, contractAddress common.Address, startBlock *big.Int, endBlock *big.Int) ([]*Refresh, error) {
+func (l *Listener) FetchRefreshEvents(ctx context.Context, contractAddress common.Address, startBlock *big.Int, endBlock *big.Int) (*Refresh, ethTypes.Log, error) {
 	logs, err := l.client.FetchEventLogs(ctx, contractAddress, string(KeyRefreshSig), startBlock, endBlock)
 	if err != nil {
-		return nil, err
+		return nil, ethTypes.Log{}, err
 	}
-	refreshEvents := make([]*Refresh, 0)
 
 	for _, re := range logs {
 		r, err := l.UnpackRefresh(l.abi, re.Data)
@@ -59,10 +58,10 @@ func (l *Listener) FetchRefreshEvents(ctx context.Context, contractAddress commo
 			continue
 		}
 
-		refreshEvents = append(refreshEvents, r)
+		return r, re, nil
 	}
 
-	return refreshEvents, nil
+	return nil, ethTypes.Log{}, nil
 }
 
 func (l *Listener) UnpackRefresh(abi abi.ABI, data []byte) (*Refresh, error) {

chains/evm/listener/tss.go

@@ -26,7 +26,7 @@ import (
 
 type EventListener interface {
 	FetchKeygenEvents(ctx context.Context, address common.Address, startBlock *big.Int, endBlock *big.Int) ([]types.Log, error)
-	FetchRefreshEvents(ctx context.Context, address common.Address, startBlock *big.Int, endBlock *big.Int) ([]*events.Refresh, error)
+	FetchRefreshEvents(ctx context.Context, address common.Address, startBlock *big.Int, endBlock *big.Int) (*events.Refresh, types.Log, error)
 }
 
 type KeygenEventHandler struct {
@@ -143,22 +143,17 @@ func (eh *RefreshEventHandler) HandleEvents(
 	startBlock *big.Int,
 	endBlock *big.Int,
 ) error {
-	refreshEvents, err := eh.eventListener.FetchRefreshEvents(
+	refreshEvent, l, err := eh.eventListener.FetchRefreshEvents(
 		context.Background(), eh.bridgeAddress, startBlock, endBlock,
 	)
 	if err != nil {
 		return fmt.Errorf("unable to fetch keygen events because of: %+v", err)
 	}
-	if len(refreshEvents) == 0 {
+	if refreshEvent == nil {
 		return nil
 	}
 
-	hash := refreshEvents[len(refreshEvents)-1].Hash
-	if hash == "" {
-		log.Error().Msgf("Hash cannot be empty string")
-		return nil
-	}
-	topology, err := eh.topologyProvider.NetworkTopology(hash)
+	topology, err := eh.topologyProvider.NetworkTopology(refreshEvent.Hash)
 	if err != nil {
 		log.Error().Err(err).Msgf("Failed fetching network topology")
 		return nil
@@ -177,7 +172,7 @@ func (eh *RefreshEventHandler) HandleEvents(
 	)
 
 	resharing := resharing.NewResharing(
-		eh.sessionID(startBlock), topology.Threshold, eh.host, eh.communication, eh.ecdsaStorer,
+		eh.sessionID(new(big.Int).SetUint64(l.BlockNumber)), topology.Threshold, eh.host, eh.communication, eh.ecdsaStorer,
 	)
 	err = eh.coordinator.Execute(context.Background(), []tss.TssProcess{resharing}, make(chan interface{}, 1), peer.ID(""))
 	if err != nil {