Skip to content

Stabilize pagination and batching for RPC retries, fee quotes, and chain interactions in src/starknet/client.ts #376

Description

@Jagadeeshftw

Description

The batching or pagination contract needs a tighter definition to stay safe under growth. src/starknet/client.ts owns RPC retries, fee quotes, and chain interactions, but the current behavior does not yet define the contract tightly enough for safe maintenance. Design the change so the runtime path, tests, and docs all describe the same behavior.

Requirements and context

  • Keep the contract specific to src/starknet/client.ts and its existing callers.
  • Add coverage for the success path and the failure or boundary path relevant to this issue.
  • Preserve compatibility where older callers or existing tests already rely on the current shape.
  • Keep the change easy to review and small enough for one focused PR.

Suggested execution

Fork the repo and create a branch

git checkout -b starknet-client-batching-190

Implement changes

  • Update/Write: src/starknet/client.ts
  • Write comprehensive tests: src/starknet/client.test.ts
  • Add documentation: docs/starknet/client.md
  • Keep any new helper names consistent with the rest of src/

Test and commit

  • Run pnpm test
  • Run pnpm lint
  • Add a note for any edge cases that are intentionally out of scope

Example commit message

fix: stabilize pagination and batching behavior

Guidelines

  • WCAG 2.1 AA is not relevant here; keep the focus on correctness and maintainability.
  • Prefer a narrow, well-documented fix over a broad refactor.
  • Timeframe: 96 hours

Metadata

Metadata

Assignees

Labels

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions