Skip to content

fix(stellar-sdk-helpers): return total fee from prepareSorobanTx - #1226

Merged
collinsezedike merged 1 commit into
drydocs:mainfrom
directorfloo:Chorestellarsdkhelperscleanupafeecommentthatcontradictsthecodeanunderstatedreturnedfeeandanunreachablefallback
Oct 25, 2026
Merged

collinsezedike merged 1 commit into
drydocs:mainfrom
directorfloo:Chorestellarsdkhelperscleanupafeecommentthatcontradictsthecodeanunderstatedreturnedfeeandanunreachablefallback

Conversation

@directorfloo

@directorfloo directorfloo commented Oct 23, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • Correct the submitKeeperOperation attempt comment to 0-indexed.
  • Return the assembled transaction total fee from prepareSorobanTx and document it.
  • Drop the unreachable fallback in vaults.ts.
  • Add a test asserting the returned fee includes the inclusion fee.

Verification

  • pnpm --filter @meridian/stellar-sdk-helpers test (576 passed)
  • pnpm --filter @meridian/stellar-sdk-helpers typecheck

Closes #1062

@vercel

vercel Bot commented Oct 23, 2026

Copy link
Copy Markdown

@elizabethklien3-pixel is attempting to deploy a commit to the Collins' projects Team on Vercel.

A member of the Team first needs to authorize it.

@directorfloo
directorfloo force-pushed the Chorestellarsdkhelperscleanupafeecommentthatcontradictsthecodeanunderstatedreturnedfeeandanunreachablefallback branch 2 times, most recently from 6d1b5f9 to 2bd4eaa Compare October 25, 2026 00:40
@directorfloo

Copy link
Copy Markdown
Contributor Author

please merge everything fixed

@collinsezedike
collinsezedike force-pushed the Chorestellarsdkhelperscleanupafeecommentthatcontradictsthecodeanunderstatedreturnedfeeandanunreachablefallback branch from 4b77547 to eb58669 Compare October 25, 2026 16:28
@collinsezedike collinsezedike changed the title chore: update comments for clarity, adjust transaction fee calculatio… fix(stellar-sdk-helpers): return total fee from prepareSorobanTx Oct 25, 2026
@collinsezedike
collinsezedike force-pushed the Chorestellarsdkhelperscleanupafeecommentthatcontradictsthecodeanunderstatedreturnedfeeandanunreachablefallback branch from eb58669 to 81e0ece Compare October 25, 2026 16:48

@collinsezedike collinsezedike left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thank you for the updates, @directorfloo. The fee documentation now matches the implementation and all required checks pass. This is ready to merge; please consider picking up another open issue.

@collinsezedike
collinsezedike merged commit 456e0a5 into drydocs:main Oct 25, 2026
10 of 11 checks passed
@collinsezedike

Copy link
Copy Markdown
Collaborator

Thank you for your contribution, @directorfloo; if you have a moment, starring the repository would help the project reach more people.

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.

[Chore] stellar-sdk-helpers cleanup, a fee comment that contradicts the code, an understated returned fee, and an unreachable fallback

2 participants