Skip to content

[SDK] Handle subgraph errors properly #3782

Merged
flopez7 merged 4 commits intodevelopfrom
feat/sdk/subgraph-errors
Feb 18, 2026
Merged

[SDK] Handle subgraph errors properly #3782
flopez7 merged 4 commits intodevelopfrom
feat/sdk/subgraph-errors

Conversation

@flopez7
Copy link
Contributor

@flopez7 flopez7 commented Feb 11, 2026

Issue tracking

Close #3745

Context behind the change

  • Added typed subgraph errors in TS (SubgraphRequestError, SubgraphBadIndexerError)
  • Updated exception filters to avoid logging bad indexers as “Unhandled exception”
  • Other subgraph failures are still logged, but under a dedicated subgraph error path

How has this been tested?

Ran unit tests

Release plan

  • Deploy new SDK version.
  • Add new monitor+alert on DataDog based on new logs

Potential risks; What to monitor; Rollback plan

None

…update exception filters to handle subgraph errors
@flopez7 flopez7 requested review from dnechay and portuu3 February 11, 2026 14:37
@flopez7 flopez7 self-assigned this Feb 11, 2026
@vercel
Copy link

vercel bot commented Feb 11, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

5 Skipped Deployments
Project Deployment Actions Updated (UTC)
faucet-frontend Ignored Ignored Preview Feb 18, 2026 10:54am
faucet-server Ignored Ignored Preview Feb 18, 2026 10:54am
human-app Skipped Skipped Feb 18, 2026 10:54am
human-dashboard-frontend Skipped Skipped Feb 18, 2026 10:54am
staking-dashboard Skipped Skipped Feb 18, 2026 10:54am

Request Review

Copy link
Collaborator

@dnechay dnechay left a comment

Choose a reason for hiding this comment

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

LGTM overall, one suggestion

Also please add to release plan that we need to add new monitor+alert on DataDog based on new logs

@vercel vercel bot temporarily deployed to Preview – staking-dashboard February 16, 2026 15:20 Inactive
@vercel vercel bot temporarily deployed to Preview – human-dashboard-frontend February 16, 2026 15:20 Inactive
@vercel vercel bot temporarily deployed to Preview – human-app February 16, 2026 15:20 Inactive
@flopez7 flopez7 requested a review from dnechay February 16, 2026 15:21
dnechay
dnechay previously approved these changes Feb 16, 2026
@vercel vercel bot temporarily deployed to Preview – human-dashboard-frontend February 18, 2026 10:54 Inactive
@vercel vercel bot temporarily deployed to Preview – human-app February 18, 2026 10:54 Inactive
@vercel vercel bot temporarily deployed to Preview – staking-dashboard February 18, 2026 10:54 Inactive
@flopez7 flopez7 merged commit a31cfc8 into develop Feb 18, 2026
13 checks passed
@flopez7 flopez7 mentioned this pull request Feb 18, 2026
25 tasks
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.

Exclude noisy subgraph errors from being logged as part of "unhandled exception" logging

3 participants