@mcp-shark/mcp-shark 1.5.9 → 1.5.10

package/core/mcp-server/auditor/audit.js

@@ -226,14 +226,13 @@ export async function withAuditRequestResponseHandler(
 
   // Extract response body as string
   const resBodyStr = resBuf.toString('utf8');
-  const resBodyJson = parseJsonSafely(resBodyStr);
 
   const jsonrpcId = reqJsonRpc?.payload?.id !== undefined ? String(reqJsonRpc.payload.id) : null;
 
   auditLogger.logResponsePacket({
     statusCode: wrappedRes.statusCode || 200,
     headers: resHeaders,
-    body: resBodyJson || resBodyStr,
+    body: resBodyStr,
     requestFrameNumber: requestResult?.frameNumber || null,
     requestTimestampNs: requestResult?.timestampNs || null,
     jsonrpcId,

package/core/repositories/AuditRepository.js

@@ -69,17 +69,68 @@ export class AuditRepository {
     return `${statusCode}${rpcInfo}`;
   }
 
+  _isCharacterCodeObject(obj) {
+    if (!obj || typeof obj !== 'object' || Array.isArray(obj)) {
+      return false;
+    }
+
+    const keys = Object.keys(obj);
+    if (keys.length === 0) {
+      return false;
+    }
+
+    // Check if all keys are numeric strings starting from "0"
+    for (let i = 0; i < keys.length; i++) {
+      if (keys[i] !== String(i)) {
+        return false;
+      }
+      // Check if values are numbers (character codes)
+      if (typeof obj[keys[i]] !== 'number') {
+        return false;
+      }
+    }
+
+    return true;
+  }
+
+  _convertCharacterCodeObjectToString(obj) {
+    const chars = [];
+    const keys = Object.keys(obj).sort((a, b) => Number(a) - Number(b));
+    for (const key of keys) {
+      chars.push(String.fromCharCode(obj[key]));
+    }
+    return chars.join('');
+  }
+
   _normalizeBody(body) {
     if (!body) {
       return { bodyRaw: '', bodyJson: null };
     }
+
+    // Handle String objects (created with new String())
+    if (Object.prototype.toString.call(body) === '[object String]') {
+      const str = String(body);
+      return { bodyRaw: str, bodyJson: str };
+    }
+
+    // Handle primitive strings
     if (typeof body === 'string') {
       return { bodyRaw: body, bodyJson: body };
     }
+
+    // Handle objects
     if (typeof body === 'object') {
+      // Check if it's a character code object and convert it back to string
+      if (this._isCharacterCodeObject(body)) {
+        const str = this._convertCharacterCodeObjectToString(body);
+        return { bodyRaw: str, bodyJson: str };
+      }
+
+      // Otherwise, stringify normally
       const raw = JSON.stringify(body);
       return { bodyRaw: raw, bodyJson: raw };
     }
+
     return { bodyRaw: '', bodyJson: null };
   }
 

package/core/repositories/StatisticsRepository.js

@@ -38,7 +38,7 @@ export class StatisticsRepository {
         COUNT(*) as total_packets,
         COUNT(CASE WHEN direction = 'request' THEN 1 END) as total_requests,
         COUNT(CASE WHEN direction = 'response' THEN 1 END) as total_responses,
-        COUNT(CASE WHEN status_code >= ${StatusCodeRanges.CLIENT_ERROR_START} THEN 1 END) as total_errors,
+        COUNT(CASE WHEN status_code >= ${StatusCodeRanges.CLIENT_ERROR_MIN} THEN 1 END) as total_errors,
         COUNT(DISTINCT session_id) as unique_sessions,
         AVG(length) as avg_packet_size,
         SUM(length) as total_bytes,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mcp-shark/mcp-shark",
-  "version": "1.5.9",
+  "version": "1.5.10",
   "description": "Aggregate multiple Model Context Protocol (MCP) servers into a single unified interface with a powerful monitoring UI. Prov deep visibility into every request and response.",
   "type": "module",
   "main": "./bin/mcp-shark.js",
@@ -25,13 +25,26 @@
   "bugs": {
     "url": "https://github.com/mcp-shark/mcp-shark/issues"
   },
-  "files": ["bin", "ui", "core", "README.md", "LICENSE", "package.json"],
+  "files": [
+    "bin",
+    "ui",
+    "core",
+    "!**/__tests__",
+    "!**/*.test.js",
+    "!**/*.spec.js",
+    "README.md",
+    "LICENSE",
+    "package.json"
+  ],
   "scripts": {
     "start": "node scripts/start-ui.js",
     "predev": "npm run build:ui",
     "dev": "node scripts/start-ui.js",
     "build": "bash -c 'cd ui && vite build'",
     "build:ui": "bash -c 'cd ui && vite build'",
+    "test": "c8 node --test --test-name-pattern='.*' --test-reporter spec $(find core ui/server -path '*/__tests__/*.test.js' -type f)",
+    "test:ui": "vitest run --coverage",
+    "test:all": "npm run test:coverage && npm run test:ui",
     "lint": "biome lint .",
     "lint:fix": "biome lint --write .",
     "format": "biome format --write .",
@@ -93,9 +106,21 @@
     "@biomejs/biome": "^1.9.4",
     "@commitlint/cli": "^19.5.0",
     "@commitlint/config-conventional": "^19.5.0",
+    "@testing-library/jest-dom": "^6.4.0",
+    "@testing-library/react": "^16.0.0",
     "@vitejs/plugin-react": "^4.2.1",
+    "@vitest/coverage-v8": "^4.0.18",
+    "c8": "^10.1.2",
     "husky": "^9.1.6",
+    "jsdom": "^25.0.0",
     "lint-staged": "^15.2.10",
-    "vite": "^7.3.0"
+    "vite": "^7.3.0",
+    "vitest": "^4.0.18"
+  },
+  "c8": {
+    "include": ["core/**/*.js", "ui/server/**/*.js"],
+    "exclude": ["**/__tests__/**", "**/*.test.js", "**/index.js"],
+    "reporter": ["text", "html"],
+    "check-coverage": false
   }
 }

package/ui/src/setupTests.js

@@ -0,0 +1 @@
+import '@testing-library/jest-dom';