feat: Add BaseModules property $registrarHasData

Author: neznaika0

app/Config/Modules.php

@@ -62,6 +62,14 @@ class Modules extends BaseModules
      */
     public $composerPackages = [];
 
+    /**
+     * If set to `true`, Registrars may have previous config values as an array.
+     * This is useful when updating arrays or checking properties.
+     *
+     * NOTE: Enable this option only for trusted modules/packages.
+     */
+    public bool $registrarHasData = false;
+
     /**
      * --------------------------------------------------------------------------
      * Auto-Discovery Rules

system/Config/BaseConfig.php

@@ -274,21 +274,36 @@ protected function registerProperties()
 
         $shortName = (new ReflectionClass($this))->getShortName();
 
+        if (static::$moduleConfig->registrarHasData) {
+            // Get all public properties for this config
+            $worker = new class () {
+                /**
+                 * @return array<string, mixed>
+                 */
+                public function getProperties(BaseConfig $obj): array
+                {
+                    return get_object_vars($obj);
+                }
+            };
+        }
+
         // Check the registrar class for a method named after this class' shortName
         foreach (static::$registrars as $callable) {
             // ignore non-applicable registrars
             if (! method_exists($callable, $shortName)) {
                 continue; // @codeCoverageIgnore
             }
 
-            $properties = $callable::$shortName();
+            $currentProps = static::$moduleConfig->registrarHasData ? $worker->getProperties($this) : [];
+            $properties   = $callable::$shortName($currentProps);
 
             if (! is_array($properties)) {
                 throw new RuntimeException('Registrars must return an array of properties and their values.');
             }
 
             foreach ($properties as $property => $value) {
-                if (isset($this->{$property}) && is_array($this->{$property}) && is_array($value)) {
+                // TODO: The array check can be removed if the option `registrarHasData` is accepted.
+                if (isset($this->{$property}) && is_array($this->{$property}) && is_array($value) && ! static::$moduleConfig->registrarHasData) {
                     $this->{$property} = array_merge($this->{$property}, $value);
                 } else {
                     $this->{$property} = $value;