Skip to content

Conversation

@weifangc
Copy link

This change replaces occurrences of interface{} with the predeclared identifier any, introduced in Go 1.18 as an alias for interface{}.

As noted in the Go 1.18 Release Notes:
This improves readability and aligns the codebase with modern Go conventions.

@weifangc weifangc requested review from a team as code owners November 18, 2025 06:34
@greptile-apps
Copy link
Contributor

greptile-apps bot commented Nov 18, 2025

Greptile Summary

  • Replaces all occurrences of interface{} with the any predeclared identifier across ~28 files to modernize the codebase and align with Go 1.18+ conventions

Important Files Changed

This PR contains only low-risk modernization changes with no files requiring special attention. All changes replace interface{} with any which are functionally identical.

Confidence score: 5/5

  • This PR is extremely safe to merge with zero risk of breaking functionality or introducing bugs
  • Score reflects that any is literally an alias for interface{} with identical runtime behavior, making this a purely cosmetic modernization with no functional changes
  • No files require special attention as all changes are syntactic improvements with zero behavioral impact

Sequence Diagram

sequenceDiagram
    participant User
    participant API
    participant Bind
    participant Committee
    participant Queue
    participant Runner
    participant BeaconNode

    User->>API: "HTTP Request"
    API->>Bind: "Bind(request, dest)"
    Bind->>Bind: "Parse form data"
    Bind->>API: "Bound struct"
    
    API->>Committee: "EnqueueMessage(msg)"
    Committee->>Queue: "TryPush(message)"
    Queue->>Committee: "Message queued"
    
    Committee->>Committee: "ConsumeQueue()"
    Committee->>Queue: "Pop(message)"
    Queue->>Committee: "Prioritized message"
    
    Committee->>Runner: "ProcessMessage(msg)"
    Runner->>Runner: "Process consensus/partial signature"
    Runner->>BeaconNode: "Submit to beacon chain"
    BeaconNode->>Runner: "Submission result"
    
    Runner->>Committee: "Processing complete"
    Committee->>API: "Response"
    API->>User: "HTTP Response"
Loading

Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

29 files reviewed, no comments

Edit Code Review Agent Settings | Greptile
React with 👍 or 👎 to share your feedback on this new summary format

Copy link
Contributor

@nkryuchkov nkryuchkov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for the contribution

@weifangc
Copy link
Author

@nkryuchkov My pleasure. If there's anything need I to do, please feel free to let me know.

Copy link
Contributor

@iurii-ssv iurii-ssv left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice!

@y0sher y0sher merged commit eb2cd44 into ssvlabs:stage Dec 3, 2025
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants