Skip to content

Commit ed39f50

Browse files
skwowetclaude
andcommitted
fix: correct bumpFactor divisor to match number of checks (CM-1137)
There are 6 metadata checks in decideMemberSimilarityUsingAdditionalChecks but the divisor was 5 — each bump was slightly too large and the sixth check was mathematically unreachable before hitting the Math.min(1) cap. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
1 parent 474d646 commit ed39f50

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

services/apps/merge_suggestions_worker/src/memberSimilarityCalculator.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -472,7 +472,7 @@ class MemberSimilarityCalculator {
472472
? startingScore
473473
: this.HIGH_CONFIDENCE_SCORE
474474

475-
const bumpFactor = (1 - confidenceScore) / 5
475+
const bumpFactor = (1 - confidenceScore) / 6
476476

477477
if (this.hasSameLocation(member, similarMember)) {
478478
isHighConfidence = true

0 commit comments

Comments
 (0)