docs(pricing): first generated Compute Unit Costs update (drift audit)#1425
docs(pricing): first generated Compute Unit Costs update (drift audit)#1425SahilAujla wants to merge 1 commit into
Conversation
Output of UpdateComputeUnitCostsFromDaikonWorkflow's generator run against latest chain-config topconfig (1100 methods). This is the drift audit PR: every diff against the hand-maintained tables is a pre-existing inaccuracy or omission the automation fixes. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
🔗 Preview Mode
|
|
@codex review |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: c807093cad
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| | getTransaction | 40 | | | ||
| | getTransactionsForAddress | 100 | | | ||
| | getSupply | 160 | 800 | | ||
| | alchemy\_requestFeePayer\_prefundRent | 1250 | 100 | |
There was a problem hiding this comment.
This row turns an override key into a public Solana JSON-RPC method even though the page already documents the real API as alchemy_requestFeePayer with prefundRent = true; repo-wide search only finds alchemy_requestFeePayer_prefundRent in this generated table and the OpenRPC ignore list, not in the public wallet docs. Users copying the method name from the pricing table would call a billing variant instead of the supported method, so render this as a note/override on alchemy_requestFeePayer instead of a row.
Useful? React with 👍 / 👎.
| | create\_file | 0 | 50 | | ||
| | upload\_metadata | 0 | 50 | |
There was a problem hiding this comment.
The generated all product is a Daikon catch-all, not a customer-facing API family: a repo-wide search for these method names only finds them in this new table, with no corresponding API docs or specs. Publishing them under a generic all API heading exposes internal billing keys as methods users cannot discover or call, so the generator should filter this product or map it to a reviewed public section before landing.
Useful? React with 👍 / 👎.
1 participant
Description
Stacked on #1424 (the marker PR) — merge that first; this PR's diff then shows exactly what the automation changes.
This is the first generated output of
UpdateComputeUnitCostsFromDaikonWorkflow(OMGWINNING/dashboard#8087), produced by running the workflow's generator locally against the latestchain-configtopconfig (1100 methods). It serves as the drift audit from the ticket's acceptance criteria: every diff against the hand-maintained tables is a pre-existing inaccuracy or omission that the daily automation will keep fixed. Once the workflow is deployed and scheduled, PRs identical to this one arrive automatically onbot/daikon-cu-costs-update.Drift audit summary
Coverage: 658 → 1038 table rows. 382 billed methods were missing from the page entirely, 53 rows had wrong values, 1 row is removed.
12 new sections auto-created (inferred titles — please review naming/placement; renaming headers or moving blocks is safe, the marker carries the key):
indexergroup), Celestia gRPC (40,celestia.*/cosmos.*services), Sui gRPC (22), Solana Yellowstone gRPC unary (7)all API(2 internal-looking methodscreate_file/upload_metadata, cu=0) — needs a human decision on naming/placementNotable value corrections (topconfig is the billing source of truth):
eth_sendRawTransactionThroughput CU 250 → 50,eth_sendRawTransactionSync250 → 125starknet_blockNumber/starknet_blockHashAndNumberCU 20 → 10getNFTsForContractThroughput CU 50 → 100getSupply/getProgramAccounts/getInflationRewardgained real throughput values (800/117/300)rateLimitComputeUnitsdefaults tocomputeUnits, so equal values carry no information)Removals (both verified intentional):
suix_getLatestBridge— markedMETHOD_PRERELEASEin topconfig; the page only lists public methods/transactions/history/by-addressis a rename totransactions/history/by-address(path normalization), not a removalFormatting normalization: uniform
\_escaping (the Wallet section was inconsistent), deterministic row sort (CU ascending, then name), NFT/signer/prices/portfolio rows keep their public display names via the workflow's display-name map.Flagged, not on the page:
/tx(custom method shared by Celestia consensus + Bitcoin) resolves to no product tag — the workflow Slack-alerts this; it needs a manual override decision.Found in the process: likely topconfig typo
aarbtrace_rawTransaction(double "a") — appears as a new Arbtrace row here; should be fixed in Daikon.Test Plan
prettier --checkpassescu:automarkers verified byte-identical to the marker PRall API)LINEAR TASK: https://linear.app/alchemyapi/issue/DX-3112/automate-the-compute-unit-costs-pricing-page-from-daikon-topconfig