Revert .gitignore changes

Remove test-markdown.json and update .gitignore
Remove rendered.html from repository
2026-01-23 23:14:13 +08:00 · 2025-05-28 18:16:13 -07:00 · 2025-05-28 18:14:13 -07:00 · 2025-05-28 18:13:49 -07:00 · 2025-05-28 18:13:39 -07:00 · 2025-05-28 18:12:07 -07:00
25 changed files with 244 additions and 1744 deletions
--- a/.github/workflows/claude.yml
+++ b/.github/workflows/claude.yml
@@ -34,6 +34,3 @@ jobs:
        uses: anthropics/claude-code-action@beta
        with:
          anthropic_api_key: ${{ secrets.ANTHROPIC_API_KEY }}
-          allowed_tools: "Bash(bun install),Bash(bun test:*),Bash(bun run format),Bash(bun typecheck)"
-          custom_instructions: "You have also been granted tools for editing files and running bun commands (install, run, test, typecheck) for testing your changes: bun install, bun test, bun run format, bun typecheck."
-          model: "claude-opus-4-20250514"
--- a/.github/workflows/issue-triage.yml
+++ b/.github/workflows/issue-triage.yml
@@ -23,20 +23,18 @@ jobs:
          mkdir -p /tmp/mcp-config
          cat > /tmp/mcp-config/mcp-servers.json << 'EOF'
          {
-            "mcpServers": {
-              "github": {
-                "command": "docker",
-                "args": [
-                  "run",
-                  "-i",
-                  "--rm",
-                  "-e",
-                  "GITHUB_PERSONAL_ACCESS_TOKEN",
-                  "ghcr.io/github/github-mcp-server:sha-7aced2b"
-                ],
-                "env": {
-                  "GITHUB_PERSONAL_ACCESS_TOKEN": "${{ secrets.GITHUB_TOKEN }}"
-                }
+            "github": {
+              "command": "docker",
+              "args": [
+                "run",
+                "-i",
+                "--rm",
+                "-e",
+                "GITHUB_PERSONAL_ACCESS_TOKEN",
+                "ghcr.io/github/github-mcp-server:sha-7aced2b"
+              ],
+              "env": {
+                "GITHUB_PERSONAL_ACCESS_TOKEN": "${{ secrets.GITHUB_TOKEN }}"
              }
            }
          }
@@ -101,6 +99,6 @@ jobs:
        with:
          prompt_file: /tmp/claude-prompts/triage-prompt.txt
          allowed_tools: "Bash(gh label list),mcp__github__get_issue,mcp__github__get_issue_comments,mcp__github__update_issue,mcp__github__search_issues,mcp__github__list_issues"
-          mcp_config: /tmp/mcp-config/mcp-servers.json
+          mcp_config_file: /tmp/mcp-config/mcp-servers.json
          timeout_minutes: "5"
          anthropic_api_key: ${{ secrets.ANTHROPIC_API_KEY }}
--- a/FAQ.md
+++ b/FAQ.md
@@ -1,156 +0,0 @@
-# Frequently Asked Questions (FAQ)
-
-This FAQ addresses common questions and gotchas when using the Claude Code GitHub Action.
-
-## Triggering and Authentication
-
-### Why doesn't tagging @claude from my automated workflow work?
-
-The `github-actions` user cannot trigger subsequent GitHub Actions workflows. This is a GitHub security feature to prevent infinite loops. To make this work, you need to use a Personal Access Token (PAT) instead, which will act as a regular user, or use a separate app token of your own. When posting a comment on an issue or PR from your workflow, use your PAT instead of the `GITHUB_TOKEN` generated in your workflow.
-
-### Why does Claude say I don't have permission to trigger it?
-
-Only users with **write permissions** to the repository can trigger Claude. This is a security feature to prevent unauthorized use. Make sure the user commenting has at least write access to the repository.
-
-### Why am I getting OIDC authentication errors?
-
-If you're using the default GitHub App authentication, you must add the `id-token: write` permission to your workflow:
-
-```yaml
-permissions:
-  contents: read
-  id-token: write # Required for OIDC authentication
-```
-
-The OIDC token is required in order for the Claude GitHub app to function. If you wish to not use the GitHub app, you can instead provide a `github_token` input to the action for Claude to operate with. See the [Claude Code permissions documentation][perms] for more.
-
-## Claude's Capabilities and Limitations
-
-### Why won't Claude update workflow files when I ask it to?
-
-The GitHub App for Claude doesn't have workflow write access for security reasons. This prevents Claude from modifying CI/CD configurations that could potentially create unintended consequences. This is something we may reconsider in the future.
-
-### Why won't Claude rebase my branch?
-
-By default, Claude only uses commit tools for non-destructive changes to the branch. Claude is configured to:
-
- Never push to branches other than where it was invoked (either its own branch or the PR branch)
- Never force push or perform destructive operations
-
-You can grant additional tools via the `allowed_tools` input if needed:
-
-```yaml
-allowed_tools: "Bash(git rebase:*)" # Use with caution
-```
-
-### Why won't Claude create a pull request?
-
-Claude doesn't create PRs by default. Instead, it pushes commits to a branch and provides a link to a pre-filled PR submission page. This approach ensures your repository's branch protection rules are still adhered to and gives you final control over PR creation.
-
-### Why can't Claude run my tests or see CI results?
-
-Claude cannot access GitHub Actions logs, test results, or other CI/CD outputs by default. It only has access to the repository files. If you need Claude to see test results, you can either:
-
-1. Instruct Claude to run tests before making commits
-2. Copy and paste CI results into a comment for Claude to analyze
-
-This limitation exists for security reasons but may be reconsidered in the future based on user feedback.
-
-### Why does Claude only update one comment instead of creating new ones?
-
-Claude is configured to update a single comment to avoid cluttering PR/issue discussions. All of Claude's responses, including progress updates and final results, will appear in the same comment with checkboxes showing task progress.
-
-## Branch and Commit Behavior
-
-### Why did Claude create a new branch when commenting on a closed PR?
-
-Claude's branch behavior depends on the context:
-
- **Open PRs**: Pushes directly to the existing PR branch
- **Closed/Merged PRs**: Creates a new branch (cannot push to closed PR branches)
- **Issues**: Always creates a new branch with a timestamp
-
-### Why are my commits shallow/missing history?
-
-For performance, Claude uses shallow clones:
-
- PRs: `--depth=20` (last 20 commits)
- New branches: `--depth=1` (single commit)
-
-If you need full history, you can configure this in your workflow before calling Claude in the `actions/checkout` step.
-
-```
- uses: actions/checkout@v4
-  depth: 0 # will fetch full repo history
-```
-
-## Configuration and Tools
-
-### What's the difference between `direct_prompt` and `custom_instructions`?
-
-These inputs serve different purposes in how Claude responds:
-
- **`direct_prompt`**: Bypasses trigger detection entirely. When provided, Claude executes this exact instruction regardless of comments or mentions. Perfect for automated workflows where you want Claude to perform a specific task on every run (e.g., "Update the API documentation based on changes in this PR").
-
- **`custom_instructions`**: Additional context added to Claude's system prompt while still respecting normal triggers. These instructions modify Claude's behavior but don't replace the triggering comment. Use this to give Claude standing instructions like "You have been granted additional tools for ...".
-
-Example:
-
-```yaml
-# Using direct_prompt - runs automatically without @claude mention
-direct_prompt: "Review this PR for security vulnerabilities"
-
-# Using custom_instructions - still requires @claude trigger
-custom_instructions: "Focus on performance implications and suggest optimizations"
-```
-
-### Why doesn't Claude execute my bash commands?
-
-The Bash tool is **disabled by default** for security. To enable individual bash commands:
-
-```yaml
-allowed_tools: "Bash(npm:*),Bash(git:*)" # Allows only npm and git commands
-```
-
-### Can Claude work across multiple repositories?
-
-No, Claude's GitHub app token is sandboxed to the current repository only. It cannot push to any other repositories. It can, however, read public repositories, but to get access to this, you must configure it with tools to do so.
-
-## MCP Servers and Extended Functionality
-
-### What MCP servers are available by default?
-
-Claude Code Action automatically configures two MCP servers:
-
-1. **GitHub MCP server**: For GitHub API operations
-2. **File operations server**: For advanced file manipulation
-
-However, tools from these servers still need to be explicitly allowed via `allowed_tools`.
-
-## Troubleshooting
-
-### How can I debug what Claude is doing?
-
-Check the GitHub Action log for Claude's run for the full execution trace.
-
-### Why can't I trigger Claude with `@claude-mention` or `claude!`?
-
-The trigger uses word boundaries, so `@claude` must be a complete word. Variations like `@claude-bot`, `@claude!`, or `claude@mention` won't work unless you customize the `trigger_phrase`.
-
-## Best Practices
-
-1. **Always specify permissions explicitly** in your workflow file
-2. **Use GitHub Secrets** for API keys - never hardcode them
-3. **Be specific with `allowed_tools`** - only enable what's necessary
-4. **Test in a separate branch** before using on important PRs
-5. **Monitor Claude's token usage** to avoid hitting API limits
-6. **Review Claude's changes** carefully before merging
-
-## Getting Help
-
-If you encounter issues not covered here:
-
-1. Check the [GitHub Issues](https://github.com/anthropics/claude-code-action/issues)
-2. Review the [example workflows](https://github.com/anthropics/claude-code-action#examples)
-
-[perms]: https://docs.anthropic.com/en/docs/claude-code/settings#permissions
--- a/README.md
+++ b/README.md
@@ -33,10 +33,6 @@ This command will guide you through setting up the GitHub app and required secre
 2. Add `ANTHROPIC_API_KEY` to your repository secrets ([Learn how to use secrets in GitHub Actions](https://docs.github.com/en/actions/security-for-github-actions/security-guides/using-secrets-in-github-actions))
 3. Copy the workflow file from [`examples/claude.yml`](./examples/claude.yml) into your repository's `.github/workflows/`

-## 📚 FAQ
-
-Having issues or questions? Check out our [Frequently Asked Questions](./FAQ.md) for solutions to common problems and detailed explanations of Claude's capabilities and limitations.
-
 ## Usage

 Add a workflow file to your repository (e.g., `.github/workflows/claude.yml`):
@@ -65,11 +61,6 @@ jobs:
          # trigger_phrase: "/claude"
          # Optional: add assignee trigger for issues
          # assignee_trigger: "claude"
-          # Optional: add custom environment variables (YAML format)
-          # claude_env: |
-          #   NODE_ENV: test
-          #   DEBUG: true
-          #   API_URL: https://api.example.com
 ```

 ## Inputs
@@ -87,70 +78,13 @@ jobs:
 | `allowed_tools`       | Additional tools for Claude to use (the base GitHub tools will always be included)                                   | No       | ""        |
 | `disallowed_tools`    | Tools that Claude should never use                                                                                   | No       | ""        |
 | `custom_instructions` | Additional custom instructions to include in the prompt for Claude                                                   | No       | ""        |
-| `mcp_config`          | Additional MCP configuration (JSON string) that merges with the built-in GitHub MCP servers                          | No       | ""        |
 | `assignee_trigger`    | The assignee username that triggers the action (e.g. @claude). Only used for issue assignment                        | No       | -         |
 | `trigger_phrase`      | The trigger phrase to look for in comments, issue/PR bodies, and issue titles                                        | No       | `@claude` |
-| `claude_env`          | Custom environment variables to pass to Claude Code execution (YAML format)                                          | No       | ""        |

 \*Required when using direct Anthropic API (default and when not using Bedrock or Vertex)

 > **Note**: This action is currently in beta. Features and APIs may change as we continue to improve the integration.

-### Using Custom MCP Configuration
-
-The `mcp_config` input allows you to add custom MCP (Model Context Protocol) servers to extend Claude's capabilities. These servers merge with the built-in GitHub MCP servers.
-
-#### Basic Example: Adding a Sequential Thinking Server
-
-```yaml
- uses: anthropics/claude-code-action@beta
-  with:
-    anthropic_api_key: ${{ secrets.ANTHROPIC_API_KEY }}
-    mcp_config: |
-      {
-        "mcpServers": {
-          "sequential-thinking": {
-            "command": "npx",
-            "args": [
-              "-y",
-              "@modelcontextprotocol/server-sequential-thinking"
-            ]
-          }
-        }
-      }
-    allowed_tools: "mcp__sequential-thinking__sequentialthinking" # Important: Each MCP tool from your server must be listed here, comma-separated
-    # ... other inputs
-```
-
-#### Passing Secrets to MCP Servers
-
-For MCP servers that require sensitive information like API keys or tokens, use GitHub Secrets in the environment variables:
-
-```yaml
- uses: anthropics/claude-code-action@beta
-  with:
-    anthropic_api_key: ${{ secrets.ANTHROPIC_API_KEY }}
-    mcp_config: |
-      {
-        "mcpServers": {
-          "custom-api-server": {
-            "command": "npx",
-            "args": ["-y", "@example/api-server"],
-            "env": {
-              "API_KEY": "${{ secrets.CUSTOM_API_KEY }}",
-              "BASE_URL": "https://api.example.com"
-            }
-          }
-        }
-      }
-    # ... other inputs
-```
-
-**Important**:
-
- Always use GitHub Secrets (`${{ secrets.SECRET_NAME }}`) for sensitive values like API keys, tokens, or passwords. Never hardcode secrets directly in the workflow file.
- Your custom servers will override any built-in servers with the same name.
-
 ## Examples

 ### Ways to Tag @claude
@@ -295,22 +229,6 @@ This action is built on top of [`anthropics/claude-code-base-action`](https://gi

 ## Advanced Configuration

-### Custom Environment Variables
-
-You can pass custom environment variables to Claude Code execution using the `claude_env` input. This is useful for CI/test setups that require specific environment variables:
-
-```yaml
- uses: anthropics/claude-code-action@beta
-  with:
-    claude_env: |
-      NODE_ENV: test
-      CI: true
-      DATABASE_URL: postgres://test:test@localhost:5432/test_db
-    # ... other inputs
-```
-
-The `claude_env` input accepts YAML format where each line defines a key-value pair. These environment variables will be available to Claude Code during execution, allowing it to run tests, build processes, or other commands that depend on specific environment configurations.
-
 ### Custom Tools

 By default, Claude only has access to:
--- a/action.yml
+++ b/action.yml
@@ -12,9 +12,6 @@ inputs:
  assignee_trigger:
    description: "The assignee username that triggers the action (e.g. @claude)"
    required: false
-  base_branch:
-    description: "The branch to use as the base/source when creating new branches (defaults to repository default branch)"
-    required: false

  # Claude Code configuration
  model:
@@ -39,12 +36,6 @@ inputs:
    description: "Direct instruction for Claude (bypasses normal trigger detection)"
    required: false
    default: ""
-  mcp_config:
-    description: "Additional MCP configuration (JSON string) that merges with the built-in GitHub MCP servers"
-  claude_env:
-    description: "Custom environment variables to pass to Claude Code execution (YAML format)"
-    required: false
-    default: ""

  # Auth configuration
  anthropic_api_key:
@@ -94,18 +85,16 @@ runs:
      env:
        TRIGGER_PHRASE: ${{ inputs.trigger_phrase }}
        ASSIGNEE_TRIGGER: ${{ inputs.assignee_trigger }}
-        BASE_BRANCH: ${{ inputs.base_branch }}
        ALLOWED_TOOLS: ${{ inputs.allowed_tools }}
        CUSTOM_INSTRUCTIONS: ${{ inputs.custom_instructions }}
        DIRECT_PROMPT: ${{ inputs.direct_prompt }}
-        MCP_CONFIG: ${{ inputs.mcp_config }}
        OVERRIDE_GITHUB_TOKEN: ${{ inputs.github_token }}
        GITHUB_RUN_ID: ${{ github.run_id }}

    - name: Run Claude Code
      id: claude-code
      if: steps.prepare.outputs.contains_trigger == 'true'
-      uses: anthropics/claude-code-base-action@1370ac97fbba9bddec20ea2924b5726bf10d8b94 # v0.0.9
+      uses: anthropics/claude-code-base-action@266585c92dd90d61d3806a3367582c4f6224e892 # https://github.com/anthropics/claude-code-base-action/releases/tag/v0.0.6
      with:
        prompt_file: /tmp/claude-prompts/claude-prompt.txt
        allowed_tools: ${{ env.ALLOWED_TOOLS }}
@@ -116,15 +105,11 @@ runs:
        use_bedrock: ${{ inputs.use_bedrock }}
        use_vertex: ${{ inputs.use_vertex }}
        anthropic_api_key: ${{ inputs.anthropic_api_key }}
-        claude_env: ${{ inputs.claude_env }}
      env:
        # Model configuration
        ANTHROPIC_MODEL: ${{ inputs.model || inputs.anthropic_model }}
        GITHUB_TOKEN: ${{ steps.prepare.outputs.GITHUB_TOKEN }}

-        # Provider configuration
-        ANTHROPIC_BASE_URL: ${{ env.ANTHROPIC_BASE_URL }}
-
        # AWS configuration
        AWS_REGION: ${{ env.AWS_REGION }}
        AWS_ACCESS_KEY_ID: ${{ env.AWS_ACCESS_KEY_ID }}
@@ -158,13 +143,10 @@ runs:
        TRIGGER_COMMENT_ID: ${{ github.event.comment.id }}
        CLAUDE_BRANCH: ${{ steps.prepare.outputs.CLAUDE_BRANCH }}
        IS_PR: ${{ github.event.issue.pull_request != null || github.event_name == 'pull_request_review_comment' }}
-        BASE_BRANCH: ${{ steps.prepare.outputs.BASE_BRANCH }}
+        DEFAULT_BRANCH: ${{ steps.prepare.outputs.DEFAULT_BRANCH }}
        CLAUDE_SUCCESS: ${{ steps.claude-code.outputs.conclusion == 'success' }}
-        CLAUDE_CANCELLED: ${{ cancelled() }}
        OUTPUT_FILE: ${{ steps.claude-code.outputs.execution_file || '' }}
        TRIGGER_USERNAME: ${{ github.event.comment.user.login || github.event.issue.user.login || github.event.pull_request.user.login || github.event.sender.login || github.triggering_actor || github.actor || '' }}
-        PREPARE_SUCCESS: ${{ steps.prepare.outcome == 'success' }}
-        PREPARE_ERROR: ${{ steps.prepare.outputs.prepare_error || '' }}

    - name: Display Claude Code Report
      if: steps.prepare.outputs.contains_trigger == 'true' && steps.claude-code.outputs.execution_file != ''
--- a/src/create-prompt/index.ts
+++ b/src/create-prompt/index.ts
@@ -31,13 +31,24 @@ const BASE_ALLOWED_TOOLS = [
  "Write",
  "mcp__github_file_ops__commit_files",
  "mcp__github_file_ops__delete_files",
-  "mcp__github_file_ops__update_claude_comment",
 ];
 const DISALLOWED_TOOLS = ["WebSearch", "WebFetch"];

-export function buildAllowedToolsString(customAllowedTools?: string): string {
+export function buildAllowedToolsString(
+  eventData: EventData,
+  customAllowedTools?: string,
+): string {
  let baseTools = [...BASE_ALLOWED_TOOLS];

+  // Add the appropriate comment tool based on event type
+  if (eventData.eventName === "pull_request_review_comment") {
+    // For inline PR review comments, only use PR comment tool
+    baseTools.push("mcp__github__update_pull_request_comment");
+  } else {
+    // For all other events (issue comments, PR reviews, issues), use issue comment tool
+    baseTools.push("mcp__github__update_issue_comment");
+  }
+
  let allAllowedTools = baseTools.join(",");
  if (customAllowedTools) {
    allAllowedTools = `${allAllowedTools},${customAllowedTools}`;
@@ -47,27 +58,10 @@ export function buildAllowedToolsString(customAllowedTools?: string): string {

 export function buildDisallowedToolsString(
  customDisallowedTools?: string,
-  allowedTools?: string,
 ): string {
-  let disallowedTools = [...DISALLOWED_TOOLS];
-
-  // If user has explicitly allowed some hardcoded disallowed tools, remove them from disallowed list
-  if (allowedTools) {
-    const allowedToolsArray = allowedTools
-      .split(",")
-      .map((tool) => tool.trim());
-    disallowedTools = disallowedTools.filter(
-      (tool) => !allowedToolsArray.includes(tool),
-    );
-  }
-
-  let allDisallowedTools = disallowedTools.join(",");
+  let allDisallowedTools = DISALLOWED_TOOLS.join(",");
  if (customDisallowedTools) {
-    if (allDisallowedTools) {
-      allDisallowedTools = `${allDisallowedTools},${customDisallowedTools}`;
-    } else {
-      allDisallowedTools = customDisallowedTools;
-    }
+    allDisallowedTools = `${allDisallowedTools},${customDisallowedTools}`;
  }
  return allDisallowedTools;
 }
@@ -75,7 +69,7 @@ export function buildDisallowedToolsString(
 export function prepareContext(
  context: ParsedGitHubContext,
  claudeCommentId: string,
-  baseBranch?: string,
+  defaultBranch?: string,
  claudeBranch?: string,
 ): PreparedContext {
  const repository = context.repository.full_name;
@@ -153,7 +147,7 @@ export function prepareContext(
        ...(commentId && { commentId }),
        commentBody,
        ...(claudeBranch && { claudeBranch }),
-        ...(baseBranch && { baseBranch }),
+        ...(defaultBranch && { defaultBranch }),
      };
      break;

@@ -175,7 +169,7 @@ export function prepareContext(
        prNumber,
        commentBody,
        ...(claudeBranch && { claudeBranch }),
-        ...(baseBranch && { baseBranch }),
+        ...(defaultBranch && { defaultBranch }),
      };
      break;

@@ -200,13 +194,13 @@ export function prepareContext(
          prNumber,
          commentBody,
          ...(claudeBranch && { claudeBranch }),
-          ...(baseBranch && { baseBranch }),
+          ...(defaultBranch && { defaultBranch }),
        };
        break;
      } else if (!claudeBranch) {
        throw new Error("CLAUDE_BRANCH is required for issue_comment event");
-      } else if (!baseBranch) {
-        throw new Error("BASE_BRANCH is required for issue_comment event");
+      } else if (!defaultBranch) {
+        throw new Error("DEFAULT_BRANCH is required for issue_comment event");
      } else if (!issueNumber) {
        throw new Error(
          "ISSUE_NUMBER is required for issue_comment event for issues",
@@ -218,7 +212,7 @@ export function prepareContext(
        commentId,
        isPR: false,
        claudeBranch: claudeBranch,
-        baseBranch,
+        defaultBranch,
        issueNumber,
        commentBody,
      };
@@ -234,8 +228,8 @@ export function prepareContext(
      if (isPR) {
        throw new Error("IS_PR must be false for issues event");
      }
-      if (!baseBranch) {
-        throw new Error("BASE_BRANCH is required for issues event");
+      if (!defaultBranch) {
+        throw new Error("DEFAULT_BRANCH is required for issues event");
      }
      if (!claudeBranch) {
        throw new Error("CLAUDE_BRANCH is required for issues event");
@@ -252,7 +246,7 @@ export function prepareContext(
          eventAction: "assigned",
          isPR: false,
          issueNumber,
-          baseBranch,
+          defaultBranch,
          claudeBranch,
          assigneeTrigger,
        };
@@ -262,7 +256,7 @@ export function prepareContext(
          eventAction: "opened",
          isPR: false,
          issueNumber,
-          baseBranch,
+          defaultBranch,
          claudeBranch,
        };
      } else {
@@ -283,7 +277,7 @@ export function prepareContext(
        isPR: true,
        prNumber,
        ...(claudeBranch && { claudeBranch }),
-        ...(baseBranch && { baseBranch }),
+        ...(defaultBranch && { defaultBranch }),
      };
      break;

@@ -436,15 +430,33 @@ ${sanitizeContent(context.directPrompt)}
 </direct_prompt>`
    : ""
 }
-${`<comment_tool_info>
-IMPORTANT: You have been provided with the mcp__github_file_ops__update_claude_comment tool to update your comment. This tool automatically handles both issue and PR comments.
+${
+  eventData.eventName === "pull_request_review_comment"
+    ? `<comment_tool_info>
+IMPORTANT: For this inline PR review comment, you have been provided with ONLY the mcp__github__update_pull_request_comment tool to update this specific review comment.

-Tool usage example for mcp__github_file_ops__update_claude_comment:
+Tool usage example for mcp__github__update_pull_request_comment:
 {
+  "owner": "${context.repository.split("/")[0]}",
+  "repo": "${context.repository.split("/")[1]}",
+  "commentId": ${eventData.commentId || context.claudeCommentId},
  "body": "Your comment text here"
 }
-Only the body parameter is required - the tool automatically knows which comment to update.
-</comment_tool_info>`}
+All four parameters (owner, repo, commentId, body) are required.
+</comment_tool_info>`
+    : `<comment_tool_info>
+IMPORTANT: For this event type, you have been provided with ONLY the mcp__github__update_issue_comment tool to update comments.
+
+Tool usage example for mcp__github__update_issue_comment:
+{
+  "owner": "${context.repository.split("/")[0]}",
+  "repo": "${context.repository.split("/")[1]}",
+  "commentId": ${context.claudeCommentId},
+  "body": "Your comment text here"
+}
+All four parameters (owner, repo, commentId, body) are required.
+</comment_tool_info>`
+}

 Your task is to analyze the context, understand the request, and provide helpful responses and/or implement code changes as needed.

@@ -458,7 +470,7 @@ Follow these steps:
 1. Create a Todo List:
   - Use your GitHub comment to maintain a detailed task list based on the request.
   - Format todos as a checklist (- [ ] for incomplete, - [x] for complete).
-   - Update the comment using mcp__github_file_ops__update_claude_comment with each task completion.
+   - Update the comment using ${eventData.eventName === "pull_request_review_comment" ? "mcp__github__update_pull_request_comment" : "mcp__github__update_issue_comment"} with each task completion.

 2. Gather Context:
   - Analyze the pre-fetched data provided above.
@@ -488,11 +500,11 @@ ${context.directPrompt ? `   - DIRECT INSTRUCTION: A direct instruction was prov
        - Look for bugs, security issues, performance problems, and other issues
        - Suggest improvements for readability and maintainability
        - Check for best practices and coding standards
-        - Reference specific code sections with file paths and line numbers${eventData.isPR ? "\n      - AFTER reading files and analyzing code, you MUST call mcp__github_file_ops__update_claude_comment to post your review" : ""}
+        - Reference specific code sections with file paths and line numbers${eventData.isPR ? "\n      - AFTER reading files and analyzing code, you MUST call mcp__github__update_issue_comment to post your review" : ""}
      - Formulate a concise, technical, and helpful response based on the context.
      - Reference specific code with inline formatting or code blocks.
      - Include relevant file paths and line numbers when applicable.
-      - ${eventData.isPR ? "IMPORTANT: Submit your review feedback by updating the Claude comment using mcp__github_file_ops__update_claude_comment. This will be displayed as your PR review." : "Remember that this feedback must be posted to the GitHub comment using mcp__github_file_ops__update_claude_comment."}
+      - ${eventData.isPR ? "IMPORTANT: Submit your review feedback by updating the Claude comment. This will be displayed as your PR review." : "Remember that this feedback must be posted to the GitHub comment."}

   B. For Straightforward Changes:
      - Use file system tools to make the change locally.
@@ -512,13 +524,13 @@ ${context.directPrompt ? `   - DIRECT INSTRUCTION: A direct instruction was prov
      ${
        eventData.claudeBranch
          ? `- Provide a URL to create a PR manually in this format:
-        [Create a PR](${GITHUB_SERVER_URL}/${context.repository}/compare/${eventData.baseBranch}...<branch-name>?quick_pull=1&title=<url-encoded-title>&body=<url-encoded-body>)
+        [Create a PR](${GITHUB_SERVER_URL}/${context.repository}/compare/${eventData.defaultBranch}...<branch-name>?quick_pull=1&title=<url-encoded-title>&body=<url-encoded-body>)
        - IMPORTANT: Use THREE dots (...) between branch names, not two (..)
          Example: ${GITHUB_SERVER_URL}/${context.repository}/compare/main...feature-branch (correct)
          NOT: ${GITHUB_SERVER_URL}/${context.repository}/compare/main..feature-branch (incorrect)
        - IMPORTANT: Ensure all URL parameters are properly encoded - spaces should be encoded as %20, not left as spaces
          Example: Instead of "fix: update welcome message", use "fix%3A%20update%20welcome%20message"
-        - The target-branch should be '${eventData.baseBranch}'.
+        - The target-branch should be '${eventData.defaultBranch}'.
        - The branch-name is the current branch: ${eventData.claudeBranch}
        - The body should include:
          - A clear description of the changes
@@ -547,8 +559,8 @@ ${context.directPrompt ? `   - DIRECT INSTRUCTION: A direct instruction was prov

 Important Notes:
 - All communication must happen through GitHub PR comments.
- Never create new comments. Only update the existing comment using mcp__github_file_ops__update_claude_comment.
- This includes ALL responses: code reviews, answers to questions, progress updates, and final results.${eventData.isPR ? "\n- PR CRITICAL: After reading files and forming your response, you MUST post it by calling mcp__github_file_ops__update_claude_comment. Do NOT just respond with a normal response, the user will not see it." : ""}
+- Never create new comments. Only update the existing comment using ${eventData.eventName === "pull_request_review_comment" ? "mcp__github__update_pull_request_comment" : "mcp__github__update_issue_comment"} with comment_id: ${context.claudeCommentId}.
+- This includes ALL responses: code reviews, answers to questions, progress updates, and final results.${eventData.isPR ? "\n- PR CRITICAL: After reading files and forming your response, you MUST post it by calling mcp__github__update_issue_comment. Do NOT just respond with a normal response, the user will not see it." : ""}
 - You communicate exclusively by editing your single comment - not through any other means.
 - Use this spinner HTML when work is in progress: <img src="https://github.com/user-attachments/assets/5ac382c7-e004-429b-8e35-7feb3e8f9c6f" width="14px" height="14px" style="vertical-align: middle; margin-left: 4px;" />
 ${eventData.isPR && !eventData.claudeBranch ? `- Always push to the existing branch when triggered on a PR.` : `- IMPORTANT: You are already on the correct branch (${eventData.claudeBranch || "the created branch"}). Never create new branches when triggered on issues or closed/merged PRs.`}
@@ -582,11 +594,6 @@ What You CANNOT Do:
 - Execute commands outside the repository context
 - Run arbitrary Bash commands (unless explicitly allowed via allowed_tools configuration)
 - Perform branch operations (cannot merge branches, rebase, or perform other git operations beyond pushing commits)
- Modify files in the .github/workflows directory (GitHub App permissions do not allow workflow modifications)
- View CI/CD results or workflow run outputs (cannot access GitHub Actions logs or test results)
-
-When users ask you to perform actions you cannot do, politely explain the limitation and, when applicable, direct them to the FAQ for more information and workarounds:
-"I'm unable to [specific action] due to [reason]. You can find more information and potential workarounds in the [FAQ](https://github.com/anthropics/claude-code-action/blob/main/FAQ.md)."

 If a user asks for something outside these capabilities (and you have no other tools provided), politely explain that you cannot perform that action and suggest an alternative approach if possible.

@@ -608,7 +615,7 @@ f. If you are unable to complete certain steps, such as running a linter or test

 export async function createPrompt(
  claudeCommentId: number,
-  baseBranch: string | undefined,
+  defaultBranch: string | undefined,
  claudeBranch: string | undefined,
  githubData: FetchDataResult,
  context: ParsedGitHubContext,
@@ -617,7 +624,7 @@ export async function createPrompt(
    const preparedContext = prepareContext(
      context,
      claudeCommentId.toString(),
-      baseBranch,
+      defaultBranch,
      claudeBranch,
    );

@@ -636,11 +643,11 @@ export async function createPrompt(

    // Set allowed tools
    const allAllowedTools = buildAllowedToolsString(
+      preparedContext.eventData,
      preparedContext.allowedTools,
    );
    const allDisallowedTools = buildDisallowedToolsString(
      preparedContext.disallowedTools,
-      preparedContext.allowedTools,
    );

    core.exportVariable("ALLOWED_TOOLS", allAllowedTools);
--- a/src/create-prompt/types.ts
+++ b/src/create-prompt/types.ts
@@ -16,7 +16,7 @@ type PullRequestReviewCommentEvent = {
  commentId?: string; // May be present for review comments
  commentBody: string;
  claudeBranch?: string;
-  baseBranch?: string;
+  defaultBranch?: string;
 };

 type PullRequestReviewEvent = {
@@ -25,7 +25,7 @@ type PullRequestReviewEvent = {
  prNumber: string;
  commentBody: string;
  claudeBranch?: string;
-  baseBranch?: string;
+  defaultBranch?: string;
 };

 type IssueCommentEvent = {
@@ -33,7 +33,7 @@ type IssueCommentEvent = {
  commentId: string;
  issueNumber: string;
  isPR: false;
-  baseBranch: string;
+  defaultBranch: string;
  claudeBranch: string;
  commentBody: string;
 };
@@ -46,7 +46,7 @@ type PullRequestCommentEvent = {
  isPR: true;
  commentBody: string;
  claudeBranch?: string;
-  baseBranch?: string;
+  defaultBranch?: string;
 };

 type IssueOpenedEvent = {
@@ -54,7 +54,7 @@ type IssueOpenedEvent = {
  eventAction: "opened";
  isPR: false;
  issueNumber: string;
-  baseBranch: string;
+  defaultBranch: string;
  claudeBranch: string;
 };

@@ -63,7 +63,7 @@ type IssueAssignedEvent = {
  eventAction: "assigned";
  isPR: false;
  issueNumber: string;
-  baseBranch: string;
+  defaultBranch: string;
  claudeBranch: string;
  assigneeTrigger: string;
 };
@@ -74,7 +74,7 @@ type PullRequestEvent = {
  isPR: true;
  prNumber: string;
  claudeBranch?: string;
-  baseBranch?: string;
+  defaultBranch?: string;
 };

 // Union type for all possible event types
--- a/src/entrypoints/prepare.ts
+++ b/src/entrypoints/prepare.ts
@@ -77,28 +77,22 @@ async function run() {
    // Step 10: Create prompt file
    await createPrompt(
      commentId,
-      branchInfo.baseBranch,
+      branchInfo.defaultBranch,
      branchInfo.claudeBranch,
      githubData,
      context,
    );

    // Step 11: Get MCP configuration
-    const additionalMcpConfig = process.env.MCP_CONFIG || "";
-    const mcpConfig = await prepareMcpConfig({
+    const mcpConfig = await prepareMcpConfig(
      githubToken,
-      owner: context.repository.owner,
-      repo: context.repository.repo,
-      branch: branchInfo.currentBranch,
-      additionalMcpConfig,
-      claudeCommentId: commentId.toString(),
-    });
+      context.repository.owner,
+      context.repository.repo,
+      branchInfo.currentBranch,
+    );
    core.setOutput("mcp_config", mcpConfig);
  } catch (error) {
-    const errorMessage = error instanceof Error ? error.message : String(error);
-    core.setFailed(`Prepare step failed with error: ${errorMessage}`);
-    // Also output the clean error message for the action to capture
-    core.setOutput("prepare_error", errorMessage);
+    core.setFailed(`Prepare step failed with error: ${error}`);
    process.exit(1);
  }
 }
--- a/src/entrypoints/update-comment-link.ts
+++ b/src/entrypoints/update-comment-link.ts
@@ -12,14 +12,13 @@ import {
 } from "../github/context";
 import { GITHUB_SERVER_URL } from "../github/api/config";
 import { checkAndDeleteEmptyBranch } from "../github/operations/branch-cleanup";
-import { updateClaudeComment } from "../github/operations/comments/update-claude-comment";

 async function run() {
  try {
    const commentId = parseInt(process.env.CLAUDE_COMMENT_ID!);
    const githubToken = process.env.GITHUB_TOKEN!;
    const claudeBranch = process.env.CLAUDE_BRANCH;
-    const baseBranch = process.env.BASE_BRANCH || "main";
+    const defaultBranch = process.env.DEFAULT_BRANCH || "main";
    const triggerUsername = process.env.TRIGGER_USERNAME;

    const context = parseGitHubContext();
@@ -93,7 +92,7 @@ async function run() {
      owner,
      repo,
      claudeBranch,
-      baseBranch,
+      defaultBranch,
    );

    // Check if we need to add PR URL when we have a new branch
@@ -103,7 +102,7 @@ async function run() {
      // Check if comment already contains a PR URL
      const serverUrlPattern = serverUrl.replace(/[.*+?^${}()|[\]\\]/g, "\\$&");
      const prUrlPattern = new RegExp(
-        `${serverUrlPattern}\\/.+\\/compare\\/${baseBranch.replace(/[.*+?^${}()|[\]\\]/g, "\\$&")}\\.\\.\\.`,
+        `${serverUrlPattern}\\/.+\\/compare\\/${defaultBranch.replace(/[.*+?^${}()|[\]\\]/g, "\\$&")}\\.\\.\\.`,
      );
      const containsPRUrl = currentBody.match(prUrlPattern);

@@ -114,7 +113,7 @@ async function run() {
            await octokit.rest.repos.compareCommitsWithBasehead({
              owner,
              repo,
-              basehead: `${baseBranch}...${claudeBranch}`,
+              basehead: `${defaultBranch}...${claudeBranch}`,
            });

          // If there are changes (commits or file changes), add the PR URL
@@ -129,7 +128,7 @@ async function run() {
            const prBody = encodeURIComponent(
              `This PR addresses ${entityType.toLowerCase()} #${context.entityNumber}\n\nGenerated with [Claude Code](https://claude.ai/code)`,
            );
-            const prUrl = `${serverUrl}/${owner}/${repo}/compare/${baseBranch}...${claudeBranch}?quick_pull=1&title=${prTitle}&body=${prBody}`;
+            const prUrl = `${serverUrl}/${owner}/${repo}/compare/${defaultBranch}...${claudeBranch}?quick_pull=1&title=${prTitle}&body=${prBody}`;
            prLink = `\n[Create a PR](${prUrl})`;
          }
        } catch (error) {
@@ -146,84 +145,71 @@ async function run() {
      duration_api_ms?: number;
    } | null = null;
    let actionFailed = false;
-    let actionCancelled = false;
-    let errorDetails: string | undefined;

-    // Check if the workflow was cancelled
-    const isCancelled = process.env.CLAUDE_CANCELLED === "true";
+    // Check for existence of output file and parse it if available
+    try {
+      const outputFile = process.env.OUTPUT_FILE;
+      if (outputFile) {
+        const fileContent = await fs.readFile(outputFile, "utf8");
+        const outputData = JSON.parse(fileContent);

-    // First check if prepare step failed
-    const prepareSuccess = process.env.PREPARE_SUCCESS !== "false";
-    const prepareError = process.env.PREPARE_ERROR;
-
-    if (!prepareSuccess && prepareError) {
-      actionFailed = true;
-      errorDetails = prepareError;
-    } else if (isCancelled) {
-      // If the workflow was cancelled, set the cancelled flag
-      actionCancelled = true;
-    } else {
-      // Check if the Claude action failed
-      // CLAUDE_SUCCESS is set to the result of: steps.claude-code.outputs.conclusion == 'success'
-      // If the step didn't run or didn't set outputs.conclusion, CLAUDE_SUCCESS will be "false"
-      // If the step succeeded, CLAUDE_SUCCESS will be "true"
-      const claudeSuccess = process.env.CLAUDE_SUCCESS === "true";
-      actionFailed = !claudeSuccess;
-
-      // Try to read execution details from output file
-      try {
-        const outputFile = process.env.OUTPUT_FILE;
-        if (outputFile) {
-          const fileContent = await fs.readFile(outputFile, "utf8");
-          const outputData = JSON.parse(fileContent);
-
-          // Output file is an array, get the last element which contains execution details
-          if (Array.isArray(outputData) && outputData.length > 0) {
-            const lastElement = outputData[outputData.length - 1];
-            if (
-              lastElement.type === "result" &&
-              "cost_usd" in lastElement &&
-              "duration_ms" in lastElement
-            ) {
-              executionDetails = {
-                cost_usd: lastElement.cost_usd,
-                duration_ms: lastElement.duration_ms,
-                duration_api_ms: lastElement.duration_api_ms,
-              };
-            }
+        // Output file is an array, get the last element which contains execution details
+        if (Array.isArray(outputData) && outputData.length > 0) {
+          const lastElement = outputData[outputData.length - 1];
+          if (
+            lastElement.role === "system" &&
+            "cost_usd" in lastElement &&
+            "duration_ms" in lastElement
+          ) {
+            executionDetails = {
+              cost_usd: lastElement.cost_usd,
+              duration_ms: lastElement.duration_ms,
+              duration_api_ms: lastElement.duration_api_ms,
+            };
          }
        }
-      } catch (error) {
-        console.error("Error reading output file:", error);
-        // Error reading output file doesn't change the action status
-        // We already determined actionFailed based on CLAUDE_SUCCESS
      }
+
+      // Check if the action failed by looking at the exit code or error marker
+      const claudeSuccess = process.env.CLAUDE_SUCCESS !== "false";
+      actionFailed = !claudeSuccess;
+    } catch (error) {
+      console.error("Error reading output file:", error);
+      // If we can't read the file, check for any failure markers
+      actionFailed = process.env.CLAUDE_SUCCESS === "false";
    }

    // Prepare input for updateCommentBody function
    const commentInput: CommentUpdateInput = {
      currentBody,
      actionFailed,
-      actionCancelled,
      executionDetails,
      jobUrl,
      branchLink,
      prLink,
      branchName: shouldDeleteBranch ? undefined : claudeBranch,
      triggerUsername,
-      errorDetails,
    };

    const updatedBody = updateCommentBody(commentInput);

+    // Update the comment using the appropriate API
    try {
-      await updateClaudeComment(octokit.rest, {
-        owner,
-        repo,
-        commentId,
-        body: updatedBody,
-        isPullRequestReviewComment: isPRReviewComment,
-      });
+      if (isPRReviewComment) {
+        await octokit.rest.pulls.updateReviewComment({
+          owner,
+          repo,
+          comment_id: commentId,
+          body: updatedBody,
+        });
+      } else {
+        await octokit.rest.issues.updateComment({
+          owner,
+          repo,
+          comment_id: commentId,
+          body: updatedBody,
+        });
+      }
      console.log(
        `✅ Updated ${isPRReviewComment ? "PR review" : "issue"} comment ${commentId} with job link`,
      );
--- a/src/github/api/client.ts
+++ b/src/github/api/client.ts
@@ -9,10 +9,7 @@ export type Octokits = {

 export function createOctokit(token: string): Octokits {
  return {
-    rest: new Octokit({
-      auth: token,
-      baseUrl: GITHUB_API_URL,
-    }),
+    rest: new Octokit({ auth: token }),
    graphql: graphql.defaults({
      baseUrl: GITHUB_API_URL,
      headers: {
--- a/src/github/context.ts
+++ b/src/github/context.ts
@@ -32,7 +32,6 @@ export type ParsedGitHubContext = {
    disallowedTools: string;
    customInstructions: string;
    directPrompt: string;
-    baseBranch?: string;
  };
 };

@@ -56,7 +55,6 @@ export function parseGitHubContext(): ParsedGitHubContext {
      disallowedTools: process.env.DISALLOWED_TOOLS ?? "",
      customInstructions: process.env.CUSTOM_INSTRUCTIONS ?? "",
      directPrompt: process.env.DIRECT_PROMPT ?? "",
-      baseBranch: process.env.BASE_BRANCH,
    },
  };

--- a/src/github/data/fetcher.ts
+++ b/src/github/data/fetcher.ts
@@ -1,17 +1,17 @@
 import { execSync } from "child_process";
-import type { Octokits } from "../api/client";
-import { ISSUE_QUERY, PR_QUERY } from "../api/queries/github";
 import type {
+  GitHubPullRequest,
+  GitHubIssue,
  GitHubComment,
  GitHubFile,
-  GitHubIssue,
-  GitHubPullRequest,
  GitHubReview,
-  IssueQueryResponse,
  PullRequestQueryResponse,
+  IssueQueryResponse,
 } from "../types";
-import type { CommentWithImages } from "../utils/image-downloader";
+import { PR_QUERY, ISSUE_QUERY } from "../api/queries/github";
+import type { Octokits } from "../api/client";
 import { downloadCommentImages } from "../utils/image-downloader";
+import type { CommentWithImages } from "../utils/image-downloader";

 type FetchDataParams = {
  octokits: Octokits;
@@ -101,14 +101,6 @@ export async function fetchGitHubData({
  let changedFilesWithSHA: GitHubFileWithSHA[] = [];
  if (isPR && changedFiles.length > 0) {
    changedFilesWithSHA = changedFiles.map((file) => {
-      // Don't compute SHA for deleted files
-      if (file.changeType === "DELETED") {
-        return {
-          ...file,
-          sha: "deleted",
-        };
-      }
-
      try {
        // Use git hash-object to compute the SHA for the current file content
        const sha = execSync(`git hash-object "${file.path}"`, {
--- a/src/github/operations/branch-cleanup.ts
+++ b/src/github/operations/branch-cleanup.ts
@@ -6,7 +6,7 @@ export async function checkAndDeleteEmptyBranch(
  owner: string,
  repo: string,
  claudeBranch: string | undefined,
-  baseBranch: string,
+  defaultBranch: string,
 ): Promise<{ shouldDeleteBranch: boolean; branchLink: string }> {
  let branchLink = "";
  let shouldDeleteBranch = false;
@@ -18,7 +18,7 @@ export async function checkAndDeleteEmptyBranch(
        await octokit.rest.repos.compareCommitsWithBasehead({
          owner,
          repo,
-          basehead: `${baseBranch}...${claudeBranch}`,
+          basehead: `${defaultBranch}...${claudeBranch}`,
        });

      // If there are no commits, mark branch for deletion
--- a/src/github/operations/branch.ts
+++ b/src/github/operations/branch.ts
@@ -14,7 +14,7 @@ import type { Octokits } from "../api/client";
 import type { FetchDataResult } from "../data/fetcher";

 export type BranchInfo = {
-  baseBranch: string;
+  defaultBranch: string;
  claudeBranch?: string;
  currentBranch: string;
 };
@@ -26,9 +26,15 @@ export async function setupBranch(
 ): Promise<BranchInfo> {
  const { owner, repo } = context.repository;
  const entityNumber = context.entityNumber;
-  const { baseBranch } = context.inputs;
  const isPR = context.isPR;

+  // Get the default branch first
+  const repoResponse = await octokits.rest.repos.get({
+    owner,
+    repo,
+  });
+  const defaultBranch = repoResponse.data.default_branch;
+
  if (isPR) {
    const prData = githubData.contextData as GitHubPullRequest;
    const prState = prData.state;
@@ -36,7 +42,7 @@ export async function setupBranch(
    // Check if PR is closed or merged
    if (prState === "CLOSED" || prState === "MERGED") {
      console.log(
-        `PR #${entityNumber} is ${prState}, creating new branch from source...`,
+        `PR #${entityNumber} is ${prState}, creating new branch from default...`,
      );
      // Fall through to create a new branch like we do for issues
    } else {
@@ -52,36 +58,17 @@ export async function setupBranch(

      console.log(`Successfully checked out PR branch for PR #${entityNumber}`);

-      // For open PRs, we need to get the base branch of the PR
-      const baseBranch = prData.baseRefName;
-
+      // For open PRs, return branch info
      return {
-        baseBranch,
+        defaultBranch,
        currentBranch: branchName,
      };
    }
  }

-  // Determine source branch - use baseBranch if provided, otherwise fetch default
-  let sourceBranch: string;
-
-  if (baseBranch) {
-    // Use provided base branch for source
-    sourceBranch = baseBranch;
-  } else {
-    // No base branch provided, fetch the default branch to use as source
-    const repoResponse = await octokits.rest.repos.get({
-      owner,
-      repo,
-    });
-    sourceBranch = repoResponse.data.default_branch;
-  }
-
  // Creating a new branch for either an issue or closed/merged PR
  const entityType = isPR ? "pr" : "issue";
-  console.log(
-    `Creating new branch for ${entityType} #${entityNumber} from source branch: ${sourceBranch}...`,
-  );
+  console.log(`Creating new branch for ${entityType} #${entityNumber}...`);

  const timestamp = new Date()
    .toISOString()
@@ -93,14 +80,14 @@ export async function setupBranch(
  const newBranch = `claude/${entityType}-${entityNumber}-${timestamp}`;

  try {
-    // Get the SHA of the source branch
-    const sourceBranchRef = await octokits.rest.git.getRef({
+    // Get the SHA of the default branch
+    const defaultBranchRef = await octokits.rest.git.getRef({
      owner,
      repo,
-      ref: `heads/${sourceBranch}`,
+      ref: `heads/${defaultBranch}`,
    });

-    const currentSHA = sourceBranchRef.data.object.sha;
+    const currentSHA = defaultBranchRef.data.object.sha;

    console.log(`Current SHA: ${currentSHA}`);

@@ -122,9 +109,9 @@ export async function setupBranch(

    // Set outputs for GitHub Actions
    core.setOutput("CLAUDE_BRANCH", newBranch);
-    core.setOutput("BASE_BRANCH", sourceBranch);
+    core.setOutput("DEFAULT_BRANCH", defaultBranch);
    return {
-      baseBranch: sourceBranch,
+      defaultBranch,
      claudeBranch: newBranch,
      currentBranch: newBranch,
    };
--- a/src/github/operations/comment-logic.ts
+++ b/src/github/operations/comment-logic.ts
@@ -9,14 +9,12 @@ export type ExecutionDetails = {
 export type CommentUpdateInput = {
  currentBody: string;
  actionFailed: boolean;
-  actionCancelled?: boolean;
  executionDetails: ExecutionDetails | null;
  jobUrl: string;
  branchLink?: string;
  prLink?: string;
  branchName?: string;
  triggerUsername?: string;
-  errorDetails?: string;
 };

 export function ensureProperlyEncodedUrl(url: string): string | null {
@@ -75,10 +73,8 @@ export function updateCommentBody(input: CommentUpdateInput): string {
    branchLink,
    prLink,
    actionFailed,
-    actionCancelled,
    branchName,
    triggerUsername,
-    errorDetails,
  } = input;

  // Extract content from the original comment body
@@ -114,13 +110,7 @@ export function updateCommentBody(input: CommentUpdateInput): string {
  // Build the header
  let header = "";

-  if (actionCancelled) {
-    header = "**Claude's task was cancelled";
-    if (durationStr) {
-      header += ` after ${durationStr}`;
-    }
-    header += "**";
-  } else if (actionFailed) {
+  if (actionFailed) {
    header = "**Claude encountered an error";
    if (durationStr) {
      header += ` after ${durationStr}`;
@@ -187,14 +177,7 @@ export function updateCommentBody(input: CommentUpdateInput): string {
  }

  // Build the new body with blank line between header and separator
-  let newBody = `${header}${links}`;
-
-  // Add error details if available
-  if (actionFailed && errorDetails) {
-    newBody += `\n\n\`\`\`\n${errorDetails}\n\`\`\``;
-  }
-
-  newBody += `\n\n---\n`;
+  let newBody = `${header}${links}\n\n---\n`;

  // Clean up the body content
  // Remove any existing View job run, branch links from the bottom
--- a/src/github/operations/comments/update-claude-comment.ts
+++ b/src/github/operations/comments/update-claude-comment.ts
@@ -1,70 +0,0 @@
-import { Octokit } from "@octokit/rest";
-
-export type UpdateClaudeCommentParams = {
-  owner: string;
-  repo: string;
-  commentId: number;
-  body: string;
-  isPullRequestReviewComment: boolean;
-};
-
-export type UpdateClaudeCommentResult = {
-  id: number;
-  html_url: string;
-  updated_at: string;
-};
-
-/**
- * Updates a Claude comment on GitHub (either an issue/PR comment or a PR review comment)
- *
- * @param octokit - Authenticated Octokit instance
- * @param params - Parameters for updating the comment
- * @returns The updated comment details
- * @throws Error if the update fails
- */
-export async function updateClaudeComment(
-  octokit: Octokit,
-  params: UpdateClaudeCommentParams,
-): Promise<UpdateClaudeCommentResult> {
-  const { owner, repo, commentId, body, isPullRequestReviewComment } = params;
-
-  let response;
-
-  try {
-    if (isPullRequestReviewComment) {
-      // Try PR review comment API first
-      response = await octokit.rest.pulls.updateReviewComment({
-        owner,
-        repo,
-        comment_id: commentId,
-        body,
-      });
-    } else {
-      // Use issue comment API (works for both issues and PR general comments)
-      response = await octokit.rest.issues.updateComment({
-        owner,
-        repo,
-        comment_id: commentId,
-        body,
-      });
-    }
-  } catch (error: any) {
-    // If PR review comment update fails with 404, fall back to issue comment API
-    if (isPullRequestReviewComment && error.status === 404) {
-      response = await octokit.rest.issues.updateComment({
-        owner,
-        repo,
-        comment_id: commentId,
-        body,
-      });
-    } else {
-      throw error;
-    }
-  }
-
-  return {
-    id: response.data.id,
-    html_url: response.data.html_url,
-    updated_at: response.data.updated_at,
-  };
-}
--- a/src/github/operations/comments/update-with-branch.ts
+++ b/src/github/operations/comments/update-with-branch.ts
@@ -15,7 +15,6 @@ import {
  isPullRequestReviewCommentEvent,
  type ParsedGitHubContext,
 } from "../../context";
-import { updateClaudeComment } from "./update-claude-comment";

 export async function updateTrackingComment(
  octokit: Octokits,
@@ -37,19 +36,25 @@ export async function updateTrackingComment(

  // Update the existing comment with the branch link
  try {
-    const isPRReviewComment = isPullRequestReviewCommentEvent(context);
-
-    await updateClaudeComment(octokit.rest, {
-      owner,
-      repo,
-      commentId,
-      body: updatedBody,
-      isPullRequestReviewComment: isPRReviewComment,
-    });
-
-    console.log(
-      `✅ Updated ${isPRReviewComment ? "PR review" : "issue"} comment ${commentId} with branch link`,
-    );
+    if (isPullRequestReviewCommentEvent(context)) {
+      // For PR review comments (inline comments), use the pulls API
+      await octokit.rest.pulls.updateReviewComment({
+        owner,
+        repo,
+        comment_id: commentId,
+        body: updatedBody,
+      });
+      console.log(`✅ Updated PR review comment ${commentId} with branch link`);
+    } else {
+      // For all other comments, use the issues API
+      await octokit.rest.issues.updateComment({
+        owner,
+        repo,
+        comment_id: commentId,
+        body: updatedBody,
+      });
+      console.log(`✅ Updated issue comment ${commentId} with branch link`);
+    }
  } catch (error) {
    console.error("Error updating comment with branch link:", error);
    throw error;
--- a/src/mcp/github-file-ops-server.ts
+++ b/src/mcp/github-file-ops-server.ts
@@ -7,8 +7,6 @@ import { readFile } from "fs/promises";
 import { join } from "path";
 import fetch from "node-fetch";
 import { GITHUB_API_URL } from "../github/api/config";
-import { Octokit } from "@octokit/rest";
-import { updateClaudeComment } from "../github/operations/comments/update-claude-comment";

 type GitHubRef = {
  object: {
@@ -441,69 +439,6 @@ server.tool(
  },
 );

-server.tool(
-  "update_claude_comment",
-  "Update the Claude comment with progress and results (automatically handles both issue and PR comments)",
-  {
-    body: z.string().describe("The updated comment content"),
-  },
-  async ({ body }) => {
-    try {
-      const githubToken = process.env.GITHUB_TOKEN;
-      const claudeCommentId = process.env.CLAUDE_COMMENT_ID;
-      const eventName = process.env.GITHUB_EVENT_NAME;
-
-      if (!githubToken) {
-        throw new Error("GITHUB_TOKEN environment variable is required");
-      }
-      if (!claudeCommentId) {
-        throw new Error("CLAUDE_COMMENT_ID environment variable is required");
-      }
-
-      const owner = REPO_OWNER;
-      const repo = REPO_NAME;
-      const commentId = parseInt(claudeCommentId, 10);
-
-      const octokit = new Octokit({
-        auth: githubToken,
-      });
-
-      const isPullRequestReviewComment =
-        eventName === "pull_request_review_comment";
-
-      const result = await updateClaudeComment(octokit, {
-        owner,
-        repo,
-        commentId,
-        body,
-        isPullRequestReviewComment,
-      });
-
-      return {
-        content: [
-          {
-            type: "text",
-            text: JSON.stringify(result, null, 2),
-          },
-        ],
-      };
-    } catch (error) {
-      const errorMessage =
-        error instanceof Error ? error.message : String(error);
-      return {
-        content: [
-          {
-            type: "text",
-            text: `Error: ${errorMessage}`,
-          },
-        ],
-        error: errorMessage,
-        isError: true,
-      };
-    }
-  },
-);
-
 async function runServer() {
  const transport = new StdioServerTransport();
  await server.connect(transport);
--- a/src/mcp/install-mcp-server.ts
+++ b/src/mcp/install-mcp-server.ts
@@ -1,27 +1,13 @@
 import * as core from "@actions/core";

-type PrepareConfigParams = {
-  githubToken: string;
-  owner: string;
-  repo: string;
-  branch: string;
-  additionalMcpConfig?: string;
-  claudeCommentId?: string;
-};
-
 export async function prepareMcpConfig(
-  params: PrepareConfigParams,
+  githubToken: string,
+  owner: string,
+  repo: string,
+  branch: string,
 ): Promise<string> {
-  const {
-    githubToken,
-    owner,
-    repo,
-    branch,
-    additionalMcpConfig,
-    claudeCommentId,
-  } = params;
  try {
-    const baseMcpConfig = {
+    const mcpConfig = {
      mcpServers: {
        github: {
          command: "docker",
@@ -49,47 +35,12 @@ export async function prepareMcpConfig(
            REPO_NAME: repo,
            BRANCH_NAME: branch,
            REPO_DIR: process.env.GITHUB_WORKSPACE || process.cwd(),
-            ...(claudeCommentId && { CLAUDE_COMMENT_ID: claudeCommentId }),
-            GITHUB_EVENT_NAME: process.env.GITHUB_EVENT_NAME || "",
-            IS_PR: process.env.IS_PR || "false",
          },
        },
      },
    };

-    // Merge with additional MCP config if provided
-    if (additionalMcpConfig && additionalMcpConfig.trim()) {
-      try {
-        const additionalConfig = JSON.parse(additionalMcpConfig);
-
-        // Validate that parsed JSON is an object
-        if (typeof additionalConfig !== "object" || additionalConfig === null) {
-          throw new Error("MCP config must be a valid JSON object");
-        }
-
-        core.info(
-          "Merging additional MCP server configuration with built-in servers",
-        );
-
-        // Merge configurations with user config overriding built-in servers
-        const mergedConfig = {
-          ...baseMcpConfig,
-          ...additionalConfig,
-          mcpServers: {
-            ...baseMcpConfig.mcpServers,
-            ...additionalConfig.mcpServers,
-          },
-        };
-
-        return JSON.stringify(mergedConfig, null, 2);
-      } catch (parseError) {
-        core.warning(
-          `Failed to parse additional MCP config: ${parseError}. Using base config only.`,
-        );
-      }
-    }
-
-    return JSON.stringify(baseMcpConfig, null, 2);
+    return JSON.stringify(mcpConfig, null, 2);
  } catch (error) {
    core.setFailed(`Install MCP server failed with error: ${error}`);
    process.exit(1);
--- a/test/comment-logic.test.ts
+++ b/test/comment-logic.test.ts
@@ -39,25 +39,6 @@ describe("updateCommentBody", () => {
      expect(result).toContain("**Claude encountered an error after 45s**");
    });

-    it("includes error details when provided", () => {
-      const input = {
-        ...baseInput,
-        currentBody: "Claude Code is working...",
-        actionFailed: true,
-        executionDetails: { duration_ms: 45000 },
-        errorDetails: "Failed to fetch issue data",
-      };
-
-      const result = updateCommentBody(input);
-      expect(result).toContain("**Claude encountered an error after 45s**");
-      expect(result).toContain("[View job]");
-      expect(result).toContain("```\nFailed to fetch issue data\n```");
-      // Ensure error details come after the header/links
-      const errorIndex = result.indexOf("```");
-      const headerIndex = result.indexOf("**Claude encountered an error");
-      expect(errorIndex).toBeGreaterThan(headerIndex);
-    });
-
    it("handles username extraction from content when not provided", () => {
      const input = {
        ...baseInput,
@@ -418,48 +399,4 @@ describe("updateCommentBody", () => {
      );
    });
  });
-
-  describe("cancellation handling", () => {
-    it("shows cancellation message when actionCancelled is true", () => {
-      const input = {
-        ...baseInput,
-        currentBody: "Claude Code is working…",
-        actionCancelled: true,
-        executionDetails: { duration_ms: 30000 }, // 30s
-        triggerUsername: "test-user",
-      };
-
-      const result = updateCommentBody(input);
-      expect(result).toContain("**Claude's task was cancelled after 30s**");
-      expect(result).not.toContain("Claude encountered an error");
-      expect(result).not.toContain("Claude finished");
-    });
-
-    it("shows cancellation message without duration when duration is missing", () => {
-      const input = {
-        ...baseInput,
-        currentBody: "Claude Code is working…",
-        actionCancelled: true,
-        triggerUsername: "test-user",
-      };
-
-      const result = updateCommentBody(input);
-      expect(result).toContain("**Claude's task was cancelled**");
-      expect(result).not.toContain("after");
-    });
-
-    it("prioritizes cancellation over failure", () => {
-      const input = {
-        ...baseInput,
-        currentBody: "Claude Code is working…",
-        actionCancelled: true,
-        actionFailed: true, // Both are true, cancellation should take precedence
-        executionDetails: { duration_ms: 45000 },
-      };
-
-      const result = updateCommentBody(input);
-      expect(result).toContain("**Claude's task was cancelled after 45s**");
-      expect(result).not.toContain("Claude encountered an error");
-    });
-  });
 });
--- a/test/create-prompt.test.ts
+++ b/test/create-prompt.test.ts
@@ -8,6 +8,7 @@ import {
  buildDisallowedToolsString,
 } from "../src/create-prompt";
 import type { PreparedContext } from "../src/create-prompt";
+import type { EventData } from "../src/create-prompt/types";

 describe("generatePrompt", () => {
  const mockGitHubData = {
@@ -126,7 +127,7 @@ describe("generatePrompt", () => {
        eventName: "issue_comment",
        commentId: "67890",
        isPR: false,
-        baseBranch: "main",
+        defaultBranch: "main",
        claudeBranch: "claude/issue-67890-20240101_120000",
        issueNumber: "67890",
        commentBody: "@claude please fix this",
@@ -182,7 +183,7 @@ describe("generatePrompt", () => {
        eventAction: "opened",
        isPR: false,
        issueNumber: "789",
-        baseBranch: "main",
+        defaultBranch: "main",
        claudeBranch: "claude/issue-789-20240101_120000",
      },
    };
@@ -209,7 +210,7 @@ describe("generatePrompt", () => {
        eventAction: "assigned",
        isPR: false,
        issueNumber: "999",
-        baseBranch: "develop",
+        defaultBranch: "develop",
        claudeBranch: "claude/issue-999-20240101_120000",
        assigneeTrigger: "claude-bot",
      },
@@ -237,7 +238,7 @@ describe("generatePrompt", () => {
        eventAction: "opened",
        isPR: false,
        issueNumber: "789",
-        baseBranch: "main",
+        defaultBranch: "main",
        claudeBranch: "claude/issue-789-20240101_120000",
      },
    };
@@ -284,7 +285,7 @@ describe("generatePrompt", () => {
        commentId: "67890",
        isPR: false,
        issueNumber: "123",
-        baseBranch: "main",
+        defaultBranch: "main",
        claudeBranch: "claude/issue-67890-20240101_120000",
        commentBody: "@claude please fix this",
      },
@@ -306,7 +307,7 @@ describe("generatePrompt", () => {
        commentId: "67890",
        isPR: false,
        issueNumber: "123",
-        baseBranch: "main",
+        defaultBranch: "main",
        claudeBranch: "claude/issue-67890-20240101_120000",
        commentBody: "@claude please fix this",
      },
@@ -361,7 +362,7 @@ describe("generatePrompt", () => {
        eventAction: "opened",
        isPR: false,
        issueNumber: "789",
-        baseBranch: "main",
+        defaultBranch: "main",
        claudeBranch: "claude/issue-789-20240101_120000",
      },
    };
@@ -399,7 +400,7 @@ describe("generatePrompt", () => {
        commentId: "67890",
        isPR: false,
        issueNumber: "123",
-        baseBranch: "main",
+        defaultBranch: "main",
        claudeBranch: "claude/issue-123-20240101_120000",
        commentBody: "@claude please fix this",
      },
@@ -431,7 +432,7 @@ describe("generatePrompt", () => {
        prNumber: "456",
        commentBody: "@claude please fix this",
        claudeBranch: "claude/pr-456-20240101_120000",
-        baseBranch: "main",
+        defaultBranch: "main",
      },
    };

@@ -469,7 +470,7 @@ describe("generatePrompt", () => {
        isPR: true,
        prNumber: "456",
        commentBody: "@claude please fix this",
-        // No claudeBranch or baseBranch for open PRs
+        // No claudeBranch or defaultBranch for open PRs
      },
    };

@@ -502,7 +503,7 @@ describe("generatePrompt", () => {
        prNumber: "789",
        commentBody: "@claude please update this",
        claudeBranch: "claude/pr-789-20240101_123000",
-        baseBranch: "develop",
+        defaultBranch: "develop",
      },
    };

@@ -530,7 +531,7 @@ describe("generatePrompt", () => {
        commentId: "review-comment-123",
        commentBody: "@claude fix this issue",
        claudeBranch: "claude/pr-999-20240101_140000",
-        baseBranch: "main",
+        defaultBranch: "main",
      },
    };

@@ -558,7 +559,7 @@ describe("generatePrompt", () => {
        isPR: true,
        prNumber: "555",
        claudeBranch: "claude/pr-555-20240101_150000",
-        baseBranch: "main",
+        defaultBranch: "main",
      },
    };

@@ -603,7 +604,7 @@ describe("getEventTypeAndContext", () => {
        eventAction: "assigned",
        isPR: false,
        issueNumber: "999",
-        baseBranch: "main",
+        defaultBranch: "main",
        claudeBranch: "claude/issue-999-20240101_120000",
        assigneeTrigger: "claude-bot",
      },
@@ -618,7 +619,15 @@ describe("getEventTypeAndContext", () => {

 describe("buildAllowedToolsString", () => {
  test("should return issue comment tool for regular events", () => {
-    const result = buildAllowedToolsString();
+    const mockEventData: EventData = {
+      eventName: "issue_comment",
+      commentId: "123",
+      isPR: true,
+      prNumber: "456",
+      commentBody: "Test comment",
+    };
+
+    const result = buildAllowedToolsString(mockEventData);

    // The base tools should be in the result
    expect(result).toContain("Edit");
@@ -627,15 +636,22 @@ describe("buildAllowedToolsString", () => {
    expect(result).toContain("LS");
    expect(result).toContain("Read");
    expect(result).toContain("Write");
-    expect(result).toContain("mcp__github_file_ops__update_claude_comment");
-    expect(result).not.toContain("mcp__github__update_issue_comment");
+    expect(result).toContain("mcp__github__update_issue_comment");
    expect(result).not.toContain("mcp__github__update_pull_request_comment");
    expect(result).toContain("mcp__github_file_ops__commit_files");
    expect(result).toContain("mcp__github_file_ops__delete_files");
  });

  test("should return PR comment tool for inline review comments", () => {
-    const result = buildAllowedToolsString();
+    const mockEventData: EventData = {
+      eventName: "pull_request_review_comment",
+      isPR: true,
+      prNumber: "456",
+      commentBody: "Test review comment",
+      commentId: "789",
+    };
+
+    const result = buildAllowedToolsString(mockEventData);

    // The base tools should be in the result
    expect(result).toContain("Edit");
@@ -644,16 +660,23 @@ describe("buildAllowedToolsString", () => {
    expect(result).toContain("LS");
    expect(result).toContain("Read");
    expect(result).toContain("Write");
-    expect(result).toContain("mcp__github_file_ops__update_claude_comment");
    expect(result).not.toContain("mcp__github__update_issue_comment");
-    expect(result).not.toContain("mcp__github__update_pull_request_comment");
+    expect(result).toContain("mcp__github__update_pull_request_comment");
    expect(result).toContain("mcp__github_file_ops__commit_files");
    expect(result).toContain("mcp__github_file_ops__delete_files");
  });

  test("should append custom tools when provided", () => {
+    const mockEventData: EventData = {
+      eventName: "issue_comment",
+      commentId: "123",
+      isPR: true,
+      prNumber: "456",
+      commentBody: "Test comment",
+    };
+
    const customTools = "Tool1,Tool2,Tool3";
-    const result = buildAllowedToolsString(customTools);
+    const result = buildAllowedToolsString(mockEventData, customTools);

    // Base tools should be present
    expect(result).toContain("Edit");
@@ -699,51 +722,4 @@ describe("buildDisallowedToolsString", () => {
    expect(parts).toContain("BadTool1");
    expect(parts).toContain("BadTool2");
  });
-
-  test("should remove hardcoded disallowed tools if they are in allowed tools", () => {
-    const customDisallowedTools = "BadTool1,BadTool2";
-    const allowedTools = "WebSearch,SomeOtherTool";
-    const result = buildDisallowedToolsString(
-      customDisallowedTools,
-      allowedTools,
-    );
-
-    // WebSearch should be removed from disallowed since it's in allowed
-    expect(result).not.toContain("WebSearch");
-
-    // WebFetch should still be disallowed since it's not in allowed
-    expect(result).toContain("WebFetch");
-
-    // Custom disallowed tools should still be present
-    expect(result).toContain("BadTool1");
-    expect(result).toContain("BadTool2");
-  });
-
-  test("should remove all hardcoded disallowed tools if they are all in allowed tools", () => {
-    const allowedTools = "WebSearch,WebFetch,SomeOtherTool";
-    const result = buildDisallowedToolsString(undefined, allowedTools);
-
-    // Both hardcoded disallowed tools should be removed
-    expect(result).not.toContain("WebSearch");
-    expect(result).not.toContain("WebFetch");
-
-    // Result should be empty since no custom disallowed tools provided
-    expect(result).toBe("");
-  });
-
-  test("should handle custom disallowed tools when all hardcoded tools are overridden", () => {
-    const customDisallowedTools = "BadTool1,BadTool2";
-    const allowedTools = "WebSearch,WebFetch";
-    const result = buildDisallowedToolsString(
-      customDisallowedTools,
-      allowedTools,
-    );
-
-    // Hardcoded tools should be removed
-    expect(result).not.toContain("WebSearch");
-    expect(result).not.toContain("WebFetch");
-
-    // Only custom disallowed tools should remain
-    expect(result).toBe("BadTool1,BadTool2");
-  });
 });
--- a/test/install-mcp-server.test.ts
+++ b/test/install-mcp-server.test.ts
@@ -1,344 +0,0 @@
-import { describe, test, expect, beforeEach, afterEach, spyOn } from "bun:test";
-import { prepareMcpConfig } from "../src/mcp/install-mcp-server";
-import * as core from "@actions/core";
-
-describe("prepareMcpConfig", () => {
-  let consoleInfoSpy: any;
-  let consoleWarningSpy: any;
-  let setFailedSpy: any;
-  let processExitSpy: any;
-
-  beforeEach(() => {
-    consoleInfoSpy = spyOn(core, "info").mockImplementation(() => {});
-    consoleWarningSpy = spyOn(core, "warning").mockImplementation(() => {});
-    setFailedSpy = spyOn(core, "setFailed").mockImplementation(() => {});
-    processExitSpy = spyOn(process, "exit").mockImplementation(() => {
-      throw new Error("Process exit");
-    });
-  });
-
-  afterEach(() => {
-    consoleInfoSpy.mockRestore();
-    consoleWarningSpy.mockRestore();
-    setFailedSpy.mockRestore();
-    processExitSpy.mockRestore();
-  });
-
-  test("should return base config when no additional config is provided", async () => {
-    const result = await prepareMcpConfig({
-      githubToken: "test-token",
-      owner: "test-owner",
-      repo: "test-repo",
-      branch: "test-branch",
-    });
-
-    const parsed = JSON.parse(result);
-    expect(parsed.mcpServers).toBeDefined();
-    expect(parsed.mcpServers.github).toBeDefined();
-    expect(parsed.mcpServers.github_file_ops).toBeDefined();
-    expect(parsed.mcpServers.github.env.GITHUB_PERSONAL_ACCESS_TOKEN).toBe(
-      "test-token",
-    );
-    expect(parsed.mcpServers.github_file_ops.env.GITHUB_TOKEN).toBe(
-      "test-token",
-    );
-    expect(parsed.mcpServers.github_file_ops.env.REPO_OWNER).toBe("test-owner");
-    expect(parsed.mcpServers.github_file_ops.env.REPO_NAME).toBe("test-repo");
-    expect(parsed.mcpServers.github_file_ops.env.BRANCH_NAME).toBe(
-      "test-branch",
-    );
-  });
-
-  test("should return base config when additional config is empty string", async () => {
-    const result = await prepareMcpConfig({
-      githubToken: "test-token",
-      owner: "test-owner",
-      repo: "test-repo",
-      branch: "test-branch",
-      additionalMcpConfig: "",
-    });
-
-    const parsed = JSON.parse(result);
-    expect(parsed.mcpServers).toBeDefined();
-    expect(parsed.mcpServers.github).toBeDefined();
-    expect(parsed.mcpServers.github_file_ops).toBeDefined();
-    expect(consoleWarningSpy).not.toHaveBeenCalled();
-  });
-
-  test("should return base config when additional config is whitespace only", async () => {
-    const result = await prepareMcpConfig({
-      githubToken: "test-token",
-      owner: "test-owner",
-      repo: "test-repo",
-      branch: "test-branch",
-      additionalMcpConfig: "   \n\t  ",
-    });
-
-    const parsed = JSON.parse(result);
-    expect(parsed.mcpServers).toBeDefined();
-    expect(parsed.mcpServers.github).toBeDefined();
-    expect(parsed.mcpServers.github_file_ops).toBeDefined();
-    expect(consoleWarningSpy).not.toHaveBeenCalled();
-  });
-
-  test("should merge valid additional config with base config", async () => {
-    const additionalConfig = JSON.stringify({
-      mcpServers: {
-        custom_server: {
-          command: "custom-command",
-          args: ["arg1", "arg2"],
-          env: {
-            CUSTOM_ENV: "custom-value",
-          },
-        },
-      },
-    });
-
-    const result = await prepareMcpConfig({
-      githubToken: "test-token",
-      owner: "test-owner",
-      repo: "test-repo",
-      branch: "test-branch",
-      additionalMcpConfig: additionalConfig,
-    });
-
-    const parsed = JSON.parse(result);
-    expect(consoleInfoSpy).toHaveBeenCalledWith(
-      "Merging additional MCP server configuration with built-in servers",
-    );
-    expect(parsed.mcpServers.github).toBeDefined();
-    expect(parsed.mcpServers.github_file_ops).toBeDefined();
-    expect(parsed.mcpServers.custom_server).toBeDefined();
-    expect(parsed.mcpServers.custom_server.command).toBe("custom-command");
-    expect(parsed.mcpServers.custom_server.args).toEqual(["arg1", "arg2"]);
-    expect(parsed.mcpServers.custom_server.env.CUSTOM_ENV).toBe("custom-value");
-  });
-
-  test("should override built-in servers when additional config has same server names", async () => {
-    const additionalConfig = JSON.stringify({
-      mcpServers: {
-        github: {
-          command: "overridden-command",
-          args: ["overridden-arg"],
-          env: {
-            OVERRIDDEN_ENV: "overridden-value",
-          },
-        },
-      },
-    });
-
-    const result = await prepareMcpConfig({
-      githubToken: "test-token",
-      owner: "test-owner",
-      repo: "test-repo",
-      branch: "test-branch",
-      additionalMcpConfig: additionalConfig,
-    });
-
-    const parsed = JSON.parse(result);
-    expect(consoleInfoSpy).toHaveBeenCalledWith(
-      "Merging additional MCP server configuration with built-in servers",
-    );
-    expect(parsed.mcpServers.github.command).toBe("overridden-command");
-    expect(parsed.mcpServers.github.args).toEqual(["overridden-arg"]);
-    expect(parsed.mcpServers.github.env.OVERRIDDEN_ENV).toBe(
-      "overridden-value",
-    );
-    expect(
-      parsed.mcpServers.github.env.GITHUB_PERSONAL_ACCESS_TOKEN,
-    ).toBeUndefined();
-    expect(parsed.mcpServers.github_file_ops).toBeDefined();
-  });
-
-  test("should merge additional root-level properties", async () => {
-    const additionalConfig = JSON.stringify({
-      customProperty: "custom-value",
-      anotherProperty: {
-        nested: "value",
-      },
-      mcpServers: {
-        custom_server: {
-          command: "custom",
-        },
-      },
-    });
-
-    const result = await prepareMcpConfig({
-      githubToken: "test-token",
-      owner: "test-owner",
-      repo: "test-repo",
-      branch: "test-branch",
-      additionalMcpConfig: additionalConfig,
-    });
-
-    const parsed = JSON.parse(result);
-    expect(parsed.customProperty).toBe("custom-value");
-    expect(parsed.anotherProperty).toEqual({ nested: "value" });
-    expect(parsed.mcpServers.github).toBeDefined();
-    expect(parsed.mcpServers.custom_server).toBeDefined();
-  });
-
-  test("should handle invalid JSON gracefully", async () => {
-    const invalidJson = "{ invalid json }";
-
-    const result = await prepareMcpConfig({
-      githubToken: "test-token",
-      owner: "test-owner",
-      repo: "test-repo",
-      branch: "test-branch",
-      additionalMcpConfig: invalidJson,
-    });
-
-    const parsed = JSON.parse(result);
-    expect(consoleWarningSpy).toHaveBeenCalledWith(
-      expect.stringContaining("Failed to parse additional MCP config:"),
-    );
-    expect(parsed.mcpServers.github).toBeDefined();
-    expect(parsed.mcpServers.github_file_ops).toBeDefined();
-  });
-
-  test("should handle non-object JSON values", async () => {
-    const nonObjectJson = JSON.stringify("string value");
-
-    const result = await prepareMcpConfig({
-      githubToken: "test-token",
-      owner: "test-owner",
-      repo: "test-repo",
-      branch: "test-branch",
-      additionalMcpConfig: nonObjectJson,
-    });
-
-    const parsed = JSON.parse(result);
-    expect(consoleWarningSpy).toHaveBeenCalledWith(
-      expect.stringContaining("Failed to parse additional MCP config:"),
-    );
-    expect(consoleWarningSpy).toHaveBeenCalledWith(
-      expect.stringContaining("MCP config must be a valid JSON object"),
-    );
-    expect(parsed.mcpServers.github).toBeDefined();
-    expect(parsed.mcpServers.github_file_ops).toBeDefined();
-  });
-
-  test("should handle null JSON value", async () => {
-    const nullJson = JSON.stringify(null);
-
-    const result = await prepareMcpConfig({
-      githubToken: "test-token",
-      owner: "test-owner",
-      repo: "test-repo",
-      branch: "test-branch",
-      additionalMcpConfig: nullJson,
-    });
-
-    const parsed = JSON.parse(result);
-    expect(consoleWarningSpy).toHaveBeenCalledWith(
-      expect.stringContaining("Failed to parse additional MCP config:"),
-    );
-    expect(consoleWarningSpy).toHaveBeenCalledWith(
-      expect.stringContaining("MCP config must be a valid JSON object"),
-    );
-    expect(parsed.mcpServers.github).toBeDefined();
-    expect(parsed.mcpServers.github_file_ops).toBeDefined();
-  });
-
-  test("should handle array JSON value", async () => {
-    const arrayJson = JSON.stringify([1, 2, 3]);
-
-    const result = await prepareMcpConfig({
-      githubToken: "test-token",
-      owner: "test-owner",
-      repo: "test-repo",
-      branch: "test-branch",
-      additionalMcpConfig: arrayJson,
-    });
-
-    const parsed = JSON.parse(result);
-    // Arrays are objects in JavaScript, so they pass the object check
-    // But they'll fail when trying to spread or access mcpServers property
-    expect(consoleInfoSpy).toHaveBeenCalledWith(
-      "Merging additional MCP server configuration with built-in servers",
-    );
-    expect(parsed.mcpServers.github).toBeDefined();
-    expect(parsed.mcpServers.github_file_ops).toBeDefined();
-    // The array will be spread into the config (0: 1, 1: 2, 2: 3)
-    expect(parsed[0]).toBe(1);
-    expect(parsed[1]).toBe(2);
-    expect(parsed[2]).toBe(3);
-  });
-
-  test("should merge complex nested configurations", async () => {
-    const additionalConfig = JSON.stringify({
-      mcpServers: {
-        server1: {
-          command: "cmd1",
-          env: { KEY1: "value1" },
-        },
-        server2: {
-          command: "cmd2",
-          env: { KEY2: "value2" },
-        },
-        github_file_ops: {
-          command: "overridden",
-          env: { CUSTOM: "value" },
-        },
-      },
-      otherConfig: {
-        nested: {
-          deeply: "value",
-        },
-      },
-    });
-
-    const result = await prepareMcpConfig({
-      githubToken: "test-token",
-      owner: "test-owner",
-      repo: "test-repo",
-      branch: "test-branch",
-      additionalMcpConfig: additionalConfig,
-    });
-
-    const parsed = JSON.parse(result);
-    expect(parsed.mcpServers.server1).toBeDefined();
-    expect(parsed.mcpServers.server2).toBeDefined();
-    expect(parsed.mcpServers.github).toBeDefined();
-    expect(parsed.mcpServers.github_file_ops.command).toBe("overridden");
-    expect(parsed.mcpServers.github_file_ops.env.CUSTOM).toBe("value");
-    expect(parsed.otherConfig.nested.deeply).toBe("value");
-  });
-
-  test("should preserve GITHUB_ACTION_PATH in file_ops server args", async () => {
-    const oldEnv = process.env.GITHUB_ACTION_PATH;
-    process.env.GITHUB_ACTION_PATH = "/test/action/path";
-
-    const result = await prepareMcpConfig({
-      githubToken: "test-token",
-      owner: "test-owner",
-      repo: "test-repo",
-      branch: "test-branch",
-    });
-
-    const parsed = JSON.parse(result);
-    expect(parsed.mcpServers.github_file_ops.args[1]).toBe(
-      "/test/action/path/src/mcp/github-file-ops-server.ts",
-    );
-
-    process.env.GITHUB_ACTION_PATH = oldEnv;
-  });
-
-  test("should use process.cwd() when GITHUB_WORKSPACE is not set", async () => {
-    const oldEnv = process.env.GITHUB_WORKSPACE;
-    delete process.env.GITHUB_WORKSPACE;
-
-    const result = await prepareMcpConfig({
-      githubToken: "test-token",
-      owner: "test-owner",
-      repo: "test-repo",
-      branch: "test-branch",
-    });
-
-    const parsed = JSON.parse(result);
-    expect(parsed.mcpServers.github_file_ops.env.REPO_DIR).toBe(process.cwd());
-
-    process.env.GITHUB_WORKSPACE = oldEnv;
-  });
-});
--- a/test/prepare-context.test.ts
+++ b/test/prepare-context.test.ts
@@ -34,7 +34,7 @@ describe("parseEnvVarsWithContext", () => {
      beforeEach(() => {
        process.env = {
          ...BASE_ENV,
-          BASE_BRANCH: "main",
+          DEFAULT_BRANCH: "main",
          CLAUDE_BRANCH: "claude/issue-67890-20240101_120000",
        };
      });
@@ -62,7 +62,7 @@ describe("parseEnvVarsWithContext", () => {
          expect(result.eventData.claudeBranch).toBe(
            "claude/issue-67890-20240101_120000",
          );
-          expect(result.eventData.baseBranch).toBe("main");
+          expect(result.eventData.defaultBranch).toBe("main");
          expect(result.eventData.commentBody).toBe(
            "@claude can you help explain how to configure the logging system?",
          );
@@ -75,7 +75,7 @@ describe("parseEnvVarsWithContext", () => {
        ).toThrow("CLAUDE_BRANCH is required for issue_comment event");
      });

-      test("should throw error when BASE_BRANCH is missing", () => {
+      test("should throw error when DEFAULT_BRANCH is missing", () => {
        expect(() =>
          prepareContext(
            mockIssueCommentContext,
@@ -83,7 +83,7 @@ describe("parseEnvVarsWithContext", () => {
            undefined,
            "claude/issue-67890-20240101_120000",
          ),
-        ).toThrow("BASE_BRANCH is required for issue_comment event");
+        ).toThrow("DEFAULT_BRANCH is required for issue_comment event");
      });
    });

@@ -151,7 +151,7 @@ describe("parseEnvVarsWithContext", () => {
    beforeEach(() => {
      process.env = {
        ...BASE_ENV,
-        BASE_BRANCH: "main",
+        DEFAULT_BRANCH: "main",
        CLAUDE_BRANCH: "claude/issue-42-20240101_120000",
      };
    });
@@ -172,7 +172,7 @@ describe("parseEnvVarsWithContext", () => {
        result.eventData.eventAction === "opened"
      ) {
        expect(result.eventData.issueNumber).toBe("42");
-        expect(result.eventData.baseBranch).toBe("main");
+        expect(result.eventData.defaultBranch).toBe("main");
        expect(result.eventData.claudeBranch).toBe(
          "claude/issue-42-20240101_120000",
        );
@@ -195,7 +195,7 @@ describe("parseEnvVarsWithContext", () => {
        result.eventData.eventAction === "assigned"
      ) {
        expect(result.eventData.issueNumber).toBe("123");
-        expect(result.eventData.baseBranch).toBe("main");
+        expect(result.eventData.defaultBranch).toBe("main");
        expect(result.eventData.claudeBranch).toBe(
          "claude/issue-123-20240101_120000",
        );
@@ -209,7 +209,7 @@ describe("parseEnvVarsWithContext", () => {
      ).toThrow("CLAUDE_BRANCH is required for issues event");
    });

-    test("should throw error when BASE_BRANCH is missing for issues", () => {
+    test("should throw error when DEFAULT_BRANCH is missing for issues", () => {
      expect(() =>
        prepareContext(
          mockIssueOpenedContext,
@@ -217,7 +217,7 @@ describe("parseEnvVarsWithContext", () => {
          undefined,
          "claude/issue-42-20240101_120000",
        ),
-      ).toThrow("BASE_BRANCH is required for issues event");
+      ).toThrow("DEFAULT_BRANCH is required for issues event");
    });
  });

--- a/test/update-claude-comment.test.ts
+++ b/test/update-claude-comment.test.ts
@@ -1,413 +0,0 @@
-import { describe, test, expect, jest, beforeEach } from "bun:test";
-import { Octokit } from "@octokit/rest";
-import {
-  updateClaudeComment,
-  type UpdateClaudeCommentParams,
-} from "../src/github/operations/comments/update-claude-comment";
-
-describe("updateClaudeComment", () => {
-  let mockOctokit: Octokit;
-
-  beforeEach(() => {
-    mockOctokit = {
-      rest: {
-        issues: {
-          updateComment: jest.fn(),
-        },
-        pulls: {
-          updateReviewComment: jest.fn(),
-        },
-      },
-    } as any as Octokit;
-  });
-
-  test("should update issue comment successfully", async () => {
-    const mockResponse = {
-      data: {
-        id: 123456,
-        html_url: "https://github.com/owner/repo/issues/1#issuecomment-123456",
-        updated_at: "2024-01-01T00:00:00Z",
-        body: "Updated comment",
-      },
-    };
-
-    // @ts-expect-error Mock implementation doesn't match full type signature
-    mockOctokit.rest.issues.updateComment = jest
-      .fn()
-      .mockResolvedValue(mockResponse);
-
-    const params: UpdateClaudeCommentParams = {
-      owner: "testowner",
-      repo: "testrepo",
-      commentId: 123456,
-      body: "Updated comment",
-      isPullRequestReviewComment: false,
-    };
-
-    const result = await updateClaudeComment(mockOctokit, params);
-
-    expect(mockOctokit.rest.issues.updateComment).toHaveBeenCalledWith({
-      owner: "testowner",
-      repo: "testrepo",
-      comment_id: 123456,
-      body: "Updated comment",
-    });
-
-    expect(result).toEqual({
-      id: 123456,
-      html_url: "https://github.com/owner/repo/issues/1#issuecomment-123456",
-      updated_at: "2024-01-01T00:00:00Z",
-    });
-  });
-
-  test("should update PR comment successfully", async () => {
-    const mockResponse = {
-      data: {
-        id: 789012,
-        html_url: "https://github.com/owner/repo/pull/2#issuecomment-789012",
-        updated_at: "2024-01-02T00:00:00Z",
-        body: "Updated PR comment",
-      },
-    };
-
-    // @ts-expect-error Mock implementation doesn't match full type signature
-    mockOctokit.rest.issues.updateComment = jest
-      .fn()
-      .mockResolvedValue(mockResponse);
-
-    const params: UpdateClaudeCommentParams = {
-      owner: "testowner",
-      repo: "testrepo",
-      commentId: 789012,
-      body: "Updated PR comment",
-      isPullRequestReviewComment: false,
-    };
-
-    const result = await updateClaudeComment(mockOctokit, params);
-
-    expect(mockOctokit.rest.issues.updateComment).toHaveBeenCalledWith({
-      owner: "testowner",
-      repo: "testrepo",
-      comment_id: 789012,
-      body: "Updated PR comment",
-    });
-
-    expect(result).toEqual({
-      id: 789012,
-      html_url: "https://github.com/owner/repo/pull/2#issuecomment-789012",
-      updated_at: "2024-01-02T00:00:00Z",
-    });
-  });
-
-  test("should update PR review comment successfully", async () => {
-    const mockResponse = {
-      data: {
-        id: 345678,
-        html_url: "https://github.com/owner/repo/pull/3#discussion_r345678",
-        updated_at: "2024-01-03T00:00:00Z",
-        body: "Updated review comment",
-      },
-    };
-
-    // @ts-expect-error Mock implementation doesn't match full type signature
-    mockOctokit.rest.pulls.updateReviewComment = jest
-      .fn()
-      .mockResolvedValue(mockResponse);
-
-    const params: UpdateClaudeCommentParams = {
-      owner: "testowner",
-      repo: "testrepo",
-      commentId: 345678,
-      body: "Updated review comment",
-      isPullRequestReviewComment: true,
-    };
-
-    const result = await updateClaudeComment(mockOctokit, params);
-
-    expect(mockOctokit.rest.pulls.updateReviewComment).toHaveBeenCalledWith({
-      owner: "testowner",
-      repo: "testrepo",
-      comment_id: 345678,
-      body: "Updated review comment",
-    });
-
-    expect(result).toEqual({
-      id: 345678,
-      html_url: "https://github.com/owner/repo/pull/3#discussion_r345678",
-      updated_at: "2024-01-03T00:00:00Z",
-    });
-  });
-
-  test("should fallback to issue comment API when PR review comment update fails with 404", async () => {
-    const mockError = new Error("Not Found") as any;
-    mockError.status = 404;
-
-    const mockResponse = {
-      data: {
-        id: 456789,
-        html_url: "https://github.com/owner/repo/pull/4#issuecomment-456789",
-        updated_at: "2024-01-04T00:00:00Z",
-        body: "Updated via fallback",
-      },
-    };
-
-    // @ts-expect-error Mock implementation doesn't match full type signature
-    mockOctokit.rest.pulls.updateReviewComment = jest
-      .fn()
-      .mockRejectedValue(mockError);
-    // @ts-expect-error Mock implementation doesn't match full type signature
-    mockOctokit.rest.issues.updateComment = jest
-      .fn()
-      .mockResolvedValue(mockResponse);
-
-    const params: UpdateClaudeCommentParams = {
-      owner: "testowner",
-      repo: "testrepo",
-      commentId: 456789,
-      body: "Updated via fallback",
-      isPullRequestReviewComment: true,
-    };
-
-    const result = await updateClaudeComment(mockOctokit, params);
-
-    expect(mockOctokit.rest.pulls.updateReviewComment).toHaveBeenCalledWith({
-      owner: "testowner",
-      repo: "testrepo",
-      comment_id: 456789,
-      body: "Updated via fallback",
-    });
-
-    expect(mockOctokit.rest.issues.updateComment).toHaveBeenCalledWith({
-      owner: "testowner",
-      repo: "testrepo",
-      comment_id: 456789,
-      body: "Updated via fallback",
-    });
-
-    expect(result).toEqual({
-      id: 456789,
-      html_url: "https://github.com/owner/repo/pull/4#issuecomment-456789",
-      updated_at: "2024-01-04T00:00:00Z",
-    });
-  });
-
-  test("should propagate error when PR review comment update fails with non-404 error", async () => {
-    const mockError = new Error("Internal Server Error") as any;
-    mockError.status = 500;
-
-    // @ts-expect-error Mock implementation doesn't match full type signature
-    mockOctokit.rest.pulls.updateReviewComment = jest
-      .fn()
-      .mockRejectedValue(mockError);
-
-    const params: UpdateClaudeCommentParams = {
-      owner: "testowner",
-      repo: "testrepo",
-      commentId: 567890,
-      body: "This will fail",
-      isPullRequestReviewComment: true,
-    };
-
-    await expect(updateClaudeComment(mockOctokit, params)).rejects.toEqual(
-      mockError,
-    );
-
-    expect(mockOctokit.rest.pulls.updateReviewComment).toHaveBeenCalledWith({
-      owner: "testowner",
-      repo: "testrepo",
-      comment_id: 567890,
-      body: "This will fail",
-    });
-
-    // Ensure fallback wasn't attempted
-    expect(mockOctokit.rest.issues.updateComment).not.toHaveBeenCalled();
-  });
-
-  test("should propagate error when issue comment update fails", async () => {
-    const mockError = new Error("Forbidden");
-
-    // @ts-expect-error Mock implementation doesn't match full type signature
-    mockOctokit.rest.issues.updateComment = jest
-      .fn()
-      .mockRejectedValue(mockError);
-
-    const params: UpdateClaudeCommentParams = {
-      owner: "testowner",
-      repo: "testrepo",
-      commentId: 678901,
-      body: "This will also fail",
-      isPullRequestReviewComment: false,
-    };
-
-    await expect(updateClaudeComment(mockOctokit, params)).rejects.toEqual(
-      mockError,
-    );
-
-    expect(mockOctokit.rest.issues.updateComment).toHaveBeenCalledWith({
-      owner: "testowner",
-      repo: "testrepo",
-      comment_id: 678901,
-      body: "This will also fail",
-    });
-  });
-
-  test("should handle empty body", async () => {
-    const mockResponse = {
-      data: {
-        id: 111222,
-        html_url: "https://github.com/owner/repo/issues/5#issuecomment-111222",
-        updated_at: "2024-01-05T00:00:00Z",
-        body: "",
-      },
-    };
-
-    // @ts-expect-error Mock implementation doesn't match full type signature
-    mockOctokit.rest.issues.updateComment = jest
-      .fn()
-      .mockResolvedValue(mockResponse);
-
-    const params: UpdateClaudeCommentParams = {
-      owner: "testowner",
-      repo: "testrepo",
-      commentId: 111222,
-      body: "",
-      isPullRequestReviewComment: false,
-    };
-
-    const result = await updateClaudeComment(mockOctokit, params);
-
-    expect(result).toEqual({
-      id: 111222,
-      html_url: "https://github.com/owner/repo/issues/5#issuecomment-111222",
-      updated_at: "2024-01-05T00:00:00Z",
-    });
-  });
-
-  test("should handle very long body", async () => {
-    const longBody = "x".repeat(10000);
-    const mockResponse = {
-      data: {
-        id: 333444,
-        html_url: "https://github.com/owner/repo/issues/6#issuecomment-333444",
-        updated_at: "2024-01-06T00:00:00Z",
-        body: longBody,
-      },
-    };
-
-    // @ts-expect-error Mock implementation doesn't match full type signature
-    mockOctokit.rest.issues.updateComment = jest
-      .fn()
-      .mockResolvedValue(mockResponse);
-
-    const params: UpdateClaudeCommentParams = {
-      owner: "testowner",
-      repo: "testrepo",
-      commentId: 333444,
-      body: longBody,
-      isPullRequestReviewComment: false,
-    };
-
-    const result = await updateClaudeComment(mockOctokit, params);
-
-    expect(mockOctokit.rest.issues.updateComment).toHaveBeenCalledWith({
-      owner: "testowner",
-      repo: "testrepo",
-      comment_id: 333444,
-      body: longBody,
-    });
-
-    expect(result).toEqual({
-      id: 333444,
-      html_url: "https://github.com/owner/repo/issues/6#issuecomment-333444",
-      updated_at: "2024-01-06T00:00:00Z",
-    });
-  });
-
-  test("should handle markdown formatting in body", async () => {
-    const markdownBody = `
-# Header
- List item 1
- List item 2
-
-\`\`\`typescript
-const code = "example";
-\`\`\`
-
-[Link](https://example.com)
-    `.trim();
-
-    const mockResponse = {
-      data: {
-        id: 555666,
-        html_url: "https://github.com/owner/repo/issues/7#issuecomment-555666",
-        updated_at: "2024-01-07T00:00:00Z",
-        body: markdownBody,
-      },
-    };
-
-    // @ts-expect-error Mock implementation doesn't match full type signature
-    mockOctokit.rest.issues.updateComment = jest
-      .fn()
-      .mockResolvedValue(mockResponse);
-
-    const params: UpdateClaudeCommentParams = {
-      owner: "testowner",
-      repo: "testrepo",
-      commentId: 555666,
-      body: markdownBody,
-      isPullRequestReviewComment: false,
-    };
-
-    const result = await updateClaudeComment(mockOctokit, params);
-
-    expect(mockOctokit.rest.issues.updateComment).toHaveBeenCalledWith({
-      owner: "testowner",
-      repo: "testrepo",
-      comment_id: 555666,
-      body: markdownBody,
-    });
-
-    expect(result).toEqual({
-      id: 555666,
-      html_url: "https://github.com/owner/repo/issues/7#issuecomment-555666",
-      updated_at: "2024-01-07T00:00:00Z",
-    });
-  });
-
-  test("should handle different response data fields", async () => {
-    const mockResponse = {
-      data: {
-        id: 777888,
-        html_url: "https://github.com/owner/repo/pull/8#discussion_r777888",
-        updated_at: "2024-01-08T12:30:45Z",
-        body: "Updated",
-        // Additional fields that might be in the response
-        created_at: "2024-01-01T00:00:00Z",
-        user: { login: "bot" },
-        node_id: "MDI0OlB1bGxSZXF1ZXN0UmV2aWV3Q29tbWVudDc3Nzg4OA==",
-      },
-    };
-
-    // @ts-expect-error Mock implementation doesn't match full type signature
-    mockOctokit.rest.pulls.updateReviewComment = jest
-      .fn()
-      .mockResolvedValue(mockResponse);
-
-    const params: UpdateClaudeCommentParams = {
-      owner: "testowner",
-      repo: "testrepo",
-      commentId: 777888,
-      body: "Updated",
-      isPullRequestReviewComment: true,
-    };
-
-    const result = await updateClaudeComment(mockOctokit, params);
-
-    // Should only return the specific fields we care about
-    expect(result).toEqual({
-      id: 777888,
-      html_url: "https://github.com/owner/repo/pull/8#discussion_r777888",
-      updated_at: "2024-01-08T12:30:45Z",
-    });
-  });
-});
--- a/test/update-comment-link-logic.test.ts
+++ b/test/update-comment-link-logic.test.ts
@@ -1,160 +0,0 @@
-import { describe, test, expect, beforeEach, afterEach } from "bun:test";
-
-describe("update-comment-link workflow status detection", () => {
-  let originalEnv: NodeJS.ProcessEnv;
-
-  beforeEach(() => {
-    originalEnv = { ...process.env };
-  });
-
-  afterEach(() => {
-    process.env = originalEnv;
-  });
-
-  test("should detect prepare step failure", () => {
-    process.env.PREPARE_SUCCESS = "false";
-    process.env.PREPARE_ERROR = "Failed to fetch issue data";
-
-    const prepareSuccess = process.env.PREPARE_SUCCESS !== "false";
-    const prepareError = process.env.PREPARE_ERROR;
-
-    let actionFailed = false;
-    let errorDetails: string | undefined;
-
-    if (!prepareSuccess && prepareError) {
-      actionFailed = true;
-      errorDetails = prepareError;
-    }
-
-    expect(actionFailed).toBe(true);
-    expect(errorDetails).toBe("Failed to fetch issue data");
-  });
-
-  test("should detect claude-code step failure when prepare succeeds", () => {
-    process.env.PREPARE_SUCCESS = "true";
-    process.env.CLAUDE_SUCCESS = "false";
-
-    const prepareSuccess = process.env.PREPARE_SUCCESS !== "false";
-    const prepareError = process.env.PREPARE_ERROR;
-
-    let actionFailed = false;
-
-    if (!prepareSuccess && prepareError) {
-      actionFailed = true;
-    } else {
-      const claudeSuccess = process.env.CLAUDE_SUCCESS === "true";
-      actionFailed = !claudeSuccess;
-    }
-
-    expect(actionFailed).toBe(true);
-  });
-
-  test("should detect success when both steps succeed", () => {
-    process.env.PREPARE_SUCCESS = "true";
-    process.env.CLAUDE_SUCCESS = "true";
-
-    const prepareSuccess = process.env.PREPARE_SUCCESS !== "false";
-    const prepareError = process.env.PREPARE_ERROR;
-
-    let actionFailed = false;
-
-    if (!prepareSuccess && prepareError) {
-      actionFailed = true;
-    } else {
-      const claudeSuccess = process.env.CLAUDE_SUCCESS === "true";
-      actionFailed = !claudeSuccess;
-    }
-
-    expect(actionFailed).toBe(false);
-  });
-
-  test("should treat missing CLAUDE_SUCCESS env var as failure", () => {
-    process.env.PREPARE_SUCCESS = "true";
-    delete process.env.CLAUDE_SUCCESS;
-
-    const prepareSuccess = process.env.PREPARE_SUCCESS !== "false";
-    const prepareError = process.env.PREPARE_ERROR;
-
-    let actionFailed = false;
-
-    if (!prepareSuccess && prepareError) {
-      actionFailed = true;
-    } else {
-      // When CLAUDE_SUCCESS is undefined, it's not === "true", so claudeSuccess = false
-      const claudeSuccess = process.env.CLAUDE_SUCCESS === "true";
-      actionFailed = !claudeSuccess;
-    }
-
-    expect(actionFailed).toBe(true);
-  });
-
-  test("should handle undefined PREPARE_SUCCESS as success", () => {
-    delete process.env.PREPARE_SUCCESS;
-    delete process.env.PREPARE_ERROR;
-    process.env.CLAUDE_SUCCESS = "true";
-
-    const prepareSuccess = process.env.PREPARE_SUCCESS !== "false";
-    const prepareError = process.env.PREPARE_ERROR;
-
-    let actionFailed = false;
-
-    if (!prepareSuccess && prepareError) {
-      actionFailed = true;
-    } else {
-      const claudeSuccess = process.env.CLAUDE_SUCCESS === "true";
-      actionFailed = !claudeSuccess;
-    }
-
-    expect(actionFailed).toBe(false);
-  });
-
-  test("should detect cancellation when CLAUDE_CANCELLED is true", () => {
-    process.env.PREPARE_SUCCESS = "true";
-    process.env.CLAUDE_SUCCESS = "false";
-    process.env.CLAUDE_CANCELLED = "true";
-
-    const prepareSuccess = process.env.PREPARE_SUCCESS !== "false";
-    const prepareError = process.env.PREPARE_ERROR;
-    const isCancelled = process.env.CLAUDE_CANCELLED === "true";
-
-    let actionFailed = false;
-    let actionCancelled = false;
-
-    if (!prepareSuccess && prepareError) {
-      actionFailed = true;
-    } else if (isCancelled) {
-      actionCancelled = true;
-    } else {
-      const claudeSuccess = process.env.CLAUDE_SUCCESS === "true";
-      actionFailed = !claudeSuccess;
-    }
-
-    expect(actionFailed).toBe(false);
-    expect(actionCancelled).toBe(true);
-  });
-
-  test("should not detect cancellation when CLAUDE_CANCELLED is false", () => {
-    process.env.PREPARE_SUCCESS = "true";
-    process.env.CLAUDE_SUCCESS = "false";
-    process.env.CLAUDE_CANCELLED = "false";
-
-    const prepareSuccess = process.env.PREPARE_SUCCESS !== "false";
-    const prepareError = process.env.PREPARE_ERROR;
-    const isCancelled = process.env.CLAUDE_CANCELLED === "true";
-
-    let actionFailed = false;
-    let actionCancelled = false;
-
-    if (!prepareSuccess && prepareError) {
-      actionFailed = true;
-    } else if (isCancelled) {
-      actionCancelled = true;
-    } else {
-      const claudeSuccess = process.env.CLAUDE_SUCCESS === "true";
-      actionFailed = !claudeSuccess;
-    }
-
-    expect(actionFailed).toBe(true);
-    expect(actionCancelled).toBe(false);
-  });
-});
Author	SHA1	Message	Date
Lina Tawfik	2a9592678e	Revert .gitignore changes	2025-05-28 18:16:13 -07:00
Lina Tawfik	7d1773e98f	Remove test-markdown.json and update .gitignore	2025-05-28 18:14:13 -07:00
Lina Tawfik	019043f2fb	Remove rendered.html from repository	2025-05-28 18:13:49 -07:00
Lina Tawfik	4ed7e5538d	Fix prettier formatting	2025-05-28 18:13:39 -07:00
Lina Tawfik	cf04e19dbc	Refactor tests to remove redundancy and improve structure - Remove redundant 'mixed input patterns' test from sanitizer.test.ts - Consolidate integration tests into 2 focused real-world scenarios - Add HTML comment stripping to sanitizeContent function - Update test expectations to match sanitization behavior - Maintain full coverage with fewer, more focused tests	2025-05-28 18:12:07 -07:00
Lina Tawfik	046ef964a9	Format code with prettier	2025-05-28 17:30:42 -07:00
Lina Tawfik	61cd297c18	Add enhanced text sanitization	2025-05-28 17:29:09 -07:00