fix: reset clears testsuite and namespace nodes from tree (#390)

handleReset() only removed TestType.class items, leaving orphaned
testsuite and namespace nodes in the VS Code test tree. On reload,
these stale nodes caused duplicate test entries.

Now deletes all non-workspace root items so the tree is fully cleared
before rediscovery.

Author: recca0120

packages/extension/src/TestCollection/TestCollection.test.ts

@@ -1763,6 +1763,28 @@ class NoDatasetTest extends TestCase
             }
         });
 
+        it('reset should clear testsuite and namespace nodes from tree (#390)', async () => {
+            const collection = givenTestCollection(`
+                <testsuites>
+                    <testsuite name="Unit">
+                        <directory>tests/Unit</directory>
+                    </testsuite>
+                    <testsuite name="Feature">
+                        <directory>tests/Feature</directory>
+                    </testsuite>
+                </testsuites>`);
+
+            await collection.add(URI.file(phpUnitProject('tests/Unit/ExampleTest.php')));
+            await collection.add(URI.file(phpUnitProject('tests/Feature/ExampleTest.php')));
+
+            expect(ctrl.items.size).toBeGreaterThan(0);
+
+            collection.reset();
+
+            // After reset, the VS Code tree should be completely empty
+            expect(ctrl.items.size).toBe(0);
+        });
+
         it('add test — all fixture files discovered', async () => {
             const collection = givenTestCollection(`
                 <testsuites>

packages/extension/src/TestCollection/TestCollection.ts

@@ -195,13 +195,11 @@ export class TestCollection extends BaseTestCollection {
     private handleReset() {
         const workspaceDefs = this.index.getDefinitionsByType(TestType.workspace);
 
-        for (const [testItem] of this.index.getDefinitionsByType(TestType.class)) {
-            if (!testItem.parent) {
-                this.rootItems.delete(testItem.id);
-                continue;
+        const keepIds = new Set(workspaceDefs.map(([item]) => item.id));
+        for (const [id] of this.rootItems) {
+            if (!keepIds.has(id)) {
+                this.rootItems.delete(id);
             }
-
-            testItem.parent.children.delete(testItem.id);
         }
         this.index.clear();