0xray 3.1.1 → 3.3.0

package/dist/mcps/knowledge-skills/code-review.server.js

@@ -4,152 +4,127 @@
  * Knowledge skill for automated code review, quality assessment,
  * and best practices validation - provides comprehensive code quality analysis
  */
-import { Server } from "@modelcontextprotocol/sdk/server/index.js";
-import { StdioServerTransport } from "@modelcontextprotocol/sdk/server/stdio.js";
-import { CallToolRequestSchema, ListToolsRequestSchema, } from "@modelcontextprotocol/sdk/types.js";
+import { XrayKnowledgeSkillBase } from "../shared/knowledge-skill-base.js";
 import * as fs from "fs";
 import * as path from "path";
-import { createGracefulShutdown } from "../../utils/shutdown-handler.js";
 import { tryLLMGovernance } from "../../governance/llm-governance-provider.js";
-class XrayCodeReviewServer {
-    server;
+class XrayCodeReviewServer extends XrayKnowledgeSkillBase {
     constructor() {
-        this.server = new Server({
-            name: "code-review", version: "3.1.0",
-        }, {
-            capabilities: {
-                tools: {},
-            },
-        });
-        this.setupToolHandlers();
-        // Server initialization - removed unnecessary startup logging
-    }
-    setupToolHandlers() {
-        this.server.setRequestHandler(ListToolsRequestSchema, async () => {
-            return {
-                tools: [
-                    {
-                        name: "analyze_code_quality",
-                        description: "Analyze code quality metrics, identify issues, and provide improvement suggestions",
-                        inputSchema: {
-                            type: "object",
-                            properties: {
-                                filePath: {
-                                    type: "string",
-                                    description: "Path to the file to analyze",
-                                },
-                                includeMetrics: {
-                                    type: "boolean",
-                                    description: "Include detailed code metrics in the analysis",
-                                    default: true,
-                                },
-                                focusAreas: {
-                                    type: "array",
-                                    items: {
-                                        type: "string",
-                                        enum: [
-                                            "security",
-                                            "performance",
-                                            "maintainability",
-                                            "style",
-                                            "logic",
-                                        ],
-                                    },
-                                    description: "Specific areas to focus the review on",
-                                },
-                            },
-                            required: ["filePath"],
+        super("code-review", "3.1.0");
+        this.tools = [
+            {
+                name: "analyze_code_quality",
+                description: "Analyze code quality metrics, identify issues, and provide improvement suggestions",
+                inputSchema: {
+                    type: "object",
+                    properties: {
+                        filePath: {
+                            type: "string",
+                            description: "Path to the file to analyze",
                         },
-                    },
-                    {
-                        name: "review_pull_request",
-                        description: "Review a pull request by analyzing changed files and providing comprehensive feedback",
-                        inputSchema: {
-                            type: "object",
-                            properties: {
-                                files: {
-                                    type: "array",
-                                    items: { type: "string" },
-                                    description: "List of file paths to review",
-                                },
-                                baseBranch: {
-                                    type: "string",
-                                    description: "Base branch for comparison",
-                                    default: "main",
-                                },
-                                focusAreas: {
-                                    type: "array",
-                                    items: {
-                                        type: "string",
-                                        enum: [
-                                            "security",
-                                            "performance",
-                                            "maintainability",
-                                            "style",
-                                            "logic",
-                                        ],
-                                    },
-                                    description: "Specific areas to focus the review on",
-                                },
+                        includeMetrics: {
+                            type: "boolean",
+                            description: "Include detailed code metrics in the analysis",
+                            default: true,
+                        },
+                        focusAreas: {
+                            type: "array",
+                            items: {
+                                type: "string",
+                                enum: [
+                                    "security",
+                                    "performance",
+                                    "maintainability",
+                                    "style",
+                                    "logic",
+                                ],
                             },
-                            required: ["files"],
+                            description: "Specific areas to focus the review on",
                         },
                     },
-                    {
-                        name: "check_best_practices",
-                        description: "Check adherence to coding best practices and standards",
-                        inputSchema: {
-                            type: "object",
-                            properties: {
-                                filePath: {
-                                    type: "string",
-                                    description: "Path to the file to check",
-                                },
-                                language: {
-                                    type: "string",
-                                    description: "Programming language (typescript, javascript, python, etc.)",
-                                },
-                                standards: {
-                                    type: "array",
-                                    items: { type: "string" },
-                                    description: "Specific standards to check (airbnb, google, microsoft, etc.)",
-                                },
+                    required: ["filePath"],
+                },
+            },
+            {
+                name: "review_pull_request",
+                description: "Review a pull request by analyzing changed files and providing comprehensive feedback",
+                inputSchema: {
+                    type: "object",
+                    properties: {
+                        files: {
+                            type: "array",
+                            items: { type: "string" },
+                            description: "List of file paths to review",
+                        },
+                        baseBranch: {
+                            type: "string",
+                            description: "Base branch for comparison",
+                            default: "main",
+                        },
+                        focusAreas: {
+                            type: "array",
+                            items: {
+                                type: "string",
+                                enum: [
+                                    "security",
+                                    "performance",
+                                    "maintainability",
+                                    "style",
+                                    "logic",
+                                ],
                             },
-                            required: ["filePath"],
+                            description: "Specific areas to focus the review on",
                         },
                     },
-                    {
-                        name: "analyze_proposal",
-                        description: "Analyze an inference proposal (pattern/bug/refactor) from a code-review perspective and return a structured governance decision",
-                        inputSchema: {
-                            type: "object",
-                            properties: {
-                                proposalTitle: { type: "string" },
-                                proposalDescription: { type: "string" },
-                                evidence: { type: "array", items: { type: "string" } },
-                                proposalType: { type: "string" },
-                            },
-                            required: ["proposalTitle", "proposalDescription"],
+                    required: ["files"],
+                },
+            },
+            {
+                name: "check_best_practices",
+                description: "Check adherence to coding best practices and standards",
+                inputSchema: {
+                    type: "object",
+                    properties: {
+                        filePath: {
+                            type: "string",
+                            description: "Path to the file to check",
+                        },
+                        language: {
+                            type: "string",
+                            description: "Programming language (typescript, javascript, python, etc.)",
+                        },
+                        standards: {
+                            type: "array",
+                            items: { type: "string" },
+                            description: "Specific standards to check (airbnb, google, microsoft, etc.)",
                         },
                     },
-                ],
-            };
-        });
-        this.server.setRequestHandler(CallToolRequestSchema, async (request) => {
-            const { name, arguments: args } = request.params;
-            switch (name) {
-                case "analyze_code_quality":
-                    return await this.analyzeCodeQuality(args);
-                case "review_pull_request":
-                    return await this.reviewPullRequest(args);
-                case "check_best_practices":
-                    return await this.checkBestPractices(args);
-                case "analyze_proposal":
-                    return await this.analyzeProposal(args);
-                default:
-                    throw new Error(`Unknown tool: ${name}`);
-            }
-        });
+                    required: ["filePath"],
+                },
+            },
+            {
+                name: "analyze_proposal",
+                description: "Analyze an inference proposal (pattern/bug/refactor) from a code-review perspective and return a structured governance decision",
+                inputSchema: {
+                    type: "object",
+                    properties: {
+                        proposalTitle: { type: "string" },
+                        proposalDescription: { type: "string" },
+                        evidence: { type: "array", items: { type: "string" } },
+                        proposalType: { type: "string" },
+                    },
+                    required: ["proposalTitle", "proposalDescription"],
+                },
+            },
+        ];
+        this.handlers = {
+            "analyze_code_quality": async (args) => this.analyzeCodeQuality(args),
+            "review_pull_request": async (args) => this.reviewPullRequest(args),
+            "check_best_practices": async (args) => this.checkBestPractices(args),
+            "analyze_proposal": async (args) => this.analyzeProposal(args),
+        };
+        this.setupToolHandlers();
+        // Server initialization - removed unnecessary startup logging
     }
     async analyzeCodeQuality(args) {
         const { filePath, includeMetrics = true, focusAreas } = args;
@@ -804,15 +779,6 @@ class XrayCodeReviewServer {
         }, 0);
         return Math.max(0, baseScore - deductions);
     }
-    async run() {
-        const transport = new StdioServerTransport();
-        await this.server.connect(transport);
-        // Use centralized shutdown handler
-        createGracefulShutdown({
-            serverName: "code-review.server",
-            server: this.server,
-        });
-    }
 }
 // Run the server if this file is executed directly
 if (import.meta.url === `file://${fs.realpathSync(process.argv[1])}`) {