Unified processing of new packages

README.md

@@ -12,7 +12,7 @@ In Composer terminology, a **repository** is a set of packages, and a **package*
 
 A **private repository** is a set of **private packages**. Packages can be required by other packages in relative terms based on their [semantic version](#semantic-versioning). That is, instead of specifying a commit hash, one can request the newest package that matches a version pattern, such as `1.1.*`.
 
-After running `composer install` or `composer update`, the `require` pattern of each package is **locked** to the specific **commit hash** that **satisfies** the requirement.
+After running composer `install`, `update` or `require`, the resulting `require` pattern of each package is **locked** to the specific **commit hash** that **satisfies** the requirement.
 
 ## How Packman works
 

src/Packman.php

@@ -142,15 +142,6 @@ public function start(array $options = [])
         // $this->log(array_keys($this->getRepositories()), 'Active repos');
     }
 
-    public function addPackages(array $options)
-    {
-        // The start() method was supposedly already called, so only
-        // call it again if there are new packages
-        if ($packages = $options['packages'] ?? false) {
-            $this->start($packages);
-        }
-    }
-
     public function listSatisRepos()
     {
         // print_r($this->getDeclaredRepos());

src/PluginLoader.php

@@ -119,44 +119,39 @@ public static function getSubscribedEvents()
         ];
     }
 
-    public static function needsWebServer($event): bool
+    public function preCommandRun(PreCommandRunEvent $event)
     {
-        $commands = ['install', 'update', 'require'];
+        $commands = ['install', 'update', 'require', 'require-dev'];
         $cmd = $event->getCommand();
 
-        return in_array($cmd, $commands);
-    }
-
-    public function preCommandRun(PreCommandRunEvent $event)
-    {
-        if (!self::needsWebServer($event)) {
+        if (!in_array($cmd, $commands)) {
             return;
         }
 
         $input = $event->getInput();
-        $require = [];
+
+        $options = [];
 
         if ($input->hasArgument('packages')) {
             $packages = $input->getArgument('packages');
 
             $parser = new VersionParser();
 
             $parsed = $parser->parseNameVersionPairs($packages);
+            $require = [];
 
             foreach ($parsed as $pkg) {
                 // Note: there might be multiple ones with the same name
                 // and different version
                 $require[$pkg['name']] = $pkg['version'] ?? 0;
             }
 
-            $options = [
-                'packages' => $require
-            ];
-
-            $this->packman->addPackages($options);
+            $options['packages'] = $require;
         }
 
-        // $this->packman->start($require);
+        // Packman::log($options, $event->getCommand());
+
+        $this->packman->start($options);
     }
 
     // public function initCommand($event)