fix: correct auto-format and phone validation edge cases

Resolves review findings on the Text field format handling:

- stripSeparator treated a whitespace separator as absent, so a space
  separator was rendered but never stripped, leaking the grouping
  characters into the stored value and payload. Only a genuinely empty
  separator now short-circuits.
- parseGroupLengths relied on parseInt alone, silently accepting
  partial values such as "3foo" and "3.5" as 3. Tokens are now
  validated as whole positive integers before parsing.
- The phone pattern matched digit-free input like "-------" or seven
  spaces. A lookahead now requires at least one digit, leaving the
  allowed characters, length range, and optional leading "+" unchanged.
- The auto-format group row resolves enableFormatting through
  getSettingValue, so the setting works in JS mode as well as literal.

Author: Lihlu

shesha-reactjs/src/designer-components/textField/settingsForm.ts

@@ -51,7 +51,7 @@ export const getSettings: SettingsFormMarkupFactory = ({ fbf, removeStyleRouter
                       { type: 'textField', propertyName: 'formatGroups', label: 'Group lengths', size: 'small', jsSetting: true, tooltip: 'Comma-separated lengths of each group, e.g. 3,4' },
                       { type: 'textField', propertyName: 'formatSeparator', label: 'Separator', size: 'small', jsSetting: true, tooltip: 'Character(s) displayed between groups, e.g. -. Visual only, not included in the stored value.' },
                     ],
-                    visibleJs: 'return data?.enableFormatting === true;',
+                    visibleJs: 'return getSettingValue(data?.enableFormatting) === true;',
                   }),
                 undefined, 'return getSettingValue(data?.textType) === "text";')
                 .stdCollapsiblePanel('Validations', (fb) => fb

shesha-reactjs/src/designer-components/textField/utils.ts

@@ -1,7 +1,7 @@
 import { IStyleValue } from "@/providers/form/models";
 import { useSettingValue } from '@/providers/settings';
 import { ISettingIdentifier } from '@/providers/settings/models';
-import { isNullOrWhiteSpace } from '@/utils/nullables';
+import { isDefined, isNullOrWhiteSpace } from '@/utils/nullables';
 import { useMemo } from 'react';
 import { TextType } from './interfaces';
 
@@ -28,7 +28,8 @@ export const TEXT_TYPE_FORMATS: Partial<Record<TextType, ITextTypeFormatConfig>>
   },
   phone: {
     // Lenient: leading "+", digits, spaces, dashes, dots and parentheses, 7-20 chars.
-    pattern: /^\+?[0-9\s\-().]{7,20}$/,
+    // The lookahead requires at least one digit, so separators alone (e.g. "-------") are rejected.
+    pattern: /^\+?(?=.*[0-9])[0-9\s\-().]{7,20}$/,
     message: 'Please enter a valid phone number',
     inputType: 'tel',
     autoComplete: 'tel',
@@ -50,13 +51,15 @@ export const parseGroupLengths = (groups: string | undefined): number[] => {
   if (isNullOrWhiteSpace(groups)) return [];
   return groups
     .split(',')
-    .map((part) => parseInt(part.trim(), 10))
+    .map((part) => part.trim())
+    .filter((part) => /^\d+$/.test(part))
+    .map((part) => parseInt(part, 10))
     .filter((length) => Number.isFinite(length) && length > 0);
 };
 
 /** Removes every occurrence of `separator` from `value`, leaving only the raw characters. */
 export const stripSeparator = (value: string, separator: string): string => {
-  if (isNullOrWhiteSpace(value) || isNullOrWhiteSpace(separator)) return value;
+  if (isNullOrWhiteSpace(value) || !isDefined(separator) || separator === '') return value;
   const escapedSeparator = separator.replace(/[.*+?^${}()|[\]\\]/g, '\\$&');
   return value.split(new RegExp(escapedSeparator, 'g')).join('');
 };