Skip to content

feat: add Stellar transaction simulator (dry-run) before submit#27

Closed
Ryjen1 wants to merge 1 commit into
wraith-protocol:developfrom
BuildersWCT:fix/24-stellar-transaction-simulator
Closed

feat: add Stellar transaction simulator (dry-run) before submit#27
Ryjen1 wants to merge 1 commit into
wraith-protocol:developfrom
BuildersWCT:fix/24-stellar-transaction-simulator

Conversation

@Ryjen1

@Ryjen1 Ryjen1 commented May 27, 2026

Copy link
Copy Markdown

Wire simulateTransaction against Soroban RPC into StellarSend so users see predicted fees, return values, and contract events before signing. Simulation failures disable the Send button with decoded contract errors; network errors fall back to letting the user try anyway.

Closes #24

Wire simulateTransaction against Soroban RPC into StellarSend so users
see predicted fees, return values, and contract events before signing.
Simulation failures disable the Send button with decoded contract errors;
network errors fall back to letting the user try anyway.

Closes wraith-protocol#24
@vercel

vercel Bot commented May 27, 2026

Copy link
Copy Markdown

Someone is attempting to deploy a commit to the truthixify's projects Team on Vercel.

A member of the Team first needs to authorize it.

@drips-wave

drips-wave Bot commented May 27, 2026

Copy link
Copy Markdown

@Ryjen1 Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@truthixify truthixify changed the base branch from main to develop May 27, 2026 17:16
@truthixify

Copy link
Copy Markdown
Contributor

Heads up — we've just set up a develop branch as the integration target for all Drips × Stellar Wave contributions, and I've retargeted this PR's base from main to develop on your behalf. The diff is unchanged; nothing to do on your end.

Going forward, please branch from and target develop. We'll periodically merge developmain as we cut releases. Thanks for contributing!

@truthixify

Copy link
Copy Markdown
Contributor

The simulator wiring is what we want for issue #24: simulateTransaction against Soroban RPC, SimulationCard UI, useSimulateTransaction hook, error decoding helpers, plus Playwright coverage.

StellarSend.tsx is rewritten heavily in this PR and #13 (input validation) just merged into develop with overlapping changes. Could you rebase?

git fetch origin
git rebase origin/develop
git push --force-with-lease

When you resolve StellarSend.tsx, keep both: the validation constants and balance-debounce from develop, layered with your debounced simulate-call flow. The SimulationCard component, hook, and lib files should land cleanly. Thanks @Ryjen1.

@truthixify

Copy link
Copy Markdown
Contributor

The simulator + SimulationCard + useSimulateTransaction + Soroban error decoding + e2e spec are the right shape for issue #24.

Real conflicts now:

git fetch origin
git rebase origin/develop
git push --force-with-lease

For StellarSend.tsx: keep develop's validation, name resolution, payment-link prefill, sponsored-withdrawal display, then layer your useSimulateTransaction call and simulate-result panel into the existing form. For playwright.config.ts: keep develop's structure, add your stellar-simulate.spec.ts alongside the existing test file.

Thanks @Ryjen1.

@truthixify

Copy link
Copy Markdown
Contributor

Closing as we wrap up this round of the Wraith × Drips × Stellar Wave program.

Thanks for the contribution. The work and review history stays on this PR for reference. A new set of issues will go up shortly under the same Stellar Wave label if you want to pick something fresh next round.

If you specifically want to land the work that's already in this PR (post-wave), feel free to reopen and rebase against develop. Otherwise, no action needed.

@truthixify truthixify closed this Jun 18, 2026
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.

Stellar transaction simulator (dry-run) before submit

2 participants