Feature request: add missing class method visibility

[Firehed]

Feature Request

Nearly identical to Rector\Php71\Rector\ClassConst\PublicConstantVisibilityRector: it would be a great addition to have a rule to add missing visibility declarations to class methods.

Note: I was able to achieve this locally by changing the aforementioned class's getNodeTypes() to include \PhpParser\Node\Stmt\ClassMethod::class. That alone seemed to be sufficient, and I didn't get any false-positives with global functions or closures.

Diff

 class Foo
 {
-    function bar()
+    public function bar()
     {

[TomasVotruba]

Hi,
this could be quite easy to implement.
Just add public modifier to ClassMethod in case it's null.

Would you like to contribute this rule?

[Firehed]

I'm happy to look into it!

A couple questions before getting started:

New rule or extend existing one to have broader scope?

If new:

• Any naming preference for the new class?
• Should it extend the existing rule (currently declared final), copy-paste most of the internals, or compose the existing one in as a proxy of sorts?

If existing:

• Should the behavior be configurable? Should it default to the current, constant-only scope?

Thanks!

[TomasVotruba]

New rule that extends AbstractRector class would be the way to go.
Check this docs on how to add a rule: https://github.com/rectorphp/rector/blob/main/docs/create_own_rule.md

Name like "ExplicitPublicClassMethodRector" sounds clear.