Revert "feat: Mark all declarations as internal (humbug#882)"

This reverts commit 7dbccc0.

Author: theofidry

specs/misc/internal-declarations.php

@@ -38,7 +38,6 @@ final class Configuration
      * @param array<string, array{string, string}> $excludedFilesWithContents Array of tuple
      *                                                                        with the first argument being the file path and
      *                                                                        the second its contents
-     * @param bool                                 $tagDeclarationsAsInternal Whether a @internal tag should be added to the symbols declarations.
      */
     public function __construct(
         private ?string $path,
@@ -47,8 +46,7 @@ public function __construct(
         private array $filesWithContents,
         private array $excludedFilesWithContents,
         private Patcher $patcher,
-        private SymbolsConfiguration $symbolsConfiguration,
-        private bool $tagDeclarationsAsInternal,
+        private SymbolsConfiguration $symbolsConfiguration
     ) {
         self::validatePrefix($prefix);
 
@@ -84,7 +82,6 @@ public function withPrefix(string $prefix): self
             $this->excludedFilesWithContents,
             $this->patcher,
             $this->symbolsConfiguration,
-            $this->tagDeclarationsAsInternal,
         );
     }
 
@@ -109,7 +106,6 @@ public function withFilesWithContents(array $filesWithContents): self
             $this->excludedFilesWithContents,
             $this->patcher,
             $this->symbolsConfiguration,
-            $this->tagDeclarationsAsInternal,
         );
     }
 
@@ -139,7 +135,6 @@ public function withPatcher(Patcher $patcher): self
             $this->excludedFilesWithContents,
             $patcher,
             $this->symbolsConfiguration,
-            $this->tagDeclarationsAsInternal,
         );
     }
 
@@ -153,11 +148,6 @@ public function getSymbolsConfiguration(): SymbolsConfiguration
         return $this->symbolsConfiguration;
     }
 
-    public function shouldTagDeclarationsAsInternal(): bool
-    {
-        return $this->tagDeclarationsAsInternal;
-    }
-
     private static function validatePrefix(string $prefix): void
     {
         if (1 !== preg_match(self::PREFIX_PATTERN, $prefix)) {

src/Configuration/Configuration.php

@@ -96,7 +96,6 @@ public function create(?string $path = null, array $paths = []): Configuration
         $finders = self::retrieveFinders($config);
         $filesFromPaths = self::retrieveFilesFromPaths($paths);
         $filesWithContents = self::retrieveFilesWithContents(chain($filesFromPaths, ...$finders));
-        $tagDeclarationsAsInternal = $config[ConfigurationKeys::TAG_DECLARATIONS_AS_INTERNAL] ?? true;
 
         return new Configuration(
             $path,
@@ -106,7 +105,6 @@ public function create(?string $path = null, array $paths = []): Configuration
             self::retrieveFilesWithContents($excludedFiles),
             new PatcherChain($patchers),
             $symbolsConfiguration,
-            $tagDeclarationsAsInternal,
         );
     }
 

src/Configuration/ConfigurationFactory.php

@@ -25,7 +25,6 @@ final class ConfigurationKeys
     public const EXCLUDED_FILES_KEYWORD = 'exclude-files';
     public const FINDER_KEYWORD = 'finders';
     public const PATCHERS_KEYWORD = 'patchers';
-    public const TAG_DECLARATIONS_AS_INTERNAL = 'tag-declarations-as-internal';
 
     public const EXPOSE_GLOBAL_CONSTANTS_KEYWORD = 'expose-global-constants';
     public const EXPOSE_GLOBAL_CLASSES_KEYWORD = 'expose-global-classes';
@@ -47,7 +46,6 @@ final class ConfigurationKeys
         self::EXCLUDED_FILES_KEYWORD,
         self::FINDER_KEYWORD,
         self::PATCHERS_KEYWORD,
-        self::TAG_DECLARATIONS_AS_INTERNAL,
         self::EXPOSE_GLOBAL_CONSTANTS_KEYWORD,
         self::EXPOSE_GLOBAL_CLASSES_KEYWORD,
         self::EXPOSE_GLOBAL_FUNCTIONS_KEYWORD,