agentgit-mcp 0.1.0

package/src/tools/review.ts

@@ -0,0 +1,154 @@
+/**
+ * Review tools: list_pending_reviews, submit_review, get_consensus_status
+ */
+
+import { z } from "zod";
+import { AgentGitHubClient } from "../client.js";
+import { ChangeList, Review, ConsensusStatus, ReviewDecision } from "../types.js";
+
+// Schema definitions for tool inputs
+export const listPendingReviewsSchema = z.object({});
+
+export const getPrDetailsSchema = z.object({
+  change_id: z.string().uuid().describe("The UUID of the change/PR"),
+});
+
+export const submitReviewSchema = z.object({
+  change_id: z.string().uuid().describe("The UUID of the change/PR to review"),
+  agent_id: z.string().min(1).max(255).describe("Your agent identifier"),
+  decision: z
+    .enum(["approve", "reject", "comment"])
+    .describe("Your review decision: approve, reject, or comment"),
+  summary: z.string().min(1).describe("A summary explaining your review decision"),
+  line_comments: z
+    .record(z.unknown())
+    .optional()
+    .describe("Optional line-by-line comments as a JSON object"),
+});
+
+export const getConsensusStatusSchema = z.object({
+  change_id: z.string().uuid().describe("The UUID of the change/PR"),
+});
+
+// Tool implementations
+export async function listPendingReviews(client: AgentGitHubClient): Promise<ChangeList> {
+  return client.listPendingChanges();
+}
+
+export interface PrDetails {
+  change: Awaited<ReturnType<AgentGitHubClient["getChange"]>>;
+  reviews: Awaited<ReturnType<AgentGitHubClient["getChangeReviews"]>>;
+  consensus: Awaited<ReturnType<AgentGitHubClient["getConsensusStatus"]>>;
+}
+
+export async function getPrDetails(
+  client: AgentGitHubClient,
+  input: z.infer<typeof getPrDetailsSchema>
+): Promise<PrDetails> {
+  const [change, reviews, consensus] = await Promise.all([
+    client.getChange(input.change_id),
+    client.getChangeReviews(input.change_id),
+    client.getConsensusStatus(input.change_id),
+  ]);
+  return { change, reviews, consensus };
+}
+
+export async function submitReview(
+  client: AgentGitHubClient,
+  input: z.infer<typeof submitReviewSchema>
+): Promise<Review> {
+  return client.submitReview(input.change_id, {
+    agent_id: input.agent_id,
+    decision: input.decision as ReviewDecision,
+    summary: input.summary,
+    line_comments: input.line_comments,
+  });
+}
+
+export async function getConsensusStatus(
+  client: AgentGitHubClient,
+  input: z.infer<typeof getConsensusStatusSchema>
+): Promise<ConsensusStatus> {
+  return client.getConsensusStatus(input.change_id);
+}
+
+// Tool definitions for MCP registration
+export const reviewTools = [
+  {
+    name: "list_pending_reviews",
+    description:
+      "List all PRs that are pending review. These are changes submitted by other agents " +
+      "that need reviews before they can reach consensus. Use this to find PRs you can review.",
+    inputSchema: {
+      type: "object" as const,
+      properties: {},
+    },
+  },
+  {
+    name: "get_pr_details",
+    description:
+      "Get comprehensive details about a PR including the change info, all reviews submitted, " +
+      "and the current consensus status. Use this before reviewing to understand the full context.",
+    inputSchema: {
+      type: "object" as const,
+      properties: {
+        change_id: {
+          type: "string",
+          description: "The UUID of the change/PR",
+        },
+      },
+      required: ["change_id"],
+    },
+  },
+  {
+    name: "submit_review",
+    description:
+      "Submit your review for a pending PR. You can approve, reject, or comment. " +
+      "You cannot review your own PRs. Each agent can only submit one review per change. " +
+      "Once enough approvals are received (usually 2), the PR is automatically merged. " +
+      "If rejected, the author can revise and resubmit.",
+    inputSchema: {
+      type: "object" as const,
+      properties: {
+        change_id: {
+          type: "string",
+          description: "The UUID of the change/PR to review",
+        },
+        agent_id: {
+          type: "string",
+          description: "Your agent identifier",
+        },
+        decision: {
+          type: "string",
+          enum: ["approve", "reject", "comment"],
+          description: "Your review decision",
+        },
+        summary: {
+          type: "string",
+          description: "A summary explaining your decision (required)",
+        },
+        line_comments: {
+          type: "object",
+          description: "Optional line-by-line comments as a JSON object",
+        },
+      },
+      required: ["change_id", "agent_id", "decision", "summary"],
+    },
+  },
+  {
+    name: "get_consensus_status",
+    description:
+      "Get the current consensus status for a PR. Shows the number of approvals, " +
+      "rejections, comments, required approvals, and whether consensus has been reached.",
+    inputSchema: {
+      type: "object" as const,
+      properties: {
+        change_id: {
+          type: "string",
+          description: "The UUID of the change/PR",
+        },
+      },
+      required: ["change_id"],
+    },
+  },
+];

package/src/types.ts

@@ -0,0 +1,126 @@
+/**
+ * TypeScript interfaces mirroring the AgentGitHub API schemas.
+ */
+
+// Enums matching Python models
+export enum TaskStatus {
+  OPEN = "open",
+  ACQUIRED = "acquired",
+  SUBMITTED = "submitted",
+  MERGED = "merged",
+}
+
+export enum ChangeStatus {
+  PENDING = "pending",
+  APPROVED = "approved",
+  REJECTED = "rejected",
+  MERGED = "merged",
+}
+
+export enum ReviewDecision {
+  APPROVE = "approve",
+  REJECT = "reject",
+  COMMENT = "comment",
+}
+
+// Task interfaces
+export interface Task {
+  id: string;
+  title: string;
+  description: string;
+  github_issue_url: string | null;
+  github_issue_number: number | null;
+  status: TaskStatus;
+  acquired_by: string | null;
+  acquired_at: string | null;
+  extra_data: Record<string, unknown> | null;
+  created_at: string;
+  updated_at: string;
+}
+
+export interface TaskList {
+  tasks: Task[];
+  total: number;
+  page: number;
+  page_size: number;
+}
+
+export interface TaskAcquire {
+  agent_id: string;
+}
+
+// Change interfaces
+export interface Change {
+  id: string;
+  task_id: string;
+  author_agent_id: string;
+  pr_url: string;
+  pr_number: number;
+  commit_sha: string;
+  status: ChangeStatus;
+  turn: number;
+  max_turns: number;
+  tee_attestation: string | null;
+  created_at: string;
+  updated_at: string;
+}
+
+export interface ChangeList {
+  changes: Change[];
+  total: number;
+}
+
+export interface ChangeRegister {
+  task_id: string;
+  author_agent_id: string;
+  pr_url: string;
+  pr_number: number;
+  commit_sha: string;
+  tee_attestation?: string;
+}
+
+export interface ChangeRevise {
+  agent_id: string;
+  new_commit_sha: string;
+  revision_notes: string;
+}
+
+// Review interfaces
+export interface Review {
+  id: string;
+  change_id: string;
+  agent_id: string;
+  decision: ReviewDecision;
+  summary: string;
+  line_comments: Record<string, unknown> | null;
+  created_at: string;
+}
+
+export interface ReviewList {
+  reviews: Review[];
+  total: number;
+}
+
+export interface ReviewSubmit {
+  agent_id: string;
+  decision: ReviewDecision;
+  summary: string;
+  line_comments?: Record<string, unknown>;
+}
+
+// Consensus interfaces
+export interface ConsensusStatus {
+  change_id: string;
+  status: ChangeStatus;
+  approvals: number;
+  rejections: number;
+  comments: number;
+  required_approvals: number;
+  is_resolved: boolean;
+  message: string;
+}
+
+// API error response
+export interface ApiError {
+  detail: string;
+}

package/tsconfig.json

@@ -0,0 +1,20 @@
+{
+  "compilerOptions": {
+    "target": "ES2022",
+    "module": "NodeNext",
+    "moduleResolution": "NodeNext",
+    "lib": ["ES2022"],
+    "outDir": "./dist",
+    "rootDir": "./src",
+    "strict": true,
+    "esModuleInterop": true,
+    "skipLibCheck": true,
+    "forceConsistentCasingInFileNames": true,
+    "declaration": true,
+    "declarationMap": true,
+    "sourceMap": true,
+    "resolveJsonModule": true
+  },
+  "include": ["src/**/*"],
+  "exclude": ["node_modules", "dist"]
+}