fix: log mode consistecny

Author: naman-contentstack

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

@@ -157,6 +157,7 @@ Use this plugin to automate the process of cloning a stack in few steps.
   async run(): Promise<void> {
     try {
       const self = this;
+      configHandler.set('log.progressSupportedModule', 'clone');
       const { flags: cloneCommandFlags } = await self.parse(StackCloneCommand);
       const {
         yes,
@@ -227,7 +228,10 @@ Use this plugin to automate the process of cloning a stack in few steps.
           config.source_stack = listOfTokens[sourceManagementTokenAlias].apiKey;
           log.debug(`Using source token alias: ${sourceManagementTokenAlias}`, cloneContext);
         } else if (sourceManagementTokenAlias) {
-          log.warn(`Provided source token alias (${sourceManagementTokenAlias}) not found in your config.!`, cloneContext);
+          log.warn(
+            `Provided source token alias (${sourceManagementTokenAlias}) not found in your config.!`,
+            cloneContext,
+          );
         }
         if (destinationManagementTokenAlias && listOfTokens?.[destinationManagementTokenAlias]) {
           config.destination_alias = destinationManagementTokenAlias;
@@ -284,7 +288,10 @@ Use this plugin to automate the process of cloning a stack in few steps.
     } catch (error: any) {
       if (error) {
         await this.cleanUp(pathdir, null, this.createCloneContext('unknown'));
-        log.error('Stack clone command failed', { ...this.createCloneContext('unknown'), error: error?.message || error });
+        log.error('Stack clone command failed', {
+          ...this.createCloneContext('unknown'),
+          error: error?.message || error,
+        });
       }
     }
   }

packages/contentstack-import/src/import/module-importer.ts

@@ -1,7 +1,7 @@
 import { resolve } from 'path';
 import { AuditFix } from '@contentstack/cli-audit';
 import messages, { $t } from '@contentstack/cli-audit/lib/messages';
-import { addLocale, cliux, ContentstackClient, log } from '@contentstack/cli-utilities';
+import { addLocale, cliux, ContentstackClient, log, CLIProgressManager } from '@contentstack/cli-utilities';
 
 import startModuleImport from './modules';
 import { ImportConfig, Modules } from '../types';
@@ -59,6 +59,14 @@ class ModuleImporter {
       if (!(await this.auditImportData())) {
         return { noSuccessMsg: true };
       }
+      // Audit overwrote the global summary with 'AUDIT'. Re-initialize for IMPORT so the final
+      // summary shows "IMPORT completed successfully!" and only import module stats.
+      const branchName = this.importConfig.branchName || '';
+      CLIProgressManager.initializeGlobalSummary(
+        branchName ? `IMPORT-${branchName}` : 'IMPORT',
+        branchName,
+        'Importing content...',
+      );
     }
 
     if (!this.importConfig.master_locale) {

packages/contentstack-utilities/src/constants/logging.ts

@@ -16,4 +16,4 @@ export const levelColors = {
   debug: 'blue',
 };
 
-export const PROGRESS_SUPPORTED_MODULES = ['export', 'import', 'audit', 'import-setup'] as const;
+export const PROGRESS_SUPPORTED_MODULES = ['export', 'import', 'audit', 'import-setup', 'clone'] as const;

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

@@ -94,13 +94,20 @@ export default class CLIProgressManager {
   }
 
   /**
-   * Print the final summary for all modules using strategies
+   * Print the final summary for all modules using strategies.
+   * When showConsoleLogs is enabled, skip the Progress Manager summary block
+   * so output is pure timestamped log lines (per documentation).
    */
   static printGlobalSummary(): void {
     if (!CLIProgressManager.globalSummary) {
       return;
     }
 
+    const showConsoleLogs = configHandler.get('log')?.showConsoleLogs;
+    if (showConsoleLogs) {
+      return;
+    }
+
     // Apply strategy-based corrections before printing
     CLIProgressManager.applyStrategyCorrections();
 
@@ -591,6 +598,11 @@ export default class CLIProgressManager {
   }
 
   private printSummary(): void {
+    // In console logs mode, use only timestamped log lines (no Progress Manager blocks)
+    if (this.showConsoleLogs) {
+      return;
+    }
+
     if (!this.enableNestedProgress) {
       // Simple summary for single progress
       this.log('\n' + chalk.bold(`${this.moduleName} Summary:`));

packages/contentstack-utilities/test/unit/cliProgressManager.test.ts

@@ -57,6 +57,7 @@ Module.prototype.require = function (id: string) {
 
 import CLIProgressManager from '../../src/progress-summary/cli-progress-manager';
 import SummaryManager from '../../src/progress-summary/summary-manager';
+import { configHandler } from '../../src';
 
 // Optimized cleanup function for fast tests
 function forceCleanupSpinners() {
@@ -237,11 +238,34 @@ describe('CLIProgressManager', () => {
     });
 
     // Note: Skipping actual withLoadingSpinner tests to avoid ora spinner issues in test environment
-    fancy.it('should print global summary when exists', () => {
+    fancy.it('should print global summary when exists and showConsoleLogs is false', () => {
       const summaryStub = sinon.stub(SummaryManager.prototype, 'printFinalSummary');
-      CLIProgressManager.initializeGlobalSummary('TEST', '');
-      CLIProgressManager.printGlobalSummary();
-      expect(summaryStub.calledOnce).to.be.true;
+      const configGetStub = sinon.stub(configHandler, 'get').callThrough();
+      configGetStub.withArgs('log').returns({ showConsoleLogs: false });
+
+      try {
+        CLIProgressManager.initializeGlobalSummary('TEST', '');
+        CLIProgressManager.printGlobalSummary();
+        expect(summaryStub.calledOnce).to.be.true;
+      } finally {
+        configGetStub.restore();
+        summaryStub.restore();
+      }
+    });
+
+    fancy.it('should skip global summary when showConsoleLogs is true (pure console log mode)', () => {
+      const summaryStub = sinon.stub(SummaryManager.prototype, 'printFinalSummary');
+      const configGetStub = sinon.stub(configHandler, 'get').callThrough();
+      configGetStub.withArgs('log').returns({ showConsoleLogs: true });
+
+      try {
+        CLIProgressManager.initializeGlobalSummary('SKIP_SUMMARY_TEST', '');
+        CLIProgressManager.printGlobalSummary();
+        expect(summaryStub.called).to.be.false;
+      } finally {
+        configGetStub.restore();
+        summaryStub.restore();
+      }
     });
   });
 
@@ -412,30 +436,28 @@ describe('CLIProgressManager', () => {
       expect(consoleLogStub.called).to.be.false;
     });
 
-    fancy.it('should print summary on stop when showConsoleLogs is true', () => {
+    fancy.it('should not print Progress Manager summary when showConsoleLogs is true (pure console log mode)', () => {
       progressManager.tick(true, 'item1');
       progressManager.tick(false, 'item2', 'error');
       progressManager.stop();
 
-      expect(consoleLogStub.called).to.be.true;
-      // Check if summary content was logged
+      // When showConsoleLogs is enabled, per-module summary blocks are skipped for consistent log output
       const logCalls = consoleLogStub.getCalls();
-      const summaryCall = logCalls.find(call => 
+      const summaryCall = logCalls.find(call =>
         call.args[0] && call.args[0].includes('TEST Summary:')
       );
-      expect(summaryCall).to.not.be.undefined;
-      
-      // Ensure progress manager is stopped
+      expect(summaryCall).to.be.undefined;
+
       progressManager = null as any;
     });
 
-    fancy.it('should print detailed summary for nested progress', () => {
+    fancy.it('should not print Detailed Summary blocks when showConsoleLogs is true (pure console log mode)', () => {
       const nestedManager = new CLIProgressManager({
         showConsoleLogs: true,
         enableNestedProgress: true,
         moduleName: 'NESTED_TEST',
       });
-      
+
       try {
         nestedManager.addProcess('process1', 5);
         nestedManager.startProcess('process1');
@@ -444,14 +466,13 @@ describe('CLIProgressManager', () => {
         nestedManager.completeProcess('process1');
         nestedManager.stop();
 
-        expect(consoleLogStub.called).to.be.true;
+        // When showConsoleLogs is enabled, Detailed Summary blocks are skipped
         const logCalls = consoleLogStub.getCalls();
-        const detailedSummaryCall = logCalls.find(call => 
+        const detailedSummaryCall = logCalls.find(call =>
           call.args[0] && call.args[0].includes('NESTED_TEST Detailed Summary:')
         );
-        expect(detailedSummaryCall).to.not.be.undefined;
+        expect(detailedSummaryCall).to.be.undefined;
       } finally {
-        // Ensure cleanup
         try {
           nestedManager.stop();
         } catch (e) {