don't allow rejecting prs

.husky/pre-commit

@@ -7,4 +7,4 @@ echo "🔨 Building action..."
 pnpm build
 
 # Add the built files and lockfile to the commit
-git add entry.js pnpm-lock.yaml
+git add entry.js mcp-server.js pnpm-lock.yaml

README.md

@@ -9,9 +9,6 @@ GitHub Action for running Claude Code and other agents via Pullfrog.
 ```bash
 # Install dependencies
 pnpm install
-
-# Test with default prompt
-npm run play              # Run locally on your machine
 ```
 
 ## Testing with play.ts

agents/claude.ts

@@ -67,7 +67,7 @@ export const claude: Agent = {
 
       // Write tarball to file
       const fileStream = createWriteStream(tarballPath);
-      await pipeline(response.body as any, fileStream);
+      await pipeline(response.body!, fileStream);
       log.info(`Downloaded tarball to ${tarballPath}`);
 
       // Extract tarball

entry.ts

@@ -6,12 +6,8 @@
 
 import * as core from "@actions/core";
 import { type Inputs, main } from "./main.ts";
-import { createMcpServer } from "./mcp/server.ts";
 import { log } from "./utils/cli.ts";
 
-// Export createMcpServer so it can be called from the spawned MCP process
-export { createMcpServer };
-
 async function run(): Promise<void> {
   // Change to GITHUB_WORKSPACE if set (this is where actions/checkout puts the repo)
   // JavaScript actions run from the action's directory, not the checked-out repo

esbuild.config.js

@@ -1,11 +1,7 @@
 import { build } from "esbuild";
 
-// Build the GitHub Action bundle only
-// For npm package builds, use zshy (pnpm build:npm)
-await build({
-  entryPoints: ["./entry.ts"],
+const sharedConfig = {
   bundle: true,
-  outfile: "./entry.js",
   format: "esm",
   platform: "node",
   target: "node20",
@@ -19,13 +15,28 @@ await build({
     "sury",
   ],
   // Provide a proper require shim for CommonJS modules bundled into ESM
+  // We use a unique variable name to avoid conflicts with bundled imports
   banner: {
-    js: `import { createRequire } from 'module'; import { fileURLToPath } from 'url'; import { dirname } from 'path'; const require = createRequire(import.meta.url); const __filename = fileURLToPath(import.meta.url); const __dirname = dirname(__filename);`,
+    js: `import { createRequire as __createRequire } from 'module'; import { fileURLToPath as __fileURLToPath } from 'url'; import { dirname as __dirnameFn } from 'path'; const require = __createRequire(import.meta.url); const __filename = __fileURLToPath(import.meta.url); const __dirname = __dirnameFn(__filename);`,
   },
   // Enable tree-shaking to remove unused code
   treeShaking: true,
   // Drop console statements in production (but keep for debugging)
   drop: [],
+};
+
+// Build the main entry bundle (without MCP)
+await build({
+  ...sharedConfig,
+  entryPoints: ["./entry.ts"],
+  outfile: "./entry.js",
+});
+
+// Build the MCP server bundle
+await build({
+  ...sharedConfig,
+  entryPoints: ["./mcp/server.ts"],
+  outfile: "./mcp-server.js",
 });
 
 console.log("✅ Build completed successfully!");

fixtures/basic.txt

@@ -1 +1 @@
-run npx cowsay "don't eat me"
+Review https://github.com/pullfrogai/scratch/pull/17

main.ts

@@ -1,5 +1,3 @@
-import { readdir } from "node:fs/promises";
-import { join } from "node:path";
 import { type } from "arktype";
 import { claude } from "./agents/claude.ts";
 import { createMcpConfigs } from "./mcp/config.ts";
@@ -23,42 +21,8 @@ export interface MainResult {
 
 export type PromptJSON = {};
 
-async function printDirectoryTree(dir: string, prefix = "", rootDir = dir): Promise<string> {
-  const entries = await readdir(dir, { withFileTypes: true });
-  const lines: string[] = [];
-
-  for (let i = 0; i < entries.length; i++) {
-    const entry = entries[i];
-    const isLast = i === entries.length - 1;
-    const currentPrefix = isLast ? "└── " : "├── ";
-    const nextPrefix = isLast ? "    " : "│   ";
-
-    const fullPath = join(dir, entry.name);
-    lines.push(`${prefix}${currentPrefix}${entry.name}`);
-
-    if (entry.isDirectory()) {
-      const subTree = await printDirectoryTree(fullPath, `${prefix}${nextPrefix}`, rootDir);
-      lines.push(subTree);
-    }
-  }
-
-  return lines.join("\n");
-}
-
 export async function main(inputs: Inputs): Promise<MainResult> {
   try {
-    // Debug: Print current directory tree before anything runs
-    const cwd = process.cwd();
-    log.info(`Current working directory: ${cwd}`);
-    try {
-      const tree = await printDirectoryTree(cwd);
-      log.info(`Directory tree:\n${tree}`);
-    } catch (error) {
-      log.warning(
-        `Failed to print directory tree: ${error instanceof Error ? error.message : String(error)}`
-      );
-    }
-
     log.info(`🐸 Running pullfrog/action@${packageJson.version}...`);
 
     setupGitConfig();

mcp/config.ts

@@ -3,6 +3,7 @@
  */
 
 import type { McpServerConfig } from "@anthropic-ai/claude-agent-sdk";
+import { fromHere } from "@ark/fs";
 import { parseRepoContext } from "../utils/github.ts";
 
 export const ghPullfrogMcpName = "gh-pullfrog";
@@ -15,19 +16,14 @@ export function createMcpConfigs(githubInstallationToken: string): McpConfigs {
   const repoContext = parseRepoContext();
   const githubRepository = `${repoContext.owner}/${repoContext.name}`;
 
-  // Get absolute path to entry.js - use GITHUB_ACTION_PATH if available, otherwise current directory
-  const entryPath = process.env.GITHUB_ACTION_PATH
-    ? `${process.env.GITHUB_ACTION_PATH}/entry.js`
-    : `${process.cwd()}/entry.js`;
+  // In production (GitHub Actions), mcp-server.js is in same directory as entry.js (where this is bundled)
+  // In development, server.ts is in the same directory as this file (config.ts)
+  const serverPath = process.env.GITHUB_ACTIONS ? fromHere("mcp-server.js") : fromHere("server.ts");
 
   return {
     [ghPullfrogMcpName]: {
       command: "node",
-      args: [
-        "--input-type=module",
-        "-e",
-        `import('${entryPath.replace(/'/g, "\\'")}').then(m => m.createMcpServer())`,
-      ],
+      args: [serverPath],
       env: {
         GITHUB_INSTALLATION_TOKEN: githubInstallationToken,
         GITHUB_REPOSITORY: githubRepository,

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) {

mcp/server.ts

@@ -8,20 +8,18 @@ import { PullRequestInfoTool } from "./prInfo.ts";
 import { ReviewTool } from "./review.ts";
 import { addTools } from "./shared.ts";
 
-export function createMcpServer(): void {
-  const server = new FastMCP({
-    name: "gh-pullfrog",
-    version: "0.0.1",
-  });
+const server = new FastMCP({
+  name: "gh-pullfrog",
+  version: "0.0.1",
+});
 
-  addTools(server, [
-    CreateCommentTool,
-    EditCommentTool,
-    IssueTool,
-    PullRequestTool,
-    ReviewTool,
-    PullRequestInfoTool,
-  ]);
+addTools(server, [
+  CreateCommentTool,
+  EditCommentTool,
+  IssueTool,
+  PullRequestTool,
+  ReviewTool,
+  PullRequestInfoTool,
+]);
 
-  server.start();
-}
+server.start();

package.json

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

todo.md

@@ -1,8 +1,10 @@
 [x] add modes to prompt 
 [x] progressively update comment
-[] don't allow rejecting prs
-[] fix pnpm caching
-[] try to avoid claude narrating the initial comment
+[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
 [] avoid exposing env
+[] test initialization trade offs for pullfrog.yml
+[] try to find heavy claude code user
+[] investigate repo config file?