[MLP-2247] Fix/debug assets compiler on build plugin archive (#175)

* Add logs

* Log before the condition

* Remove lines to isolate problem

* Return early if assets-compiler exists

* Rename variables for clarity

* Remove conditionals to add tools

* Rename step

* Use another syntax

* Use an id instead of name

* Rollback this branch as is in use

* Remove conditional tools

* Conditionally run php-scoper

if there is no config it creates it and runs, we need to avoid that

* Continue on error

* Check the conclusion instead

* Check with failure()

* Use outputs

* Use hashFiles()

* Remove composer config

* Try verbose to see why translate fails

* Try v2 as verbose was not failing

* Try verbose again without config

* Rollback removing the config

* Add composer/installers to the list of tools & remove leftover

---------

Co-authored-by: Orestis Samaras <o.samaras@inpsyde.com>
Co-authored-by: Philipp Bammes <8144115+tyrann0us@users.noreply.github.com>
Co-authored-by: Moritz Meißelbach <m.meisselbach@inpsyde.com>
Co-authored-by: Oleksandr Zhyian <oleksandr.zhyian@gmail.com>

Author: 5 people

.github/workflows/build-plugin-archive.yml

@@ -164,41 +164,7 @@ jobs:
         uses: shivammathur/setup-php@v2
         with:
           php-version: ${{ inputs.PHP_VERSION_BUILD }}
-          tools: humbug/php-scoper, sniccowp/php-scoper-wordpress-excludes, rector
-
-      - name: Check optional Composer build tools
-        id: composer-tools
-        run: |
-          hasAssetConfig(){
-            test -f assets-compiler.json
-            local EXIT=$?
-            if [ ! $EXIT ]; then
-              echo "$EXIT"
-              exit 0
-            fi
-            jq '.extra | has("composer-asset-compiler")' --exit-status < composer.json >/dev/null 2>&1
-            local EXIT=$?
-            echo "$EXIT"
-          }
-          hasTranslateConfig(){
-            jq '.extra | has("wp-translation-downloader")' --exit-status < composer.json >/dev/null 2>&1
-            local EXIT=$?
-            echo "$EXIT"
-          }
-          hasScoperConfig(){
-            test -f scoper.inc.php
-            local EXIT=$?
-            echo "$EXIT"
-          }
-          hasRectorConfig(){
-            test -f rector.php
-            local EXIT=$?
-            echo "$EXIT"
-          }
-          echo "assets-compiler=$( hasAssetConfig )" >> $GITHUB_OUTPUT
-          echo "translation-downloader=$( hasTranslateConfig )" >> $GITHUB_OUTPUT
-          echo "php-scoper=$( hasScoperConfig )" >> $GITHUB_OUTPUT
-          echo "rector=$( hasRectorConfig )" >> $GITHUB_OUTPUT
+          tools: rector, php-scoper, sniccowp/php-scoper-wordpress-excludes, composer/installers, inpsyde/composer-assets-compiler, inpsyde/wp-translation-downloader
 
       - name: Set up node cache mode
         run: |
@@ -211,7 +177,6 @@ jobs:
           fi
 
       - name: Set up node
-        if: steps.composer-tools.outputs.assets-compiler == '0'
         uses: actions/setup-node@v4
         env:
           NODE_OPTIONS: ${{ inputs.NODE_OPTIONS }}
@@ -221,32 +186,21 @@ jobs:
           registry-url: ${{ inputs.NPM_REGISTRY_DOMAIN }}
           cache: ${{ env.NODE_CACHE_MODE }}
 
-      - name: Install and run Composer Asset Compiler
-        if: steps.composer-tools.outputs.assets-compiler == '0'
-        env:
-          COMPOSER_AUTH: '${{ secrets.COMPOSER_AUTH_JSON }}'
+      - name: Run Composer Asset Compiler
         run: |
-          composer global config --no-plugins --no-interaction allow-plugins.inpsyde/composer-assets-compiler true
-          composer global require inpsyde/composer-assets-compiler
           composer compile-assets ${{ inputs.COMPILE_ASSETS_ARGS }}
 
-      - name: Install and run WordPress Translation Downloader
-        if: steps.composer-tools.outputs.translation-downloader == '0'
-        env:
-          COMPOSER_AUTH: '${{ secrets.COMPOSER_AUTH_JSON }}'
+      - name: Run WordPress Translation Downloader
         run: |
-          composer global config --no-plugins --no-interaction allow-plugins.composer/installers true
-          composer global config --no-plugins --no-interaction allow-plugins.inpsyde/wp-translation-downloader true
-          composer global require inpsyde/wp-translation-downloader
           composer wp-translation-downloader:download
 
       - name: Run Rector
-        if: steps.composer-tools.outputs.rector == '0'
+        if: ${{ hashFiles('rector.php') != '' }}
         run: |
           rector
 
       - name: Run PHP-Scoper ensuring isolated file autoloading
-        if: steps.composer-tools.outputs.php-scoper == '0'
+        if: ${{ hashFiles('scoper.inc.php') != '' }}
         run: |
           php-scoper add-prefix --force --output-dir=build
           composer --working-dir=build dump-autoload -o
@@ -256,7 +210,7 @@ jobs:
           sed -i "s/'__composer_autoload_files'/\'__composer_autoload_files_${{ github.sha }}'/g" "build/vendor/composer/autoload_real.php"
 
       - name: Move code to the `build/` directory
-        if: steps.composer-tools.outputs.php-scoper != '0'
+        if: ${{ hashFiles('scoper.inc.php') == '' }}
         run: |
           shopt -s extglob dotglob
           mkdir build