Skip to content

fix: propagate pending user operation RPC errors#366

Open
realmehmetali wants to merge 1 commit into
base:masterfrom
realmehmetali:agent/fix-userop-lookup-errors
Open

fix: propagate pending user operation RPC errors#366
realmehmetali wants to merge 1 commit into
base:masterfrom
realmehmetali:agent/fix-userop-lookup-errors

Conversation

@realmehmetali

@realmehmetali realmehmetali commented Jul 18, 2026

Copy link
Copy Markdown

Summary

  • detect JSON-RPC errors returned by eth_getUserOperationByHash
  • propagate them instead of reporting the payment as not_found
  • add regression coverage for the error and telemetry paths

Why

When the receipt lookup returned null, the follow-up user-operation lookup ignored its JSON-RPC error field. A bundler failure could therefore be misreported as a missing operation.

Agent impact

Autonomous agents need to distinguish a genuinely absent user operation from an RPC failure before deciding whether to retry, alert, or advance a payment workflow. Accurate failure semantics are foundational for safe agent-run payment loops.

Validation

  • focused payment-status suite: 20 tests passed
  • package typecheck
  • package build
  • Biome check on changed files

@cb-heimdall

cb-heimdall commented Jul 18, 2026

Copy link
Copy Markdown
Collaborator

🟡 Heimdall Review Status

Requirement Status More Info
Reviews 🟡 0/1
Denominator calculation
Show calculation
1 if user is bot 0
1 if user is external 0
2 if repo is sensitive 0
From .codeflow.yml 1
Additional review requirements
Show calculation
Max 0
0
From CODEOWNERS 0
Global minimum 0
Max 1
1
1 if commit is unverified 0
Sum 1

@realmehmetali
realmehmetali force-pushed the agent/fix-userop-lookup-errors branch from 6d4a3da to bf986b9 Compare July 18, 2026 13:45
@realmehmetali
realmehmetali marked this pull request as ready for review July 18, 2026 14:05
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.

2 participants