Remove excessive debug logging across various services and components to streamline code and improve readability. Update logging levels from info to error where appropriate, and ensure consistent handling of locale data in migration processes.

Author: sauravraw

api/src/services/drupal/query.service.ts

@@ -344,10 +344,6 @@ export const createQuery = async (
       destination_stack_id
     );
 
-    console.info(
-      `🔍 query.service.ts - Database connection established successfully`
-    );
-
     // SQL query to extract field configuration from Drupal
     const configQuery =
       "SELECT *, CONVERT(data USING utf8) as data FROM config WHERE name LIKE '%field.field.node%'";
@@ -376,7 +372,7 @@ export const createQuery = async (
           });
         }
       } catch (err: any) {
-        console.warn(`Couldn't parse row ${i}:`, err.message);
+        console.error(`Couldn't parse row ${i}:`, err.message);
       }
     }
 

api/src/services/migration.service.ts

@@ -1073,19 +1073,6 @@ const startMigration = async (req: Request): Promise<any> => {
         );
         // 🔍 DEBUG: Log master_locale before passing to createEntry
         const masterLocaleForContentful = project?.stackDetails?.master_locale;
-        console.info(
-          '🔍 Contentful startMigration - master_locale before createEntry:',
-          {
-            master_locale: masterLocaleForContentful,
-            master_locale_type: typeof masterLocaleForContentful,
-            master_locale_isLowercase:
-              masterLocaleForContentful ===
-              masterLocaleForContentful?.toLowerCase?.(),
-            master_locale_toLowerCase:
-              masterLocaleForContentful?.toLowerCase?.(),
-            project_stackDetails: project?.stackDetails,
-          }
-        );
 
         await contentfulService?.createEntry(
           cleanLocalPath,
@@ -1603,15 +1590,6 @@ export const createSourceLocales = async (req: Request) => {
     if (index > -1) {
       ProjectModelLowdb?.update?.((data: any) => {
         data.projects[index].source_locales = locales;
-
-        console.info(
-          '✅ [createSourceLocales] Saved source_locales to project:',
-          {
-            projectId,
-            saved_source_locales: locales,
-            first_element_master: locales[0] || 'NONE',
-          }
-        );
       });
     } else {
       logger.error(`Project with ID: ${projectId} not found`, {
@@ -1691,23 +1669,6 @@ export const updateLocaleMapper = async (req: Request) => {
         .get('projects')
         .find({ id: projectId })
         .value();
-      console.info(
-        '================================================================================'
-      );
-      console.info(
-        '🔍 [API updateLocaleMapper] Saved locale data to database:'
-      );
-      console.info('  Project ID:', projectId);
-      console.info('  master_locale:', updatedProject?.master_locale);
-      console.info('  locales:', updatedProject?.locales);
-      console.info('  localeMapping:', updatedProject?.localeMapping);
-      console.info(
-        '  localeMapping keys:',
-        Object.keys(updatedProject?.localeMapping || {})
-      );
-      console.info(
-        '================================================================================'
-      );
 
       // 🔍 LOGGING: Log after update
       await ProjectModelLowdb?.read?.();

api/src/utils/batch-processor.utils.ts

@@ -15,7 +15,7 @@ export class BatchProcessor<T> {
   constructor(options: BatchProcessorOptions) {
     this.options = {
       delayBetweenBatches: 100, // Default 100ms delay
-      ...options
+      ...options,
     };
   }
 
@@ -25,23 +25,27 @@ export class BatchProcessor<T> {
   async processBatches<R>(
     items: T[],
     processor: (item: T) => Promise<R>,
-    onBatchComplete?: (batchIndex: number, totalBatches: number, results: R[]) => void
+    onBatchComplete?: (
+      batchIndex: number,
+      totalBatches: number,
+      results: R[]
+    ) => void
   ): Promise<R[]> {
     const { batchSize, concurrency, delayBetweenBatches } = this.options;
     const totalBatches = Math.ceil(items.length / batchSize);
     const allResults: R[] = [];
 
-    console.log(`📦 Processing ${items.length} items in ${totalBatches} batches (${batchSize} items per batch, concurrency: ${concurrency})`);
-
     for (let batchIndex = 0; batchIndex < totalBatches; batchIndex++) {
       const startIndex = batchIndex * batchSize;
       const endIndex = Math.min(startIndex + batchSize, items.length);
       const batch = items.slice(startIndex, endIndex);
 
-      console.log(`📋 Processing batch ${batchIndex + 1}/${totalBatches} (${batch.length} items)`);
-
       // Process batch with controlled concurrency
-      const batchResults = await this.processBatchWithConcurrency(batch, processor, concurrency);
+      const batchResults = await this.processBatchWithConcurrency(
+        batch,
+        processor,
+        concurrency
+      );
       allResults.push(...batchResults);
 
       // Callback for batch completion
@@ -50,7 +54,11 @@ export class BatchProcessor<T> {
       }
 
       // Delay between batches to allow file handles to close
-      if (batchIndex < totalBatches - 1 && delayBetweenBatches && delayBetweenBatches > 0) {
+      if (
+        batchIndex < totalBatches - 1 &&
+        delayBetweenBatches &&
+        delayBetweenBatches > 0
+      ) {
         await this.delay(delayBetweenBatches);
       }
 
@@ -60,7 +68,6 @@ export class BatchProcessor<T> {
       }
     }
 
-    console.log(`✅ Completed processing ${items.length} items in ${totalBatches} batches`);
     return allResults;
   }
 
@@ -73,7 +80,7 @@ export class BatchProcessor<T> {
     concurrency: number
   ): Promise<R[]> {
     const results: R[] = [];
-    
+
     for (let i = 0; i < batch.length; i += concurrency) {
       const chunk = batch.slice(i, i + concurrency);
       const chunkPromises = chunk.map(processor);
@@ -88,7 +95,7 @@ export class BatchProcessor<T> {
    * Delay utility
    */
   private delay(ms: number): Promise<void> {
-    return new Promise(resolve => setTimeout(resolve, ms));
+    return new Promise((resolve) => setTimeout(resolve, ms));
   }
 }
 
@@ -99,7 +106,11 @@ export async function processBatches<T, R>(
   items: T[],
   processor: (item: T) => Promise<R>,
   options: BatchProcessorOptions,
-  onBatchComplete?: (batchIndex: number, totalBatches: number, results: R[]) => void
+  onBatchComplete?: (
+    batchIndex: number,
+    totalBatches: number,
+    results: R[]
+  ) => void
 ): Promise<R[]> {
   const batchProcessor = new BatchProcessor<T>(options);
   return batchProcessor.processBatches(items, processor, onBatchComplete);

api/src/utils/content-type-creator.utils.ts

@@ -960,7 +960,7 @@ export const convertToSchemaFormate = ({ field, advanced = true, marketPlacePath
           "non_localizable": field.advanced?.nonLocalizable ?? false,
         }
       } else {
-        console.info('Content Type Field', field?.contentstackField)
+        console.error('Content Type Field', field?.contentstackField)
       }
     }
   }
@@ -1174,6 +1174,6 @@ export const contenTypeMaker = async ({ contentType, destinationStackId, project
       await saveContent(ct, contentSave);
     }
   } else {
-    console.info(contentType?.contentstackUid, 'missing');
+    console.error(contentType?.contentstackUid, 'missing');
   }
 };

api/src/utils/custom-logger.utils.ts

@@ -68,17 +68,13 @@ const customLogger = async (
 
     if (!fs.existsSync(logFilePath)) {
       // If the file does not exist, create it and write an initial log entry
-      console.info(`Creating new log file at: ${logFilePath}`);
       // Conditionally log stack trace based on environment
       if (process.env.NODE_ENV !== 'production') {
         console.info(new Error().stack);
       }
       await fs.promises.writeFile(logFilePath, 'Log file created\n', {
         flag: 'a',
       });
-      console.info(
-        `Log file created and initial entry written: ${logFilePath}`
-      );
     }
     // Create a logger instance with a file transport
     const log = createLogger({

api/src/utils/sanitize-path.utils.ts

@@ -7,7 +7,7 @@ import path from 'path';
  * @param filename - The input filename to sanitize.
  * @returns A safe, sanitized filename.
  */
-export const sanitizeFilename = (filename: string): string => {
+const sanitizeFilename = (filename: string): string => {
   return path.basename(filename).replace(/[^a-zA-Z0-9_.\s-]/g, '');
 };
 

ui/src/components/Common/AddStack/addStack.tsx

@@ -114,26 +114,6 @@ const AddStack = (props: any): JSX.Element => {
         const rawMasterLocale = sourceLocales.length > 0 ? sourceLocales[0] : 'en-us';
         const masterLocale = (typeof rawMasterLocale === 'string' ? rawMasterLocale : rawMasterLocale?.label || rawMasterLocale?.value || 'en-us').toLowerCase();
 
-        // 🔍 DEBUG: Log master locale detection (FIRST element from source_locales)
-        console.info('================================================================================');
-        console.info('🔍 AddStack - Master locale detection (source_locales[0]):', {
-          sourceLocales,
-          sourceLocales_length: sourceLocales.length,
-          sourceLocales_first_element: sourceLocales[0],
-          rawMasterLocale,
-          masterLocale_after_lowercase: masterLocale,
-          masterLocale_type: typeof masterLocale,
-          isLowercase: masterLocale === masterLocale?.toLowerCase?.(),
-        });
-        console.info('🔍 AddStack - Contentstack locales sample (first 3):', 
-          rawMappedLocalesMapped?.slice(0, 3)?.map(l => ({
-            value: l.value,
-            value_lowercase: l.value?.toLowerCase(),
-            label: l.label
-          }))
-        );
-        console.info('================================================================================');
-        
         // 🔧 CRITICAL: Find matching Contentstack locale (all lowercase comparison)
         const matchingLocale = rawMappedLocalesMapped.find(locale => {
           const localeValueLower = (locale.value || '').toLowerCase();
@@ -149,15 +129,6 @@ const AddStack = (props: any): JSX.Element => {
           return false;
         });
 
-        // 🔍 DEBUG: Log matching result
-        console.info('🔍 AddStack - Matching Contentstack locale:', {
-          matchingLocale,
-          matchingLocale_value: matchingLocale?.value,
-          matchingLocale_value_lowercase: matchingLocale?.value?.toLowerCase(),
-          will_auto_select: !!matchingLocale
-        });
-        console.info('================================================================================');
-        
         setAllLocales(rawMappedLocalesMapped);
 
         // Update form with correct master locale after locales are loaded
@@ -184,15 +155,6 @@ const AddStack = (props: any): JSX.Element => {
       const rawMasterLocale = sourceLocales.length > 0 ? sourceLocales[0] : 'en-us';
       const masterLocale = (typeof rawMasterLocale === 'string' ? rawMasterLocale : rawMasterLocale?.label || rawMasterLocale?.value || 'en-us').toLowerCase();
 
-      // 🔍 DEBUG: Log master locale in useEffect (source_locales[0])
-      console.info('🔍 AddStack useEffect - Re-checking master locale:', {
-        sourceLocales,
-        sourceLocales_first_element: sourceLocales[0],
-        rawMasterLocale,
-        masterLocale_after_lowercase: masterLocale,
-        allLocales_count: allLocales.length
-      });
-      
       // 🔧 CRITICAL: Find matching Contentstack locale (all lowercase comparison)
       const matchingLocale = allLocales.find(locale => {
         const localeValueLower = (locale.value || '').toLowerCase();
@@ -208,15 +170,7 @@ const AddStack = (props: any): JSX.Element => {
         return false;
       });
 
-      // 🔍 DEBUG: Log matching result in useEffect
-      console.info('🔍 AddStack useEffect - Final matching locale:', {
-        matchingLocale,
-        matchingLocale_value: matchingLocale?.value,
-        will_update_form: !!matchingLocale
-      });
-      
       if (matchingLocale) {
-        console.info('✅ AddStack - Auto-selecting master locale:', matchingLocale.value);
         formRef.current.change('locale', matchingLocale);
       } else {
         console.warn('⚠️ AddStack - No matching Contentstack locale found for:', masterLocale);

ui/src/components/ContentMapper/index.tsx

@@ -691,7 +691,6 @@ const ContentMapper = forwardRef(({ handleStepChange }: contentMapperProps, ref:
       //       delete updatedOptions[key];
       //     }
       //   });
-      //   console.info("updatedOptions", updatedOptions);
       //   return updatedOptions;
       // });
 

ui/src/components/DestinationStack/Actions/LoadStacks.tsx

@@ -114,13 +114,6 @@ const LoadStacks = (props: LoadFileFormatProps) => {
       // 🔧 CRITICAL: Ensure master_locale is always lowercase
       const masterLocale = (data?.locale || '').toLowerCase();
 
-      console.info('🔍 LoadStacks - Creating new stack:', {
-        stack_name: data?.name,
-        raw_locale: data?.locale,
-        master_locale_lowercase: masterLocale,
-        description: data?.description
-      });
-      
       // Post data to backend
       const resp = await createStacksInOrg(selectedOrganisation?.value, {
         ...data,

ui/src/components/LegacyCms/Actions/LoadUploadFile.tsx

@@ -126,14 +126,6 @@ const LoadUploadFile = (props: LoadUploadFileProps) => {
 
       const { data, status } = await fileValidation(projectId, newMigrationData?.legacy_cms?.affix);
 
-      /* eslint-disable no-console */
-      console.log('📥 UI - API Response received:');
-      console.log('  status:', status);
-      console.log('  data.file_details.isSQL:', data?.file_details?.isSQL);
-      console.log('  data.file_details.cmsType:', data?.file_details?.cmsType);
-      console.log('  Full data:', data);
-      /* eslint-enable no-console */
-
       setProgressPercentage(70);
       setProcessing('Processing...70%');
 
@@ -312,7 +304,6 @@ const LoadUploadFile = (props: LoadUploadFileProps) => {
       //setIsFormatValid(isFormatValid); 
       setIsDisabled(!isFormatValid || isEmptyString(newMigrationDataRef?.current?.legacy_cms?.affix));
       if(!isFormatValid){
-        console.warn('⚠️ LoadUploadFile: File format is not valid, setting isValidated to false');
         setValidationMessage('');
         // ✅ FIX: Properly spread existing data to prevent data loss
         dispatch(updateNewMigrationData({
@@ -526,21 +517,13 @@ const LoadUploadFile = (props: LoadUploadFileProps) => {
             disabled={!(reValidate || (!isDisabled))}
           > 
             {(() => {
-              /* eslint-disable no-console */
-              console.log('=== BUTTON LABEL DEBUG ===');
-              console.log('fileDetails:', fileDetails);
-              console.log('fileDetails.isLocalPath:', fileDetails?.isLocalPath);
-              console.log('fileDetails.isSQL:', fileDetails?.isSQL);
-              
               // Logic: If using local path, always "File Validate"
               // If not using local path AND using SQL, then "Check Connection"
               // Otherwise "File Validate"
               const buttonText = fileDetails?.isLocalPath 
                 ? 'File Validate' 
                 : (fileDetails?.isSQL ? 'Check Connection' : 'File Validate');
 
-              console.log('Button text will be:', buttonText);
-              /* eslint-enable no-console */
               return buttonText;
             })()}
           </Button>