tapimo 0.9.0 → 0.10.1

package/src/apps/data/routes/transactions.ts

@@ -186,6 +186,12 @@ export namespace schema {
               'The chain ID that the transaction is valid on; legacy transactions may omit it.',
             ),
           ),
+        feePayer: z
+          .optional(Schema.Address)
+          .check(
+            z.describe('The address that paid the transaction fee.'),
+            z.meta({ examples: ['0xbe058e1c4df8a4366a387bf595b284246a93039e'] }),
+          ),
         feePayerSignature: z
           .nullish(SignatureEnvelope)
           .check(
@@ -811,8 +817,8 @@ export function transactions() {
 
 /**
  * Lists humanized transactions with required fee-token metadata and opt-in
- * resources. Status filters page through receipts because execution status
- * lives there.
+ * resources. Status and fee-payer filters page through receipts because those
+ * fields are indexed there.
  */
 export async function listTransactions(
   c: Context<App.Environment>,
@@ -820,31 +826,28 @@ export async function listTransactions(
 ) {
   const { include, ...rest } = options
 
-  // `status` filters on the receipt's execution status, which `txs` does not
-  // store: page the receipts list (which already supports `status`) and hydrate
-  // each receipt's underlying transaction. The receipts `(block_num, tx_idx)`
-  // keyset cursor spans the same domain as the transactions `(block_num, idx)`
-  // cursor, so cursors stay interchangeable between the two paths.
-  const receiptPage =
-    rest.status === undefined
-      ? undefined
-      : await Receipts.getReceipts(c, {
-          address: rest.address,
-          chainId: rest.chainId,
-          cursor: rest.cursor,
-          feePayer: rest.feePayer,
-          feeToken: rest.feeToken,
-          fromBlock: rest.fromBlock,
-          fromTimestamp: rest.fromTimestamp,
-          limit: rest.limit,
-          order: rest.order,
-          page: rest.page,
-          recipient: rest.recipient,
-          sender: rest.sender,
-          status: rest.status,
-          toBlock: rest.toBlock,
-          toTimestamp: rest.toTimestamp,
-        })
+  // `status` lives on receipts, whose fee-payer index also avoids an unbounded
+  // `txs.fee_payer` scan. Its cursor matches the transaction cursor domain.
+  const receiptDriven = rest.status !== undefined || rest.feePayer !== undefined
+  const receiptPage = receiptDriven
+    ? await Receipts.getReceipts(c, {
+        address: rest.address,
+        chainId: rest.chainId,
+        cursor: rest.cursor,
+        feePayer: rest.feePayer,
+        feeToken: rest.feeToken,
+        fromBlock: rest.fromBlock,
+        fromTimestamp: rest.fromTimestamp,
+        limit: rest.limit,
+        order: rest.order,
+        page: rest.page,
+        recipient: rest.recipient,
+        sender: rest.sender,
+        status: rest.status,
+        toBlock: rest.toBlock,
+        toTimestamp: rest.toTimestamp,
+      })
+    : undefined
   const page = receiptPage
     ? {
         data: await transactionsForReceipts(c, {
@@ -885,7 +888,7 @@ export declare namespace listTransactions {
  * Fetches the underlying transactions for a page of receipts in one indexed
  * `txs` query and returns them humanized, ordered to match the receipt page.
  * Receipts whose transaction row is missing (not yet indexed) are skipped.
- * Backs the `status`-filtered transactions list, which pages receipts first.
+ * Backs status- and fee-payer-filtered lists, which page receipts first.
  */
 async function transactionsForReceipts(
   c: Context<App.Environment>,
@@ -1275,20 +1278,21 @@ function countFilters(options: countTransactions.Options): string[] {
 /**
  * Capped total count of transactions matching the same filters as the page.
  *
- * A `status` filter is delegated to {@link Receipts.countReceipts}, mirroring
- * the page's receipts-driven `status` path (execution status lives on receipts,
- * not `txs`). The address scope reuses the page's `UNION` of the two sides —
- * each side hits its own (`from`/`to`) index, and the deduped keyset count
- * avoids double-counting self-transactions (`from = to`). `totalCountCapped: true` is a
- * lower bound — at least {@link Schema.countCap} matches.
+ * Status and fee-payer filters are delegated to {@link Receipts.countReceipts},
+ * mirroring the page's receipts-driven path. The address scope reuses the
+ * page's `UNION` of the two sides; each side hits its own (`from`/`to`) index,
+ * and the deduped keyset count avoids double-counting self-transactions
+ * (`from = to`). `totalCountCapped: true` is a lower bound — at least
+ * {@link Schema.countCap} matches.
  */
 export function countTransactions(
   c: Context<App.Environment>,
   options: countTransactions.Options,
 ): Promise<z.output<typeof Schema.CountMeta>> {
-  // Status lives on receipts, so route the count through the same path the page
-  // uses: count receipts under the identical filters.
-  if (options.status !== undefined) return Receipts.countReceipts(c, options)
+  // Match the receipt-driven page path and avoid scanning unindexed
+  // `txs.fee_payer` values when counting.
+  if (options.status !== undefined || options.feePayer !== undefined)
+    return Receipts.countReceipts(c, options)
 
   const { address, chainId } = options
   const store = c.get('store')
@@ -1373,8 +1377,8 @@ export declare namespace scanTransactions {
 
 /**
  * The `txs` columns {@link toRpcTransaction} reconstructs a transaction from.
- * Shared by the list scan and the receipt-driven `status` path so the two
- * queries cannot drift from the reconstruction.
+ * Shared by the list scan and receipt-driven paths so the queries cannot drift
+ * from the reconstruction.
  */
 const transactionColumns = `block_num, block_timestamp, idx, hash, type, "from", "to", value, input,
                    gas_limit, max_fee_per_gas, max_priority_fee_per_gas,
@@ -1396,6 +1400,7 @@ function toRpcTransaction(row: Record<string, unknown>) {
     blockHash: null,
     blockNumber: Value.toHex(row['block_num']) ?? null,
     blockTimestamp: timestamp ? Value.toHex(Math.floor(Date.parse(timestamp) / 1000)) : undefined,
+    feePayer: typeof row['fee_payer'] === 'string' ? row['fee_payer'] : undefined,
     feeToken: typeof row['fee_token'] === 'string' ? row['fee_token'] : undefined,
     from: row['from'],
     gas: Value.toHex(row['gas_limit']),

package/src/apps/data/routes/transfers.bench.ts

@@ -0,0 +1,28 @@
+import * as QueryBench from '../../../../test/QueryBench.js'
+
+const address = {
+  4217: '0xc24357119432595a547cb2c1e41c51d753645519',
+  42431: '0x15496d3123afb2cfadc0ac068eadd11d0f8ae799',
+} as const
+
+QueryBench.bench({
+  name: 'getTransfers',
+  path: '/v1/transfers',
+  queries: [
+    'include=crossToken',
+    'include=crossToken,totalCount',
+    'include=memo',
+    'include=token.logoUri',
+    'include=token.verified',
+    'include=totalCount',
+    'valuation.currency=USD',
+  ],
+})
+
+QueryBench.bench({
+  name: 'getTransfers?address&include=totalCount',
+  path: {
+    4217: `/v1/transfers?address=${address[4217]}&include=totalCount`,
+    42431: `/v1/transfers?address=${address[42431]}&include=totalCount`,
+  },
+})