Warning: This build has drifted.
The coverage report for this pull request build may be inaccurate because its base commit is no longer the HEAD of its target branch.
This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

    • Learn more: For more information on this, see Tracking coverage changes for pull request builds.
    • Fix now: For a quick fix, rebase this PR at GitHub. Your next report should be accurate.
    • Prevent going forward: To avoid this issue with future PRs, see these Recommended CI Configurations.

Address review comments

To support the fuzz contract:
- extractField now returns the last match for duplicate fields, matching
  proto3's last-writer-wins semantics for scalar fields. Note that proto.Marshal
  would not add duplicate fields but proto.Unmarshal parses them correctly even
  if there are duplicate fields.
- Remove the "advance past field 1" optimization in ChannelHeader parsing
  since last-writer-wins requires searching from the beginning for each field.
  Example bytes (field order): field5, field1, field5, field1
  - Last field 1 is at position 4. Advancing past it means we only search positions 5+ for field 5.
  - But the last field 5 is at position 3 — we'd miss it.

Signed-off-by: Senthilnathan <cendhu@gmail.com>

fix all bugs

Signed-off-by: Senthilnathan <cendhu@gmail.com>