spiral / framework

src/Router/src/CoreHandler.php

Braces +28 added lines, -12 removed lines

@@ -66,7 +66,8 @@ 
      */
     public function handle(Request $request): Response
     {
-        if ($this->controller === null) {
+        if ($this->controller === null)
+        {
             throw new HandlerException('Controller and action pair is not set');
         }
 
@@ -76,7 +77,8 @@ 
         $output = $result = null;
 
         $response = $this->responseFactory->createResponse(200);
-        try {
+        try
+        {
             $action = $this->getAction($request);
 
             // run the core withing PSR-7 Request/Response scope
@@ -99,14 +101,21 @@ 
                     ]
                 )
             );
-        } catch (ControllerException $e) {
+        }
+        catch (ControllerException $e)
+        {
             \ob_get_clean();
             throw $this->mapException($e);
-        } catch (\Throwable $e) {
+        }
+        catch (\Throwable $e)
+        {
             \ob_get_clean();
             throw $e;
-        } finally {
-            while (\ob_get_level() > $outputLevel + 1) {
+        }
+        finally
+        {
+            while (\ob_get_level() > $outputLevel + 1)
+            {
                 $output = \ob_get_clean() . $output;
             }
         }
@@ -120,7 +129,8 @@ 
 
     private function getAction(Request $request): string
     {
-        if ($this->verbActions) {
+        if ($this->verbActions)
+        {
             return \strtolower($request->getMethod()) . \ucfirst($this->action);
         }
 
@@ -136,21 +146,27 @@ 
      */
     private function wrapResponse(Response $response, mixed $result = null, string $output = ''): Response
     {
-        if ($result instanceof Response) {
-            if ($output !== '' && $result->getBody()->isWritable()) {
+        if ($result instanceof Response)
+        {
+            if ($output !== '' && $result->getBody()->isWritable())
+            {
                 $result->getBody()->write($output);
             }
 
             return $result;
         }
 
-        if ($result instanceof \Generator) {
+        if ($result instanceof \Generator)
+        {
             return $response->withBody(new GeneratorStream($result));
         }
 
-        if (\is_array($result) || $result instanceof \JsonSerializable) {
+        if (\is_array($result) || $result instanceof \JsonSerializable)
+        {
             $response = $this->writeJson($response, $result);
-        } else {
+        }
+        else
+        {
             $response->getBody()->write((string)$result);
         }
 

Spacing +16 added lines, -16 removed lines

@@ -38,7 +38,7 @@ 
         private readonly ScopeInterface $scope,
         private readonly ResponseFactoryInterface $responseFactory,
         ?TracerInterface $tracer = null
-    ) {
+    ){
         $this->tracer = $tracer ?? new NullTracer($scope);
     }
 
@@ -69,7 +69,7 @@ 
      */
     public function handle(Request $request): Response
     {
-        if ($this->controller === null) {
+        if ($this->controller === null){
             throw new HandlerException('Controller and action pair is not set');
         }
 
@@ -79,7 +79,7 @@ 
         $output = $result = null;
 
         $response = $this->responseFactory->createResponse(200);
-        try {
+        try{
             $action = $this->getAction($request);
 
             // run the core withing PSR-7 Request/Response scope
@@ -102,29 +102,29 @@ 
                     ]
                 )
             );
-        } catch (ControllerException $e) {
+        }catch (ControllerException $e){
             \ob_get_clean();
             throw $this->mapException($e);
-        } catch (\Throwable $e) {
+        }catch (\Throwable $e){
             \ob_get_clean();
             throw $e;
-        } finally {
-            while (\ob_get_level() > $outputLevel + 1) {
-                $output = \ob_get_clean() . $output;
+        }finally{
+            while (\ob_get_level() > $outputLevel + 1){
+                $output = \ob_get_clean().$output;
             }
         }
 
         return $this->wrapResponse(
             $response,
             $result,
-            \ob_get_clean() . $output
+            \ob_get_clean().$output
         );
     }
 
     private function getAction(Request $request): string
     {
-        if ($this->verbActions) {
-            return \strtolower($request->getMethod()) . \ucfirst($this->action);
+        if ($this->verbActions){
+            return \strtolower($request->getMethod()).\ucfirst($this->action);
         }
 
         return $this->action;
@@ -139,21 +139,21 @@ 
      */
     private function wrapResponse(Response $response, mixed $result = null, string $output = ''): Response
     {
-        if ($result instanceof Response) {
-            if ($output !== '' && $result->getBody()->isWritable()) {
+        if ($result instanceof Response){
+            if ($output !== '' && $result->getBody()->isWritable()){
                 $result->getBody()->write($output);
             }
 
             return $result;
         }
 
-        if ($result instanceof \Generator) {
+        if ($result instanceof \Generator){
             return $response->withBody(new GeneratorStream($result));
         }
 
-        if (\is_array($result) || $result instanceof \JsonSerializable) {
+        if (\is_array($result) || $result instanceof \JsonSerializable){
             $response = $this->writeJson($response, $result);
-        } else {
+        }else{
             $response->getBody()->write((string)$result);
         }
 

src/Router/src/Router.php

Braces +46 added lines, -21 removed lines

@@ -63,14 +63,19 @@ 
 
         return $this->tracer->trace(
             name: 'Routing',
-            callback: function (SpanInterface $span) use ($request) {
-                try {
+            callback: function (SpanInterface $span) use ($request)
+            {
+                try
+                {
                     $route = $this->matchRoute($request, $routeName);
-                } catch (RouteException $e) {
+                }
+                catch (RouteException $e)
+                {
                     throw new RouterException('Invalid route definition', $e->getCode(), $e);
                 }
 
-                if ($route === null) {
+                if ($route === null)
+                {
                     $this->eventDispatcher?->dispatch(new RouteNotFound($request));
                     throw new RouteNotFoundException($request->getUri());
                 }
@@ -105,7 +110,8 @@ 
 
     public function getRoute(string $name): RouteInterface
     {
-        if (isset($this->routes[$name])) {
+        if (isset($this->routes[$name]))
+        {
             return $this->routes[$name];
         }
 
@@ -114,7 +120,8 @@ 
 
     public function getRoutes(): array
     {
-        if (!empty($this->default)) {
+        if (!empty($this->default))
+        {
             return $this->routes + [null => $this->default];
         }
 
@@ -123,9 +130,12 @@ 
 
     public function uri(string $route, iterable $parameters = []): UriInterface
     {
-        try {
+        try
+        {
             return $this->getRoute($route)->uri($parameters);
-        } catch (UndefinedRouteException) {
+        }
+        catch (UndefinedRouteException)
+        {
             //In some cases route name can be provided as controller:action pair, we can try to
             //generate such route automatically based on our default/fallback route
             return $this->castRoute($route)->uri($parameters);
@@ -137,9 +147,11 @@ 
         /** @var GroupRegistry $groups */
         $groups = $this->container->get(GroupRegistry::class);
 
-        foreach ($routes->getCollection() as $name => $configurator) {
+        foreach ($routes->getCollection() as $name => $configurator)
+        {
             $target = $configurator->target;
-            if ($configurator->core !== null && $target instanceof AbstractTarget) {
+            if ($configurator->core !== null && $target instanceof AbstractTarget)
+            {
                 $target = $target->withCore($configurator->core);
             }
 
@@ -149,19 +161,23 @@ 
                 $configurator->defaults
             );
 
-            if ($configurator->middleware !== null) {
+            if ($configurator->middleware !== null)
+            {
                 $route = $route->withMiddleware(...$configurator->middleware);
             }
 
-            if ($configurator->methods !== null) {
+            if ($configurator->methods !== null)
+            {
                 $route = $route->withVerbs(...$configurator->methods);
             }
 
-            if (!isset($this->routes[$name]) && $name !== RoutingConfigurator::DEFAULT_ROUTE_NAME) {
+            if (!isset($this->routes[$name]) && $name !== RoutingConfigurator::DEFAULT_ROUTE_NAME)
+            {
                 $groups->getGroup($configurator->group ?? $groups->getDefaultGroup())->addRoute($name, $route);
             }
 
-            if ($name === RoutingConfigurator::DEFAULT_ROUTE_NAME) {
+            if ($name === RoutingConfigurator::DEFAULT_ROUTE_NAME)
+            {
                 $this->setDefault($route);
             }
         }
@@ -172,17 +188,20 @@ 
      */
     protected function matchRoute(ServerRequestInterface $request, string &$routeName = null): ?RouteInterface
     {
-        foreach ($this->routes as $name => $route) {
+        foreach ($this->routes as $name => $route)
+        {
             // Matched route will return new route instance with matched parameters
             $matched = $route->match($request);
 
-            if ($matched !== null) {
+            if ($matched !== null)
+            {
                 $routeName = $name;
                 return $matched;
             }
         }
 
-        if ($this->default !== null) {
+        if ($this->default !== null)
+        {
             return $this->default->match($request);
         }
 
@@ -195,7 +214,8 @@ 
      */
     protected function configure(RouteInterface $route): RouteInterface
     {
-        if ($route instanceof ContainerizedInterface && !$route->hasContainer()) {
+        if ($route instanceof ContainerizedInterface && !$route->hasContainer())
+        {
             // isolating route in a given container
             $route = $route->withContainer($this->container);
         }
@@ -225,11 +245,16 @@ 
             );
         }
 
-        if (!empty($matches['name'])) {
+        if (!empty($matches['name']))
+        {
             $routeObject = $this->getRoute($matches['name']);
-        } elseif ($this->default !== null) {
+        }
+        elseif ($this->default !== null)
+        {
             $routeObject = $this->default;
-        } else {
+        }
+        else
+        {
             throw new UndefinedRouteException(\sprintf('Unable to locate route candidate for `%s`', $route));
         }
 

Spacing +24 added lines, -24 removed lines

@@ -49,8 +49,8 @@ 
         private readonly ContainerInterface $container,
         private readonly ?EventDispatcherInterface $eventDispatcher = null,
         private readonly ?TracerInterface $tracer = new NullTracer(),
-    ) {
-        $this->basePath = '/' . \ltrim($basePath, '/');
+    ){
+        $this->basePath = '/'.\ltrim($basePath, '/');
     }
 
     /**
@@ -64,13 +64,13 @@ 
         return $this->tracer->trace(
             name: 'Routing',
             callback: function (SpanInterface $span) use ($request) {
-                try {
+                try{
                     $route = $this->matchRoute($request, $routeName);
-                } catch (RouteException $e) {
+                }catch (RouteException $e){
                     throw new RouterException('Invalid route definition', $e->getCode(), $e);
                 }
 
-                if ($route === null) {
+                if ($route === null){
                     $this->eventDispatcher?->dispatch(new RouteNotFound($request));
                     throw new RouteNotFoundException($request->getUri());
                 }
@@ -105,7 +105,7 @@ 
 
     public function getRoute(string $name): RouteInterface
     {
-        if (isset($this->routes[$name])) {
+        if (isset($this->routes[$name])){
             return $this->routes[$name];
         }
 
@@ -114,7 +114,7 @@ 
 
     public function getRoutes(): array
     {
-        if (!empty($this->default)) {
+        if (!empty($this->default)){
             return $this->routes + [null => $this->default];
         }
 
@@ -123,9 +123,9 @@ 
 
     public function uri(string $route, iterable $parameters = []): UriInterface
     {
-        try {
+        try{
             return $this->getRoute($route)->uri($parameters);
-        } catch (UndefinedRouteException) {
+        }catch (UndefinedRouteException){
             //In some cases route name can be provided as controller:action pair, we can try to
             //generate such route automatically based on our default/fallback route
             return $this->castRoute($route)->uri($parameters);
@@ -137,31 +137,31 @@ 
         /** @var GroupRegistry $groups */
         $groups = $this->container->get(GroupRegistry::class);
 
-        foreach ($routes->getCollection() as $name => $configurator) {
+        foreach ($routes->getCollection() as $name => $configurator){
             $target = $configurator->target;
-            if ($configurator->core !== null && $target instanceof AbstractTarget) {
+            if ($configurator->core !== null && $target instanceof AbstractTarget){
                 $target = $target->withCore($configurator->core);
             }
 
             $route = new Route(
-                $configurator->prefix . '/' . \ltrim($configurator->pattern, '/'),
+                $configurator->prefix.'/'.\ltrim($configurator->pattern, '/'),
                 $target,
                 $configurator->defaults
             );
 
-            if ($configurator->middleware !== null) {
+            if ($configurator->middleware !== null){
                 $route = $route->withMiddleware(...$configurator->middleware);
             }
 
-            if ($configurator->methods !== null) {
+            if ($configurator->methods !== null){
                 $route = $route->withVerbs(...$configurator->methods);
             }
 
-            if (!isset($this->routes[$name]) && $name !== RoutingConfigurator::DEFAULT_ROUTE_NAME) {
+            if (!isset($this->routes[$name]) && $name !== RoutingConfigurator::DEFAULT_ROUTE_NAME){
                 $groups->getGroup($configurator->group ?? $groups->getDefaultGroup())->addRoute($name, $route);
             }
 
-            if ($name === RoutingConfigurator::DEFAULT_ROUTE_NAME) {
+            if ($name === RoutingConfigurator::DEFAULT_ROUTE_NAME){
                 $this->setDefault($route);
             }
         }
@@ -172,17 +172,17 @@ 
      */
     protected function matchRoute(ServerRequestInterface $request, string &$routeName = null): ?RouteInterface
     {
-        foreach ($this->routes as $name => $route) {
+        foreach ($this->routes as $name => $route){
             // Matched route will return new route instance with matched parameters
             $matched = $route->match($request);
 
-            if ($matched !== null) {
+            if ($matched !== null){
                 $routeName = $name;
                 return $matched;
             }
         }
 
-        if ($this->default !== null) {
+        if ($this->default !== null){
             return $this->default->match($request);
         }
 
@@ -195,7 +195,7 @@ 
      */
     protected function configure(RouteInterface $route): RouteInterface
     {
-        if ($route instanceof ContainerizedInterface && !$route->hasContainer()) {
+        if ($route instanceof ContainerizedInterface && !$route->hasContainer()){
             // isolating route in a given container
             $route = $route->withContainer($this->container);
         }
@@ -219,17 +219,17 @@ 
                 $route,
                 $matches
             )
-        ) {
+        ){
             throw new UndefinedRouteException(
                 "Unable to locate route or use default route with 'name/controller:action' pattern"
             );
         }
 
-        if (!empty($matches['name'])) {
+        if (!empty($matches['name'])){
             $routeObject = $this->getRoute($matches['name']);
-        } elseif ($this->default !== null) {
+        } elseif ($this->default !== null){
             $routeObject = $this->default;
-        } else {
+        }else{
             throw new UndefinedRouteException(\sprintf('Unable to locate route candidate for `%s`', $route));
         }
 

src/Prototype/src/Command/DumpCommand.php

Spacing +8 added lines, -8 removed lines

@@ -22,7 +22,7 @@ 
     public function perform(PrototypeBootloader $prototypeBootloader): int
     {
         $dependencies = $this->registry->getPropertyBindings();
-        if ($dependencies === []) {
+        if ($dependencies === []){
             $this->writeln('<comment>No prototyped shortcuts found.</comment>');
 
             return self::SUCCESS;
@@ -32,7 +32,7 @@ 
 
         $trait = new \ReflectionClass(PrototypeTrait::class);
         $docComment = $trait->getDocComment();
-        if ($docComment === false) {
+        if ($docComment === false){
             $this->write('<fg=red>DOCComment is missing</fg=red>');
 
             return self::FAILURE;
@@ -40,7 +40,7 @@ 
 
         $filename = $trait->getFileName();
 
-        try {
+        try{
             \file_put_contents(
                 $filename,
                 \str_replace(
@@ -49,18 +49,18 @@ 
                     \file_get_contents($filename)
                 )
             );
-        } catch (\Throwable $e) {
-            $this->write('<fg=red>' . $e->getMessage() . "</fg=red>\n");
+        }catch (\Throwable $e){
+            $this->write('<fg=red>'.$e->getMessage()."</fg=red>\n");
 
             return self::FAILURE;
         }
 
         $this->write("<fg=green>complete</fg=green>\n");
 
-        if ($this->isVerbose()) {
+        if ($this->isVerbose()){
             $grid = $this->table(['Property:', 'Target:']);
 
-            foreach ($dependencies as $dependency) {
+            foreach ($dependencies as $dependency){
                 $grid->addRow([$dependency->var, $dependency->type->fullName]);
             }
 
@@ -78,7 +78,7 @@ 
         );
         $an->lines[] = new Annotation\Line('');
 
-        foreach ($dependencies as $dependency) {
+        foreach ($dependencies as $dependency){
             $an->lines[] = new Annotation\Line(
                 \sprintf('\\%s $%s', $dependency->type->fullName, $dependency->var),
                 'property'

Braces +15 added lines, -7 removed lines

@@ -22,7 +22,8 @@ 
     public function perform(PrototypeBootloader $prototypeBootloader): int
     {
         $dependencies = $this->registry->getPropertyBindings();
-        if ($dependencies === []) {
+        if ($dependencies === [])
+        {
             $this->writeln('<comment>No prototyped shortcuts found.</comment>');
 
             return self::SUCCESS;
@@ -32,7 +33,8 @@ 
 
         $trait = new \ReflectionClass(PrototypeTrait::class);
         $docComment = $trait->getDocComment();
-        if ($docComment === false) {
+        if ($docComment === false)
+        {
             $this->write('<fg=red>DOCComment is missing</fg=red>');
 
             return self::FAILURE;
@@ -40,7 +42,8 @@ 
 
         $filename = $trait->getFileName();
 
-        try {
+        try
+        {
             \file_put_contents(
                 $filename,
                 \str_replace(
@@ -49,7 +52,9 @@ 
                     \file_get_contents($filename)
                 )
             );
-        } catch (\Throwable $e) {
+        }
+        catch (\Throwable $e)
+        {
             $this->write('<fg=red>' . $e->getMessage() . "</fg=red>\n");
 
             return self::FAILURE;
@@ -57,10 +62,12 @@ 
 
         $this->write("<fg=green>complete</fg=green>\n");
 
-        if ($this->isVerbose()) {
+        if ($this->isVerbose())
+        {
             $grid = $this->table(['Property:', 'Target:']);
 
-            foreach ($dependencies as $dependency) {
+            foreach ($dependencies as $dependency)
+            {
                 $grid->addRow([$dependency->var, $dependency->type->fullName]);
             }
 
@@ -78,7 +85,8 @@ 
         );
         $an->lines[] = new Annotation\Line('');
 
-        foreach ($dependencies as $dependency) {
+        foreach ($dependencies as $dependency)
+        {
             $an->lines[] = new Annotation\Line(
                 \sprintf('\\%s $%s', $dependency->type->fullName, $dependency->var),
                 'property'

src/Telemetry/src/Span.php

Spacing +2 added lines, -2 removed lines

@@ -13,7 +13,7 @@ 
     public function __construct(
         private string $name,
         private array $attributes = []
-    ) {
+    ){
     }
 
     public function getName(): string
@@ -57,7 +57,7 @@ 
         return $this->attributes[$name] ?? null;
     }
 
-    public function setStatus(string|int $code, string $description = null): self
+    public function setStatus(string | int $code, string $description = null): self
     {
         $this->status = new Status($code, $description);
 

src/Telemetry/src/Config/TelemetryConfig.php

Spacing +5 added lines, -5 removed lines

@@ -25,7 +25,7 @@ 
      */
     public function getDefaultDriver(): string
     {
-        if (!\is_string($this->config['default'])) {
+        if (!\is_string($this->config['default'])){
             throw new InvalidArgumentException('Default trace driver config value must be a string');
         }
 
@@ -37,9 +37,9 @@ 
      * @return class-string<TracerFactoryInterface>|Autowire|TracerFactoryInterface
      * @throws InvalidArgumentException
      */
-    public function getDriverConfig(string $name): string|Autowire|TracerFactoryInterface
+    public function getDriverConfig(string $name): string | Autowire | TracerFactoryInterface
     {
-        if (!isset($this->config['drivers'][$name])) {
+        if (!isset($this->config['drivers'][$name])){
             throw new InvalidArgumentException(
                 \sprintf('Config for telemetry driver `%s` is not defined.', $name)
             );
@@ -47,11 +47,11 @@ 
 
         $driver = $this->config['drivers'][$name];
 
-        if ($driver instanceof TracerFactoryInterface) {
+        if ($driver instanceof TracerFactoryInterface){
             return $driver;
         }
 
-        if (!\is_string($driver) && !$driver instanceof Autowire) {
+        if (!\is_string($driver) && !$driver instanceof Autowire){
             throw new InvalidArgumentException(
                 \sprintf('Trace type value for `%s` must be a string or %s', $name, Autowire::class)
             );

Braces +8 added lines, -4 removed lines

@@ -25,7 +25,8 @@ 
      */
     public function getDefaultDriver(): string
     {
-        if (!\is_string($this->config['default'])) {
+        if (!\is_string($this->config['default']))
+        {
             throw new InvalidArgumentException('Default trace driver config value must be a string');
         }
 
@@ -39,7 +40,8 @@ 
      */
     public function getDriverConfig(string $name): string|Autowire|TracerFactoryInterface
     {
-        if (!isset($this->config['drivers'][$name])) {
+        if (!isset($this->config['drivers'][$name]))
+        {
             throw new InvalidArgumentException(
                 \sprintf('Config for telemetry driver `%s` is not defined.', $name)
             );
@@ -47,11 +49,13 @@ 
 
         $driver = $this->config['drivers'][$name];
 
-        if ($driver instanceof TracerFactoryInterface) {
+        if ($driver instanceof TracerFactoryInterface)
+        {
             return $driver;
         }
 
-        if (!\is_string($driver) && !$driver instanceof Autowire) {
+        if (!\is_string($driver) && !$driver instanceof Autowire)
+        {
             throw new InvalidArgumentException(
                 \sprintf('Trace type value for `%s` must be a string or %s', $name, Autowire::class)
             );

src/Telemetry/src/LogTracerFactory.php

Spacing +1 added lines, -1 removed lines

@@ -19,7 +19,7 @@
         private readonly ClockInterface $clock,
         LogsInterface $logs,
         string $channel = self::LOG_CHANNEL
-    ) {
+    ){
         $this->logger = $logs->getLogger($channel);
     }
 

src/Telemetry/src/Span/Status.php

Spacing +2 added lines, -2 removed lines

@@ -11,8 +11,8 @@
      * @param non-empty-string|null $description
      */
     public function __construct(
-        public readonly string|int $code,
+        public readonly string | int $code,
         public readonly ?string $description = null
-    ) {
+    ){
     }
 }

src/Telemetry/src/NullTracerFactory.php

Spacing +1 added lines, -1 removed lines

@@ -11,7 +11,7 @@
 {
     public function __construct(
         private readonly ?ScopeInterface $scope = new Container(),
-    ) {
+    ){
     }
 
     public function make(array $context = []): TracerInterface

src/Telemetry/src/Bootloader/TelemetryBootloader.php

Spacing +2 added lines, -2 removed lines

@@ -36,7 +36,7 @@ 
 
     public function __construct(
         private readonly ConfiguratorInterface $config
-    ) {
+    ){
     }
 
     public function init(EnvironmentInterface $env): void
@@ -47,7 +47,7 @@ 
     /**
      * @param class-string<TracerFactoryInterface>|TracerFactoryInterface|Autowire $driver
      */
-    public function registerTracer(string $name, string|TracerFactoryInterface|Autowire $driver): void
+    public function registerTracer(string $name, string | TracerFactoryInterface | Autowire $driver): void
     {
         $this->config->modify(
             TelemetryConfig::CONFIG,