Skip to content

docs: document devnet simulation false-positive troubleshooting (#489)#499

Open
akshitj11 wants to merge 1 commit into
phantom:masterfrom
akshitj11:docs/devnet-simulation-faq-489
Open

docs: document devnet simulation false-positive troubleshooting (#489)#499
akshitj11 wants to merge 1 commit into
phantom:masterfrom
akshitj11:docs/devnet-simulation-faq-489

Conversation

@akshitj11
Copy link
Copy Markdown

@akshitj11 akshitj11 commented May 25, 2026

Add FAQ explaining when Phantom's devnet simulation warning is a false
positive, with local verification steps and signTransaction guidance.

Fixes #489

Summary by CodeRabbit

Documentation

  • Enhanced documentation on alternative transaction signing methods, explaining pre-sign simulation behavior and RPC context handling
  • New devnet troubleshooting guide addressing "transaction reverted during simulation" warnings, complete with local verification techniques, false-positive identification checklist, and escalation recommendations

Review Change Stack

Add FAQ explaining when Phantom's devnet simulation warning is a false
positive, with local verification steps and signTransaction guidance.

Fixes phantom#489

Co-authored-by: Cursor <cursoragent@cursor.com>
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 25, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 7939fe76-0fc1-462f-bde2-3fcf41f86318

📥 Commits

Reviewing files that changed from the base of the PR and between f1e613e and abaa108.

📒 Files selected for processing (2)
  • getting-started-with-solana/sending-a-transaction.md
  • resources/faq.md

📝 Walkthrough

Walkthrough

This PR adds documentation explaining Phantom's devnet "transaction reverted during simulation" false-positive warning. Updates span tutorial guidance on signing methods and a comprehensive FAQ entry covering problem diagnosis, verification techniques, and escalation steps.

Changes

Devnet Simulation Warning Guidance

Layer / File(s) Summary
Tutorial guidance on signing methods and devnet simulation behavior
getting-started-with-solana/sending-a-transaction.md
The "Other Signing Methods" section clarifies that signAndSendTransaction keeps simulation and submission in the same RPC context to reduce misleading warnings. The "Signing a Transaction (Without Sending)" section adds a devnet-specific warning explaining that signTransaction triggers pre-sign simulation and may show false "reverted during simulation" messages, with instructions to locally simulate and verify Testnet Mode alignment.
FAQ explanation and diagnostic framework for the warning
resources/faq.md
A new FAQ section explains the "transaction reverted during simulation" warning message, why it occurs even when transactions later succeed on-chain, and the simulation versus broadcast pipeline that leads to false positives. A related warnings table and checklist distinguish the simulation revert from other Phantom approval warnings and help determine whether a warning is a false positive or a real failure.
FAQ verification guidance and escalation steps
resources/faq.md
Added sections for local transaction verification (including a JavaScript helper using simulateTransaction with blockhash refresh), commitment-alignment rules for dapp RPC integration, and escalation criteria for persistent failures, real errors, and when to seek help via GitHub issues or the Phantom developer Discord.
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'docs: document devnet simulation false-positive troubleshooting' directly and accurately summarizes the main change: adding documentation about devnet simulation false-positive warnings.
Linked Issues check ✅ Passed The PR directly addresses issue #489 by adding FAQ documentation explaining the devnet simulation false-positive warning, providing verification steps, and offering guidance for dapps using signTransaction.
Out of Scope Changes check ✅ Passed All changes are documentation-only and directly related to addressing issue #489's requirements: explaining false-positives, providing troubleshooting steps, and offering implementation guidance.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Comment @coderabbitai help to get the list of available commands and usage tips.

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.

Devnet: "transaction reverted during simulation" false positive on every transaction (succeeds on submit) — same symptom as #208

1 participant