docs(specs): update OpenRPC specifications from Daikon [Bot] - #1579
Conversation
🔗 Preview Mode
|
![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: 1f2f9f789d
ℹ️ About Codex in GitHub
Codex has been enabled to automatically 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 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
| ../_components/custom/methods.yaml#/components/methods/eth_getBlockReceipts | ||
| - $ref: ../_components/custom/methods.yaml#/components/methods/eth_getProof | ||
| - $ref: >- | ||
| ../_components/custom/methods.yaml#/components/methods/eth_getTransactionBySenderAndNonce |
There was a problem hiding this comment.
Add the eth_getTransactionBySenderAndNonce component before referencing it: the target commit contains no matching key in src/openrpc/chains/_components/custom/methods.yaml, and a repository-wide search finds only this and the five equivalent newly added references. Because generate:rpc dereferences every chain specification, Base, BNB Smart Chain, Ethereum, Ink, OP Mainnet, and X Layer now fail generation with an unresolved JSON pointer instead of producing their API specifications.
Useful? React with 👍 / 👎.
| | [`isBlockhashValid`](/docs/chains/solana/solana-api-endpoints/is-blockhash-valid) | [`minimumLedgerSlot`](/docs/chains/solana/solana-api-endpoints/minimum-ledger-slot) | | ||
| | [`requestAirdrop`](/docs/chains/solana/solana-api-endpoints/request-airdrop) | [`sendTransaction`](/docs/chains/solana/solana-api-endpoints/send-transaction) | | ||
| | [`simulateBundle`](/docs/chains/solana/solana-api-endpoints/simulate-bundle) | [`simulateTransaction`](/docs/chains/solana/solana-api-endpoints/simulate-transaction) | | ||
| | [`getBalanceByOwnerAtSlot`](/docs/chains/solana/solana-api-endpoints/get-balance-by-owner-at-slot) | [`getBalanceByOwnerAtSlot`](/docs/chains/solana/solana-api-endpoints/get-balance-by-owner-at-slot) | |
There was a problem hiding this comment.
Remove one of these identical links (and the duplicate source reference before regenerating): the published Solana API overview now lists getBalanceByOwnerAtSlot twice while the parent version listed it once. This gives users a duplicate endpoint in the supposedly comprehensive method index and unnecessarily shifts the rest of the table.
Useful? React with 👍 / 👎.
Daikon batch PR #1579 added $ref entries for eth_getTransactionBySenderAndNonce across Base, BNB, Ethereum, Ink, OP Mainnet, and X Layer chain specs, but the underlying method definition was missing from _components/custom/methods.yaml. This adds the definition (originally an Otterscan-namespace method exposed under eth_* on Alchemy) so the $refs resolve and the per-method pages render. Requested-by: @dslovinsky
|
Added the missing This PR wired up Method background: originally introduced by the Otterscan project as Signature: Requested by: @dslovinsky (via Slack). |
2 participants
🤖 Automated update of OpenRPC specs
This PR updates JSON-RPC method references based on the latest Daikon config. See this comprehensive guide
7 chains created or updated:
This workflow run:
Something look wrong? If you are an Alchemy employee please open a PR with changes in Daikon. If you are a member of the public please reach out to our team.