[isaacdoidge]

What

#36 was squash-merged into #30's branch (fix/paging-and-request-params) after #30 had been merged, so its changes
are not on master. This PR applies them to master: its first commit is #36's squash commit (0b39ac1),
cherry-picked onto master.

The two commits after it do the work that was left over when #30 and #31 were both merged:

• account_fa_transactions reads its cursor with paging.next_cursor. feat(client): add fungible asset balance, transfer, migration and transaction helpers #31 had a private copy of the same rule in
  account.py, which is removed. The two functions return the same result for every input.
• iter_account_fa_transactions is added, like the other iterators.
• account_fa_transactions and account_coin_transactions treat an empty transaction_type the same way: no type
  parameter is sent.
• account_coin_transactions says that feature 64 or feature 65 puts SUPRA in primary stores.

Behaviour

From #36 (unchanged; its description has the details):

• account_history(account_address, pagination=None, categories=None) and iter_account_history for
  GET /rpc/v3/accounts/{address}/history:
  • Returns (entries, cursor). Each entry is {"categories": [...], "transaction": {...}}.
  • categories takes AccountHistoryCategory members or their names (sent, coin, fa, da). An empty,
    unknown or comma-joined value raises ValueError before any request.
  • The feed ends only when the response has no x-supra-cursor: "0" is a valid cursor, and an empty page with a
    cursor continues. The iterator also stops on a repeated cursor.
  • Needs a node from supra_node_v12.0.0. A v11.7.x node answers 404, raised as ApiError.
  • The method does not return the x-supra-history-indexed-from header.
• transaction_type on account_coin_transactions and iter_account_coin_transactions:
  • Sent as the node's type query parameter. TransactionType gains RECORD and SETTLEMENT. settlement needs
    a v12 node; a v11.7.x node answers 400.
  • With a filter, the first request of an ascending read keeps a "0" cursor on an empty page, which costs at most
    one more request.
• paging.next_cursor gains the keyword only_missing_header_ends (default False), used by the history feed.
• Docs: a README section says which method lists SUPRA movements on each SUPRA layout; the coin-transaction
  docstrings point to account_fa_transactions and account_history.

New here:

• iter_account_fa_transactions(account_address, pagination=None, metadata_address=0xa, transaction_type=None, *, store_address=None): the arguments of account_fa_transactions. It follows the cursor until the feed ends or
  the node repeats one. An invalid address raises ValueError when the iteration starts.
• account_fa_transactions sends no type parameter for an empty transaction_type, and then does not apply
  the "0" rule for typed ascending reads. Before, it sent type= with an empty value.

Checks

• The cherry-pick had one conflict, in the import list of account.py (feat(rest): account history, a transaction type filter on coin transactions, and SUPRA layout docs #36's ACCOUNT_HISTORY_ENDPOINT and feat(client): add fungible asset balance, transfer, migration and transaction helpers #31's
  ACCOUNT_FA_TRANSACTIONS_ENDPOINT); both are kept. paging.py and tests/unit/test_paging.py are identical to
  0b39ac1, and the other files differ from it only by feat(client): add fungible asset balance, transfer, migration and transaction helpers #31's additions.
• A fourth commit corrects a node-backed test from test: run the node-backed tests and the automation example on v11.7 and v12 nodes #37, written before fix(rest)!: end paged feeds at a missing cursor, add iterators, correct request parameters #30 merged: an ended
  account_coin_transactions feed now reports its cursor as None, not "". CI skips the node-backed tests, so
  master did not show it.
• make fmt-check, make lint, make typecheck, make test-spec (248 scenarios), make build: pass.
• make test-unit: 350 tests, OK (26 node-backed skipped). New offline tests cover iter_account_fa_transactions
  ending on a missing header, continuing past an empty filtered page, and with store_address; and an empty
  transaction_type on account_fa_transactions.
• On a fresh supra_node_v12.0.0.rc7 localnet:
  • make test-unit with SUPRA_RPC_URL set: 350 tests, OK, none skipped.
  • With an account funded by the faucet that then transferred SUPRA, account_history listed the transfer as
    sent and fa and the faucet grant as fa; the receiver's history listed the transfer under fa and nothing
    under coin. Both history iterators read to the end. account_coin_transactions returned empty pages with
    cursor None for both accounts with every filter, settlement included.
• The v11.7.2 checks in feat(rest): account history, a transaction type filter on coin transactions, and SUPRA layout docs #36 (404 for /history, the coin-transaction filters, 400 for settlement) were not repeated:
  the code they exercised is unchanged by this PR's later commits.

🤖 Generated with Claude Code