fix: make predefined filters API generic and prevent dropping regular sort params if provided

[MartinCupela]

Goal

The original predefined filters implementation omitted DX, when it used Record<string, unknown> for filters definitions in predefined filters types. We have ChannelsFilter type that can be passed to predefined filters types to help users type the filters easier.

Also the channelsQueryRequest method was dropping regular sort parameter, if it detected predefined_filters in the methods payload. But this is incorrect in cases, when somebody wants to use predefined filter that has not sort param internally and wants to rely on dynamically provided regular sort object. If the server-side stored predefined filter definition contains its own sort, the regular sort param is ignored server-side.

I have added fixme comment so that channelsQueryRequest returns the whole response payload as it contains predefined_filters payload, that is currently being discarded and cannot be used client-side to follow sorting and filtering rules client-side. This change would be breaking and so it is not implemented with this PR.

[github-actions]

Size Change: +62 B (+0.02%)

Total Size: 381 kB

📦 View Changed

Filename	Size	Change
dist/cjs/index.browser.js	127 kB	+21 B (+0.02%)
dist/cjs/index.node.js	128 kB	+20 B (+0.02%)
dist/esm/index.mjs	126 kB	+21 B (+0.02%)

_{compressed-size-action}

[stream-ci-bot]

🎉 This PR is included in version 9.43.2 🎉

The release is available on:

• GitHub release
• npm package (@latest dist-tag)

Your semantic-release bot 📦🚀