@bryan-thompson/inspector-assessment 1.26.5 → 1.26.7

package/client/lib/services/assessment/modules/AuthenticationAssessor.js

@@ -246,7 +246,7 @@ export class AuthenticationAssessor extends BaseAssessor {
      * Run authentication assessment
      */
     async assess(context) {
-        this.log("Starting authentication assessment");
+        this.logger.info("Starting authentication assessment");
         this.testCount = 0;
         const oauthIndicators = [];
         const localResourceIndicators = [];
@@ -323,7 +323,7 @@ export class AuthenticationAssessor extends BaseAssessor {
         // Generate additional recommendations from auth config findings
         const authConfigRecommendations = authConfigAnalysis.findings.map((f) => f.recommendation ||
             `Review ${f.type}: ${f.message} (${f.file || "unknown file"})`);
-        this.log(`Assessment complete: auth=${authMethod}, localDeps=${hasLocalDependencies}, tlsEnforced=${transportSecurity.tlsEnforced}, authConfigFindings=${authConfigAnalysis.totalFindings}`);
+        this.logger.info(`Assessment complete: auth=${authMethod}, localDeps=${hasLocalDependencies}, tlsEnforced=${transportSecurity.tlsEnforced}, authConfigFindings=${authConfigAnalysis.totalFindings}`);
         return {
             authMethod,
             hasLocalDependencies,
@@ -594,7 +594,7 @@ export class AuthenticationAssessor extends BaseAssessor {
         // Issue #65: Apply file limit to prevent performance issues on large codebases
         let sourceFiles = Array.from(context.sourceCodeFiles);
         if (sourceFiles.length > MAX_FILES) {
-            this.log(`Rate limiting: Analyzing ${MAX_FILES} of ${sourceFiles.length} files`);
+            this.logger.info(`Rate limiting: Analyzing ${MAX_FILES} of ${sourceFiles.length} files`);
             sourceFiles = sourceFiles.slice(0, MAX_FILES);
         }
         for (const [filePath, content] of sourceFiles) {
@@ -776,7 +776,7 @@ export class AuthenticationAssessor extends BaseAssessor {
             }
             catch (error) {
                 // Warning 4 fix: Handle malformed files gracefully
-                this.log(`Error analyzing ${filePath}: ${error}`);
+                this.logger.info(`Error analyzing ${filePath}: ${error}`);
                 continue;
             }
         }

package/client/lib/services/assessment/modules/BaseAssessor.d.ts

@@ -10,7 +10,6 @@ export declare abstract class BaseAssessor<T = unknown> {
     protected config: AssessmentConfiguration;
     protected logger: Logger;
     protected testCount: number;
-    private deprecationWarningsEmitted;
     constructor(config: AssessmentConfiguration);
     /**
      * Abstract method that each assessor must implement
@@ -20,19 +19,6 @@ export declare abstract class BaseAssessor<T = unknown> {
      * Common method to determine status based on pass rate
      */
     protected determineStatus(passed: number, total: number, threshold?: number): AssessmentStatus;
-    /**
-     * Log assessment progress
-     * @deprecated Use this.logger.info() directly for structured logging with context. Will be removed in v2.0.0.
-     */
-    protected log(message: string): void;
-    /**
-     * Log error with optional context
-     * @deprecated Use this.logger.error() directly for structured logging with context. Will be removed in v2.0.0.
-     *
-     * @param message - Description of what operation failed
-     * @param error - The error that occurred (optional)
-     */
-    protected logError(message: string, error?: unknown): void;
     /**
      * Handle an error with logging and structured result
      *

package/client/lib/services/assessment/modules/BaseAssessor.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"BaseAssessor.d.ts","sourceRoot":"","sources":["../../../../src/services/assessment/modules/BaseAssessor.ts"],"names":[],"mappings":"AAAA;;;GAGG;AAEH,OAAO,EACL,uBAAuB,EACvB,gBAAgB,EAEjB,MAAM,uBAAuB,CAAC;AAC/B,OAAO,EAAE,iBAAiB,EAAE,MAAM,2BAA2B,CAAC;AAC9D,OAAO,EAAE,MAAM,EAAwC,MAAM,eAAe,CAAC;AAC7E,OAAO,EAGL,WAAW,EAEZ,MAAM,eAAe,CAAC;AAGvB,8BAAsB,YAAY,CAAC,CAAC,GAAG,OAAO;IAC5C,SAAS,CAAC,MAAM,EAAE,uBAAuB,CAAC;IAC1C,SAAS,CAAC,MAAM,EAAE,MAAM,CAAC;IACzB,SAAS,CAAC,SAAS,EAAE,MAAM,CAAK;IAGhC,OAAO,CAAC,0BAA0B,CAGhC;gBAEU,MAAM,EAAE,uBAAuB;IAS3C;;OAEG;IACH,QAAQ,CAAC,MAAM,CAAC,OAAO,EAAE,iBAAiB,GAAG,OAAO,CAAC,CAAC,CAAC;IAEvD;;OAEG;IACH,SAAS,CAAC,eAAe,CACvB,MAAM,EAAE,MAAM,EACd,KAAK,EAAE,MAAM,EACb,SAAS,GAAE,MAAY,GACtB,gBAAgB;IAUnB;;;OAGG;IACH,SAAS,CAAC,GAAG,CAAC,OAAO,EAAE,MAAM,GAAG,IAAI;IAWpC;;;;;;OAMG;IACH,SAAS,CAAC,QAAQ,CAAC,OAAO,EAAE,MAAM,EAAE,KAAK,CAAC,EAAE,OAAO,GAAG,IAAI;IAW1D;;;;;;;;;;;;;;;;;;;;OAoBG;IACH,SAAS,CAAC,WAAW,CAAC,CAAC,SAAS,WAAW,EACzC,KAAK,EAAE,OAAO,EACd,OAAO,EAAE,MAAM,EACf,QAAQ,GAAE,OAAO,CAAC,CAAC,CAAM,GACxB,CAAC;IAsBJ;;OAEG;IACH,YAAY,IAAI,MAAM;IAItB;;OAEG;IACH,cAAc,IAAI,IAAI;IAItB;;OAEG;IACH,SAAS,CAAC,gBAAgB,CACxB,OAAO,EAAE,MAAM,uBAAuB,CAAC,sBAAsB,CAAC,GAC7D,OAAO;IAIV;;OAEG;cACa,KAAK,CAAC,EAAE,EAAE,MAAM,GAAG,OAAO,CAAC,IAAI,CAAC;IAIhD;;;;;;;;;;OAUG;cACa,kBAAkB,CAAC,CAAC,EAClC,OAAO,EAAE,OAAO,CAAC,CAAC,CAAC,EACnB,SAAS,GAAE,MAAgC,GAC1C,OAAO,CAAC,CAAC,CAAC;IAOb;;OAEG;IACH,SAAS,CAAC,aAAa,CAAC,IAAI,EAAE,MAAM,GAAG,OAAO;IAS9C;;OAEG;IACH,SAAS,CAAC,mBAAmB,CAAC,KAAK,EAAE,OAAO,GAAG,MAAM;IAgBrD;;;;;;OAMG;IACH,SAAS,CAAC,eAAe,CACvB,QAAQ,EAAE,OAAO,EACjB,UAAU,GAAE,OAAe,GAC1B,OAAO;IA+CV;;OAEG;IACH,SAAS,CAAC,gBAAgB,CAAC,QAAQ,EAAE,OAAO,GAAG;QAC7C,IAAI,CAAC,EAAE,MAAM,GAAG,MAAM,CAAC;QACvB,OAAO,CAAC,EAAE,MAAM,CAAC;KAClB;CA2BF"}
+{"version":3,"file":"BaseAssessor.d.ts","sourceRoot":"","sources":["../../../../src/services/assessment/modules/BaseAssessor.ts"],"names":[],"mappings":"AAAA;;;GAGG;AAEH,OAAO,EACL,uBAAuB,EACvB,gBAAgB,EAEjB,MAAM,uBAAuB,CAAC;AAC/B,OAAO,EAAE,iBAAiB,EAAE,MAAM,2BAA2B,CAAC;AAC9D,OAAO,EAAE,MAAM,EAAwC,MAAM,eAAe,CAAC;AAC7E,OAAO,EAGL,WAAW,EAEZ,MAAM,eAAe,CAAC;AAGvB,8BAAsB,YAAY,CAAC,CAAC,GAAG,OAAO;IAC5C,SAAS,CAAC,MAAM,EAAE,uBAAuB,CAAC;IAC1C,SAAS,CAAC,MAAM,EAAE,MAAM,CAAC;IACzB,SAAS,CAAC,SAAS,EAAE,MAAM,CAAK;gBAEpB,MAAM,EAAE,uBAAuB;IAS3C;;OAEG;IACH,QAAQ,CAAC,MAAM,CAAC,OAAO,EAAE,iBAAiB,GAAG,OAAO,CAAC,CAAC,CAAC;IAEvD;;OAEG;IACH,SAAS,CAAC,eAAe,CACvB,MAAM,EAAE,MAAM,EACd,KAAK,EAAE,MAAM,EACb,SAAS,GAAE,MAAY,GACtB,gBAAgB;IAUnB;;;;;;;;;;;;;;;;;;;;OAoBG;IACH,SAAS,CAAC,WAAW,CAAC,CAAC,SAAS,WAAW,EACzC,KAAK,EAAE,OAAO,EACd,OAAO,EAAE,MAAM,EACf,QAAQ,GAAE,OAAO,CAAC,CAAC,CAAM,GACxB,CAAC;IAsBJ;;OAEG;IACH,YAAY,IAAI,MAAM;IAItB;;OAEG;IACH,cAAc,IAAI,IAAI;IAItB;;OAEG;IACH,SAAS,CAAC,gBAAgB,CACxB,OAAO,EAAE,MAAM,uBAAuB,CAAC,sBAAsB,CAAC,GAC7D,OAAO;IAIV;;OAEG;cACa,KAAK,CAAC,EAAE,EAAE,MAAM,GAAG,OAAO,CAAC,IAAI,CAAC;IAIhD;;;;;;;;;;OAUG;cACa,kBAAkB,CAAC,CAAC,EAClC,OAAO,EAAE,OAAO,CAAC,CAAC,CAAC,EACnB,SAAS,GAAE,MAAgC,GAC1C,OAAO,CAAC,CAAC,CAAC;IAOb;;OAEG;IACH,SAAS,CAAC,aAAa,CAAC,IAAI,EAAE,MAAM,GAAG,OAAO;IAS9C;;OAEG;IACH,SAAS,CAAC,mBAAmB,CAAC,KAAK,EAAE,OAAO,GAAG,MAAM;IAgBrD;;;;;;OAMG;IACH,SAAS,CAAC,eAAe,CACvB,QAAQ,EAAE,OAAO,EACjB,UAAU,GAAE,OAAe,GAC1B,OAAO;IA+CV;;OAEG;IACH,SAAS,CAAC,gBAAgB,CAAC,QAAQ,EAAE,OAAO,GAAG;QAC7C,IAAI,CAAC,EAAE,MAAM,GAAG,MAAM,CAAC;QACvB,OAAO,CAAC,EAAE,MAAM,CAAC;KAClB;CA2BF"}

package/client/lib/services/assessment/modules/BaseAssessor.js

@@ -9,11 +9,6 @@ export class BaseAssessor {
     config;
     logger;
     testCount = 0;
-    // Track deprecation warnings to emit only once per instance
-    deprecationWarningsEmitted = {
-        log: false,
-        logError: false,
-    };
     constructor(config) {
         this.config = config;
         // Create logger from config, using class name as prefix
@@ -32,33 +27,6 @@ export class BaseAssessor {
             return "NEED_MORE_INFO";
         return "FAIL";
     }
-    /**
-     * Log assessment progress
-     * @deprecated Use this.logger.info() directly for structured logging with context. Will be removed in v2.0.0.
-     */
-    log(message) {
-        if (!this.deprecationWarningsEmitted.log) {
-            this.logger.warn("BaseAssessor.log() is deprecated. Use this.logger.info() instead. " +
-                "This method will be removed in v2.0.0.");
-            this.deprecationWarningsEmitted.log = true;
-        }
-        this.logger.info(message);
-    }
-    /**
-     * Log error with optional context
-     * @deprecated Use this.logger.error() directly for structured logging with context. Will be removed in v2.0.0.
-     *
-     * @param message - Description of what operation failed
-     * @param error - The error that occurred (optional)
-     */
-    logError(message, error) {
-        if (!this.deprecationWarningsEmitted.logError) {
-            this.logger.warn("BaseAssessor.logError() is deprecated. Use this.logger.error() instead. " +
-                "This method will be removed in v2.0.0.");
-            this.deprecationWarningsEmitted.logError = true;
-        }
-        this.logger.error(message, error ? { error: String(error) } : undefined);
-    }
     /**
      * Handle an error with logging and structured result
      *
@@ -147,7 +115,7 @@ export class BaseAssessor {
             return JSON.parse(text);
         }
         catch (error) {
-            this.logError(`Failed to parse JSON: ${text}`, error);
+            this.logger.error(`Failed to parse JSON: ${text}`, { error });
             return null;
         }
     }

package/client/lib/services/assessment/modules/CrossCapabilitySecurityAssessor.js

@@ -65,7 +65,7 @@ export class CrossCapabilitySecurityAssessor extends BaseAssessor {
         const tools = context.tools || [];
         const resources = context.resources || [];
         const prompts = context.prompts || [];
-        this.log(`Testing cross-capability security: ${tools.length} tools, ${resources.length} resources, ${prompts.length} prompts`);
+        this.logger.info(`Testing cross-capability security: ${tools.length} tools, ${resources.length} resources, ${prompts.length} prompts`);
         // Test 1: Tool->Resource access patterns
         const toolResourceResults = this.testToolResourceAccess(tools, resources);
         results.push(...toolResourceResults);

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

@@ -15,7 +15,7 @@
 import { BaseAssessor } from "./BaseAssessor.js";
 export class DeveloperExperienceAssessor extends BaseAssessor {
     async assess(context) {
-        this.log("Starting developer experience assessment");
+        this.logger.info("Starting developer experience assessment");
         const readmeContent = context.readmeContent || "";
         // Assess documentation
         const documentationMetrics = this.analyzeDocumentation(readmeContent, context.tools, "verbose");

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

@@ -16,7 +16,7 @@ export class DocumentationAssessor extends BaseAssessor {
         });
     }
     async assess(context) {
-        this.log("Starting documentation assessment");
+        this.logger.info("Starting documentation assessment");
         const readmeContent = context.readmeContent || "";
         const validVerbosityLevels = ["minimal", "standard", "verbose"];
         const configVerbosity = this.config.documentationVerbosity;
@@ -27,7 +27,7 @@ export class DocumentationAssessor extends BaseAssessor {
                 verbosity = configVerbosity;
             }
             else {
-                this.log(`Warning: Invalid documentationVerbosity "${configVerbosity}". ` +
+                this.logger.info(`Warning: Invalid documentationVerbosity "${configVerbosity}". ` +
                     `Valid options: ${validVerbosityLevels.join(", ")}. Using "verbose".`);
             }
         }

package/client/lib/services/assessment/modules/ErrorHandlingAssessor.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"ErrorHandlingAssessor.d.ts","sourceRoot":"","sources":["../../../../src/services/assessment/modules/ErrorHandlingAssessor.ts"],"names":[],"mappings":"AAAA;;;GAGG;AAEH,OAAO,EACL,uBAAuB,EAIxB,MAAM,uBAAuB,CAAC;AAC/B,OAAO,EAAE,YAAY,EAAE,MAAM,gBAAgB,CAAC;AAC9C,OAAO,EAAE,iBAAiB,EAAE,MAAM,2BAA2B,CAAC;AAG9D,qBAAa,qBAAsB,SAAQ,YAAY;IAC/C,MAAM,CAAC,OAAO,EAAE,iBAAiB,GAAG,OAAO,CAAC,uBAAuB,CAAC;IAiE1E,OAAO,CAAC,qBAAqB;YAqDf,qBAAqB;YAuBrB,qBAAqB;YAmGrB,cAAc;YAmFd,iBAAiB;YA8DjB,kBAAkB;IA6DhC,OAAO,CAAC,aAAa;IAOrB,OAAO,CAAC,uBAAuB;IAgC/B,OAAO,CAAC,0BAA0B;IAgClC,OAAO,CAAC,uBAAuB;IA4B/B,OAAO,CAAC,gBAAgB;IAoGxB,OAAO,CAAC,4BAA4B;IAapC,OAAO,CAAC,mBAAmB;IAuE3B,OAAO,CAAC,uBAAuB;CA4ChC"}
+{"version":3,"file":"ErrorHandlingAssessor.d.ts","sourceRoot":"","sources":["../../../../src/services/assessment/modules/ErrorHandlingAssessor.ts"],"names":[],"mappings":"AAAA;;;GAGG;AAEH,OAAO,EACL,uBAAuB,EAIxB,MAAM,uBAAuB,CAAC;AAC/B,OAAO,EAAE,YAAY,EAAE,MAAM,gBAAgB,CAAC;AAC9C,OAAO,EAAE,iBAAiB,EAAE,MAAM,2BAA2B,CAAC;AAG9D,qBAAa,qBAAsB,SAAQ,YAAY;IAC/C,MAAM,CAAC,OAAO,EAAE,iBAAiB,GAAG,OAAO,CAAC,uBAAuB,CAAC;IAiE1E,OAAO,CAAC,qBAAqB;YAuDf,qBAAqB;YAuBrB,qBAAqB;YAmGrB,cAAc;YAmFd,iBAAiB;YA8DjB,kBAAkB;IA6DhC,OAAO,CAAC,aAAa;IAOrB,OAAO,CAAC,uBAAuB;IAgC/B,OAAO,CAAC,0BAA0B;IAgClC,OAAO,CAAC,uBAAuB;IA4B/B,OAAO,CAAC,gBAAgB;IAoGxB,OAAO,CAAC,4BAA4B;IAapC,OAAO,CAAC,mBAAmB;IAuE3B,OAAO,CAAC,uBAAuB;CA4ChC"}

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

@@ -6,7 +6,7 @@ import { BaseAssessor } from "./BaseAssessor.js";
 import { createConcurrencyLimit } from "../lib/concurrencyLimit.js";
 export class ErrorHandlingAssessor extends BaseAssessor {
     async assess(context) {
-        this.log("Starting error handling assessment");
+        this.logger.info("Starting error handling assessment");
         const testDetails = [];
         let passedTests = 0;
         // Test a sample of tools for error handling
@@ -14,7 +14,7 @@ export class ErrorHandlingAssessor extends BaseAssessor {
         // Parallel tool testing with concurrency limit
         const concurrency = this.config.maxParallelTests ?? 5;
         const limit = createConcurrencyLimit(concurrency, this.logger);
-        this.log(`Testing ${toolsToTest.length} tools for error handling with concurrency limit of ${concurrency}`);
+        this.logger.info(`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);
             // Add delay between tests to avoid rate limiting
@@ -50,34 +50,34 @@ export class ErrorHandlingAssessor extends BaseAssessor {
         if (this.config.selectedToolsForTesting !== undefined) {
             // Warn if deprecated maxToolsToTestForErrors is also set
             if (this.config.maxToolsToTestForErrors !== undefined) {
-                this.log(`Warning: Both selectedToolsForTesting and maxToolsToTestForErrors are set. ` +
+                this.logger.info(`Warning: Both selectedToolsForTesting and maxToolsToTestForErrors are set. ` +
                     `Using selectedToolsForTesting (maxToolsToTestForErrors is deprecated).`);
             }
             const selectedNames = new Set(this.config.selectedToolsForTesting);
             const selectedTools = tools.filter((tool) => selectedNames.has(tool.name));
             // Empty array means user explicitly selected 0 tools
             if (this.config.selectedToolsForTesting.length === 0) {
-                this.log(`User selected 0 tools for error handling - skipping tests`);
+                this.logger.info(`User selected 0 tools for error handling - skipping tests`);
                 return [];
             }
             // If no tools matched the names (config out of sync), log warning but respect selection
             if (selectedTools.length === 0) {
-                this.log(`Warning: No tools matched selection (${this.config.selectedToolsForTesting.join(", ")})`);
+                this.logger.info(`Warning: No tools matched selection (${this.config.selectedToolsForTesting.join(", ")})`);
                 return [];
             }
-            this.log(`Testing ${selectedTools.length} selected tools out of ${tools.length} for error handling`);
+            this.logger.info(`Testing ${selectedTools.length} selected tools out of ${tools.length} for error handling`);
             return selectedTools;
         }
         // Backward compatibility: use old maxToolsToTestForErrors configuration
         const configLimit = this.config.maxToolsToTestForErrors;
         // If -1, test all tools
         if (configLimit === -1) {
-            this.log(`Testing all ${tools.length} tools for error handling`);
+            this.logger.info(`Testing all ${tools.length} tools for error handling`);
             return tools;
         }
         // Otherwise use the configured limit (default to 5 if not set)
         const maxTools = Math.min(configLimit ?? 5, tools.length);
-        this.log(`Testing ${maxTools} out of ${tools.length} tools for error handling`);
+        this.logger.info(`Testing ${maxTools} out of ${tools.length} tools for error handling`);
         return tools.slice(0, maxTools);
     }
     async testToolErrorHandling(tool, callTool) {

package/client/lib/services/assessment/modules/ExternalAPIScannerAssessor.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"ExternalAPIScannerAssessor.d.ts","sourceRoot":"","sources":["../../../../src/services/assessment/modules/ExternalAPIScannerAssessor.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;GAUG;AAEH,OAAO,EAAE,YAAY,EAAE,MAAM,gBAAgB,CAAC;AAC9C,OAAO,KAAK,EAAE,iBAAiB,EAAE,MAAM,2BAA2B,CAAC;AACnE,OAAO,KAAK,EAEV,4BAA4B,EAE7B,MAAM,uBAAuB,CAAC;AAmE/B,qBAAa,0BAA2B,SAAQ,YAAY;IACpD,MAAM,CACV,OAAO,EAAE,iBAAiB,GACzB,OAAO,CAAC,4BAA4B,CAAC;IA6DxC;;OAEG;IACH,OAAO,CAAC,cAAc;IAItB;;OAEG;IACH,OAAO,CAAC,eAAe;IA0BvB;;OAEG;IACH,OAAO,CAAC,aAAa;IAIrB;;OAEG;IACH,OAAO,CAAC,QAAQ;IAIhB;;OAEG;IACH,OAAO,CAAC,eAAe;IASvB;;OAEG;IACH,OAAO,CAAC,gBAAgB;IAsBxB;;OAEG;IACH,OAAO,CAAC,aAAa;IAcrB;;OAEG;IACH,OAAO,CAAC,mBAAmB;IAyB3B;;OAEG;IACH,OAAO,CAAC,uBAAuB;IA2B/B;;OAEG;IACH,OAAO,CAAC,oBAAoB;CAa7B"}
+{"version":3,"file":"ExternalAPIScannerAssessor.d.ts","sourceRoot":"","sources":["../../../../src/services/assessment/modules/ExternalAPIScannerAssessor.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;GAUG;AAEH,OAAO,EAAE,YAAY,EAAE,MAAM,gBAAgB,CAAC;AAC9C,OAAO,KAAK,EAAE,iBAAiB,EAAE,MAAM,2BAA2B,CAAC;AACnE,OAAO,KAAK,EAEV,4BAA4B,EAE7B,MAAM,uBAAuB,CAAC;AAmE/B,qBAAa,0BAA2B,SAAQ,YAAY;IACpD,MAAM,CACV,OAAO,EAAE,iBAAiB,GACzB,OAAO,CAAC,4BAA4B,CAAC;IA+DxC;;OAEG;IACH,OAAO,CAAC,cAAc;IAItB;;OAEG;IACH,OAAO,CAAC,eAAe;IA0BvB;;OAEG;IACH,OAAO,CAAC,aAAa;IAIrB;;OAEG;IACH,OAAO,CAAC,QAAQ;IAIhB;;OAEG;IACH,OAAO,CAAC,eAAe;IASvB;;OAEG;IACH,OAAO,CAAC,gBAAgB;IAsBxB;;OAEG;IACH,OAAO,CAAC,aAAa;IAcrB;;OAEG;IACH,OAAO,CAAC,mBAAmB;IAyB3B;;OAEG;IACH,OAAO,CAAC,uBAAuB;IA2B/B;;OAEG;IACH,OAAO,CAAC,oBAAoB;CAa7B"}

package/client/lib/services/assessment/modules/ExternalAPIScannerAssessor.js

@@ -74,13 +74,13 @@ const SKIP_FILE_PATTERNS = [
 ];
 export class ExternalAPIScannerAssessor extends BaseAssessor {
     async assess(context) {
-        this.log("Starting external API scanner assessment");
+        this.logger.info("Starting external API scanner assessment");
         this.resetTestCount();
         const detectedAPIs = [];
         let scannedFiles = 0;
         // Check if source code analysis is enabled
         if (!context.sourceCodeFiles || !context.config.enableSourceCodeAnalysis) {
-            this.log("Source code analysis not enabled, skipping external API scan");
+            this.logger.info("Source code analysis not enabled, skipping external API scan");
             return this.createNoSourceResult();
         }
         // Scan each source file
@@ -100,7 +100,7 @@ export class ExternalAPIScannerAssessor extends BaseAssessor {
         const status = this.computeStatus(detectedAPIs, affiliationWarning);
         const explanation = this.generateExplanation(detectedAPIs, uniqueServices, affiliationWarning, scannedFiles);
         const recommendations = this.generateRecommendations(uniqueServices, affiliationWarning);
-        this.log(`External API scan complete: ${detectedAPIs.length} APIs found in ${scannedFiles} files`);
+        this.logger.info(`External API scan complete: ${detectedAPIs.length} APIs found in ${scannedFiles} files`);
         return {
             detectedAPIs,
             uniqueServices,

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

@@ -23,23 +23,23 @@ export class FunctionalityAssessor extends BaseAssessor {
             const selectedTools = tools.filter((tool) => selectedNames.has(tool.name));
             // Empty array means user explicitly selected 0 tools
             if (this.config.selectedToolsForTesting.length === 0) {
-                this.log(`User selected 0 tools for functionality testing - skipping tests`);
+                this.logger.info(`User selected 0 tools for functionality testing - skipping tests`);
                 return [];
             }
             // If no tools matched the names (config out of sync), log warning but respect selection
             if (selectedTools.length === 0) {
-                this.log(`Warning: No tools matched selection (${this.config.selectedToolsForTesting.join(", ")})`);
+                this.logger.info(`Warning: No tools matched selection (${this.config.selectedToolsForTesting.join(", ")})`);
                 return [];
             }
-            this.log(`Testing ${selectedTools.length} selected tools out of ${tools.length} for functionality`);
+            this.logger.info(`Testing ${selectedTools.length} selected tools out of ${tools.length} for functionality`);
             return selectedTools;
         }
         // Default: test all tools
-        this.log(`Testing all ${tools.length} tools for functionality`);
+        this.logger.info(`Testing all ${tools.length} tools for functionality`);
         return tools;
     }
     async assess(context) {
-        this.log(`Starting functionality assessment${this.config.reviewerMode ? " (reviewer mode - quick verification)" : ""}`);
+        this.logger.info(`Starting functionality assessment${this.config.reviewerMode ? " (reviewer mode - quick verification)" : ""}`);
         const toolResults = [];
         const brokenTools = [];
         let workingTools = 0;
@@ -71,7 +71,7 @@ export class FunctionalityAssessor extends BaseAssessor {
             batchCount = 0;
             lastBatchTime = Date.now();
         };
-        this.log(`Testing ${toolsToTest.length} tools with concurrency limit of ${concurrency}`);
+        this.logger.info(`Testing ${toolsToTest.length} tools with concurrency limit of ${concurrency}`);
         const results = await Promise.all(toolsToTest.map((tool) => limit(async () => {
             this.testCount++;
             completedTests++;
@@ -103,7 +103,7 @@ export class FunctionalityAssessor extends BaseAssessor {
             else if (result.status === "broken") {
                 brokenTools.push(result.toolName);
                 if (this.config.skipBrokenTools) {
-                    this.log(`Skipping further tests for broken tool: ${result.toolName}`);
+                    this.logger.info(`Skipping further tests for broken tool: ${result.toolName}`);
                 }
             }
         }
@@ -141,7 +141,7 @@ export class FunctionalityAssessor extends BaseAssessor {
             const cleanedParams = schema
                 ? cleanParams(testParams, schema)
                 : testParams;
-            this.log(`Testing tool: ${tool.name} with params: ${JSON.stringify(cleanedParams)}`);
+            this.logger.info(`Testing tool: ${tool.name} with params: ${JSON.stringify(cleanedParams)}`);
             // Execute tool with timeout
             const response = await this.executeWithTimeout(callTool(tool.name, cleanedParams), this.config.testTimeout);
             const executionTime = Date.now() - startTime;
@@ -197,7 +197,7 @@ export class FunctionalityAssessor extends BaseAssessor {
             };
         }
         catch (error) {
-            this.logError(`Tool execution failed: ${tool.name}`, error);
+            this.logger.error(`Tool execution failed: ${tool.name}`, { error });
             return {
                 toolName: tool.name,
                 tested: true,

package/client/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,EAKxB,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;IAyHvC;;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;;;OAGG;IACH,OAAO,CAAC,2BAA2B;IAoDnC;;;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,EAKxB,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;IAyHvC;;OAEG;IACH,OAAO,CAAC,sBAAsB;IA4B9B;;;OAGG;YACW,sBAAsB;IA6BpC;;OAEG;IACH,OAAO,CAAC,uBAAuB;IAyB/B;;OAEG;IACH,OAAO,CAAC,qBAAqB;IA0C7B;;;OAGG;YACW,mBAAmB;IAsCjC;;;OAGG;IACH,OAAO,CAAC,2BAA2B;IAoDnC;;;OAGG;IACH,OAAO,CAAC,2BAA2B;IA8FnC;;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/client/lib/services/assessment/modules/MCPSpecComplianceAssessor.js

@@ -90,7 +90,7 @@ export class MCPSpecComplianceAssessor extends BaseAssessor {
         const totalChecks = checksArray.length;
         const complianceScore = (passedCount / totalChecks) * 100;
         // Log score/check consistency for debugging
-        this.log(`MCP Compliance: ${passedCount}/${totalChecks} checks passed (${complianceScore.toFixed(1)}%)`);
+        this.logger.info(`MCP Compliance: ${passedCount}/${totalChecks} checks passed (${complianceScore.toFixed(1)}%)`);
         // Determine status based on protocol checks only
         let status;
         if (!protocolChecks.serverInfoValidity.passed) {
@@ -135,16 +135,16 @@ export class MCPSpecComplianceAssessor extends BaseAssessor {
         const metadata = context.serverInfo?.metadata;
         const protocolVersion = metadata?.protocolVersion;
         if (protocolVersion) {
-            this.log(`Using protocol version from metadata: ${protocolVersion}`);
+            this.logger.info(`Using protocol version from metadata: ${protocolVersion}`);
             return protocolVersion;
         }
         // Fall back to server version
         if (context.serverInfo?.version) {
-            this.log(`Using server version as protocol version: ${context.serverInfo.version}`);
+            this.logger.info(`Using server version as protocol version: ${context.serverInfo.version}`);
             return context.serverInfo.version;
         }
         // Default if no version information available
-        this.log("No protocol version information available, using default");
+        this.logger.info("No protocol version information available, using default");
         return "2025-06-18"; // Current MCP spec version as fallback
     }
     /**
@@ -183,14 +183,14 @@ export class MCPSpecComplianceAssessor extends BaseAssessor {
         // Check if name is properly set (should be a string, not null/undefined)
         if (serverInfo.name !== undefined && serverInfo.name !== null) {
             if (typeof serverInfo.name !== "string") {
-                this.log("Server info name is not a string");
+                this.logger.info("Server info name is not a string");
                 return false;
             }
         }
         // Check if metadata is properly formatted (should be an object if present)
         if (serverInfo.metadata !== undefined && serverInfo.metadata !== null) {
             if (typeof serverInfo.metadata !== "object") {
-                this.log("Server info metadata is not an object");
+                this.logger.info("Server info metadata is not an object");
                 return false;
             }
         }
@@ -297,7 +297,7 @@ export class MCPSpecComplianceAssessor extends BaseAssessor {
         const totalTools = tools.length;
         const coveragePercent = totalTools > 0 ? Math.round((withOutputSchema / totalTools) * 100) : 0;
         // Log for debugging
-        this.log(`Structured output support: ${withOutputSchema}/${totalTools} tools (${coveragePercent}%)`);
+        this.logger.info(`Structured output support: ${withOutputSchema}/${totalTools} tools (${coveragePercent}%)`);
         const coverage = {
             totalTools,
             withOutputSchema,
@@ -344,11 +344,11 @@ export class MCPSpecComplianceAssessor extends BaseAssessor {
             }
             // Check listChanged notification support
             if (capabilities.resources.listChanged) {
-                this.log("Server declares resources.listChanged notification support");
+                this.logger.info("Server declares resources.listChanged notification support");
             }
             // Check subscribe support
             if (capabilities.resources.subscribe) {
-                this.log("Server declares resource subscription support");
+                this.logger.info("Server declares resource subscription support");
             }
             this.testCount++;
         }
@@ -362,13 +362,13 @@ export class MCPSpecComplianceAssessor extends BaseAssessor {
             }
             // Check listChanged notification support
             if (capabilities.prompts.listChanged) {
-                this.log("Server declares prompts.listChanged notification support");
+                this.logger.info("Server declares prompts.listChanged notification support");
             }
             this.testCount++;
         }
         // Check logging capability
         if (capabilities.logging) {
-            this.log("Server declares logging capability");
+            this.logger.info("Server declares logging capability");
             this.testCount++;
         }
         // Determine pass/fail
@@ -433,7 +433,7 @@ export class MCPSpecComplianceAssessor extends BaseAssessor {
             (transport === "http" || transport === "streamable-http")) {
             // For HTTP transport on 2025-06-18+, headers are required
             // We assume compliance if using the new protocol version
-            this.log(`HTTP transport detected with protocol ${protocolVersion} - header compliance assumed`);
+            this.logger.info(`HTTP transport detected with protocol ${protocolVersion} - header compliance assumed`);
         }
         // Determine confidence based on detection method
         const confidence = hasTransportMetadata ? "medium" : "low";

package/client/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;IAoFvC;;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;IAqFvC;;OAEG;IACH,OAAO,CAAC,uBAAuB;IAsB/B;;OAEG;IACH,OAAO,CAAC,mBAAmB;IA0C3B;;OAEG;IACH,OAAO,CAAC,uBAAuB;CA+ChC"}

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

@@ -19,7 +19,7 @@ export class ManifestValidationAssessor extends BaseAssessor {
      * Run manifest validation assessment
      */
     async assess(context) {
-        this.log("Starting manifest validation assessment");
+        this.logger.info("Starting manifest validation assessment");
         this.testCount = 0;
         // Check if manifest is available
         if (!context.manifestJson && !context.manifestRaw) {
@@ -91,7 +91,7 @@ export class ManifestValidationAssessor extends BaseAssessor {
         if (manifest.privacy_policies &&
             Array.isArray(manifest.privacy_policies) &&
             manifest.privacy_policies.length > 0) {
-            this.log(`Validating ${manifest.privacy_policies.length} privacy policy URL(s)`);
+            this.logger.info(`Validating ${manifest.privacy_policies.length} privacy policy URL(s)`);
             const policyResults = await this.validatePrivacyPolicyUrls(manifest.privacy_policies);
             privacyPolicies = {
                 declared: manifest.privacy_policies,
@@ -121,7 +121,7 @@ export class ManifestValidationAssessor extends BaseAssessor {
         const status = this.determineManifestStatus(validationResults, hasRequiredFields);
         const explanation = this.generateExplanation(validationResults, hasRequiredFields, hasIcon, privacyPolicies);
         const recommendations = this.generateRecommendations(validationResults, privacyPolicies);
-        this.log(`Assessment complete: ${validationResults.filter((r) => r.valid).length}/${validationResults.length} checks passed`);
+        this.logger.info(`Assessment complete: ${validationResults.filter((r) => r.valid).length}/${validationResults.length} checks passed`);
         return {
             hasManifest: true,
             manifestVersion: manifest.manifest_version,
@@ -413,7 +413,9 @@ export class ManifestValidationAssessor extends BaseAssessor {
                 new URL(url);
             }
             catch (error) {
-                this.logError(`Invalid privacy policy URL format: ${url}`, error);
+                this.logger.error(`Invalid privacy policy URL format: ${url}`, {
+                    error,
+                });
                 results.push({
                     url,
                     accessible: false,
@@ -460,7 +462,9 @@ export class ManifestValidationAssessor extends BaseAssessor {
                     });
                 }
                 catch (fetchError) {
-                    this.logError(`Failed to fetch privacy policy URL: ${url}`, fetchError);
+                    this.logger.error(`Failed to fetch privacy policy URL: ${url}`, {
+                        error: fetchError,
+                    });
                     results.push({
                         url,
                         accessible: false,

package/client/lib/services/assessment/modules/PortabilityAssessor.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"PortabilityAssessor.d.ts","sourceRoot":"","sources":["../../../../src/services/assessment/modules/PortabilityAssessor.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;GAYG;AAEH,OAAO,EAAE,YAAY,EAAE,MAAM,gBAAgB,CAAC;AAC9C,OAAO,EAAE,iBAAiB,EAAE,MAAM,2BAA2B,CAAC;AAC9D,OAAO,KAAK,EACV,qBAAqB,EAGtB,MAAM,uBAAuB,CAAC;AA8H/B,qBAAa,mBAAoB,SAAQ,YAAY;IACnD;;OAEG;IACG,MAAM,CAAC,OAAO,EAAE,iBAAiB,GAAG,OAAO,CAAC,qBAAqB,CAAC;IAuHxE;;OAEG;IACH,OAAO,CAAC,oBAAoB;IAiD5B;;OAEG;IACH,OAAO,CAAC,uBAAuB;IAuE/B;;OAEG;IACH,OAAO,CAAC,QAAQ;IAiIhB;;OAEG;IACH,OAAO,CAAC,cAAc;IAkBtB;;OAEG;IACH,OAAO,CAAC,0BAA0B;IAmClC;;OAEG;IACH,OAAO,CAAC,mBAAmB;IAqC3B;;OAEG;IACH,OAAO,CAAC,uBAAuB;CAmDhC"}
+{"version":3,"file":"PortabilityAssessor.d.ts","sourceRoot":"","sources":["../../../../src/services/assessment/modules/PortabilityAssessor.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;GAYG;AAEH,OAAO,EAAE,YAAY,EAAE,MAAM,gBAAgB,CAAC;AAC9C,OAAO,EAAE,iBAAiB,EAAE,MAAM,2BAA2B,CAAC;AAC9D,OAAO,KAAK,EACV,qBAAqB,EAGtB,MAAM,uBAAuB,CAAC;AA8H/B,qBAAa,mBAAoB,SAAQ,YAAY;IACnD;;OAEG;IACG,MAAM,CAAC,OAAO,EAAE,iBAAiB,GAAG,OAAO,CAAC,qBAAqB,CAAC;IAyHxE;;OAEG;IACH,OAAO,CAAC,oBAAoB;IAiD5B;;OAEG;IACH,OAAO,CAAC,uBAAuB;IAuE/B;;OAEG;IACH,OAAO,CAAC,QAAQ;IAiIhB;;OAEG;IACH,OAAO,CAAC,cAAc;IAkBtB;;OAEG;IACH,OAAO,CAAC,0BAA0B;IAmClC;;OAEG;IACH,OAAO,CAAC,mBAAmB;IAqC3B;;OAEG;IACH,OAAO,CAAC,uBAAuB;CAmDhC"}

package/client/lib/services/assessment/modules/PortabilityAssessor.js

@@ -124,7 +124,7 @@ export class PortabilityAssessor extends BaseAssessor {
      * Run portability assessment
      */
     async assess(context) {
-        this.log("Starting portability assessment");
+        this.logger.info("Starting portability assessment");
         this.testCount = 0;
         const issues = [];
         let scannedFiles = 0;
@@ -159,7 +159,7 @@ export class PortabilityAssessor extends BaseAssessor {
         }
         // Check source code files if available
         if (context.sourceCodeFiles && context.config.enableSourceCodeAnalysis) {
-            this.log("Scanning source code files for portability issues...");
+            this.logger.info("Scanning source code files for portability issues...");
             for (const [filePath, content] of context.sourceCodeFiles) {
                 // Skip irrelevant files
                 if (this.shouldSkipFile(filePath))
@@ -185,7 +185,7 @@ export class PortabilityAssessor extends BaseAssessor {
         const status = this.determinePortabilityStatus(issues, usesDirname, usesBundleRoot);
         const explanation = this.generateExplanation(issues, usesDirname, usesBundleRoot, scannedFiles);
         const recommendations = this.generateRecommendations(issues, usesDirname, usesBundleRoot);
-        this.log(`Assessment complete: ${issues.length} portability issues found`);
+        this.logger.info(`Assessment complete: ${issues.length} portability issues found`);
         // NEW: Analyze shell commands and platform coverage (Issue #9)
         const shellCommands = this.analyzeShellCommands(context);
         const platformCoverage = this.analyzePlatformCoverage(issues);

package/client/lib/services/assessment/modules/ProhibitedLibrariesAssessor.js

@@ -16,7 +16,7 @@ export class ProhibitedLibrariesAssessor extends BaseAssessor {
      * Run prohibited libraries assessment
      */
     async assess(context) {
-        this.log("Starting prohibited libraries assessment");
+        this.logger.info("Starting prohibited libraries assessment");
         this.testCount = 0;
         const matches = [];
         const scannedFiles = [];
@@ -24,7 +24,7 @@ export class ProhibitedLibrariesAssessor extends BaseAssessor {
         let hasMediaLibraries = false;
         // Check package.json dependencies
         if (context.packageJson) {
-            this.log("Scanning package.json dependencies...");
+            this.logger.info("Scanning package.json dependencies...");
             this.testCount++;
             scannedFiles.push("package.json");
             const packageJson = context.packageJson;
@@ -64,7 +64,7 @@ export class ProhibitedLibrariesAssessor extends BaseAssessor {
         }
         // Check source code files if available
         if (context.sourceCodeFiles && context.config.enableSourceCodeAnalysis) {
-            this.log("Scanning source code files...");
+            this.logger.info("Scanning source code files...");
             for (const [filePath, content] of context.sourceCodeFiles) {
                 // Check Python requirements files
                 if (filePath.endsWith("requirements.txt") ||
@@ -126,7 +126,7 @@ export class ProhibitedLibrariesAssessor extends BaseAssessor {
         const status = this.calculateStatusFromMatches(uniqueMatches);
         const explanation = this.generateExplanation(uniqueMatches, hasFinancialLibraries, hasMediaLibraries, scannedFiles);
         const recommendations = this.generateRecommendations(uniqueMatches);
-        this.log(`Assessment complete: ${uniqueMatches.length} prohibited libraries found`);
+        this.logger.info(`Assessment complete: ${uniqueMatches.length} prohibited libraries found`);
         return {
             matches: uniqueMatches,
             scannedFiles,

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

@@ -67,7 +67,7 @@ export class PromptAssessor extends BaseAssessor {
         if (!context.prompts || context.prompts.length === 0) {
             return this.createNoPromptsResponse();
         }
-        this.log(`Testing ${context.prompts.length} prompts`);
+        this.logger.info(`Testing ${context.prompts.length} prompts`);
         // Test each prompt
         for (const prompt of context.prompts) {
             this.testCount++;
@@ -216,7 +216,7 @@ export class PromptAssessor extends BaseAssessor {
             return { success: true, unsafeContent, executionTime };
         }
         catch (error) {
-            this.logError(`Prompt execution failed: ${prompt.name}`, error);
+            this.logger.error(`Prompt execution failed: ${prompt.name}`, { error });
             return {
                 success: false,
                 unsafeContent: false,

package/client/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,EAKxB,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;IAyIxC;;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;;;OAGG;IACH,OAAO,CAAC,2BAA2B;IAiDnC;;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,EAKxB,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;IAyIxC;;OAEG;IACH,OAAO,CAAC,sBAAsB;IAyB9B;;OAEG;YACW,sBAAsB;IAuBpC;;OAEG;IACH,OAAO,CAAC,uBAAuB;IAsB/B;;OAEG;IACH,OAAO,CAAC,qBAAqB;IAwC7B;;OAEG;YACW,mBAAmB;IAiCjC;;;OAGG;IACH,OAAO,CAAC,2BAA2B;IAiDnC;;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"}