Remove composer dependency, insights, and tests (#575)

* remove composer dependency, insights, and tests

* rm more class references

Author: cmgmyr

composer.json

@@ -22,7 +22,7 @@
         "ext-json": "*",
         "ext-mbstring": "*",
         "ext-tokenizer": "*",
-        "composer/composer": "^2.0",
+        "composer/semver": "^3.3",
         "friendsofphp/php-cs-fixer": "^3.0.0",
         "justinrainbow/json-schema": "^5.1",
         "league/container": "^3.2|^4.2",

docs/insights/architecture.md

@@ -3,7 +3,6 @@
 The following insights are organised in different metrics :
 
 * `NunoMaduro\PhpInsights\Domain\Metrics\Architecture\Classes` <Badge text="Architecture\Classes" type="warn" vertical="middle"/>
-* `NunoMaduro\PhpInsights\Domain\Metrics\Architecture\Composer` <Badge text="Architecture\Composer" type="warn" vertical="middle"/>
 * `NunoMaduro\PhpInsights\Domain\Metrics\Architecture\Constants` <Badge text="Architecture\Constants" type="warn" vertical="middle"/>
 * `NunoMaduro\PhpInsights\Domain\Metrics\Architecture\Files` <Badge text="Architecture\Files" type="warn" vertical="middle"/>
 * `NunoMaduro\PhpInsights\Domain\Metrics\Architecture\Functions` <Badge text="Architecture\Functions" type="warn" vertical="middle"/>
@@ -95,42 +94,6 @@ This sniff reports use of superfluous prefix or suffix "Abstract" for abstract c
 
 **Insight Class**: `SlevomatCodingStandard\Sniffs\Classes\SuperfluousAbstractClassNamingSniff`
 
-## `composer.json` must exist <Badge text="^1.0"/> <Badge text="Architecture\Composer" type="warn"/>
-
-This insight verifies there is `composer.json`.
-
-**Insight Class**: `NunoMaduro\PhpInsights\Domain\Insights\Composer\ComposerMustExist`
-
-## The name property in the `composer.json` <Badge text="^1.0"/> <Badge text="Architecture\Composer" type="warn"/>
-
-This insight checks if the name section in `composer.json` contains default values (e.g. `laravel/laravel` or `symfony/symfony`).
-
-**Insight Class**: `NunoMaduro\PhpInsights\Domain\Insights\Composer\ComposerMustContainName`
-
-## Composer.json must be valid <Badge text="^1.7"/> <Badge text="Architecture\Composer" type="warn"/>
-
-This insight checks if the `composer.json` is valid.
-
-**Insight Class**: `NunoMaduro\PhpInsights\Domain\Insights\Composer\ComposerMustBeValid`
-
-<details>
-   <summary>Configuration</summary>
-
-```php
-ComposerMustBeValid::class => [
-    //optional configuration value for composer version 2 only
-    //you can allow 'version' in your composer.json and disable the check of it so that you don't get an error
-    'composerVersionCheck' => 0,
-]
-```
-</details>
-
-## Composer.lock must be fresh <Badge text="^1.7"/> <Badge text="Architecture\Composer" type="warn"/>
-
-This insight verifies that the `composer.lock` is not outdated.
-
-**Insight Class**: `NunoMaduro\PhpInsights\Domain\Insights\Composer\ComposerLockMustBeFresh`
-
 ## Define `globals` is prohibited <Badge text="^1.0"/> <Badge text="Architecture\Constants" type="warn"/>
 
 This insight disallows defining `globals`.

src/Application/Composer.php

@@ -4,12 +4,8 @@
 
 namespace NunoMaduro\PhpInsights\Application;
 
-use Composer\Semver\Semver;
-
 /**
  * @internal
- *
- * @see \Tests\Application\ComposerTest
  */
 final class Composer
 {
@@ -51,23 +47,4 @@ public function getName(): string
     {
         return $this->config['name'] ?? '';
     }
-
-    public function hasPhpVersion(): bool
-    {
-        return isset($this->getRequirements()['php']);
-    }
-
-    public function lowestPhpVersionIsGreaterThenOrEqualTo(string $version): bool
-    {
-        $composerVersion = $this->getPhpVersion();
-        preg_match("/\d+(\.\d+)/", $composerVersion, $matches);
-        $composerVersion = $matches[0];
-
-        return Semver::satisfies($composerVersion, $version);
-    }
-
-    private function getPhpVersion(): string
-    {
-        return $this->getRequirements()['php'];
-    }
 }

src/Application/DefaultPreset.php

@@ -52,9 +52,7 @@ public static function get(Composer $composer): array
                     'ignoreUnusedValuesWhenOnlyKeysAreUsedInForeach' => true,
                 ],
                 PropertyTypeHintSniff::class => [
-                    'enableNativeTypeHint' => $composer->hasPhpVersion()
-                        ? $composer->lowestPhpVersionIsGreaterThenOrEqualTo('7.4')
-                        : PHP_VERSION_ID >= 70_400,
+                    'enableNativeTypeHint' => PHP_VERSION_ID >= 70_400,
                 ],
             ],
         ];