feat(rector): Skip classes with a name too common from auto use statement

Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>

diff --git a/build/rector.php b/build/rector.php
index 9df995a00448642f26b75c5039fa98c77f9286e9..db39fb4ce485d8c5b349fdadfa5d4f49ee096036 100644 (file)
--- a/build/rector.php
+++ b/build/rector.php
@@ -21,7 +21,21 @@ class NextcloudNamespaceSkipVoter implements ClassNameImportSkipVoterInterface {
                'OCA',
                'OCP',
        ];
+       private array $skippedClassNames = [
+               'Backend',
+               'Connection',
+               'Exception',
+               'IManager',
+               'IProvider',
+               'Manager',
+               'Plugin',
+               'Provider',
+       ];
        public function shouldSkip(File $file, FullyQualifiedObjectType $fullyQualifiedObjectType, Node $node) : bool {
+               if (in_array($fullyQualifiedObjectType->getShortName(), $this->skippedClassNames)) {
+                       // Skip common class names to avoid confusion
+                       return true;
+               }
                foreach ($this->namespacePrefixes as $prefix) {
                        if (str_starts_with($fullyQualifiedObjectType->getClassName(), $prefix . '\\')) {
                                // Import Nextcloud namespaces