feat: Refactor URL import handling and add comprehensive tests for various scenarios

Author: butschster

src/Config/Import/Source/ImportSourceProvider.php

@@ -10,14 +10,15 @@
 use Butschster\ContextGenerator\Config\Import\Source\Registry\ImportSourceRegistry;
 use Psr\Log\LoggerInterface;
 use Psr\Log\NullLogger;
+use Spiral\Core\Container;
 
 /**
  * Service provider for accessing import sources
  */
 final readonly class ImportSourceProvider
 {
     public function __construct(
-        private ImportSourceRegistry $sourceRegistry,
+        private Container $container,
         #[LoggerPrefix(prefix: 'import-sources')]
         private ?LoggerInterface $logger = null,
     ) {}
@@ -31,7 +32,7 @@ public function __construct(
      */
     public function getSource(string $name): ImportSourceInterface
     {
-        return $this->sourceRegistry->get($name);
+        return $this->getSourceRegistry()->get($name);
     }
 
     /**
@@ -44,15 +45,15 @@ public function findSourceForConfig(SourceConfigInterface $config): ?ImportSourc
     {
         // First try to find source by type
         $sourceName = $config->getType();
-        if ($this->sourceRegistry->has($sourceName)) {
-            $source = $this->sourceRegistry->get($sourceName);
+        if ($this->getSourceRegistry()->has($sourceName)) {
+            $source = $this->getSourceRegistry()->get($sourceName);
             if ($source->supports($config)) {
                 return clone $source;
             }
         }
 
         // If not found by type, try all registered sources
-        foreach ($this->sourceRegistry->all() as $source) {
+        foreach ($this->getSourceRegistry()->all() as $source) {
             if ($source->supports($config)) {
                 return clone $source;
             }
@@ -66,23 +67,8 @@ public function findSourceForConfig(SourceConfigInterface $config): ?ImportSourc
         return null;
     }
 
-    /**
-     * Get a logger with a source-specific prefix
-     *
-     * @param string $sourceName Name of the source for logger prefixing
-     * @return LoggerInterface Logger with appropriate prefix
-     */
-    public function getSourceLogger(string $sourceName): LoggerInterface
+    private function getSourceRegistry(): ImportSourceRegistry
     {
-        if ($this->logger === null) {
-            return new NullLogger();
-        }
-
-        // Check if logger supports prefixing
-        if ($this->logger instanceof HasPrefixLoggerInterface) {
-            return $this->logger->withPrefix("import-{$sourceName}");
-        }
-
-        return $this->logger;
+        return $this->container->get(ImportSourceRegistry::class);
     }
 }

src/Config/Import/Source/Url/UrlImportSource.php

@@ -61,7 +61,7 @@ public function load(SourceConfigInterface $config): array
         }
 
         try {
-            $url = $config->url;
+            $url = $this->container->get(VariableResolver::class)->resolve($config->url);
             $headers = $this->container->get(VariableResolver::class)->resolve($config->headers);
 
             $this->logger->debug('Loading URL import', [