From 251ec93150000096ed163a1311373510a76332b8 Mon Sep 17 00:00:00 2001 From: Nikita Levyankov Date: Thu, 18 Dec 2025 12:59:22 +0200 Subject: [PATCH] refactor: remove dead code (profileModules/Agents/Workflows) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Removed unused properties that were set but never read: - normalized.profileModules - normalized.profileAgents - normalized.profileWorkflows These values were: 1. Stored as unwrapped (strings like 'all' or arrays) 2. Never accessed anywhere in the codebase 3. Created confusion - actual values used are normalized.modules/agents/workflows 4. Inconsistent with the wrapped versions actually used downstream The profile values are already correctly processed and stored in normalized.modules/agents/workflows (with proper array wrapping), which are then passed to installer via config.cliOptions. No functional change - just removing dead code that served no purpose. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude --- tools/cli/installers/lib/core/options-parser.js | 7 +------ 1 file changed, 1 insertion(+), 6 deletions(-) diff --git a/tools/cli/installers/lib/core/options-parser.js b/tools/cli/installers/lib/core/options-parser.js index 36bb110b..036ee6a0 100644 --- a/tools/cli/installers/lib/core/options-parser.js +++ b/tools/cli/installers/lib/core/options-parser.js @@ -121,12 +121,7 @@ function parseOptions(cliOptions) { throw new Error(`Unknown profile: ${normalized.profile}. Valid profiles: minimal, full, solo-dev, team`); } - // Profile provides defaults, but CLI options override - normalized.profileModules = profile.modules; - normalized.profileAgents = profile.agents; - normalized.profileWorkflows = profile.workflows; - - // If no explicit modules/agents/workflows, use profile values + // Use profile values as defaults when CLI options not provided // Ensure strings like 'all' are wrapped in arrays for consistency if (!normalized.modules) { normalized.modules = Array.isArray(profile.modules) ? profile.modules : [profile.modules];