sprintf ESLint rule does not check for numbered placeholders

[swissspidy]

Describe the bug

While working on #19777 I noticed that the @wordpress/valid-sprintf rule does not catch the following issue:

sprintf(
	/* translators: 1: number of blocks. 2: average rating. */
	_n(
		'This author has %d block, with an average rating of %d.',
		'This author has %d blocks, with an average rating of %d.',
		authorBlockCount
	),
	authorBlockCount,
	authorBlockRating
)

As you can see, there are two occurrences of %d in each string, which makes it impossible for translators to change the order of placeholders in the sentence.

Expected behavior

It should warn about multiple placeholders being found, and should suggest using numbered placeholders instead:

sprintf(
	/* translators: 1: number of blocks. 2: average rating. */
	_n(
		'This author has %1$d block, with an average rating of %2$d.',
		'This author has %1$d blocks, with an average rating of %2$d.',
		authorBlockCount
	),
	authorBlockCount,
	authorBlockRating
)

[swissspidy]

The rule can actually be much improved here by simply following what WPCS does in PHP: https://github.com/WordPress/WordPress-Coding-Standards/blob/2f927b0ba2bfcbffaa8f3251c086e109302d6622/WordPress/Sniffs/WP/I18nSniff.php

That one contains two different regex for catching unordered and ordered placeholders, making it easy to verify whether the code is wrong. Shouldn't be difficult to port that part.