@bryan-thompson/inspector-assessment-client 1.25.4 → 1.25.6

package/lib/services/assessment/modules/DeveloperExperienceAssessor.js

@@ -16,20 +16,28 @@ import { BaseAssessor } from "./BaseAssessor.js";
 export class DeveloperExperienceAssessor extends BaseAssessor {
     async assess(context) {
         this.log("Starting developer experience assessment");
+        const readmeContent = context.readmeContent || "";
         // Assess documentation
-        const documentationMetrics = this.analyzeDocumentation(context.readmeContent || "", context.tools, "verbose");
-        const documentationScore = this.calculateDocumentationScore(documentationMetrics);
+        const documentationMetrics = this.analyzeDocumentation(readmeContent, context.tools, "verbose");
+        // Issue #55: Add quality scoring
+        const { checks: qualityChecks, score: qualityScore } = this.assessDocumentationQuality(readmeContent, context);
+        // Add quality data to metrics
+        documentationMetrics.qualityChecks = qualityChecks;
+        documentationMetrics.qualityScore = qualityScore;
+        documentationMetrics.readmeSizeBytes = Buffer.byteLength(readmeContent, "utf8");
+        // Use quality score for documentation scoring (Issue #55)
+        const documentationScore = qualityScore.total;
         // Assess usability
         const usabilityMetrics = this.analyzeUsability(context.tools);
         const usabilityScore = this.calculateUsabilityScore(usabilityMetrics);
-        // Calculate overall score (weighted average)
+        // Calculate overall score (weighted average: 60% docs, 40% usability)
         const overallScore = Math.round(documentationScore * 0.6 + usabilityScore * 0.4);
-        // Determine status
+        // Determine status using Issue #55 thresholds
         const status = this.determineOverallStatus(overallScore);
         // Generate explanation and recommendations
         const explanation = this.generateExplanation(documentationMetrics, usabilityMetrics, context.tools);
         const recommendations = this.generateRecommendations(documentationMetrics, usabilityMetrics);
-        this.testCount = 9; // Documentation (5) + Usability (4) checks
+        this.testCount = 15; // Documentation (5) + Quality (6) + Usability (4) checks
         return {
             documentation: documentationMetrics,
             usability: usabilityMetrics,
@@ -321,20 +329,178 @@ export class DeveloperExperienceAssessor extends BaseAssessor {
         }
         return "functional";
     }
-    calculateDocumentationScore(metrics) {
-        let score = 0;
-        const maxScore = 5;
-        if (metrics.hasReadme)
-            score++;
-        if (metrics.hasInstallInstructions)
-            score++;
-        if (metrics.hasUsageGuide)
-            score++;
-        if (metrics.hasAPIReference)
-            score++;
-        if (metrics.exampleCount >= metrics.requiredExamples)
-            score++;
-        return Math.round((score / maxScore) * 100);
+    // ============================================================================
+    // Issue #55: Documentation Quality Scoring
+    // ============================================================================
+    /**
+     * Assess documentation quality using Issue #55 point-based scoring
+     * Max 100 points: README (30), Install (20), Config (20), Examples (20), License (10)
+     */
+    assessDocumentationQuality(content, context) {
+        const checks = {
+            hasReadme: content.length > 0,
+            readmeQuality: this.determineReadmeQuality(content),
+            hasInstallation: this.checkInstallInstructions(content),
+            hasConfiguration: this.checkConfigurationSection(content),
+            hasExamples: this.checkUsageGuide(content),
+            hasLicense: this.detectLicense(context),
+            licenseType: this.detectLicenseType(context),
+        };
+        const score = this.calculateQualityScore(checks, content);
+        return { checks, score };
+    }
+    /**
+     * Determine README quality tier based on size
+     * - minimal: <5KB
+     * - adequate: 5KB-15KB
+     * - comprehensive: >15KB
+     */
+    determineReadmeQuality(content) {
+        const sizeBytes = Buffer.byteLength(content, "utf8");
+        const sizeKB = sizeBytes / 1024;
+        if (sizeKB > 15)
+            return "comprehensive";
+        if (sizeKB > 5)
+            return "adequate";
+        return "minimal";
+    }
+    /**
+     * Calculate point-based quality score per Issue #55
+     * Max 100 points:
+     * - README exists: +10
+     * - README >5KB: +10 (adequate)
+     * - README >15KB: +10 more (comprehensive = +20 total)
+     * - Installation section: +20
+     * - Configuration section: +20
+     * - Examples present: +20
+     * - License file: +10
+     */
+    calculateQualityScore(checks, content) {
+        const sizeBytes = Buffer.byteLength(content, "utf8");
+        const sizeKB = sizeBytes / 1024;
+        // Calculate README size bonus
+        let readmeComprehensive = 0;
+        if (checks.hasReadme) {
+            if (sizeKB > 15) {
+                readmeComprehensive = 20; // comprehensive: +10 + +10
+            }
+            else if (sizeKB > 5) {
+                readmeComprehensive = 10; // adequate: +10
+            }
+        }
+        const breakdown = {
+            readmeExists: checks.hasReadme ? 10 : 0,
+            readmeComprehensive,
+            installation: checks.hasInstallation ? 20 : 0,
+            configuration: checks.hasConfiguration ? 20 : 0,
+            examples: checks.hasExamples ? 20 : 0,
+            license: checks.hasLicense ? 10 : 0,
+        };
+        return {
+            total: Object.values(breakdown).reduce((sum, v) => sum + v, 0),
+            breakdown,
+        };
+    }
+    /**
+     * Check for configuration/environment section
+     * Looks for: configuration, config, environment, env vars, .env
+     */
+    checkConfigurationSection(content) {
+        const configKeywords = [
+            "configuration",
+            "config",
+            "environment variable",
+            "env var",
+            ".env",
+            "api key",
+            "api_key",
+            "apikey",
+            "setup",
+        ];
+        const contentLower = content.toLowerCase();
+        return configKeywords.some((keyword) => contentLower.includes(keyword));
+    }
+    /**
+     * Detect license presence from context
+     * Checks sourceCodeFiles for LICENSE/LICENSE.md or README for license section
+     */
+    detectLicense(context) {
+        // Check source code files if available
+        if (context.sourceCodeFiles) {
+            const licenseFiles = [
+                "LICENSE",
+                "LICENSE.md",
+                "LICENSE.txt",
+                "LICENCE",
+                "LICENCE.md",
+            ];
+            for (const file of licenseFiles) {
+                if (context.sourceCodeFiles.has(file))
+                    return true;
+            }
+        }
+        // Fallback: check README for license section
+        const content = context.readmeContent || "";
+        return /^#+\s*licen[sc]e/im.test(content);
+    }
+    /**
+     * Detect license type (MIT, Apache-2.0, GPL, BSD, etc.)
+     */
+    detectLicenseType(context) {
+        if (!context.sourceCodeFiles)
+            return undefined;
+        // Try common license file names
+        const licenseFiles = [
+            "LICENSE",
+            "LICENSE.md",
+            "LICENSE.txt",
+            "LICENCE",
+            "LICENCE.md",
+        ];
+        let licenseContent;
+        for (const file of licenseFiles) {
+            if (context.sourceCodeFiles.has(file)) {
+                licenseContent = context.sourceCodeFiles.get(file);
+                break;
+            }
+        }
+        if (!licenseContent)
+            return undefined;
+        // Simple license detection patterns
+        if (licenseContent.includes("MIT License") ||
+            licenseContent.includes("Permission is hereby granted, free of charge")) {
+            return "MIT";
+        }
+        if (licenseContent.includes("Apache License") &&
+            licenseContent.includes("2.0")) {
+            return "Apache-2.0";
+        }
+        if (licenseContent.includes("GNU GENERAL PUBLIC LICENSE")) {
+            if (licenseContent.includes("Version 3"))
+                return "GPL-3.0";
+            if (licenseContent.includes("Version 2"))
+                return "GPL-2.0";
+            return "GPL";
+        }
+        if (licenseContent.includes("BSD")) {
+            if (licenseContent.includes("3-Clause") || licenseContent.includes("New"))
+                return "BSD-3-Clause";
+            if (licenseContent.includes("2-Clause") ||
+                licenseContent.includes("Simplified"))
+                return "BSD-2-Clause";
+            return "BSD";
+        }
+        if (licenseContent.includes("ISC License")) {
+            return "ISC";
+        }
+        if (licenseContent.includes("Mozilla Public License")) {
+            return "MPL-2.0";
+        }
+        if (licenseContent.includes("UNLICENSE") ||
+            licenseContent.includes("unlicense")) {
+            return "Unlicense";
+        }
+        return "Unknown";
     }
     // ============================================================================
     // Usability Analysis (from UsabilityAssessor)

package/lib/services/assessment/modules/DocumentationAssessor.d.ts

@@ -3,9 +3,14 @@
  * Evaluates documentation quality and completeness
  */
 import { DocumentationAssessment } from "../../../lib/assessmentTypes.js";
+import { AssessmentConfiguration } from "../../../lib/assessment/configTypes.js";
 import { BaseAssessor } from "./BaseAssessor.js";
 import { AssessmentContext } from "../AssessmentOrchestrator.js";
+/**
+ * @deprecated Use DeveloperExperienceAssessor instead. Will be removed in v2.0.0.
+ */
 export declare class DocumentationAssessor extends BaseAssessor {
+    constructor(config: AssessmentConfiguration);
     assess(context: AssessmentContext): Promise<DocumentationAssessment>;
     private analyzeDocumentation;
     /**

package/lib/services/assessment/modules/DocumentationAssessor.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"DocumentationAssessor.d.ts","sourceRoot":"","sources":["../../../../src/services/assessment/modules/DocumentationAssessor.ts"],"names":[],"mappings":"AAAA;;;GAGG;AAEH,OAAO,EACL,uBAAuB,EAKxB,MAAM,uBAAuB,CAAC;AAC/B,OAAO,EAAE,YAAY,EAAE,MAAM,gBAAgB,CAAC;AAC9C,OAAO,EAAE,iBAAiB,EAAE,MAAM,2BAA2B,CAAC;AAE9D,qBAAa,qBAAsB,SAAQ,YAAY;IAC/C,MAAM,CAAC,OAAO,EAAE,iBAAiB,GAAG,OAAO,CAAC,uBAAuB,CAAC;IAyC1E,OAAO,CAAC,oBAAoB;IAuJ5B;;;OAGG;IACH,OAAO,CAAC,yBAAyB;IAuEjC;;OAEG;IACH,OAAO,CAAC,wBAAwB;IAiBhC;;OAEG;IACH,OAAO,CAAC,sBAAsB;IAmC9B;;OAEG;IACH,OAAO,CAAC,aAAa;IAKrB;;OAEG;IACH,OAAO,CAAC,mBAAmB;IAqB3B;;;OAGG;IACH,OAAO,CAAC,mBAAmB;IA4C3B,OAAO,CAAC,wBAAwB;IAchC,OAAO,CAAC,eAAe;IAavB,OAAO,CAAC,iBAAiB;IAezB,OAAO,CAAC,cAAc;IAUtB;;;OAGG;IACH,OAAO,CAAC,sBAAsB;IAY9B;;;OAGG;IACH,OAAO,CAAC,mBAAmB;IA4B3B,OAAO,CAAC,4BAA4B;IAmBpC,OAAO,CAAC,mBAAmB;IAyB3B,OAAO,CAAC,uBAAuB;CA+BhC"}
+{"version":3,"file":"DocumentationAssessor.d.ts","sourceRoot":"","sources":["../../../../src/services/assessment/modules/DocumentationAssessor.ts"],"names":[],"mappings":"AAAA;;;GAGG;AAEH,OAAO,EACL,uBAAuB,EAKxB,MAAM,uBAAuB,CAAC;AAC/B,OAAO,EAAE,uBAAuB,EAAE,MAAM,8BAA8B,CAAC;AACvE,OAAO,EAAE,YAAY,EAAE,MAAM,gBAAgB,CAAC;AAC9C,OAAO,EAAE,iBAAiB,EAAE,MAAM,2BAA2B,CAAC;AAE9D;;GAEG;AACH,qBAAa,qBAAsB,SAAQ,YAAY;gBACzC,MAAM,EAAE,uBAAuB;IAYrC,MAAM,CAAC,OAAO,EAAE,iBAAiB,GAAG,OAAO,CAAC,uBAAuB,CAAC;IAyC1E,OAAO,CAAC,oBAAoB;IAuJ5B;;;OAGG;IACH,OAAO,CAAC,yBAAyB;IAuEjC;;OAEG;IACH,OAAO,CAAC,wBAAwB;IAiBhC;;OAEG;IACH,OAAO,CAAC,sBAAsB;IAmC9B;;OAEG;IACH,OAAO,CAAC,aAAa;IAKrB;;OAEG;IACH,OAAO,CAAC,mBAAmB;IAqB3B;;;OAGG;IACH,OAAO,CAAC,mBAAmB;IA4C3B,OAAO,CAAC,wBAAwB;IAchC,OAAO,CAAC,eAAe;IAavB,OAAO,CAAC,iBAAiB;IAezB,OAAO,CAAC,cAAc;IAUtB;;;OAGG;IACH,OAAO,CAAC,sBAAsB;IAY9B;;;OAGG;IACH,OAAO,CAAC,mBAAmB;IA4B3B,OAAO,CAAC,4BAA4B;IAmBpC,OAAO,CAAC,mBAAmB;IAyB3B,OAAO,CAAC,uBAAuB;CA+BhC"}

package/lib/services/assessment/modules/DocumentationAssessor.js

@@ -3,7 +3,18 @@
  * Evaluates documentation quality and completeness
  */
 import { BaseAssessor } from "./BaseAssessor.js";
+/**
+ * @deprecated Use DeveloperExperienceAssessor instead. Will be removed in v2.0.0.
+ */
 export class DocumentationAssessor extends BaseAssessor {
+    constructor(config) {
+        super(config);
+        this.logger.warn("DocumentationAssessor is deprecated. Use DeveloperExperienceAssessor instead. " +
+            "This module will be removed in v2.0.0.", {
+            module: "DocumentationAssessor",
+            replacement: "DeveloperExperienceAssessor",
+        });
+    }
     async assess(context) {
         this.log("Starting documentation assessment");
         const readmeContent = context.readmeContent || "";

package/lib/services/assessment/modules/ErrorHandlingAssessor.js

@@ -13,7 +13,7 @@ export class ErrorHandlingAssessor extends BaseAssessor {
         const toolsToTest = this.selectToolsForTesting(context.tools);
         // Parallel tool testing with concurrency limit
         const concurrency = this.config.maxParallelTests ?? 5;
-        const limit = createConcurrencyLimit(concurrency);
+        const limit = createConcurrencyLimit(concurrency, this.logger);
         this.log(`Testing ${toolsToTest.length} tools for error handling with concurrency limit of ${concurrency}`);
         const allToolTests = await Promise.all(toolsToTest.map((tool) => limit(async () => {
             const toolTests = await this.testToolErrorHandling(tool, context.callTool);

package/lib/services/assessment/modules/FunctionalityAssessor.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"FunctionalityAssessor.d.ts","sourceRoot":"","sources":["../../../../src/services/assessment/modules/FunctionalityAssessor.ts"],"names":[],"mappings":"AAAA;;;GAGG;AAEH,OAAO,EACL,uBAAuB,EAGxB,MAAM,uBAAuB,CAAC;AAC/B,OAAO,EAAE,YAAY,EAAE,MAAM,gBAAgB,CAAC;AAC9C,OAAO,EAAE,iBAAiB,EAAE,MAAM,2BAA2B,CAAC;AAS9D,qBAAa,qBAAsB,SAAQ,YAAY;IACrD,OAAO,CAAC,cAAc,CAAwB;IAE9C;;OAEG;IACH,OAAO,CAAC,qBAAqB;IAoCvB,MAAM,CAAC,OAAO,EAAE,iBAAiB,GAAG,OAAO,CAAC,uBAAuB,CAAC;YAkI5D,QAAQ;IAgGtB,OAAO,CAAC,qBAAqB;IAmE7B,OAAO,CAAC,kBAAkB;IA4G1B;;;OAGG;IACH,OAAO,CAAC,MAAM,CAAC,QAAQ,CAAC,uBAAuB,CAe7C;IAEF;;;OAGG;IACH,OAAO,CAAC,mCAAmC;IAsF3C;;OAEG;IACH,OAAO,CAAC,iBAAiB;IAWlB,iBAAiB,CAAC,MAAM,EAAE,GAAG,GAAG,OAAO;IAI9C,OAAO,CAAC,mBAAmB;CA+B5B"}
+{"version":3,"file":"FunctionalityAssessor.d.ts","sourceRoot":"","sources":["../../../../src/services/assessment/modules/FunctionalityAssessor.ts"],"names":[],"mappings":"AAAA;;;GAGG;AAEH,OAAO,EACL,uBAAuB,EAGxB,MAAM,uBAAuB,CAAC;AAC/B,OAAO,EAAE,YAAY,EAAE,MAAM,gBAAgB,CAAC;AAC9C,OAAO,EAAE,iBAAiB,EAAE,MAAM,2BAA2B,CAAC;AAU9D,qBAAa,qBAAsB,SAAQ,YAAY;IACrD,OAAO,CAAC,cAAc,CAAwB;IAE9C;;OAEG;IACH,OAAO,CAAC,qBAAqB;IAoCvB,MAAM,CAAC,OAAO,EAAE,iBAAiB,GAAG,OAAO,CAAC,uBAAuB,CAAC;YAmI5D,QAAQ;IAiGtB,OAAO,CAAC,qBAAqB;IAmE7B,OAAO,CAAC,kBAAkB;IA4G1B;;;OAGG;IACH,OAAO,CAAC,MAAM,CAAC,QAAQ,CAAC,uBAAuB,CAe7C;IAEF;;;OAGG;IACH,OAAO,CAAC,mCAAmC;IAsF3C;;OAEG;IACH,OAAO,CAAC,iBAAiB;IAWlB,iBAAiB,CAAC,MAAM,EAAE,GAAG,GAAG,OAAO;IAI9C,OAAO,CAAC,mBAAmB;CA+B5B"}

package/lib/services/assessment/modules/FunctionalityAssessor.js

@@ -9,6 +9,7 @@ import { ToolClassifier, ToolCategory } from "../ToolClassifier.js";
 import { TestDataGenerator } from "../TestDataGenerator.js";
 import { cleanParams } from "../../../utils/paramUtils.js";
 import { resolveRef, normalizeUnionType } from "../../../utils/schemaUtils.js";
+import { DEFAULT_PERFORMANCE_CONFIG } from "../config/performanceConfig.js";
 export class FunctionalityAssessor extends BaseAssessor {
     toolClassifier = new ToolClassifier();
     /**
@@ -46,14 +47,15 @@ export class FunctionalityAssessor extends BaseAssessor {
         const toolsToTest = this.selectToolsForTesting(context.tools);
         // Parallel tool testing with concurrency limit
         const concurrency = this.config.maxParallelTests ?? 5;
-        const limit = createConcurrencyLimit(concurrency);
+        const limit = createConcurrencyLimit(concurrency, this.logger);
         // Progress tracking for batched events
+        // Uses centralized PerformanceConfig values (Issue #37)
         const totalEstimate = toolsToTest.length;
         let completedTests = 0;
         let lastBatchTime = Date.now();
         const startTime = Date.now();
-        const BATCH_INTERVAL = 500;
-        const BATCH_SIZE = 5; // Smaller batch for functionality (fewer tests)
+        const BATCH_INTERVAL = DEFAULT_PERFORMANCE_CONFIG.batchFlushIntervalMs;
+        const BATCH_SIZE = DEFAULT_PERFORMANCE_CONFIG.functionalityBatchSize;
         let batchCount = 0;
         const emitProgressBatch = () => {
             if (context.onProgress) {
@@ -195,6 +197,7 @@ export class FunctionalityAssessor extends BaseAssessor {
             };
         }
         catch (error) {
+            this.logError(`Tool execution failed: ${tool.name}`, error);
             return {
                 toolName: tool.name,
                 tested: true,

package/lib/services/assessment/modules/MCPSpecComplianceAssessor.d.ts

@@ -5,6 +5,9 @@
 import { MCPSpecComplianceAssessment, AssessmentConfiguration } from "../../../lib/assessmentTypes.js";
 import { BaseAssessor } from "./BaseAssessor.js";
 import { AssessmentContext } from "../AssessmentOrchestrator.js";
+/**
+ * @deprecated Use ProtocolComplianceAssessor instead. Will be removed in v2.0.0.
+ */
 export declare class MCPSpecComplianceAssessor extends BaseAssessor {
     private ajv;
     constructor(config: AssessmentConfiguration);

package/lib/services/assessment/modules/MCPSpecComplianceAssessor.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"MCPSpecComplianceAssessor.d.ts","sourceRoot":"","sources":["../../../../src/services/assessment/modules/MCPSpecComplianceAssessor.ts"],"names":[],"mappings":"AAAA;;;GAGG;AAEH,OAAO,EACL,2BAA2B,EAM3B,uBAAuB,EAGxB,MAAM,uBAAuB,CAAC;AAO/B,OAAO,EAAE,YAAY,EAAE,MAAM,gBAAgB,CAAC;AAC9C,OAAO,EAAE,iBAAiB,EAAE,MAAM,2BAA2B,CAAC;AAE9D,qBAAa,yBAA0B,SAAQ,YAAY;IACzD,OAAO,CAAC,GAAG,CAAc;gBAEb,MAAM,EAAE,uBAAuB;IAK3C;;;OAGG;IACG,MAAM,CACV,OAAO,EAAE,iBAAiB,GACzB,OAAO,CAAC,2BAA2B,CAAC;IAmHvC;;OAEG;IACH,OAAO,CAAC,sBAAsB;IAwB9B;;;OAGG;YACW,sBAAsB;IA6BpC;;OAEG;IACH,OAAO,CAAC,uBAAuB;IAyB/B;;OAEG;IACH,OAAO,CAAC,qBAAqB;IAyC7B;;;OAGG;YACW,mBAAmB;IAsCjC;;OAEG;IACH,OAAO,CAAC,4BAA4B;IAiBpC;;;OAGG;IACH,OAAO,CAAC,2BAA2B;IA0FnC;;OAEG;IACH,OAAO,CAAC,yBAAyB;IAyFjC;;OAEG;IACH,OAAO,CAAC,uBAAuB;IAsB/B;;OAEG;IACH,OAAO,CAAC,sBAAsB;IA4B9B;;OAEG;IACH,OAAO,CAAC,qBAAqB;IA2C7B;;;OAGG;IACH,OAAO,CAAC,oBAAoB;IAoF5B;;OAEG;IACH,OAAO,CAAC,yBAAyB;IAyBjC;;OAEG;IACH,OAAO,CAAC,6BAA6B;CA0DtC"}
+{"version":3,"file":"MCPSpecComplianceAssessor.d.ts","sourceRoot":"","sources":["../../../../src/services/assessment/modules/MCPSpecComplianceAssessor.ts"],"names":[],"mappings":"AAAA;;;GAGG;AAEH,OAAO,EACL,2BAA2B,EAM3B,uBAAuB,EAGxB,MAAM,uBAAuB,CAAC;AAO/B,OAAO,EAAE,YAAY,EAAE,MAAM,gBAAgB,CAAC;AAC9C,OAAO,EAAE,iBAAiB,EAAE,MAAM,2BAA2B,CAAC;AAE9D;;GAEG;AACH,qBAAa,yBAA0B,SAAQ,YAAY;IACzD,OAAO,CAAC,GAAG,CAAc;gBAEb,MAAM,EAAE,uBAAuB;IAa3C;;;OAGG;IACG,MAAM,CACV,OAAO,EAAE,iBAAiB,GACzB,OAAO,CAAC,2BAA2B,CAAC;IAmHvC;;OAEG;IACH,OAAO,CAAC,sBAAsB;IAwB9B;;;OAGG;YACW,sBAAsB;IA6BpC;;OAEG;IACH,OAAO,CAAC,uBAAuB;IAyB/B;;OAEG;IACH,OAAO,CAAC,qBAAqB;IA0C7B;;;OAGG;YACW,mBAAmB;IAsCjC;;OAEG;IACH,OAAO,CAAC,4BAA4B;IAiBpC;;;OAGG;IACH,OAAO,CAAC,2BAA2B;IA0FnC;;OAEG;IACH,OAAO,CAAC,yBAAyB;IAyFjC;;OAEG;IACH,OAAO,CAAC,uBAAuB;IAsB/B;;OAEG;IACH,OAAO,CAAC,sBAAsB;IA4B9B;;OAEG;IACH,OAAO,CAAC,qBAAqB;IA2C7B;;;OAGG;IACH,OAAO,CAAC,oBAAoB;IAoF5B;;OAEG;IACH,OAAO,CAAC,yBAAyB;IAyBjC;;OAEG;IACH,OAAO,CAAC,6BAA6B;CA0DtC"}

package/lib/services/assessment/modules/MCPSpecComplianceAssessor.js

@@ -4,10 +4,18 @@
  */
 import Ajv from "ajv";
 import { BaseAssessor } from "./BaseAssessor.js";
+/**
+ * @deprecated Use ProtocolComplianceAssessor instead. Will be removed in v2.0.0.
+ */
 export class MCPSpecComplianceAssessor extends BaseAssessor {
     ajv;
     constructor(config) {
         super(config);
+        this.logger.warn("MCPSpecComplianceAssessor is deprecated. Use ProtocolComplianceAssessor instead. " +
+            "This module will be removed in v2.0.0.", {
+            module: "MCPSpecComplianceAssessor",
+            replacement: "ProtocolComplianceAssessor",
+        });
         this.ajv = new Ajv({ allErrors: true });
     }
     /**
@@ -198,7 +206,9 @@ export class MCPSpecComplianceAssessor extends BaseAssessor {
                         hasErrors = true;
                         const errorMsg = `${tool.name}: ${JSON.stringify(this.ajv.errors)}`;
                         errors.push(errorMsg);
-                        console.warn(`Invalid schema for tool ${tool.name}:`, this.ajv.errors);
+                        this.logger.warn(`Invalid schema for tool ${tool.name}`, {
+                            errors: this.ajv.errors,
+                        });
                     }
                 }
             }
@@ -210,7 +220,9 @@ export class MCPSpecComplianceAssessor extends BaseAssessor {
             };
         }
         catch (error) {
-            console.error("Schema compliance check failed:", error);
+            this.logger.error("Schema compliance check failed", {
+                error: String(error),
+            });
             return {
                 passed: false,
                 confidence: "low",

package/lib/services/assessment/modules/ManifestValidationAssessor.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"ManifestValidationAssessor.d.ts","sourceRoot":"","sources":["../../../../src/services/assessment/modules/ManifestValidationAssessor.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;GAWG;AAEH,OAAO,EAAE,YAAY,EAAE,MAAM,gBAAgB,CAAC;AAC9C,OAAO,EAAE,iBAAiB,EAAE,MAAM,2BAA2B,CAAC;AAC9D,OAAO,KAAK,EACV,4BAA4B,EAK7B,MAAM,uBAAuB,CAAC;AAM/B,qBAAa,0BAA2B,SAAQ,YAAY;IAC1D;;OAEG;IACG,MAAM,CACV,OAAO,EAAE,iBAAiB,GACzB,OAAO,CAAC,4BAA4B,CAAC;IA6JxC;;OAEG;IACH,OAAO,CAAC,sBAAsB;IAyB9B;;OAEG;IACH,OAAO,CAAC,uBAAuB;IAmB/B;;OAEG;IACH,OAAO,CAAC,uBAAuB;IAgC/B;;OAEG;IACH,OAAO,CAAC,qBAAqB;IAiC7B;;OAEG;IACH,OAAO,CAAC,wBAAwB;IAiChC;;OAEG;IACH,OAAO,CAAC,iBAAiB;IA+CzB;;OAEG;IACH,OAAO,CAAC,YAAY;IAqCpB;;OAEG;IACH,OAAO,CAAC,kBAAkB;IA+B1B;;OAEG;IACH,OAAO,CAAC,qBAAqB;IA8B7B;;OAEG;YACW,yBAAyB;IA2EvC;;OAEG;IACH,OAAO,CAAC,uBAAuB;IAsB/B;;OAEG;IACH,OAAO,CAAC,mBAAmB;IA0C3B;;OAEG;IACH,OAAO,CAAC,uBAAuB;CA+ChC"}
+{"version":3,"file":"ManifestValidationAssessor.d.ts","sourceRoot":"","sources":["../../../../src/services/assessment/modules/ManifestValidationAssessor.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;GAWG;AAEH,OAAO,EAAE,YAAY,EAAE,MAAM,gBAAgB,CAAC;AAC9C,OAAO,EAAE,iBAAiB,EAAE,MAAM,2BAA2B,CAAC;AAC9D,OAAO,KAAK,EACV,4BAA4B,EAK7B,MAAM,uBAAuB,CAAC;AAM/B,qBAAa,0BAA2B,SAAQ,YAAY;IAC1D;;OAEG;IACG,MAAM,CACV,OAAO,EAAE,iBAAiB,GACzB,OAAO,CAAC,4BAA4B,CAAC;IA6JxC;;OAEG;IACH,OAAO,CAAC,sBAAsB;IAyB9B;;OAEG;IACH,OAAO,CAAC,uBAAuB;IAmB/B;;OAEG;IACH,OAAO,CAAC,uBAAuB;IAgC/B;;OAEG;IACH,OAAO,CAAC,qBAAqB;IAiC7B;;OAEG;IACH,OAAO,CAAC,wBAAwB;IAiChC;;OAEG;IACH,OAAO,CAAC,iBAAiB;IA+CzB;;OAEG;IACH,OAAO,CAAC,YAAY;IAqCpB;;OAEG;IACH,OAAO,CAAC,kBAAkB;IA+B1B;;OAEG;IACH,OAAO,CAAC,qBAAqB;IA8B7B;;OAEG;YACW,yBAAyB;IAoFvC;;OAEG;IACH,OAAO,CAAC,uBAAuB;IAsB/B;;OAEG;IACH,OAAO,CAAC,mBAAmB;IA0C3B;;OAEG;IACH,OAAO,CAAC,uBAAuB;CA+ChC"}

package/lib/services/assessment/modules/ManifestValidationAssessor.js

@@ -412,7 +412,8 @@ export class ManifestValidationAssessor extends BaseAssessor {
             try {
                 new URL(url);
             }
-            catch {
+            catch (error) {
+                this.logError(`Invalid privacy policy URL format: ${url}`, error);
                 results.push({
                     url,
                     accessible: false,
@@ -437,8 +438,11 @@ export class ManifestValidationAssessor extends BaseAssessor {
                     contentType: response.headers.get("content-type") || undefined,
                 });
             }
-            catch {
+            catch (headError) {
                 // Try GET request as fallback (some servers reject HEAD)
+                this.logger.debug(`HEAD request failed for ${url}, trying GET`, {
+                    error: headError instanceof Error ? headError.message : String(headError),
+                });
                 try {
                     const controller = new AbortController();
                     const timeoutId = setTimeout(() => controller.abort(), 5000);
@@ -456,6 +460,7 @@ export class ManifestValidationAssessor extends BaseAssessor {
                     });
                 }
                 catch (fetchError) {
+                    this.logError(`Failed to fetch privacy policy URL: ${url}`, fetchError);
                     results.push({
                         url,
                         accessible: false,

package/lib/services/assessment/modules/PromptAssessor.d.ts

@@ -32,6 +32,7 @@ export declare class PromptAssessor extends BaseAssessor {
     private analyzePromptTemplate;
     /**
      * Analyze dynamic content characteristics for enrichment (Issue #9)
+     * Enhanced with SanitizationDetector for library-aware detection (Issue #56)
      */
     private analyzeDynamicContent;
 }

package/lib/services/assessment/modules/PromptAssessor.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"PromptAssessor.d.ts","sourceRoot":"","sources":["../../../../src/services/assessment/modules/PromptAssessor.ts"],"names":[],"mappings":"AAAA;;;;;;;;;GASG;AAEH,OAAO,EACL,gBAAgB,EAGjB,MAAM,uBAAuB,CAAC;AAC/B,OAAO,EAAE,YAAY,EAAE,MAAM,gBAAgB,CAAC;AAC9C,OAAO,EAAE,iBAAiB,EAAa,MAAM,2BAA2B,CAAC;AA6DzE,qBAAa,cAAe,SAAQ,YAAY;IACxC,MAAM,CAAC,OAAO,EAAE,iBAAiB,GAAG,OAAO,CAAC,gBAAgB,CAAC;IAqDnE,OAAO,CAAC,uBAAuB;YAajB,UAAU;IAsFxB,OAAO,CAAC,oBAAoB;IAK5B,OAAO,CAAC,kBAAkB;IAa1B,OAAO,CAAC,qBAAqB;YAuBf,mBAAmB;IAuCjC,OAAO,CAAC,6BAA6B;YAqBvB,mBAAmB;IA6CjC,OAAO,CAAC,qBAAqB;YAsCf,sBAAsB;IA+BpC,OAAO,CAAC,qBAAqB;IAe7B,OAAO,CAAC,mBAAmB;IAmC3B,OAAO,CAAC,uBAAuB;IAqC/B;;OAEG;IACH,OAAO,CAAC,qBAAqB;IAkC7B;;OAEG;IACH,OAAO,CAAC,qBAAqB;CA+C9B"}
+{"version":3,"file":"PromptAssessor.d.ts","sourceRoot":"","sources":["../../../../src/services/assessment/modules/PromptAssessor.ts"],"names":[],"mappings":"AAAA;;;;;;;;;GASG;AAEH,OAAO,EACL,gBAAgB,EAGjB,MAAM,uBAAuB,CAAC;AAC/B,OAAO,EAAE,YAAY,EAAE,MAAM,gBAAgB,CAAC;AAC9C,OAAO,EAAE,iBAAiB,EAAa,MAAM,2BAA2B,CAAC;AA8DzE,qBAAa,cAAe,SAAQ,YAAY;IACxC,MAAM,CAAC,OAAO,EAAE,iBAAiB,GAAG,OAAO,CAAC,gBAAgB,CAAC;IAqDnE,OAAO,CAAC,uBAAuB;YAajB,UAAU;IAsFxB,OAAO,CAAC,oBAAoB;IAK5B,OAAO,CAAC,kBAAkB;IAa1B,OAAO,CAAC,qBAAqB;YAuBf,mBAAmB;IAwCjC,OAAO,CAAC,6BAA6B;YAqBvB,mBAAmB;IAmDjC,OAAO,CAAC,qBAAqB;YAsCf,sBAAsB;IAqCpC,OAAO,CAAC,qBAAqB;IAe7B,OAAO,CAAC,mBAAmB;IAmC3B,OAAO,CAAC,uBAAuB;IAqC/B;;OAEG;IACH,OAAO,CAAC,qBAAqB;IAkC7B;;;OAGG;IACH,OAAO,CAAC,qBAAqB;CAsD9B"}

package/lib/services/assessment/modules/PromptAssessor.js

@@ -9,6 +9,7 @@
  * - Required vs optional argument handling
  */
 import { BaseAssessor } from "./BaseAssessor.js";
+import { SanitizationDetector } from "./securityTests/SanitizationDetector.js";
 // AUP violation patterns in prompt descriptions/content
 const AUP_VIOLATION_PATTERNS = [
     // Harmful content generation
@@ -215,6 +216,7 @@ export class PromptAssessor extends BaseAssessor {
             return { success: true, unsafeContent, executionTime };
         }
         catch (error) {
+            this.logError(`Prompt execution failed: ${prompt.name}`, error);
             return {
                 success: false,
                 unsafeContent: false,
@@ -265,8 +267,11 @@ export class PromptAssessor extends BaseAssessor {
             }
             return { vulnerable: false };
         }
-        catch {
+        catch (error) {
             // Error handling payload is good - not vulnerable
+            this.logger.debug(`Injection payload rejected for ${prompt.name} (good)`, {
+                error: error instanceof Error ? error.message : String(error),
+            });
             return { vulnerable: false };
         }
     }
@@ -311,8 +316,11 @@ export class PromptAssessor extends BaseAssessor {
                 // If we got here without error, validation failed
                 return false;
             }
-            catch {
+            catch (error) {
                 // Expected - missing required arg should throw
+                this.logger.debug(`Missing arg ${arg.name} correctly rejected for ${prompt.name}`, {
+                    error: error instanceof Error ? error.message : String(error),
+                });
                 continue;
             }
         }
@@ -399,6 +407,7 @@ export class PromptAssessor extends BaseAssessor {
     }
     /**
      * Analyze dynamic content characteristics for enrichment (Issue #9)
+     * Enhanced with SanitizationDetector for library-aware detection (Issue #56)
      */
     analyzeDynamicContent(prompt) {
         const description = prompt.description || "";
@@ -409,25 +418,26 @@ export class PromptAssessor extends BaseAssessor {
             /\$\{.*\}/i.test(fullText) ||
             /\{[a-zA-Z_][a-zA-Z0-9_]*\}/i.test(fullText) ||
             (prompt.arguments?.length || 0) > 0;
-        // Detect escaping mechanisms mentioned
-        const escapingApplied = [];
-        if (/sanitiz/i.test(fullText))
-            escapingApplied.push("sanitization");
-        if (/escap/i.test(fullText))
-            escapingApplied.push("escaping");
-        if (/encod/i.test(fullText))
-            escapingApplied.push("encoding");
-        if (/validat/i.test(fullText))
-            escapingApplied.push("validation");
-        if (/filter/i.test(fullText))
-            escapingApplied.push("filtering");
+        // Issue #56: Use SanitizationDetector for library-aware detection
+        const sanitizationDetector = new SanitizationDetector();
+        const sanitizationResult = sanitizationDetector.detectFromText(fullText);
+        // Combine library detection with generic patterns for escapingApplied
+        const escapingApplied = [
+            ...sanitizationResult.libraries,
+            ...sanitizationResult.genericPatterns,
+        ];
         // Infer injection safety from multiple signals
         const hasTypeChecks = prompt.arguments?.some((a) => a.description?.toLowerCase().includes("type") ||
             a.description?.toLowerCase().includes("must be"));
         const hasLengthLimits = prompt.arguments?.some((a) => a.description?.toLowerCase().includes("max") ||
             a.description?.toLowerCase().includes("limit"));
-        // Consider injection safe if escaping is mentioned or validation exists
-        const injectionSafe = escapingApplied.length > 0 || hasTypeChecks || hasLengthLimits || false;
+        // Issue #56: Enhanced injection safety determination
+        // Now considers specific libraries (stronger signal) in addition to generic patterns
+        const injectionSafe = sanitizationResult.libraries.length > 0 || // Specific library = strong signal
+            sanitizationResult.genericPatterns.length >= 2 || // Multiple generic patterns
+            hasTypeChecks ||
+            hasLengthLimits ||
+            false;
         return {
             hasInterpolation,
             injectionSafe,

package/lib/services/assessment/modules/ProtocolComplianceAssessor.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"ProtocolComplianceAssessor.d.ts","sourceRoot":"","sources":["../../../../src/services/assessment/modules/ProtocolComplianceAssessor.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;GAgBG;AAEH,OAAO,EACL,2BAA2B,EAM3B,uBAAuB,EAGxB,MAAM,uBAAuB,CAAC;AAC/B,OAAO,KAAK,EAAE,aAAa,EAAE,MAAM,gCAAgC,CAAC;AAOpE,OAAO,EAAE,YAAY,EAAE,MAAM,gBAAgB,CAAC;AAC9C,OAAO,EAAE,iBAAiB,EAAE,MAAM,2BAA2B,CAAC;AAmB9D;;;GAGG;AACH,MAAM,WAAW,4BAA6B,SAAQ,2BAA2B;IAC/E,2EAA2E;IAC3E,iBAAiB,CAAC,EAAE;QAClB,mBAAmB,EAAE,aAAa,CAAC;QACnC,kBAAkB,EAAE,aAAa,CAAC;QAClC,uBAAuB,EAAE,aAAa,CAAC;KACxC,CAAC;CACH;AAED,qBAAa,0BAA2B,SAAQ,YAAY,CAAC,4BAA4B,CAAC;IACxF,OAAO,CAAC,GAAG,CAAc;gBAEb,MAAM,EAAE,uBAAuB;IAK3C;;OAEG;IACH,OAAO,CAAC,cAAc;IAItB;;OAEG;IACH,OAAO,CAAC,cAAc;IAItB;;;OAGG;IACG,MAAM,CACV,OAAO,EAAE,iBAAiB,GACzB,OAAO,CAAC,4BAA4B,CAAC;IAmIxC;;OAEG;IACH,OAAO,CAAC,sBAAsB;IAqB9B;;OAEG;YACW,sBAAsB;IAuBpC;;OAEG;IACH,OAAO,CAAC,uBAAuB;IAsB/B;;OAEG;IACH,OAAO,CAAC,qBAAqB;IAuC7B;;OAEG;YACW,mBAAmB;IAiCjC;;OAEG;IACH,OAAO,CAAC,4BAA4B;IAYpC;;OAEG;IACH,OAAO,CAAC,2BAA2B;IAkEnC;;OAEG;IACH,OAAO,CAAC,qBAAqB;IAS7B;;OAEG;YACW,wBAAwB;IA4GtC;;OAEG;YACW,uBAAuB;IA2FrC;;OAEG;YACW,4BAA4B;IAoD1C,OAAO,CAAC,yBAAyB;IAkEjC,OAAO,CAAC,uBAAuB;IAqB/B,OAAO,CAAC,sBAAsB;IA0B9B,OAAO,CAAC,qBAAqB;IAgC7B,OAAO,CAAC,oBAAoB;IA8E5B;;OAEG;IACH,OAAO,CAAC,mBAAmB;IAoC3B;;OAEG;IACH,OAAO,CAAC,uBAAuB;CAqEhC"}
+{"version":3,"file":"ProtocolComplianceAssessor.d.ts","sourceRoot":"","sources":["../../../../src/services/assessment/modules/ProtocolComplianceAssessor.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;GAgBG;AAEH,OAAO,EACL,2BAA2B,EAM3B,uBAAuB,EAGxB,MAAM,uBAAuB,CAAC;AAC/B,OAAO,KAAK,EAAE,aAAa,EAAE,MAAM,gCAAgC,CAAC;AAOpE,OAAO,EAAE,YAAY,EAAE,MAAM,gBAAgB,CAAC;AAC9C,OAAO,EAAE,iBAAiB,EAAE,MAAM,2BAA2B,CAAC;AAmB9D;;;GAGG;AACH,MAAM,WAAW,4BAA6B,SAAQ,2BAA2B;IAC/E,2EAA2E;IAC3E,iBAAiB,CAAC,EAAE;QAClB,mBAAmB,EAAE,aAAa,CAAC;QACnC,kBAAkB,EAAE,aAAa,CAAC;QAClC,uBAAuB,EAAE,aAAa,CAAC;KACxC,CAAC;CACH;AAED,qBAAa,0BAA2B,SAAQ,YAAY,CAAC,4BAA4B,CAAC;IACxF,OAAO,CAAC,GAAG,CAAc;gBAEb,MAAM,EAAE,uBAAuB;IAK3C;;OAEG;IACH,OAAO,CAAC,cAAc;IAItB;;OAEG;IACH,OAAO,CAAC,cAAc;IAItB;;;OAGG;IACG,MAAM,CACV,OAAO,EAAE,iBAAiB,GACzB,OAAO,CAAC,4BAA4B,CAAC;IAmIxC;;OAEG;IACH,OAAO,CAAC,sBAAsB;IAqB9B;;OAEG;YACW,sBAAsB;IAuBpC;;OAEG;IACH,OAAO,CAAC,uBAAuB;IAsB/B;;OAEG;IACH,OAAO,CAAC,qBAAqB;IAwC7B;;OAEG;YACW,mBAAmB;IAiCjC;;OAEG;IACH,OAAO,CAAC,4BAA4B;IAYpC;;OAEG;IACH,OAAO,CAAC,2BAA2B;IAkEnC;;OAEG;IACH,OAAO,CAAC,qBAAqB;IAS7B;;OAEG;YACW,wBAAwB;IA4GtC;;OAEG;YACW,uBAAuB;IA2FrC;;OAEG;YACW,4BAA4B;IAoD1C,OAAO,CAAC,yBAAyB;IAkEjC,OAAO,CAAC,uBAAuB;IAqB/B,OAAO,CAAC,sBAAsB;IA0B9B,OAAO,CAAC,qBAAqB;IAgC7B,OAAO,CAAC,oBAAoB;IA8E5B;;OAEG;IACH,OAAO,CAAC,mBAAmB;IAoC3B;;OAEG;IACH,OAAO,CAAC,uBAAuB;CAqEhC"}

package/lib/services/assessment/modules/ProtocolComplianceAssessor.js

@@ -229,7 +229,9 @@ export class ProtocolComplianceAssessor extends BaseAssessor {
                         hasErrors = true;
                         const errorMsg = `${tool.name}: ${JSON.stringify(this.ajv.errors)}`;
                         errors.push(errorMsg);
-                        console.warn(`Invalid schema for tool ${tool.name}:`, this.ajv.errors);
+                        this.logger.warn(`Invalid schema for tool ${tool.name}`, {
+                            errors: this.ajv.errors,
+                        });
                     }
                 }
             }
@@ -240,7 +242,9 @@ export class ProtocolComplianceAssessor extends BaseAssessor {
             };
         }
         catch (error) {
-            console.error("Schema compliance check failed:", error);
+            this.logger.error("Schema compliance check failed", {
+                error: String(error),
+            });
             return {
                 passed: false,
                 confidence: "low",

package/lib/services/assessment/modules/ProtocolConformanceAssessor.d.ts

@@ -11,10 +11,15 @@
  *
  * @module assessment/modules/ProtocolConformanceAssessor
  */
+import { AssessmentConfiguration } from "../../../lib/assessment/configTypes.js";
 import type { ProtocolConformanceAssessment } from "../../../lib/assessment/extendedTypes.js";
 import { BaseAssessor } from "./BaseAssessor.js";
 import { AssessmentContext } from "../AssessmentOrchestrator.js";
+/**
+ * @deprecated Use ProtocolComplianceAssessor instead. Will be removed in v2.0.0.
+ */
 export declare class ProtocolConformanceAssessor extends BaseAssessor<ProtocolConformanceAssessment> {
+    constructor(config: AssessmentConfiguration);
     /**
      * Select representative tools for testing (first, middle, last for diversity)
      */

package/lib/services/assessment/modules/ProtocolConformanceAssessor.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"ProtocolConformanceAssessor.d.ts","sourceRoot":"","sources":["../../../../src/services/assessment/modules/ProtocolConformanceAssessor.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;GAYG;AAGH,OAAO,KAAK,EACV,6BAA6B,EAE9B,MAAM,gCAAgC,CAAC;AACxC,OAAO,EAAE,YAAY,EAAE,MAAM,gBAAgB,CAAC;AAC9C,OAAO,EAAE,iBAAiB,EAAE,MAAM,2BAA2B,CAAC;AAmB9D,qBAAa,2BAA4B,SAAQ,YAAY,CAAC,6BAA6B,CAAC;IAC1F;;OAEG;IACH,OAAO,CAAC,qBAAqB;IAS7B;;OAEG;IACH,OAAO,CAAC,cAAc;IAItB;;OAEG;IACH,OAAO,CAAC,cAAc;IAItB;;OAEG;IACH,OAAO,CAAC,mBAAmB;IAI3B;;OAEG;IACH,OAAO,CAAC,eAAe;IAIjB,MAAM,CACV,OAAO,EAAE,iBAAiB,GACzB,OAAO,CAAC,6BAA6B,CAAC;IAqCzC;;;;;;;;;OASG;YACW,wBAAwB;IAoHtC;;;;;OAKG;YACW,uBAAuB;IAkGrC;;;;;;;;OAQG;YACW,4BAA4B;IAkD1C;;OAEG;IACH,OAAO,CAAC,yBAAyB;IA6BjC;;OAEG;IACH,OAAO,CAAC,mBAAmB;IAmC3B;;OAEG;IACH,OAAO,CAAC,uBAAuB;CA6ChC"}
+{"version":3,"file":"ProtocolConformanceAssessor.d.ts","sourceRoot":"","sources":["../../../../src/services/assessment/modules/ProtocolConformanceAssessor.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;GAYG;AAGH,OAAO,EAAE,uBAAuB,EAAE,MAAM,8BAA8B,CAAC;AACvE,OAAO,KAAK,EACV,6BAA6B,EAE9B,MAAM,gCAAgC,CAAC;AACxC,OAAO,EAAE,YAAY,EAAE,MAAM,gBAAgB,CAAC;AAC9C,OAAO,EAAE,iBAAiB,EAAE,MAAM,2BAA2B,CAAC;AAmB9D;;GAEG;AACH,qBAAa,2BAA4B,SAAQ,YAAY,CAAC,6BAA6B,CAAC;gBAC9E,MAAM,EAAE,uBAAuB;IAY3C;;OAEG;IACH,OAAO,CAAC,qBAAqB;IAS7B;;OAEG;IACH,OAAO,CAAC,cAAc;IAItB;;OAEG;IACH,OAAO,CAAC,cAAc;IAItB;;OAEG;IACH,OAAO,CAAC,mBAAmB;IAI3B;;OAEG;IACH,OAAO,CAAC,eAAe;IAIjB,MAAM,CACV,OAAO,EAAE,iBAAiB,GACzB,OAAO,CAAC,6BAA6B,CAAC;IAqCzC;;;;;;;;;OASG;YACW,wBAAwB;IA0HtC;;;;;OAKG;YACW,uBAAuB;IAmGrC;;;;;;;;OAQG;YACW,4BAA4B;IAkD1C;;OAEG;IACH,OAAO,CAAC,yBAAyB;IA6BjC;;OAEG;IACH,OAAO,CAAC,mBAAmB;IAmC3B;;OAEG;IACH,OAAO,CAAC,uBAAuB;CA6ChC"}

package/lib/services/assessment/modules/ProtocolConformanceAssessor.js

@@ -20,7 +20,18 @@ const VALID_CONTENT_TYPES = [
     "resource",
     "resource_link",
 ];
+/**
+ * @deprecated Use ProtocolComplianceAssessor instead. Will be removed in v2.0.0.
+ */
 export class ProtocolConformanceAssessor extends BaseAssessor {
+    constructor(config) {
+        super(config);
+        this.logger.warn("ProtocolConformanceAssessor is deprecated. Use ProtocolComplianceAssessor instead. " +
+            "This module will be removed in v2.0.0.", {
+            module: "ProtocolConformanceAssessor",
+            replacement: "ProtocolComplianceAssessor",
+        });
+    }
     /**
      * Select representative tools for testing (first, middle, last for diversity)
      */
@@ -147,6 +158,9 @@ export class ProtocolConformanceAssessor extends BaseAssessor {
             }
             catch (error) {
                 // Tool threw exception instead of returning error response
+                this.logger.debug(`Tool ${testTool.name} threw exception instead of error response`, {
+                    error: error instanceof Error ? error.message : String(error),
+                });
                 results.push({
                     toolName: testTool.name,
                     passed: false,
@@ -254,6 +268,7 @@ export class ProtocolConformanceAssessor extends BaseAssessor {
             };
         }
         catch (error) {
+            this.logError("Content type validation failed", error);
             return {
                 passed: false,
                 confidence: "medium",

package/lib/services/assessment/modules/ResourceAssessor.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"ResourceAssessor.d.ts","sourceRoot":"","sources":["../../../../src/services/assessment/modules/ResourceAssessor.ts"],"names":[],"mappings":"AAAA;;;;;;;;;GASG;AAEH,OAAO,EACL,kBAAkB,EAGnB,MAAM,uBAAuB,CAAC;AAC/B,OAAO,EAAE,YAAY,EAAE,MAAM,gBAAgB,CAAC;AAC9C,OAAO,EAAE,iBAAiB,EAAE,MAAM,2BAA2B,CAAC;AAoN9D,qBAAa,gBAAiB,SAAQ,YAAY;IAC1C,MAAM,CAAC,OAAO,EAAE,iBAAiB,GAAG,OAAO,CAAC,kBAAkB,CAAC;IAgFrE,OAAO,CAAC,yBAAyB;YAiBnB,YAAY;IAoG1B;;OAEG;IACH,OAAO,CAAC,uBAAuB;IAY/B;;OAEG;IACH,OAAO,CAAC,mBAAmB;IA4B3B;;OAEG;IACH,OAAO,CAAC,uBAAuB;YAsBjB,oBAAoB;IA4FlC,OAAO,CAAC,UAAU;IAmBlB,OAAO,CAAC,kBAAkB;IAM1B,OAAO,CAAC,cAAc;IAItB,OAAO,CAAC,wBAAwB;IAIhC;;;OAGG;IACH,OAAO,CAAC,qBAAqB;IAc7B,OAAO,CAAC,yBAAyB;IAYjC,OAAO,CAAC,uBAAuB;IAqB/B,OAAO,CAAC,mBAAmB;IAoC3B,OAAO,CAAC,uBAAuB;CA+DhC"}
+{"version":3,"file":"ResourceAssessor.d.ts","sourceRoot":"","sources":["../../../../src/services/assessment/modules/ResourceAssessor.ts"],"names":[],"mappings":"AAAA;;;;;;;;;GASG;AAEH,OAAO,EACL,kBAAkB,EAGnB,MAAM,uBAAuB,CAAC;AAC/B,OAAO,EAAE,YAAY,EAAE,MAAM,gBAAgB,CAAC;AAC9C,OAAO,EAAE,iBAAiB,EAAE,MAAM,2BAA2B,CAAC;AAoN9D,qBAAa,gBAAiB,SAAQ,YAAY;IAC1C,MAAM,CAAC,OAAO,EAAE,iBAAiB,GAAG,OAAO,CAAC,kBAAkB,CAAC;IAgFrE,OAAO,CAAC,yBAAyB;YAiBnB,YAAY;IAoG1B;;OAEG;IACH,OAAO,CAAC,uBAAuB;IAY/B;;OAEG;IACH,OAAO,CAAC,mBAAmB;IA4B3B;;OAEG;IACH,OAAO,CAAC,uBAAuB;YAsBjB,oBAAoB;IAkGlC,OAAO,CAAC,UAAU;IAsBlB,OAAO,CAAC,kBAAkB;IAM1B,OAAO,CAAC,cAAc;IAItB,OAAO,CAAC,wBAAwB;IAIhC;;;OAGG;IACH,OAAO,CAAC,qBAAqB;IAc7B,OAAO,CAAC,yBAAyB;IAYjC,OAAO,CAAC,uBAAuB;IAqB/B,OAAO,CAAC,mBAAmB;IAoC3B,OAAO,CAAC,uBAAuB;CA+DhC"}