Skip to content

feat: self hosted sponsorship new arch#115

Open
vr16x wants to merge 3 commits intodevelopfrom
feat/self-hosted-sponsorship-new-arch
Open

feat: self hosted sponsorship new arch#115
vr16x wants to merge 3 commits intodevelopfrom
feat/self-hosted-sponsorship-new-arch

Conversation

@vr16x
Copy link
Collaborator

@vr16x vr16x commented Jul 4, 2025

PR-Codex overview

This PR introduces a new merkle-tree utility, updates the DEFAULT_PREFIX to be exported, and modifies the sponsorSupertransaction function to utilize the new createMerkleTree function for signature verification. Additionally, it increases the size limit and updates some dependencies.

Detailed summary

  • Added export for merkle-tree in src/sdk/modules/utils/index.ts.
  • Changed DEFAULT_PREFIX to be an exported constant in src/sdk/clients/decorators/mee/signQuote.ts.
  • Implemented createMerkleTree in src/sdk/modules/utils/merkle-tree.ts.
  • Updated sponsorSupertransaction to use createMerkleTree for signature verification.
  • Increased the size limit in size-limit.json from 65 kB to 98 kB.
  • Updated dependencies in package.json for @openzeppelin/merkle-tree, @vitest/coverage-istanbul, and vitest.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

@vr16x vr16x requested review from alexcambose, fichiokaku and filmakarov and removed request for filmakarov July 4, 2025 13:17
@github-actions
Copy link

github-actions bot commented Jul 4, 2025

size-limit report 📦

Path Size
core (esm) 92.25 KB (+54.19% 🔺)
core (cjs) 190.34 KB (+17.25% 🔺)
bundler (tree-shaking) 5.8 KB (0%)
paymaster (tree-shaking) 543 B (0%)


console.log("trigger.amount", trigger.amount)
console.log("fusionQuote.trigger.amount", fusionQuote.trigger.amount)
console.log(getMeeScanLink(fusionQuote.quote.hash))
Copy link
Contributor

Choose a reason for hiding this comment

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

nit remove logs

})

const quote = await meeClient.getQuote({
instructions: [...transferInstruction, ...instructions, ...instructions],
Copy link
Contributor

Choose a reason for hiding this comment

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

should these ...instructions be added 2 times?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Yeah, it is part of the test.

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.

2 participants

Comments