Skip to content

fix: envio indexer config + separate per-reader checkpoint tracking#650

Merged
preethamr merged 4 commits intomainnet-prodfrom
fix/envio-indexer-config
Apr 2, 2026
Merged

fix: envio indexer config + separate per-reader checkpoint tracking#650
preethamr merged 4 commits intomainnet-prodfrom
fix/envio-indexer-config

Conversation

@otsybizov
Copy link
Copy Markdown
Contributor

🤖 Linear

Closes CONG-XXX

Copilot AI review requested due to automatic review settings April 2, 2026 00:38
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the Envio indexer configuration to capture gasLimit and refactors Cartographer/Subgraph reading to track and persist checkpoints per reader implementation (e.g., Goldsky vs Envio), with corresponding test updates.

Changes:

  • Add gasLimit capture in the Envio indexer (field_selection, event handler metadata extraction, and GraphQL schema fields).
  • Introduce per-reader checkpoint load/save utilities and update Cartographer operations to use reader-specific cursors.
  • Extend the subgraph adapter composite reader to return merged results along with per-reader checkpoint updates; update unit tests/mocks accordingly.

Reviewed changes

Copilot reviewed 18 out of 18 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
packages/indexer/src/EventHandlers.ts Populate gasLimit from transaction metadata.
packages/indexer/schema.graphql Expose gasLimit on multiple entities for parity.
packages/indexer/config.yaml Include gasLimit in Envio transaction_fields selection.
packages/agents/cartographer/poller/test/mock.ts Update subgraph adapter mocks for new *WithCheckpoints APIs and reader types.
packages/agents/cartographer/poller/test/lib/operations/monitor.spec.ts Adjust monitor tests for per-reader checkpoint calls.
packages/agents/cartographer/poller/test/lib/operations/invoices.spec.ts Adjust invoice/deposit tests for per-reader checkpoint saving semantics.
packages/agents/cartographer/poller/test/lib/operations/intents.spec.ts Update intents tests to use new checkpoint-returning reader APIs.
packages/agents/cartographer/poller/test/lib/operations/depositors.spec.ts Update depositor tests for per-reader checkpoints.
packages/agents/cartographer/core/src/operations/monitor.ts Use per-reader checkpoints for message polling and persistence.
packages/agents/cartographer/core/src/operations/invoices.ts Use per-reader checkpoints for hub invoices/deposits operations.
packages/agents/cartographer/core/src/operations/intents.ts Use per-reader checkpoints for origin/destination/settlement intents and orders.
packages/agents/cartographer/core/src/operations/helper.ts Treat domains as supported if Envio is available (in addition to Goldsky URLs).
packages/agents/cartographer/core/src/operations/depositors.ts Use per-reader checkpoints for depositor event ingestion.
packages/agents/cartographer/core/src/operations/checkpoints.ts New helper module to load/save checkpoints keyed by reader type.
packages/adapters/subgraph/src/reader.ts Add readerType, add *WithCheckpoints APIs, and implement per-reader checkpoint merging helpers.
packages/adapters/subgraph/src/lib/types/subgraph.ts Introduce ReaderCheckpoints type.
packages/adapters/subgraph/src/graph/reader.ts Set Graph reader readerType to goldsky.
packages/adapters/subgraph/src/envio/reader.ts Set Envio reader readerType to envio.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread packages/adapters/subgraph/src/reader.ts Outdated
Comment thread packages/agents/cartographer/core/src/operations/checkpoints.ts
Comment thread packages/agents/cartographer/core/src/operations/monitor.ts
@preethamr preethamr merged commit fcc0bb3 into mainnet-prod Apr 2, 2026
23 checks passed
@preethamr preethamr deleted the fix/envio-indexer-config branch April 2, 2026 03:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants