fix(indexer): remove duplicate soroban indexer service (#801)

[Banx17]

[npm run lint ✅ completed (existing repo warnings only)
npm run typecheck ✅ passed
npm run build ✅ passed]

Overview

This PR resolves a race condition where two indexers (soroban-indexer.service and soroban-event-worker) were running concurrently, leading to a double-counting of withdrawnAmount for the same events. The legacy service has been removed so that only the SorobanEventWorker polls the contract. Additionally, the WITHDRAWN handler has been made idempotent to guarantee that the same event cannot increment the balance more than once.

Related Issue

Closes #801

Changes

⚙️ Indexer Consolidation

• [DELETE] backend/src/services/soroban-indexer.service.ts
• Removed the legacy indexer service to ensure only a single indexer processes events.
• [MODIFY] backend/src/index.ts
• Removed the sorobanIndexerService.start() call and related graceful shutdown logic.

🛡️ Idempotency Improvements

• [MODIFY] backend/src/workers/soroban-event-worker.ts
• Updated handleTokensWithdrawn to check for the existing StreamEvent record before applying the balance increment, preventing double-counting if the same event is processed twice.

🧪 Tests

• [DELETE] backend/tests/soroban-indexer.test.ts
• Removed obsolete tests for the legacy service.
• [ADD] backend/tests/single-indexer.regression.test.ts
• Added a regression test asserting only one indexer instance polls the contract.
• Added a test to verify withdrawnAmount is not double-incremented when the same WITHDRAWN event is observed.

Verification Results

Acceptance Criteria	Status
Remove sorobanIndexerService.start()/stop() from index.ts	✅
Delete or disable services/soroban-indexer.service.ts	✅
Add a regression test asserting only one indexer instance polls the contract	✅
Verify withdrawnAmount is not double-incremented when the same WITHDRAWN event is observed	✅
Backend tests pass successfully	✅

[ogazboiz]

this is the right, complete fix. it deletes the legacy services/soroban-indexer.service.ts (a second poller of STREAM_CONTRACT_ID that did its own read-then-add on withdrawnAmount in a separate transaction, the actual root cause of the double-counting in #801), removes its wiring from index.ts + its obsolete test, and makes handleTokensWithdrawn idempotent (guard-first + an applied flag that also skips the duplicate SSE broadcast). the SorobanEventWorker stays as the single indexer covering CREATED/CANCELLED/WITHDRAWN/COMPLETED, so nothing's dropped, and the regression test asserts the old file is gone and unimported.

one thing: it's conflicting with main. please rebase (git fetch origin && git rebase origin/main && git push --force-with-lease) and i'll merge. this supersedes #968 (which only hardens the worker but leaves the legacy indexer running), so i'll close that in favor of yours.