Fixed issues related to jq clearing te composer.json when adding composer autoloads

Author: gplanchat

src/PHP/ComposerAutoload.php

@@ -6,16 +6,14 @@
 
 use Kiboko\Component\Dockerfile\Dockerfile;
 
-final class ComposerAutoload implements Dockerfile\LayerInterface
+final readonly class ComposerAutoload implements Dockerfile\LayerInterface, \Stringable
 {
-    /**
-     * @param array<string, array<string, string|list<string>>> $autoloads
-     */
-    private array $autoloads;
-
-    public function __construct(array $autoloads)
-    {
-        $this->autoloads = $autoloads;
+    public function __construct(
+        /**
+         * @param array<string, array<string, string|list<string>>> $autoloads
+         */
+        private array $autoloads
+    ) {
     }
 
     private static function command(string ...$command): string
@@ -50,22 +48,45 @@ private static function pipe(string ...$commands): string
         return implode(' | ', $commands);
     }
 
-    public function __toString()
+    private static function indirection(string $command, string $destination): string
+    {
+        return $command . ' > ' . self::escapeArgument($destination);
+    }
+
+    private static function and(string ...$commands): string
     {
-        $commands = implode(' \\' . PHP_EOL . '    && ', array_map(fn ($type, $autoload) => match ($type) {
-            'psr4' => self::pipe(
-                self::command('cat', 'composer.json'),
-                self::command('jq', '--indent', '4', sprintf('.autoload."psr-4" |= . + %s', json_encode($autoload))),
-                self::command('tee', 'composer.json')
+        return implode(' \\'.\PHP_EOL.'    && ', $commands);
+    }
+
+    public function __toString(): string
+    {
+        if (\count($this->autoloads) <= 0) {
+            return '';
+        }
+
+        $commands = self::and(
+            ...array_map(
+                fn ($type, $autoload) => match ($type) {
+                    'psr4' => self::and(
+                        self::pipe(
+                            self::command('cat', 'composer.json'),
+                            self::command('jq', '--indent', '4', sprintf('.autoload."psr-4" |= . + %s', json_encode($autoload, \JSON_THROW_ON_ERROR))),
+                            self::command('tee', 'composer.temp')
+                        ),
+                        self::indirection(self::command('cat', 'composer.temp'), 'composer.json'),
+                        self::command('rm', 'composer.temp'),
+                    )
+                },
+                array_keys($this->autoloads),
+                array_values($this->autoloads)
             )
-        }, array_keys($this->autoloads), array_values($this->autoloads)));
+        );
 
-        $d = $commands;
         return (string) new Dockerfile\Run(
             <<<RUN
-            set -ex \\
-                && {$commands}
-            RUN
+                set -ex \\
+                    && {$commands}
+                RUN
         );
     }
 }