Skip to content

fix: recreate smart account client after Safe deployment - #1945

Merged
MusabShakeel576 merged 1 commit into
masterfrom
claude/cherrypick-safe-recreate-to-master
Apr 15, 2026
Merged

fix: recreate smart account client after Safe deployment#1945
MusabShakeel576 merged 1 commit into
masterfrom
claude/cherrypick-safe-recreate-to-master

Conversation

@MusabShakeel576

Copy link
Copy Markdown
Contributor

Summary

Cherry-pick from qa (merged PR #1943):

  • Recreate the Safe smart account client after deploying the Safe on the card withdrawal chain. The permissionless library caches factory/initCode at client creation time, so the original client still includes deployment data in subsequent UserOps even after the Safe is deployed, causing InvalidSignature (0x8baa579f) reverts.

Test plan

  • Verify card withdrawal works for a user whose Safe was just deployed on Base
  • Verify card withdrawal still works for users with an already-deployed Safe (deployment step is skipped)

https://claude.ai/code/session_01UdFXo6FkqQhCozNm7xXm7U

The permissionless library caches factory/initCode at client creation time.
After deploying the Safe via no-op, the same client still sends factory data
in subsequent UserOps causing simulation failures. Recreate the client after
deployment so the withdrawal UserOp has no factory fields.

https://claude.ai/code/session_01UdFXo6FkqQhCozNm7xXm7U
@vercel

vercel Bot commented Apr 15, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

2 Skipped Deployments
Project Deployment Actions Updated (UTC)
solid-app Ignored Ignored Apr 15, 2026 7:19am
solid-app-staging Ignored Ignored Apr 15, 2026 7:19am

Request Review

@MusabShakeel576
MusabShakeel576 merged commit 81c2736 into master Apr 15, 2026
4 of 5 checks passed
@MusabShakeel576
MusabShakeel576 deleted the claude/cherrypick-safe-recreate-to-master branch April 15, 2026 07:31
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