From ac3175ba75554b3d1b394f4a11ae1f67a243cf45 Mon Sep 17 00:00:00 2001 From: naman-contentstack Date: Wed, 7 Jan 2026 17:49:42 +0530 Subject: [PATCH] feat:fix progress manager logging --- .../contentstack-export/src/utils/export-config-handler.ts | 6 ++++-- .../src/utils/import-config-handler.ts | 4 ++++ .../contentstack-import/src/utils/import-config-handler.ts | 7 ++++--- packages/contentstack-utilities/src/logger/logger.ts | 4 ++-- 4 files changed, 14 insertions(+), 7 deletions(-) diff --git a/packages/contentstack-export/src/utils/export-config-handler.ts b/packages/contentstack-export/src/utils/export-config-handler.ts index 80250fb54d..b2accb4729 100644 --- a/packages/contentstack-export/src/utils/export-config-handler.ts +++ b/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 => { + // 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 => { 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 }); diff --git a/packages/contentstack-import-setup/src/utils/import-config-handler.ts b/packages/contentstack-import-setup/src/utils/import-config-handler.ts index e099d43ae8..8b2c0d7a05 100644 --- a/packages/contentstack-import-setup/src/utils/import-config-handler.ts +++ b/packages/contentstack-import-setup/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 => { + // 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']) { diff --git a/packages/contentstack-import/src/utils/import-config-handler.ts b/packages/contentstack-import/src/utils/import-config-handler.ts index 1a3ff963c8..639fd7f3d1 100644 --- a/packages/contentstack-import/src/utils/import-config-handler.ts +++ b/packages/contentstack-import/src/utils/import-config-handler.ts @@ -15,6 +15,10 @@ import login from './login-handler'; import { ImportConfig } from '../types'; const setupConfig = async (importCmdFlags: any): Promise => { + // 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'; @@ -135,9 +139,6 @@ const setupConfig = async (importCmdFlags: any): Promise => { config['exclude-global-modules'] = importCmdFlags['exclude-global-modules']; } - // Set progress supported module to check and display console logs - configHandler.set('log.progressSupportedModule', 'import'); - // Add authentication details to config for context tracking config.authenticationMethod = authenticationMethod; log.debug('Import configuration setup completed', { ...config }); diff --git a/packages/contentstack-utilities/src/logger/logger.ts b/packages/contentstack-utilities/src/logger/logger.ts index c3612b7dbd..ff1fc544b1 100644 --- a/packages/contentstack-utilities/src/logger/logger.ts +++ b/packages/contentstack-utilities/src/logger/logger.ts @@ -75,8 +75,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;