fix: Adjust block access service to be correct without workarounds

[jsync-swirlds]

Reviewer Notes

• Modified the proto definition to remove unverified and prevent calling with both latest and a block number.
• Added an "invalid" response and removed the verified/unverified responses
• Removed unnecessary prefixes in the response enum names
• Removed check for both fields set in Java (it's no longer possible).
• Fixed up all references
• Removed tests that set both fields as it's no longer possible (the last one set is the only one set).

Related Issue(s)

Fixes #1148

[Nana-EC]

1st pass looks looks good on simplifying the logic

[Nana-EC]

LG

[AlfredoG87]

LGTM

fun fact:

you can actually request the latest block either way:

{
    "retrieve_latest": true
}

{
    "block_number": "18446744073709551615"
}

[jsync-swirlds]

LGTM

fun fact:

you can actually request the latest block either way:

Yes, I thought it might smooth the change if either works. We can always make the small change so it only works with the flag set in a later update.

[codecov]

Codecov Report

Attention: Patch coverage is 45.45455% with 6 lines in your changes missing coverage. Please review.

Files with missing lines	Patch %	Lines
.../node/access/service/BlockAccessServicePlugin.java	45.45%	6 Missing ⚠️

❌ Your patch status has failed because the patch coverage (45.45%) is below the target coverage (80.00%). You can increase the patch coverage or adjust the target coverage.

@@             Coverage Diff              @@
##               main    #1147      +/-   ##
============================================
+ Coverage     84.48%   84.66%   +0.18%     
+ Complexity      919      915       -4     
============================================
  Files           104      104              
  Lines          3725     3717       -8     
  Branches        400      398       -2     
============================================
  Hits           3147     3147              
+ Misses          427      418       -9     
- Partials        151      152       +1     

Files with missing lines	Coverage Δ	Complexity Δ
.../node/access/service/BlockAccessServicePlugin.java	77.58% <45.45%> (-4.24%)	8.00 <0.00> (-3.00)

... and 2 files with indirect coverage changes

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.