[DX] Deprecate NonPhpRectorInterface, the only rule and its file processor, to make Rector handle exlusively PHP

[TomasVotruba]

Rector as any other tools like ECS, PHPStan, PHP-CS-Fixer, PHPUnit, Pest etc. focuses on working with PHP files.
Yet, in some cases it can change yaml, twig or latte files too. While using PHP classes in templates is a bad practise, it cannot be avoided like in Symfony configs.

That's why added a "special file processor" that was able to handle some non-php files too. This is not-so-known feature, that works in a magic way:

• those files are processed only if passed into the explicit paths, e.g.

# this will process them
bin/rector config src

# this will not
bin/rector src

• it only renames class names matching regex pattern:

# this is skipped
use App\SomeType;

class: SomeType

• it only renames classes, not constant, nor method names etc.

This magic is hidden deep and it leads to unnexpected cases, that some class are renamed and some not.
Instead these files should be skipped completely and Rector handle PHP files only. This will show same behavior as other CLI tools mention above and make you handle non-PHP files yourself in consistent way 👍

That's why I've decided to remove this magic feature before going Rector 1.0 stable

[staabm]

I like this direction 👍

[TomasVotruba]

@staabm Thanks for feedback, I feel this has been a mistake past 2 years 🙏 Time to fix bad design choices

[samsonasik]

/cc @grandmaster44 @sabbelasichon FYI