tapimo 0.14.0 → 0.15.1

package/src/internal/MetricSink.test.ts

@@ -1,5 +1,6 @@
 import * as Metrics from '../Metrics.js'
 import type * as Webhooks from '../Webhooks.js'
+import type * as RoutesProvider from './routes/Provider.js'
 import type * as Log from './Log.js'
 import * as MetricSink from './MetricSink.js'
 
@@ -10,8 +11,103 @@ type MetricEvent = {
   value: number
 }
 
+function describeProviderMetric(event: MetricEvent) {
+  return `${event.type}:${event.name}:${event.value}:${event.tags?.['provider']}:${event.tags?.['operation']}:${event.tags?.['outcome']}:${event.tags?.['failure'] ?? 'none'}`
+}
+
+describe('routesProviderOperations', () => {
+  test('normalizes every provider operation and outcome', () => {
+    const events: MetricEvent[] = []
+    let flushes = 0
+    const metrics = Metrics.from({
+      count(name, value, tags) {
+        events.push({ name, tags, type: 'count', value })
+      },
+      flush() {
+        flushes += 1
+      },
+      gauge() {},
+      histogram(name, value, tags) {
+        events.push({ name, tags, type: 'histogram', value })
+      },
+    })
+    const record = MetricSink.routesProviderOperations(metrics)
+    const attempts = [
+      {
+        durationMs: 1,
+        id: 'relay',
+        operation: 'createDepositAddress',
+        outcome: 'available',
+      },
+      { durationMs: 2, id: 'relay', operation: 'getQuote', outcome: 'thin' },
+      {
+        durationMs: 3,
+        id: 'relay',
+        operation: 'listDepositAddressRequests',
+        outcome: 'unavailable',
+      },
+      {
+        durationMs: 4,
+        failure: 'network',
+        id: 'relay',
+        operation: 'observeDepositAddressRequest',
+        outcome: 'failed',
+      },
+      {
+        durationMs: 5,
+        id: 'stargate',
+        operation: 'prepareTransfer',
+        outcome: 'available',
+      },
+      {
+        durationMs: 6,
+        id: 'relay',
+        operation: 'receiveWebhook',
+        outcome: 'success',
+      },
+      {
+        durationMs: 7,
+        id: 'stargate',
+        operation: 'verifyDestinationDelivery',
+        outcome: 'success',
+      },
+      {
+        durationMs: 8,
+        failure: 'timeout',
+        id: 'stargate',
+        operation: 'verifySourceTransaction',
+        outcome: 'failed',
+      },
+    ] as const satisfies readonly RoutesProvider.ProviderAttemptResult[]
+
+    for (const attempt of attempts) record(attempt)
+
+    expect(events.map(describeProviderMetric)).toMatchInlineSnapshot(`
+      [
+        "count:routes_provider_operation_count:1:relay:createDepositAddress:success:none",
+        "histogram:routes_provider_operation_duration_ms:1:relay:createDepositAddress:success:none",
+        "count:routes_provider_operation_count:1:relay:getQuote:success:none",
+        "histogram:routes_provider_operation_duration_ms:2:relay:getQuote:success:none",
+        "count:routes_provider_operation_count:1:relay:listDepositAddressRequests:unavailable:none",
+        "histogram:routes_provider_operation_duration_ms:3:relay:listDepositAddressRequests:unavailable:none",
+        "count:routes_provider_operation_count:1:relay:observeDepositAddressRequest:failed:network",
+        "histogram:routes_provider_operation_duration_ms:4:relay:observeDepositAddressRequest:failed:network",
+        "count:routes_provider_operation_count:1:stargate:prepareTransfer:success:none",
+        "histogram:routes_provider_operation_duration_ms:5:stargate:prepareTransfer:success:none",
+        "count:routes_provider_operation_count:1:relay:receiveWebhook:success:none",
+        "histogram:routes_provider_operation_duration_ms:6:relay:receiveWebhook:success:none",
+        "count:routes_provider_operation_count:1:stargate:verifyDestinationDelivery:success:none",
+        "histogram:routes_provider_operation_duration_ms:7:stargate:verifyDestinationDelivery:success:none",
+        "count:routes_provider_operation_count:1:stargate:verifySourceTransaction:failed:timeout",
+        "histogram:routes_provider_operation_duration_ms:8:stargate:verifySourceTransaction:failed:timeout",
+      ]
+    `)
+    expect(flushes).toBe(0)
+  })
+})
+
 describe('requests', () => {
-  test('records optional funding count failures on successful pages', () => {
+  test('records optional routes count failures on successful pages', () => {
     const events: MetricEvent[] = []
     const metrics = Metrics.from({
       count(name, value, tags) {
@@ -25,42 +121,42 @@ describe('requests', () => {
 
     record({
       duration: 10,
-      fundingDepositCountFailed: true,
-      fundingTransferCountFailed: true,
+      routesDepositCountFailed: true,
+      routesTransferCountFailed: true,
       level: 'info',
       method: 'GET',
-      path: '/v1/funding/transfers',
+      path: '/v1/routes/transfers',
       requestId: 'request_private',
-      route: '/v1/funding/transfers',
+      route: '/v1/routes/transfers',
       status: 200,
     })
 
-    expect(events.find((event) => event.name === 'funding_deposit_count_failure_count'))
+    expect(events.find((event) => event.name === 'routes_deposit_count_failure_count'))
       .toMatchInlineSnapshot(`
       {
-        "name": "funding_deposit_count_failure_count",
+        "name": "routes_deposit_count_failure_count",
         "tags": {
           "cache": "miss",
           "method": "GET",
           "principal_environment": "none",
           "principal_type": "unknown",
-          "route": "/v1/funding/transfers",
+          "route": "/v1/routes/transfers",
         },
         "type": "count",
         "value": 1,
       }
     `)
 
-    expect(events.find((event) => event.name === 'funding_transfer_count_failure_count'))
+    expect(events.find((event) => event.name === 'routes_transfer_count_failure_count'))
       .toMatchInlineSnapshot(`
       {
-        "name": "funding_transfer_count_failure_count",
+        "name": "routes_transfer_count_failure_count",
         "tags": {
           "cache": "miss",
           "method": "GET",
           "principal_environment": "none",
           "principal_type": "unknown",
-          "route": "/v1/funding/transfers",
+          "route": "/v1/routes/transfers",
         },
         "type": "count",
         "value": 1,
@@ -86,9 +182,16 @@ describe('requests', () => {
       level: 'error',
       method: 'POST',
       path: '/rpc/sponsor',
+      provider: {
+        chainId: 4217,
+        failure: 'network',
+        id: 'rpc',
+        operation: 'eth_call',
+      },
       providerFailures: [
         {
           chainId: 4217,
+          code: 'UPSTREAM_FAILURE',
           failure: 'http',
           id: 'tidx',
           operation: 'query',
@@ -151,6 +254,23 @@ describe('requests', () => {
           "type": "count",
           "value": 2,
         },
+        {
+          "name": "upstream_failure_count",
+          "tags": {
+            "cache": "miss",
+            "chain_id": "4217",
+            "failure": "network",
+            "method": "POST",
+            "operation": "eth_call",
+            "principal_environment": "none",
+            "principal_type": "unknown",
+            "provider": "rpc",
+            "route": "/rpc/sponsor",
+            "status": "unknown",
+          },
+          "type": "count",
+          "value": 1,
+        },
         {
           "name": "upstream_failure_count",
           "tags": {
@@ -242,7 +362,7 @@ describe('requests', () => {
     `)
   })
 
-  test('records bounded funding provider attempts without caller data', () => {
+  test('records bounded route provider attempts without caller data', () => {
     const events: MetricEvent[] = []
     let flushes = 0
     const metrics = Metrics.from({
@@ -263,7 +383,7 @@ describe('requests', () => {
 
     record({
       duration: 100,
-      fundingProviderAttempts: [
+      routesProviderAttempts: [
         {
           durationMs: 10,
           id: 'relay',
@@ -289,10 +409,16 @@ describe('requests', () => {
           operation: 'prepareTransfer',
           outcome: 'failed',
         },
+        {
+          durationMs: 50,
+          id: 'relay',
+          operation: 'receiveWebhook',
+          outcome: 'success',
+        },
       ],
       level: 'info',
       method: 'GET',
-      path: '/v1/funding/quotes',
+      path: '/v1/routes/quotes',
       principal: {
         id: 'key_private',
         orgId: 'org_private',
@@ -301,15 +427,15 @@ describe('requests', () => {
       },
       query: 'sourceAmount=secret',
       requestId: 'request_private',
-      route: '/v1/funding/quotes',
+      route: '/v1/routes/quotes',
       status: 200,
     } satisfies Log.Entry)
 
-    const funding = events.filter((event) => event.name.startsWith('funding_quote_provider_'))
-    expect(funding).toMatchInlineSnapshot(`
+    const routes = events.filter((event) => event.name.startsWith('routes_quote_provider_'))
+    expect(routes).toMatchInlineSnapshot(`
       [
         {
-          "name": "funding_quote_provider_attempt_count",
+          "name": "routes_quote_provider_attempt_count",
           "tags": {
             "operation": "getQuote",
             "outcome": "available",
@@ -319,7 +445,7 @@ describe('requests', () => {
           "value": 1,
         },
         {
-          "name": "funding_quote_provider_duration_ms",
+          "name": "routes_quote_provider_duration_ms",
           "tags": {
             "operation": "getQuote",
             "outcome": "available",
@@ -329,7 +455,7 @@ describe('requests', () => {
           "value": 10,
         },
         {
-          "name": "funding_quote_provider_attempt_count",
+          "name": "routes_quote_provider_attempt_count",
           "tags": {
             "operation": "getQuote",
             "outcome": "thin",
@@ -339,7 +465,7 @@ describe('requests', () => {
           "value": 1,
         },
         {
-          "name": "funding_quote_provider_duration_ms",
+          "name": "routes_quote_provider_duration_ms",
           "tags": {
             "operation": "getQuote",
             "outcome": "thin",
@@ -349,7 +475,7 @@ describe('requests', () => {
           "value": 20,
         },
         {
-          "name": "funding_quote_provider_attempt_count",
+          "name": "routes_quote_provider_attempt_count",
           "tags": {
             "operation": "getQuote",
             "outcome": "unavailable",
@@ -359,7 +485,7 @@ describe('requests', () => {
           "value": 1,
         },
         {
-          "name": "funding_quote_provider_duration_ms",
+          "name": "routes_quote_provider_duration_ms",
           "tags": {
             "operation": "getQuote",
             "outcome": "unavailable",
@@ -369,7 +495,7 @@ describe('requests', () => {
           "value": 30,
         },
         {
-          "name": "funding_quote_provider_attempt_count",
+          "name": "routes_quote_provider_attempt_count",
           "tags": {
             "failure": "timeout",
             "operation": "prepareTransfer",
@@ -380,7 +506,7 @@ describe('requests', () => {
           "value": 1,
         },
         {
-          "name": "funding_quote_provider_duration_ms",
+          "name": "routes_quote_provider_duration_ms",
           "tags": {
             "failure": "timeout",
             "operation": "prepareTransfer",
@@ -392,12 +518,173 @@ describe('requests', () => {
         },
       ]
     `)
-    expect(JSON.stringify(funding)).not.toMatch(
+    expect(
+      events
+        .filter((event) => event.name === 'routes_provider_operation_count')
+        .map(describeProviderMetric),
+    ).toMatchInlineSnapshot(`
+      [
+        "count:routes_provider_operation_count:1:relay:getQuote:success:none",
+        "count:routes_provider_operation_count:1:rhino:getQuote:success:none",
+        "count:routes_provider_operation_count:1:squid:getQuote:unavailable:none",
+        "count:routes_provider_operation_count:1:across:prepareTransfer:failed:timeout",
+        "count:routes_provider_operation_count:1:relay:receiveWebhook:success:none",
+      ]
+    `)
+    expect(JSON.stringify(routes)).not.toMatch(
       /key_private|org_private|project_private|request_private|secret/,
     )
     expect(flushes).toBe(1)
   })
 
+  test('records route quote availability and freshness', () => {
+    const events: MetricEvent[] = []
+    const metrics = Metrics.from({
+      count(name, value, tags) {
+        events.push({ name, tags, type: 'count', value })
+      },
+      flush() {},
+      gauge() {},
+      histogram(name, value, tags) {
+        events.push({ name, tags, type: 'histogram', value })
+      },
+    })
+    const record = MetricSink.requests(metrics)
+    const base = {
+      duration: 100,
+      level: 'info',
+      method: 'GET',
+      path: '/v1/routes/quotes',
+      requestId: 'request_private',
+      route: '/v1/routes/quotes',
+      status: 200,
+    } satisfies Log.Entry
+
+    record({
+      ...base,
+      routesQuoteResult: {
+        candidateCount: 2,
+        outcome: 'available',
+        quotes: [
+          {
+            ageMs: 1_250,
+            customerValueLossBps: 100,
+            destinationChain: 'eip155:4217',
+            destinationToken: 'eip155:4217/erc20:0x20c000000000000000000000b9537d11c60e8b50',
+            futureSkewMs: 0,
+            provider: 'relay',
+            sourceChain: 'eip155:8453',
+            sourceToken: 'eip155:8453/erc20:0x833589fcd6edb6e08f4c7c32d4f71b54bda02913',
+          },
+          {
+            ageMs: 0,
+            destinationChain: 'eip155:4217',
+            destinationToken: 'eip155:4217/erc20:0x20c000000000000000000000b9537d11c60e8b50',
+            futureSkewMs: 2_500,
+            provider: 'stargate',
+            sourceChain: 'eip155:8453',
+            sourceToken: 'eip155:8453/erc20:0x833589fcd6edb6e08f4c7c32d4f71b54bda02913',
+          },
+        ],
+      },
+    })
+    record({
+      ...base,
+      routesQuoteResult: { candidateCount: 0, outcome: 'unsupported', quotes: [] },
+    })
+
+    expect(
+      events.filter((event) =>
+        [
+          'routes_quote_age_ms',
+          'routes_quote_future_skew_ms',
+          'routes_quote_request_count',
+          'routes_quote_customer_value_loss_bps',
+        ].includes(event.name),
+      ),
+    ).toMatchInlineSnapshot(`
+      [
+        {
+          "name": "routes_quote_request_count",
+          "tags": {
+            "cache": "miss",
+            "method": "GET",
+            "outcome": "available",
+            "principal_environment": "none",
+            "principal_type": "unknown",
+            "route": "/v1/routes/quotes",
+          },
+          "type": "count",
+          "value": 1,
+        },
+        {
+          "name": "routes_quote_customer_value_loss_bps",
+          "tags": {
+            "destination_chain": "eip155:4217",
+            "destination_token": "eip155:4217/erc20:0x20c000000000000000000000b9537d11c60e8b50",
+            "provider": "relay",
+            "source_chain": "eip155:8453",
+            "source_token": "eip155:8453/erc20:0x833589fcd6edb6e08f4c7c32d4f71b54bda02913",
+          },
+          "type": "histogram",
+          "value": 100,
+        },
+        {
+          "name": "routes_quote_age_ms",
+          "tags": {
+            "cache": "miss",
+            "method": "GET",
+            "principal_environment": "none",
+            "principal_type": "unknown",
+            "provider": "relay",
+            "route": "/v1/routes/quotes",
+          },
+          "type": "histogram",
+          "value": 1250,
+        },
+        {
+          "name": "routes_quote_age_ms",
+          "tags": {
+            "cache": "miss",
+            "method": "GET",
+            "principal_environment": "none",
+            "principal_type": "unknown",
+            "provider": "stargate",
+            "route": "/v1/routes/quotes",
+          },
+          "type": "histogram",
+          "value": 0,
+        },
+        {
+          "name": "routes_quote_future_skew_ms",
+          "tags": {
+            "cache": "miss",
+            "method": "GET",
+            "principal_environment": "none",
+            "principal_type": "unknown",
+            "provider": "stargate",
+            "route": "/v1/routes/quotes",
+          },
+          "type": "histogram",
+          "value": 2500,
+        },
+        {
+          "name": "routes_quote_request_count",
+          "tags": {
+            "cache": "miss",
+            "method": "GET",
+            "outcome": "unsupported",
+            "principal_environment": "none",
+            "principal_type": "unknown",
+            "route": "/v1/routes/quotes",
+          },
+          "type": "count",
+          "value": 1,
+        },
+      ]
+    `)
+  })
+
   test('records MPP relay outcomes without credential data', () => {
     const events: MetricEvent[] = []
     const metrics = Metrics.from({

package/src/internal/MetricSink.ts

@@ -1,5 +1,6 @@
 import type { Metrics } from '../Metrics.js'
 import type * as Webhooks from '../Webhooks.js'
+import type * as RoutesProvider from './routes/Provider.js'
 import type * as Log from './Log.js'
 import * as WebhookDestination from './WebhookDestination.js'
 
@@ -147,6 +148,26 @@ type RouteTags = {
   route: string
 }
 
+/** Builds a normalized route provider operation sink over an optional metrics backend. */
+export function routesProviderOperations(backend: Metrics | undefined) {
+  return (attempt: RoutesProvider.ProviderAttemptResult) => {
+    if (!backend) return
+    const outcome = (() => {
+      if (attempt.outcome === 'failed') return 'failed'
+      if (attempt.outcome === 'unavailable') return 'unavailable'
+      return 'success'
+    })()
+    const tags = {
+      ...(attempt.failure === undefined ? {} : { failure: attempt.failure }),
+      operation: attempt.operation,
+      outcome,
+      provider: attempt.id,
+    }
+    backend.count('routes_provider_operation_count', 1, tags)
+    backend.histogram('routes_provider_operation_duration_ms', attempt.durationMs, tags)
+  }
+}
+
 /**
  * Builds a request (HTTP) metrics sink over `backend`: emits the bounded Worker
  * metrics derived from one request-log entry, then flushes (one entry == one
@@ -157,6 +178,7 @@ type RouteTags = {
  * StatsD-style series.
  */
 export function requests(backend: Metrics): RequestSink {
+  const recordRoutesProviderOperation = routesProviderOperations(backend)
   return (entry) => {
     const tags = routeTags(entry)
 
@@ -196,24 +218,67 @@ export function requests(backend: Metrics): RequestSink {
     for (const [operation, duration] of Object.entries(entry.timings ?? {}))
       backend.histogram('upstream_duration_ms', duration, { ...tags, operation })
 
-    for (const attempt of entry.fundingProviderAttempts ?? []) {
+    if (entry.routesQuoteResult) {
+      backend.count('routes_quote_request_count', 1, {
+        ...tags,
+        outcome: entry.routesQuoteResult.outcome,
+      })
+      for (const quote of entry.routesQuoteResult.quotes) {
+        const tags_quote = {
+          destination_chain: quote.destinationChain,
+          destination_token: quote.destinationToken,
+          provider: quote.provider,
+          source_chain: quote.sourceChain,
+          source_token: quote.sourceToken,
+        }
+        if (quote.customerValueLossBps !== undefined)
+          backend.histogram(
+            'routes_quote_customer_value_loss_bps',
+            quote.customerValueLossBps,
+            tags_quote,
+          )
+        backend.histogram('routes_quote_age_ms', quote.ageMs, {
+          ...tags,
+          provider: quote.provider,
+        })
+        if (quote.futureSkewMs > 0)
+          backend.histogram('routes_quote_future_skew_ms', quote.futureSkewMs, {
+            ...tags,
+            provider: quote.provider,
+          })
+      }
+    }
+
+    for (const attempt of entry.routesProviderAttempts ?? []) {
+      recordRoutesProviderOperation(attempt)
+      // Keep the legacy family scoped to operations it already measured.
+      if (
+        attempt.operation !== 'createDepositAddress' &&
+        attempt.operation !== 'getQuote' &&
+        attempt.operation !== 'prepareTransfer'
+      )
+        continue
       const tags_attempt = {
         ...(attempt.failure === undefined ? {} : { failure: attempt.failure }),
         operation: attempt.operation,
         outcome: attempt.outcome,
         provider: attempt.id,
       }
-      backend.count('funding_quote_provider_attempt_count', 1, tags_attempt)
-      backend.histogram('funding_quote_provider_duration_ms', attempt.durationMs, tags_attempt)
+      backend.count('routes_quote_provider_attempt_count', 1, tags_attempt)
+      backend.histogram('routes_quote_provider_duration_ms', attempt.durationMs, tags_attempt)
     }
 
-    if (entry.fundingDepositCountFailed)
-      backend.count('funding_deposit_count_failure_count', 1, tags)
+    if (entry.routesDepositCountFailed) backend.count('routes_deposit_count_failure_count', 1, tags)
 
-    if (entry.fundingTransferCountFailed)
-      backend.count('funding_transfer_count_failure_count', 1, tags)
+    if (entry.routesTransferCountFailed)
+      backend.count('routes_transfer_count_failure_count', 1, tags)
 
-    for (const failure of entry.providerFailures ?? (entry.provider ? [entry.provider] : []))
+    const providerFailures = entry.providerFailures ?? []
+    const failures =
+      entry.provider && !providerFailures.includes(entry.provider)
+        ? [entry.provider, ...providerFailures]
+        : providerFailures
+    for (const failure of failures)
       backend.count('upstream_failure_count', 1, {
         ...tags,
         chain_id: String(failure.chainId ?? 'unknown'),

package/src/internal/Mpp.test.ts

@@ -1,4 +1,5 @@
 import * as Mpp from './Mpp.js'
+import * as Store from './Store.js'
 
 const fingerprint = Mpp.fingerprintForRealm('openai.mpp.tempo.xyz')
 // Layout: 0x + tag(4 bytes) + version(1 byte) + fingerprint(10 bytes) + padding(17 bytes).
@@ -12,6 +13,28 @@ describe('fingerprintForRealm', () => {
   })
 })
 
+describe('fingerprintMap', () => {
+  test('bounds stalled cache reads', async () => {
+    vi.useFakeTimers()
+    try {
+      const pending = deferred<string | null>()
+      const store = Store.from({
+        ...Store.memory(),
+        get: () => pending.promise,
+      })
+      const failure = expect(Mpp.fingerprintMap({ store })).rejects.toBeInstanceOf(
+        Store.MemoizeTimeoutError,
+      )
+
+      await vi.advanceTimersByTimeAsync(1_000)
+      await failure
+      pending.resolve(null)
+    } finally {
+      vi.useRealTimers()
+    }
+  })
+})
+
 describe('isAttributionMemo', () => {
   test('accepts a 32-byte memo prefixed with the attribution tag', () => {
     expect(Mpp.isAttributionMemo(attributionMemo)).toBe(true)
@@ -62,3 +85,11 @@ describe('resolve', () => {
     ).toBe(undefined)
   })
 })
+
+function deferred<value>() {
+  let resolve!: (value: PromiseLike<value> | value) => void
+  const promise = new Promise<value>((resolve_) => {
+    resolve = resolve_
+  })
+  return { promise, resolve }
+}