test(jest): Fix fs.remove and logger.info errors - Issue #618

### Errors Fixed (6 occurrences total)

**1. fs.remove is not a function (4 occurrences)**
- File: tests/integration/cli/logCommands.test.js (lines 386, 462)
- Problem: fs-extra's fs.remove() deprecated/unavailable in v11.x
- Fix: Use Node's built-in fs/promises.rm() with recursive option
- Pattern: `await rm(tempLogDir, { recursive: true, force: true })`

**2. logger.info is not a function (2 occurrences)**
- File: src/services/PersonaService.js (line 23)
- Problem: Import as `const logger = require(...)` instead of destructuring
- Fix: Destructure logger from module: `const { logger } = require('../utils/logger')`
- Root cause: Jest mocks export `{ logger: {...} }`, not default export

### Why This Matters

Both errors follow the same pattern as roast.test.js fixes:
- Module-level initialization in Jest environment
- Incorrect import patterns (not matching Jest mock structure)
- Using deprecated/unavailable methods

### Test Results

**Before:**
- fs.remove errors: 4 occurrences
- logger.info errors: 2 occurrences
- autoApprovalSecurityV2.test.js: "logger.info is not a function"

**After:**
- fs.remove errors: ✅ RESOLVED (switched to Node built-in rm())
- logger.info errors: ✅ RESOLVED (destructured import)
- autoApprovalSecurityV2.test.js: Different error (app.address) = progress!

### Files Modified

1. tests/integration/cli/logCommands.test.js
   - Added: `const { rm } = require('fs/promises')`
   - Changed: `fs.remove()` → `rm()` (2 occurrences)

2. src/services/PersonaService.js
   - Changed: `const logger = require(...)` → `const { logger } = require(...)`

### Pattern for coderabbit-lessons.md

This extends Jest compatibility pattern #9:
- Always check if methods exist in target library version
- Prefer Node built-ins over library methods when available
- Ensure imports match Jest mock structure (destructure if mock exports object)

Related: Issue #618 (24/24 roast.test.js passing)

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

Co-Authored-By: Claude <[email protected]>

Author: Eibon7

src/services/PersonaService.js

@@ -20,7 +20,7 @@
 const { createClient } = require('@supabase/supabase-js');
 const { encryptField, decryptField } = require('../utils/encryption');
 const EmbeddingsService = require('./embeddingsService');
-const logger = require('../utils/logger');
+const { logger } = require('../utils/logger'); // Issue #618 - destructure logger from module
 const { mockMode } = require('../config/mockMode');
 
 // Plan-based access control configuration

tests/integration/cli/logCommands.test.js

@@ -12,6 +12,7 @@
 const { execSync, spawn } = require('child_process');
 const path = require('path');
 const fs = require('fs-extra');
+const { rm } = require('fs/promises'); // Node built-in for fs.rm (Issue #618)
 
 // Test configuration
 const CLI_PATH = path.join(__dirname, '../../../cli.js');
@@ -382,8 +383,8 @@ describe('Log Commands CLI Integration', () => {
     });
 
     afterAll(async () => {
-      // Clean up temp directory
-      await fs.remove(tempLogDir);
+      // Clean up temp directory (Issue #618 - use Node's built-in fs/promises.rm)
+      await rm(tempLogDir, { recursive: true, force: true });
     });
 
     test('should perform complete backup and cleanup cycle', async () => {
@@ -458,8 +459,8 @@ describe('Log Commands CLI Integration', () => {
     });
 
     afterAll(async () => {
-      // Clean up temp directory
-      await fs.remove(tempLogDir);
+      // Clean up temp directory (Issue #618 - use Node's built-in fs/promises.rm)
+      await rm(tempLogDir, { recursive: true, force: true });
     });
 
     test('should perform complete backup and cleanup cycle', async () => {