@bryan-thompson/inspector-assessment 1.35.1 → 1.35.3

package/cli/build/__tests__/assess-full-e2e.test.js

@@ -76,6 +76,10 @@ async function spawnCLI(args, timeout = 60000) {
         // Set timeout
         const timer = setTimeout(() => {
             if (proc && !proc.killed) {
+                // Destroy streams before killing to prevent memory leaks
+                proc.stdout?.destroy();
+                proc.stderr?.destroy();
+                proc.stdin?.destroy();
                 proc.kill("SIGTERM");
                 exitCode = -1; // Indicate timeout
             }
@@ -83,6 +87,10 @@ async function spawnCLI(args, timeout = 60000) {
         // Handle process exit
         proc.on("close", (code) => {
             clearTimeout(timer);
+            // Destroy streams to prevent memory leaks
+            proc?.stdout?.destroy();
+            proc?.stderr?.destroy();
+            proc?.stdin?.destroy();
             // Don't overwrite timeout exit code (-1)
             if (exitCode !== -1) {
                 exitCode = code;
@@ -100,6 +108,10 @@ async function spawnCLI(args, timeout = 60000) {
         // Handle errors
         proc.on("error", (err) => {
             clearTimeout(timer);
+            // Destroy streams to prevent memory leaks
+            proc?.stdout?.destroy();
+            proc?.stderr?.destroy();
+            proc?.stdin?.destroy();
             stderr += `\nProcess error: ${err.message}`;
             resolve({
                 stdout,
@@ -151,10 +163,10 @@ function parseJSONLEvents(stderr) {
  * @returns True if server responds, false otherwise
  */
 async function checkServerAvailable(url) {
+    const controller = new AbortController();
+    // Give enough time to receive initial response but not wait forever
+    const timeoutId = setTimeout(() => controller.abort(), 5000);
     try {
-        const controller = new AbortController();
-        // Give enough time to receive initial response but not wait forever
-        const timeoutId = setTimeout(() => controller.abort(), 5000);
         const response = await fetch(url, {
             method: "POST",
             headers: DEFAULT_HEADERS,
@@ -172,26 +184,22 @@ async function checkServerAvailable(url) {
         });
         // Server responded with a status code - check if it's OK
         if (response.status >= 500) {
-            clearTimeout(timeoutId);
             return false;
         }
         // For SSE responses, check if we can read any data
         // This confirms the server is actually responding
         const reader = response.body?.getReader();
         if (!reader) {
-            clearTimeout(timeoutId);
             return response.status < 500;
         }
         try {
             // Try to read the first chunk
             const { done, value } = await reader.read();
-            clearTimeout(timeoutId);
             reader.cancel(); // Cancel the stream - we don't need more data
             // If we got any data, the server is available
             return !done && value && value.length > 0;
         }
         catch {
-            clearTimeout(timeoutId);
             // If read fails after successful fetch, server still responded
             return true;
         }
@@ -199,6 +207,11 @@ async function checkServerAvailable(url) {
     catch {
         return false;
     }
+    finally {
+        // Always clean up timeout and abort controller
+        clearTimeout(timeoutId);
+        controller.abort();
+    }
 }
 /**
  * Create a temporary config file for testing
@@ -238,6 +251,9 @@ function createInvalidConfig(content, filename) {
 // Test Setup
 // ============================================================================
 describe("CLI E2E Integration Tests", () => {
+    afterEach(() => {
+        jest.clearAllMocks();
+    });
     let vulnerableAvailable = false;
     let hardenedAvailable = false;
     beforeAll(async () => {

package/cli/build/__tests__/assess-full.test.js

@@ -13,6 +13,9 @@ import * as path from "path";
  * without needing to import the actual module (which has side effects)
  */
 describe("CLI Argument Parsing Concepts", () => {
+    afterEach(() => {
+        jest.clearAllMocks();
+    });
     describe("Profile Flag Parsing", () => {
         const VALID_PROFILES = ["quick", "security", "compliance", "full"];
         function parseProfile(args) {

package/cli/build/__tests__/assessment-runner/assessment-executor.test.js

@@ -3,7 +3,7 @@
  *
  * Tests for runFullAssessment() orchestration logic.
  */
-import { jest, describe, it, expect, beforeEach, afterEach, } from "@jest/globals";
+import { jest, describe, it, expect, beforeEach, afterEach, afterAll, } from "@jest/globals";
 // Mock all dependencies with explicit any types for flexibility
 const mockLoadServerConfig = jest.fn();
 const mockConnectToServer = jest.fn();
@@ -137,6 +137,19 @@ describe("runFullAssessment", () => {
     afterEach(() => {
         jest.restoreAllMocks();
     });
+    afterAll(() => {
+        // Clean up module mocks to prevent memory leaks
+        jest.unmock("../../lib/assessment-runner/server-config.js");
+        jest.unmock("../../lib/assessment-runner/server-connection.js");
+        jest.unmock("../../lib/assessment-runner/source-loader.js");
+        jest.unmock("../../lib/assessment-runner/tool-wrapper.js");
+        jest.unmock("../../lib/assessment-runner/config-builder.js");
+        jest.unmock("../../../../client/lib/services/assessment/AssessmentOrchestrator.js");
+        jest.unmock("../../assessmentState.js");
+        jest.unmock("../../lib/jsonl-events.js");
+        jest.unmock("fs");
+        jest.unmock("../../../../client/lib/lib/assessmentTypes.js");
+    });
     describe("orchestration flow", () => {
         it("should load server config", async () => {
             await runFullAssessment(defaultOptions);

package/cli/build/__tests__/assessment-runner/config-builder.test.js

@@ -62,6 +62,12 @@ describe("buildConfig", () => {
     afterEach(() => {
         process.env = originalEnv;
     });
+    afterAll(() => {
+        jest.unmock("../../profiles.js");
+        jest.unmock("../../../../client/lib/lib/assessmentTypes.js");
+        jest.unmock("../../../../client/lib/services/assessment/lib/claudeCodeBridge.js");
+        jest.unmock("../../../../client/lib/services/assessment/config/performanceConfig.js");
+    });
     describe("default configuration", () => {
         it("should spread DEFAULT_ASSESSMENT_CONFIG", () => {
             const result = buildConfig({ serverName: "test" });

package/cli/build/__tests__/assessment-runner/index.test.js

@@ -7,6 +7,9 @@ import { describe, it, expect } from "@jest/globals";
 // Import the barrel/facade module
 import * as assessmentRunner from "../../lib/assessment-runner/index.js";
 describe("assessment-runner index exports", () => {
+    afterEach(() => {
+        jest.clearAllMocks();
+    });
     describe("function exports", () => {
         it("should export all 6 public functions", () => {
             expect(typeof assessmentRunner.loadServerConfig).toBe("function");

package/cli/build/__tests__/assessment-runner/server-config.test.js

@@ -22,6 +22,9 @@ describe("loadServerConfig", () => {
         jest.clearAllMocks();
         mockExistsSync.mockReturnValue(false);
     });
+    afterAll(() => {
+        jest.unmock("fs");
+    });
     describe("config path resolution", () => {
         it("should search explicit configPath first when provided", () => {
             const configPath = "/custom/path/config.json";

package/cli/build/__tests__/assessment-runner/server-connection.test.js

@@ -39,10 +39,18 @@ const { StreamableHTTPClientTransport } = await import("@modelcontextprotocol/sd
 const { connectToServer } = await import("../../lib/assessment-runner/server-connection.js");
 describe("connectToServer", () => {
     beforeEach(() => {
-        jest.clearAllMocks();
         mockConnect.mockResolvedValue(undefined);
         mockStdioTransport.stderr.on.mockClear();
     });
+    afterEach(() => {
+        jest.clearAllMocks();
+    });
+    afterAll(() => {
+        jest.unmock("@modelcontextprotocol/sdk/client/index.js");
+        jest.unmock("@modelcontextprotocol/sdk/client/stdio.js");
+        jest.unmock("@modelcontextprotocol/sdk/client/sse.js");
+        jest.unmock("@modelcontextprotocol/sdk/client/streamableHttp.js");
+    });
     describe("HTTP transport", () => {
         it("should create StreamableHTTPClientTransport for transport:http", async () => {
             const config = {

package/cli/build/__tests__/assessment-runner/source-loader.test.js

@@ -34,10 +34,15 @@ describe("loadSourceFiles", () => {
     const mockReadFileSync = fs.readFileSync;
     const mockReaddirSync = fs.readdirSync;
     beforeEach(() => {
-        jest.clearAllMocks();
         mockExistsSync.mockReturnValue(false);
         mockReaddirSync.mockReturnValue([]);
     });
+    afterEach(() => {
+        jest.clearAllMocks();
+    });
+    afterAll(() => {
+        jest.unmock("fs");
+    });
     describe("README discovery", () => {
         it("should find README.md in source directory", () => {
             const sourcePath = "/project";
@@ -118,13 +123,17 @@ describe("loadSourceFiles", () => {
             const consoleSpy = jest
                 .spyOn(console, "warn")
                 .mockImplementation(() => { });
-            mockExistsSync.mockImplementation((p) => p === path.join(sourcePath, "manifest.json"));
-            mockReadFileSync.mockReturnValue("{ invalid json }");
-            const result = loadSourceFiles(sourcePath);
-            expect(result.manifestRaw).toBe("{ invalid json }");
-            expect(result.manifestJson).toBeUndefined();
-            expect(consoleSpy).toHaveBeenCalledWith(expect.stringContaining("Failed to parse manifest.json"));
-            consoleSpy.mockRestore();
+            try {
+                mockExistsSync.mockImplementation((p) => p === path.join(sourcePath, "manifest.json"));
+                mockReadFileSync.mockReturnValue("{ invalid json }");
+                const result = loadSourceFiles(sourcePath);
+                expect(result.manifestRaw).toBe("{ invalid json }");
+                expect(result.manifestJson).toBeUndefined();
+                expect(consoleSpy).toHaveBeenCalledWith(expect.stringContaining("Failed to parse manifest.json"));
+            }
+            finally {
+                consoleSpy.mockRestore();
+            }
         });
     });
     describe("source file collection", () => {
@@ -276,14 +285,18 @@ describe("loadSourceFiles", () => {
             const consoleSpy = jest
                 .spyOn(console, "warn")
                 .mockImplementation(() => { });
-            mockExistsSync.mockReturnValue(false);
-            mockReaddirSync.mockImplementation(() => {
-                throw new Error("Permission denied");
-            });
-            const result = loadSourceFiles(sourcePath);
-            expect(result.sourceCodeFiles?.size).toBe(0);
-            expect(consoleSpy).toHaveBeenCalledWith(expect.stringContaining("Could not load source files"), expect.any(Error));
-            consoleSpy.mockRestore();
+            try {
+                mockExistsSync.mockReturnValue(false);
+                mockReaddirSync.mockImplementation(() => {
+                    throw new Error("Permission denied");
+                });
+                const result = loadSourceFiles(sourcePath);
+                expect(result.sourceCodeFiles?.size).toBe(0);
+                expect(consoleSpy).toHaveBeenCalledWith(expect.stringContaining("Could not load source files"), expect.any(Error));
+            }
+            finally {
+                consoleSpy.mockRestore();
+            }
         });
         it("should skip unreadable files silently", () => {
             const sourcePath = "/project";

package/cli/build/__tests__/assessment-runner/tool-wrapper.test.js

@@ -15,6 +15,9 @@ describe("createCallToolWrapper", () => {
             callTool: mockCallTool,
         };
     });
+    afterEach(() => {
+        jest.clearAllMocks();
+    });
     describe("successful tool calls", () => {
         it("should wrap successful tool response with content array", async () => {
             mockCallTool.mockResolvedValue({

package/cli/build/__tests__/assessment-runner-facade.test.js

@@ -12,6 +12,9 @@ import { loadServerConfig, loadSourceFiles, connectToServer, createCallToolWrapp
 // Test namespace import
 import * as AssessmentRunner from "../lib/assessment-runner.js";
 describe("Assessment Runner Facade", () => {
+    afterEach(() => {
+        jest.clearAllMocks();
+    });
     describe("Function Exports", () => {
         it("should export loadServerConfig function", () => {
             expect(typeof loadServerConfig).toBe("function");

package/cli/build/__tests__/cli-build-fixes.test.js

@@ -11,6 +11,9 @@
 import { describe, it, expect } from "@jest/globals";
 import { ScopedListenerConfig } from "../lib/event-config.js";
 describe("CLI Build Fixes Regression Tests", () => {
+    afterEach(() => {
+        jest.clearAllMocks();
+    });
     describe("event-config.ts - CLI_DEFAULT_MAX_LISTENERS constant", () => {
         it("should use local constant instead of cross-workspace import", () => {
             // Fix: Replaced DEFAULT_PERFORMANCE_CONFIG.eventEmitterMaxListeners

package/cli/build/__tests__/flag-parsing.test.js

@@ -669,8 +669,9 @@ describe("parseArgs Zod Schema Integration", () => {
         // Run any pending timers and restore
         jest.runAllTimers();
         jest.useRealTimers();
-        processExitSpy.mockRestore();
-        consoleErrorSpy.mockRestore();
+        // Use optional chaining in case spies weren't created (prevents memory leaks)
+        processExitSpy?.mockRestore();
+        consoleErrorSpy?.mockRestore();
     });
     describe("LogLevelSchema integration", () => {
         it("parseArgs validates log level with LogLevelSchema", () => {

package/cli/build/__tests__/http-transport-integration.test.js

@@ -24,9 +24,9 @@ const DEFAULT_HEADERS = {
  * Check if a server is available by sending a basic HTTP request
  */
 async function checkServerAvailable(url) {
+    const controller = new AbortController();
+    const timeoutId = setTimeout(() => controller.abort(), 5000);
     try {
-        const controller = new AbortController();
-        const timeoutId = setTimeout(() => controller.abort(), 5000);
         const response = await fetch(url, {
             method: "POST",
             headers: DEFAULT_HEADERS,
@@ -42,13 +42,17 @@ async function checkServerAvailable(url) {
             }),
             signal: controller.signal,
         });
-        clearTimeout(timeoutId);
         // Accept any response (200 or error) as indication server is up
         return response.status < 500;
     }
     catch {
         return false;
     }
+    finally {
+        // Always clean up timeout and abort controller to prevent memory leaks
+        clearTimeout(timeoutId);
+        controller.abort();
+    }
 }
 /**
  * Parse SSE response to extract JSON data
@@ -101,6 +105,9 @@ async function sendMcpRequest(url, method, params = {}, headers = {}) {
     return { response, data };
 }
 describe("HTTP Transport Integration", () => {
+    afterEach(() => {
+        jest.clearAllMocks();
+    });
     let vulnerableServerAvailable = false;
     let hardenedServerAvailable = false;
     beforeAll(async () => {

package/cli/build/__tests__/lib/server-configSchemas.test.js

@@ -7,7 +7,7 @@
  * Addresses QA requirement: Test that type guards are mutually exclusive
  * and correctly discriminate between transport types.
  */
-import { describe, it, expect } from "@jest/globals";
+import { jest, describe, it, expect } from "@jest/globals";
 import { z } from "zod";
 // Define schemas inline to avoid import issues with client/lib in CLI tests
 const HttpSseServerConfigSchema = z.object({
@@ -36,6 +36,9 @@ function isStdioConfig(entry) {
     return "command" in entry && !("url" in entry);
 }
 describe("server-configSchemas type guards", () => {
+    afterEach(() => {
+        jest.clearAllMocks();
+    });
     describe("isHttpSseConfig", () => {
         it("should return true for HTTP transport config", () => {
             const config = {

package/cli/build/__tests__/lib/zodErrorFormatter.test.js

@@ -4,10 +4,13 @@
  * Tests for formatZodError utility to ensure helpful error messages.
  * Addresses QA requirement: verify Zod error messages are helpful (not just generic "Invalid").
  */
-import { describe, it, expect } from "@jest/globals";
+import { jest, describe, it, expect } from "@jest/globals";
 import { z } from "zod";
 import { formatZodError, formatZodIssue, formatZodErrorIndented, zodErrorToArray, formatUserFriendlyError, formatZodErrorForJson, } from "../../lib/zodErrorFormatter.js";
 describe("zodErrorFormatter", () => {
+    afterEach(() => {
+        jest.clearAllMocks();
+    });
     describe("formatZodIssue", () => {
         it("should format issue with path", () => {
             const issue = {

package/cli/build/__tests__/profiles.test.js

@@ -7,6 +7,9 @@
 import { jest, describe, it, expect } from "@jest/globals";
 import { ASSESSMENT_PROFILES, PROFILE_METADATA, MODULE_ALIASES, DEPRECATED_MODULES, TIER_1_CORE_SECURITY, TIER_2_COMPLIANCE, TIER_3_CAPABILITY, TIER_4_EXTENDED, ALL_MODULES, resolveModuleNames, getProfileModules, isValidProfileName, getProfileHelpText, mapLegacyConfigToModules, modulesToLegacyConfig, } from "../profiles.js";
 describe("Profile Definitions", () => {
+    afterEach(() => {
+        jest.restoreAllMocks();
+    });
     describe("Profile Constants", () => {
         it("should have four profiles defined", () => {
             const profiles = Object.keys(ASSESSMENT_PROFILES);
@@ -124,17 +127,25 @@ describe("resolveModuleNames", () => {
     });
     it("should emit warnings for deprecated modules when warn=true", () => {
         const consoleSpy = jest.spyOn(console, "warn").mockImplementation(() => { });
-        const modules = ["documentation"];
-        resolveModuleNames(modules, true);
-        expect(consoleSpy).toHaveBeenCalledWith(expect.stringContaining("deprecated"));
-        consoleSpy.mockRestore();
+        try {
+            const modules = ["documentation"];
+            resolveModuleNames(modules, true);
+            expect(consoleSpy).toHaveBeenCalledWith(expect.stringContaining("deprecated"));
+        }
+        finally {
+            consoleSpy.mockRestore();
+        }
     });
     it("should not emit warnings when warn=false", () => {
         const consoleSpy = jest.spyOn(console, "warn").mockImplementation(() => { });
-        const modules = ["documentation"];
-        resolveModuleNames(modules, false);
-        expect(consoleSpy).not.toHaveBeenCalled();
-        consoleSpy.mockRestore();
+        try {
+            const modules = ["documentation"];
+            resolveModuleNames(modules, false);
+            expect(consoleSpy).not.toHaveBeenCalled();
+        }
+        finally {
+            consoleSpy.mockRestore();
+        }
     });
 });
 describe("getProfileModules", () => {

package/cli/build/__tests__/security/security-pattern-count.test.js

@@ -22,6 +22,9 @@ const __filename = fileURLToPath(import.meta.url);
 const __dirname = path.dirname(__filename);
 const projectRoot = path.resolve(__dirname, "../../../.."); // From cli/src/__tests__/security to root
 describe("Security Pattern Count Consistency", () => {
+    afterEach(() => {
+        jest.clearAllMocks();
+    });
     describe("CLI assess-security references", () => {
         it("should use consistent pattern count in assess-security.ts", () => {
             const filePath = path.join(projectRoot, "cli/src/assess-security.ts");

package/cli/build/__tests__/stage3-fix-validation.test.js

@@ -9,6 +9,9 @@
 import { describe, it, expect } from "@jest/globals";
 import { AssessmentOptionsSchema, safeParseAssessmentOptions, validateAssessmentOptions, } from "../lib/cli-parserSchemas.js";
 describe("Stage 3 Fix Validation Tests", () => {
+    afterEach(() => {
+        jest.clearAllMocks();
+    });
     describe("[TEST-001] cli-parserSchemas.ts - stageBVerbose field (FIX-001)", () => {
         describe("stageBVerbose field validation", () => {
             it("should accept stageBVerbose with true value (happy path)", () => {

package/cli/build/__tests__/testbed-integration.test.js

@@ -26,9 +26,9 @@ const DEFAULT_HEADERS = {
  * Check if a server is available by sending an initialize request
  */
 async function checkServerAvailable(url) {
+    const controller = new AbortController();
+    const timeoutId = setTimeout(() => controller.abort(), 5000);
     try {
-        const controller = new AbortController();
-        const timeoutId = setTimeout(() => controller.abort(), 5000);
         const response = await fetch(url, {
             method: "POST",
             headers: DEFAULT_HEADERS,
@@ -44,12 +44,16 @@ async function checkServerAvailable(url) {
             }),
             signal: controller.signal,
         });
-        clearTimeout(timeoutId);
         return response.status < 500;
     }
     catch {
         return false;
     }
+    finally {
+        // Always clean up timeout and abort controller to prevent memory leaks
+        clearTimeout(timeoutId);
+        controller.abort();
+    }
 }
 /**
  * Parse SSE response to extract JSON data
@@ -120,6 +124,9 @@ async function callTool(url, toolName, args) {
     return data;
 }
 describe("Testbed A/B Comparison", () => {
+    afterEach(() => {
+        jest.clearAllMocks();
+    });
     let bothServersAvailable = false;
     let vulnerableAvailable = false;
     let hardenedAvailable = false;

package/cli/build/__tests__/transport.test.js

@@ -8,6 +8,9 @@
 import { describe, it, expect } from "@jest/globals";
 import { createTransport } from "../transport.js";
 describe("Transport Creation", () => {
+    afterEach(() => {
+        jest.clearAllMocks();
+    });
     describe("Input Validation", () => {
         it("should throw error when URL is missing for HTTP transport", () => {
             const options = {

package/cli/build/lib/__tests__/cli-parserSchemas.test.js

@@ -9,6 +9,9 @@
 import { ZodError } from "zod";
 import { AssessmentProfileNameSchema, AssessmentModuleNameSchema, ServerConfigSchema, AssessmentOptionsSchema, ValidationResultSchema, validateAssessmentOptions, validateServerConfig, parseAssessmentOptions, safeParseAssessmentOptions, parseModuleNames, safeParseModuleNames, LogLevelSchema, ReportFormatSchema, TransportTypeSchema, ZOD_SCHEMA_VERSION, } from "../cli-parserSchemas.js";
 describe("cli-parserSchemas", () => {
+    afterEach(() => {
+        jest.clearAllMocks();
+    });
     describe("Re-exported schemas", () => {
         test("exports ZOD_SCHEMA_VERSION", () => {
             expect(ZOD_SCHEMA_VERSION).toBe(1);

package/cli/build/lib/assessment-runner/__tests__/server-configSchemas.test.js

@@ -9,6 +9,9 @@
 import { ZodError } from "zod";
 import { HttpSseServerConfigSchema, StdioServerConfigSchema, ServerEntrySchema, ClaudeDesktopConfigSchema, StandaloneConfigSchema, ConfigFileSchema, parseConfigFile, safeParseConfigFile, validateServerEntry, isHttpSseConfig, isStdioConfig, TransportTypeSchema, } from "../server-configSchemas.js";
 describe("server-configSchemas", () => {
+    afterEach(() => {
+        jest.clearAllMocks();
+    });
     describe("Re-exported schemas", () => {
         test("exports TransportTypeSchema", () => {
             expect(TransportTypeSchema.safeParse("stdio").success).toBe(true);

package/cli/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bryan-thompson/inspector-assessment-cli",
-  "version": "1.35.1",
+  "version": "1.35.3",
   "description": "CLI for the Enhanced MCP Inspector with assessment capabilities",
   "license": "MIT",
   "author": "Bryan Thompson <bryan@triepod.ai>",

package/client/dist/assets/{OAuthCallback-CUf6mvrP.js → OAuthCallback-BglPlsRX.js}

@@ -1,4 +1,4 @@
-import { u as useToast, r as reactExports, j as jsxRuntimeExports, p as parseOAuthCallbackParams, g as generateOAuthErrorDescription, S as SESSION_KEYS, I as InspectorOAuthClientProvider, a as auth } from "./index-uinVACY4.js";
+import { u as useToast, r as reactExports, j as jsxRuntimeExports, p as parseOAuthCallbackParams, g as generateOAuthErrorDescription, S as SESSION_KEYS, I as InspectorOAuthClientProvider, a as auth } from "./index-iCeeS8vg.js";
 const OAuthCallback = ({ onConnect }) => {
   const { toast } = useToast();
   const hasProcessedRef = reactExports.useRef(false);

package/client/dist/assets/{OAuthDebugCallback-DkfPOggR.js → OAuthDebugCallback-CWrUH2KB.js}

@@ -1,4 +1,4 @@
-import { r as reactExports, S as SESSION_KEYS, p as parseOAuthCallbackParams, j as jsxRuntimeExports, g as generateOAuthErrorDescription } from "./index-uinVACY4.js";
+import { r as reactExports, S as SESSION_KEYS, p as parseOAuthCallbackParams, j as jsxRuntimeExports, g as generateOAuthErrorDescription } from "./index-iCeeS8vg.js";
 const OAuthDebugCallback = ({ onConnect }) => {
   reactExports.useEffect(() => {
     let isProcessed = false;

package/client/dist/assets/{index-uinVACY4.js → index-iCeeS8vg.js}

@@ -16373,7 +16373,7 @@ object({
   token_type_hint: string().optional()
 }).strip();
 const name = "@bryan-thompson/inspector-assessment-client";
-const version$1 = "1.35.1";
+const version$1 = "1.35.3";
 const packageJson = {
   name,
   version: version$1
@@ -48920,7 +48920,7 @@ const useTheme = () => {
     [theme, setThemeWithSideEffect]
   );
 };
-const version = "1.35.1";
+const version = "1.35.3";
 var [createTooltipContext] = createContextScope("Tooltip", [
   createPopperScope
 ]);
@@ -52515,13 +52515,13 @@ const App = () => {
   ) });
   if (window.location.pathname === "/oauth/callback") {
     const OAuthCallback = React.lazy(
-      () => __vitePreload(() => import("./OAuthCallback-CUf6mvrP.js"), true ? [] : void 0)
+      () => __vitePreload(() => import("./OAuthCallback-BglPlsRX.js"), true ? [] : void 0)
     );
     return /* @__PURE__ */ jsxRuntimeExports.jsx(reactExports.Suspense, { fallback: /* @__PURE__ */ jsxRuntimeExports.jsx("div", { children: "Loading..." }), children: /* @__PURE__ */ jsxRuntimeExports.jsx(OAuthCallback, { onConnect: onOAuthConnect }) });
   }
   if (window.location.pathname === "/oauth/callback/debug") {
     const OAuthDebugCallback = React.lazy(
-      () => __vitePreload(() => import("./OAuthDebugCallback-DkfPOggR.js"), true ? [] : void 0)
+      () => __vitePreload(() => import("./OAuthDebugCallback-CWrUH2KB.js"), true ? [] : void 0)
     );
     return /* @__PURE__ */ jsxRuntimeExports.jsx(reactExports.Suspense, { fallback: /* @__PURE__ */ jsxRuntimeExports.jsx("div", { children: "Loading..." }), children: /* @__PURE__ */ jsxRuntimeExports.jsx(OAuthDebugCallback, { onConnect: onOAuthDebugConnect }) });
   }

package/client/dist/index.html

@@ -5,7 +5,7 @@
     <link rel="icon" type="image/svg+xml" href="/mcp.svg" />
     <meta name="viewport" content="width=device-width, initial-scale=1.0" />
     <title>MCP Inspector</title>
-    <script type="module" crossorigin src="/assets/index-uinVACY4.js"></script>
+    <script type="module" crossorigin src="/assets/index-iCeeS8vg.js"></script>
     <link rel="stylesheet" crossorigin href="/assets/index-BoUA5OL1.css">
   </head>
   <body>

package/client/lib/lib/assessment/coreTypes.d.ts

@@ -280,4 +280,27 @@ export interface PackageJson {
 export type ToolInputSchema = JSONSchema7 & {
     type: "object";
 };
+/**
+ * Result of namespace/prefix detection for tool naming.
+ * Used to identify when tools share a common namespace (e.g., calc_add, calc_subtract).
+ * This helps downstream analyzers (like mcp-auditor) understand intentional naming patterns
+ * and reduces false positives for "naming conflicts".
+ * @public
+ */
+export interface NamespaceDetectionResult {
+    /** Whether a namespace was detected */
+    detected: boolean;
+    /** The detected namespace/prefix (e.g., "calc" from calc_add, calc_subtract) */
+    namespace?: string;
+    /** Confidence level of the detection */
+    confidence: "high" | "medium" | "low";
+    /** Number of tools that match this namespace */
+    toolsCovered: number;
+    /** Total number of tools analyzed */
+    totalTools: number;
+    /** How the namespace was detected */
+    matchPattern?: "prefix" | "serverName" | "none";
+    /** Sample tool names showing the pattern */
+    evidence?: string[];
+}
 //# sourceMappingURL=coreTypes.d.ts.map