Reduce logging for SQL exceptions from queries with calculated columns (#1840)

### version 6.58.5
*Released*: 14 August 2025
- Issue 52026 and 51862: Reduce the logging for calculated expression column SQL errors
  - add comment tag to executeSql query to indicate it is from a calculated column validation
  - use string value compatible with CAST to NUMERIC for parseCalculatedColumn() temp data

Author: cnathe

packages/components/package-lock.json

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

packages/components/package.json

@@ -1,6 +1,12 @@
 # @labkey/components
 Components, models, actions, and utility functions for LabKey applications and pages
 
+### version 6.58.5
+*Released*: 14 August 2025
+- Issue 52026 and 51862: Reduce the logging for calculated expression column SQL errors
+  - add comment tag to executeSql query to indicate it is from a calculated column validation
+  - use string value compatible with CAST to NUMERIC for parseCalculatedColumn() temp data
+
 ### version 6.58.4
 *Released*: 12 August 2025
 - Adding new parent alias on designer doesn't always add to the bottom of the list

packages/components/releaseNotes/components.md

@@ -867,17 +867,15 @@ describe('domain properties actions', () => {
         expect(getCastStatement('key', 'VISITDATE')).toBe('CAST(CURDATE() AS TIMESTAMP) AS "key"');
         expect(getCastStatement('key', 'DATE')).toBe('CAST(CURDATE() AS DATE) AS "key"');
         expect(getCastStatement('key', 'TIME')).toBe('CAST(\'13:00\' AS TIME) AS "key"');
-        expect(getCastStatement('key', 'TEXT')).toBe('CAST(\'Testing\' AS VARCHAR) AS "key"');
-        expect(getCastStatement('key', 'OTHER')).toBe('CAST(\'Testing\' AS VARCHAR) AS "key"');
+        expect(getCastStatement('key', 'TEXT')).toBe('CAST(\'1\' AS VARCHAR) AS "key"');
+        expect(getCastStatement('key', 'OTHER')).toBe('CAST(\'1\' AS VARCHAR) AS "key"');
     });
 
     // Issue 52608
     test('getCastStatement for special char field', () => {
-        expect(getCastStatement('a"key" with quotes', 'TEXT')).toBe(
-            'CAST(\'Testing\' AS VARCHAR) AS "a""key"" with quotes"'
-        );
+        expect(getCastStatement('a"key" with quotes', 'TEXT')).toBe('CAST(\'1\' AS VARCHAR) AS "a""key"" with quotes"');
         expect(getCastStatement('a,./key with!@#$specCHARS', 'TEXT')).toBe(
-            'CAST(\'Testing\' AS VARCHAR) AS "a,./key with!@#$specCHARS"'
+            'CAST(\'1\' AS VARCHAR) AS "a,./key with!@#$specCHARS"'
         );
     });
 

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

@@ -49,6 +49,7 @@ import { IImportAlias } from '../entities/models';
 import { DATA_CLASS_IMPORT_PREFIX, SAMPLE_SET_IMPORT_PREFIX } from '../entities/constants';
 
 import {
+    CALCULATED_COLUMN_SQL_TAG,
     DOMAIN_ERROR_ID,
     DOMAIN_FIELD_CLIENT_SIDE_ERROR,
     DOMAIN_FIELD_LOOKUP_CONTAINER,
@@ -442,7 +443,7 @@ export function getCastStatement(key: string, type: string): string {
         case 'TIME':
             return `CAST('13:00' AS TIME) AS "${quotedKey}"`;
         default:
-            return `CAST('Testing' AS VARCHAR) AS "${quotedKey}"`;
+            return `CAST('1' AS VARCHAR) AS "${quotedKey}"`;
     }
 }
 
@@ -465,7 +466,8 @@ export async function parseCalculatedColumn(
                 .map(key => getCastStatement(key, columnMap[key]))
                 .join(',\n') +
             ') AS x' +
-            ' WHERE 1=0';
+            ' WHERE 1=0' +
+            CALCULATED_COLUMN_SQL_TAG; // Issue 52026 and 51862
 
         await executeSql({
             schemaName: 'core',

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

@@ -230,3 +230,5 @@ export const MAX_VALID_TEXT_CHOICES = 200;
 export const LOOKUP_VALIDATOR_VALUES = { type: 'Lookup', name: 'Lookup Validator' };
 
 export const DOMAIN_ERROR_ID = 'domain-error';
+
+export const CALCULATED_COLUMN_SQL_TAG = '/* CALCULATED-EXPRESSION-COLUMN-QUERY */';