refactor: remove dead code (profileModules/Agents/Workflows)
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 <noreply@anthropic.com>
This commit is contained in:
parent
3a663a9a92
commit
251ec93150
|
|
@ -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];
|
||||
|
|
|
|||
Loading…
Reference in New Issue