fix(Configure): prevent aroundRadius="all" throwing

[Haroenv]

Summary

Because React props are not writable, behaviour inside _parseNumbers > merge in the helper requires the configure props to be writable.

fixes #6136

See #6011 for the ultimate cause of this issue; _parseNumbers mutates the arguments as it's much more performant than making a copy on every interaction (this is a very hot function).

Result

This bug is reproducible by simply rendering <Configure aroundRadius="all" /> or any other numeric (

instantsearch/packages/algoliasearch-helper/src/SearchParameters/index.js

Lines 239 to 251 in d1aa720

	var numberKeys = [
	'aroundPrecision',
	'aroundRadius',
	'getRankingInfo',
	'minWordSizefor2Typos',
	'minWordSizefor1Typo',
	'page',
	'maxValuesPerFacet',
	'distinct',
	'minimumAroundRadius',
	'hitsPerPage',
	'minProximity',
	];

) key as a non-number string. After this PR, the bug no longer happens.

Note that of course this has a performance penalty (or regression) for the Configure widget. There's no way around this as far as I can tell, unless we'd manually make the props writable, but that seems like a bad idea.

[codesandbox-ci]

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit bd84738:

Sandbox	Source
example-instantsearch-getting-started	Configuration
example-react-instantsearch-getting-started	Configuration
example-react-instantsearch-next-app-dir-example	Configuration
example-react-instantsearch-next-routing-example	Configuration
example-vue-instantsearch-getting-started	Configuration