Address PR 32 review comments

[Copilot]

Addresses review feedback on PR #32 around partial range validation, duplicated tree cleanup/move logic, and coverage test diagnostics.

• Range validation

  • Simplifies partial serialization range checks into one ordered bounds expression:

    min_index() <= from && from <= to && to <= max_index()

• Tree ownership cleanup

  • Extracts repeated TreeT teardown into clear().
  • Extracts move-transfer/reset behavior into move_from().
  • Reuses both helpers from move construction, move assignment, copy assignment cleanup, destruction, and deserialization reset paths.

• Coverage diagnostics

  • Sends coverage test error output to stderr instead of stdout.

[achamayou]

@copilot fix the clang tidy errors in https://github.com/microsoft/merklecpp/actions/runs/27033275945/job/79790938759?pr=34

[Copilot]

@copilot fix the clang tidy errors in https://github.com/microsoft/merklecpp/actions/runs/27033275945/job/79790938759?pr=34

Fixed in c182cd5. I also validated locally with the CI-style clang-tidy header check and tests.

[Copilot]

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.