ox as an optional peer dependency

[carlosrsabreu]

When installing v0.2.43, I ran into a problem when building my project using pnpm:

Import trace for requested module:
../../node_modules/.pnpm/[email protected][email protected][email protected][email protected][email protected]_/node_modules/permissionless/_esm/accounts/index.js
../../node_modules/.pnpm/@[email protected]_@[email protected]_@[email protected]_@[email protected]__@types_424d5a67b0e25619433d3b7545479202/node_modules/@fourt/sdk/dist/index.js
./providers/UserProvider/index.tsx
./app/layout.tsx

../../node_modules/.pnpm/[email protected][email protected][email protected][email protected][email protected]_/node_modules/permissionless/_esm/accounts/kernel/utils/signMessage.js
Module not found: Can't resolve 'ox'

Then, I suspect the problem lies in considering the ox as an optional peer dependency, which is why it is not being installed. Can anyone explain to me why ox is an optional peer dependency?

permissionless.js/packages/permissionless/package.json

Lines 97 to 101 in 4287124

	"peerDependenciesMeta": {
	"ox": {
	"optional": true
	}
	}

I resolved this issue by installing the ox package in version 0.6.7 in my application, and the build started working again.

[plusminushalf]

ox is an optional peer dependency because not all smart accounts depends on it. You have to use ox only when using kernel smart account. Now that I think about it, maybe it is better to remove optional for now.

[plusminushalf]

Will release a new version - #400

[carlosrsabreu]

Got it, thank you!

[chaitanyapotti]

@plusminushalf Can you look at #401 as well. It solves a common peer dep issue when using latest viem with permissionless