Refactored grant related code for readability

[thehenrytsai]

• Turned RecordsGrantAuthorization.authorizeQuery/Read/Write parameters into input instead for readability
• Renamed authorize() to authorizeOwner()
• Renamed authorizeProtocolRecord() to verifyProtocolRecordScope()
• Renamed authorizeFlatRecord() to verifyFlatRecordScope()
• Renamed GrantAuthorization.authorizeGenericMessage() to GrantAuthorization.performBaseValidation()
• Renamed validateMessageSignatureIntegrity() to validateSignatureStructure()
• Introduced RecordsPermissionsGrantMessage for readability

[codesandbox]

Review or Edit in CodeSandbox

Open the branch in Web Editor • VS Code • Insiders

Open Preview

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (41d3550) 98.82% compared to head (365935e) 98.83%.

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #647   +/-   ##
=======================================
  Coverage   98.82%   98.83%           
=======================================
  Files          71       71           
  Lines        9181     9240   +59     
  Branches     1391     1391           
=======================================
+ Hits         9073     9132   +59     
  Misses        100      100           
  Partials        8        8           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[LiranCohen]

This is great! Definitely an improvement for readability without keeping too much context in certain areas.