Fixed PR comments

Author: sunil-lakshman

src/commands/cm/stacks/export-query.ts

@@ -105,7 +105,7 @@ export default class ExportQueryCommand extends Command {
       await queryExporter.execute();
       log.debug('Query exporter completed successfully', context);
 
-      log.info('Query-based export completed successfully!', context);
+      log.success('Query-based export completed successfully!', context);
       log.info(`Export files saved to: ${this.exportDir}`, context);
     } catch (error) {
       const errorConfig = { exportDir: this.exportDir, stackApiKey: '' } as QueryExportConfig;

src/core/module-exporter.ts

@@ -38,7 +38,7 @@ export class ModuleExporter {
       }
 
       // success message
-      log.info(`Successfully exported ${moduleName}`, this.logContext);
+      log.success(`Successfully exported ${moduleName}`, this.logContext);
     } catch (error) {
       log.error(`Failed to export ${moduleName}: ${formatError(error)}`, this.logContext);
       throw error;

src/core/query-executor.ts

@@ -170,7 +170,7 @@ export class QueryExporter {
       }
 
       fsUtil.writeFile(sanitizePath(contentTypesFilePath), contentTypes);
-      log.info('Referenced content types export completed successfully', this.logContext);
+      log.success('Referenced content types export completed successfully', this.logContext);
     } catch (error) {
       log.error(`Error exporting referenced content types: ${error.message}`, this.logContext);
       throw error;
@@ -249,7 +249,7 @@ export class QueryExporter {
       // export personalize
       await this.moduleExporter.exportModule('personalize');
 
-      log.info('Dependent modules export completed successfully', this.logContext);
+      log.success('Dependent modules export completed successfully', this.logContext);
     } catch (error) {
       log.error(`Error exporting dependent modules: ${error.message}`, this.logContext);
       throw error;
@@ -284,7 +284,7 @@ export class QueryExporter {
       // and export entries for all of them
       await this.moduleExporter.exportModule('entries');
 
-      log.info('Entries export completed successfully', this.logContext);
+      log.success('Entries export completed successfully', this.logContext);
     } catch (error) {
       log.error(`Error exporting entries: ${error.message}`, this.logContext);
       throw error;
@@ -421,7 +421,7 @@ export class QueryExporter {
         fsUtil.removeFile(sanitizePath(tempAssetFilePath));
 
         log.info(`Temporary files cleaned up`, this.logContext);
-        log.info('Referenced assets exported successfully', this.logContext);
+        log.success('Referenced assets exported successfully', this.logContext);
       } else {
         log.info('No referenced assets found in entries', this.logContext);
       }

src/utils/branch-helper.ts

@@ -35,7 +35,7 @@ export const setupBranches = async (config: QueryExportConfig, stackAPIClient: a
         });
 
       if (result && typeof result === 'object') {
-        log.info(`Branch '${config.branchName}' found`, context);
+        log.success(`Branch '${config.branchName}' found`, context);
       } else {
         throw new Error(`No branch found named ${config.branchName}.`);
       }

src/utils/index.ts

@@ -1,6 +1,6 @@
 export * as fileHelper from './file-helper';
 export { fsUtil } from './file-helper';
-export { log, unlinkFileLogger, createLogContext, redactSensitiveData } from './logger';
+export { log, unlinkFileLogger, createLogContext } from './logger';
 export * from './common-helper';
 export * from './config-handler';
 export * from './content-type-helper';

src/utils/logger.ts

@@ -174,35 +174,11 @@ export function createLogContext(config: QueryExportConfig): any {
   return {
     command: 'cm:stacks:export-query',
     module: '',
-    userId: configHandler.get('userUid') || '',
     email: configHandler.get('email') || '',
-    sessionId: '',
+    sessionId: configHandler.get('sessionId') || '',
     apiKey: config.stackApiKey || '',
     orgId: configHandler.get('oauthOrgUid') || '',
     authenticationMethod: config.managementToken ? 'Management Token' : 'Basic Auth',
   };
 }
 
-/**
- * Redacts API keys and other sensitive information from message strings
- */
-export function redactSensitiveData(message: string, apiKey?: string): string {
-  if (!message || typeof message !== 'string') {
-    return message;
-  }
-
-  let redactedMessage = message;
-
-  // Redact the provided API key if it exists
-  if (apiKey && apiKey.length > 0) {
-    const escapedApiKey = apiKey.replace(/[.*+?^${}()|[\]\\]/g, '\\$&');
-    const apiKeyRegex = new RegExp(escapedApiKey, 'gi');
-    redactedMessage = redactedMessage.replace(apiKeyRegex, '[REDACTED]');
-  }
-
-  // Redact common API key patterns (blt + 32 hex chars or bltc + 32 hex chars)
-  const apiKeyPattern = /\bbltc?[a-f0-9]{32}\b/gi;
-  redactedMessage = redactedMessage.replace(apiKeyPattern, '[REDACTED]');
-
-  return redactedMessage;
-}