docs(specs): update OpenRPC specifications from Daikon [Bot] - #1558
Conversation
🔗 Preview Mode
|
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 216b5b5fc3
ℹ️ 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".
Codex flagged an unresolved $ref in src/openrpc/chains/solana/solana.yaml pointing at `components.methods.getBalanceByOwnerAtSlot` that was not defined in the shared `_components/solana/methods.yaml`. Verified the method is real on Alchemy's Solana mainnet RPC: it returns a wallet's balance of an SPL Token / Token-2022 mint (or wrapped SOL) as of a specific finalized slot, with required `owner`, `mint`, and `slot` params. Adds the method definition (co-located with getBalance / getTokenAccountsByOwnerAtSlot in the Account Archive family) and a `TokenBalanceByOwnerAtSlot` response schema in account.yaml. Example values sampled from a live call against solana-mainnet.g.alchemy.com. Refs Daikon PR #1558 Requested-by: @SahilAujla
|
@codex review |
…tion" This reverts commit 49efc5a. Reason: PRs #1551 and #1552 (approved by @dslovinsky earlier today) already document `getBalanceByOwnerAtSlot` with a richer spec (adds a `scope: all|ata` param and a `wallet`/`owner` alias caught during live-check). Reverting so PR #1552 can merge cleanly; once it does, this Daikon PR's unresolved $ref resolves without conflict. Refs Daikon PR #1558 Requested-by: @SahilAujla
|
Codex Review: Didn't find any major issues. 👍 Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
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". |
Brings in PR #1552 which defines `getBalanceByOwnerAtSlot` in `_components/solana/methods.yaml`, resolving the unresolved $ref in `src/openrpc/chains/solana/solana.yaml` that codex flagged. Also picks up unrelated main advances (wallet EIP-7702 docs, misc). Conflict resolved in `content/api-reference/solana/solana-api-overview.mdx`: whitespace-only diff on the `getFirstAvailableBlock` row between Daikon's regen (HEAD, wider alignment) and #1552's manual edit. Kept HEAD's alignment to match the rest of the Daikon-regenerated table. `pnpm run generate:rpc` and `pnpm run validate:rpc` both clean after merge. Refs Daikon PR #1558 Requested-by: @dslovinsky
🤖 Automated update of OpenRPC specs
This PR updates JSON-RPC method references based on the latest Daikon config. See this comprehensive guide
5 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.