clay / claycli / 35010494890

85%

master: 83%

Pull #262

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. Master is broken for `npm test` right now as a result.

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.

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

771 of 996 branches covered (77.41%)

Branch coverage included in aggregate %.

7 of 7 new or added lines in 1 file covered. (100.0%)

1670 of 1889 relevant lines covered (88.41%)

10.91 hits per line