Skip to content

docs(developer-hub): add Sui integration guide for Pyth Pro/Lazer#3498

Merged
aditya520 merged 9 commits intomainfrom
devin/1771919839-fix-sui-lazer-docs
Feb 24, 2026
Merged

docs(developer-hub): add Sui integration guide for Pyth Pro/Lazer#3498
aditya520 merged 9 commits intomainfrom
devin/1771919839-fix-sui-lazer-docs

Conversation

@devin-ai-integration
Copy link
Contributor

@devin-ai-integration devin-ai-integration bot commented Feb 24, 2026

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:

  • New file: sui.mdx — three-step guide covering WebSocket subscription (leEcdsa format), PTB verification via the Sui JS SDK, and consuming the verified Update in Move contracts
  • Updated: meta.json and index.mdx to include Sui in navigation and listing

Rationale

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?

Verification performed:

  • Move code examples were cross-checked against the actual contract source at lazer/contracts/sui/sources/Feed accessor names, I64 methods (get_is_negative, get_magnitude_if_positive, get_magnitude_if_negative), and Update methods all match
  • TypeScript SDK usage matches the pattern in lazer/contracts/sui/sdk/js/README.md

Important review notes

  • move syntax highlighting: The source.config.ts rehypeCodeOptions.langs list does not include move. 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).
  • Content accuracy: The Move examples are documentation-only (not compiled), so correctness depends on matching the actual contract API. The reviewer should verify the Feed/Update/I64 accessor names haven't changed since the original PR was written.

Link to Devin run: https://app.devin.ai/sessions/5fb891efbd31447c9097c416b498b3da
Requested by: @matej-douro


Open with Devin

- 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
@devin-ai-integration
Copy link
Contributor Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

@vercel
Copy link

vercel bot commented Feb 24, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
component-library Ready Ready Preview, Comment Feb 24, 2026 4:01pm
developer-hub Ready Ready Preview, Comment Feb 24, 2026 4:01pm
5 Skipped Deployments
Project Deployment Actions Updated (UTC)
api-reference Skipped Skipped Feb 24, 2026 4:01pm
entropy-explorer Skipped Skipped Feb 24, 2026 4:01pm
insights Skipped Skipped Feb 24, 2026 4:01pm
proposals Skipped Skipped Feb 24, 2026 4:01pm
staking Skipped Skipped Feb 24, 2026 4:01pm

Request Review

@github-actions
Copy link
Contributor

github-actions bot commented Feb 24, 2026

🤖 llms.txt Content Review

Analyzed documentation changes — no updates to curated content required.

Products reviewed: llms-price-feeds-pro.txt, llms-price-feeds.txt

Note: Product files contain curated content. Deep dive page URLs in each file link to individual .mdx pages for full detail.

Files analyzed
apps/developer-hub/content/docs/price-feeds/pro/integrate-as-consumer/index.mdx
apps/developer-hub/content/docs/price-feeds/pro/integrate-as-consumer/sui.mdx

Copy link
Contributor Author

@devin-ai-integration devin-ai-integration bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 3 additional findings.

Open in Devin Review

Copy link
Contributor Author

@devin-ai-integration devin-ai-integration bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Devin Review found 1 new potential issue.

View 7 additional findings in Devin Review.

Open in Devin Review

@aditya520 aditya520 merged commit f80a3f9 into main Feb 24, 2026
11 checks passed
@aditya520 aditya520 deleted the devin/1771919839-fix-sui-lazer-docs branch February 24, 2026 18:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants