🧹 [Code Health] Replace console statements with proper logger

src/index.js

@@ -10,6 +10,7 @@ import parser from './lib/parser';
 import getUniqueID from './lib/util/getUniqueID';
 import hasParents from './lib/util/hasParents';
 import openUrl from './lib/util/openUrl';
+import logger from './lib/util/logger.js';
 import tokensToAST from './lib/util/tokensToAST';
 import renderRules from './lib/renderRules';
 import AstRenderer from './lib/AstRenderer';
@@ -91,13 +92,13 @@ const getRenderer = (
   debugPrintTree,
 ) => {
   if (renderer && rules) {
-    console.warn(
+    logger.warn(
       'react-native-markdown-display you are using renderer and rules at the same time. This is not possible, props.rules is ignored',
     );
   }
 
   if (renderer && style) {
-    console.warn(
+    logger.warn(
       'react-native-markdown-display you are using renderer and style at the same time. This is not possible, props.style is ignored',
     );
   }

src/lib/AstRenderer.js

@@ -1,6 +1,7 @@
 import {StyleSheet} from 'react-native';
 
 import {textStylePropsSet} from './data/textStyleProps';
+import logger from './util/logger.js';
 import convertAdditionalStyles from './util/convertAdditionalStyles';
 import getUniqueID from './util/getUniqueID';
 
@@ -30,6 +31,16 @@ export default class AstRenderer {
     this._debugPrintTree = debugPrintTree;
   }
 
+  _log = (...args) => {
+    if (this._debugPrintTree === true) {
+      logger.log(...args);
+    }
+  };
+
+  _warn = (...args) => {
+    logger.warn(...args);
+  };
+
   /**
    *
    * @param {string} type
@@ -39,7 +50,7 @@ export default class AstRenderer {
     const renderFunction = this._renderRules[type];
 
     if (!renderFunction) {
-      console.warn(
+      this._warn(
         `Warning, unknown render rule encountered: ${type}. 'unknown' render rule used (by default, returns null - nothing rendered)`,
       );
       return this._renderRules.unknown;
@@ -58,9 +69,7 @@ export default class AstRenderer {
     const renderFunction = this.getRenderFunction(node.type);
     const parents = [...parentNodes];
 
-    if (this._debugPrintTree === true) {
-      console.log(`${'-'.repeat(parents.length)}${node.type}`);
-    }
+    this._log(`${'-'.repeat(parents.length)}${node.type}`);
 
     parents.unshift(node);
 

src/lib/util/logger.js

@@ -0,0 +1,19 @@
+const logger = {
+  log: (...args) => {
+    if (typeof console !== 'undefined' && typeof console.log === 'function') {
+      console.log(...args);
+    }
+  },
+  warn: (...args) => {
+    if (typeof console !== 'undefined' && typeof console.warn === 'function') {
+      console.warn(...args);
+    }
+  },
+  error: (...args) => {
+    if (typeof console !== 'undefined' && typeof console.error === 'function') {
+      console.error(...args);
+    }
+  },
+};
+
+export default logger;

src/lib/util/stringToTokens.js

@@ -1,9 +1,10 @@
+import logger from './logger.js';
 export function stringToTokens(source, markdownIt) {
   let result = [];
   try {
     result = markdownIt.parse(source, {});
   } catch (err) {
-    console.warn(err);
+    logger.warn(err);
   }
 
   return result;