I think this one need e2e test to avoid regression and back and forth check, eg:

• multiple --dry-run check should ok.
• multiple succesfull keep changing, eg, the following code got twice changed should ok.

if ($a && $b) {
    return true;
}

on config:

<?php

declare(strict_types=1);

use Rector\CodingStyle\Rector\Stmt\NewlineAfterStatementRector;
use Rector\Config\RectorConfig;
use Rector\EarlyReturn\Rector\If_\ChangeAndIfToEarlyReturnRector;

return static function (RectorConfig $rectorConfig): void {
    $rectorConfig->rule(ChangeAndIfToEarlyReturnRector::class);
    $rectorConfig->rule(NewlineAfterStatementRector::class);
};

First: https://getrector.com/demo/55b308a8-af17-48cb-b2be-9c0d26ab60c4
Second: https://getrector.com/demo/53cd71fb-1851-44d0-aa1e-ba7e5329db77

Thanks for guidance! Will work on separate PRs this week-end to add those e2e tests

Is this still WIP ?

It needs one more e2e test for consecutive succesfull.

for consecutive dry-run, there was already PR #3616

I planned to work on that e2e test this week-end

@samsonasik @staabm here it is: #3666

I think the PR itself needs a new test-case reproducing the issue it is trying to fix

@staabm good point! I'll try to come with a test in following days, but the final code is ready if you want to have a look, see that all tests pass now, even the consecutive runs test.
@samsonasik Idea is to cache files only when Rector didn't changed them, so when they are in their final state. So following runs will still go through previously changed files to attempt to change them again.
Maybe not the best way, but at least it preserve actual Rector vision on multiple runs and makes the cache reliable

Credits to @stof for the idea! Thank you

Please rename commit message with double quote "" , that will cause failure build downgrade

@samsonasik that revert commit is a temporary one just to generate a failing build, to show the issue.
will remove it from the final PR, showing that the PR solves the issue

Here the failing test without the fix: https://github.com/rectorphp/rector-src/actions/runs/4850942011/jobs/8644319975?pr=3614
See that with the fix by caching at the end instead of caching at the beginning, that previously failing test is now green: https://github.com/rectorphp/rector-src/actions/runs/4850961546/jobs/8644357227?pr=3614

@TomasVotruba @samsonasik @staabm this is now ready for review!

Thank you 👍

I think we need the same fix in https://github.com/easy-coding-standard/easy-coding-standard. @yguedidi could you check that?

@staabm I'm not using ECS, so I can't guarantee that I can come with a PR any time soon there sorry.
It's fine if some else inspire from this PR to implement it in ECS :)