Address Gemini code review comments

Improvements based on Gemini's medium-priority feedback:

1. Centralize debug module imports:
   - test/shared/setup.mjs: Use import at top instead of require in hooks
   - test/unit/core/utils.test.mjs: Add import debug at top, reuse in hooks
   - test/unit/core/utils-enhanced.test.mjs: Add import debug at top, reuse in hooks

2. Clean up redundant configuration:
   - vitest.config.mjs: Remove redundant silent and reporters defaults

Benefits:
- Consistent ES module import style across all test files
- DRY principle - import once, reuse throughout the file
- Cleaner vitest configuration without unnecessary defaults

All 632 tests still passing with zero debug output.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>

Author: Claude Code

test/shared/setup.mjs

@@ -1,13 +1,13 @@
 import { beforeEach, afterEach, vi } from 'vitest';
 import { stopAllServers } from '../helpers/test-server.mjs';
+import debug from 'debug';
 
 // Increase max listeners to avoid warnings when running many tests with child processes
 // Vitest adds exit/beforeExit listeners for each test file with spawned processes
 process.setMaxListeners(30);
 
 // Disable debug output during tests unless explicitly enabled
 if (!process.env.DEBUG) {
-  const debug = require('debug');
   debug.disable();
 }
 
@@ -30,7 +30,6 @@ afterEach(async () => {
 
   // Re-disable debug after each test to prevent leakage
   if (!process.env.DEBUG) {
-    const debug = require('debug');
     debug.disable();
   }
 });

test/unit/core/utils-enhanced.test.mjs

@@ -9,6 +9,7 @@
  */
 import { describe, it, expect, beforeEach, afterEach, vi } from 'vitest';
 import * as utils from '../../../lib/utils.js';
+import debug from 'debug';
 
 describe('Utils Module - Enhanced Coverage', () => {
   describe('BufferingLogger.printOutput() behavior', () => {
@@ -20,7 +21,6 @@ describe('Utils Module - Enhanced Coverage', () => {
     });
 
     afterEach(() => {
-      const debug = require('debug');
       debug.disable();
       if (originalDebugEnv !== undefined) {
         process.env.DEBUG = originalDebugEnv;
@@ -67,7 +67,6 @@ describe('Utils Module - Enhanced Coverage', () => {
     });
 
     afterEach(() => {
-      const debug = require('debug');
       debug.disable();
       if (originalDebugEnv !== undefined) {
         process.env.DEBUG = originalDebugEnv;

test/unit/core/utils.test.mjs

@@ -11,6 +11,7 @@
 import { describe, it, expect, beforeEach, afterEach, vi } from 'vitest';
 import { EventEmitter } from 'events';
 import * as utils from '../../../lib/utils.js';
+import debug from 'debug';
 
 describe('Utils Module', () => {
   describe('noop()', () => {
@@ -251,7 +252,6 @@ describe('Utils Module', () => {
     });
 
     afterEach(() => {
-      const debug = require('debug');
       debug.disable();
       if (originalDebugEnv !== undefined) {
         process.env.DEBUG = originalDebugEnv;

vitest.config.mjs

@@ -11,9 +11,6 @@ export default defineConfig({
         singleThread: true
       }
     },
-    // Silence debug output during tests
-    silent: false,
-    reporters: ['default'],
     // Timeouts for WebSocket operations
     testTimeout: 15000,
     hookTimeout: 15000,