feat: remove node-html-parser from browser bundle

Author: SettingDust

build.ts

@@ -9,7 +9,7 @@ import { nodeExternalsPlugin } from 'esbuild-node-externals';
     target: 'es2017',
     logLevel: 'info',
     color: true,
-    // minify: true,
+    minify: true,
     sourcemap: true,
     legalComments: 'external'
   }
@@ -20,6 +20,9 @@ import { nodeExternalsPlugin } from 'esbuild-node-externals';
     outfile: 'dist/node-html-markdown.js',
     format: 'esm',
     platform: 'node',
+    define: {
+      'IS_BROWSER': 'false'
+    },
     plugins: [ nodeExternalsPlugin({
       packagePath: './package.json'
     }) ]
@@ -30,6 +33,9 @@ import { nodeExternalsPlugin } from 'esbuild-node-externals';
     outfile: 'dist/node-html-markdown.cjs',
     format: 'cjs',
     platform: 'node',
+    define: {
+      'IS_BROWSER': 'false'
+    },
     plugins: [ nodeExternalsPlugin({
       packagePath: './package.json'
     }) ]
@@ -45,8 +51,12 @@ import { nodeExternalsPlugin } from 'esbuild-node-externals';
     format: 'esm',
     platform: 'browser',
     define: {
-      'process.env.LOG_PERF': process.env.LOG_PERF ?? 'false'
-    }
+      // This will remove the perfStart/End after minified by esbuild.
+      'process.env.LOG_PERF': process.env.LOG_PERF ?? 'false',
+      // This will remove the node-html-parser usage
+      'IS_BROWSER': 'true'
+    },
+    external: ['node-html-parser']
   }).then()
 
   await build({
@@ -55,7 +65,9 @@ import { nodeExternalsPlugin } from 'esbuild-node-externals';
     format: 'cjs',
     platform: 'browser',
     define: {
-      'process.env.LOG_PERF': process.env.LOG_PERF ?? 'false'
-    }
+      'process.env.LOG_PERF': process.env.LOG_PERF ?? 'false',
+      'IS_BROWSER': 'true'
+    },
+    external: ['node-html-parser']
   }).then()
 })()

src/config.ts

@@ -1,7 +1,7 @@
 import { isWhiteSpaceOnly, splitSpecial, surround, tagSurround, trimNewLines } from './utilities';
 import { PostProcessResult, TranslatorConfigObject } from './translator';
 import { NodeHtmlMarkdownOptions } from './options';
-import { Options as NodeHtmlParserOptions } from 'node-html-parser'
+import type { Options as NodeHtmlParserOptions } from 'node-html-parser'
 
 
 /* ****************************************************************************************************************** */

src/nodes.ts

@@ -1,13 +1,10 @@
-import * as NHParser from 'node-html-parser';
-import { CommentNode, NodeType } from 'node-html-parser';
-
+import type * as NHParser from 'node-html-parser';
+import type { CommentNode } from 'node-html-parser';
 
 /* ****************************************************************************************************************** */
 // region: Types
 /* ****************************************************************************************************************** */
 
-export { NodeType, CommentNode }
-
 /* ********************************************************* *
  * Merged Nodes - Unions of node-html-parser and common DOM
  * ********************************************************* */
@@ -25,8 +22,8 @@ export type TextNode = (NHParser.TextNode) & NodeBase
 // region: TypeGuards
 /* ****************************************************************************************************************** */
 
-export const isTextNode = (node: HtmlNode): node is TextNode => node.nodeType === NodeType.TEXT_NODE;
-export const isCommentNode = (node: HtmlNode): node is CommentNode => node.nodeType === NodeType.COMMENT_NODE;
-export const isElementNode = (node: HtmlNode): node is ElementNode => node.nodeType === NodeType.ELEMENT_NODE;
+export const isTextNode = (node: HtmlNode): node is TextNode => node.nodeType === 3;
+export const isCommentNode = (node: HtmlNode): node is CommentNode => node.nodeType === 8;
+export const isElementNode = (node: HtmlNode): node is ElementNode => node.nodeType === 1;
 
 // endregion

src/utilities.ts

@@ -2,7 +2,6 @@ import { NodeHtmlMarkdownOptions } from './options';
 import { ElementNode, HtmlNode } from './nodes';
 import { nodeHtmlParserConfig } from './config';
 
-
 /* ****************************************************************************************************************** */
 // region: String Utils
 /* ****************************************************************************************************************** */
@@ -123,6 +122,9 @@ export const truthyStr = (v: any, value?: string): string => v ? ((value !== und
 /* ****************************************************************************************************************** */
 // region: Parser
 /* ****************************************************************************************************************** */
+// For esbuild removing code
+declare global {const IS_BROWSER: boolean}
+
 
 function tryParseWithNativeDom(html: string): ElementNode | undefined {
   try {
@@ -169,24 +171,25 @@ const getNodeHtmlParser = () => {
 export function parseHTML(html: string, options: NodeHtmlMarkdownOptions): ElementNode {
   let nodeHtmlParse: ReturnType<typeof getNodeHtmlParser>;
 
-  /* If specified, try to parse with native engine, fallback to node-html-parser */
   perfStart('parse');
   let el: ElementNode | undefined;
-  if (options.preferNativeParser) {
+  /* If specified, try to parse with native engine, fallback to node-html-parser */
+  if (IS_BROWSER || options.preferNativeParser) {
     try {
       el = tryParseWithNativeDom(html);
     }
     catch (e) {
+      if (IS_BROWSER) throw e;
       nodeHtmlParse = getNodeHtmlParser();
       if (nodeHtmlParse) console.warn('Native DOM parser encountered an error during parse', e);
       else throw e;
     }
   } else nodeHtmlParse = getNodeHtmlParser();
 
-  if (!el) el = nodeHtmlParse!(html, nodeHtmlParserConfig);
+  if (!IS_BROWSER && !el) el = nodeHtmlParse!(html, nodeHtmlParserConfig);
   perfStop('parse');
 
-  return el;
+  return el!;
 }
 
 // endregion