Annotate `load`/`loads`/`parse` return value as `Any`

[jonathandung]

Currently, I'm getting a bunch of extremely frustrating type checker errors since the return type of the parse function was inferred to be something like (Unknown | Any | list[Unknown | Any | list[Unknown | Any | list[Unknown] | bool | None] | bool | None] | bool | None), even if I know the json5 I'm parsing is a top-level object. The above type doesn't have the required method, and I don't want to use # type: ignore[attr-defined,reportOptionalMemberAccess,reportAttributeAccessIssue] every time.

I propose to add an inline return type annotation of Any to each affected function, so that type checkers think all attributes are there. This is what the stub file for the builtin json module does for their load and loads. Of course, it would then be the user's responsibility to use only the properties and methods they know are present.

[jonathandung]

It appears that this has already been done, and I just haven't updated the package to the latest version. I will close this issue now.

[jonathandung]

I see that it has only been done for load. loads should return Any, and parse should return tuple[Any, None, int] | tuple[None, str, int]. I will fix this in my PR.

[jonathandung]

I find the helper functions difficult to type, since they are recursive, and my knowledge is insufficient. Maybe merge this first? Thanks

[dpranke]

Sure, merged. Don't worry if you don't get anywhere on this; it's possible I didn't do it already because it didn't make enough sense to bother :).