Replace jsdom with node-html-parser for better bundling

Author: elieteyssedou

.changeset/replace-jsdom-with-node-html-parser.md

@@ -0,0 +1,9 @@
+---
+"@mcp-pointer/server": patch
+---
+
+Replace jsdom with node-html-parser for better bundling
+
+- Reduced bundle size
+- Fixes bundling issues with esbuild
+- faster HTML parsing

packages/server/package.json

@@ -25,13 +25,12 @@
   "dependencies": {
     "@modelcontextprotocol/sdk": "^0.5.0",
     "commander": "^11.1.0",
-    "jsdom": "^27.0.0",
+    "node-html-parser": "^7.0.1",
     "ws": "^8.16.0"
   },
   "devDependencies": {
     "@mcp-pointer/shared": "workspace:*",
     "@types/jest": "^30.0.0",
-    "@types/jsdom": "^21.1.7",
     "@types/ws": "^8.5.10",
     "esbuild": "catalog:",
     "jest": "^30.1.3",

packages/server/src/services/element-processor.ts

@@ -8,12 +8,10 @@ import logger from '../logger';
 
 function safeGet<T>(obj: any, path: string, defaultValue: T): T {
   const keys = path.split('.');
-  let result = obj;
-  for (const key of keys) {
-    result = result?.[key];
+  return keys.reduce((result, key) => {
     if (result === undefined) return defaultValue;
-  }
-  return result ?? defaultValue;
+    return result?.[key];
+  }, obj) ?? defaultValue;
 }
 
 export default class ElementProcessor {
@@ -89,7 +87,7 @@ export default class ElementProcessor {
       framework: 'react' as const,
     };
 
-    const sourceFile = safeGet(reactFiber, '_debugSource.fileName', '');
+    const sourceFile = safeGet<string>(reactFiber, '_debugSource.fileName', '');
     if (sourceFile && typeof sourceFile === 'string') {
       const fileName = sourceFile.split('/').pop() || sourceFile;
       const lineNumber = safeGet(reactFiber, '_debugSource.lineNumber', 0);

packages/server/src/services/mcp-service.ts

@@ -67,7 +67,7 @@ export default class MCPService {
   }
 
   private async getPointedElement() {
-    const processedElement = await this.sharedState.getProcessedElement();
+    const processedElement = await this.sharedState.getPointedElement();
 
     if (!processedElement) {
       return {

packages/server/src/utils/dom-extractor.ts

@@ -1,11 +1,11 @@
-import { JSDOM } from 'jsdom';
+import { parse } from 'node-html-parser';
 
 export function extractFromHTML(html: string) {
   const warnings: string[] = [];
 
   try {
-    const fragment = JSDOM.fragment(html);
-    const element = fragment.firstChild as Element;
+    const root = parse(html);
+    const element = root.firstChild as unknown as Element;
 
     if (!element) {
       warnings.push('No element found in HTML');