clay / claycli / 35010408035

83%

master: 83%

Pull #260

github

🍕 Fix normalizeRequestedSteps complexity regression from #259

`npx eslint lib cli index.js` -- the full lint command `npm test` actually
runs -- currently fails on master: normalizeRequestedSteps has a cyclomatic
complexity of 10 (max 8), introduced in #259 and missed there because that
PR's own verification only linted the specific files it touched, not the
full repo command.

Splits the function into splitOnlyEntries() (CSV/repeated-flag parsing) and
classifyOnlyEntries() (recognized vs. unrecognized step names), leaving
normalizeRequestedSteps as a thin orchestrator (complexity 7). No behavior
change -- reverified the exact case #259 added: an unrecognized --only value
still throws with the same message.

Discovered while rebasing this branch onto master post-#259; fixing it here
so this branch's own CI is green. Also opening a standalone fix directly
against master, since master itself fails `npm test` right now independent
of this branch.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>

784 of 1040 branches covered (75.38%)

Branch coverage included in aggregate %.

23 of 53 new or added lines in 2 files covered. (43.4%)

1685 of 1934 relevant lines covered (87.13%)

10.75 hits per line