ergebnis · fredden · Dec 31, 2023
@@ -627,6 +627,19 @@ sections, the `Vendor\Composer\VersionConstraintNormalizer` will ensure that
    }
   ```
 
+- empty sections (which are defined as optional in the schema) are automatically removed
+
+  ```diff
+   {
+      "require": {
+          "foo/bar": "^2.3.4"
+      }
+  -   "config": {
+  -       "preferred-install": {}
+  -   }
+   }
+  ```
+
 ## Changelog
 
 The maintainers of this project record notable changes to this project in a [changelog](CHANGELOG.md).

@@ -28,17 +28,20 @@
     private SchemaValidator\SchemaValidator $schemaValidator;
     private SchemaStorage $schemaStorage;
     private string $schemaUri;
+    private bool $pruneEmpty;
 
     public function __construct(
         string $schemaUri,
         SchemaStorage $schemaStorage,
         SchemaValidator\SchemaValidator $schemaValidator,
-        Pointer\Specification $specificationForPointerToDataThatShouldNotBeSorted
+        Pointer\Specification $specificationForPointerToDataThatShouldNotBeSorted,
+        bool $pruneEmpty = false
     ) {
         $this->schemaUri = $schemaUri;
         $this->schemaStorage = $schemaStorage;
         $this->schemaValidator = $schemaValidator;
         $this->specificationForPointerToDataThatShouldNotBeSorted = $specificationForPointerToDataThatShouldNotBeSorted;
+        $this->pruneEmpty = $pruneEmpty;
     }
 
     public function normalize(Json $json): Json
@@ -223,6 +226,14 @@
                     $pointerToData->append(Pointer\ReferenceToken::fromString($name)),
                 );
 
+                if (
+                    $this->pruneEmpty
+                    && [] === (array) $normalized->{$name}
+                    && self::isKeyOptionalInSchema($schema, $name)
+                ) {
+                    unset($normalized->{$name});
+                }
+
                 unset($data->{$name});
             }
         }
@@ -327,4 +338,17 @@
 
         return $schema;
     }
+
+    private static function isKeyOptionalInSchema(object $schema, string $name): bool
+    {
+        if (
+            \property_exists($schema, 'required')
+            && \is_array($schema->required)
+            && \in_array($name, $schema->required, true)
+        ) {
+            return false;
+        }
+
+        return true;
+    }
 }
@@ -89,6 +89,7 @@ public function __construct(string $schemaUri)
                      */
                     Pointer\Specification::equals(Pointer\JsonPointer::fromJsonString('/scripts/auto-scripts')),
                 ),
+                true,
             ),
             new BinNormalizer(),
             new ConfigHashNormalizer(),

@@ -13,6 +13,5 @@
             "email": "am@localheinz.com"
         }
     ],
-    "homepage": "https://getcomposer.org/doc/04-schema.md#bin",
-    "bin": []
+    "homepage": "https://getcomposer.org/doc/04-schema.md#bin"
 }
@@ -13,6 +13,5 @@
             "email": "am@localheinz.com"
         }
     ],
-    "homepage": "https://getcomposer.org/doc/06-config.md",
-    "config": {}
+    "homepage": "https://getcomposer.org/doc/06-config.md"
 }
@@ -13,8 +13,5 @@
             "email": "am@localheinz.com"
         }
     ],
-    "homepage": "https://getcomposer.org/doc/06-config.md#allow-plugins",
-    "config": {
-        "allow-plugins": {}
-    }
+    "homepage": "https://getcomposer.org/doc/06-config.md#allow-plugins"
 }
@@ -13,8 +13,5 @@
             "email": "am@localheinz.com"
         }
     ],
-    "homepage": "https://getcomposer.org/doc/06-config.md#preferred-install",
-    "config": {
-        "preferred-install": {}
-    }
+    "homepage": "https://getcomposer.org/doc/06-config.md#preferred-install"
 }
@@ -15,7 +15,6 @@
     ],
     "homepage": "https://getcomposer.org/doc/06-config.md",
     "config": {
-        "allow-plugins": {},
         "platform": {
             "php": "8.0.25"
         },

@@ -15,7 +15,6 @@
     ],
     "homepage": "https://getcomposer.org/doc/06-config.md",
     "config": {
-        "allow-plugins": {},
         "platform": {
             "php": "8.0.25"
         },

@@ -13,6 +13,5 @@
             "email": "am@localheinz.com"
         }
     ],
-    "homepage": "https://getcomposer.org/doc/04-schema.md#extra",
-    "extra": {}
+    "homepage": "https://getcomposer.org/doc/04-schema.md#extra"
 }
@@ -13,6 +13,5 @@
             "email": "am@localheinz.com"
         }
     ],
-    "homepage": "https://getcomposer.org/doc/04-schema.md#repositories",
-    "repositories": []
+    "homepage": "https://getcomposer.org/doc/04-schema.md#repositories"
 }
@@ -1,6 +1,5 @@
 {
     "description": "This test exists to cover certain mutants from infection/infection",
-    "require": {},
     "require-dev": {
         "ergebnis/overlapping-version-constraints-1": "^1.0 || 3.4.5",
         "ergebnis/overlapping-version-constraints-2": "^1.0 || ^2.0",

@@ -13,6 +13,5 @@
             "email": "am@localheinz.com"
         }
     ],
-    "homepage": "https://getcomposer.org/doc/04-schema.md#scripts",
-    "scripts": {}
+    "homepage": "https://getcomposer.org/doc/04-schema.md#scripts"
 }
@@ -1,4 +1,3 @@
 {
-    "homepage": "https://getcomposer.org/doc/04-schema.md#suggest",
-    "suggest": {}
+    "homepage": "https://getcomposer.org/doc/04-schema.md#suggest"
 }
@@ -1,6 +1,5 @@
 {
     "homepage": "https://getcomposer.org/doc/04-schema.md#suggest",
-    "require": {},
     "suggest": {
         "php": "Nothing works without it",
         "hhvm": "Okay",

@@ -1,3 +1 @@
-{
-    "value-contains-packages-and-version-constraints": {}
-}
+{}