fix(indexer): unblock testnet backfill#74
Conversation
|
Caution Review failedPull request was closed or merged during review 📝 WalkthroughWalkthroughThis PR enhances the indexer's block-handling and error recovery by introducing RPC block parsing support, rewriting the chain client's getBlock to use raw JSON-RPC, and integrating missing-block error handling into the sync loop with monotonic height tracking. It adds a 10-second timeout to getNativeTransaction, improves 429 retry detection, and ensures last_synced_height advances monotonically via GREATEST comparisons. Configuration changes include making the Docker healthcheck port configurable and updating the testnet compose file to use host networking with explicit localhost endpoints for RPC and Postgres connections. Estimated code review effort🎯 3 (Moderate) | ⏱️ ~22 minutes 🚥 Pre-merge checks | ✅ 3 | ❌ 2❌ Failed checks (2 warnings)
✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Warning There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure. 🔧 ESLint
ESLint skipped: no ESLint configuration detected in root package.json. To enable, add Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Summary
Test plan
Related
Summary by CodeRabbit
New Features
Bug Fixes