cbrowser 19.1.4 → 19.1.6

package/dist/mcp-server.js

@@ -24,6 +24,7 @@ import { registerPersonaComparisonTools } from "./mcp-tools/base/persona-compari
 import { registerPersonaCreationTools } from "./mcp-tools/persona-creation-tools.js";
 import { registerPersonaLifecycleTools } from "./mcp-tools/persona-lifecycle-tools.js";
 import { registerUiResources } from "./mcp-tools/ui-resources.js";
+import { countRegisteredTools } from "./mcp-tools/base/browser-management-tools.js";
 import { applySecurityLayer } from "./mcp-tools/security-layer.js";
 // Visual module imports
 import { runVisualRegression, runCrossBrowserTest, runResponsiveTest, runABComparison, crossBrowserDiff, captureVisualBaseline, listVisualBaselines, } from "./visual/index.js";
@@ -31,7 +32,7 @@ import { runVisualRegression, runCrossBrowserTest, runResponsiveTest, runABCompa
 import { runNLTestSuite, parseNLTestSuite, dryRunNLTestSuite, repairTest, detectFlakyTests, generateCoverageMap, } from "./testing/index.js";
 // Analysis module imports
 import { huntBugs, runChaosTest, findElementByIntent, runAgentReadyAudit, runCompetitiveBenchmark, } from "./analysis/index.js";
-import { listAccessibilityPersonas, getAccessibilityPersona } from "./personas.js";
+import { listAccessibilityPersonas, getAccessibilityPersona, describeAccessibilityPersona } from "./personas.js";
 // Persona imports for cognitive journey
 import { getPersona, getAnyPersona, listPersonas, getCognitiveProfile, createCognitivePersona, listEmotionalPersonas, getEmotionalPersona, isAgentPersonaObject, } from "./personas.js";
 // Emotional state functions (v13.1.0)
@@ -665,6 +666,7 @@ async function registerCBrowserTools() {
         }
         catch (error) {
             return {
+                isError: true,
                 content: [{
                         type: "text",
                         text: JSON.stringify({
@@ -743,6 +745,7 @@ async function registerCBrowserTools() {
         }
         catch (error) {
             return {
+                isError: true,
                 content: [{
                         type: "text",
                         text: JSON.stringify({
@@ -762,6 +765,7 @@ async function registerCBrowserTools() {
         }
         catch (error) {
             return {
+                isError: true,
                 content: [{
                         type: "text",
                         text: JSON.stringify({
@@ -1129,7 +1133,7 @@ async function registerCBrowserTools() {
     }, async ({ filepath, dryRun, fuzzyMatch }) => {
         const fs = await import("fs");
         if (!fs.existsSync(filepath)) {
-            return { content: [{ type: "text", text: JSON.stringify({ error: `Test file not found: ${filepath}` }) }] };
+            return { isError: true, content: [{ type: "text", text: JSON.stringify({ error: `Test file not found: ${filepath}` }) }] };
         }
         const fileContent = fs.readFileSync(filepath, "utf-8");
         const suiteName = filepath.split("/").pop()?.replace(/\.[^.]+$/, "") || "Test Suite";
@@ -1486,6 +1490,7 @@ async function registerCBrowserTools() {
         // v17.0.0: Check for agent personas - cognitive journeys don't support them yet
         if (existingPersona && isAgentPersonaObject(existingPersona)) {
             return {
+                isError: true,
                 content: [{
                         type: "text",
                         text: JSON.stringify({
@@ -1836,43 +1841,11 @@ Begin the simulation now. Narrate your thoughts as this persona.
             const p = getAccessibilityPersona(name);
             if (!p)
                 return null;
-            // v16.11.0: Compute disabilityType and barrierTypes from accessibilityTraits
-            const traits = p.accessibilityTraits;
-            let disabilityType = "General accessibility";
-            const barrierTypes = [];
-            if (traits?.tremor) {
-                disabilityType = "Motor impairment (tremor)";
-                barrierTypes.push("motor_precision", "touch_target");
-            }
-            if (traits?.visionLevel !== undefined && traits.visionLevel < 0.5) {
-                disabilityType = "Low vision";
-                barrierTypes.push("visual_clarity", "contrast");
-            }
-            if (traits?.colorBlindness) {
-                disabilityType = `Color blindness (${traits.colorBlindness})`;
-                barrierTypes.push("sensory");
-            }
-            if (traits?.processingSpeed !== undefined && traits.processingSpeed < 0.6) {
-                disabilityType = "Cognitive (Processing)";
-                barrierTypes.push("cognitive_load", "temporal");
-            }
-            if (traits?.attentionSpan !== undefined && traits.attentionSpan < 0.5) {
-                if (!disabilityType.includes("Cognitive")) {
-                    disabilityType = "Cognitive (ADHD/Attention)";
-                }
-                barrierTypes.push("cognitive_load");
-            }
-            // Name-based fallback
-            if (disabilityType === "General accessibility") {
-                if (p.name.includes("deaf") || p.name.includes("hearing"))
-                    disabilityType = "Hearing impairment";
-                else if (p.name.includes("motor"))
-                    disabilityType = "Motor impairment";
-                else if (p.name.includes("vision") || p.name.includes("blind"))
-                    disabilityType = "Vision impairment";
-                else if (p.name.includes("cognitive") || p.name.includes("adhd"))
-                    disabilityType = "Cognitive";
-            }
+            // Shared with the remote roster (personas.ts); a declared label wins
+            // over inference. This copy and that one were identical inference
+            // blocks, so a fix to one could only ever reach half the users.
+            // (2026-10-07)
+            const { disabilityType, barrierTypes } = describeAccessibilityPersona(p);
             // v16.12.0: Include Schwartz values for accessibility personas
             const values = resolveValuesForPersona(p.name);
             return {
@@ -1882,7 +1855,7 @@ Begin the simulation now. Narrate your thoughts as this persona.
                 disabilityType,
                 demographics: p.demographics,
                 cognitiveTraits: p.cognitiveTraits || {},
-                barrierTypes: [...new Set(barrierTypes)], // Deduplicate
+                barrierTypes,
                 values: values ? {
                     schwartz: {
                         selfDirection: values.selfDirection,
@@ -1966,6 +1939,7 @@ Begin the simulation now. Narrate your thoughts as this persona.
         const session = comparisonSessions.get(sessionId);
         if (!session) {
             return {
+                isError: true,
                 content: [{ type: "text", text: JSON.stringify({ error: "Session not found", sessionId }) }],
             };
         }
@@ -2016,11 +1990,13 @@ Begin the simulation now. Narrate your thoughts as this persona.
         const session = comparisonSessions.get(sessionId);
         if (!session) {
             return {
+                isError: true,
                 content: [{ type: "text", text: JSON.stringify({ error: "Session not found", sessionId }) }],
             };
         }
         if (session.results.length === 0) {
             return {
+                isError: true,
                 content: [{ type: "text", text: JSON.stringify({ error: "No results recorded yet", sessionId }) }],
             };
         }
@@ -2137,8 +2113,11 @@ Begin the simulation now. Narrate your thoughts as this persona.
                 accessibilityTraits: persona.accessibilityTraits,
                 cognitiveTraits: persona.cognitiveTraits,
             };
-            // Compute disability type from traits
-            sessionPersona.disabilityType = getDisabilityTypeFromPersona(sessionPersona);
+            // A declared label wins; traits are inferred only for personas that
+            // declare none. Inferring here gave dyslexic-user "Cognitive
+            // (ADHD/Memory)" in this session and "Cognitive (Processing)" in the
+            // roster. (2026-10-07)
+            sessionPersona.disabilityType = persona.disabilityType ?? getDisabilityTypeFromPersona(sessionPersona);
             return sessionPersona;
         });
         // Store session
@@ -2198,6 +2177,7 @@ Begin the simulation now. Narrate your thoughts as this persona.
         const session = empathyAuditSessions.get(sessionId);
         if (!session) {
             return {
+                isError: true,
                 content: [{ type: "text", text: JSON.stringify({ error: "Session not found. Call empathy_audit_init first." }) }],
             };
         }
@@ -2247,6 +2227,7 @@ Begin the simulation now. Narrate your thoughts as this persona.
         const session = empathyAuditSessions.get(sessionId);
         if (!session) {
             return {
+                isError: true,
                 content: [{ type: "text", text: JSON.stringify({ error: "Session not found." }) }],
             };
         }
@@ -2301,11 +2282,13 @@ Begin the simulation now. Narrate your thoughts as this persona.
         const session = empathyAuditSessions.get(sessionId);
         if (!session) {
             return {
+                isError: true,
                 content: [{ type: "text", text: JSON.stringify({ error: "Session not found." }) }],
             };
         }
         if (session.personaResults.length === 0) {
             return {
+                isError: true,
                 content: [{ type: "text", text: JSON.stringify({ error: "No persona results recorded. Complete at least one persona journey first." }) }],
             };
         }
@@ -2562,8 +2545,12 @@ Begin the simulation now. Narrate your thoughts as this persona.
     // Diagnostics Tools
     // =========================================================================
     server.tool("status", "Get CBrowser environment status and diagnostics including data directories, installed browsers, configuration, self-healing cache statistics, and MCP tool count", {}, async () => {
-        // v18.22.0: Include tool count for self-diagnosis of tool discrepancies
-        const info = await getStatusInfo(VERSION, collectedTools.length);
+        // v18.22.0: Include tool count for self-diagnosis of tool discrepancies.
+        // collectedTools sees only server.tool(...) calls, so every family this
+        // server registers through registerTool (empathy_audit, values, persona
+        // comparison/creation/lifecycle) was missing: 55 reported, 82 listed.
+        // Count what tools/list actually serves. (2026-10-07)
+        const info = await getStatusInfo(VERSION, countRegisteredTools(server) ?? collectedTools.length);
         return {
             content: [
                 {
@@ -2630,6 +2617,7 @@ Begin the simulation now. Narrate your thoughts as this persona.
         const personaData = getPersona(persona);
         if (!personaData) {
             return {
+                isError: true,
                 content: [{ type: "text", text: JSON.stringify({ error: `Persona not found: ${persona}` }) }],
             };
         }
@@ -2663,6 +2651,7 @@ Begin the simulation now. Narrate your thoughts as this persona.
         const personaData = getPersona(persona);
         if (!personaData) {
             return {
+                isError: true,
                 content: [{ type: "text", text: JSON.stringify({ error: `Persona not found: ${persona}` }) }],
             };
         }