Skip to content

Cover indexer reorg final state rewrites - #385

Open
kitWarse wants to merge 2 commits into
SO4-Markets:mainfrom
kitWarse:test/336-indexer-reorg-final-state
Open

Cover indexer reorg final state rewrites#385
kitWarse wants to merge 2 commits into
SO4-Markets:mainfrom
kitWarse:test/336-indexer-reorg-final-state

Conversation

@kitWarse

Copy link
Copy Markdown

Summary

  • Added reorg fixtures for an orphaned ledger sequence and its canonical replacement.
  • Updated deterministic IDs for market config snapshots and position changes so canonical replay for the same ledger rewrites affected records instead of leaving stale duplicates.
  • Added mapping test coverage that replays orphaned events, replays canonical events twice, and asserts the final position, position change, and order state is deterministic.

Issue

Changes

  • Keyed market config snapshots by market key and ledger.
  • Keyed position changes by position key, ledger, and change type.
  • Added
    eorged-ledger-sequence.json fixture data.
  • Added assertions for stale value replacement and stable final entity counts.

Validation

  • Attempted: �un test apps/s03-indexer/tests/mappingHandlers.test.ts
  • Result: blocked locally because @stellar/stellar-sdk is not installed in this fresh clone.

Notes

  • Full CI/package validation was not run locally per request.
  • No dependency or lockfile changes were committed.

@vercel

vercel Bot commented Jul 25, 2026

Copy link
Copy Markdown

@kitWarse is attempting to deploy a commit to the Ijai's projects Team on Vercel.

A member of the Team first needs to authorize it.

@drips-wave

drips-wave Bot commented Jul 25, 2026

Copy link
Copy Markdown

@kitWarse Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

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.

Indexer: ledger reorg handling

1 participant