From 69f5e9e7304bd3e2931b5517996282dfce40ad7e Mon Sep 17 00:00:00 2001
From: Andrew Nicols <andrew@nicols.co.uk>
Date: Wed, 19 Jun 2024 23:44:13 +0800
Subject: [PATCH] Address all phpcs issues

---
 .gitignore                |  1 +
 .phpcs.xml.dist           | 27 +++++++++++++++++++++++++++
 composer.json             |  8 +++++++-
 src/Bridge.php            |  8 ++++----
 src/CallableResolver.php  |  6 ++----
 src/ControllerInvoker.php |  8 +++-----
 6 files changed, 44 insertions(+), 14 deletions(-)
 create mode 100644 .phpcs.xml.dist

diff --git a/.gitignore b/.gitignore
index bd4e81c..b607c93 100644
--- a/.gitignore
+++ b/.gitignore
@@ -1,3 +1,4 @@
 /vendor/
 /composer.phar
 /composer.lock
+.phpcs-cache
diff --git a/.phpcs.xml.dist b/.phpcs.xml.dist
new file mode 100644
index 0000000..84ee281
--- /dev/null
+++ b/.phpcs.xml.dist
@@ -0,0 +1,27 @@
+<?xml version="1.0"?>
+<ruleset>
+    <arg name="basepath" value="."/>
+    <arg name="extensions" value="php"/>
+    <arg name="cache" value=".phpcs-cache"/>
+    <!-- Show sniff names -->
+    <arg value="s"/>
+
+    <file>src</file>
+
+    <rule ref="HardMode"/>
+
+    <rule ref="SlevomatCodingStandard.TypeHints.ParameterTypeHint.MissingAnyTypeHint">
+        <exclude-pattern type="relative">src/CallableResolver.php</exclude-pattern>
+    </rule>
+    <rule ref="SlevomatCodingStandard.TypeHints.ReturnTypeHint.MissingAnyTypeHint">
+        <exclude-pattern type="relative">src/CallableResolver.php</exclude-pattern>
+    </rule>
+
+    <rule ref="SlevomatCodingStandard.Classes.SuperfluousInterfaceNaming.SuperfluousSuffix">
+        <severity>0</severity>
+    </rule>
+    <rule ref="SlevomatCodingStandard.Classes.SuperfluousExceptionNaming.SuperfluousSuffix">
+        <severity>0</severity>
+    </rule>
+
+</ruleset>
diff --git a/composer.json b/composer.json
index 63794ec..9540396 100644
--- a/composer.json
+++ b/composer.json
@@ -21,6 +21,12 @@
     },
     "require-dev": {
         "laminas/laminas-diactoros": "^2.1",
-        "phpunit/phpunit": ">= 7.0 < 10"
+        "phpunit/phpunit": ">= 7.0 < 10",
+        "mnapoli/hard-mode": "^0.3.0"
+    },
+    "config": {
+        "allow-plugins": {
+            "dealerdirect/phpcodesniffer-composer-installer": true
+        }
     }
 }
diff --git a/src/Bridge.php b/src/Bridge.php
index f97770d..127cdc9 100644
--- a/src/Bridge.php
+++ b/src/Bridge.php
@@ -3,6 +3,7 @@
 namespace DI\Bridge\Slim;
 
 use DI\Container;
+use Invoker\CallableResolver as InvokerCallableResolver;
 use Invoker\Invoker;
 use Invoker\ParameterResolver\AssociativeArrayResolver;
 use Invoker\ParameterResolver\Container\TypeHintContainerResolver;
@@ -11,7 +12,6 @@
 use Psr\Container\ContainerInterface;
 use Slim\App;
 use Slim\Factory\AppFactory;
-use \Invoker\CallableResolver as InvokerCallableResolver;
 use Slim\Interfaces\CallableResolverInterface;
 
 /**
@@ -22,7 +22,7 @@
  */
 class Bridge
 {
-    public static function create(ContainerInterface $container = null): App
+    public static function create(?ContainerInterface $container = null): App
     {
         $container = $container ?: new Container;
 
@@ -43,11 +43,11 @@ private static function createControllerInvoker(ContainerInterface $container):
     {
         $resolvers = [
             // Inject parameters by name first
-            new AssociativeArrayResolver(),
+            new AssociativeArrayResolver,
             // Then inject services by type-hints for those that weren't resolved
             new TypeHintContainerResolver($container),
             // Then fall back on parameters default values for optional route parameters
-            new DefaultValueResolver(),
+            new DefaultValueResolver,
         ];
 
         $invoker = new Invoker(new ResolverChain($resolvers), $container);
diff --git a/src/CallableResolver.php b/src/CallableResolver.php
index 7172ef1..3cf789c 100644
--- a/src/CallableResolver.php
+++ b/src/CallableResolver.php
@@ -1,4 +1,4 @@
-<?php
+<?php declare(strict_types=1);
 
 namespace DI\Bridge\Slim;
 
@@ -12,9 +12,7 @@
  */
 class CallableResolver implements AdvancedCallableResolverInterface
 {
-    /**
-     * @var \Invoker\CallableResolver
-     */
+    /** @var \Invoker\CallableResolver */
     private $callableResolver;
 
     public function __construct(\Invoker\CallableResolver $callableResolver)
diff --git a/src/ControllerInvoker.php b/src/ControllerInvoker.php
index ba3bfe8..bc1a632 100644
--- a/src/ControllerInvoker.php
+++ b/src/ControllerInvoker.php
@@ -1,4 +1,4 @@
-<?php
+<?php declare(strict_types=1);
 
 namespace DI\Bridge\Slim;
 
@@ -9,15 +9,14 @@
 
 class ControllerInvoker implements InvocationStrategyInterface
 {
-    /**
-     * @var InvokerInterface
-     */
+    /** @var InvokerInterface */
     private $invoker;
 
     public function __construct(InvokerInterface $invoker)
     {
         $this->invoker = $invoker;
     }
+
     /**
      * Invoke a route callable.
      *
@@ -25,7 +24,6 @@ public function __construct(InvokerInterface $invoker)
      * @param ServerRequestInterface $request        The request object.
      * @param ResponseInterface      $response       The response object.
      * @param array                  $routeArguments The route's placeholder arguments
-     *
      * @return ResponseInterface|string The response from the callable.
      */
     public function __invoke(