Skip to content

Label RPC errors by network, and export the chain tip #188

Open

Description

@Miracle656

Background

rpcErrorsTotal (src/metrics.ts:58-63) has only an outcome label, and recordRpcError is called from src/rpc.ts:167, 170 without a network. With two loops running you cannot tell which chain's provider is failing — which is the entire point of the metric.

Separately, last_indexed_ledger is exported (src/metrics.ts:71-76) but the chain tip is not, so Prometheus cannot compute indexer lag. /status computes it (src/api.ts:445), but that is a JSON endpoint, not a scrape target.

Acceptance criteria

  • rpc_errors_total gains a network label, threaded from the withRetry callers
  • A chain_tip_ledger{network} gauge is set on each successful tip read
  • The README /metrics sample block is updated
  • Tests assert the new labels and series appear in renderMetrics() — fails before

Drips Wave · Complexity: Easy · 100 points


How this repo is laid out

Wraith is a Soroban incoming-transfer indexer: it fills the gap Horizon leaves for SAC/SEP-41 token events by recipient address. Roughly 14k lines under src/, REST + GraphQL + WebSocket, Postgres via Prisma, deployed on Render.

  • Unit tests are jest (src/**/__tests__, tests/*.test.ts), with a coverage gate.
  • Integration tests are vitest (tests/integration/, vitest.integration.config.ts, Docker).
  • Don't mix the two runners in one file.

Known state of main before you start

tsc --noEmit and the unit suite are green. The Integration tests CI job is red, and so is Load test — both predate this batch. Open PRs #177 and #178 address them. If the integration job is red when you open a PR, that is not your fault; say so in the PR and carry on.

Four modules are shipped, closed as done, and imported by nothing: src/indexer/checkpoint.ts, src/ingest/reorg.ts, src/middleware/opa.ts, and src/api/candles.ts + src/workers/ohlc-refresh.ts. So the live indexer today has no reorg handling and no atomic checkpointing, despite both features being closed. Two issues below start unpicking that — don't assume a closed issue means a wired feature.

Ground rules

  • Every PR needs a test that fails before the change and passes after, unless the issue says otherwise.
  • Don't widen a Prometheus label to something unbounded — addresses and contract ids must never become label values.
  • Nothing may log or return a raw database error, a connection string, or a provider URL.
  • Don't reformat a file you're changing. A diff that is 90% whitespace hides the 10% that matters.


Required: Before submitting, join the contributor Telegram so your work can be tracked and counted toward the Stellar Wave: https://t.me/+fxHXq8f1SwlkZDBk

Activity

  1. N-otorious commented on Sep 24, 2026

    @N-otorious
    Contributor

    @N-otorious has applied to work on this issue as part of the Stellar Wave Program's 9th wave.

    Would like to work on this issue, assign it to me please.

    ℹ️ Repo Maintainers: To accept this application, review their application or assign @N-otorious to this issue.

  2. biqqest commented on Sep 24, 2026

    @biqqest

    @biqqest has applied to work on this issue as part of the Stellar Wave Program's 9th wave.

    I will love to work on this issue please

    ℹ️ Repo Maintainers: To accept this application, review their application or assign @biqqest to this issue.

  3. collinsezedike commented on Sep 28, 2026

    @collinsezedike
    Contributor

    @collinsezedike has applied to work on this issue as part of the Stellar Wave Program's 9th wave.

    The RPC error metric lacks a network label, so operators cannot tell which chain's provider is failing when both networks run, and the chain tip gauge is missing entirely so Prometheus cannot compute indexer lag. The fix requires threading network context through the retry callers and exposing a chain tip gauge per network.

    Here's my plan:

    • Add network label to rpc_errors_total metric definition
    • Thread network label through withRetry callers in src/rpc.ts
    • Add chain_tip_ledger gauge with network label set on each successful tip read
    • Update README /metrics sample block with the new labels and series
    • Add tests asserting new labels and series appear in renderMetrics output

    Happy to start as soon as I'm assigned

    ℹ️ Repo Maintainers: To accept this application, review their application or assign @collinsezedike to this issue.

  4. drips-wave commented on Sep 29, 2026

    @drips-wave

    Congratulations, @biqqest! 🎉 Your application was accepted by the repo's maintainers, and the issue is due on September 30, 2026.

    🧑‍💻 @biqqest: Please resolve the issue such that the repo's maintainers have enough time to review your contribution before the due date. You'll earn Points for completing the issue on-time, which will make you eligible for a share of the Stellar Wave Program's reward pool.

    Warning

    When opening a PR, please link it to this issue to ensure it gets tracked accurately. Points are awarded when this issue is marked as completed by the maintainer.

    🤠 Repo maintainers: Please keep an eye on the contributor's progress and review their work before the due date. You can manage this issue, including adjusting its complexity and points, here.

    🌊 Happy Wave 🌊

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions