Outdated UserOperation type

[bh2smith]

I am running into some errors with sendUserOperation attempting to use the relay-kit:

Error: {"message":"UserOperation mismatch, EntryPoint expects UserOperation of version 0.7","code":-32602}

According to these docs:
https://eips.ethereum.org/EIPS/eip-4337#rpc-methods-eth-namespace

The initCode field should be split into factory & factoryData

safe-core-sdk/packages/relay-kit/src/packs/safe-4337/SafeOperation.ts

Lines 71 to 89 in 4b8db6d

	toUserOperation(): UserOperation {
	return {
	sender: this.data.safe,
	nonce: ethers.toBeHex(this.data.nonce),
	initCode: this.data.initCode,
	callData: this.data.callData,
	callGasLimit: this.data.callGasLimit,
	verificationGasLimit: this.data.verificationGasLimit,
	preVerificationGas: this.data.preVerificationGas,
	maxFeePerGas: this.data.maxFeePerGas,
	maxPriorityFeePerGas: this.data.maxPriorityFeePerGas,
	paymasterAndData: this.data.paymasterAndData,
	signature: ethers.solidityPacked(
	['uint48', 'uint48', 'bytes'],
	[this.data.validAfter, this.data.validUntil, this.encodedSignatures()]
	)
	}
	}
	}

Note that we are using the entry point supplied by @safe-global/safe-modules-deployments

Its a bit funny because the PimlicoFeeEstimator doesn't complian about this userOperation type on gas estimation.

After splitting the initCode into factory and factoryData we also ran into an issue with the packed PaymasterData:

Error: {"message":"Validation error: Unrecognized key(s) in object: 'paymasterAndData' at \"params[0]\"","code":-32601}

It seems that we should be using:

      paymaster, // address
      paymasterVerificationGasLimit, // uint256
      paymasterPostOpGasLimit, // uint256
      paymasterData, // bytes

[dasanra]

Hey @bh2smith , currently the relay kit is only compatible with entrypoint v0.6. There is no expected yet to support v0.7.

Using a v0.6 entrypoint should solve your issue.

[dasanra]

About to add v0.7 support with #1103