Merge pull request #2320 from contentstack/revert-2314-fix/DX-3610

Revert "fix: inconsistent log messages in case of failures"

Author: naman-contentstack

packages/contentstack-import/src/commands/cm/stacks/import.ts

@@ -148,7 +148,14 @@ export default class ImportCommand extends Command {
       const moduleImporter = new ModuleImporter(managementAPIClient, importConfig);
       const result = await moduleImporter.start();
       backupDir = importConfig.backupDir;
-      //Note: Final summary is now handled by summary manager
+
+      if (!result?.noSuccessMsg) {
+        const successMessage = importConfig.stackName
+          ? `Successfully imported the content to the stack named ${importConfig.stackName} with the API key ${importConfig.apiKey} .`
+          : `The content has been imported to the stack ${importConfig.apiKey} successfully!`;
+        log.success(successMessage, importConfig.context);
+      }
+
       CLIProgressManager.printGlobalSummary();
       this.logSuccessAndBackupMessages(backupDir, importConfig);
       // Clear progress module setting now that import is complete

packages/contentstack-utilities/src/progress-summary/cli-progress-manager.ts

@@ -108,16 +108,6 @@ export default class CLIProgressManager {
     CLIProgressManager.globalSummary.printFinalSummary();
   }
 
-  /**
-   * Check if there are any failures in the global summary
-   */
-  static hasFailures(): boolean {
-    if (!CLIProgressManager.globalSummary) {
-      return false;
-    }
-    return CLIProgressManager.globalSummary.hasFailures();
-  }
-
   /**
    * Apply strategy-based corrections to module data
    */

packages/contentstack-utilities/src/progress-summary/summary-manager.ts

@@ -131,12 +131,10 @@ export default class SummaryManager {
 
     // Final Status
     console.log('\n' + chalk.bold('Final Status:'));
-    if (!this.hasFailures() && failedModules === 0) {
-      console.log(chalk.bold.green(`✅ ${this.operationName} completed successfully!`));
-    } else if (this.hasFailures() || failedModules > 0) {
-      console.log(
-        chalk.bold.yellow(`⚠️ ${this.operationName} completed with failures, see the logs for more details.`),
-      );
+    if (failedModules === 0) {
+      console.log(chalk.bold.green(`🎉 ${this.operationName} completed successfully!`));
+    } else if (completedModules > 0) {
+      console.log(chalk.bold.yellow(`⚠️  ${this.operationName} completed with ${failedModules} failed modules`));
     } else {
       console.log(chalk.bold.red(`❌ ${this.operationName} failed`));
     }
@@ -148,13 +146,6 @@ export default class SummaryManager {
     this.printFailureSummaryWithLogReference();
   }
 
-  /**
-   * Check if there are any failures across all modules
-   */
-  hasFailures(): boolean {
-    return Array.from(this.modules.values()).some((m) => m.failures.length > 0 || m.failureCount > 0);
-  }
-
   private printFailureSummaryWithLogReference(): void {
     const modulesWithFailures = Array.from(this.modules.values()).filter((m) => m.failures.length > 0);