clean up schemas

Author: sayansisodiya

src/github/process-review.ts

@@ -81,6 +81,8 @@ export async function processReview(
           const inlineComments = comments.filter(c => c.type === 'inline');
           const generalComments = comments.filter(c => c.type === 'general');
 
+          // TODO(sayans): GitHub API allows <= 30 comments per review, so we need to add splitting logic if there are > 30 comments
+
           console.log(`📝 Collected ${inlineComments.length} inline comments and ${generalComments.length} general comments`);
 
           // Create review summary from general comments

src/mcp/tools/leave_comment.ts

@@ -2,9 +2,6 @@ import { getCollector } from '../comment-collector.js';
 
 export interface LeaveGeneralCommentArgs {
   message: string;
-  owner: string;
-  repo: string;
-  pr_number: number;
 }
 
 export async function leaveGeneralComment(

src/mcp/tools/leave_inline_comment.ts

@@ -2,9 +2,6 @@ import { getCollector } from '../comment-collector.js';
 
 export interface LeaveInlineCommentArgs {
   message: string;
-  owner: string;
-  repo: string;
-  pr_number: number;
   path: string;
   line: number;
   commit_sha?: string;

src/mcp/validation.ts

@@ -11,11 +11,11 @@ const ownerRepoPr = z.object({
 });
 
 // Schema definitions
-const LeaveGeneralCommentArgsSchema = ownerRepoPr.extend({
+const LeaveGeneralCommentArgsSchema = z.object({
   message: z.string(),
 });
 
-const LeaveInlineCommentArgsSchema = ownerRepoPr.extend({
+const LeaveInlineCommentArgsSchema = z.object({
   message: z.string(),
   path: z.string(),
   line: z.number(),