Merge pull request #144 from php-api-clients/store-state-cache-file-to-prevent-unesecary-file-overwriting-and-spec-generating

Store state cache to prevent unnecessary file overwriting with the same contents

Author: WyriHaximus

Makefile

@@ -21,6 +21,7 @@ else
 		-v "`pwd`:`pwd`" \
 		-v "${COMPOSER_CACHE_DIR}:/home/app/.composer/cache" \
 		-w "`pwd`" \
+		-e "FORCE_GENERATION=$$FORCE_GENERATION" \
 		"wyrihaximusnet/php:8.2-nts-alpine-slim-dev"
 endif
 

bin/openapi-client-generator.source

@@ -20,12 +20,12 @@ use Symfony\Component\Yaml\Yaml;
     exit((function (string $configurationFile): int {
         $configuration = (new ObjectMapperUsingReflection())->hydrateObject(Configuration::class, Yaml::parseFile($configurationFile));
         (new Generator(
-            $configuration->spec
+            $configuration,
+            dirname($configurationFile) . DIRECTORY_SEPARATOR,
         ))->generate(
             $configuration->namespace->source . '\\',
             $configuration->namespace->test . '\\',
             dirname($configurationFile) . DIRECTORY_SEPARATOR,
-            $configuration,
         );
 
         return 0;

example/openapi-client-one.yaml

@@ -1,3 +1,8 @@
+state:
+  file: etc/openapi-client-generator.state
+  additionalFiles:
+    - composer.json
+    - composer.lock
 spec: https://raw.githubusercontent.com/github/rest-api-description/main/descriptions-next/api.github.com/api.github.com.yaml
 #spec: file:///home/wyrihaximus/Projects/PHPAPIClients/openapi-client-generator/example/api.github.com.yaml
 namespace:

example/openapi-client-subsplit.yaml

@@ -1,3 +1,8 @@
+state:
+  file: etc/openapi-client-generator.state
+  additionalFiles:
+    - composer.json
+    - composer.lock
 spec: https://raw.githubusercontent.com/github/rest-api-description/main/descriptions-next/api.github.com/api.github.com.yaml
 #spec: file:///home/wyrihaximus/Projects/PHPAPIClients/openapi-client-generator/example/api.github.com.yaml
 namespace:

src/Configuration.php

@@ -17,6 +17,7 @@
      * @param array<class-string<ContentType>>|null $contentType
      */
     public function __construct(
+        public \ApiClients\Tools\OpenApiClientGenerator\Configuration\State $state,
         public string $spec,
         public Templates $templates,
         public Namespace_ $namespace,

src/Configuration/State.php

@@ -0,0 +1,20 @@
+<?php
+
+namespace ApiClients\Tools\OpenApiClientGenerator\Configuration;
+
+use EventSauce\ObjectHydrator\MapFrom;
+use EventSauce\ObjectHydrator\PropertyCasters\CastListToType;
+
+final readonly class State
+{
+    /**
+     * @param array<string> $additionalFiles
+     */
+    public function __construct(
+        public string $file,
+        #[MapFrom('additionalFiles')]
+        #[CastListToType('string')]
+        public ?array $additionalFiles,
+    ) {
+    }
+}