Add missing return types

src/Symfony/Component/DependencyInjection/Argument/ReferenceSetArgumentTrait.php

@@ -44,6 +44,8 @@ public function getValues(): array
 
     /**
      * @param Reference[] $values The service references to put in the set
+     *
+     * @return void
      */
     public function setValues(array $values)
     {

src/Symfony/Component/DependencyInjection/Container.php

@@ -217,7 +217,7 @@ public function get(string $id, int $invalidBehavior = self::EXCEPTION_ON_INVALI
      *
      * As a separate method to allow "get()" to use the really fast `??` operator.
      */
-    private static function make(self $container, string $id, int $invalidBehavior)
+    private static function make(self $container, string $id, int $invalidBehavior): ?object
     {
         if (isset($container->loading[$id])) {
             throw new ServiceCircularReferenceException($id, array_merge(array_keys($container->loading), [$id]));
@@ -338,6 +338,8 @@ public static function underscore(string $id): string
 
     /**
      * Creates a service by requiring its factory file.
+     *
+     * @return mixed
      */
     protected function load(string $file)
     {

src/Symfony/Component/DependencyInjection/Dumper/PhpDumper.php

@@ -1325,7 +1325,7 @@ public function isCompiled(): bool
         if ($this->asFiles && !$this->inlineFactories) {
             $code .= <<<'EOF'
 
-    protected function load($file, $lazyLoad = true)
+    protected function load($file, $lazyLoad = true): mixed
     {
         if (class_exists($class = __NAMESPACE__.'\\'.$file, false)) {
             return $class::do($this, $lazyLoad);

src/Symfony/Component/DependencyInjection/Extension/Extension.php

@@ -28,11 +28,17 @@ abstract class Extension implements ExtensionInterface, ConfigurationExtensionIn
 {
     private array $processedConfigs = [];
 
+    /**
+     * @return string|false
+     */
     public function getXsdValidationBasePath()
     {
         return false;
     }
 
+    /**
+     * @return string
+     */
     public function getNamespace()
     {
         return 'http://example.org/schema/dic/'.$this->getAlias();
@@ -67,6 +73,9 @@ public function getAlias(): string
         return Container::underscore($classBaseName);
     }
 
+    /**
+     * @return ConfigurationInterface|null
+     */
     public function getConfiguration(array $config, ContainerBuilder $container)
     {
         $class = static::class;

src/Symfony/Component/DependencyInjection/Loader/YamlFileLoader.php

@@ -332,7 +332,7 @@ private function isUsingShortSyntax(array $service): bool
     /**
      * @throws InvalidArgumentException When tags are invalid
      */
-    private function parseDefinition(string $id, array|string|null $service, string $file, array $defaults, bool $return = false, bool $trackBindings = true)
+    private function parseDefinition(string $id, array|string|null $service, string $file, array $defaults, bool $return = false, bool $trackBindings = true): Definition|Alias|null
     {
         if (preg_match('/^_[a-zA-Z0-9_]*$/', $id)) {
             throw new InvalidArgumentException(sprintf('Service names that start with an underscore are reserved. Rename the "%s" service or define it in XML instead.', $id));
@@ -706,6 +706,8 @@ private function parseDefinition(string $id, array|string|null $service, string
         } else {
             $this->setDefinition($id, $definition);
         }
+
+        return null;
     }
 
     /**

src/Symfony/Component/DependencyInjection/ParameterBag/FrozenParameterBag.php

@@ -34,26 +34,41 @@ public function __construct(
         $this->resolved = true;
     }
 
+    /**
+     * @return never
+     */
     public function clear()
     {
         throw new LogicException('Impossible to call clear() on a frozen ParameterBag.');
     }
 
+    /**
+     * @return never
+     */
     public function add(array $parameters)
     {
         throw new LogicException('Impossible to call add() on a frozen ParameterBag.');
     }
 
+    /**
+     * @return never
+     */
     public function set(string $name, array|bool|string|int|float|\UnitEnum|null $value)
     {
         throw new LogicException('Impossible to call set() on a frozen ParameterBag.');
     }
 
+    /**
+     * @return never
+     */
     public function deprecate(string $name, string $package, string $version, string $message = 'The parameter "%s" is deprecated.')
     {
         throw new LogicException('Impossible to call deprecate() on a frozen ParameterBag.');
     }
 
+    /**
+     * @return never
+     */
     public function remove(string $name)
     {
         throw new LogicException('Impossible to call remove() on a frozen ParameterBag.');

src/Symfony/Component/DependencyInjection/ParameterBag/ParameterBag.php

@@ -255,6 +255,9 @@ public function resolveString(string $value, array $resolving = []): mixed
         }, $value);
     }
 
+    /**
+     * @return bool
+     */
     public function isResolved()
     {
         return $this->resolved;

src/Symfony/Component/DependencyInjection/Tests/Fixtures/php/services10_as_files.txt

@@ -93,7 +93,7 @@ class ProjectServiceContainer extends Container
         return require $this->containerDir.\DIRECTORY_SEPARATOR.'removed-ids.php';
     }
 
-    protected function load($file, $lazyLoad = true)
+    protected function load($file, $lazyLoad = true): mixed
     {
         if (class_exists($class = __NAMESPACE__.'\\'.$file, false)) {
             return $class::do($this, $lazyLoad);

src/Symfony/Component/DependencyInjection/Tests/Fixtures/php/services9_as_files.txt

@@ -627,7 +627,7 @@ class ProjectServiceContainer extends Container
         return require $this->containerDir.\DIRECTORY_SEPARATOR.'removed-ids.php';
     }
 
-    protected function load($file, $lazyLoad = true)
+    protected function load($file, $lazyLoad = true): mixed
     {
         if (class_exists($class = __NAMESPACE__.'\\'.$file, false)) {
             return $class::do($this, $lazyLoad);

src/Symfony/Component/DependencyInjection/Tests/Fixtures/php/services_deprecated_parameters_as_files.txt

@@ -73,7 +73,7 @@ class ProjectServiceContainer extends Container
         return true;
     }
 
-    protected function load($file, $lazyLoad = true)
+    protected function load($file, $lazyLoad = true): mixed
     {
         if (class_exists($class = __NAMESPACE__.'\\'.$file, false)) {
             return $class::do($this, $lazyLoad);

src/Symfony/Component/DependencyInjection/Tests/Fixtures/php/services_non_shared_lazy_as_files.txt

@@ -103,7 +103,7 @@ class ProjectServiceContainer extends Container
         return true;
     }
 
-    protected function load($file, $lazyLoad = true)
+    protected function load($file, $lazyLoad = true): mixed
     {
         if (class_exists($class = __NAMESPACE__.'\\'.$file, false)) {
             return $class::do($this, $lazyLoad);

src/Symfony/Component/DomCrawler/AbstractUriElement.php

@@ -115,6 +115,8 @@ protected function canonicalizePath(string $path): string
      *
      * @param \DOMElement $node A \DOMElement instance
      *
+     * @return void
+     *
      * @throws \LogicException If given node is not an anchor
      */
     abstract protected function setNode(\DOMElement $node);

src/Symfony/Component/DomCrawler/Field/FormField.php

@@ -118,6 +118,8 @@ public function isDisabled(): bool
 
     /**
      * Initializes the form field.
+     *
+     * @return void
      */
     abstract protected function initialize();
 }

src/Symfony/Component/ExpressionLanguage/Compiler.php

@@ -28,6 +28,9 @@ public function __construct(array $functions)
         $this->functions = $functions;
     }
 
+    /**
+     * @return array
+     */
     public function getFunction(string $name)
     {
         return $this->functions[$name];

src/Symfony/Component/ExpressionLanguage/Node/Node.php

@@ -80,6 +80,11 @@ public function evaluate(array $functions, array $values)
         return $results;
     }
 
+    /**
+     * @return array
+     *
+     * @throws \BadMethodCallException when this node cannot be transformed to an array
+     */
     public function toArray()
     {
         throw new \BadMethodCallException(sprintf('Dumping a "%s" instance is not supported yet.', static::class));

src/Symfony/Component/ExpressionLanguage/Parser.php

@@ -130,6 +130,9 @@ private function doParse(TokenStream $stream, ?array $names = []): Node\Node
         return $node;
     }
 
+    /**
+     * @return Node\Node
+     */
     public function parseExpression(int $precedence = 0)
     {
         $expr = $this->getPrimary();
@@ -151,6 +154,9 @@ public function parseExpression(int $precedence = 0)
         return $expr;
     }
 
+    /**
+     * @return Node\Node
+     */
     protected function getPrimary()
     {
         $token = $this->stream->current;

src/Symfony/Component/ExpressionLanguage/SerializedParsedExpression.php

@@ -11,6 +11,8 @@
 
 namespace Symfony\Component\ExpressionLanguage;
 
+use Symfony\Component\ExpressionLanguage\Node\Node;
+
 /**
  * Represents an already parsed expression.
  *
@@ -30,6 +32,9 @@ public function __construct(string $expression, string $nodes)
         $this->nodes = $nodes;
     }
 
+    /**
+     * @return Node
+     */
     public function getNodes()
     {
         return unserialize($this->nodes);

src/Symfony/Component/Filesystem/Filesystem.php

@@ -401,7 +401,7 @@ public function hardlink(string $originFile, string|iterable $targetFiles)
     /**
      * @param string $linkType Name of the link type, typically 'symbolic' or 'hard'
      */
-    private function linkException(string $origin, string $target, string $linkType)
+    private function linkException(string $origin, string $target, string $linkType): never
     {
         if (self::$lastError) {
             if ('\\' === \DIRECTORY_SEPARATOR && str_contains(self::$lastError, 'error code(1314)')) {

src/Symfony/Component/Form/AbstractType.php

@@ -48,11 +48,17 @@ public function configureOptions(OptionsResolver $resolver)
     {
     }
 
+    /**
+     * @return string
+     */
     public function getBlockPrefix()
     {
         return StringUtil::fqcnToBlockPrefix(static::class) ?: '';
     }
 
+    /**
+     * @return string|null
+     */
     public function getParent()
     {
         return FormType::class;