update action

action.yml

@@ -32,9 +32,10 @@ runs:
       shell: bash
       working-directory: ${{ github.action_path }}
     - name: Run agent
-      run: node entry.ts
+      run: |
+        cd $GITHUB_WORKSPACE
+        node ${{ github.action_path }}/entry.ts
       shell: bash
-      working-directory: ${{ github.action_path }}
       env:
         INPUTS_JSON: ${{ toJSON(inputs) }}
 

agents/claude.ts

@@ -1,6 +1,7 @@
 import { access, constants } from "node:fs/promises";
 import * as core from "@actions/core";
 import { createMcpConfig } from "../mcp/config.ts";
+import { debugLog, isDebug } from "../utils/logging.ts";
 import { spawn } from "../utils/subprocess.ts";
 import { boxString, tableString } from "../utils/table.ts";
 import { instructions } from "./shared.ts";
@@ -77,24 +78,27 @@ export class ClaudeAgent implements Agent {
 
       const env = {
         ANTHROPIC_API_KEY: this.apiKey,
-        GITHUB_TOKEN: this.githubInstallationToken,
+        ...(isDebug() && { LOG_LEVEL: "debug" }),
       };
 
       console.log(boxString(prompt, { title: "Prompt" }));
 
       const mcpConfig = createMcpConfig(this.githubInstallationToken);
-      console.log("📋 MCP Config:", mcpConfig);
+
+      if (isDebug()) {
+        debugLog(`📋 MCP Config: ${mcpConfig}`);
+      }
 
       const args = [
         "--print",
         "--output-format",
         "stream-json",
         "--verbose",
-        "--debug",
         "--permission-mode",
         "bypassPermissions",
         "--mcp-config",
         mcpConfig,
+        ...(isDebug() ? ["--debug"] : []),
       ];
 
       core.startGroup("🔄 Run details");
@@ -115,7 +119,9 @@ export class ClaudeAgent implements Agent {
         input: `${instructions} ${prompt}`,
         timeout: 10 * 60 * 1000, // 10 minutes
         onStdout: (_chunk) => {
-          processJSONChunk(_chunk, this);
+          if (_chunk.trim()) {
+            processJSONChunk(_chunk, this);
+          }
         },
         onStderr: (_chunk) => {
           if (_chunk.trim()) {
@@ -166,14 +172,19 @@ export class ClaudeAgent implements Agent {
  */
 function processJSONChunk(chunk: string, agent?: ClaudeAgent): void {
   try {
-    // Skip debug lines that start with [DEBUG] or [debug]
     const trimmedChunk = chunk.trim();
     if (trimmedChunk.startsWith("[DEBUG]") || trimmedChunk.startsWith("[debug]")) {
       console.log(chunk);
       return;
     }
-    
-    console.log(chunk);
+
+    if (trimmedChunk.startsWith("[ERROR]") || trimmedChunk.startsWith("[error]")) {
+      console.error(chunk);
+      return;
+    }
+
+    debugLog(trimmedChunk);
+
     const parsedChunk = JSON.parse(trimmedChunk);
 
     switch (parsedChunk.type) {
@@ -310,11 +321,11 @@ function processJSONChunk(chunk: string, agent?: ClaudeAgent): void {
         break;
 
       default:
-        core.debug(`📦 Unknown chunk type: ${parsedChunk.type}`);
+        debugLog(`📦 Unknown chunk type: ${parsedChunk.type}`);
         break;
     }
   } catch (error) {
-    core.debug(`Failed to parse chunk: ${error}`);
-    core.debug(`Raw chunk: ${chunk.substring(0, 200)}...`);
+    debugLog(`Failed to parse chunk: ${error}`);
+    debugLog(`Raw chunk: ${chunk.substring(0, 200)}...`);
   }
 }

main.ts

@@ -1,7 +1,8 @@
 import * as core from "@actions/core";
 import { type } from "arktype";
 import { ClaudeAgent } from "./agents/claude.ts";
-import { setupGitHubInstallationToken } from "./utils/github.ts";
+import { setupGitHubInstallationToken, parseRepoContext } from "./utils/github.ts";
+import { setupGitConfig, setupGitAuth } from "./utils/setup.ts";
 
 export const Inputs = type({
   prompt: "string",
@@ -20,8 +21,12 @@ export async function main(inputs: Inputs): Promise<MainResult> {
   try {
     core.info(`→ Starting agent run with Claude Code`);
 
-    // Setup GitHub installation token
+    setupGitConfig();
+
     const githubInstallationToken = await setupGitHubInstallationToken();
+    const repoContext = parseRepoContext();
+    
+    setupGitAuth(githubInstallationToken, repoContext);
 
     const agent = new ClaudeAgent({
       apiKey: inputs.anthropic_api_key!,

mcp/config.ts

@@ -2,18 +2,15 @@
  * Simple MCP configuration helper for adding our minimal GitHub comment server
  */
 import { fromHere } from "@ark/fs";
+import { parseRepoContext } from "../utils/github.ts";
 
 const actionPath = fromHere("..");
 
 export const mcpServerName = "gh-pullfrog";
 
 export function createMcpConfig(githubInstallationToken: string) {
-  const githubRepository = process.env.GITHUB_REPOSITORY;
-  if (!githubRepository) {
-    throw new Error(
-      "GITHUB_REPOSITORY environment variable is required for MCP GitHub integration"
-    );
-  }
+  const repoContext = parseRepoContext();
+  const githubRepository = `${repoContext.owner}/${repoContext.name}`;
 
   return JSON.stringify(
     {
@@ -24,7 +21,7 @@ export function createMcpConfig(githubInstallationToken: string) {
           env: {
             GITHUB_INSTALLATION_TOKEN: githubInstallationToken,
             GITHUB_REPOSITORY: githubRepository,
-            LOG_LEVEL: "debug",
+            LOG_LEVEL: process.env.LOG_LEVEL,
           },
         },
       },

mcp/shared.ts

@@ -9,7 +9,6 @@ export interface ToolResult {
     type: "text";
     text: string;
   }[];
-  error?: string;
   isError?: boolean;
 }
 
@@ -72,7 +71,6 @@ const handleToolError = (error: unknown): ToolResult => {
         text: `Error: ${errorMessage}`,
       },
     ],
-    error: errorMessage,
     isError: true,
   };
 };

package.json

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

utils/act.ts

@@ -3,7 +3,7 @@ import { existsSync } from "node:fs";
 import { dirname, join } from "node:path";
 import { fileURLToPath } from "node:url";
 import { config } from "dotenv";
-import { buildAction, setupTestRepo } from "./setup.ts";
+import { setupTestRepo } from "./setup.ts";
 
 const __filename = fileURLToPath(import.meta.url);
 const __dirname = dirname(__filename);
@@ -19,8 +19,6 @@ export function runAct(prompt: string): void {
 
   config({ path: envPath });
 
-  buildAction(actionPath);
-
   const workflowPath = join(tempDir, ".github", "workflows", "pullfrog.yml");
 
   const distPath = join(actionPath, ".act-dist");
@@ -54,7 +52,6 @@ export function runAct(prompt: string): void {
       }
     });
 
-
     const actCommand = actCommandParts.join(" ");
 
     console.log("🚀 Running act with prompt:");

utils/logging.ts

@@ -0,0 +1,30 @@
+/**
+ * Centralized debug logging utility
+ * Controls debug behavior based on LOG_LEVEL environment variable
+ */
+
+import * as core from "@actions/core";
+
+const isDebugEnabled = process.env.LOG_LEVEL === "debug";
+const isGitHubActions = !!process.env.GITHUB_ACTIONS;
+
+/**
+ * Check if debug logging is enabled
+ */
+export function isDebug(): boolean {
+  return isDebugEnabled;
+}
+
+/**
+ * Log debug message if debug is enabled
+ * Uses core.debug() in GitHub Actions, console.log() locally
+ */
+export function debugLog(message: string): void {
+  if (isDebugEnabled) {
+    if (isGitHubActions) {
+      core.debug(message);
+    } else {
+      console.log(`[DEBUG] ${message}`);
+    }
+  }
+}

utils/setup.ts

@@ -1,5 +1,6 @@
 import { execSync } from "node:child_process";
 import { existsSync, rmSync } from "node:fs";
+import type { RepoContext } from "./github.ts";
 
 export interface SetupOptions {
   tempDir: string;
@@ -38,12 +39,30 @@ export function setupTestRepo(options: SetupOptions): void {
 }
 
 /**
- * Build the action bundles
+ * Setup git configuration to avoid identity errors
  */
-export function buildAction(actionPath: string): void {
-  console.log("🔨 Building fresh bundles with esbuild...");
-  execSync("node esbuild.config.js", {
-    cwd: actionPath,
-    stdio: "inherit",
-  });
+export function setupGitConfig(): void {
+  console.log("🔧 Setting up git configuration...");
+  execSync('git config user.email "action@pullfrog.ai"', { stdio: "inherit" });
+  execSync('git config user.name "Pullfrog Action"', { stdio: "inherit" });
+}
+
+/**
+ * Setup git authentication using GitHub token
+ */
+export function setupGitAuth(githubToken: string, repoContext: RepoContext): void {
+  console.log("🔐 Setting up git authentication...");
+
+  // Remove existing git auth headers that actions/checkout might have set
+  try {
+    execSync("git config --unset-all http.https://github.com/.extraheader", { stdio: "inherit" });
+    console.log("✓ Removed existing authentication headers");
+  } catch {
+    console.log("No existing authentication headers to remove");
+  }
+
+  // Update remote URL to embed the token
+  const remoteUrl = `https://x-access-token:${githubToken}@github.com/${repoContext.owner}/${repoContext.name}.git`;
+  execSync(`git remote set-url origin "${remoteUrl}"`, { stdio: "inherit" });
+  console.log("✓ Updated remote URL with authentication token");
 }

utils/subprocess.ts

@@ -6,6 +6,7 @@ export interface SpawnOptions {
   env?: Record<string, string>;
   input?: string;
   timeout?: number;
+  cwd?: string;
   onStdout?: (chunk: string) => void;
   onStderr?: (chunk: string) => void;
 }
@@ -21,7 +22,7 @@ export interface SpawnResult {
  * Spawn a subprocess with streaming callbacks and buffered results
  */
 export async function spawn(options: SpawnOptions): Promise<SpawnResult> {
-  const { cmd, args, env, input, timeout, onStdout, onStderr } = options;
+  const { cmd, args, env, input, timeout, cwd, onStdout, onStderr } = options;
 
   const startTime = Date.now();
   let stdoutBuffer = "";
@@ -31,6 +32,7 @@ export async function spawn(options: SpawnOptions): Promise<SpawnResult> {
     const child = nodeSpawn(cmd, args, {
       env: env ? { ...process.env, ...env } : process.env,
       stdio: ["pipe", "pipe", "pipe"],
+      cwd: cwd || process.cwd(),
     });
 
     let timeoutId: NodeJS.Timeout | undefined;