From be307a15794246e95799123d9515e1ade0cedf56 Mon Sep 17 00:00:00 2001
From: David Snelling <david@soulcraft.com>
Date: Thu, 3 Sep 2026 09:06:13 -0700
Subject: [PATCH] test(hygiene): close every brain the vfs unit suite creates

Each file opened a Brainy per test (beforeEach) and never closed it.
---
 tests/vfs/tree-operations.unit.test.ts    | 6 +++++-
 tests/vfs/vfs-bug-fixes.unit.test.ts      | 6 +++++-
 tests/vfs/vfs-bulkwrite-race.unit.test.ts | 6 +++++-
 3 files changed, 15 insertions(+), 3 deletions(-)

diff --git a/tests/vfs/tree-operations.unit.test.ts b/tests/vfs/tree-operations.unit.test.ts
index 8c717115..91743227 100644
--- a/tests/vfs/tree-operations.unit.test.ts
+++ b/tests/vfs/tree-operations.unit.test.ts
@@ -3,7 +3,7 @@
  * Ensures tree methods prevent recursion and work correctly
  */
 
-import { describe, it, expect, beforeEach } from 'vitest'
+import { describe, it, expect, beforeEach, afterEach } from 'vitest'
 import { Brainy } from '../../src/brainy.js'
 import { VirtualFileSystem } from '../../src/vfs/VirtualFileSystem.js'
 import { VFSTreeUtils } from '../../src/vfs/TreeUtils.js'
@@ -24,6 +24,10 @@ describe('VFS Tree Operations', () => {
     await vfs.init()
   })
 
+  afterEach(async () => {
+    await brain.close()
+  })
+
   describe('Critical: No Self-Inclusion Bug', () => {
     it('should NEVER return a directory as its own child', async () => {
       // Create test structure
diff --git a/tests/vfs/vfs-bug-fixes.unit.test.ts b/tests/vfs/vfs-bug-fixes.unit.test.ts
index f98d6a76..12199c8b 100644
--- a/tests/vfs/vfs-bug-fixes.unit.test.ts
+++ b/tests/vfs/vfs-bug-fixes.unit.test.ts
@@ -6,7 +6,7 @@
  * - Issue #2: File read decompression error
  */
 
-import { describe, it, expect, beforeEach } from 'vitest'
+import { describe, it, expect, beforeEach, afterEach } from 'vitest'
 import { Brainy } from '../../src/brainy.js'
 import { VirtualFileSystem } from '../../src/vfs/VirtualFileSystem.js'
 
@@ -25,6 +25,10 @@ describe('VFS Bug Fixes', () => {
     await vfs.init()
   })
 
+  afterEach(async () => {
+    await brain.close()
+  })
+
   describe('Issue #1: Duplicate Directory Nodes', () => {
     it('should not create duplicate directory entries when writing multiple files to same directory', async () => {
       // Write multiple files to the same directory (reproduce the bug scenario)
diff --git a/tests/vfs/vfs-bulkwrite-race.unit.test.ts b/tests/vfs/vfs-bulkwrite-race.unit.test.ts
index 238ac6b9..09d68568 100644
--- a/tests/vfs/vfs-bulkwrite-race.unit.test.ts
+++ b/tests/vfs/vfs-bulkwrite-race.unit.test.ts
@@ -12,7 +12,7 @@
  * other operations in parallel batches.
  */
 
-import { describe, it, expect, beforeEach } from 'vitest'
+import { describe, it, expect, beforeEach, afterEach } from 'vitest'
 import { Brainy } from '../../src/brainy.js'
 import { VirtualFileSystem } from '../../src/vfs/VirtualFileSystem.js'
 
@@ -30,6 +30,10 @@ describe('VFS bulkWrite Race Condition Fix', () => {
     await vfs.init()
   })
 
+  afterEach(async () => {
+    await brain.close()
+  })
+
   describe('operation ordering', () => {
     it('should create directories before files when mixed in same batch', async () => {
       // This is the exact scenario that triggered the race condition: