spiral / framework

src/Framework/Bootloader/Http/SessionBootloader.php

Spacing +5 added lines, -5 removed lines

@@ -26,7 +26,7 @@ 
 {
     public function __construct(
         private readonly BinderInterface $binder,
-    ) {
+    ){
     }
 
     public function defineBindings(): array
@@ -37,7 +37,7 @@ 
                 SessionInterface::class,
                 new Proxy(SessionInterface::class, false, $this->resolveSession(...)),
             );
-        $this->binder->bind(SessionInterface::class, new Proxy(SessionInterface::class, true), );
+        $this->binder->bind(SessionInterface::class, new Proxy(SessionInterface::class, true),);
 
         return [];
     }
@@ -71,7 +71,7 @@ 
                 'handler' => new Autowire(
                     FileHandler::class,
                     [
-                        'directory' => $directories->get('runtime') . 'session',
+                        'directory' => $directories->get('runtime').'session',
                         'lifetime' => 86400,
                     ]
                 ),
@@ -90,14 +90,14 @@ 
 
     private function resolveSession(ContainerInterface $container): SessionInterface
     {
-        try {
+        try{
             /** @var ServerRequestInterface $request */
             $request = $container->get(ServerRequestInterface::class);
             return $request->getAttribute(SessionMiddleware::ATTRIBUTE) ?? throw new ContextualObjectNotFoundException(
                 SessionInterface::class,
                 SessionMiddleware::ATTRIBUTE,
             );
-        } catch (InvalidRequestScopeException $e) {
+        }catch (InvalidRequestScopeException $e){
             throw new InvalidRequestScopeException(SessionInterface::class, previous: $e);
         }
     }

src/Framework/Bootloader/Http/Exception/InvalidRequestScopeException.php

Spacing +3 added lines, -3 removed lines

@@ -19,9 +19,9 @@
 {
     public function __construct(
         string $id,
-        string|Container|null $scopeOrContainer = null,
-        \Throwable|null $previous = null,
-    ) {
+        string | Container | null $scopeOrContainer = null,
+        \Throwable | null $previous = null,
+    ){
         parent::__construct($id, $scopeOrContainer, Spiral::HttpRequest->value, $previous);
     }
 }

src/Framework/Bootloader/Http/CookiesBootloader.php

Spacing +3 added lines, -3 removed lines

@@ -22,7 +22,7 @@ 
     public function __construct(
         private readonly ConfiguratorInterface $config,
         private readonly BinderInterface $binder,
-    ) {
+    ){
     }
 
     public function defineBindings(): array
@@ -54,12 +54,12 @@ 
 
     private function cookieQueue(ServerRequestInterface $request): CookieQueue
     {
-        try {
+        try{
             return $request->getAttribute(CookieQueue::ATTRIBUTE) ?? throw new ContextualObjectNotFoundException(
                 CookieQueue::class,
                 CookieQueue::ATTRIBUTE,
             );
-        } catch (InvalidRequestScopeException $e) {
+        }catch (InvalidRequestScopeException $e){
             throw new InvalidRequestScopeException(CookieQueue::class, previous: $e);
         }
     }

src/Core/src/Exception/Shared/InvalidContainerScopeException.php

Spacing +3 added lines, -3 removed lines

@@ -16,10 +16,10 @@
     protected string $scope;
     public function __construct(
         protected readonly string $id,
-        Container|string|null $scopeOrContainer = null,
+        Container | string | null $scopeOrContainer = null,
         protected readonly ?string $requiredScope = null,
-        \Throwable|null $previous = null,
-    ) {
+        \Throwable | null $previous = null,
+    ){
         $this->scope = \is_string($scopeOrContainer)
             ? $scopeOrContainer
             : Introspector::scopeName($scopeOrContainer);

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

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 {
         $instance = $binding->autowire->resolve($this, $arguments);
@@ -246,17 +246,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(),
@@ -270,24 +270,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(
@@ -297,7 +297,7 @@ 
                         )
                     )
                 );
-            } finally {
+            }finally{
                 $this->tracer->pop();
             }
         }
@@ -308,23 +308,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,
@@ -334,19 +334,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);
         }
     }
@@ -367,7 +367,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(),
@@ -393,13 +393,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);
                 }
             }
@@ -428,26 +428,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',
@@ -460,12 +460,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(
@@ -475,22 +475,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();
         }
@@ -508,13 +508,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;
         }
 
@@ -526,12 +526,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;
         }
 
@@ -545,7 +545,7 @@ 
          * @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 +559,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);
@@ -578,9 +578,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),
         ));
     }

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;
     }
 }