GitHub Issue 954: Add error for duplicate values for parent inputs

packages/components/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@labkey/components",
-  "version": "7.29.1",
+  "version": "7.29.2-fb-duplicateParents.0",
   "description": "Components, models, actions, and utility functions for LabKey applications and pages",
   "sideEffects": false,
   "files": [

packages/components/releaseNotes/components.md

@@ -9,6 +9,10 @@ Components, models, actions, and utility functions for LabKey applications and p
 *Released*: 7 April 2026
 - Update `FREEZER_ITEM_SAMPLE_MAPPER` to return undefined if not matched, for consistency with other mappers
 
+### version 7.X
+*Released*: X April 2026
+- GitHub Issue 954: Add error for duplicate values for parent inputs
+
 ### version 7.28.1
 *Released*: 3 April 2026
 - GitHub Issue 613: Use "Status" instead of "SampleState" in error messaging.

packages/components/src/internal/components/editable/actions.test.ts

@@ -836,6 +836,48 @@ describe('parsePastedLookup', () => {
             ]),
         });
     });
+
+    test('duplicate values', () => {
+        // single duplicate
+        expect(parsePastedLookup(multiValueLookup, stringLookupValues, 'A,A')).toStrictEqual({
+            message: { message: 'Duplicate values not allowed: "A".' },
+            valueDescriptors: List([
+                { display: 'A', raw: 'a' },
+                { display: 'A', raw: 'a' },
+            ]),
+        });
+
+        // multiple duplicates
+        expect(parsePastedLookup(multiValueLookup, stringLookupValues, 'A,b,A,b')).toStrictEqual({
+            message: { message: 'Duplicate values not allowed: "A", "b".' },
+            valueDescriptors: List([
+                { display: 'A', raw: 'a' },
+                { display: 'b', raw: 'B' },
+                { display: 'A', raw: 'a' },
+                { display: 'b', raw: 'B' },
+            ]),
+        });
+
+        // duplicate with other valid values
+        expect(parsePastedLookup(multiValueLookup, stringLookupValues, 'A,C,A')).toStrictEqual({
+            message: { message: 'Duplicate values not allowed: "A".' },
+            valueDescriptors: List([
+                { display: 'A', raw: 'a' },
+                { display: 'C', raw: 'C' },
+                { display: 'A', raw: 'a' },
+            ]),
+        });
+
+        // unmatched takes precedence over duplicates
+        expect(parsePastedLookup(multiValueLookup, stringLookupValues, 'A,A,notfound')).toStrictEqual({
+            message: { message: 'Could not find "notfound"' },
+            valueDescriptors: List([
+                { display: 'A', raw: 'a' },
+                { display: 'A', raw: 'a' },
+                { display: 'notfound', raw: 'notfound' },
+            ]),
+        });
+    });
 });
 
 describe('insertPastedData', () => {

packages/components/src/internal/components/editable/actions.ts

@@ -1118,6 +1118,7 @@ export function parsePastedLookup(
     let message: CellMessage;
     let values: ValueDescriptor[];
     const unmatched: string[] = [];
+    const dupValues = new Set<string>();
 
     // Parse pasted strings to split properly around quoted values.
     // Remove the quotes for storing the actual values in the grid.
@@ -1129,10 +1130,15 @@ export function parsePastedLookup(
         unmatched.push(vt);
         values = [{ display: vt, raw: vt }];
     } else {
+        const foundValues = new Set<string>();
         values = parsedValues.flatMap(v => {
             const vt = v.trim();
             if (!vt) return [];
 
+            if (foundValues.has(vt)) {
+                dupValues.add(vt);
+            }
+            foundValues.add(vt);
             const vl = vt.toLowerCase();
             const vd = descriptors.find(d => d.display && d.display.toString().toLowerCase() === vl);
             if (vd) return [vd];
@@ -1148,6 +1154,12 @@ export function parsePastedLookup(
             .map(u => '"' + u + '"')
             .join(', ');
         message = { message: lookupValidationErrorMessage(valueStr, true) };
+    } else if (dupValues.size > 0) {
+        const valueStr = Array.from(dupValues)
+            .slice(0, 4)
+            .map(u => '"' + u + '"')
+            .join(', ');
+        message = { message: `Duplicate values not allowed: ${valueStr}.` };
     }
 
     return { message, valueDescriptors: List(values) };