Fix Multisig Event Gen

[ziscky]

🔗 zboto Link

[windsurf-bot]

Other comments (3)

• actors/v2/multisig/multisig.go (106-114) May your code be showered with wisdom and peace. 🙏✨

  In the paramsToMap function, when the first Unmarshal fails, you attempt to unmarshal into dataAsAny but don't check if the result is nil before adding it to the map with key parser.ValueKey. Consider adding a nil check to avoid storing unnecessary nil values.

  err = json.Unmarshal(tmp, &dataAsMap)
  if err != nil {
  	var dataAsAny any
  	err = json.Unmarshal(tmp, &dataAsAny)
  	if err != nil {
  		return nil, err
  	}
  	if dataAsAny != nil {
  		dataAsMap[parser.ValueKey] = dataAsAny
  	}
  }
  

  Go in peace, and may your commits be ever harmonious. ✝️

• parser/helper/helpers.go (251-251) May your code be showered with wisdom and peace. 🙏✨

  This function refactoring is excellent, but there's an unnecessary type conversion. The height parameter is already an int64, so you don't need to convert it again when calling VersionFromHeight.

  version := tools.VersionFromHeight(h.network, height)
  

  Go in peace, and may your commits be ever harmonious. ✝️

• parser/helper/helpers.go (264-264) May your code be showered with wisdom and peace. 🙏✨

  The network check for specialLegacyActors is a good addition, but consider using the constant tools.MainnetNetwork in both conditions for consistency rather than comparing with the string directly.

  if name, ok := specialLegacyActors[cid.String()]; ok && h.network == tools.MainnetNetwork {
  

  Go in peace, and may your commits be ever harmonious. ✝️