try esm action

.husky/pre-commit

@@ -1,6 +1,3 @@
-#!/usr/bin/env sh
-. "$(dirname -- "$0")/_/husky.sh"
-
 # Ensure lockfile is up to date
 echo "🔒 Updating lockfile..."
 pnpm install --lockfile-only
@@ -10,4 +7,4 @@ echo "🔨 Building action..."
 pnpm build
 
 # Add the built files and lockfile to the commit
-git add entry.cjs pnpm-lock.yaml
+git add entry.js pnpm-lock.yaml

action.yml

@@ -13,7 +13,7 @@ inputs:
 
 runs:
   using: "node20"
-  main: "entry.cjs"
+  main: "entry.js"
 
 branding:
   icon: "code"

entry.build.ts

@@ -1,39 +0,0 @@
-#!/usr/bin/env node
-
-/**
- * Build entry point - wraps entry.ts to avoid top-level await in CJS output
- */
-
-import * as core from "@actions/core";
-import { type } from "arktype";
-import { Inputs, main } from "./main.ts";
-import packageJson from "./package.json" with { type: "json" };
-import { log } from "./utils/cli.ts";
-
-async function run(): Promise<void> {
-  try {
-    log.info(`🐸 Running pullfrog/action@${packageJson.version}...`);
-
-    const inputsJson = process.env.INPUTS_JSON;
-    if (!inputsJson) {
-      throw new Error("INPUTS_JSON environment variable not found");
-    }
-
-    const parsed = type("string.json.parse").assert(inputsJson);
-    const inputs = Inputs.assert(parsed);
-
-    const result = await main(inputs);
-
-    if (!result.success) {
-      throw new Error(result.error || "Agent execution failed");
-    }
-  } catch (error) {
-    const errorMessage = error instanceof Error ? error.message : "Unknown error occurred";
-    core.setFailed(`Action failed: ${errorMessage}`);
-  }
-}
-
-// Wrap in IIFE to avoid top-level await in CJS
-(async () => {
-  await run();
-})();

entry.ts

@@ -7,9 +7,13 @@
 import * as core from "@actions/core";
 import { type } from "arktype";
 import { Inputs, main } from "./main.ts";
+import { createMcpServer } from "./mcp/server.ts";
 import packageJson from "./package.json" with { type: "json" };
 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> {
   try {
     log.info(`🐸 Running pullfrog/action@${packageJson.version}...`);

esbuild.config.js

@@ -3,17 +3,23 @@ import { build } from "esbuild";
 // Build the GitHub Action bundle only
 // For npm package builds, use zshy (pnpm build:npm)
 await build({
-  entryPoints: ["./entry.build.ts"],
+  entryPoints: ["./entry.ts"],
   bundle: true,
-  outfile: "./entry.cjs",
-  format: "cjs",
+  outfile: "./entry.js",
+  format: "esm",
   platform: "node",
   target: "node20",
   minify: true,
   sourcemap: false,
-  // @actions/core is provided by GitHub Actions runtime, but we still need it bundled
-  // for local testing. However, we can mark it to reduce duplication if needed.
-  external: [],
+  // Mark all node_modules as external - Node.js will handle ESM/CJS interop natively
+  // This avoids esbuild's require() polyfill which doesn't work in ESM
+  packages: "external",
+  // Mark optional peer dependencies as external
+  external: [
+    "@valibot/to-json-schema",
+    "effect",
+    "sury",
+  ],
   // Enable tree-shaking to remove unused code
   treeShaking: true,
   // Drop console statements in production (but keep for debugging)

mcp/config.ts

@@ -3,11 +3,8 @@
  */
 
 import type { McpServerConfig } from "@anthropic-ai/claude-agent-sdk";
-import { fromHere } from "@ark/fs";
 import { parseRepoContext } from "../utils/github.ts";
 
-const actionPath = fromHere("..");
-
 export const ghPullfrogMcpName = "gh-pullfrog";
 
 export type McpName = typeof ghPullfrogMcpName;
@@ -18,10 +15,19 @@ 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`;
+
   return {
     [ghPullfrogMcpName]: {
       command: "node",
-      args: [`${actionPath}/mcp/server.ts`],
+      args: [
+        "--input-type=module",
+        "-e",
+        `import('${entryPath.replace(/'/g, "\\'")}').then(m => m.createMcpServer())`,
+      ],
       env: {
         GITHUB_INSTALLATION_TOKEN: githubInstallationToken,
         GITHUB_REPOSITORY: githubRepository,

mcp/server.ts

@@ -8,18 +8,20 @@ import { PullRequestInfoTool } from "./prInfo.ts";
 import { ReviewTool } from "./review.ts";
 import { addTools } from "./shared.ts";
 
-const server = new FastMCP({
-  name: "gh-pullfrog",
-  version: "0.0.1",
-});
+export function createMcpServer(): void {
+  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.74",
+  "version": "0.0.76",
   "type": "module",
   "files": [
     "index.js",

utils/setup.ts

@@ -50,8 +50,16 @@ export function setupGitConfig(): void {
 
 /**
  * Setup git authentication using GitHub token
+ * Only runs in GitHub Actions environment to avoid breaking local git remotes
  */
 export function setupGitAuth(githubToken: string, repoContext: RepoContext): void {
+  // Only set up git auth in GitHub Actions environment
+  // In local testing, this would overwrite the real git remote with fake credentials
+  if (!process.env.GITHUB_ACTIONS) {
+    log.info("⚠️  Skipping git authentication setup (not in GitHub Actions)");
+    return;
+  }
+
   log.info("🔐 Setting up git authentication...");
 
   // Remove existing git auth headers that actions/checkout might have set