Skip to content

Conversation

dk1a
Copy link
Contributor

@dk1a dk1a commented Jul 29, 2025

Currently it's impossible to use entrykit with a custom paymaster client

This override is a simple solution, it may be better to move all the configuration there from Chain and making it just paymaster, rather than paymasterOverride. Related to the TODO for configuring createBundlerClient in general.

Since it's probably not relevant for quarry stuff, I ignore the override there

@dk1a dk1a requested a review from alvrs as a code owner July 29, 2025 15:51
Copy link

changeset-bot bot commented Jul 29, 2025

⚠️ No Changeset found

Latest commit: c8a44e5

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

Copy link

vercel bot commented Jul 29, 2025

@dk1a is attempting to deploy a commit to the Lattice Team on Vercel.

A member of the Team first needs to authorize it.

@dk1a dk1a changed the title feat: add custom paymaster client override feat(entrykit): add custom paymaster client override Jul 29, 2025
@frolic
Copy link
Member

frolic commented Jul 30, 2025

can you share more about your use case? what do you need a custom client for that the chain config options doesn't help with?

@dk1a
Copy link
Contributor Author

dk1a commented Jul 30, 2025

I just need a custom paymaster, the other aspects of the client are untouched.
More specifically I'm helping moving castles use base's paymaster, see https://docs.cdp.coinbase.com/paymaster/guides/quickstart for example, and passing the paymaster client to entrykit as an override

The chain config only allows a simple static contract address, not the full getPaymasterData or other details

This is relevant for other custom paymaster clients too, not just base

@dk1a
Copy link
Contributor Author

dk1a commented Jul 30, 2025

The main reason I need to fork entrykit is that it does registerDelegation, which I also want to be aware of the custom paymaster, otherwise I could've probably just added it to the session client within the app itself

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