spiral / framework

src/Core/src/Exception/Container/RecursiveProxyException.php

Spacing +3 added lines, -3 removed lines

@@ -13,11 +13,11 @@
     public function __construct(
         public readonly string $alias,
         public readonly ?string $bindingScope = null,
-        public readonly ?array $callingScope = null,
-    ) {
+        public readonly ? array $callingScope = null,
+    ){
         $message = "Recursive proxy detected for `$alias`.";
         $bindingScope === null or $message .= "\nBinding scope: `$bindingScope`.";
-        $callingScope === null or $message .= "\nCalling scope: `" . \implode('.', $callingScope) . '`.';
+        $callingScope === null or $message .= "\nCalling scope: `".\implode('.', $callingScope).'`.';
         parent::__construct($message);
     }
 }

src/Core/src/Config/Proxy.php

Spacing +2 added lines, -2 removed lines

@@ -17,8 +17,8 @@
     public function __construct(
         protected readonly string $interface,
         public readonly bool $singleton = false,
-        public readonly ?\Closure $fallbackFactory = null,
-    ) {
+        public readonly ? \Closure $fallbackFactory = null,
+    ){
         \interface_exists($interface) or throw new \InvalidArgumentException(
             "Interface `{$interface}` does not exist.",
         );

src/Core/src/Internal/Factory.php

Braces +129 added lines, -56 removed lines

@@ -70,17 +70,20 @@ 
      */
     public function make(string $alias, array $parameters = [], Stringable|string|null $context = null): mixed
     {
-        if ($parameters === [] && \array_key_exists($alias, $this->state->singletons)) {
+        if ($parameters === [] && \array_key_exists($alias, $this->state->singletons))
+        {
             return $this->state->singletons[$alias];
         }
 
         $binding = $this->state->bindings[$alias] ?? null;
 
-        if ($binding === null) {
+        if ($binding === null)
+        {
             return $this->resolveWithoutBinding($alias, $parameters, $context);
         }
 
-        try {
+        try
+        {
             $this->tracer->push(
                 false,
                 action: 'resolve from binding',
@@ -110,7 +113,9 @@ 
                     ->resolveWeakReference($binding, $alias, $context, $parameters),
                 default => $binding,
             };
-        } finally {
+        }
+        finally
+        {
             $this->state->bindings[$alias] ??= $binding;
             $this->tracer->pop(true);
             $this->tracer->pop(false);
@@ -124,18 +129,23 @@ 
     private function resolveInjector(Config\Injectable $binding, Ctx $ctx, array $arguments)
     {
         $context = $ctx->context;
-        try {
+        try
+        {
             $reflection = $ctx->reflection ??= new \ReflectionClass($ctx->class);
-        } catch (\ReflectionException $e) {
+        }
+        catch (\ReflectionException $e)
+        {
             throw new ContainerException($e->getMessage(), $e->getCode(), $e);
         }
 
         $injector = $binding->injector;
 
-        try {
+        try
+        {
             $injectorInstance = \is_object($injector) ? $injector : $this->container->get($injector);
 
-            if (!$injectorInstance instanceof InjectorInterface) {
+            if (!$injectorInstance instanceof InjectorInterface)
+            {
                 throw new InjectionException(
                     \sprintf(
                         "Class '%s' must be an instance of InjectorInterface for '%s'.",
@@ -162,7 +172,8 @@ 
                 default => (string)$context,
             });
 
-            if (!$reflection->isInstance($instance)) {
+            if (!$reflection->isInstance($instance))
+            {
                 throw new InjectionException(
                     \sprintf(
                         "Invalid injection response for '%s'.",
@@ -172,7 +183,9 @@ 
             }
 
             return $instance;
-        } finally {
+        }
+        finally
+        {
             $this->state->bindings[$ctx->class] ??= $binding;
         }
     }
@@ -191,7 +204,8 @@ 
             //Binding is pointing to something else
             : $this->make($binding->alias, $arguments, $context);
 
-        if ($binding->singleton && $arguments === []) {
+        if ($binding->singleton && $arguments === [])
+        {
             $this->state->singletons[$alias] = $result;
         }
 
@@ -200,7 +214,8 @@ 
 
     private function resolveProxy(Config\Proxy $binding, string $alias, Stringable|string|null $context): mixed
     {
-        if ($context instanceof RetryContext) {
+        if ($context instanceof RetryContext)
+        {
             return $binding->fallbackFactory === null
                 ? throw new RecursiveProxyException(
                     $alias,
@@ -211,7 +226,8 @@ 
 
         $result = Proxy::create(new \ReflectionClass($binding->getInterface()), $context, new Attribute\Proxy());
 
-        if ($binding->singleton) {
+        if ($binding->singleton)
+        {
             $this->state->singletons[$alias] = $result;
         }
 
@@ -252,11 +268,14 @@ 
         array $arguments,
     ): mixed {
         $ctx = new Ctx(alias: $alias, class: $alias, context: $context, singleton: $binding->singleton);
-        try {
+        try
+        {
             $instance = $binding::class === Config\Factory::class && $binding->getParametersCount() === 0
                 ? ($binding->factory)()
                 : $this->invoker->invoke($binding->factory, $arguments);
-        } catch (NotCallableException $e) {
+        }
+        catch (NotCallableException $e)
+        {
             throw new ContainerException(
                 $this->tracer->combineTraceMessage(\sprintf('Invalid binding for `%s`.', $ctx->alias)),
                 $e->getCode(),
@@ -275,19 +294,24 @@ 
     ): ?object {
         $avoidCache = $arguments !== [];
 
-        if (($avoidCache || $binding->reference->get() === null) && \class_exists($alias)) {
-            try {
+        if (($avoidCache || $binding->reference->get() === null) && \class_exists($alias))
+        {
+            try
+            {
                 $this->tracer->push(false, alias: $alias, source: WeakReference::class, context: $context);
 
                 $object = $this->createInstance(
                     new Ctx(alias: $alias, class: $alias, context: $context),
                     $arguments,
                 );
-                if ($avoidCache) {
+                if ($avoidCache)
+                {
                     return $object;
                 }
                 $binding->reference = WeakReference::create($object);
-            } catch (\Throwable) {
+            }
+            catch (\Throwable)
+            {
                 throw new ContainerException(
                     $this->tracer->combineTraceMessage(
                         \sprintf(
@@ -297,7 +321,9 @@ 
                         )
                     )
                 );
-            } finally {
+            }
+            finally
+            {
                 $this->tracer->pop();
             }
         }
@@ -312,19 +338,26 @@ 
     ): mixed {
         $parent = $this->scope->getParentFactory();
 
-        if ($parent !== null) {
-            try {
+        if ($parent !== null)
+        {
+            try
+            {
                 $this->tracer->push(false, ...[
                     'current scope' => $this->scope->getScopeName(),
                     'jump to parent scope' => $this->scope->getParentScope()->getScopeName(),
                 ]);
                 /** @psalm-suppress TooManyArguments */
                 return $parent->make($alias, $parameters, $context);
-            } catch (BadScopeException $e) {
-                if ($this->scope->getScopeName() !== $e->getScope()) {
+            }
+            catch (BadScopeException $e)
+            {
+                if ($this->scope->getScopeName() !== $e->getScope())
+                {
                     throw $e;
                 }
-            } catch (ContainerExceptionInterface $e) {
+            }
+            catch (ContainerExceptionInterface $e)
+            {
                 $className = match (true) {
                     $e instanceof NotFoundException => NotFoundException::class,
                     $e instanceof RecursiveProxyException => throw $e,
@@ -334,19 +367,24 @@ 
                     'Can\'t resolve `%s`.',
                     $alias,
                 )), previous: $e);
-            } finally {
+            }
+            finally
+            {
                 $this->tracer->pop(false);
             }
         }
 
         $this->tracer->push(false, action: 'autowire', alias: $alias, context: $context);
-        try {
+        try
+        {
             //No direct instructions how to construct class, make is automatically
             return $this->autowire(
                 new Ctx(alias: $alias, class: $alias, context: $context),
                 $parameters,
             );
-        } finally {
+        }
+        finally
+        {
             $this->tracer->pop(false);
         }
     }
@@ -393,13 +431,16 @@ 
         Ctx $ctx,
         array $arguments,
     ): object {
-        if ($this->options->checkScope) {
+        if ($this->options->checkScope)
+        {
             // Check scope name
             $ctx->reflection = new \ReflectionClass($instance);
             $scopeName = ($ctx->reflection->getAttributes(Attribute\Scope::class)[0] ?? null)?->newInstance()->name;
-            if ($scopeName !== null) {
+            if ($scopeName !== null)
+            {
                 $scope = $this->scope;
-                while ($scope->getScopeName() !== $scopeName) {
+                while ($scope->getScopeName() !== $scopeName)
+                {
                     $scope = $scope->getParentScope() ?? throw new BadScopeException($scopeName, $instance::class);
                 }
             }
@@ -428,26 +469,33 @@ 
         array $arguments,
     ): object {
         $class = $ctx->class;
-        try {
+        try
+        {
             $ctx->reflection = $reflection = new \ReflectionClass($class);
-        } catch (\ReflectionException $e) {
+        }
+        catch (\ReflectionException $e)
+        {
             throw new ContainerException($e->getMessage(), $e->getCode(), $e);
         }
 
         // Check scope name
-        if ($this->options->checkScope) {
+        if ($this->options->checkScope)
+        {
             $scope = ($reflection->getAttributes(Attribute\Scope::class)[0] ?? null)?->newInstance()->name;
-            if ($scope !== null && $scope !== $this->scope->getScopeName()) {
+            if ($scope !== null && $scope !== $this->scope->getScopeName())
+            {
                 throw new BadScopeException($scope, $class);
             }
         }
 
         // We have to construct class using external injector when we know the exact context
-        if ($arguments === [] && $this->binder->hasInjector($class)) {
+        if ($arguments === [] && $this->binder->hasInjector($class))
+        {
             return $this->resolveInjector($this->state->bindings[$ctx->class], $ctx, $arguments);
         }
 
-        if (!$reflection->isInstantiable()) {
+        if (!$reflection->isInstantiable())
+        {
             $itIs = match (true) {
                 $reflection->isEnum() => 'Enum',
                 $reflection->isAbstract() => 'Abstract class',
@@ -460,12 +508,16 @@ 
 
         $constructor = $reflection->getConstructor();
 
-        if ($constructor !== null) {
-            try {
+        if ($constructor !== null)
+        {
+            try
+            {
                 $this->tracer->push(false, action: 'resolve arguments', signature: $constructor);
                 $this->tracer->push(true);
                 $args = $this->resolver->resolveArguments($constructor, $arguments, $this->options->validateArguments);
-            } catch (ValidationException $e) {
+            }
+            catch (ValidationException $e)
+            {
                 throw new ContainerException(
                     $this->tracer->combineTraceMessage(
                         \sprintf(
@@ -475,22 +527,31 @@ 
                         )
                     ),
                 );
-            } finally {
+            }
+            finally
+            {
                 $this->tracer->pop(true);
                 $this->tracer->pop(false);
             }
-            try {
+            try
+            {
                 // Using constructor with resolved arguments
                 $this->tracer->push(false, call: "$class::__construct", arguments: $args);
                 $this->tracer->push(true);
                 $instance = new $class(...$args);
-            } catch (\TypeError $e) {
+            }
+            catch (\TypeError $e)
+            {
                 throw new WrongTypeException($constructor, $e);
-            } finally {
+            }
+            finally
+            {
                 $this->tracer->pop(true);
                 $this->tracer->pop(false);
             }
-        } else {
+        }
+        else
+        {
             // No constructor specified
             $instance = $reflection->newInstance();
         }
@@ -508,13 +569,15 @@ 
         $instance = $this->runInflector($instance);
 
         //Declarative singletons
-        if ($this->isSingleton($ctx)) {
+        if ($this->isSingleton($ctx))
+        {
             $this->state->singletons[$ctx->alias] = $instance;
         }
 
         // Register finalizer
         $finalizer = $this->getFinalizer($ctx, $instance);
-        if ($finalizer !== null) {
+        if ($finalizer !== null)
+        {
             $this->state->finalizers[] = $finalizer;
         }
 
@@ -526,12 +589,14 @@ 
      */
     private function isSingleton(Ctx $ctx): bool
     {
-        if ($ctx->singleton === true) {
+        if ($ctx->singleton === true)
+        {
             return true;
         }
 
         /** @psalm-suppress RedundantCondition https://github.com/vimeo/psalm/issues/9489 */
-        if ($ctx->reflection->implementsInterface(SingletonInterface::class)) {
+        if ($ctx->reflection->implementsInterface(SingletonInterface::class))
+        {
             return true;
         }
 
@@ -545,7 +610,8 @@ 
          * @var Attribute\Finalize|null $attribute
          */
         $attribute = ($ctx->reflection->getAttributes(Attribute\Finalize::class)[0] ?? null)?->newInstance();
-        if ($attribute === null) {
+        if ($attribute === null)
+        {
             return null;
         }
 
@@ -559,10 +625,14 @@ 
     {
         $scope = $this->scope;
 
-        while ($scope !== null) {
-            foreach ($this->state->inflectors as $class => $inflectors) {
-                if ($instance instanceof $class) {
-                    foreach ($inflectors as $inflector) {
+        while ($scope !== null)
+        {
+            foreach ($this->state->inflectors as $class => $inflectors)
+            {
+                if ($instance instanceof $class)
+                {
+                    foreach ($inflectors as $inflector)
+                    {
                         $instance = $inflector->getParametersCount() > 1
                             ? $this->invoker->invoke($inflector->inflector, [$instance])
                             : ($inflector->inflector)($instance);
@@ -578,9 +648,12 @@ 
 
     private function validateArguments(ContextFunction $reflection, array $arguments = []): bool
     {
-        try {
+        try
+        {
             $this->resolver->validateArguments($reflection, $arguments);
-        } catch (\Throwable) {
+        }
+        catch (\Throwable)
+        {
             return false;
         }
 

Spacing +68 added lines, -68 removed lines

@@ -68,19 +68,19 @@ 
      *
      * @throws \Throwable
      */
-    public function make(string $alias, array $parameters = [], Stringable|string|null $context = null): mixed
+    public function make(string $alias, array $parameters = [], Stringable | string | null $context = null): mixed
     {
-        if ($parameters === [] && \array_key_exists($alias, $this->state->singletons)) {
+        if ($parameters === [] && \array_key_exists($alias, $this->state->singletons)){
             return $this->state->singletons[$alias];
         }
 
         $binding = $this->state->bindings[$alias] ?? null;
 
-        if ($binding === null) {
+        if ($binding === null){
             return $this->resolveWithoutBinding($alias, $parameters, $context);
         }
 
-        try {
+        try{
             $this->tracer->push(
                 false,
                 action: 'resolve from binding',
@@ -110,7 +110,7 @@ 
                     ->resolveWeakReference($binding, $alias, $context, $parameters),
                 default => $binding,
             };
-        } finally {
+        }finally{
             $this->state->bindings[$alias] ??= $binding;
             $this->tracer->pop(true);
             $this->tracer->pop(false);
@@ -124,18 +124,18 @@ 
     private function resolveInjector(Config\Injectable $binding, Ctx $ctx, array $arguments)
     {
         $context = $ctx->context;
-        try {
+        try{
             $reflection = $ctx->reflection ??= new \ReflectionClass($ctx->class);
-        } catch (\ReflectionException $e) {
+        }catch (\ReflectionException $e){
             throw new ContainerException($e->getMessage(), $e->getCode(), $e);
         }
 
         $injector = $binding->injector;
 
-        try {
+        try{
             $injectorInstance = \is_object($injector) ? $injector : $this->container->get($injector);
 
-            if (!$injectorInstance instanceof InjectorInterface) {
+            if (!$injectorInstance instanceof InjectorInterface){
                 throw new InjectionException(
                     \sprintf(
                         "Class '%s' must be an instance of InjectorInterface for '%s'.",
@@ -162,7 +162,7 @@ 
                 default => (string)$context,
             });
 
-            if (!$reflection->isInstance($instance)) {
+            if (!$reflection->isInstance($instance)){
                 throw new InjectionException(
                     \sprintf(
                         "Invalid injection response for '%s'.",
@@ -172,7 +172,7 @@ 
             }
 
             return $instance;
-        } finally {
+        }finally{
             $this->state->bindings[$ctx->class] ??= $binding;
         }
     }
@@ -180,27 +180,27 @@ 
     private function resolveAlias(
         Config\Alias $binding,
         string $alias,
-        Stringable|string|null $context,
+        Stringable | string | null $context,
         array $arguments,
     ): mixed {
         $result = $binding->alias === $alias
             ? $this->autowire(
-                new Ctx(alias: $alias, class: $binding->alias, context: $context, singleton: $binding->singleton),
+                new Ctx(alias : $alias, class : $binding->alias, context : $context, singleton : $binding->singleton),
                 $arguments,
             )
             //Binding is pointing to something else
             : $this->make($binding->alias, $arguments, $context);
 
-        if ($binding->singleton && $arguments === []) {
+        if ($binding->singleton && $arguments === []){
             $this->state->singletons[$alias] = $result;
         }
 
         return $result;
     }
 
-    private function resolveProxy(Config\Proxy $binding, string $alias, Stringable|string|null $context): mixed
+    private function resolveProxy(Config\Proxy $binding, string $alias, Stringable | string | null $context): mixed
     {
-        if ($context instanceof RetryContext) {
+        if ($context instanceof RetryContext){
             return $binding->fallbackFactory === null
                 ? throw new RecursiveProxyException(
                     $alias,
@@ -211,7 +211,7 @@ 
 
         $result = Proxy::create(new \ReflectionClass($binding->getInterface()), $context, new Attribute\Proxy());
 
-        if ($binding->singleton) {
+        if ($binding->singleton){
             $this->state->singletons[$alias] = $result;
         }
 
@@ -221,13 +221,13 @@ 
     private function resolveShared(
         Config\Shared $binding,
         string $alias,
-        Stringable|string|null $context,
+        Stringable | string | null $context,
         array $arguments,
     ): object {
         $avoidCache = $arguments !== [];
         return $avoidCache
             ? $this->createInstance(
-                new Ctx(alias: $alias, class: $binding->value::class, context: $context),
+                new Ctx(alias : $alias, class : $binding->value::class, context : $context),
                 $arguments,
             )
             : $binding->value;
@@ -236,7 +236,7 @@ 
     private function resolveAutowire(
         Config\Autowire $binding,
         string $alias,
-        Stringable|string|null $context,
+        Stringable | string | null $context,
         array $arguments,
     ): mixed {
         $target = $binding->autowire->alias;
@@ -250,17 +250,17 @@ 
     }
 
     private function resolveFactory(
-        Config\Factory|Config\DeferredFactory $binding,
+        Config\Factory | Config\DeferredFactory $binding,
         string $alias,
-        Stringable|string|null $context,
+        Stringable | string | null $context,
         array $arguments,
     ): mixed {
         $ctx = new Ctx(alias: $alias, class: $alias, context: $context, singleton: $binding->singleton);
-        try {
+        try{
             $instance = $binding::class === Config\Factory::class && $binding->getParametersCount() === 0
                 ? ($binding->factory)()
                 : $this->invoker->invoke($binding->factory, $arguments);
-        } catch (NotCallableException $e) {
+        }catch (NotCallableException $e){
             throw new ContainerException(
                 $this->tracer->combineTraceMessage(\sprintf('Invalid binding for `%s`.', $ctx->alias)),
                 $e->getCode(),
@@ -274,24 +274,24 @@ 
     private function resolveWeakReference(
         Config\WeakReference $binding,
         string $alias,
-        Stringable|string|null $context,
+        Stringable | string | null $context,
         array $arguments,
     ): ?object {
         $avoidCache = $arguments !== [];
 
-        if (($avoidCache || $binding->reference->get() === null) && \class_exists($alias)) {
-            try {
+        if (($avoidCache || $binding->reference->get() === null) && \class_exists($alias)){
+            try{
                 $this->tracer->push(false, alias: $alias, source: WeakReference::class, context: $context);
 
                 $object = $this->createInstance(
                     new Ctx(alias: $alias, class: $alias, context: $context),
                     $arguments,
                 );
-                if ($avoidCache) {
+                if ($avoidCache){
                     return $object;
                 }
                 $binding->reference = WeakReference::create($object);
-            } catch (\Throwable) {
+            }catch (\Throwable){
                 throw new ContainerException(
                     $this->tracer->combineTraceMessage(
                         \sprintf(
@@ -301,7 +301,7 @@ 
                         )
                     )
                 );
-            } finally {
+            }finally{
                 $this->tracer->pop();
             }
         }
@@ -312,23 +312,23 @@ 
     private function resolveWithoutBinding(
         string $alias,
         array $parameters = [],
-        Stringable|string|null $context = null
+        Stringable | string | null $context = null
     ): mixed {
         $parent = $this->scope->getParentFactory();
 
-        if ($parent !== null) {
-            try {
+        if ($parent !== null){
+            try{
                 $this->tracer->push(false, ...[
                     'current scope' => $this->scope->getScopeName(),
                     'jump to parent scope' => $this->scope->getParentScope()->getScopeName(),
                 ]);
                 /** @psalm-suppress TooManyArguments */
                 return $parent->make($alias, $parameters, $context);
-            } catch (BadScopeException $e) {
-                if ($this->scope->getScopeName() !== $e->getScope()) {
+            }catch (BadScopeException $e){
+                if ($this->scope->getScopeName() !== $e->getScope()){
                     throw $e;
                 }
-            } catch (ContainerExceptionInterface $e) {
+            }catch (ContainerExceptionInterface $e){
                 $className = match (true) {
                     $e instanceof NotFoundException => NotFoundException::class,
                     $e instanceof RecursiveProxyException => throw $e,
@@ -338,19 +338,19 @@ 
                     'Can\'t resolve `%s`.',
                     $alias,
                 )), previous: $e);
-            } finally {
+            }finally{
                 $this->tracer->pop(false);
             }
         }
 
         $this->tracer->push(false, action: 'autowire', alias: $alias, context: $context);
-        try {
+        try{
             //No direct instructions how to construct class, make is automatically
             return $this->autowire(
                 new Ctx(alias: $alias, class: $alias, context: $context),
                 $parameters,
             );
-        } finally {
+        }finally{
             $this->tracer->pop(false);
         }
     }
@@ -371,7 +371,7 @@ 
                 &&
                 (isset($this->state->injectors[$ctx->class]) || $this->binder->hasInjector($ctx->class))
         ))
-        ) {
+        ){
             throw new NotFoundException($this->tracer->combineTraceMessage(\sprintf(
                 'Can\'t resolve `%s`: undefined class or binding `%s`.',
                 $this->tracer->getRootAlias(),
@@ -397,13 +397,13 @@ 
         Ctx $ctx,
         array $arguments,
     ): object {
-        if ($this->options->checkScope) {
+        if ($this->options->checkScope){
             // Check scope name
             $ctx->reflection = new \ReflectionClass($instance);
             $scopeName = ($ctx->reflection->getAttributes(Attribute\Scope::class)[0] ?? null)?->newInstance()->name;
-            if ($scopeName !== null) {
+            if ($scopeName !== null){
                 $scope = $this->scope;
-                while ($scope->getScopeName() !== $scopeName) {
+                while ($scope->getScopeName() !== $scopeName){
                     $scope = $scope->getParentScope() ?? throw new BadScopeException($scopeName, $instance::class);
                 }
             }
@@ -432,26 +432,26 @@ 
         array $arguments,
     ): object {
         $class = $ctx->class;
-        try {
+        try{
             $ctx->reflection = $reflection = new \ReflectionClass($class);
-        } catch (\ReflectionException $e) {
+        }catch (\ReflectionException $e){
             throw new ContainerException($e->getMessage(), $e->getCode(), $e);
         }
 
         // Check scope name
-        if ($this->options->checkScope) {
+        if ($this->options->checkScope){
             $scope = ($reflection->getAttributes(Attribute\Scope::class)[0] ?? null)?->newInstance()->name;
-            if ($scope !== null && $scope !== $this->scope->getScopeName()) {
+            if ($scope !== null && $scope !== $this->scope->getScopeName()){
                 throw new BadScopeException($scope, $class);
             }
         }
 
         // We have to construct class using external injector when we know the exact context
-        if ($arguments === [] && $this->binder->hasInjector($class)) {
+        if ($arguments === [] && $this->binder->hasInjector($class)){
             return $this->resolveInjector($this->state->bindings[$ctx->class], $ctx, $arguments);
         }
 
-        if (!$reflection->isInstantiable()) {
+        if (!$reflection->isInstantiable()){
             $itIs = match (true) {
                 $reflection->isEnum() => 'Enum',
                 $reflection->isAbstract() => 'Abstract class',
@@ -464,12 +464,12 @@ 
 
         $constructor = $reflection->getConstructor();
 
-        if ($constructor !== null) {
-            try {
+        if ($constructor !== null){
+            try{
                 $this->tracer->push(false, action: 'resolve arguments', signature: $constructor);
                 $this->tracer->push(true);
                 $args = $this->resolver->resolveArguments($constructor, $arguments, $this->options->validateArguments);
-            } catch (ValidationException $e) {
+            }catch (ValidationException $e){
                 throw new ContainerException(
                     $this->tracer->combineTraceMessage(
                         \sprintf(
@@ -479,22 +479,22 @@ 
                         )
                     ),
                 );
-            } finally {
+            }finally{
                 $this->tracer->pop(true);
                 $this->tracer->pop(false);
             }
-            try {
+            try{
                 // Using constructor with resolved arguments
                 $this->tracer->push(false, call: "$class::__construct", arguments: $args);
                 $this->tracer->push(true);
                 $instance = new $class(...$args);
-            } catch (\TypeError $e) {
+            }catch (\TypeError $e){
                 throw new WrongTypeException($constructor, $e);
-            } finally {
+            }finally{
                 $this->tracer->pop(true);
                 $this->tracer->pop(false);
             }
-        } else {
+        }else{
             // No constructor specified
             $instance = $reflection->newInstance();
         }
@@ -512,13 +512,13 @@ 
         $instance = $this->runInflector($instance);
 
         //Declarative singletons
-        if ($this->isSingleton($ctx)) {
+        if ($this->isSingleton($ctx)){
             $this->state->singletons[$ctx->alias] = $instance;
         }
 
         // Register finalizer
         $finalizer = $this->getFinalizer($ctx, $instance);
-        if ($finalizer !== null) {
+        if ($finalizer !== null){
             $this->state->finalizers[] = $finalizer;
         }
 
@@ -530,12 +530,12 @@ 
      */
     private function isSingleton(Ctx $ctx): bool
     {
-        if ($ctx->singleton === true) {
+        if ($ctx->singleton === true){
             return true;
         }
 
         /** @psalm-suppress RedundantCondition https://github.com/vimeo/psalm/issues/9489 */
-        if ($ctx->reflection->implementsInterface(SingletonInterface::class)) {
+        if ($ctx->reflection->implementsInterface(SingletonInterface::class)){
             return true;
         }
 
@@ -549,7 +549,7 @@ 
          * @var Attribute\Finalize|null $attribute
          */
         $attribute = ($ctx->reflection->getAttributes(Attribute\Finalize::class)[0] ?? null)?->newInstance();
-        if ($attribute === null) {
+        if ($attribute === null){
             return null;
         }
 
@@ -563,10 +563,10 @@ 
     {
         $scope = $this->scope;
 
-        while ($scope !== null) {
-            foreach ($this->state->inflectors as $class => $inflectors) {
-                if ($instance instanceof $class) {
-                    foreach ($inflectors as $inflector) {
+        while ($scope !== null){
+            foreach ($this->state->inflectors as $class => $inflectors){
+                if ($instance instanceof $class){
+                    foreach ($inflectors as $inflector){
                         $instance = $inflector->getParametersCount() > 1
                             ? $this->invoker->invoke($inflector->inflector, [$instance])
                             : ($inflector->inflector)($instance);
@@ -582,9 +582,9 @@ 
 
     private function validateArguments(ContextFunction $reflection, array $arguments = []): bool
     {
-        try {
+        try{
             $this->resolver->validateArguments($reflection, $arguments);
-        } catch (\Throwable) {
+        }catch (\Throwable){
             return false;
         }
 

src/Core/src/Internal/Proxy/Resolver.php

Spacing +10 added lines, -10 removed lines

@@ -19,27 +19,27 @@ 
 {
     public static function resolve(
         string $alias,
-        \Stringable|string|null $context = null,
+        \Stringable | string | null $context = null,
         ?ContainerInterface $c = null,
     ): object {
         $c ??= ContainerScope::getContainer() ?? throw new ContainerException('Proxy is out of scope.');
 
-        try {
+        try{
             /** @psalm-suppress TooManyArguments */
             $result = $c->get($alias, $context) ?? throw new ContainerException(
                 'Resolved `null` from the container.',
             );
-        } catch (\Throwable $e) {
+        }catch (\Throwable $e){
             $scope = self::getScope($c);
             throw new ContainerException(
                 $scope === null
                     ? "Unable to resolve `{$alias}` in a Proxy."
                     : \sprintf('Unable to resolve `%s` in a Proxy in `%s` scope.', $alias, \implode('.', $scope)),
-                previous: $e,
+                previous : $e,
             );
         }
 
-        if (!Proxy::isProxy($result)) {
+        if (!Proxy::isProxy($result)){
             return $result;
         }
 
@@ -48,10 +48,10 @@ 
          * to try to get the instance from the Proxy Fallback Factory.
          * If there is no the Proxy Fallback Factory, {@see RecursiveProxyException} will be thrown.
          */
-        try {
+        try{
             /** @psalm-suppress TooManyArguments */
             $result = $c->get($alias, new RetryContext($context));
-        } catch (RecursiveProxyException $e) {
+        }catch (RecursiveProxyException $e){
             throw new RecursiveProxyException($e->alias, $e->bindingScope, self::getScope($c));
         }
 
@@ -66,8 +66,8 @@ 
      */
     private static function getScope(ContainerInterface $c): ?array
     {
-        if (!$c instanceof Container) {
-            if (!Proxy::isProxy($c)) {
+        if (!$c instanceof Container){
+            if (!Proxy::isProxy($c)){
                 return null;
             }
 
@@ -75,7 +75,7 @@ 
         }
 
         return \array_reverse(\array_map(
-            static fn (?string $name): string => $name ?? 'null',
+            static fn (?string $name) : string => $name ?? 'null',
             Introspector::scopeNames($c),
         ));
     }

Braces +16 added lines, -7 removed lines

@@ -24,12 +24,15 @@ 
     ): object {
         $c ??= ContainerScope::getContainer() ?? throw new ContainerException('Proxy is out of scope.');
 
-        try {
+        try
+        {
             /** @psalm-suppress TooManyArguments */
             $result = $c->get($alias, $context) ?? throw new ContainerException(
                 'Resolved `null` from the container.',
             );
-        } catch (\Throwable $e) {
+        }
+        catch (\Throwable $e)
+        {
             $scope = self::getScope($c);
             throw new ContainerException(
                 $scope === null
@@ -39,7 +42,8 @@ 
             );
         }
 
-        if (!Proxy::isProxy($result)) {
+        if (!Proxy::isProxy($result))
+        {
             return $result;
         }
 
@@ -48,10 +52,13 @@ 
          * to try to get the instance from the Proxy Fallback Factory.
          * If there is no the Proxy Fallback Factory, {@see RecursiveProxyException} will be thrown.
          */
-        try {
+        try
+        {
             /** @psalm-suppress TooManyArguments */
             $result = $c->get($alias, new RetryContext($context));
-        } catch (RecursiveProxyException $e) {
+        }
+        catch (RecursiveProxyException $e)
+        {
             throw new RecursiveProxyException($e->alias, $e->bindingScope, self::getScope($c));
         }
 
@@ -66,8 +73,10 @@ 
      */
     private static function getScope(ContainerInterface $c): ?array
     {
-        if (!$c instanceof Container) {
-            if (!Proxy::isProxy($c)) {
+        if (!$c instanceof Container)
+        {
+            if (!Proxy::isProxy($c))
+            {
                 return null;
             }
 

src/Core/src/Internal/Proxy/RetryContext.php

Spacing +3 added lines, -3 removed lines

@@ -15,12 +15,12 @@
      * @param \Stringable|string|null $context Original context.
      */
     public function __construct(
-        public \Stringable|string|null $context = null,
-    ) {
+        public \Stringable | string | null $context = null,
+    ){
     }
 
     public function __toString(): string
     {
-        return (string) $this->context;
+        return (string)$this->context;
     }
 }

src/Framework/Bootloader/Http/RoutesBootloader.php

Spacing +5 added lines, -5 removed lines

@@ -19,7 +19,7 @@ 
 {
     public function init(HttpBootloader $http): void
     {
-        foreach ($this->globalMiddleware() as $middleware) {
+        foreach ($this->globalMiddleware() as $middleware){
             $http->addMiddleware($middleware);
         }
     }
@@ -61,11 +61,11 @@ 
 
     private function registerMiddlewareGroups(BinderInterface $binder, array $groups): void
     {
-        foreach ($groups as $group => $middleware) {
+        foreach ($groups as $group => $middleware){
             $binder
                 ->getBinder('http')
                 ->bind(
-                    'middleware:' . $group,
+                    'middleware:'.$group,
                     static function (FactoryInterface $factory) use ($middleware): LazyPipeline {
                         return $factory->make(LazyPipeline::class)->withAddedMiddleware(...$middleware);
                     }
@@ -75,8 +75,8 @@ 
 
     private function registerMiddlewareForRouteGroups(GroupRegistry $registry, array $groups): void
     {
-        foreach ($groups as $group) {
-            $registry->getGroup($group)->addMiddleware('middleware:' . $group);
+        foreach ($groups as $group){
+            $registry->getGroup($group)->addMiddleware('middleware:'.$group);
         }
     }
 }

Braces +6 added lines, -3 removed lines

@@ -19,7 +19,8 @@ 
 {
     public function init(HttpBootloader $http): void
     {
-        foreach ($this->globalMiddleware() as $middleware) {
+        foreach ($this->globalMiddleware() as $middleware)
+        {
             $http->addMiddleware($middleware);
         }
     }
@@ -61,7 +62,8 @@ 
 
     private function registerMiddlewareGroups(BinderInterface $binder, array $groups): void
     {
-        foreach ($groups as $group => $middleware) {
+        foreach ($groups as $group => $middleware)
+        {
             $binder
                 ->getBinder('http')
                 ->bind(
@@ -75,7 +77,8 @@ 
 
     private function registerMiddlewareForRouteGroups(GroupRegistry $registry, array $groups): void
     {
-        foreach ($groups as $group) {
+        foreach ($groups as $group)
+        {
             $registry->getGroup($group)->addMiddleware('middleware:' . $group);
         }
     }

src/Http/src/LazyPipeline.php

Spacing +7 added lines, -7 removed lines

@@ -39,7 +39,7 @@ 
     public function __construct(
         #[Proxy] private readonly ContainerInterface $container,
         private readonly ?EventDispatcherInterface $dispatcher = null,
-    ) {
+    ){
     }
 
     /**
@@ -47,7 +47,7 @@ 
      *
      * @param MiddlewareInterface ...$middleware List of middleware or its definition.
      */
-    public function withAddedMiddleware(MiddlewareInterface|Autowire|string ...$middleware): self
+    public function withAddedMiddleware(MiddlewareInterface | Autowire | string ...$middleware): self
     {
         $pipeline = clone $this;
         $pipeline->middleware = \array_merge($pipeline->middleware, $middleware);
@@ -59,7 +59,7 @@ 
      *
      * @param MiddlewareInterface ...$middleware List of middleware or its definition.
      */
-    public function withMiddleware(MiddlewareInterface|Autowire|string ...$middleware): self
+    public function withMiddleware(MiddlewareInterface | Autowire | string ...$middleware): self
     {
         $pipeline = clone $this;
         $pipeline->middleware = $middleware;
@@ -92,9 +92,9 @@ 
 
         $previousRequest = $currentRequest->get();
         $currentRequest->set($request);
-        try {
+        try{
             // There is no middleware to process, let's pass the request to the handler
-            if (!\array_key_exists($this->position, $this->middleware)) {
+            if (!\array_key_exists($this->position, $this->middleware)){
                 return $this->handler->handle($request);
             }
 
@@ -108,7 +108,7 @@ 
                 ? \sprintf('%s=%s', $this->middleware[$this->position], $middleware::class)
                 : $middleware::class;
             // Init a tracing span when the pipeline starts
-            if ($span === null) {
+            if ($span === null){
                 /** @var TracerInterface $tracer */
                 $tracer = $this->container->get(TracerInterface::class);
                 return $tracer->trace(
@@ -126,7 +126,7 @@ 
             $span->setAttribute('http.middleware', $middlewares);
 
             return $middleware->process($request, $this->next($span));
-        } finally {
+        }finally{
             $currentRequest->set($previousRequest);
         }
     }

Braces +9 added lines, -4 removed lines

@@ -92,9 +92,11 @@ 
 
         $previousRequest = $currentRequest->get();
         $currentRequest->set($request);
-        try {
+        try
+        {
             // There is no middleware to process, let's pass the request to the handler
-            if (!\array_key_exists($this->position, $this->middleware)) {
+            if (!\array_key_exists($this->position, $this->middleware))
+            {
                 return $this->handler->handle($request);
             }
 
@@ -108,7 +110,8 @@ 
                 ? \sprintf('%s=%s', $this->middleware[$this->position], $middleware::class)
                 : $middleware::class;
             // Init a tracing span when the pipeline starts
-            if ($span === null) {
+            if ($span === null)
+            {
                 /** @var TracerInterface $tracer */
                 $tracer = $this->container->get(TracerInterface::class);
                 return $tracer->trace(
@@ -126,7 +129,9 @@ 
             $span->setAttribute('http.middleware', $middlewares);
 
             return $middleware->process($request, $this->next($span));
-        } finally {
+        }
+        finally
+        {
             $currentRequest->set($previousRequest);
         }
     }

src/Http/src/Http.php

Spacing +11 added lines, -11 removed lines

@@ -25,21 +25,21 @@ 
 {
     private ?RequestHandlerInterface $handler = null;
     private readonly TracerFactoryInterface $tracerFactory;
-    private readonly Pipeline|LazyPipeline $pipeline;
+    private readonly Pipeline | LazyPipeline $pipeline;
 
     public function __construct(
         private readonly HttpConfig $config,
-        Pipeline|LazyPipeline $pipeline,
+        Pipeline | LazyPipeline $pipeline,
         private readonly ResponseFactoryInterface $responseFactory,
         private readonly ContainerInterface $container,
         ?TracerFactoryInterface $tracerFactory = null,
         private readonly ?EventDispatcherInterface $dispatcher = null,
-    ) {
-        if ($pipeline instanceof Pipeline) {
-            foreach ($this->config->getMiddleware() as $middleware) {
+    ){
+        if ($pipeline instanceof Pipeline){
+            foreach ($this->config->getMiddleware() as $middleware){
                 $pipeline->pushMiddleware($this->container->get($middleware));
             }
-        } else {
+        }else{
             $pipeline = $pipeline->withAddedMiddleware(
                 ...$this->config->getMiddleware()
             );
@@ -53,12 +53,12 @@ 
     /**
      * @internal
      */
-    public function getPipeline(): Pipeline|LazyPipeline
+    public function getPipeline(): Pipeline | LazyPipeline
     {
         return $this->pipeline;
     }
 
-    public function setHandler(callable|RequestHandlerInterface $handler): self
+    public function setHandler(callable | RequestHandlerInterface $handler): self
     {
         $this->handler = $handler instanceof RequestHandlerInterface
             ? $handler
@@ -77,7 +77,7 @@ 
 
             $this->dispatcher?->dispatch(new RequestReceived($request));
 
-            if ($this->handler === null) {
+            if ($this->handler === null){
                 throw new HttpException('Unable to run HttpCore, no handler is set.');
             }
 
@@ -107,7 +107,7 @@ 
             callback: $callback,
             attributes: [
                 'http.method' => $request->getMethod(),
-                'http.url' => (string) $request->getUri(),
+                'http.url' => (string)$request->getUri(),
                 'http.headers' => \array_map(
                     static fn (array $values): string => \implode(',', $values),
                     $request->getHeaders(),
@@ -117,7 +117,7 @@ 
             traceKind: TraceKind::SERVER,
         );
 
-        foreach ($tracer->getContext() as $key => $value) {
+        foreach ($tracer->getContext() as $key => $value){
             $response = $response->withHeader($key, $value);
         }
 

Braces +11 added lines, -5 removed lines

@@ -35,11 +35,15 @@ 
         ?TracerFactoryInterface $tracerFactory = null,
         private readonly ?EventDispatcherInterface $dispatcher = null,
     ) {
-        if ($pipeline instanceof Pipeline) {
-            foreach ($this->config->getMiddleware() as $middleware) {
+        if ($pipeline instanceof Pipeline)
+        {
+            foreach ($this->config->getMiddleware() as $middleware)
+            {
                 $pipeline->pushMiddleware($this->container->get($middleware));
             }
-        } else {
+        }
+        else
+        {
             $pipeline = $pipeline->withAddedMiddleware(
                 ...$this->config->getMiddleware()
             );
@@ -77,7 +81,8 @@ 
 
             $this->dispatcher?->dispatch(new RequestReceived($request));
 
-            if ($this->handler === null) {
+            if ($this->handler === null)
+            {
                 throw new HttpException('Unable to run HttpCore, no handler is set.');
             }
 
@@ -117,7 +122,8 @@ 
             traceKind: TraceKind::SERVER,
         );
 
-        foreach ($tracer->getContext() as $key => $value) {
+        foreach ($tracer->getContext() as $key => $value)
+        {
             $response = $response->withHeader($key, $value);
         }
 

src/Router/src/Traits/PipelineTrait.php

Spacing +8 added lines, -8 removed lines

@@ -20,7 +20,7 @@ 
 {
     use ContainerTrait;
 
-    protected Pipeline|LazyPipeline|null $pipeline = null;
+    protected Pipeline | LazyPipeline | null $pipeline = null;
 
     /** @psalm-var array<array-key, MiddlewareType> */
     protected array $middleware = [];
@@ -44,16 +44,16 @@ 
         $route = clone $this;
 
         // array fallback
-        if (\count($middleware) === 1 && \is_array($middleware[0])) {
+        if (\count($middleware) === 1 && \is_array($middleware[0])){
             $middleware = $middleware[0];
         }
 
         /** @var MiddlewareType[] $middleware */
-        foreach ($middleware as $item) {
+        foreach ($middleware as $item){
             $route->middleware[] = $item;
         }
 
-        if ($route->pipeline !== null) {
+        if ($route->pipeline !== null){
             $route->pipeline = $route->makeLazyPipeline();
         }
 
@@ -80,11 +80,11 @@ 
     protected function makePipeline(): Pipeline
     {
         \assert($this->container !== null);
-        try {
+        try{
             return $this->container
                 ->get(PipelineFactory::class)
                 ->createWithMiddleware($this->middleware);
-        } catch (ContainerExceptionInterface $e) {
+        }catch (ContainerExceptionInterface $e){
             throw new RouteException($e->getMessage(), $e->getCode(), $e);
         }
     }
@@ -97,11 +97,11 @@ 
     protected function makeLazyPipeline(): LazyPipeline
     {
         \assert($this->container !== null);
-        try {
+        try{
             /** @var LazyPipeline $pipeline */
             $pipeline = $this->container->get(LazyPipeline::class);
             return $pipeline->withMiddleware(...$this->middleware);
-        } catch (ContainerExceptionInterface $e) {
+        }catch (ContainerExceptionInterface $e){
             throw new RouteException($e->getMessage(), $e->getCode(), $e);
         }
     }

Braces +16 added lines, -7 removed lines

@@ -44,16 +44,19 @@ 
         $route = clone $this;
 
         // array fallback
-        if (\count($middleware) === 1 && \is_array($middleware[0])) {
+        if (\count($middleware) === 1 && \is_array($middleware[0]))
+        {
             $middleware = $middleware[0];
         }
 
         /** @var MiddlewareType[] $middleware */
-        foreach ($middleware as $item) {
+        foreach ($middleware as $item)
+        {
             $route->middleware[] = $item;
         }
 
-        if ($route->pipeline !== null) {
+        if ($route->pipeline !== null)
+        {
             $route->pipeline = $route->makeLazyPipeline();
         }
 
@@ -80,11 +83,14 @@ 
     protected function makePipeline(): Pipeline
     {
         \assert($this->container !== null);
-        try {
+        try
+        {
             return $this->container
                 ->get(PipelineFactory::class)
                 ->createWithMiddleware($this->middleware);
-        } catch (ContainerExceptionInterface $e) {
+        }
+        catch (ContainerExceptionInterface $e)
+        {
             throw new RouteException($e->getMessage(), $e->getCode(), $e);
         }
     }
@@ -97,11 +103,14 @@ 
     protected function makeLazyPipeline(): LazyPipeline
     {
         \assert($this->container !== null);
-        try {
+        try
+        {
             /** @var LazyPipeline $pipeline */
             $pipeline = $this->container->get(LazyPipeline::class);
             return $pipeline->withMiddleware(...$this->middleware);
-        } catch (ContainerExceptionInterface $e) {
+        }
+        catch (ContainerExceptionInterface $e)
+        {
             throw new RouteException($e->getMessage(), $e->getCode(), $e);
         }
     }