don't allow rejecting prs

entry.js

@@ -40481,7 +40481,7 @@ function query({
 // package.json
 var package_default = {
   name: "@pullfrog/action",
-  version: "0.0.87",
+  version: "0.0.88",
   type: "module",
   files: [
     "index.js",

fixtures/basic.txt

@@ -1 +1 @@
-create a new pr to pullfrogai/scratch that calculates the number of toonies in @ShawnMorreau's bank account and estimates how long it will take to complete his toonie chamber
+Review https://github.com/pullfrogai/scratch/pull/17

mcp-server.js

@@ -102593,7 +102593,6 @@ var PullRequestInfoTool = tool({
 // mcp/review.ts
 var Review = type({
   pull_number: type.number.describe("The pull request number to review"),
-  event: type.enumerated("APPROVE", "REQUEST_CHANGES", "COMMENT").describe("'APPROVE', 'REQUEST_CHANGES', or 'COMMENT' (the review action)"),
   body: type.string.describe(
     "Brief summary or general feedback that doesn't apply to specific code locations. Keep it concise - most feedback should be in the 'comments' array."
   ).optional(),
@@ -102616,7 +102615,7 @@ var ReviewTool = tool({
   name: "submit_pull_request_review",
   description: "Submit a review (approve, request changes, or comment) for an existing pull request. IMPORTANT: Use 'comments' array for ALL specific code issues at the line-level. Only use 'body' for a brief summary or feedback that doesn't apply to a specific location.",
   parameters: Review,
-  execute: contextualize(async ({ pull_number, event, body, commit_id, comments = [] }, ctx) => {
+  execute: contextualize(async ({ pull_number, body, commit_id, comments = [] }, ctx) => {
     const pr = await ctx.octokit.rest.pulls.get({
       owner: ctx.owner,
       repo: ctx.name,
@@ -102626,7 +102625,7 @@ var ReviewTool = tool({
       owner: ctx.owner,
       repo: ctx.name,
       pull_number,
-      event
+      event: "COMMENT"
     };
     if (body) params.body = body;
     if (commit_id) {

mcp/review.ts

@@ -4,9 +4,6 @@ import { contextualize, tool } from "./shared.ts";
 
 export const Review = type({
   pull_number: type.number.describe("The pull request number to review"),
-  event: type
-    .enumerated("APPROVE", "REQUEST_CHANGES", "COMMENT")
-    .describe("'APPROVE', 'REQUEST_CHANGES', or 'COMMENT' (the review action)"),
   body: type.string
     .describe(
       "Brief summary or general feedback that doesn't apply to specific code locations. Keep it concise - most feedback should be in the 'comments' array."
@@ -45,7 +42,7 @@ export const ReviewTool = tool({
     "IMPORTANT: Use 'comments' array for ALL specific code issues at the line-level. " +
     "Only use 'body' for a brief summary or feedback that doesn't apply to a specific location.",
   parameters: Review,
-  execute: contextualize(async ({ pull_number, event, body, commit_id, comments = [] }, ctx) => {
+  execute: contextualize(async ({ pull_number, body, commit_id, comments = [] }, ctx) => {
     // Get the PR to determine the head commit if commit_id not provided
     const pr = await ctx.octokit.rest.pulls.get({
       owner: ctx.owner,
@@ -58,7 +55,7 @@ export const ReviewTool = tool({
       owner: ctx.owner,
       repo: ctx.name,
       pull_number,
-      event,
+      event: "COMMENT",
     };
     if (body) params.body = body;
     if (commit_id) {

package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pullfrog/action",
-  "version": "0.0.87",
+  "version": "0.0.88",
   "type": "module",
   "files": [
     "index.js",

todo.md

@@ -1,6 +1,6 @@
 [x] add modes to prompt 
 [x] progressively update comment
-[] don't allow rejecting prs
+[x] don't allow rejecting prs
 [x] fix pnpm caching
 [] fix prompt to avoid narration like "I just read all tools from MCP server"
 [] investigate including terminal output from bash commands as collapsed groups