ldy26098
/
BMAD-METHOD
mirror of https://github.com/bmad-code-org/BMAD-METHOD.git
1
0
Fork 0
Code Packages Projects Releases Wiki Activity

fix: remove useless ternary operators in options-parser 

The ternary operators were checking `Array.isArray()` but returning the same value in both branches, making them completely pointless. Since profiles can contain both arrays (e.g., `['dev', 'architect']`) and strings (e.g., `'all'`), and both are valid, we should just assign the value directly.

Fixed lines:
- normalized.modules = profile.modules
- normalized.agents = profile.agents
- normalized.workflows = profile.workflows

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

Co-Authored-By: Claude <noreply@anthropic.com>
This commit is contained in:
Nikita Levyankov 2025-12-18 12:32:04 +02:00
parent 1dca5fb4b7
commit b88845c0fe
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
tools/cli/installers/lib/core/options-parser.js
View File

@ -128,13 +128,13 @@ function parseOptions(cliOptions) {
// If no explicit modules/agents/workflows, use profile values // If no explicit modules/agents/workflows, use profile values
if (!normalized.modules) { if (!normalized.modules) {
normalized.modules = Array.isArray(profile.modules) ? profile.modules : profile.modules; normalized.modules = profile.modules;
} }
if (!normalized.agents) { if (!normalized.agents) {
normalized.agents = Array.isArray(profile.agents) ? profile.agents : profile.agents; normalized.agents = profile.agents;
} }
if (!normalized.workflows) { if (!normalized.workflows) {
normalized.workflows = Array.isArray(profile.workflows) ? profile.workflows : profile.workflows; normalized.workflows = profile.workflows;
} }
} }