Refactor file pattern matching in StreamWrapper and improve test coverage for FileFilter exclusions and make sure vendor/src will never included and only in current directory src

Author: rcalicdan

src/Contract/FileFilter.php

@@ -156,7 +156,7 @@ private static function compileGlobToRegex(string $glob, string $baseDir): strin
         } elseif (str_starts_with($glob, '**')) {
             $pattern = '.*' . substr($regex, 4) . '$';
         } else {
-            $pattern = '(^' . preg_quote($baseDir . '/', '#') . '|^.*\/)' . $regex . '$';
+            $pattern = '^' . preg_quote($baseDir . '/', '#') . $regex . '$';
         }
 
         return '#' . $pattern . '#i';

src/Internal/StreamWrapper.php

@@ -111,7 +111,7 @@ public static function unregister(): void
     public static function transformSource(string $source, string $filePath = ''): string
     {
         // Respect per-file suppression tag unless respect_ignore_tags is false
-        if ((bool) (Config::get()['respect_ignore_tags'] ?? true) && (str_contains($source, '@typephp-ignore-file') || str_contains($source, '@typephp-disable-file'))) {
+        if (Config::isRespectIgnoreTagsEnabled() && (str_contains($source, '@typephp-ignore-file') || str_contains($source, '@typephp-disable-file'))) {
             return $source;
         }
 
@@ -242,16 +242,6 @@ public function stream_lock(int $operation): bool
             return true;
         }
 
-        // CRITICAL: PHPStan's internal stub narrows flock's $operation parameter to int<0, 7>.
-        // At PHP runtime, valid bitwise lock operations (such as LOCK_UN = 8 or LOCK_UN | LOCK_NB = 12)
-        // range from 1 to 15. Passing 0 to flock() causes PHP to throw a warning ("must be one of LOCK_SH,
-        // LOCK_EX, or LOCK_UN").
-        //
-        // CONSEQUENCE OF IGNORING: if it bypass PHPStan's narrow stub check here.
-        // RUNTIME SAFETY: The guard clause above ($operation < 1 || $operation > 15) guarantees that invalid
-        // operations (like 0) never reach flock(), preserving full runtime safety and compatibility
-        // with Pest/PHPUnit result-caching (file_put_contents).
-        //
         // @phpstan-ignore argument.type
         return @flock($this->handle, $operation);
     }
@@ -464,7 +454,7 @@ private static function compileGlobToRegex(string $glob): string
         } elseif (str_starts_with($glob, '**')) {
             $pattern = '.*' . substr($regex, 4) . '$';
         } else {
-            $pattern = '(^' . preg_quote(self::$baseDir . '/', '#') . '|^.*\/)' . $regex . '$';
+            $pattern = '^' . preg_quote(self::$baseDir . '/', '#') . $regex . '$';
         }
 
         return '#' . $pattern . '#i';
@@ -507,7 +497,7 @@ private static function isReadOnlyCall(): bool
      */
     private static function isApplicationFile(string $path, string|false $resolvedPath): bool
     {
-        if (! (bool) (Config::get()['enabled'] ?? true)) {
+        if (! Config::isEnabled()) {
             return false; // TypePHP is globally disabled!
         }
 
@@ -661,4 +651,4 @@ private static function extractAndSeedFileMetadata(array $stmts, string $filePat
 
         SpecialTypeResolver::seedFileMetadata($filePath, $namespace, $imports, $classTraitUseDocs);
     }
-}
+}

tests/Internal/StreamWrapperTest.php

@@ -2,6 +2,8 @@
 
 declare(strict_types=1);
 
+use TypePHP\Contract\FileFilter;
+use TypePHP\Internal\Config;
 use TypePHP\Internal\StreamWrapper;
 
 describe('StreamWrapper Unit Tests', function () {
@@ -53,4 +55,32 @@ function testGen(): Generator
             ->and($transformed)->toContain('RuntimeTypeChecker::checkSend')
         ;
     });
+
+   test('strictly isolates vendor files with nested src directories when application includes specific src subpackages', function () {
+        Config::set([
+            'include' => [
+                'src/**',
+                'src/Core/**',         
+                'src/Storefront/**',
+                'src/Administration/**',
+            ],
+            'exclude' => [
+                'vendor/**',          
+                'storage/**',
+                'var/**',
+                'cache/**',
+            ],
+        ]);
+
+        $projectRoot = Config::getProjectRoot();
+
+        $vendorFile = str_replace('\\', '/', $projectRoot . '/vendor/doctrine/dbal/src/Core/Table.php');
+        $appFile = str_replace('\\', '/', $projectRoot . '/src/Core/Framework/Util.php');
+
+        expect(FileFilter::isFileExcluded($vendorFile))->toBeTrue()
+            ->and(FileFilter::isFileExcluded($appFile))->toBeFalse()
+        ;
+
+        Config::reset();
+    });
 });