remove extra mcp tools; they were causing issues. update docs accordingly

Author: sayansisodiya

README.md

@@ -230,9 +230,6 @@ The app includes a Model Context Protocol (MCP) server that exposes GitHub opera
 ### Available MCP Tools
 - `leave_general_comment` - Leave general comments on pull requests
 - `leave_inline_comment` - Leave inline comments on specific lines
-- `create_check_run` - Create or update check run status
-- `get_pr_info` - Get pull request details and optionally the diff
-- `trigger_review` - Start the code review process
 - `get_pr_comments` - Get all comments on a pull request
 
 ### Usage with AI Agents

config.yml

@@ -57,7 +57,6 @@ amp:
     1. Look at the pull request details to fetch the context before reviewing.
     2. Review the diff content to identify issues.
     3. Leave specific inline comments or code suggestions for each issue found.
-    4. Create check run status when the code review is complete.
 
     Always leave actionable, specific comments with suggested fixes.
 
@@ -129,28 +128,6 @@ amp:
         - "Required: message, owner, repo, pr_number, path, line"
         - "Optional: commit_sha (will be fetched from PR if not provided)"
         - "Example: Comment on line 25 of src/auth.js about missing error handling"
-    - name: create_check_run
-      description: Create or update GitHub check run status
-      instructions: 
-        - "Use to mark review completion with check run status"
-        - "Required: owner, repo, commit_sha, status ('queued', 'in_progress', 'completed')"
-        - "Optional: conclusion ('success', 'failure', 'neutral'), title, summary, details_url"
-        - "Use 'completed' status with 'success' conclusion when review is done"
-        - "Use 'completed' status with 'failure' conclusion for critical issues"
-    - name: get_pr_info
-      description: Get pull request details
-      instructions: 
-        - "Use to understand context about the pull request before reviewing the diff"
-        - "Required: owner, repo, pr_number"
-        - "Optional: include_diff (boolean) to also fetch the diff content"
-        - "Returns PR info, repository info, and optionally diff content"
-    - name: trigger_review
-      description: Start code review process
-      instructions: 
-        - "Usually called automatically, but available if needed to retrigger code review"
-        - "Required: owner, repo, pr_number"
-        - "Optional: commit_sha, force (boolean)"
-        - "Creates check run and initiates review process"
     - name: get_pr_comments
       description: Get all comments on a pull request
       instructions: 

src/mcp/README.md

@@ -25,39 +25,7 @@ Leave inline comments on specific lines in pull requests.
 - `line` (number, required): Line number for the inline comment
 - `commit_sha` (string, optional): Commit SHA (will be fetched if not provided)
 
-### 3. `create_check_run`
-Create or update check run status.
-
-**Parameters:**
-- `owner` (string, required): Repository owner
-- `repo` (string, required): Repository name
-- `commit_sha` (string, required): Commit SHA to create check run for
-- `status` (string, required): Check run status - "queued", "in_progress", or "completed"
-- `conclusion` (string, optional): Check run conclusion - "success", "failure", "neutral", "cancelled", "skipped", or "timed_out"
-- `title` (string, optional): Check run title
-- `summary` (string, optional): Check run summary
-- `details_url` (string, optional): Details URL for the check run
-
-### 4. `get_pr_info`
-Get pull request details and optionally the diff.
-
-**Parameters:**
-- `owner` (string, required): Repository owner
-- `repo` (string, required): Repository name
-- `pr_number` (number, required): Pull request number
-- `include_diff` (boolean, optional): Include diff content (default: false)
-
-### 5. `trigger_review`
-Start the code review process.
-
-**Parameters:**
-- `owner` (string, required): Repository owner
-- `repo` (string, required): Repository name
-- `pr_number` (number, required): Pull request number
-- `commit_sha` (string, optional): Specific commit SHA to review
-- `force` (boolean, optional): Force re-review even if already reviewed (default: false)
-
-### 6. `get_pr_comments`
+### 3. `get_pr_comments`
 Get all comments on a pull request.
 
 **Parameters:**
@@ -160,10 +128,10 @@ This allows Amp to automatically use the GitHub MCP tools during code reviews.
 ```
 You have access to GitHub tools. To review a pull request:
 
-1. Use get_pr_info to understand the context
+1. Use get_pr_comments to see existing feedback
 2. Review the code changes  
 3. Use leave_inline_comment to add specific feedback
-4. Use create_check_run to mark the review complete
+4. Use leave_general_comment for overall observations
 
 Owner: octocat
 Repo: Hello-World
@@ -174,23 +142,17 @@ PR Number: 123
 
 The tools are designed to work together in a typical code review workflow:
 
-1. **Get Context**: `get_pr_info` to fetch PR details and optionally the diff
-2. **Check Existing Feedback**: `get_pr_comments` to see what's already been discussed
-3. **Leave Feedback**: 
+1. **Check Existing Feedback**: `get_pr_comments` to see what's already been discussed
+2. **Leave Feedback**: 
    - `leave_inline_comment` for specific line-level issues
    - `leave_general_comment` for overall observations
-4. **Update Status**: `create_check_run` to mark the review as complete
-5. **Re-trigger if Needed**: `trigger_review` to restart the process
 
 ## Development
 
 The tools are organized in the `/src/mcp/tools/` directory:
 - `leave_comment.ts` - General comment functionality
 - `leave_inline_comment.ts` - Inline comment functionality
-- `create_check_run.ts` - Check run status updates
-- `get_pr_info.ts` - PR information retrieval  
 - `get_pr_comments.ts` - PR comments retrieval
-- `trigger_review.ts` - Review triggering
 
 The server implementations are:
 - `server.ts` - Standalone stdio MCP server

src/mcp/http-server.ts

@@ -3,16 +3,10 @@ import { cors } from "hono/cors";
 import { getConfig, Config } from "../config.js";
 import { leaveGeneralComment } from "./tools/leave_comment.js";
 import { leaveInlineComment } from "./tools/leave_inline_comment.js";
-import { createCheckRun } from "./tools/create_check_run.js";
-import { getPRInfo } from "./tools/get_pr_info.js";
-import { triggerReview } from "./tools/trigger_review.js";
 import { getPRComments } from "./tools/get_pr_comments.js";
 import {
   validateLeaveGeneralCommentArgs,
   validateLeaveInlineCommentArgs,
-  validateCreateCheckRunArgs,
-  validateGetPRInfoArgs,
-  validateTriggerReviewArgs,
   validateGetPRCommentsArgs,
 } from "./validation.js";
 
@@ -263,108 +257,6 @@ function handleToolsList(c: Context, body: unknown) {
           required: ["message", "owner", "repo", "pr_number", "path", "line"],
         },
       },
-      {
-        name: "create_check_run",
-        description: "Create or update check run status",
-        inputSchema: {
-          type: "object",
-          properties: {
-            owner: {
-              type: "string",
-              description: "Repository owner",
-            },
-            repo: {
-              type: "string",
-              description: "Repository name",
-            },
-            commit_sha: {
-              type: "string",
-              description: "Commit SHA to create check run for",
-            },
-            status: {
-              type: "string",
-              enum: ["queued", "in_progress", "completed"],
-              description: "Check run status",
-            },
-            conclusion: {
-              type: "string",
-              enum: ["success", "failure", "neutral", "cancelled", "skipped", "timed_out"],
-              description: "Check run conclusion (optional)",
-            },
-            title: {
-              type: "string",
-              description: "Check run title (optional)",
-            },
-            summary: {
-              type: "string",
-              description: "Check run summary (optional)",
-            },
-            details_url: {
-              type: "string",
-              description: "Details URL for the check run (optional)",
-            },
-          },
-          required: ["owner", "repo", "commit_sha", "status"],
-        },
-      },
-      {
-        name: "get_pr_info",
-        description: "Get pull request details",
-        inputSchema: {
-          type: "object",
-          properties: {
-            owner: {
-              type: "string",
-              description: "Repository owner",
-            },
-            repo: {
-              type: "string",
-              description: "Repository name",
-            },
-            pr_number: {
-              type: "number",
-              description: "Pull request number",
-            },
-            include_diff: {
-              type: "boolean",
-              description: "Include diff content (optional, default: false)",
-              default: false,
-            },
-          },
-          required: ["owner", "repo", "pr_number"],
-        },
-      },
-      {
-        name: "trigger_review",
-        description: "Start code review process",
-        inputSchema: {
-          type: "object",
-          properties: {
-            owner: {
-              type: "string",
-              description: "Repository owner",
-            },
-            repo: {
-              type: "string",
-              description: "Repository name",
-            },
-            pr_number: {
-              type: "number",
-              description: "Pull request number",
-            },
-            commit_sha: {
-              type: "string",
-              description: "Specific commit SHA to review (optional)",
-            },
-            force: {
-              type: "boolean",
-              description: "Force re-review even if already reviewed (optional)",
-              default: false,
-            },
-          },
-          required: ["owner", "repo", "pr_number"],
-        },
-      },
       {
         name: "get_pr_comments",
         description: "Get all comments on a pull request",
@@ -446,26 +338,6 @@ async function handleToolsCall(c: Context, body: unknown, config: Config) {
         }
         break;
 
-      case "create_check_run":
-        result = await createCheckRun(
-          validateCreateCheckRunArgs(args),
-          config
-        );
-        break;
-
-      case "get_pr_info":
-        result = await getPRInfo(
-          validateGetPRInfoArgs(args),
-          config
-        );
-        break;
-
-      case "trigger_review":
-        result = await triggerReview(
-          validateTriggerReviewArgs(args),
-          config
-        );
-        break;
 
       case "get_pr_comments":
         result = await getPRComments(