Skip to content

fix: reduce false-positive merge suggestions from single-word display names (CM-1137)#4123

Merged
skwowet merged 1 commit into
mainfrom
hotfix-improve-CM-1137
May 15, 2026
Merged

fix: reduce false-positive merge suggestions from single-word display names (CM-1137)#4123
skwowet merged 1 commit into
mainfrom
hotfix-improve-CM-1137

Conversation

@skwowet
Copy link
Copy Markdown
Collaborator

@skwowet skwowet commented May 15, 2026

Summary

Follow-up to #4115.

The first production run surfaced:

  • ~21k single-word first-name collisions (Alex, David, Scott, etc.) scoring 0.9 with no corroborating signal.
  • ~259 placeholder-name pairs ([[Unknown]] [Not Provided], Deleted User) incorrectly treated as valid merge candidates.

Changes

  • Single-word display names now require at least one corroborating signal (location, org, timezone, or local-part).
  • Only full names (i.e. names containing a space) are treated as high-confidence on their own.
  • Added [[Unknown]] [Not Provided] and Deleted User to the placeholder-name blocklist.

Note

Medium Risk
Adjusts merge-similarity scoring logic used to propose member merges; mistakes could reduce recall or still allow some false positives, but changes are localized to heuristics.

Overview
Reduces false-positive merge suggestions when members share the same displayName.

Exact displayName matches now return low confidence for OS/placeholder names via isOsReservedName, and treat single-word names as requiring corroborating signals by starting decideMemberSimilarityUsingAdditionalChecks at HIGH_CONFIDENCE_SCORE (only multi-word “full names” remain high-confidence on display name alone).

Expands the OS_RESERVED_NAMES blocklist in utils.ts to include [[Unknown]] [Not Provided] and Deleted User.

Reviewed by Cursor Bugbot for commit 4a87858. Bugbot is set up for automated code reviews on this repo. Configure here.

…holder names (CM-1137)

Signed-off-by: Yeganathan S <63534555+skwowet@users.noreply.github.com>
@skwowet skwowet self-assigned this May 15, 2026
Copilot AI review requested due to automatic review settings May 15, 2026 17:37
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR tightens member merge similarity scoring to reduce false positives from ambiguous single-word display names and known placeholder names.

Changes:

  • Adds [[Unknown]] [Not Provided] and Deleted User to the non-identity name blocklist.
  • Changes exact display-name scoring so single-word names require corroborating signals, while multi-word names remain high-confidence.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
services/apps/merge_suggestions_worker/src/utils.ts Expands reserved/non-identity display name handling.
services/apps/merge_suggestions_worker/src/memberSimilarityCalculator.ts Adjusts exact display-name scoring for single-word vs full-name cases.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@skwowet skwowet merged commit cd27dc9 into main May 15, 2026
22 checks passed
@skwowet skwowet deleted the hotfix-improve-CM-1137 branch May 15, 2026 18:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants