Add a workaround with globals to ensure service mapping

Author: mglaman

src/Drupal/DrupalAutoloader.php

@@ -191,8 +191,14 @@ class: Drupal\jsonapi\Routing\JsonApiParamEnhancer
 
         $service_map = $container->getByType(ServiceMap::class);
         assert($service_map instanceof ServiceMap);
-        // @todo this is not updating the reference in the container.
+        // @todo this is a hack that needs investigation.
+        // We cannot manipulate the service container and add parameters, so we take the existing
+        // service and modify it's properties so that its reference is updated within the container.
+        //
+        // During debug this works, but other times it fails.
         $service_map->setDrupalServices($this->serviceMap);
+        // So, to work around whatever is happening we force it into globals.
+        $GLOBALS['drupalServiceMap'] = $service_map->getServices();
     }
 
     protected function loadLegacyIncludes(): void

src/Drupal/ServiceMap.php

@@ -11,12 +11,25 @@ class ServiceMap
 
     public function getService(string $id): ?DrupalServiceDefinition
     {
+        // @see notes in DrupalAutoloader.
+        // This is all a work around due to inability to set container parameters.
         if (count($this->services) === 0) {
-            throw new ShouldNotHappenException('No Drupal service map was registered.');
+            $this->services = $GLOBALS['drupalServiceMap'];
+            if (count($this->services) === 0) {
+                throw new ShouldNotHappenException('No Drupal service map was registered.');
+            }
         }
         return $this->services[$id] ?? null;
     }
 
+    /**
+     * @return \PHPStan\Drupal\DrupalServiceDefinition[]
+     */
+    public function getServices(): array
+    {
+        return $this->services;
+    }
+
     public function setDrupalServices(array $drupalServices): void
     {
         $this->services = [];