Fixed local cluster bug and added datasource version compatibility check before importing workflows

Signed-off-by: saimedhi <saimedhi@amazon.com>

Author: saimedhi

public/pages/workflows/import_workflow/import_workflow_modal.tsx

@@ -44,7 +44,7 @@ import {
   WORKFLOW_NAME_RESTRICTIONS,
 } from '../../../../common';
 import { WORKFLOWS_TAB } from '../workflows';
-import { getDataSourceId } from '../../../utils/utils';
+import { getDataSourceId, getEffectiveVersion, formatDisplayVersion } from '../../../utils/utils';
 
 interface ImportWorkflowModalProps {
   isImportModalOpen: boolean;
@@ -62,6 +62,17 @@ interface ImportWorkflowModalProps {
 export function ImportWorkflowModal(props: ImportWorkflowModalProps) {
   const dispatch = useAppDispatch();
   const dataSourceId = getDataSourceId();
+  const [dataSourceVersion, setDataSourceVersion] = useState<
+    string | undefined
+  >(undefined);
+  useEffect(() => {
+    async function getVersion() {
+      if (dataSourceId !== undefined) {
+        setDataSourceVersion(await getEffectiveVersion(dataSourceId));
+      }
+    }
+    getVersion();
+  }, [dataSourceId]);
   const { workflows } = useSelector((state: AppState) => state.workflows);
 
   // workflow name state
@@ -87,10 +98,8 @@ export function ImportWorkflowModal(props: ImportWorkflowModalProps) {
     return description.length > MAX_DESCRIPTION_LENGTH;
   }
 
-  // States for tracking workflow template compatibility with current data source version
+  // State for tracking workflow template compatibility with current data source version
   const [isCompatible, setIsCompatible] = useState<boolean>(true);
-  const [inCompatibleDataSourceVersion, setInCompatibleDataSourceVersion] = useState<string | undefined>();
-
 
   // transient importing state for button state
   const [isImporting, setIsImporting] = useState<boolean>(false);
@@ -124,11 +133,8 @@ export function ImportWorkflowModal(props: ImportWorkflowModalProps) {
   useEffect(() => {
     async function checkCompatibility() {
       if (isValidWorkflow(fileObj)) {
-        const [isCompatible, dataSourceVersion] = await isCompatibleWorkflow(fileObj, dataSourceId);
+        const isCompatible = await isCompatibleWorkflow(fileObj, dataSourceId);
         setIsCompatible(isCompatible);
-        if (dataSourceVersion) {
-          setInCompatibleDataSourceVersion(dataSourceVersion);
-        }
       }
     }
     checkCompatibility();
@@ -161,11 +167,11 @@ export function ImportWorkflowModal(props: ImportWorkflowModalProps) {
               <EuiSpacer size="m" />
             </>
           )}
-          {isValidWorkflow(fileObj) && !isCompatible && inCompatibleDataSourceVersion && (
+          {isValidWorkflow(fileObj) && !isCompatible && dataSourceVersion && (
             <>
               <EuiFlexItem>
                 <EuiCallOut
-                  title={`The uploaded file is not compatible with the current data source version ${inCompatibleDataSourceVersion}. Upload a compatible file or switch to another data source.`}
+                  title={`The uploaded file is not compatible with the current data source version ${formatDisplayVersion(dataSourceVersion)}. Upload a compatible file or switch to another data source.`}
                   iconType={'alert'}
                   color="danger"
                 />

public/utils/utils.tsx

@@ -157,27 +157,26 @@ export function isValidWorkflow(workflowObj: any): boolean {
   return workflowObj?.name !== undefined;
 }
 
+// Determines if a file used for import workflow is compatible with the current data source version.
 export async function isCompatibleWorkflow(
   workflowObj: any, 
   dataSourceId?: string | undefined
-): Promise<[boolean, string | undefined]> {
+): Promise<boolean> {
   const compatibility = workflowObj?.version?.compatibility;
 
   // Default to true when compatibility cannot be assessed (empty/invalid compatibility array or MDS disabled.)
   if (!Array.isArray(compatibility) || compatibility.length === 0 || dataSourceId === undefined) {
-    return [true, undefined];
+    return true;
   }
 
   const dataSourceVersion = await getEffectiveVersion(dataSourceId);  
   const [effectiveMajorVersion, effectiveMinorVersion] = dataSourceVersion.split('.').map(Number);
 
   // Checks if any version in compatibility array matches the current dataSourceVersion (major.minor)
-  const isCompatible = compatibility.some(compatibleVersion => {
+  return compatibility.some(compatibleVersion => {
     const [compatibleMajor, compatibleMinor] = compatibleVersion.split('.').map(Number);
     return effectiveMajorVersion === compatibleMajor && effectiveMinorVersion === compatibleMinor;
   });
-
-  return [isCompatible, `${effectiveMajorVersion}.${effectiveMinorVersion}`];
 }
 
 
@@ -936,3 +935,15 @@ export const getEffectiveVersion = async (
     return MIN_SUPPORTED_VERSION;
   }
 };
+
+    
+/**
+ * Formats version string to show only major.minor numbers
+ * Example: "3.0.0-alpha1" -> "3.0"
+ */
+export function formatDisplayVersion(version: string): string {
+  // Take first two parts of version number (major.minor)
+  const [major, minor] = version.split('.');
+  return `${major}.${minor}`;
+}
+