fix: Update getInitialsFromName to sanitize name

modules/preview-react/avatar/lib/getInitialsFromName.ts

@@ -1,11 +1,24 @@
+/**
+  Extracts initials from a person's name by taking the first letter of the first and last name parts. It also sanitizes input to filter out common name prefixes and suffixes.
+*/
 export const getInitialsFromName = (name: string) => {
-  // Trim and split by one or more whitespace characters
-  const nameParts = name.trim().split(/\s+/).filter(Boolean);
+  // Remove parenthetical content (e.g., titles, departments) and trim
+  const sanitized = name.replace(/\([^)]*\)/g, '').trim();
 
-  const first = nameParts[0];
+  // Split by one or more whitespace characters
+  const nameParts = sanitized.split(/\s+/).filter(Boolean);
+
+  // Filter out parts that are purely numeric or Roman numerals (suffixes like III, IV)
+  const filteredParts = nameParts.filter(part => {
+    const isNumeric = /^\d+$/.test(part);
+    const isRomanNumeral = /^[IVX]+$/i.test(part);
+    return !isNumeric && !isRomanNumeral;
+  });
+
+  const first = filteredParts[0];
   const firstInitial = first?.[0] || '';
-  const last = nameParts.length > 1 ? nameParts[nameParts.length - 1] : undefined;
+  const last = filteredParts.length > 1 ? filteredParts[filteredParts.length - 1] : undefined;
   const lastInitial = last?.[0] || '';
 
-  return `${firstInitial}${lastInitial}`;
+  return `${firstInitial}${lastInitial}`.toUpperCase();
 };

modules/preview-react/avatar/spec/getInitialsFromName.spec.tsx

@@ -0,0 +1,55 @@
+import {getInitialsFromName} from '../lib/getInitialsFromName';
+
+describe('getInitialsFromName', () => {
+  it('should return initials from a simple first and last name', () => {
+    expect(getInitialsFromName('Logan McNeil')).toBe('LM');
+  });
+
+  it('should return initials from a single name', () => {
+    expect(getInitialsFromName('Logan')).toBe('L');
+  });
+
+  it('should remove parenthetical content', () => {
+    expect(getInitialsFromName('Logan McNeil (SFV, VP Director)')).toBe('LM');
+  });
+
+  it('should filter out Roman numeral suffixes', () => {
+    expect(getInitialsFromName('Logan McNeil III')).toBe('LM');
+    expect(getInitialsFromName('John Smith IV')).toBe('JS');
+    expect(getInitialsFromName('Mary Johnson II')).toBe('MJ');
+  });
+
+  it('should handle multiple middle names', () => {
+    expect(getInitialsFromName('John Paul Jones')).toBe('JJ');
+  });
+
+  it('should handle names with both parenthetical content and suffixes', () => {
+    expect(getInitialsFromName('Logan McNeil III (VP Director)')).toBe('LM');
+  });
+
+  it('should handle extra whitespace', () => {
+    expect(getInitialsFromName('  Logan   McNeil  ')).toBe('LM');
+  });
+
+  it('should return empty string for empty input', () => {
+    expect(getInitialsFromName('')).toBe('');
+  });
+
+  it('should handle names with only parenthetical content', () => {
+    expect(getInitialsFromName('(VP Director)')).toBe('');
+  });
+
+  it('should handle names with numeric suffixes', () => {
+    expect(getInitialsFromName('Logan McNeil 3')).toBe('LM');
+  });
+
+  it('should work with names from various cultures', () => {
+    expect(getInitialsFromName('José García')).toBe('JG');
+    expect(getInitialsFromName('Mohammed Al-Rashid')).toBe('MA');
+  });
+
+  it('should handle mixed case Roman numerals', () => {
+    expect(getInitialsFromName('Logan McNeil iii')).toBe('LM');
+    expect(getInitialsFromName('Logan McNeil Iii')).toBe('LM');
+  });
+});