SAFE Wallet Account Abstraction ERC 4337 fix

[Sharqiewicz]

The SAFE Wallet AA Issue

When using Safe Wallet, the initial transaction hash returned is a safeTxHash which differs from the final on-chain transaction hash. This is because Safe uses a multi-signature approach where:

1. The initial safeTxHash is generated when a transaction is proposed
2. This transaction needs to be confirmed by the required number of owners (based on the threshold)
3. Only after sufficient confirmations and execution will there be an actual on-chain transaction with a different hash
   This creates challenges when tracking transaction status using standard wagmi/viem hooks, as they look for the initial safeTxHash on-chain which doesn't exist.

Instead of wagmi waitForTransactionReceipt use

import { waitForTransactionConfirmationReceipt } from '...';

await waitForTransactionConfirmationReceipt(hash, wagmiConfig);

[netlify]

✅ Deploy Preview for pendulum-pay ready!

Name	Link
🔨 Latest commit	5f7898a
🔍 Latest deploy log	https://app.netlify.com/sites/pendulum-pay/deploys/67b5d66b5eaaf900087ed8bd
😎 Deploy Preview	https://deploy-preview-480--pendulum-pay.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify site configuration.

[github-actions]

yarn.lock changes

Click to toggle table visibility

Name	Status	Previous	Current
@peculiar/asn1-schema		-	2.3.15
@safe-global/api-kit		-	2.5.9
@safe-global/protocol-kit		-	5.2.2
@safe-global/safe-deployments		-	1.37.29
@safe-global/safe-modules-deployments		-	2.2.7
@safe-global/types-kit		-	1.0.2
asn1js		-	3.0.5
es-shim-unscopables		1.0.2	1.1.0
pathe		2.0.2	2.0.3
pvtsutils		-	1.3.6
pvutils		-	1.1.3
viem		2.23.1	2.23.2

[Sharqiewicz]

@pendulum-chain/devs Ready for review ✅

[Sharqiewicz]

One thing worth mentioning is that for setting the maximum and current number of signers, we use Zustand outside of a React component and custom hook, directly within a plain function.

https://github.com/pmndrs/zustand#readingwriting-state-and-reacting-to-changes-outside-of-components

[ebma]

Great changes, looks good to me in general 🙏 I only left some questions regarding the structure.

[Sharqiewicz]

@pendulum-chain/devs Ready ✅

[gianfra-t]

Looks good! I think we should test one offramp, if you haven't already. Do you have a safe wallet to test?

[Sharqiewicz]

@gianfra-t I tested 2 days ago, should I test again?

[gianfra-t]

We can test on staging then, to save some funds.

[ebma]

Yes, let's do that 👍