hiero-ledger / hiero-ledger/hiero-consensus-node

Weird protobuf field names in virtual_map_state.proto

Open
#21,023 0 comments 0 reactions 0 assignees View on GitHub
Tech Debt Reduced
Dominant language
Java
Stars
406
Forks
226
Avg merge
3d 4h
Merged PRs (30d)
210

Description

Some field names in `virtual_map_state.proto` are weird, they are a mix of pascal, snake, and upper case notations. Usually protobuf names are snake, for example, `TokenService_I_ALIASES` should be changed to `tokenservice_i_aliases`.

Classes generated from `virtual_map_state.proto` should not be affected, since everything is capitalized anyway, but there may be other places where the field names pop up. These places should be updated accordingly.

Contributor guide

Open the contributing guide

Research direction

Start with virtual_map_state.proto and inspect all field declarations with mixed casing, including TokenService_I_ALIASES. Search the repository for those field names and update affected references while leaving generated class names unchanged. Done means the protobuf fields use consistent snake_case and all remaining references match.

Written by the indexing model from the issue text.

Assessment

Tech stack
java
Domain
backend
Issue type
Refactor
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
48/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.