fix: assessment flow

Author: snehaljha-sf

messages/assess.json

@@ -123,15 +123,15 @@
   "errorEvaluatingExpression": "Error evaluating expression: %s, %s",
   "foundFlexiPages": "Found %s FlexiPages to assess",
   "assessingFlexiPages": "Assessing FlexiPages",
-  "completedProcessingAllFlexiPages": "Completed processing all flexipage files. Total processed: %s",
-  "completedProcessingFlexiPage": "Completed processing %s - Errors: %s",
-  "processingFlexiPage": "Processing flexipage file: %s",
-  "startingFlexiPageProcessing": "Starting to process flexipage: %s",
+  "completedProcessingAllFlexiPages": "Completed processing all FlexiPages. Total processed: %s",
+  "completedProcessingFlexiPage": "Completed processing %s. Errors: %s",
+  "processingFlexiPage": "Processing FlexiPage: %s",
+  "startingFlexiPageProcessing": "Starting to process FlexiPage: %s",
   "readFlexiPageContent": "Read file content, size: %s characters",
-  "updatedModifiedContent": "updated content in file: %s",
-  "generatedDiffForFile": "Generated diff for file: %s",
-  "errorProcessingFlexiPage": "Error processing flexipage file: %s, %s",
-  "flexipagesWithChanges": "Found %s flexipage files with changes",
+  "updatedModifiedContent": "updated content in %s file",
+  "generatedDiffForFile": "Generated difference for the file: %s",
+  "errorProcessingFlexiPage": "Error processing FlexiPage: %s, %s",
+  "flexipagesWithChanges": "Found %s FlexiPages with changes",
   "migratingFlexiPages": "Migrating FlexiPages",
   "experienceSiteMetadataConsent": "The consent for exp site is %s",
   "experienceSiteConsentNotProvidedWarning": "Consent for experience sites is not provided. Experience sites will not be processed",

messages/migrate.json

@@ -117,15 +117,15 @@
   "migrationConsentNotGiven": "Couldn't confirm whether assessment errors are resolved",
   "foundFlexiPages": "Found %s FlexiPages to migrate",
   "migratingFlexiPages": "Migrating FlexiPages",
-  "completedProcessingAllFlexiPages": "Completed processing all flexipage files. Total processed: %s",
-  "completedProcessingFlexiPage": "Completed processing %s - Errors: %s",
-  "processingFlexiPage": "Processing flexipage file: %s",
-  "startingFlexiPageProcessing": "Starting to process flexipage: %s",
+  "completedProcessingAllFlexiPages": "Completed processing all flexipages. Total processed: %s",
+  "completedProcessingFlexiPage": "Completed processing %s. Errors: %s",
+  "processingFlexiPage": "Processing FlexiPage: %s",
+  "startingFlexiPageProcessing": "Starting to process FlexiPage: %s",
   "readFlexiPageContent": "Read file content, size: %s characters",
-  "updatedModifiedContent": "updated content in file: %s",
-  "generatedDiffForFile": "Generated diff for file: %s",
-  "errorProcessingFlexiPage": "Error processing flexipage file: %s, %s",
-  "flexipagesWithChanges": "Found %s flexipage files with changes",
+  "updatedModifiedContent": "updated content in %s file",
+  "generatedDiffForFile": "Generated difference for the file: %s",
+  "errorProcessingFlexiPage": "Error processing FlexiPage: %s, %s",
+  "flexipagesWithChanges": "Found %s FlexiPages with changes",
   "omniGlobalAutoNumberPrefEnabled": "Successfully enabled OmniGlobalAutoNumberPref org preference",
   "errorEnablingOmniGlobalAutoNumberPref": "We couldn’t enable the Global Auto Number setting. Turn on the Global Auto Number setting manually, and try again. See <a href='https://help.salesforce.com/s/articleView?id=xcloud.os_standard_global_auto_number.htm&type=5' target='_blank'>Set Up Omni Global Auto Number</a>.",
   "performingPreMigrationChecks": "Performing pre-migration checks for Global Auto Number...",
@@ -188,16 +188,16 @@
   "errorCheckingGlobalAutoNumber": "We couldn’t check whether the Global Auto Number setting is enabled: %s. Try again later.",
   "errorMigrationMessage": "Error migrating object: %s",
   "nameMappingUndefined": "Name Mapping is undefined",
-  "autoDeployConsentMessage": "Do you want to auto deploy the related objects? [y/n]",
-  "errorDeployingComponents": "Error deploying related objects post migration",
+  "autoDeployConsentMessage": "Do you want to deploy the related objects automatically? [y/n]",
+  "errorDeployingComponents": "We couldn’t deploy related objects after migration.",
   "installingDependency": "Installing node dependency %s",
   "dependencyInstalled": "Node dependency %s installed",
   "deployingFromManifest": "Deploying from metadata packages",
-  "manifestDeployFailed": "Metadata packages deployment failed, check logs or status for more details",
+  "manifestDeployFailed": "We couldn’t deploy metadata packages. Check the logs or the status for more details.",
   "installingRequiredDependencies": "Installing required node dependencies",
-  "creatingNPMConfigFile": "Creating npm config file",
+  "creatingNPMConfigFile": "Creating npm configuration file",
   "npmConfigFileCreated": "Npm config file created",
-  "authKeyEnvVarNotSet": "OMA_AUTH_KEY environment variable is not set, LWCs will not be deployed",
+  "authKeyEnvVarNotSet": "The OMA_AUTH_KEY environment variable isn’t set. LWCs won't be deployed.",
   "experienceSiteException": "Exception occurred while processing experience sites",
   "reservedKeysFoundInPropertySet": "Reserved keys found in any of output response transformation fields: %s.",
   "incompleteMigrationDetected": "We couldn't complete the migration process",
@@ -210,7 +210,7 @@
   "skipFlexcardAngularOmniScriptDependencyWarning": "Skipping FlexCard %s due to Angular OmniScript dependencies",
   "flexCardMigrationProcessingMessage": "Processing %s FlexCards for migration (%s skipped due to Angular dependencies)",
   "noMetadataToDeploy": "No metadata to deploy",
-  "manifestDeployementStarted": "Manifest deployment started with id: %s",
+  "manifestDeployementStarted": "Manifest deployment started with ID: %s",
   "migrationConsentMessage": "Make sure that all records listed in the Assessment Status column in the Assessment Report are marked as Green before proceeding with the migration. Do you want to proceed? Yes or No?",
   "truncationFailed": "Truncation failed for %s: %s",
   "invalidTypeMigrateErrorMessage": "We couldn't migrate your Omnistudio components in the %s namespace. Select the correct namespace and try again",
@@ -231,4 +231,4 @@
   "enableStandardRuntimeAfterDesigner": "Standard designer setup successful, proceeding with Standard Runtime enablement",
   "skipStandardRuntimeDueToFailure": "Skipping Standard Runtime enablement due to designer setup failure",
   "errorCheckingStandardDesigner": "Failed to check standard designer status for package %s: %s"
-}
+}

src/commands/omnistudio/migration/migrate.ts

@@ -646,10 +646,10 @@ export default class Migrate extends OmniStudioBaseCommand {
     const mergedResults: MigratedRecordInfo[] = [];
 
     for (const record of Array.from(migrationResults.records.values())) {
-      const obj = {
+      const obj: MigratedRecordInfo = {
         id: record['Id'],
         name: migrationTool.getRecordName(record),
-        status: messages.getMessage('labelStatusSkipped'),
+        status: 'Skipped',
         errors: record['errors'],
         migratedId: undefined,
         warnings: [],
@@ -663,11 +663,11 @@ export default class Migrate extends OmniStudioBaseCommand {
         errors = errors.concat(recordResults.errors || []);
 
         if (recordResults?.skipped) {
-          obj.status = messages.getMessage('labelStatusSkipped');
+          obj.status = 'Skipped';
         } else if (!recordResults || recordResults.hasErrors) {
-          obj.status = messages.getMessage('labelStatusFailed');
+          obj.status = 'Failed';
         } else {
-          obj.status = messages.getMessage('labelStatusComplete');
+          obj.status = 'Complete';
         }
         obj.errors = errors;
         obj.migratedId = recordResults.id;

src/javascripts/reportGeneratorUtility.js

@@ -102,8 +102,11 @@ function filterAndSearchTable(tableId) {
   const visibleRows = Array.from(table.rows).filter(
     (row) => row.style.display !== 'none' && row.id !== 'no-rows-message'
   );
-  reportTable.querySelector('#row-count').textContent = `Showing ${visibleRows.length} record${
-    visibleRows.length !== 1 ? 's' : ''
+
+  // filter only distinct classes from visibleRows
+  const distinctClasses = [...new Set(visibleRows.map((row) => row.classList[0]))];
+  reportTable.querySelector('#row-count').textContent = `Showing ${distinctClasses.length} record${
+    distinctClasses.length !== 1 ? 's' : ''
   }`;
 }
 
@@ -153,6 +156,21 @@ document.addEventListener('DOMContentLoaded', () => {
   document.querySelectorAll('.rpt-table-container').forEach((tableContainer) => {
     filterAndSearchTable(tableContainer.id);
   });
+
+  document.querySelectorAll('tr').forEach((row) => {
+    row.addEventListener('mouseover', (event) => {
+      const className = event.currentTarget.classList[0];
+      document.querySelectorAll(`.${className}`).forEach((r) => {
+        r.classList.add('highlight');
+      });
+    });
+    row.addEventListener('mouseout', (event) => {
+      const className = event.currentTarget.classList[0];
+      document.querySelectorAll(`.${className}`).forEach((r) => {
+        r.classList.remove('highlight');
+      });
+    });
+  });
 });
 
 function openReport(ele) {

src/migration/dataraptor.ts

@@ -288,7 +288,7 @@ export class DataRaptorMigrationTool extends BaseMigrationTool implements Migrat
     Logger.info(this.messages.getMessage('processingDataRaptor', [drName]));
     const warnings: string[] = [];
     const existingDRNameVal = new StringVal(drName, 'name');
-    let assessmentStatus = 'Ready for migration';
+    let assessmentStatus: 'Ready for migration' | 'Warnings' | 'Needs Manual Intervention' = 'Ready for migration';
 
     if (!existingDRNameVal.isNameCleaned()) {
       warnings.push(
@@ -302,7 +302,7 @@ export class DataRaptorMigrationTool extends BaseMigrationTool implements Migrat
     }
     if (existingDataRaptorNames.has(existingDRNameVal.cleanName())) {
       warnings.push(this.messages.getMessage('duplicatedName') + '  ' + existingDRNameVal.cleanName());
-      assessmentStatus = 'Need Manual Intervention';
+      assessmentStatus = 'Needs Manual Intervention';
     } else {
       existingDataRaptorNames.add(existingDRNameVal.cleanName());
     }

src/migration/flexcard.ts

@@ -211,13 +211,13 @@ export class CardMigrationTool extends BaseMigrationTool implements MigrationToo
       infos: [],
       warnings: [],
       errors: [],
-      migrationStatus: '',
+      migrationStatus: 'Ready for migration',
     };
 
     // Check for name changes due to API naming requirements
     const originalName: string = flexCardName;
     const cleanedName: string = this.cleanName(originalName);
-    let assessmentStatus = 'Ready for migration';
+    let assessmentStatus: 'Ready for migration' | 'Warnings' | 'Needs Manual Intervention' = 'Ready for migration';
     flexCardAssessmentInfo.name = this.allVersions ? `${cleanedName}_${version}` : cleanedName;
     if (cleanedName !== originalName) {
       flexCardAssessmentInfo.warnings.push(
@@ -229,7 +229,7 @@ export class CardMigrationTool extends BaseMigrationTool implements MigrationToo
     // Check for duplicate names
     if (uniqueNames.has(cleanedName)) {
       flexCardAssessmentInfo.warnings.push(this.messages.getMessage('duplicateCardNameMessage', [cleanedName]));
-      assessmentStatus = 'Need Manual Intervention';
+      assessmentStatus = 'Needs Manual Intervention';
     }
     uniqueNames.add(cleanedName);
 
@@ -302,7 +302,7 @@ export class CardMigrationTool extends BaseMigrationTool implements MigrationToo
           flexCardAssessmentInfo.warnings.push(
             this.messages.getMessage('integrationProcedureManualUpdateMessage', [originalIpMethod])
           );
-          flexCardAssessmentInfo.migrationStatus = 'Need Manual Intervention';
+          flexCardAssessmentInfo.migrationStatus = 'Needs Manual Intervention';
         }
       }
     } else if (dataSource.type === Constants.ApexRemoteComponentName) {

src/migration/omniscript.ts

@@ -470,15 +470,15 @@ export class OmniScriptMigrationTool extends BaseMigrationTool implements Migrat
     const existingSubTypeVal = new StringVal(existingSubType, 'sub type');
     const omniScriptName = omniscript[this.namespacePrefix + 'Name'];
     const existingOmniScriptNameVal = new StringVal(omniScriptName, 'name');
-    let assessmentStatus = 'Ready for migration';
+    let assessmentStatus: 'Ready for migration' | 'Warnings' | 'Needs Manual Intervention' = 'Ready for migration';
 
     const warnings: string[] = [];
 
     // Check for Angular OmniScript dependencies
     for (const osDep of dependencyOS) {
       if (this.nameRegistry.isAngularOmniScript(osDep.name)) {
         warnings.push(this.messages.getMessage('angularOmniScriptDependencyWarning', [osDep.location, osDep.name]));
-        assessmentStatus = 'Need Manual Intervention';
+        assessmentStatus = 'Needs Manual Intervention';
       }
     }
 
@@ -557,7 +557,7 @@ export class OmniScriptMigrationTool extends BaseMigrationTool implements Migrat
       const type = omniscript[this.namespacePrefix + 'IsLwcEnabled__c'] ? 'LWC' : 'Angular';
       if (type === 'Angular') {
         warnings.unshift(this.messages.getMessage('angularOSWarning'));
-        assessmentStatus = 'Need Manual Intervention';
+        assessmentStatus = 'Needs Manual Intervention';
       }
     }
 
@@ -627,7 +627,9 @@ export class OmniScriptMigrationTool extends BaseMigrationTool implements Migrat
           value.migrationSuccess = true;
         }
 
-        let finalKey = `${nameMapping.oldType}${nameMapping.oldSubtype}${nameMapping.oldLanguage}`;
+        let finalKey = `${nameMapping.oldType}${nameMapping.oldSubtype}${this.cleanLanguageName(
+          nameMapping.oldLanguage
+        )}`;
 
         finalKey = finalKey.toLowerCase();
         if (storage.osStorage.has(finalKey)) {
@@ -647,6 +649,11 @@ export class OmniScriptMigrationTool extends BaseMigrationTool implements Migrat
     StorageUtil.printAssessmentStorage();
   }
 
+  private cleanLanguageName(language: string): string {
+    // replace -, ( and ) and space with _
+    return language.replace(/[-() ]/g, '');
+  }
+
   async migrate(): Promise<MigrationResult[]> {
     // Get All Records from OmniScript__c (IP & OS Parent Records)
     const omniscripts = await this.getAllOmniScripts();
@@ -1039,7 +1046,7 @@ export class OmniScriptMigrationTool extends BaseMigrationTool implements Migrat
 
           let finalKey = `${oldrecord[this.namespacePrefix + 'Type__c']}${
             oldrecord[this.namespacePrefix + 'SubType__c']
-          }${oldrecord[this.namespacePrefix + 'Language__c']}`;
+          }${this.cleanLanguageName(oldrecord[this.namespacePrefix + 'Language__c'])}`;
 
           finalKey = finalKey.toLowerCase();
           if (storage.osStorage.has(finalKey)) {

src/migration/related/ExperienceSiteMigration.ts

@@ -120,7 +120,7 @@ export class ExperienceSiteMigration extends BaseRelatedObjectMigration {
       path: file.location,
       diff: JSON.stringify([]),
       hasOmnistudioContent: false,
-      status: 'Can be Automated',
+      status: 'Ready for migration',
     };
 
     const lookupComponentName = `${this.namespace}:vlocityLWCOmniWrapper`;
@@ -179,7 +179,7 @@ export class ExperienceSiteMigration extends BaseRelatedObjectMigration {
         path: file.location,
         diff: JSON.stringify([]),
         hasOmnistudioContent: false,
-        status: 'Errors',
+        status: 'Failed',
       };
 
       experienceSiteAssessmentInfos.push(experienceSiteAssessmentInfo);
@@ -256,7 +256,7 @@ export class ExperienceSiteMigration extends BaseRelatedObjectMigration {
       experienceSiteAssessmentInfo.warnings.push(
         'Target exists as empty string. Please check experience site configuration'
       );
-      experienceSiteAssessmentInfo.status = 'Errors';
+      experienceSiteAssessmentInfo.status = 'Needs Manual Intervention';
       return;
     }
 
@@ -287,7 +287,7 @@ export class ExperienceSiteMigration extends BaseRelatedObjectMigration {
     if (this.shouldAddWarning(targetDataFromStorageFC)) {
       const warningMsg: string = this.getWarningMessage(flexcardName, targetDataFromStorageFC);
       experienceSiteAssessmentInfo.warnings.push(warningMsg);
-      experienceSiteAssessmentInfo.status = 'Errors';
+      experienceSiteAssessmentInfo.status = 'Needs Manual Intervention';
     } else {
       component.componentName = TARGET_COMPONENT_NAME_FC;
 
@@ -316,7 +316,7 @@ export class ExperienceSiteMigration extends BaseRelatedObjectMigration {
     if (this.shouldAddWarning(targetDataFromStorage)) {
       const warningMsg: string = this.getWarningMessage(targetName, targetDataFromStorage);
       experienceSiteAssessmentInfo.warnings.push(warningMsg);
-      experienceSiteAssessmentInfo.status = 'Errors';
+      experienceSiteAssessmentInfo.status = 'Needs Manual Intervention';
     } else {
       component.componentName = TARGET_COMPONENT_NAME_OS;