fix(parser): prevent trait use statements from being parsed as namespace imports

Fixes issue #133: when a class uses traits without any namespace use imports,
the parser would treat 'use Trait;' inside the class body as namespace imports,
causing new imports to be inserted in the wrong location.

- Add classDeclaration null check before matching use statements
- Add regression tests for trait use with and without namespace imports

Author: MehediDracula

src/core/DeclarationParser.ts

@@ -39,7 +39,7 @@ export class DeclarationParser {
                 declarationLines.declare = line + 1;
             } else if (/^\s*(namespace\s|<\?php\s+namespace\s)/.test(text)) {
                 declarationLines.namespace = line + 1;
-            } else if (/^\s*use\s+/.test(text) && !/^\s*use\s*\(/.test(text)) {
+            } else if (declarationLines.classDeclaration === null && /^\s*use\s+/.test(text) && !/^\s*use\s*\(/.test(text)) {
                 // Match 'use Foo\Bar;', 'use function ...', 'use const ...' but not 'use ($var)' (closure use)
                 const kindMatch = text.match(/^\s*use\s+(function|const)\s/);
                 const kind: 'class' | 'function' | 'const' = kindMatch ? kindMatch[1] as 'function' | 'const' : 'class';

src/test/suite/declarationParser.test.ts

@@ -370,6 +370,24 @@ suite('DeclarationParser (VS Code Integration)', () => {
         assert.deepStrictEqual(names, ['User']);
     });
 
+    test('should not parse trait use statements as namespace imports (#133)', async () => {
+        const doc = await createDocument(
+            '<?php\n\nnamespace App\\Models;\n\nclass Project extends Model {\n    use \\App\\Traits\\Validation;\n}'
+        );
+        const { useStatements, declarationLines } = parser.parse(doc);
+        assert.strictEqual(useStatements.length, 0);
+        assert.strictEqual(declarationLines.lastUseStatement, null);
+    });
+
+    test('should not parse trait use when namespace imports exist (#133)', async () => {
+        const doc = await createDocument(
+            '<?php\n\nnamespace App\\Models;\n\nuse Illuminate\\Support\\Carbon;\n\nclass Project extends Model {\n    use \\App\\Traits\\Validation;\n}'
+        );
+        const { useStatements } = parser.parse(doc);
+        assert.strictEqual(useStatements.length, 1);
+        assert.strictEqual(useStatements[0].className, 'Carbon');
+    });
+
     test('should throw when class in grouped import is already imported', async () => {
         const doc = await createDocument(
             '<?php\n\nuse App\\Models\\{User, Post};\n\nclass Foo {}'