fix(bench): aggregate per-route vector before exposing success rate - #584
Merged
Conversation
Flotapponnier
added a commit
that referenced
this pull request
Jun 19, 2026
Flotapponnier
added a commit
that referenced
this pull request
Jun 19, 2026
…ee (#593) Adds the Near Intents 1Click API provider to both bridge benchmarks. Metrics will start populating once the bridge-monitor harness is redeployed with the nearintents_bridge.go integration shipped on mobula-api side and the NEARINTENTS_API_KEY env var set on Railway. Includes Near Protocol logo (public/logos/near-intents.svg), provider registry entry with longDescription + docs link, and YAML provider entries on both bridge-quote-latency and bridge-fee with the post-#584 avg(...) wrapper on success queries. SEO cohort copy updated to include Near Intents in both benches.
Flotapponnier
added a commit
that referenced
this pull request
Jun 19, 2026
…ee (#593) (#594) Adds the Near Intents 1Click API provider to both bridge benchmarks. Metrics will start populating once the bridge-monitor harness is redeployed with the nearintents_bridge.go integration shipped on mobula-api side and the NEARINTENTS_API_KEY env var set on Railway. Includes Near Protocol logo (public/logos/near-intents.svg), provider registry entry with longDescription + docs link, and YAML provider entries on both bridge-quote-latency and bridge-fee with the post-#584 avg(...) wrapper on success queries. SEO cohort copy updated to include Near Intents in both benches.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
Wrap the success queries on bridge-quote-latency and bridge-fee in avg(...) so they collapse the per route times per amount vector into a single scalar.
Why
The previous queries returned a vector (one entry per route times amount times token times region). The materialize layer picked one entry from that vector, which made the rendered success rate wrong:
After this patch the rendered values match the true 24h average across all measured routes.
Out of scope
The underlying Mobula API failures on Arbitrum to Solana corridor (where the 0% per route came from) are an upstream issue, not a bench issue.