Merge pull request #2296 from contentstack/enhancement/DX-3611

feat:fix progress manager logging

Author: naman-contentstack

.talismanrc

@@ -1,4 +1,60 @@
 fileignoreconfig:
   - filename: package-lock.json
     checksum: 3f28d87bef29c352de910a0654318a847b5ced07cdaf271e9a155c98ae677b16
+  - filename: pnpm-lock.yaml
+    checksum: 6f278bce220f922eef8cd7d2a5c74e978892621e66049b927df6ed01964a76c8
+  - filename: packages/contentstack-import/src/commands/cm/stacks/import.ts
+    checksum: ee643d664d69eafc6f82923048a85c4dc26f78b1ac93cfe1e152c69945b65d9e
+  - filename: packages/contentstack-seed/src/commands/cm/stacks/seed.ts
+    checksum: d04770564196b080878566255ea0faf1c82c1460161d2004d2b1edece0546493
+  - filename: packages/contentstack-import/README.md
+    checksum: f701583669afcf9be38c8b1698aede39658c0df5afe6691ba1301408d20c7c69
+  - filename: packages/contentstack-auth/test/unit/commands/tokens-remove.test.ts
+    checksum: 7e256db86c516b40c47eab78b3881b7ece525607281ebcd0b8711c089bdac056
+  - filename: packages/contentstack-export/src/commands/cm/stacks/export.ts
+    checksum: 0f419db76a2e4132e8f833ae2b194d2c2658a0417265390a6036b63b3401c876
+  - filename: packages/contentstack-export/README.md
+    checksum: ed79c7f29e1cb17a4889c9b19877fd2d7596aec4cc8fcbf4f83643155ef1800c
+  - filename: packages/contentstack-import-setup/README.md
+    checksum: f1640c025b0f7603b5f81b511802bf9de18ee6daefbf01521b8d2347490c7757
+  - filename: packages/contentstack-import-setup/src/commands/cm/stacks/import-setup.ts
+    checksum: a903276f842ec00e49ed4751d6df8f4f7213fb1aaa631323f6ce593dad0b236c
+  - filename: packages/contentstack-migration/README.md
+    checksum: f25b5601b30057471827d44b41483ea9664586f6c65559ac04a3d39bfb319526
+  - filename: packages/contentstack-import-setup/src/utils/login-handler.ts
+    checksum: ca22ae9649dbc11eba4a4399ef517bdd3f3e3b0ca815307a3313bc9332be3f85
+  - filename: packages/contentstack-migration/src/commands/cm/stacks/migration.js
+    checksum: ebc17ed46356ec89d294fa0caa84a654d09823410ba5f122331c0ebd83decd92
+  - filename: packages/contentstack-import-setup/test/unit/login-handler.test.ts
+    checksum: 51155d629b9825ab8218023cba367ea41264ce534f891e3a6e14a829c2399654
+  - filename: packages/contentstack-bootstrap/test/bootstrap-integration.test.js
+    checksum: d3e3902b2ee72aa41483da5c135e5c4bcec85f65939695708e9bec9478f6336c
+  - filename: packages/contentstack-migration/test/unit/utils/map.test.js
+    checksum: 7d570280b2f379531dde84946b06171d50f92489ff0de6336f7fbd81c336ee89
+  - filename: packages/contentstack-seed/test/commands/cm/stacks/seed.test.ts
+    checksum: 61143bbb2ac86c24afc6972d17d9179c6181ec68a909b84570afdad2aaa13ade
+  - filename: packages/contentstack-config/test/unit/commands/remove-base-branch.test.ts
+    checksum: 3e73b079b749120221f357d66ac7351ddb300d283f28410d5e6f3b0309ab3962
+  - filename: packages/contentstack-migration/test/unit/validators/edit-content-type-validator.test.js
+    checksum: bde4bc6b2a90e7ce5872e6fbbabef9f2db352705be4d2f8d28d71d84209a714e
+  - filename: packages/contentstack-bootstrap/test/bootstrap.test.js
+    checksum: b1f46b3447b1b358f80d6404d9d5b385fb385714e5c1f865ca97d64d6edaefc2
+  - filename: packages/contentstack-seed/README.md
+    checksum: 67378f35a33ae250a8099bbb42a5e644486e7279715685e3ba5c65add6451c46
+  - filename: packages/contentstack-utilities/test/unit/contentstack-marketplace-sdk.test.ts
+    checksum: ccf521d7eafe03d5c4b597c8b30c4cceac00e75d597685fbfb411da4603fbd29
+  - filename: packages/contentstack-migration/test/unit/validators/create-content-type-validator.test.js
+    checksum: f92e39a542cd2d561c441d23395515cadc24c9514de55c3edb038f70bd2458b3
+  - filename: packages/contentstack-migration/test/unit/modules/parser.test.js
+    checksum: 243fa1c45875675f719f660c0c988e2ba9266c562a37aaeb09b0db93a0cb037d
+  - filename: packages/contentstack-bootstrap/test/interactive.test.js
+    checksum: fb0c32cd846cce3a53927316699a1c5aaa814939fe9b33bcd9141addbbe447d0
+  - filename: packages/contentstack-seed/test/seed/interactive.test.ts
+    checksum: e7a823051b5eb27f2674ca2c31719205fa822e9cac1524dbd14e48b1ec078c06
+  - filename: packages/contentstack/README.md
+    checksum: e650b0eb54404b08fb2b802a05883dc5984bff43842876df70bcbb5821034fe5
+  - filename: packages/contentstack-seed/test/seed/importer.test.ts
+    checksum: 77bc27f5217c6d69c21bac51afc94d677ad67374c1b39b0575646300eb0decd3
+  - filename: packages/contentstack-seed/test/seed/contentstack/client.test.ts
+    checksum: f1bc369c9c3c4a84ddd590864c0f3e8b13be956b8fb8891b6324f44cdcc7d568
 version: '1.0'

packages/contentstack-export/src/utils/export-config-handler.ts

@@ -9,6 +9,10 @@ import { filter, includes } from 'lodash';
 import { ExportConfig } from '../types';
 
 const setupConfig = async (exportCmdFlags: any): Promise<ExportConfig> => {
+  // Set progress supported module FIRST, before any log calls
+  // This ensures the logger respects the showConsoleLogs setting correctly
+  configHandler.set('log.progressSupportedModule', 'export');
+  
   let config = merge({}, defaultConfig);
 
   // Track authentication method
@@ -132,8 +136,6 @@ const setupConfig = async (exportCmdFlags: any): Promise<ExportConfig> => {
       throw new Error(`Invalid query format: ${error.message}`);
     }
   }
-  // Set progress supported module to check and display console logs
-  configHandler.set('log.progressSupportedModule', 'export');
   // Add authentication details to config for context tracking
   config.authenticationMethod = authenticationMethod;
   log.debug('Export configuration setup completed.', { ...config });

packages/contentstack-import-setup/src/utils/import-config-handler.ts

@@ -7,7 +7,11 @@ import login from './login-handler';
 import { ImportConfig } from '../types';
 
 const setupConfig = async (importCmdFlags: any): Promise<ImportConfig> => {
-  const config: ImportConfig = merge({}, defaultConfig);
+  // Set progress supported module FIRST, before any log calls
+  // This ensures the logger respects the showConsoleLogs setting correctly
+  configHandler.set('log.progressSupportedModule', 'import-setup');
+
+  let config: ImportConfig = merge({}, defaultConfig);
   // setup the config
   // if (importCmdFlags['config']) {
   //   let externalConfig = await readFile(importCmdFlags['config']);
@@ -18,7 +22,9 @@ const setupConfig = async (importCmdFlags: any): Promise<ImportConfig> => {
   //   config = merge.recursive(config, externalConfig);
   // }
 
-  config.contentDir = sanitizePath(importCmdFlags['data'] || importCmdFlags['data-dir'] || config.data || (await askContentDir()));
+  config.contentDir = sanitizePath(
+    importCmdFlags['data'] || importCmdFlags['data-dir'] || config.data || (await askContentDir()),
+  );
   const pattern = /[*$%#<>{}!&?]/g;
   if (pattern.test(config.contentDir)) {
     cliux.print(`\nPlease enter a directory path without special characters: (*,&,{,},[,],$,%,<,>,?,!)`, {

packages/contentstack-import/src/utils/import-config-handler.ts

@@ -9,6 +9,10 @@ import login from './login-handler';
 import { ImportConfig } from '../types';
 
 const setupConfig = async (importCmdFlags: any): Promise<ImportConfig> => {
+  // Set progress supported module FIRST, before any log calls
+  // This ensures the logger respects the showConsoleLogs setting correctly
+  configHandler.set('log.progressSupportedModule', 'import');
+
   let config: ImportConfig = merge({}, defaultConfig);
   // Track authentication method
   let authenticationMethod = 'unknown';

packages/contentstack-utilities/src/logger/logger.ts

@@ -77,8 +77,8 @@ export default class Logger {
       const hasProgressSupport = currentModule && PROGRESS_SUPPORTED_MODULES.includes(currentModule);
 
       if (hasProgressSupport) {
-        // Plugin has progress bars - respect user's showConsoleLogs setting
-        showConsoleLogs = logConfig.showConsoleLogs ?? true;
+        // Plugin has progress bars - respect user's explicit setting, or default to false (show progress bars)
+        showConsoleLogs = logConfig.showConsoleLogs ?? false;
       } else {
         // Plugin doesn't have progress support - always show console logs
         showConsoleLogs = true;