refactor: unify post-install and post-update event handling to run sync command

Signed-off-by: Felipe Sayão Lobato Abreu <github@mentordosnerds.com>

Author: coisa

src/Composer/Plugin.php

@@ -60,50 +60,33 @@ public function getCapabilities(): array
     public static function getSubscribedEvents(): array
     {
         return [
-            ScriptEvents::POST_INSTALL_CMD => 'onPostInstall',
-            ScriptEvents::POST_UPDATE_CMD => 'onPostUpdate',
+            ScriptEvents::POST_INSTALL_CMD => 'runSyncCommand',
+            ScriptEvents::POST_UPDATE_CMD => 'runSyncCommand',
         ];
     }
 
     /**
      * Handles the automated script installation.
      *
-     * This method MUST be triggered by `POST_INSTALL_CMD` and SHALL delegate
-     * the actual work to the `installScripts` utility.
+     * This method MUST execute the `dev-tools:sync` command after relevant Composer operations to ensure
+     * the development tools are correctly synchronized with the current project state.
      *
      * @param Event $event the Composer script event context
      *
      * @return void
      */
-    public function onPostInstall(Event $event): void
+    public function runSyncCommand(Event $event): void
     {
         $event->getComposer()
-            ->getEventDispatcher()
-            ->dispatchScript('dev-tools:sync', true);
-    }
-
-    /**
-     * Handles the automated script synchronization after updates.
-     *
-     * This method MUST be triggered by `POST_UPDATE_CMD` and SHALL ensure
-     * that all development scripts are correctly aligned in the root configuration.
-     *
-     * @param Event $event the Composer script event context
-     *
-     * @return void
-     */
-    public function onPostUpdate(Event $event): void
-    {
-        $event->getComposer()
-            ->getEventDispatcher()
-            ->dispatchScript('dev-tools:sync', true);
+            ->getLoop()
+            ->getProcessExecutor()
+            ->execute('vendor/bin/dev-tools dev-tools:sync');
     }
 
     /**
      * Handles activation lifecycle events for the Composer session.
      *
-     * The method MUST ensure the `dev-tools` script capability exists inside `composer.json` extras.
-     * It SHOULD append it if currently missing.
+     * This method MUST adhere to the standard Composer plugin activation protocol, even if no specific logic is required.
      *
      * @param Composer $composer the primary package configuration instance over Composer
      * @param IOInterface $io interactive communication channels

tests/Composer/PluginTest.php

@@ -18,12 +18,12 @@
 
 namespace FastForward\DevTools\Tests\Composer;
 
-use Composer\Package\RootPackageInterface;
-use Composer\EventDispatcher\EventDispatcher;
 use Composer\Composer;
 use Composer\IO\IOInterface;
 use Composer\Plugin\Capability\CommandProvider;
 use Composer\Script\Event as ScriptEvent;
+use Composer\Util\Loop;
+use Composer\Util\ProcessExecutor;
 use FastForward\DevTools\Composer\Capability\DevToolsCommandProvider;
 use FastForward\DevTools\Composer\Plugin;
 use PHPUnit\Framework\Attributes\CoversClass;
@@ -125,62 +125,29 @@ public function activateWillDoNothing(): void
      * @return void
      */
     #[Test]
-    public function onPostInstallWillInstallScripts(): void
+    public function runSyncCommandWillExecuteDevToolsSync(): void
     {
         $event = $this->prophesize(ScriptEvent::class);
 
-        // Mock RootPackageInterface para getPackage()
-        $package = $this->prophesize(RootPackageInterface::class);
-        $package->getName()
-            ->willReturn('fast-forward/dev-tools');
-        $this->composer->getPackage()
-            ->willReturn($package->reveal());
+        // Mock ProcessExecutor
+        $processExecutor = $this->prophesize(ProcessExecutor::class);
+        $processExecutor->execute('vendor/bin/dev-tools dev-tools:sync')
+            ->shouldBeCalled();
 
-        // Mock EventDispatcher
-        $eventDispatcher = $this->prophesize(EventDispatcher::class);
-        $eventDispatcher->dispatchScript('dev-tools:sync', true)
-            ->willReturn(0);
-        $this->composer->getEventDispatcher()
-            ->willReturn($eventDispatcher->reveal());
+        // Mock Loop
+        $loop = $this->prophesize(Loop::class);
+        $loop->getProcessExecutor()
+            ->willReturn($processExecutor->reveal());
 
-        $event->getComposer()
-            ->willReturn($this->composer->reveal());
-        $event->getIO()
-            ->willReturn($this->io->reveal());
-
-        $this->plugin->onPostInstall($event->reveal());
-
-        self::assertTrue(true); // Evita teste risky
-    }
-
-    /**
-     * @return void
-     */
-    #[Test]
-    public function onPostUpdateWillInstallScripts(): void
-    {
-        $event = $this->prophesize(ScriptEvent::class);
-
-        // Mock RootPackageInterface para getPackage()
-        $package = $this->prophesize(RootPackageInterface::class);
-        $package->getName()
-            ->willReturn('fast-forward/dev-tools');
-        $this->composer->getPackage()
-            ->willReturn($package->reveal());
-
-        // Mock EventDispatcher
-        $eventDispatcher = $this->prophesize(EventDispatcher::class);
-        $eventDispatcher->dispatchScript('dev-tools:sync', true)
-            ->willReturn(0);
-        $this->composer->getEventDispatcher()
-            ->willReturn($eventDispatcher->reveal());
+        // Mock Composer
+        $composer = $this->prophesize(Composer::class);
+        $composer->getLoop()
+            ->willReturn($loop->reveal());
 
         $event->getComposer()
-            ->willReturn($this->composer->reveal());
-        $event->getIO()
-            ->willReturn($this->io->reveal());
+            ->willReturn($composer->reveal());
 
-        $this->plugin->onPostUpdate($event->reveal());
+        $this->plugin->runSyncCommand($event->reveal());
 
         self::assertTrue(true); // Evita teste risky
     }