Freegle / Iznik / 21292

71%

push

circleci

fix(namegen): enforce GenerateName min-length contract (fix flaky test)

fillWord() breaks out of its loop when the trigram chain dead-ends, so
GenerateName could return a name shorter than its own minLen=4 — down to the
2-char start bigram. That intermittently failed TestGenerateNameReasonableLength
("2" is not >= "4") on master (a rare, seed-dependent flake). Guard the result:
fall back to "A freegler" (which the test already skips) when it's too short, so
the function honours the [minLen,maxLen] contract it documents.

Verified: utils namegen tests pass across 50×20 = 1000 GenerateName calls.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

10826 of 14417 branches covered (75.09%)

Branch coverage included in aggregate %.

3 of 5 new or added lines in 1 file covered. (60.0%)

117789 of 166219 relevant lines covered (70.86%)

35.82 hits per line