docs(developer-hub): add Sui integration guide for Pyth Pro/Lazer#3498
Merged
docs(developer-hub): add Sui integration guide for Pyth Pro/Lazer#3498
Conversation
- Restructure to show PTB-based verification pattern (avoids version coupling) - Use modern Move syntax: &feeds[0] instead of vector::borrow - Use method syntax: .is_some(), .borrow() instead of option::is_some()
- Use @pythnetwork/pyth-lazer-sui-js SDK with addParseAndVerifyLeEcdsaUpdateCall - Link to sdk/js subdirectory for Sui TypeScript SDK - Fix wording about version compatibility - Remove 'modern syntax' references - just use proper syntax
- Fix I64 API: use get_is_negative(), get_magnitude_if_positive/negative() - Remove market_session/ema fields from main example (keep core fields) - Update contract addresses callout to point to SDK README - Simplify signed integer example with correct method signatures
Contributor
Author
🤖 Devin AI EngineerI'll be helping with this pull request! Here's what you should know: ✅ I will automatically:
Note: I can only respond to comments from users who have write access to this repository. ⚙️ Control Options:
|
|
The latest updates on your projects. Learn more about Vercel for GitHub.
5 Skipped Deployments
|
Contributor
🤖 llms.txt Content ReviewAnalyzed documentation changes — no updates to curated content required. Products reviewed:
Files analyzed |
2 tasks
apps/developer-hub/content/docs/price-feeds/pro/integrate-as-consumer/sui.mdx
Show resolved
Hide resolved
aditya520
approved these changes
Feb 24, 2026
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
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
Adds documentation for integrating Pyth Pro/Lazer on Sui, following the same structure as the existing EVM and SVM guides. This is a re-creation of #3495 from a non-fork branch to resolve Vercel deployment authorization failures (the original PR was from a fork, which Vercel cannot build without manual team authorization).
Changes:
sui.mdx— three-step guide covering WebSocket subscription (leEcdsa format), PTB verification via the Sui JS SDK, and consuming the verifiedUpdatein Move contractsmeta.jsonandindex.mdxto include Sui in navigation and listingRationale
PR #3495 had all GitHub Actions CI passing but Vercel deployments were stuck on "Authorization required to deploy" because it originated from a fork (
pythia-assistant/pyth-crosschain). This PR cherry-picks the same 6 commits onto a branch in the main repo so Vercel can build and generate a preview.How has this been tested?
buildandtestpassed on the original PR docs(developer-hub): add Sui integration guide for Pyth Pro/Lazer #3495)Verification performed:
lazer/contracts/sui/sources/—Feedaccessor names,I64methods (get_is_negative,get_magnitude_if_positive,get_magnitude_if_negative), andUpdatemethods all matchlazer/contracts/sui/sdk/js/README.mdImportant review notes
movesyntax highlighting: Thesource.config.tsrehypeCodeOptions.langslist does not includemove. Shiki 3.x bundles a Move grammar, so it may work without explicit registration — but this should be confirmed by checking the Vercel preview renders the Move code blocks correctly (not as errors or blank).Link to Devin run: https://app.devin.ai/sessions/5fb891efbd31447c9097c416b498b3da
Requested by: @matej-douro