mirror of
https://github.com/KeygraphHQ/shannon.git
synced 2026-05-25 18:07:54 +02:00
f13c7421f4
Comprehensive codebase cleanup based on parallel agent analysis and automated dead code detection (knip, depcheck). Reduces codebase by ~10% with zero functional changes. ## Phase 1: Obsolete MCP Setup Removal (~82 lines) - Delete setupMCP() and cleanupMCP() functions from environment.js - Remove all calls to cleanupMCP() (8 instances across 3 files) - Migrate from claude CLI to SDK's mcpServers option - Remove --log flag (obsolete logging system) ## Phase 2: Dead Code Removal (~317 lines) - Delete src/utils/logger.js entirely (127 lines, superseded by audit system) - Remove handleConfigError() and handleError() from error-handling.js - Remove isToolAvailable() from tool-checker.js - Remove 5 dead methods from audit-session.js (logSessionFailure, logMessage, markRolledBack, updateValidation, getValidation) - Remove 6 wrapper methods from audit/logger.js (all callers use logEvent directly) - Remove formatCost(), updateMessage(), compose() utilities (unused) ## Phase 3: Consolidation (~195 lines) - Extract SessionMutex to src/utils/concurrency.js (was duplicated in 2 files) - Consolidate formatDuration to src/audit/utils.js (was in 3 files) - Extract readline prompts to src/cli/prompts.js (was duplicated in 2 files) - Create validator factories in constants.js (reduce 72 lines to 30) ## Impact - Total reduction: 488 lines (20 files modified, 2 created, 1 deleted) - Codebase: ~4,900 → ~4,400 LOC (10% reduction) - Zero functional changes, all tests pass - Improved maintainability and DRY compliance 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
55 lines
1.4 KiB
JavaScript
55 lines
1.4 KiB
JavaScript
/**
|
|
* Concurrency Control Utilities
|
|
*
|
|
* Provides mutex implementation for preventing race conditions during
|
|
* concurrent session operations.
|
|
*/
|
|
|
|
/**
|
|
* SessionMutex - Promise-based mutex for session file operations
|
|
*
|
|
* Prevents race conditions when multiple agents or operations attempt to
|
|
* modify the same session data simultaneously. This is particularly important
|
|
* during parallel execution of vulnerability analysis and exploitation phases.
|
|
*
|
|
* Usage:
|
|
* ```js
|
|
* const mutex = new SessionMutex();
|
|
* const unlock = await mutex.lock(sessionId);
|
|
* try {
|
|
* // Critical section - modify session data
|
|
* } finally {
|
|
* unlock(); // Always release the lock
|
|
* }
|
|
* ```
|
|
*/
|
|
export class SessionMutex {
|
|
constructor() {
|
|
// Map of sessionId -> Promise (represents active lock)
|
|
this.locks = new Map();
|
|
}
|
|
|
|
/**
|
|
* Acquire lock for a session
|
|
* @param {string} sessionId - Session ID to lock
|
|
* @returns {Promise<Function>} Unlock function to release the lock
|
|
*/
|
|
async lock(sessionId) {
|
|
if (this.locks.has(sessionId)) {
|
|
// Wait for existing lock to be released
|
|
await this.locks.get(sessionId);
|
|
}
|
|
|
|
// Create new lock promise
|
|
let resolve;
|
|
const promise = new Promise(r => resolve = r);
|
|
this.locks.set(sessionId, promise);
|
|
|
|
// Return unlock function
|
|
return () => {
|
|
this.locks.delete(sessionId);
|
|
resolve();
|
|
};
|
|
}
|
|
}
|