Skip to content

rpc-cost audit minors: dedicated units per request, stale copy, dRPC free re-measured - #3167

Merged
Flotapponnier merged 1 commit into
devfrom
fix/rpc-cost-audit-minors-1022
Oct 22, 2026
Merged

Flotapponnier merged 1 commit into
devfrom
fix/rpc-cost-audit-minors-1022

Conversation

@Flotapponnier

Copy link
Copy Markdown
Collaborator

Minor findings of the 2026-10-22 audit of bench 282: units per request for the dedicated cohort (A10, test added), stale README and profiles.go copy (A8), spec comments (A9), title wording (minor 10), chains bullet (A12), and dRPC's free plan re-measured on Solana (still error 35, kept Ethereum-only, conflict with plans.mdx noted).

Checks: gofmt, go vet, go test, pnpm validate, tsc, bun test src/ (695 pass).

…free re-measured

From the 2026-10-22 audit of bench 282.

- A10: units_per_request was never set for the dedicated cohort although
  the model bills it at 1 unit a request, so the panel read blank beside a
  priced bill. publishedUnitsPerRequest gives 1 on a chain one of the
  provider's plans serves, nothing elsewhere. Test added.
- README: provider and pricing-API counts (16 of 19, four APIs), the
  archive list, the dedicated cohort (unranked, priced in the calculator,
  break-even emitted but unread), and the real metric labels (kind, tier).
- profiles.go: the Alchemy/QuickNode ratio and the "one flat unit for
  every method" line no longer matched the spec.
- Spec: header comment says tier not cohort, an orphaned comment removed,
  "ranked" dropped from the title (the reference cohort's sentence read
  "across 19 ranked providers. 1 provider measured; none is ranked"), and
  the chains bullet no longer claims every provider prices Solana.
- dRPC Free re-measured: Solana still answers error 35 on the free plan,
  against plans.mdx's new "~10M" Solana figure; kept Ethereum-only.
- gofmt on catalogue.go (whitespace only).
@Flotapponnier
Flotapponnier merged commit 6d9e7ed into dev Oct 22, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant