Possible regression due sprintf: TooManyArguments

[ro0NL]

After bumping global phar installation from v5 to master, including a cache removal, we see a 130 issues about sprintf in our codebase:

Too many arguments for the number of placeholders in sprintf (see https://psalm.dev/026)
$output->writeln(sprintf('Handling product %d => %d (%d)', $productId,  $categoryId, $position));

[psalm-github-bot]

Hey @ro0NL, can you reproduce the issue on https://psalm.dev ?

[ro0NL]

Sadly no

[orklah]

Can you tell us in which PHP version you use to run Psalm (the runtime version, not the analysis version)

cc @kkmuffme

[ro0NL]

right, both php 7.4 still

[orklah]

Note: I also reproduce on my work codebase on PHP 7.4. Seems like detection based on sprintf return is broken before PHP 8

[kkmuffme]

@ro0NL could you test the code of #9943 to see if it fixes the issue for you?