fix: @W-19821724: Flexcard multiple active versions and CX labels feedback

Changes:
- Addressed the Flexcards having multipe versions causing issues with assess/migrate
- Fixed the issue with OMA to consider version with -a flag and not flag dup/manual
- Addressed theCX label feedbakc items
- README and messages updates

Author: sf-kishore-kurri

.gitignore

@@ -46,6 +46,6 @@ oclif.manifest.json
 omnistudio_migration
 
 assessment_reports/
-migration_report/
+migration_reports/
 .sfdx/
 package.xml

README.md

@@ -34,7 +34,7 @@ You can also authenticate using a consumer key (client ID) and secret key throug
 3. In a new terminal session, Install the Omnistudio Migration Assistant by running the `sf plugins install` command. For example
 
 ```
-sf plugins install @salesforce/[email protected].50
+sf plugins install @salesforce/[email protected].51
 ```
 
 4. Run the Omnistudio Migration Assistant from the Salesforce CLI:
@@ -114,7 +114,7 @@ sfdx omnistudio:migration:migrate -u [email protected] --only=autonum
 
 ```
 USAGE
-  $ sf omnistudio:migration:assess [-f] [-v <string>] [-u <string>] [--loglevel
+  $ sf omnistudio:migration:assess [-v <string>] [-u <string>] [--loglevel
   trace|debug|info|warn|error|fatal|TRACE|DEBUG|INFO|WARN|ERROR|FATAL]
 
 OPTIONS
@@ -149,7 +149,7 @@ OPTIONS
 
 ```
 USAGE
-  $ sf omnistudio:migration:migrate [-f] [-v <string>] [-u <string>] [--loglevel
+  $ sf omnistudio:migration:migrate [-v <string>] [-u <string>] [--loglevel
   trace|debug|info|warn|error|fatal|TRACE|DEBUG|INFO|WARN|ERROR|FATAL]
 
 OPTIONS

messages/assess.json

@@ -25,7 +25,7 @@
   "errorWhileUploadingCard": "We couldn't deploy your Flexcard to your org. Review these errors:",
   "errorWhileCreatingElements": "We couldn't save elements in your Omniscript. Review these errors:",
   "allVersionsDescription": "Migrate all versions of a component",
-  "changeMessage": "%s will be changed from %s to %s",
+  "changeMessage": "The %s %s will be changed from %s to %s",
   "angularOSWarning": "We couldn't migrate Angular Omniscripts. Convert all Angular Omniscripts to LWC Omniscripts, and try again.",
   "relatedObjectGA": "Select the component type that you want to migrate: 'apex' for Apex classes, 'lwc' for Lightning Web Components, 'flexipage' for FlexiPages, 'expsites' for Experience Sites, or 'apex, lwc, flexipage, expsites' if you want to include all types.",
   "noPackageInstalled": "No managed package found in your org.",
@@ -93,7 +93,7 @@
   "cardNameChangeMessage": "The Flexcard name will be changed from %s to %s to adhere to the API naming standards.",
   "authordNameChangeMessage": "The Flexcard author name will be changed from %s to %s to adhere to the naming rules: %s",
   "omniScriptNameChangeMessage": "The Omniscript reference name %s will be changed to %s during migration.",
-  "dataRaptorNameChangeMessage": "The Data Mapper %s will be renamed to %s during migration.",
+  "dataRaptorNameChangeMessage": "The Data Mapper name will be changed from %s to %s during migration.",
   "integrationProcedureNameChangeMessage": "The Integration Procedure %s will be renamed to %s during migration.",
   "integrationProcedureManualUpdateMessage": "All references to %s in this Integration Procedure must be manually updated after migration.",
   "duplicateCardNameMessage": "A Flexcard with the same name \"%s\" already exists after name cleaning, which could lead to conflicts during migration.",

messages/migrate.json

@@ -84,13 +84,13 @@
   "packageSelectionPrompt": "Enter the number of the package to use (1-%s):",
   "invalidPackageSelection": "Invalid selection. Please enter a number between 1 and %s.",
   "selectedPackage": "Selected package: %s (Version: %s)",
-  "dataRaptorNameChangeMessage": "The Data Mapper %s will be renamed to %s during migration.",
+  "dataRaptorNameChangeMessage": "The Data Mapper name %s will be renamed to %s during migration.",
   "integrationProcedureNameChangeMessage": "The Integration Procedure %s will be renamed to %s during migration.",
   "integrationProcedureManualUpdateMessage": "All references to %s in this Integration Procedure must be manually updated after migration.",
   "cardAuthorNameChangeMessage": "We've modified the name of this Flexcard author to adhere to the naming rules: %s",
   "cardNameChangeMessage": "We've modified the name of this Flexcard to adhere to the naming rules: %s",
   "duplicateCardNameMessage": "A Flexcard with the same name \"%s\" already exists after name cleaning, which could lead to conflicts during migration.",
-  "angularOmniscriptWarningMessage": "We couldn't migrate Angular Omniscripts. Convert all Angular Omniscripts to LWC Omniscripts, and try again.",
+  "angularOmniscriptWarningMessage": "We couldn't migrate Angular Omniscripts.",
   "apexFileChangesApplied": "Applied changes to the %s Apex class.",
   "generatingComponentReports": "Generating report for components",
   "generatingRelatedObjectReports": "Generating report for related objects",
@@ -137,7 +137,7 @@
   "enableExperienceBundleMetadataAPIProgramatically": "The ExperienceBundle metadata API needs to be programmatically enabled.",
   "unableToEnableExperienceBundleMetadataAPI": "We couldn’t enable the ExperienceBundle metadata API. Experience Cloud sites won’t be processed.",
   "relatedObjectsToProcess": "These objects will be processed: %s",
-  "consentForExperienceSites": "By proceeding further, you hereby consent to enable the ExperienceBundle metadata API and migrate Experience Cloud sites. Do you want to proceed?",
+  "consentForExperienceSites": "By proceeding further, you hereby consent to enable the ExperienceBundle metadata API and migrate Experience Cloud sites. Do you want to proceed? [y/n]",
   "noRelatedObjects": "No related objects to process",
   "turnOffExperienceBundleAPI": "Turning off the ExperienceBundle metadata API as it was enabled programmatically.",
   "errorRevertingExperienceBundleMetadataAPI": "We couldn’t revert the ExperienceBundle metadata API. Manually revert it in your org.",
@@ -192,7 +192,7 @@
   "completeComponentMappingMessage": "Registered name mappings for %s components",
   "componentMappingNotFound": "No registry mapping found for %s component: %s, using fallback cleaning",
   "integrationProcedureInvalidUnderscoreFormat": "Integration Procedure reference '%s' contains underscores in Type/SubType names. Skipping reference update to avoid incorrect parsing.",
-  "flexCardWithAngularOmniScriptWarning": "Flexcard has dependencies on Angular Omniscript(s) which are not migrated. Please convert Omniscript(s) to LWC before migrating this Flexcard.",
+  "flexCardWithAngularOmniScriptWarning": "We couldn't migrate Flexcards with dependencies on Angular Omniscripts.",
   "angularOmniScriptDependencyWarning": "Element '%s' references Angular Omniscript '%s' which will not be migrated. Consider converting the referenced Omniscript to LWC",
   "skipFlexcardAngularOmniScriptDependencyWarning": "Skipping Flexcard %s due to Angular Omniscript dependencies",
   "flexCardMigrationProcessingMessage": "Processing %s Flexcards for migration (%s skipped due to Angular dependencies)",
@@ -203,7 +203,7 @@
   "integrationProcedureTypeEmptyAfterCleaning": "Integration Procedure Type '%s' becomes empty after name cleaning. Please provide a valid Type value.",
   "integrationProcedureSubtypeEmptyAfterCleaning": "Integration Procedure SubType '%s' becomes empty after name cleaning. Please provide a valid SubType value.",
   "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 Ready for Migration before proceeding with the migration. Do you want to proceed? Yes or No?",
+  "migrationConsentMessage": "Make sure that all records listed in the Assessment Status column in the Assessment Report are marked as either Ready for Migration or Warnings 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",
   "errorCheckingStandardRuntime": "We couldn't check whether the Standard OmniStudio Runtime setting is enabled: %s. Try again later.",

src/migration/dataraptor.ts

@@ -23,6 +23,7 @@ import {
 import { StringVal } from '../utils/StringValue/stringval';
 import { Logger } from '../utils/logger';
 import { createProgressBar } from './base';
+import { Constants } from '../utils/constants/stringContants';
 
 export class DataRaptorMigrationTool extends BaseMigrationTool implements MigrationTool {
   static readonly DRBUNDLE_NAME = 'DRBundle__c';
@@ -317,11 +318,12 @@ 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' | 'Warnings' | 'Needs Manual Intervention' = 'Ready for migration';
+    let assessmentStatus: 'Ready for migration' | 'Warnings' | 'Needs manual intervention' = 'Ready for migration';
 
     if (!existingDRNameVal.isNameCleaned()) {
       warnings.push(
         this.messages.getMessage('changeMessage', [
+          Constants.DataMapperComponentName,
           existingDRNameVal.type,
           existingDRNameVal.val,
           existingDRNameVal.cleanName(),
@@ -333,12 +335,12 @@ export class DataRaptorMigrationTool extends BaseMigrationTool implements Migrat
     if (drName && /^[0-9]/.test(drName)) {
       const proposedName = 'DM' + this.cleanName(drName);
       warnings.push(this.messages.getMessage('dataMapperNameStartsWithNumber', [drName, proposedName]));
-      assessmentStatus = 'Needs Manual Intervention';
+      assessmentStatus = 'Needs manual intervention';
     }
 
     if (existingDataRaptorNames.has(existingDRNameVal.cleanName())) {
       warnings.push(this.messages.getMessage('duplicatedName') + '  ' + existingDRNameVal.cleanName());
-      assessmentStatus = 'Needs Manual Intervention';
+      assessmentStatus = 'Needs manual intervention';
     } else {
       existingDataRaptorNames.add(existingDRNameVal.cleanName());
     }

src/migration/flexcard.ts

@@ -219,7 +219,7 @@ export class CardMigrationTool extends BaseMigrationTool implements MigrationToo
     // Check for name changes due to API naming requirements
     const originalName: string = flexCardName;
     const cleanedName: string = this.cleanName(originalName);
-    let assessmentStatus: 'Ready for migration' | 'Warnings' | 'Needs Manual Intervention' | 'Failed' =
+    let assessmentStatus: 'Ready for migration' | 'Warnings' | 'Needs manual intervention' | 'Failed' =
       'Ready for migration';
     flexCardAssessmentInfo.name = this.allVersions ? `${cleanedName}_${version}` : cleanedName;
     if (cleanedName !== originalName) {
@@ -229,12 +229,13 @@ export class CardMigrationTool extends BaseMigrationTool implements MigrationToo
       assessmentStatus = getUpdatedAssessmentStatus(assessmentStatus, 'Warnings');
     }
 
-    // Check for duplicate names
-    if (uniqueNames.has(cleanedName)) {
-      flexCardAssessmentInfo.warnings.push(this.messages.getMessage('duplicateCardNameMessage', [cleanedName]));
-      assessmentStatus = getUpdatedAssessmentStatus(assessmentStatus, 'Needs Manual Intervention');
+    // Check for duplicate names (include version when allVersions is true)
+    const uniqueCleanedName = this.allVersions ? `${cleanedName}_${version}` : cleanedName;
+    if (uniqueNames.has(uniqueCleanedName)) {
+      flexCardAssessmentInfo.warnings.push(this.messages.getMessage('duplicateCardNameMessage', [uniqueCleanedName]));
+      assessmentStatus = getUpdatedAssessmentStatus(assessmentStatus, 'Needs manual intervention');
     }
-    uniqueNames.add(cleanedName);
+    uniqueNames.add(uniqueCleanedName);
 
     // Check for author name changes
     const originalAuthor = flexCard[this.namespacePrefix + 'Author__c'];
@@ -315,8 +316,12 @@ export class CardMigrationTool extends BaseMigrationTool implements MigrationToo
             this.messages.getMessage('integrationProcedureManualUpdateMessage', [originalIpMethod])
           );
           flexCardAssessmentInfo.migrationStatus = getUpdatedAssessmentStatus(
-            flexCardAssessmentInfo.migrationStatus,
-            'Needs Manual Intervention'
+            flexCardAssessmentInfo.migrationStatus as
+              | 'Warnings'
+              | 'Needs manual intervention'
+              | 'Ready for migration'
+              | 'Failed',
+            'Needs manual intervention'
           );
         }
       }
@@ -773,14 +778,19 @@ export class CardMigrationTool extends BaseMigrationTool implements MigrationToo
       }
       const transformedCardAuthorName = transformedCard['AuthorName'];
 
-      if (uniqueNames.has(transformedCard['Name'])) {
-        this.setRecordErrors(card, this.messages.getMessage('duplicatedCardName', [transformedCard['Name']]));
+      // Check for duplicates using version-aware name when allVersions is true
+      const uniqueCheckName = this.allVersions
+        ? `${transformedCard['Name']}_${transformedCard['VersionNumber']}`
+        : transformedCard['Name'];
+
+      if (uniqueNames.has(uniqueCheckName)) {
+        this.setRecordErrors(card, this.messages.getMessage('duplicatedCardName', [uniqueCheckName]));
         originalRecords.set(recordId, card);
         return;
       }
 
-      // Save the name for duplicated names check
-      uniqueNames.add(transformedCard['Name']);
+      // Save the name for duplicated names check (with version if allVersions is true)
+      uniqueNames.add(uniqueCheckName);
 
       // Create a map of the original records
       originalRecords.set(recordId, card);
@@ -879,6 +889,7 @@ export class CardMigrationTool extends BaseMigrationTool implements MigrationToo
         } else {
           value.migrationSuccess = true;
         }
+        // Use the oldName from nameMapping which already includes version when allVersions is true
         let finalKey = `${flexCardAssessmentInfo.nameMapping.oldName}`;
         finalKey = finalKey.toLowerCase();
         if (storage.fcStorage.has(finalKey)) {

src/migration/omniscript.ts

@@ -471,7 +471,7 @@ 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' | 'Warnings' | 'Needs Manual Intervention' = 'Ready for migration';
+    let assessmentStatus: 'Ready for migration' | 'Warnings' | 'Needs manual intervention' = 'Ready for migration';
 
     const warnings: string[] = [];
     const errors: string[] = [];
@@ -480,19 +480,19 @@ export class OmniScriptMigrationTool extends BaseMigrationTool implements Migrat
     if (omniProcessType === 'Integration Procedure') {
       if (!existingType || existingType.trim() === '') {
         errors.push(this.messages.getMessage('missingMandatoryField', ['Type', 'Integration Procedure']));
-        assessmentStatus = 'Needs Manual Intervention';
+        assessmentStatus = 'Needs manual intervention';
       }
       if (!existingSubType || existingSubType.trim() === '') {
         errors.push(this.messages.getMessage('missingMandatoryField', ['SubType', 'Integration Procedure']));
-        assessmentStatus = 'Needs Manual Intervention';
+        assessmentStatus = 'Needs manual intervention';
       }
     }
 
     // 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 = 'Needs Manual Intervention';
+        assessmentStatus = 'Needs manual intervention';
       }
     }
 
@@ -519,10 +519,11 @@ export class OmniScriptMigrationTool extends BaseMigrationTool implements Migrat
     if (!existingTypeVal.isNameCleaned()) {
       if (omniProcessType === 'Integration Procedure' && (!newType || newType.trim() === '')) {
         warnings.push(this.messages.getMessage('integrationProcedureTypeEmptyAfterCleaning', [existingTypeVal.val]));
-        assessmentStatus = 'Needs Manual Intervention';
+        assessmentStatus = 'Needs manual intervention';
       } else {
         warnings.push(
           this.messages.getMessage('changeMessage', [
+            omniProcessType,
             existingTypeVal.type,
             existingTypeVal.val,
             existingTypeVal.cleanName(),
@@ -536,10 +537,11 @@ export class OmniScriptMigrationTool extends BaseMigrationTool implements Migrat
         warnings.push(
           this.messages.getMessage('integrationProcedureSubtypeEmptyAfterCleaning', [existingSubTypeVal.val])
         );
-        assessmentStatus = 'Needs Manual Intervention';
+        assessmentStatus = 'Needs manual intervention';
       } else {
         warnings.push(
           this.messages.getMessage('changeMessage', [
+            omniProcessType,
             existingSubTypeVal.type,
             existingSubTypeVal.val,
             existingSubTypeVal.cleanName(),
@@ -552,6 +554,7 @@ export class OmniScriptMigrationTool extends BaseMigrationTool implements Migrat
     if (!existingOmniScriptNameVal.isNameCleaned()) {
       warnings.push(
         this.messages.getMessage('changeMessage', [
+          omniProcessType,
           existingOmniScriptNameVal.type,
           existingOmniScriptNameVal.val,
           existingOmniScriptNameVal.cleanName(),
@@ -561,7 +564,7 @@ export class OmniScriptMigrationTool extends BaseMigrationTool implements Migrat
     }
     if (existingOmniscriptNames.has(recordName)) {
       warnings.push(this.messages.getMessage('duplicatedName') + '  ' + recordName);
-      assessmentStatus = 'Needs Manual Intervention';
+      assessmentStatus = 'Needs manual intervention';
     } else {
       existingOmniscriptNames.add(recordName);
     }
@@ -570,21 +573,21 @@ export class OmniScriptMigrationTool extends BaseMigrationTool implements Migrat
     if (duplicateElementNames.size > 0) {
       const duplicateNamesList = Array.from(duplicateElementNames).join(', ');
       warnings.unshift(this.messages.getMessage('invalidOrRepeatingOmniscriptElementNames', [duplicateNamesList]));
-      assessmentStatus = 'Needs Manual Intervention';
+      assessmentStatus = 'Needs manual intervention';
     }
 
     // Add warning for reserved keys found in PropertySet
     if (foundReservedKeys.size > 0) {
       const reservedKeysList = Array.from(foundReservedKeys).join(', ');
       warnings.unshift(this.messages.getMessage('reservedKeysFoundInPropertySet', [reservedKeysList]));
-      assessmentStatus = 'Needs Manual Intervention';
+      assessmentStatus = 'Needs manual intervention';
     }
 
     if (omniProcessType === this.OMNISCRIPT) {
       const type = omniscript[this.namespacePrefix + 'IsLwcEnabled__c'] ? 'LWC' : 'Angular';
       if (type === 'Angular') {
         warnings.unshift(this.messages.getMessage('angularOSWarning'));
-        assessmentStatus = 'Needs Manual Intervention';
+        assessmentStatus = 'Needs manual intervention';
       }
     }