Merge pull request #176 from context-hub/hotfix/168

feat: refactor file handling to use FSPath for temporary file creation

Author: butschster

src/Console/MCPServerCommand.php

@@ -109,6 +109,7 @@ public function __invoke(Container $container, DirectoriesInterface $dirs, Appli
                     bindings: new Scope(
                         name: AppScope::Mcp,
                         bindings: [
+                            DirectoriesInterface::class => $dirs,
                             ProjectServiceInterface::class => $projectServiceFactory->create(),
                             HasPrefixLoggerInterface::class => $logger,
                             ConfigLoaderInterface::class => $loader,

src/Source/GitDiff/Fetcher/Source/AbstractGitSource.php

@@ -4,6 +4,7 @@
 
 namespace Butschster\ContextGenerator\Source\GitDiff\Fetcher\Source;
 
+use Butschster\ContextGenerator\Application\FSPath;
 use Butschster\ContextGenerator\Lib\Git\Command;
 use Butschster\ContextGenerator\Lib\Git\CommandsExecutorInterface;
 use Butschster\ContextGenerator\Lib\Git\Exception\GitCommandException;
@@ -42,14 +43,14 @@ public function createFileInfos(string $repository, string $commitReference, str
             }
 
             // Create the temporary file
-            $tempFile = $tempDir . '/' . $file;
-            $tempDirname = \dirname($tempFile);
+            $tempFile = FSPath::create($tempDir)->join($file);
+            $tempDirname = (string) $tempFile->parent();
 
             $this->files->ensureDirectory($tempDirname, 0777);
-            $this->files->write($tempFile, $diff);
+            $this->files->write((string) $tempFile, $diff);
 
             // Create a file info object with additional metadata
-            $fileInfos[] = new class($tempFile, $file, $diff) extends SplFileInfo {
+            $fileInfos[] = new class((string) $tempFile, $file, $diff) extends SplFileInfo {
                 private readonly string $originalPath;
 
                 public function __construct(

src/Source/Tree/TreeSourceBootloader.php

@@ -5,9 +5,7 @@
 namespace Butschster\ContextGenerator\Source\Tree;
 
 use Butschster\ContextGenerator\Application\Bootloader\SourceFetcherBootloader;
-use Butschster\ContextGenerator\Application\Logger\HasPrefixLoggerInterface;
 use Butschster\ContextGenerator\DirectoriesInterface;
-use Butschster\ContextGenerator\Lib\Content\ContentBuilderFactory;
 use Butschster\ContextGenerator\Source\Registry\SourceRegistryInterface;
 use Spiral\Boot\Bootloader\Bootloader;
 use Spiral\Core\FactoryInterface;
@@ -21,8 +19,6 @@ public function defineSingletons(): array
             TreeSourceFetcher::class => static fn(
                 FactoryInterface $factory,
                 DirectoriesInterface $dirs,
-                ContentBuilderFactory $builderFactory,
-                HasPrefixLoggerInterface $logger,
             ): TreeSourceFetcher => $factory->make(TreeSourceFetcher::class, [
                 'basePath' => (string) $dirs->getRootPath(),
             ]),

src/Source/Tree/TreeSourceFetcher.php

@@ -49,6 +49,7 @@ public function fetch(SourceInterface $source, ModifiersApplierInterface $modifi
         $this->logger?->info('Fetching tree source content', [
             'description' => $source->getDescription(),
             'basePath' => $this->basePath,
+            'sourcePaths' => $source->sourcePaths,
             'renderFormat' => $source->renderFormat,
             'hasModifiers' => !empty($source->modifiers),
         ]);