Move beforeTraverse() to global Node since beforeTraverse() not overrideable

[samsonasik]

since beforeTraverse() become final,

• [internal] Mark beforeTraverse() in AbstractRector as @final to prevent override and miss-use rectorphp/rector-src#7727

the use of beforeTraverse here is invalid,

Using custom node visitor is not ideal for shareable rules, since user then need another step to register visitor, this PR make it keep working automatically, by register on global Node early on the particular rules that requires set attributes.

[samsonasik]

@GeniJaho @driftingly @peterfox Ready to merge 👍

after this PR merged, I think new immediate release is needed so next composer update when sometime rector released new version, there will be less to no regression as possible :)

[samsonasik]

For note: after next rector release, which new FileNode will exists, instead of global node that can be any node, the FileNode and Namespace_ should be used instead as a "top" node :)

This PR ensure keep work first whenever next rector release anytime soon when we are may not available to immediatelly patch :)

[samsonasik]

@GeniJaho @driftingly @peterfox I've bumped to latest rector ^2.2.13 and fix compatible code:

• ensure use isInFirstLevelStatement() from scope to reduce repetitive init traverse 9cc30cf
• fix patch of static SimpleCallableNodeTraverser::traverseNodesWithCallable usage c1e50cc

Ready to merge and this needs immediate new release :)

[peterfox]

Looks good. I'm wandering if what we need is a mechanism where a default config is loaded with the Laravel Package or the package has it's own Config object at this point that preloads a few things. Then we could use NodeVisitors.

This will be good to resolve the current situation.

[samsonasik]

There is default config:

https://github.com/driftingly/rector-laravel/blob/main/config/config.php

already included in sets, but then user that only want to use single rule and done, will require manually include it, that's another step for user.