[Writer] docs(solana): separate API overview into Core RPC, DAS, and Photon sections#1238
Open
JackReacher0807 wants to merge 3 commits intomainfrom
Open
[Writer] docs(solana): separate API overview into Core RPC, DAS, and Photon sections#1238JackReacher0807 wants to merge 3 commits intomainfrom
JackReacher0807 wants to merge 3 commits intomainfrom
Conversation
…ctions Reorganize the Solana API overview page into three clearly-labeled sections so users can tell at a glance which methods belong to the core Solana JSON-RPC surface, the Metaplex DAS API, and the Photon (ZK Compression) API. - Adds short intro for each surface - Links DAS methods to the dedicated /docs/reference/alchemy-das-apis-for-solana page - Notes that Photon methods are powered by the Helius Photon indexer - Preserves every existing endpoint link; no OpenRPC spec changes Requested by @cosmin on Slack.
🔗 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: 8a081990dd
ℹ️ 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".
|
|
||
| ## Core Solana RPC methods | ||
|
|
||
| Standard Solana JSON-RPC methods for reading on-chain state, submitting transactions, and inspecting the cluster. These match the official [Solana RPC specification](https://solana.com/docs/rpc) and are supported on every Alchemy Solana endpoint. |
There was a problem hiding this comment.
This new sentence states that methods in this section match the official Solana RPC spec and are supported on every Alchemy Solana endpoint, but the same table includes getPriorityFeeEstimate and simulateBundle (lines 34 and 46), which are documented as provider-specific functionality (getPriorityFeeEstimate recommendation logic and Jito bundle simulation in src/openrpc/chains/_components/solana/methods.yaml, lines 643 and 886). This can mislead readers about cross-provider compatibility, so the wording should be narrowed or those methods separated.
Useful? React with 👍 / 👎.
Collaborator
Author
There was a problem hiding this comment.
Addressed in 4a4456b. Moved getPriorityFeeEstimate and simulateBundle into a new ### Alchemy-specific extensions subsection under Core Solana RPC methods, with a note that they are not part of the official Solana RPC specification and are not available on other Solana RPC providers. The Core RPC table now only lists methods from the official Solana RPC spec that work against any provider.
| | [`getAssetsByAuthority`](/docs/reference/alchemy-das-apis-for-solana/solana-das-api-endpoints/get-assets-by-authority) | [`getAssetsByCreator`](/docs/reference/alchemy-das-apis-for-solana/solana-das-api-endpoints/get-assets-by-creator) | | ||
| | [`getAssetsByGroup`](/docs/reference/alchemy-das-apis-for-solana/solana-das-api-endpoints/get-assets-by-group) | [`getAssetsByOwner`](/docs/reference/alchemy-das-apis-for-solana/solana-das-api-endpoints/get-assets-by-owner) | | ||
| | [`getAssetSignatures`](/docs/reference/alchemy-das-apis-for-solana/solana-das-api-endpoints/get-asset-signatures) | [`getNftEditions`](/docs/reference/alchemy-das-apis-for-solana/solana-das-api-endpoints/get-nft-editions) | | ||
| | [`searchAssets`](/docs/reference/alchemy-das-apis-for-solana/solana-das-api-endpoints/search-assets) | | |
There was a problem hiding this comment.
The DAS section is presented as the method list for this API surface, but getTokenAccounts is missing even though it is part of the DAS reference methods table (content/api-reference/data/nft-api/alchemy-das-apis-for-solana.mdx, Methods section). Users using this page as the overview can incorrectly conclude that getTokenAccounts is unavailable from the Solana DAS surface.
Useful? React with 👍 / 👎.
Collaborator
Author
There was a problem hiding this comment.
Already addressed in the previous commit 198d4f7 — getTokenAccounts is in the DAS API table on line 68 of the current file. You can verify with: grep getTokenAccounts content/api-reference/solana/solana-api-overview.mdx. Matches the method list in content/api-reference/data/nft-api/alchemy-das-apis-for-solana.mdx.
🔍 Link CheckStatus: ❌ Failed Summary
Broken links (80) — click to expandErrors per inputErrors in ./content/wallets/pages/bundler-api/bundler-faqs.mdx
Errors in ./content/api-reference/op-mainnet/op-mainnet-api-faq/op-mainnet-api-faq.mdx
Errors in ./content/wallets/pages/recipes/smart-wallets-aave.mdx
Errors in ./content/wallets/pages/smart-wallets/how-to-stamp-requests.mdx
Errors in ./content/wallets/pages/concepts/smart-account-client.mdx
Errors in ./content/wallets/pages/smart-wallets/quickstart/index.mdx
Errors in ./content/wallets/pages/index.mdx
Errors in ./content/wallets/pages/authentication/overview.mdx
Errors in ./content/api-reference/introduction/api-overview.mdx
Errors in ./content/tutorials/alchemy-university/smart-contract-basics/multi-sig-contracts.mdx
Errors in ./content/wallets/pages/recipes/send-usdc.mdx
Errors in ./content/wallets/pages/recipes/upgrade-to-smart-accounts.mdx
Errors in ./content/wallets/pages/react-native/getting-started/getting-started-expo.mdx
Errors in ./content/changelog/2025-08-28.md
Errors in ./content/api-reference/hyperevm/hyperevm-api-overview.mdx
Errors in ./content/wallets/pages/react-native/getting-started/getting-started-rn-bare.mdx
Errors in ./content/changelog/2026-04-09.md
Errors in ./content/api-reference/katana/katana-api-overview.mdx
Errors in ./content/changelog/2025-09-18.md
Errors in ./content/changelog/2026-03-26.md
Errors in ./content/api-reference/arbitrum-nova/arbitrum-nova-deprecation-notice.mdx
Errors in ./content/api-reference/arbitrum/arbitrum-api-faq/arbitrum-api-faq.mdx
Errors in ./content/wallets/pages/recipes/onramp-funds.mdx
|
cosmingamanusi
approved these changes
Apr 20, 2026
…okenAccounts to DAS table
cosmingamanusi
approved these changes
Apr 20, 2026
cosmingamanusi
requested changes
Apr 20, 2026
Contributor
cosmingamanusi
left a comment
cosmingamanusi
left a comment
There was a problem hiding this comment.
section not created
- Address Codex P2: narrow Core RPC portability claim to strictly standard Solana RPC methods; move getPriorityFeeEstimate and simulateBundle into a new ### Alchemy-specific extensions subsection with a note that they are not available on other Solana RPC providers. - Addresses Cosmin's CHANGES_REQUESTED feedback on PR #1238.
Collaborator
Author
|
@cosmingamanusi thanks for the re-review! A couple of things:
|
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
2 participants
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.
Summary
Reorganizes
content/api-reference/solana/solana-api-overview.mdxinto three clearly-labeled H2 sections so developers can quickly find the method they need instead of scanning one long alphabetical table that mixes three different API surfaces.New structure
solana-rpcmethods (getAccountInfo, getBalance, getBlock, sendTransaction, etc.)/docs/reference/alchemy-das-apis-for-solana.Verification
src/openrpc/chains/solana/solana.yamlends up in exactly one section; all existing endpoint links are preserved.docs.ymlnav structure is unchanged.Context
Linear: DOCS-45
Requested by @cosmin on Slack.