[command] Bundle dependency analysers with dev-tools (#10)

Author: coisa

README.md

@@ -83,12 +83,9 @@ composer dev-tools gitignore
 composer dev-tools:sync
 ```
 
-The `dependencies` command expects both dependency analyzers to be installed in
-the target project:
-
-```bash
-composer require --dev shipmonk/composer-dependency-analyser icanhazstring/composer-unused
-```
+The `dependencies` command ships with both dependency analyzers as direct
+dependencies of `fast-forward/dev-tools`, so it works without extra
+installation in the consumer project.
 
 The `skills` command keeps `.agents/skills` aligned with the packaged Fast
 Forward skill set. It creates missing links, repairs broken links, and

composer.json

@@ -34,6 +34,7 @@
         "fakerphp/faker": "^1.24",
         "fast-forward/phpdoc-bootstrap-template": "^1.0",
         "friendsofphp/php-cs-fixer": "^3.94",
+        "icanhazstring/composer-unused": "^0.9.6",
         "jolicode/jolinotif": "^3.3",
         "phpdocumentor/shim": "^3.9",
         "phpro/grumphp": "^2.19",
@@ -42,6 +43,7 @@
         "pyrech/composer-changelogs": "^2.2",
         "rector/rector": "^2.3",
         "saggre/phpdocumentor-markdown": "^1.0",
+        "shipmonk/composer-dependency-analyser": "^1.8.4",
         "symfony/var-dumper": "^7.4",
         "symfony/var-exporter": "^7.4",
         "symplify/easy-coding-standard": "^13.0",

docs/running/specialized-commands.rst

@@ -33,8 +33,9 @@ Analyzes missing and unused Composer dependencies.
 
 Important details:
 
-- it requires ``shipmonk/composer-dependency-analyser`` and
-  ``icanhazstring/composer-unused`` to be installed in the target project;
+- it ships ``shipmonk/composer-dependency-analyser`` and
+  ``icanhazstring/composer-unused`` as direct dependencies of
+  ``fast-forward/dev-tools``;
 - it uses ``composer-dependency-analyser`` only for missing dependency checks
   and leaves unused-package reporting to ``composer-unused``;
 - it returns a non-zero exit code when missing or unused dependencies are

src/Command/DependenciesCommand.php

@@ -183,11 +183,11 @@ private function resolveMissingRequirements(
         }
 
         if (! $this->filesystem->exists($dependencyAnalyser)) {
-            $missing[] = '- vendor/bin/composer-dependency-analyser not found. Install: composer require --dev shipmonk/composer-dependency-analyser';
+            $missing[] = '- vendor/bin/composer-dependency-analyser not found. Reinstall fast-forward/dev-tools dependencies.';
         }
 
         if (! $this->filesystem->exists($composerUnused)) {
-            $missing[] = '- vendor/bin/composer-unused not found. Install: composer require --dev icanhazstring/composer-unused';
+            $missing[] = '- vendor/bin/composer-unused not found. Reinstall fast-forward/dev-tools dependencies.';
         }
 
         return $missing;

tests/Command/DependenciesCommandTest.php

@@ -105,11 +105,11 @@ public function executeWillFailWhenRequiredFilesAreMissing(): void
         $this->output->writeln('<error>Dependency analysis requires the following files:</error>')
             ->shouldBeCalled();
         $this->output->writeln(
-            '- vendor/bin/composer-dependency-analyser not found. Install: composer require --dev shipmonk/composer-dependency-analyser'
+            '- vendor/bin/composer-dependency-analyser not found. Reinstall fast-forward/dev-tools dependencies.'
         )
             ->shouldBeCalled();
         $this->output->writeln(
-            '- vendor/bin/composer-unused not found. Install: composer require --dev icanhazstring/composer-unused'
+            '- vendor/bin/composer-unused not found. Reinstall fast-forward/dev-tools dependencies.'
         )
             ->shouldBeCalled();