ts-expect-error will error if it's no longer needed, so it's better in almost all cases. There's some moments where we need to keep ts-ignore though:
- things that fail only with algoliasearch v3 or v4
- optional packages (places)
- code that doesn't always error

At the same time updated the algoliasearch-helper to a version that includes `showParentLevel` in the typings

…nfiniteHits (#4820)

* fix(types): change results type to use Hit instead of AlgoliaHits

The types Hit and AlgoliaHit are mostly the same.

type Hit = {
  __position: number;
  __queryID?: string;
} & AlgoliaHit;

source: https://github.com/algolia/instantsearch.js/blob/21b6c53d4b1a3f3dd3a2375a5cfe0a31bef6ded6/src/types/results.ts#L60-L63

As of today, the hits exposed via results indeed [contain __position and
__queryID](https://github.com/algolia/instantsearch.js/blob/21b6c53d4b1a3f3dd3a2375a5cfe0a31bef6ded6/src/connectors/hits/connectHits.ts#L158).

* fix(types): add `hits` and `results` to `InfiniteHitsRenderState`

While trying use `InfiniteHitsRenderState` in Angular InstantSearch I
noticed how `hits` and `results` while being [part of the state expored
by
`connectInfiniteHits`](https://github.com/algolia/instantsearch.js/blob/master/src/connectors/infinite-hits/connectInfiniteHits.ts#L331-L342)
are not actually visible on the type.

* chore: release v4.25.3

* Update CHANGELOG.md

Co-authored-by: Clément Vannicatte <20689156+shortcuts@users.noreply.github.com>

Co-authored-by: Haroen Viaene <hello@haroen.me>
Co-authored-by: Clément Vannicatte <20689156+shortcuts@users.noreply.github.com>