feat(indexer): fetch v3.5.0 and v1.4.0 events from their upgrade blocks - #123
Merged
Merged
Conversation
Duplicate contract addresses but for the new versions so we can start from their upgrade blocks rather than from the complete START.
rvagg
force-pushed
the
rvagg/upgrade-start-blocks
branch
from
September 10, 2026 12:48
857ebfd to
c1d0f39
Compare
There was a problem hiding this comment.
🟢 Approval recommended
The configuration, handler retargeting, and added tests are consistent and verify the intended start-block behavior for the upgraded event ranges.
Pull request overview
This PR updates the indexer configuration and event handlers to start indexing newly introduced events (PDPVerifier v3.5.0 and FWSS v1.4.0) from their respective upgrade blocks, while keeping the proxy addresses unchanged.
Changes:
- Added per-network “upgrade block” constants for PDPVerifier v3.5.0 and FWSS v1.4.0.
- Split Ponder contract entries (same address, later
startBlock) and retargeted the relevant event handlers to those new entries. - Added a test to assert the upgrade entries share the parent address and start after the parent (and in the expected order).
File summaries
| File | Description |
|---|---|
| indexer/ponder.config.ts | Adds PDPVerifierV35 and FWSSV14 contract entries that start at upgrade blocks while sharing the parent addresses. |
| indexer/src/networks.ts | Introduces PDP_VERIFIER_V3_5_BLOCK and FWSS_V1_4_BLOCK in the network type and per-network constants. |
| indexer/src/pdp-verifier.ts | Retargets v3.5.0-only events (PiecesAddedV2, PiecesScheduledForRemoval) to PDPVerifierV35. |
| indexer/src/fwss.ts | Retargets the v1.4.0-only event (DataSetAuthorizerSet) to FWSSV14. |
| indexer/test/networks.test.ts | Updates expected network registry objects to include the new upgrade block constants. |
| indexer/test/upgrade-entries.test.ts | Adds coverage ensuring upgrade entries share addresses but start later than their parent entries. |
Review details
- Files reviewed: 6/6 changed files
- Comments generated: 0
- Review effort level: Lite
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Duplicate contract addresses but for the new versions so we can start from their upgrade blocks rather than from the complete START.