fix CI/CD: Fixed path builds for build, tar and extract the builds in Release flow

Author: Wtyd

.github/workflows/release.yml

@@ -51,6 +51,7 @@ jobs:
         php --version
         php githooks app:pre-build php
         php githooks app:build
+        git status --short
   
     - name: Upload build artifact
       uses: actions/upload-artifact@v4
@@ -113,32 +114,57 @@ jobs:
     name: Commit the Build
     needs: test_rc
     runs-on: ubuntu-latest
+    strategy:
+      matrix:
+        operating-system: [ubuntu-latest]
+        php-versions: ['7.1'] 
     permissions:
       contents: write
     steps:
       - name: Checkout
         uses: actions/checkout@v4
 
+      - name: Install PHP
+        uses: shivammathur/setup-php@v2 
+        with:
+          php-version: ${{ matrix.php-versions }}
+          coverage: none
+
+      - name: Get composer cache directory
+        id: composer-cache
+        run: echo "dir=$(composer config cache-files-dir)" >> $GITHUB_OUTPUT
+      
+      - name: Cache dependencies
+        uses: actions/cache@v4
+        with:
+          path: ${{ steps.composer-cache.outputs.dir }}
+          key: ${{ runner.os }}-rc-${{ matrix.php-versions }}-${{ hashFiles('**/composer.json') }}
+          restore-keys: |
+            ${{ runner.os }}-
+    
+      - name: Install dependencies
+        run: |
+          chmod -R +x tools/*
+          tools/composer install --prefer-dist --no-progress
+
       - name: Delete Old Builds
         run: rm builds/githooks builds/php7.1/githooks builds/php7.3/githooks
 
       - uses: actions/download-artifact@v4
         with:
-          name: githooks-71
-
-      - uses: actions/download-artifact@v4
-        with:
-          name: githooks-73
+          path: ${{ github.GITHUB_WORKSPACE }}
 
-      - uses: actions/download-artifact@v4
-        with:
-          name: githooks-81
+      - name: Display structure of downloaded files
+        run: |
+          ls -R | grep githooks
+          ls
 
       - name: Extract and check builds
         run: |
-          tar -xvf githooks-71.tar
-          tar -xvf githooks-73.tar
-          tar -xvf githooks-81.tar
+          php githooks app:extract-build --all
+          # tar -xvf githooks-7.1/githooks-7.1.tar
+          # tar -xvf githooks-7.3/githooks-7.3.tar
+          # tar -xvf githooks-8.1/githooks-8.1.tar
           ls -lah builds
           git status
 

app/Commands/ExtractBuildCommand.php

@@ -10,7 +10,7 @@
 
 class ExtractBuildCommand extends Command
 {
-    protected $signature = 'app:extract-build';
+    protected $signature = 'app:extract-build {--all}';
     protected $description = 'Extracts the build from the tar file after the build process.';
 
     /**  @var \Wtyd\GitHooks\Build\Build */
@@ -25,20 +25,40 @@ public function handle()
     {
         $this->title('Extract build');
 
-        $this->task(
-            '   <fg=yellow>1. Extracting build</>',
-            $this->extractBuild()
-        );
+        $allBuilds = boolval($this->option('all'));
 
-        $this->task(
-            '   <fg=yellow>2. Check build</>',
-            $this->checkBuild()
-        );
+        if ($allBuilds) {
+            $this->task(
+                '   <fg=yellow>1. Extracting all builds</>',
+                function () {
+                    $builds = Build::ALL_BUILDS;
+                    foreach ($builds as $tarFile) {
+                        $this->extractBuild($tarFile);
+                    }
+                }
+            );
+        } else {
+            $this->task(
+                '   <fg=yellow>1. Extracting build</>',
+                $this->extractBuild()
+            );
+
+            $this->task(
+                '   <fg=yellow>2. Check build</>',
+                $this->checkBuild()
+            );
+        }
     }
 
-    private function extractBuild(): void
+    /**
+     * Files are in directory with him name. For example ./githooks-7.1/githooks-7.1.tar
+     *
+     * @param string $tarName For example: 'githooks-7.1.tar'
+     */
+    private function extractBuild($tarName = null): void
     {
-        $zipFile = File::name($this->build->getTarName()) . DIRECTORY_SEPARATOR . $this->build->getTarName();
+        $tarName = $tarName ?? $this->build->getTarName();
+        $zipFile = File::name($tarName) . DIRECTORY_SEPARATOR . $tarName;
         $zip = new PharData($zipFile);
         $resultado = $zip->extractTo('./', null, true); // extract to $this->build->getBuildPath();
         if (true === $resultado) {

app/Commands/Zero/BuildCommand.php

@@ -103,7 +103,7 @@ private function build(string $name): BuildCommand
         // $this->tarBuild($name);
         // TODO la compilación bien pero falta mencionar el tar
         $this->output->writeln(
-            sprintf('    Compiled successfully: <fg=green>%s</>', $this->build->getBuildPath() . DIRECTORY_SEPARATOR . $name)
+            sprintf('    Compiled successfully: <fg=green>%s</>', $this->build->getBuildPath() . $name)
         );
 
         return $this;

src/Build/Build.php

@@ -8,6 +8,8 @@
 
 class Build
 {
+    const ALL_BUILDS = ['githooks-7.1.tar', 'githooks-7.3.tar', 'githooks-8.1.tar'];
+
     /** @var string */
     private $phpVersion;
 
@@ -25,7 +27,7 @@ public function __construct()
      */
     private function setPhpVersion(): void
     {
-        $this->phpVersion = implode('.', array_slice(explode('.', phpversion()), 0, 2));
+        $this->phpVersion = phpversion();
     }
 
     /**
@@ -38,7 +40,7 @@ private function setPhpVersion(): void
     private function setBuildPath(): void
     {
         $path = 'builds' . DIRECTORY_SEPARATOR;
-
+        // TODO: Refactor this to use a switch statement like getTarName()
         if (version_compare($this->phpVersion, '7.1', '<')) {
             throw new Exception('GitHooks only supports php 7.1 or greater.', 1);
         }
@@ -56,6 +58,11 @@ public function getBuildPath(): string
         return $this->buildPath;
     }
 
+    public function getBinary(): string
+    {
+        return $this->buildPath . 'githooks';
+    }
+
     public function getPhpVersion(): string
     {
         return $this->phpVersion;
@@ -67,7 +74,7 @@ public function getPhpVersion(): string
     public function getTarName(): string
     {
         $tarName = '';
-        switch ($this->phpVersion) {
+        switch (implode('.', array_slice(explode('.', $this->phpVersion), 0, 2))) {
             case '7.1':
             case '7.2':
                 $tarName = 'githooks-7.1.tar';

src/Utils/ComposerUpdater.php

@@ -17,10 +17,10 @@ public static function phpOldVersions(): void
             return;
         }
 
-        $rootPaht = getcwd();
+        $rootPath = getcwd();
         $build = new Build();
-        $origin = str_replace('/', DIRECTORY_SEPARATOR, "$rootPaht/vendor/wtyd/githooks") . $build->getBuildPath() . DIRECTORY_SEPARATOR . 'githooks';
-        $destiny = str_replace('/', DIRECTORY_SEPARATOR, "$rootPaht/vendor/bin/githooks");
+        $origin = str_replace('/', DIRECTORY_SEPARATOR, "$rootPath/vendor/wtyd/githooks/") . $build->getBinary();
+        $destiny = str_replace('/', DIRECTORY_SEPARATOR, "$rootPath/vendor/bin/githooks");
         if (file_exists($destiny)) {
             unlink($destiny);
         }