makeCommaSeparatedString: add lastSeparator and postfixargs (#1830)

- makeCommaSeparatedString
    - Add lastSeparator and postfix args
    - export

Author: labkey-alan

packages/components/package-lock.json

@@ -1,6 +1,6 @@
 {
   "name": "@labkey/components",
-  "version": "6.54.3",
+  "version": "6.55.0",
   "description": "Components, models, actions, and utility functions for LabKey applications and pages",
   "sideEffects": false,
   "files": [

packages/components/package.json

@@ -1,6 +1,10 @@
 # @labkey/components
 Components, models, actions, and utility functions for LabKey applications and pages
 
+### version 6.55.0
+*Released*: 10 July 2025
+- makeCommaSeparatedString: Add lastSeparator and postfix args, export
+
 ### version 6.54.3
 *Released*: 10 July 2025
 - Issue 52657: LKSM: We shouldn't allow creating sample names that differ only in case

packages/components/releaseNotes/components.md

@@ -67,6 +67,7 @@ import {
     isNonNegativeFloat,
     isNonNegativeInteger,
     isSetEqual,
+    makeCommaSeparatedString,
     parseCsvString,
     parseScientificInt,
     quoteValueWithDelimiters,
@@ -1557,6 +1558,7 @@ export {
     LockIcon,
     LOOK_AND_FEEL_METRIC,
     LookupSelectInput,
+    makeCommaSeparatedString,
     makeQueryInfo,
     makeTestActions,
     makeTestISelectRowsResult,

packages/components/src/index.ts

@@ -1416,6 +1416,24 @@ describe('makeCommaSeparatedString', () => {
         expect(makeCommaSeparatedString([123, 456])).toBe('123 and 456');
         expect(makeCommaSeparatedString([123, 'blood', true])).toBe('123, blood and true');
     });
+
+    test('lastSeparator', () => {
+        expect(makeCommaSeparatedString(['blood'], ', or ')).toBe('blood');
+        expect(makeCommaSeparatedString(['blood', 'saliva', 'dna'], ', or ')).toEqual('blood, saliva, or dna');
+        expect(makeCommaSeparatedString([1, 2, 3], ', and ')).toEqual('1, 2, and 3');
+    });
+
+    test('postfix', () => {
+        expect(makeCommaSeparatedString(['blood'], undefined, '.')).toBe('blood.');
+        expect(makeCommaSeparatedString(['blood', 'saliva', 'dna'], undefined, '.')).toEqual('blood, saliva and dna.');
+        expect(makeCommaSeparatedString([1, 2, 3], undefined, '.')).toEqual('1, 2 and 3.');
+    });
+
+    test('lastSeparator & postfix', () => {
+        expect(makeCommaSeparatedString(['blood'], ', or ', '.')).toBe('blood.');
+        expect(makeCommaSeparatedString(['blood', 'saliva', 'dna'], ', or ', '.')).toEqual('blood, saliva, or dna.');
+        expect(makeCommaSeparatedString([1, 2, 3], ', and ', '.')).toEqual('1, 2, and 3.');
+    });
 });
 
 describe('getValuesSummary', () => {

packages/components/src/internal/util/utils.test.ts

@@ -694,13 +694,13 @@ export function getValueFromRow(row: Record<string, any>, col: string): string |
     return val;
 }
 
-export function makeCommaSeparatedString<T>(values: T[]): string {
+export function makeCommaSeparatedString<T>(values: T[], lastSeparator = ' and ', postfix = ''): string {
     if (!values || values.length === 0) return '';
-    if (values.length === 1) return values[0] + '';
+    if (values.length === 1) return values[0] + postfix;
 
     const firsts = values.slice(0, values.length - 1);
     const last = values[values.length - 1];
-    return firsts.join(', ') + ' and ' + last;
+    return firsts.join(', ') + lastSeparator + last + postfix;
 }
 
 /**