Skip to content

Make Soroban confirmation polling configurable - #246

Merged
Jaydbrown merged 1 commit into
conduit-protocol:mainfrom
louiss72:agent/configurable-polling-217
Jul 29, 2026
Merged

Jaydbrown merged 1 commit into
conduit-protocol:mainfrom
louiss72:agent/configurable-polling-217

Conversation

@louiss72

Copy link
Copy Markdown
Contributor

Summary

  • add confirmation polling defaults to ConduitConfig with current behavior preserved (1000ms, 30 attempts)
  • add ConfirmationPollingOptions to low-level invokeContract()
  • make StreamsModule._sendAndPoll() honor the client-level polling config
  • add regression coverage for both stream mutations and direct invokeContract() polling options

Closes #217.

Validation

  • npm test -- src/tests/streams-success.test.ts src/tests/soroban-rate-limit.test.ts -> 2 files / 17 tests passed
  • npm run build -> passed
  • npm run lint -> passed
  • git diff --check -> passed
  • npm run typecheck is blocked by an existing WalletConnect edge-case test issue unrelated to this change: chainId: undefined is not assignable with exactOptionalPropertyTypes
  • npm test is blocked by an existing pagination cursor expectation in src/tests/pagination-cursor.test.ts; the focused polling suites pass

Payout

This is Stellar Wave / USDC-compatible. EVM/Base USDC address: 0x2f8081562ac67467d1cbd40ab3120849c1f587da.

If payment must be Stellar-native USDC, please ask for a Stellar public key before sending.

@Jaydbrown
Jaydbrown force-pushed the agent/configurable-polling-217 branch from ada6bd3 to e333201 Compare July 29, 2026 10:31
@Jaydbrown
Jaydbrown merged commit bd5f70a into conduit-protocol:main Jul 29, 2026
1 of 2 checks passed
@louiss72

Copy link
Copy Markdown
Contributor Author

Thanks for merging this and the related streamFi-sdk PRs (#238, #240, #242, #243, #245, #246).

Could you confirm the payout process for these merged Stellar Wave items?

Preferred payout options:

  • USDC / USDT / ETH on EVM-compatible chains: 0x2f8081562ac67467d1cbd40ab3120849c1f587da
  • XMR: 4DSQMNzzq46N1z2pZWAVdeA6JvUL9TCB2bnBiA3ZzoqEdYJnMydt5akCa3vtmapeDsbVKGPFdNkzqTcJS8M8oyK7WGjNk99k1B6CoKjvH5
  • SOL / Solana assets: 2h5Qhgw16SLyzpwwfKV53gmS9PiXyRkME8oavhsgRBU6

If Stellar-native payout is required, please let me know and I will provide a Stellar public key.

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.

Enhancement: make RPC confirmation-polling interval/attempts configurable

3 participants