export json types in utils

[dynst]

cosmjs/packages/json-rpc/src/types.ts

Lines 12 to 16 in ca243f5

	export interface JsonRpcSuccessResponse {
	readonly jsonrpc: "2.0";
	readonly id: JsonRpcId;
	readonly result: any;
	}

JsonCompatibleDictionary seems like it should be used in a lot of the many places that currently use any #1840, and also seems like it should be renamed JsonObject.

I'm guessing migrating to using real json validation like zod would be overkill.

https://www.npmjs.com/package/zod

But maybe it's not needed to reinvent JsonValue if we use this:

https://www.npmjs.com/package/type-fest

[webmaster128]

Interesting, looks good at first glance

I'm guessing migrating to using real json validation like zod would be overkill.

Yeah, agreed. Also I don't like external libraries in our public types which makes it very hard to replace dependencies.

[dynst]

external libraries in our public types which makes it very hard to replace dependencies.

And even just upgrading them, zod had a whole thing about how carefully the upgrade had to be designed due to how entangled it is with APIs.

colinhacks/zod#4371
https://zod.dev/v4

[dynst]

type-fest actually has its own widely adopted definition of JsonValue, package has 219 million downloads a week. Thoughts on using that in a public API?

https://github.com/sindresorhus/type-fest/blob/v5.1.0/source/json-value.d.ts
https://github.com/sindresorhus/type-fest/blob/v5.1.0/source/readonly-deep.d.ts