Skip duplicate blocks in committer

[iuwqyir]

TL;DR

Added duplicate block handling in the committer and comprehensive test coverage for the committer component.

What changed?

• Added logic to skip duplicate blocks in the sequential block processing
• Created new test file committer_test.go with extensive test coverage
• Added test cases for duplicate block handling, gap detection, and committer functionality
• Renamed reorg handler test to avoid naming conflicts

How to test?

1. Run the test suite with go test ./internal/orchestrator
2. Verify all committer tests pass, including:
   • Duplicate block handling
   • Sequential block processing
   • Gap detection
   • Committer initialization
   • Block commit operations

Why make this change?

The committer component previously lacked test coverage and could start filling gaps when encountering duplicate blocks. This change improves reliability by properly handling duplicate blocks and adds comprehensive testing to ensure the committer behaves correctly under various scenarios.

[iuwqyir]

• Skip duplicate blocks in committer #112 👈
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @iuwqyir and the rest of your teammates on Graphite