fix: wrap github MCP config with mcpServers in issue-triage workflow

Update the MCP configuration structure to properly wrap the github
server configuration within an mcpServers object, matching the
expected format for MCP config files.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>

.github/workflows/claude-review.yml

@@ -29,4 +29,4 @@ jobs:
 
             Be constructive and specific in your feedback. Give inline comments where applicable.
           anthropic_api_key: ${{ secrets.ANTHROPIC_API_KEY }}
-          allowed_tools: "mcp__github__create_pending_pull_request_review,mcp__github__add_pull_request_review_comment_to_pending_review,mcp__github__submit_pending_pull_request_review,mcp__github__get_pull_request_diff"
+          allowed_tools: "mcp__github__add_pull_request_review_comment"

.github/workflows/issue-triage.yml

@@ -32,7 +32,7 @@ jobs:
                   "--rm",
                   "-e",
                   "GITHUB_PERSONAL_ACCESS_TOKEN",
-                  "ghcr.io/github/github-mcp-server:sha-6d69797"
+                  "ghcr.io/github/github-mcp-server:sha-7aced2b"
                 ],
                 "env": {
                   "GITHUB_PERSONAL_ACCESS_TOKEN": "${{ secrets.GITHUB_TOKEN }}"

.github/workflows/release.yml

@@ -1,138 +0,0 @@
-name: Create Release
-
-on:
-  workflow_dispatch:
-    inputs:
-      dry_run:
-        description: "Dry run (only show what would be created)"
-        required: false
-        type: boolean
-        default: false
-
-jobs:
-  create-release:
-    runs-on: ubuntu-latest
-    permissions:
-      contents: write
-    outputs:
-      next_version: ${{ steps.next_version.outputs.next_version }}
-    steps:
-      - name: Checkout code
-        uses: actions/checkout@v4
-        with:
-          fetch-depth: 0
-
-      - name: Get latest tag
-        id: get_latest_tag
-        run: |
-          # Get only version tags (v + number pattern)
-          latest_tag=$(git tag -l 'v[0-9]*' | sort -V | tail -1 || echo "v0.0.0")
-          if [ -z "$latest_tag" ]; then
-            latest_tag="v0.0.0"
-          fi
-          echo "latest_tag=$latest_tag" >> $GITHUB_OUTPUT
-          echo "Latest tag: $latest_tag"
-
-      - name: Calculate next version
-        id: next_version
-        run: |
-          latest_tag="${{ steps.get_latest_tag.outputs.latest_tag }}"
-          # Remove 'v' prefix and split by dots
-          version=${latest_tag#v}
-          IFS='.' read -ra VERSION_PARTS <<< "$version"
-
-          # Increment patch version
-          major=${VERSION_PARTS[0]:-0}
-          minor=${VERSION_PARTS[1]:-0}
-          patch=${VERSION_PARTS[2]:-0}
-          patch=$((patch + 1))
-
-          next_version="v${major}.${minor}.${patch}"
-          echo "next_version=$next_version" >> $GITHUB_OUTPUT
-          echo "Next version: $next_version"
-
-      - name: Display dry run info
-        if: ${{ inputs.dry_run }}
-        run: |
-          echo "🔍 DRY RUN MODE"
-          echo "Would create tag: ${{ steps.next_version.outputs.next_version }}"
-          echo "From commit: ${{ github.sha }}"
-          echo "Previous tag: ${{ steps.get_latest_tag.outputs.latest_tag }}"
-
-      - name: Create and push tag
-        if: ${{ !inputs.dry_run }}
-        run: |
-          next_version="${{ steps.next_version.outputs.next_version }}"
-          git config user.name "github-actions[bot]"
-          git config user.email "github-actions[bot]@users.noreply.github.com"
-
-          git tag -a "$next_version" -m "Release $next_version"
-          git push origin "$next_version"
-
-      - name: Create Release
-        if: ${{ !inputs.dry_run }}
-        env:
-          GH_TOKEN: ${{ github.token }}
-        run: |
-          next_version="${{ steps.next_version.outputs.next_version }}"
-
-          gh release create "$next_version" \
-            --title "$next_version" \
-            --generate-notes \
-            --latest=false # We want to keep beta as the latest
-
-  update-beta-tag:
-    needs: create-release
-    if: ${{ !inputs.dry_run }}
-    runs-on: ubuntu-latest
-    permissions:
-      contents: write
-    steps:
-      - name: Checkout code
-        uses: actions/checkout@v4
-        with:
-          fetch-depth: 0
-
-      - name: Update beta tag
-        run: |
-          # Get the latest version tag
-          VERSION=$(git tag -l 'v[0-9]*' | sort -V | tail -1)
-
-          # Update the beta tag to point to this release
-          git config user.name "github-actions[bot]"
-          git config user.email "github-actions[bot]@users.noreply.github.com"
-          git tag -fa beta -m "Update beta tag to ${VERSION}"
-          git push origin beta --force
-
-      - name: Update beta release to be latest
-        env:
-          GH_TOKEN: ${{ github.token }}
-        run: |
-          # Update beta release to be marked as latest
-          gh release edit beta --latest
-
-  update-major-tag:
-    needs: create-release
-    if: ${{ !inputs.dry_run }}
-    runs-on: ubuntu-latest
-    permissions:
-      contents: write
-    steps:
-      - name: Checkout code
-        uses: actions/checkout@v4
-        with:
-          fetch-depth: 0
-
-      - name: Update major version tag
-        run: |
-          next_version="${{ needs.create-release.outputs.next_version }}"
-          # Extract major version (e.g., v0 from v0.0.20)
-          major_version=$(echo "$next_version" | cut -d. -f1)
-
-          # Update the major version tag to point to this release
-          git config user.name "github-actions[bot]"
-          git config user.email "github-actions[bot]@users.noreply.github.com"
-          git tag -fa "$major_version" -m "Update $major_version tag to $next_version"
-          git push origin "$major_version" --force
-
-          echo "Updated $major_version tag to point to $next_version"

.prettierignore

@@ -1,2 +0,0 @@
-# Test fixtures should not be formatted to preserve exact output matching
-test/fixtures/

CONTRIBUTING.md

@@ -50,6 +50,20 @@ Thank you for your interest in contributing to Claude Code Action! This document
    bun test
    ```
 
+2. **Integration Tests** (using GitHub Actions locally):
+
+   ```bash
+   ./test-local.sh
+   ```
+
+   This script:
+
+   - Installs `act` if not present (requires Homebrew on macOS)
+   - Runs the GitHub Action workflow locally using Docker
+   - Requires your `ANTHROPIC_API_KEY` to be set
+
+   On Apple Silicon Macs, the script automatically adds the `--container-architecture linux/amd64` flag to avoid compatibility issues.
+
 ## Pull Request Process
 
 1. Create a new branch from `main`:
@@ -89,7 +103,13 @@ Thank you for your interest in contributing to Claude Code Action! This document
 
 When modifying the action:
 
-1. Test in a real GitHub Actions workflow by:
+1. Test locally with the test script:
+
+   ```bash
+   ./test-local.sh
+   ```
+
+2. Test in a real GitHub Actions workflow by:
    - Creating a test repository
    - Using your branch as the action source:
      ```yaml

FAQ.md

@@ -12,10 +12,6 @@ The `github-actions` user cannot trigger subsequent GitHub Actions workflows. Th
 
 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 can't I assign @claude to an issue on my repository?
-
-If you're in a public repository, you should be able to assign to Claude without issue. If it's a private organization repository, you can only assign to users in your own organization, which Claude isn't. In this case, you'll need to make a custom user in that case.
-
 ### 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:

README.md

@@ -49,7 +49,7 @@ on:
   pull_request_review_comment:
     types: [created]
   issues:
-    types: [opened, assigned, labeled]
+    types: [opened, assigned]
   pull_request_review:
     types: [submitted]
 
@@ -65,46 +65,32 @@ jobs:
           # trigger_phrase: "/claude"
           # Optional: add assignee trigger for issues
           # assignee_trigger: "claude"
-          # Optional: add label trigger for issues
-          # label_trigger: "claude"
           # Optional: add custom environment variables (YAML format)
           # claude_env: |
           #   NODE_ENV: test
           #   DEBUG: true
           #   API_URL: https://api.example.com
-          # Optional: limit the number of conversation turns
-          # max_turns: "5"
-          # Optional: grant additional permissions (requires corresponding GitHub token permissions)
-          # additional_permissions: |
-          #   actions: read
 ```
 
 ## Inputs
 
-| Input                    | Description                                                                                                          | Required | Default   |
+| Input                 | Description                                                                                                          | Required | Default   |
-| ------------------------ | -------------------------------------------------------------------------------------------------------------------- | -------- | --------- |
+| --------------------- | -------------------------------------------------------------------------------------------------------------------- | -------- | --------- |
-| `anthropic_api_key`      | Anthropic API key (required for direct API, not needed for Bedrock/Vertex)                                           | No\*     | -         |
+| `anthropic_api_key`   | Anthropic API key (required for direct API, not needed for Bedrock/Vertex)                                           | No\*     | -         |
-| `direct_prompt`          | Direct prompt for Claude to execute automatically without needing a trigger (for automated workflows)                | No       | -         |
+| `direct_prompt`       | Direct prompt for Claude to execute automatically without needing a trigger (for automated workflows)                | No       | -         |
-| `base_branch`            | The base branch to use for creating new branches (e.g., 'main', 'develop')                                           | No       | -         |
+| `timeout_minutes`     | Timeout in minutes for execution                                                                                     | No       | `30`      |
-| `max_turns`              | Maximum number of conversation turns Claude can take (limits back-and-forth exchanges)                               | No       | -         |
+| `github_token`        | GitHub token for Claude to operate with. **Only include this if you're connecting a custom GitHub app of your own!** | No       | -         |
-| `timeout_minutes`        | Timeout in minutes for execution                                                                                     | No       | `30`      |
+| `model`               | Model to use (provider-specific format required for Bedrock/Vertex)                                                  | No       | -         |
-| `use_sticky_comment`     | Use just one comment to deliver PR comments (only applies for pull_request event workflows)                          | No       | `false`   |
+| `anthropic_model`     | **DEPRECATED**: Use `model` instead. Kept for backward compatibility.                                                | No       | -         |
-| `github_token`           | GitHub token for Claude to operate with. **Only include this if you're connecting a custom GitHub app of your own!** | No       | -         |
+| `use_bedrock`         | Use Amazon Bedrock with OIDC authentication instead of direct Anthropic API                                          | No       | `false`   |
-| `model`                  | Model to use (provider-specific format required for Bedrock/Vertex)                                                  | No       | -         |
+| `use_vertex`          | Use Google Vertex AI with OIDC authentication instead of direct Anthropic API                                        | No       | `false`   |
-| `fallback_model`         | Enable automatic fallback to specified model when primary model is unavailable                                       | No       | -         |
+| `allowed_tools`       | Additional tools for Claude to use (the base GitHub tools will always be included)                                   | No       | ""        |
-| `anthropic_model`        | **DEPRECATED**: Use `model` instead. Kept for backward compatibility.                                                | No       | -         |
+| `disallowed_tools`    | Tools that Claude should never use                                                                                   | No       | ""        |
-| `use_bedrock`            | Use Amazon Bedrock with OIDC authentication instead of direct Anthropic API                                          | No       | `false`   |
+| `custom_instructions` | Additional custom instructions to include in the prompt for Claude                                                   | No       | ""        |
-| `use_vertex`             | Use Google Vertex AI with OIDC authentication instead of direct Anthropic API                                        | No       | `false`   |
+| `mcp_config`          | Additional MCP configuration (JSON string) that merges with the built-in GitHub MCP servers                          | No       | ""        |
-| `allowed_tools`          | Additional tools for Claude to use (the base GitHub tools will always be included)                                   | No       | ""        |
+| `assignee_trigger`    | The assignee username that triggers the action (e.g. @claude). Only used for issue assignment                        | No       | -         |
-| `disallowed_tools`       | Tools that Claude should never use                                                                                   | No       | ""        |
+| `trigger_phrase`      | The trigger phrase to look for in comments, issue/PR bodies, and issue titles                                        | No       | `@claude` |
-| `custom_instructions`    | Additional custom instructions to include in the prompt for Claude                                                   | No       | ""        |
+| `claude_env`          | Custom environment variables to pass to Claude Code execution (YAML format)                                          | 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       | -         |
-| `label_trigger`          | The label name that triggers the action when applied to an issue (e.g. "claude")                                     | No       | -         |
-| `trigger_phrase`         | The trigger phrase to look for in comments, issue/PR bodies, and issue titles                                        | No       | `@claude` |
-| `branch_prefix`          | The prefix to use for Claude branches (defaults to 'claude/', use 'claude-' for dash format)                         | No       | `claude/` |
-| `claude_env`             | Custom environment variables to pass to Claude Code execution (YAML format)                                          | No       | ""        |
-| `additional_permissions` | Additional permissions to enable. Currently supports 'actions: read' for viewing workflow results                    | No       | ""        |
 
 \*Required when using direct Anthropic API (default and when not using Bedrock or Vertex)
 
@@ -160,40 +146,6 @@ For MCP servers that require sensitive information like API keys or tokens, use
     # ... other inputs
 ```
 
-#### Using Python MCP Servers with uv
-
-For Python-based MCP servers managed with `uv`, you need to specify the directory containing your server:
-
-```yaml
-- uses: anthropics/claude-code-action@beta
-  with:
-    anthropic_api_key: ${{ secrets.ANTHROPIC_API_KEY }}
-    mcp_config: |
-      {
-        "mcpServers": {
-          "my-python-server": {
-            "type": "stdio",
-            "command": "uv",
-            "args": [
-              "--directory",
-              "${{ github.workspace }}/path/to/server/",
-              "run",
-              "server_file.py"
-            ]
-          }
-        }
-      }
-    allowed_tools: "my-python-server__<tool_name>" # Replace <tool_name> with your server's tool names
-    # ... other inputs
-```
-
-For example, if your Python MCP server is at `mcp_servers/weather.py`, you would use:
-
-```yaml
-"args":
-  ["--directory", "${{ github.workspace }}/mcp_servers/", "run", "weather.py"]
-```
-
 **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.
@@ -343,75 +295,6 @@ This action is built on top of [`anthropics/claude-code-base-action`](https://gi
 
 ## Advanced Configuration
 
-### Additional Permissions for CI/CD Integration
-
-The `additional_permissions` input allows Claude to access GitHub Actions workflow information when you grant the necessary permissions. This is particularly useful for analyzing CI/CD failures and debugging workflow issues.
-
-#### Enabling GitHub Actions Access
-
-To allow Claude to view workflow run results, job logs, and CI status:
-
-1. **Grant the necessary permission to your GitHub token**:
-
-   - When using the default `GITHUB_TOKEN`, add the `actions: read` permission to your workflow:
-
-   ```yaml
-   permissions:
-     contents: write
-     pull-requests: write
-     issues: write
-     actions: read # Add this line
-   ```
-
-2. **Configure the action with additional permissions**:
-
-   ```yaml
-   - uses: anthropics/claude-code-action@beta
-     with:
-       anthropic_api_key: ${{ secrets.ANTHROPIC_API_KEY }}
-       additional_permissions: |
-         actions: read
-       # ... other inputs
-   ```
-
-3. **Claude will automatically get access to CI/CD tools**:
-   When you enable `actions: read`, Claude can use the following MCP tools:
-   - `mcp__github_ci__get_ci_status` - View workflow run statuses
-   - `mcp__github_ci__get_workflow_run_details` - Get detailed workflow information
-   - `mcp__github_ci__download_job_log` - Download and analyze job logs
-
-#### Example: Debugging Failed CI Runs
-
-```yaml
-name: Claude CI Helper
-on:
-  issue_comment:
-    types: [created]
-
-permissions:
-  contents: write
-  pull-requests: write
-  issues: write
-  actions: read # Required for CI access
-
-jobs:
-  claude-ci-helper:
-    runs-on: ubuntu-latest
-    steps:
-      - uses: anthropics/claude-code-action@beta
-        with:
-          anthropic_api_key: ${{ secrets.ANTHROPIC_API_KEY }}
-          additional_permissions: |
-            actions: read
-          # Now Claude can respond to "@claude why did the CI fail?"
-```
-
-**Important Notes**:
-
-- The GitHub token must have the `actions: read` permission in your workflow
-- If the permission is missing, Claude will warn you and suggest adding it
-- Currently, only `actions: read` is supported, but the format allows for future extensions
-
 ### 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:
@@ -428,24 +311,6 @@ You can pass custom environment variables to Claude Code execution using the `cl
 
 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.
 
-### Limiting Conversation Turns
-
-You can use the `max_turns` parameter to limit the number of back-and-forth exchanges Claude can have during task execution. This is useful for:
-
-- Controlling costs by preventing runaway conversations
-- Setting time boundaries for automated workflows
-- Ensuring predictable behavior in CI/CD pipelines
-
-```yaml
-- uses: anthropics/claude-code-action@beta
-  with:
-    anthropic_api_key: ${{ secrets.ANTHROPIC_API_KEY }}
-    max_turns: "5" # Limit to 5 conversation turns
-    # ... other inputs
-```
-
-When the turn limit is reached, Claude will stop execution gracefully. Choose a value that gives Claude enough turns to complete typical tasks while preventing excessive usage.
-
 ### Custom Tools
 
 By default, Claude only has access to:
@@ -461,15 +326,8 @@ Claude does **not** have access to execute arbitrary Bash commands by default. I
 ```yaml
 - uses: anthropics/claude-code-action@beta
   with:
-    allowed_tools: |
+    allowed_tools: "Bash(npm install),Bash(npm run test),Edit,Replace,NotebookEditCell"
-      Bash(npm install)
+    disallowed_tools: "TaskOutput,KillTask"
-      Bash(npm run test)
-      Edit
-      Replace
-      NotebookEditCell
-    disallowed_tools: |
-      TaskOutput
-      KillTask
     # ... other inputs
 ```
 

ROADMAP.md

@@ -1,20 +0,0 @@
-# Claude Code GitHub Action Roadmap
-
-Thank you for trying out the beta of our GitHub Action! This document outlines our path to `v1.0`. Items are not necessarily in priority order.
-
-## Path to 1.0
-
-- ~**Ability to see GitHub Action CI results** - This will enable Claude to look at CI failures and make updates to PRs to fix test failures, lint errors, and the like.~
-- **Cross-repo support** - Enable Claude to work across multiple repositories in a single session
-- **Ability to modify workflow files** - Let Claude update GitHub Actions workflows and other CI configuration files
-- **Support for workflow_dispatch and repository_dispatch events** - Dispatch Claude on events triggered via API from other workflows or from other services
-- **Ability to disable commit signing** - Option to turn off GPG signing for environments where it's not required. This will enable Claude to use normal `git` bash commands for committing. This will likely become the default behavior once added.
-- **Better code review behavior** - Support inline comments on specific lines, provide higher quality reviews with more actionable feedback
-- **Support triggering @claude from bot users** - Allow automation and bot accounts to invoke Claude
-- **Customizable base prompts** - Full control over Claude's initial context with template variables like `$PR_COMMENTS`, `$PR_FILES`, etc. Users can replace our default prompt entirely while still accessing key contextual data
-
----
-
-**Note:** This roadmap represents our current vision for reaching `v1.0` and is subject to change based on user feedback and development priorities.
-
-We welcome feedback on these planned features! If you're interested in contributing to any of these features, please open an issue to discuss implementation details with us. We're also open to suggestions for new features not listed here.

action.yml

@@ -12,17 +12,9 @@ inputs:
   assignee_trigger:
     description: "The assignee username that triggers the action (e.g. @claude)"
     required: false
-  label_trigger:
-    description: "The label that triggers the action (e.g. claude)"
-    required: false
-    default: "claude"
   base_branch:
     description: "The branch to use as the base/source when creating new branches (defaults to repository default branch)"
     required: false
-  branch_prefix:
-    description: "The prefix to use for Claude branches (defaults to 'claude/', use 'claude-' for dash format)"
-    required: false
-    default: "claude/"
 
   # Claude Code configuration
   model:
@@ -31,9 +23,6 @@ inputs:
   anthropic_model:
     description: "DEPRECATED: Use 'model' instead. Model to use (provider-specific format required for Bedrock/Vertex)"
     required: false
-  fallback_model:
-    description: "Enable automatic fallback to specified model when primary model is unavailable"
-    required: false
   allowed_tools:
     description: "Additional tools for Claude to use (the base GitHub tools will always be included)"
     required: false
@@ -52,10 +41,6 @@ inputs:
     default: ""
   mcp_config:
     description: "Additional MCP configuration (JSON string) that merges with the built-in GitHub MCP servers"
-  additional_permissions:
-    description: "Additional permissions to enable. Currently supports 'actions: read' for viewing workflow results"
-    required: false
-    default: ""
   claude_env:
     description: "Custom environment variables to pass to Claude Code execution (YAML format)"
     required: false
@@ -77,18 +62,10 @@ inputs:
     required: false
     default: "false"
 
-  max_turns:
-    description: "Maximum number of conversation turns"
-    required: false
-    default: ""
   timeout_minutes:
     description: "Timeout in minutes for execution"
     required: false
     default: "30"
-  use_sticky_comment:
-    description: "Use just one comment to deliver issue/PR comments"
-    required: false
-    default: "false"
 
 outputs:
   execution_file:
@@ -106,43 +83,35 @@ runs:
     - name: Install Dependencies
       shell: bash
       run: |
-        cd ${GITHUB_ACTION_PATH}
+        cd ${{ github.action_path }}
         bun install
 
     - name: Prepare action
       id: prepare
       shell: bash
       run: |
-        bun run ${GITHUB_ACTION_PATH}/src/entrypoints/prepare.ts
+        bun run ${{ github.action_path }}/src/entrypoints/prepare.ts
       env:
         TRIGGER_PHRASE: ${{ inputs.trigger_phrase }}
         ASSIGNEE_TRIGGER: ${{ inputs.assignee_trigger }}
-        LABEL_TRIGGER: ${{ inputs.label_trigger }}
         BASE_BRANCH: ${{ inputs.base_branch }}
-        BRANCH_PREFIX: ${{ inputs.branch_prefix }}
         ALLOWED_TOOLS: ${{ inputs.allowed_tools }}
-        DISALLOWED_TOOLS: ${{ inputs.disallowed_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 }}
-        USE_STICKY_COMMENT: ${{ inputs.use_sticky_comment }}
-        ACTIONS_TOKEN: ${{ github.token }}
-        ADDITIONAL_PERMISSIONS: ${{ inputs.additional_permissions }}
 
     - name: Run Claude Code
       id: claude-code
       if: steps.prepare.outputs.contains_trigger == 'true'
-      uses: anthropics/claude-code-base-action@a835717b36becf75584224421f4094aae288cad7 # v0.0.31
+      uses: anthropics/claude-code-base-action@1370ac97fbba9bddec20ea2924b5726bf10d8b94 # v0.0.9
       with:
-        prompt_file: ${{ runner.temp }}/claude-prompts/claude-prompt.txt
+        prompt_file: /tmp/claude-prompts/claude-prompt.txt
         allowed_tools: ${{ env.ALLOWED_TOOLS }}
         disallowed_tools: ${{ env.DISALLOWED_TOOLS }}
         timeout_minutes: ${{ inputs.timeout_minutes }}
-        max_turns: ${{ inputs.max_turns }}
         model: ${{ inputs.model || inputs.anthropic_model }}
-        fallback_model: ${{ inputs.fallback_model }}
         mcp_config: ${{ steps.prepare.outputs.mcp_config }}
         use_bedrock: ${{ inputs.use_bedrock }}
         use_vertex: ${{ inputs.use_vertex }}
@@ -178,7 +147,7 @@ runs:
       if: steps.prepare.outputs.contains_trigger == 'true' && steps.prepare.outputs.claude_comment_id && always()
       shell: bash
       run: |
-        bun run ${GITHUB_ACTION_PATH}/src/entrypoints/update-comment-link.ts
+        bun run ${{ github.action_path }}/src/entrypoints/update-comment-link.ts
       env:
         REPOSITORY: ${{ github.repository }}
         PR_NUMBER: ${{ github.event.issue.number || github.event.pull_request.number }}
@@ -195,24 +164,15 @@ runs:
         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 || '' }}
-        USE_STICKY_COMMENT: ${{ inputs.use_sticky_comment }}
 
     - name: Display Claude Code Report
       if: steps.prepare.outputs.contains_trigger == 'true' && steps.claude-code.outputs.execution_file != ''
       shell: bash
       run: |
-        # Try to format the turns, but if it fails, dump the raw JSON
+        echo "## Claude Code Report" >> $GITHUB_STEP_SUMMARY
-        if bun run ${{ github.action_path }}/src/entrypoints/format-turns.ts "${{ steps.claude-code.outputs.execution_file }}" >> $GITHUB_STEP_SUMMARY 2>/dev/null; then
+        echo '```json' >> $GITHUB_STEP_SUMMARY
-          echo "Successfully formatted Claude Code report"
+        cat "${{ steps.claude-code.outputs.execution_file }}" >> $GITHUB_STEP_SUMMARY
-        else
+        echo '```' >> $GITHUB_STEP_SUMMARY
-          echo "## Claude Code Report (Raw Output)" >> $GITHUB_STEP_SUMMARY
-          echo "" >> $GITHUB_STEP_SUMMARY
-          echo "Failed to format output (please report). Here's the raw JSON:" >> $GITHUB_STEP_SUMMARY
-          echo "" >> $GITHUB_STEP_SUMMARY
-          echo '```json' >> $GITHUB_STEP_SUMMARY
-          cat "${{ steps.claude-code.outputs.execution_file }}" >> $GITHUB_STEP_SUMMARY
-          echo '```' >> $GITHUB_STEP_SUMMARY
-        fi
 
     - name: Revoke app token
       if: always() && inputs.github_token == ''

bun.lock

@@ -2,7 +2,7 @@
   "lockfileVersion": 1,
   "workspaces": {
     "": {
-      "name": "@anthropic-ai/claude-code-action",
+      "name": "claude-pr-action",
       "dependencies": {
         "@actions/core": "^1.10.1",
         "@actions/github": "^6.0.1",

examples/claude-auto-review.yml

@@ -35,4 +35,4 @@ jobs:
 
             Provide constructive feedback with specific suggestions for improvement.
             Use inline comments to highlight specific areas of concern.
-          # allowed_tools: "mcp__github__create_pending_pull_request_review,mcp__github__add_pull_request_review_comment_to_pending_review,mcp__github__submit_pending_pull_request_review,mcp__github__get_pull_request_diff"
+          # allowed_tools: "mcp__github__add_pull_request_review_comment"

package.json

@@ -1,5 +1,5 @@
 {
-  "name": "@anthropic-ai/claude-code-action",
+  "name": "claude-pr-action",
   "version": "1.0.0",
   "private": true,
   "scripts": {

src/create-prompt/index.ts

@@ -24,7 +24,6 @@ export type { CommonFields, PreparedContext } from "./types";
 
 const BASE_ALLOWED_TOOLS = [
   "Edit",
-  "MultiEdit",
   "Glob",
   "Grep",
   "LS",
@@ -36,47 +35,38 @@ const BASE_ALLOWED_TOOLS = [
 ];
 const DISALLOWED_TOOLS = ["WebSearch", "WebFetch"];
 
-export function buildAllowedToolsString(
+export function buildAllowedToolsString(customAllowedTools?: string): string {
-  customAllowedTools?: string[],
-  includeActionsTools: boolean = false,
-): string {
   let baseTools = [...BASE_ALLOWED_TOOLS];
 
-  // Add GitHub Actions MCP tools if enabled
-  if (includeActionsTools) {
-    baseTools.push(
-      "mcp__github_ci__get_ci_status",
-      "mcp__github_ci__get_workflow_run_details",
-      "mcp__github_ci__download_job_log",
-    );
-  }
-
   let allAllowedTools = baseTools.join(",");
-  if (customAllowedTools && customAllowedTools.length > 0) {
+  if (customAllowedTools) {
-    allAllowedTools = `${allAllowedTools},${customAllowedTools.join(",")}`;
+    allAllowedTools = `${allAllowedTools},${customAllowedTools}`;
   }
   return allAllowedTools;
 }
 
 export function buildDisallowedToolsString(
-  customDisallowedTools?: string[],
+  customDisallowedTools?: string,
-  allowedTools?: string[],
+  allowedTools?: string,
 ): string {
   let disallowedTools = [...DISALLOWED_TOOLS];
 
   // If user has explicitly allowed some hardcoded disallowed tools, remove them from disallowed list
-  if (allowedTools && allowedTools.length > 0) {
+  if (allowedTools) {
+    const allowedToolsArray = allowedTools
+      .split(",")
+      .map((tool) => tool.trim());
     disallowedTools = disallowedTools.filter(
-      (tool) => !allowedTools.includes(tool),
+      (tool) => !allowedToolsArray.includes(tool),
     );
   }
 
   let allDisallowedTools = disallowedTools.join(",");
-  if (customDisallowedTools && customDisallowedTools.length > 0) {
+  if (customDisallowedTools) {
     if (allDisallowedTools) {
-      allDisallowedTools = `${allDisallowedTools},${customDisallowedTools.join(",")}`;
+      allDisallowedTools = `${allDisallowedTools},${customDisallowedTools}`;
     } else {
-      allDisallowedTools = customDisallowedTools.join(",");
+      allDisallowedTools = customDisallowedTools;
     }
   }
   return allDisallowedTools;
@@ -93,7 +83,6 @@ export function prepareContext(
   const eventAction = context.eventAction;
   const triggerPhrase = context.inputs.triggerPhrase || "@claude";
   const assigneeTrigger = context.inputs.assigneeTrigger;
-  const labelTrigger = context.inputs.labelTrigger;
   const customInstructions = context.inputs.customInstructions;
   const allowedTools = context.inputs.allowedTools;
   const disallowedTools = context.inputs.disallowedTools;
@@ -131,10 +120,8 @@ export function prepareContext(
     triggerPhrase,
     ...(triggerUsername && { triggerUsername }),
     ...(customInstructions && { customInstructions }),
-    ...(allowedTools.length > 0 && { allowedTools: allowedTools.join(",") }),
+    ...(allowedTools && { allowedTools }),
-    ...(disallowedTools.length > 0 && {
+    ...(disallowedTools && { disallowedTools }),
-      disallowedTools: disallowedTools.join(","),
-    }),
     ...(directPrompt && { directPrompt }),
     ...(claudeBranch && { claudeBranch }),
   };
@@ -255,7 +242,7 @@ export function prepareContext(
       }
 
       if (eventAction === "assigned") {
-        if (!assigneeTrigger && !directPrompt) {
+        if (!assigneeTrigger) {
           throw new Error(
             "ASSIGNEE_TRIGGER is required for issue assigned event",
           );
@@ -267,20 +254,7 @@ export function prepareContext(
           issueNumber,
           baseBranch,
           claudeBranch,
-          ...(assigneeTrigger && { assigneeTrigger }),
+          assigneeTrigger,
-        };
-      } else if (eventAction === "labeled") {
-        if (!labelTrigger) {
-          throw new Error("LABEL_TRIGGER is required for issue labeled event");
-        }
-        eventData = {
-          eventName: "issues",
-          eventAction: "labeled",
-          isPR: false,
-          issueNumber,
-          baseBranch,
-          claudeBranch,
-          labelTrigger,
         };
       } else if (eventAction === "opened") {
         eventData = {
@@ -354,17 +328,10 @@ export function getEventTypeAndContext(envVars: PreparedContext): {
           eventType: "ISSUE_CREATED",
           triggerContext: `new issue with '${envVars.triggerPhrase}' in body`,
         };
-      } else if (eventData.eventAction === "labeled") {
-        return {
-          eventType: "ISSUE_LABELED",
-          triggerContext: `issue labeled with '${eventData.labelTrigger}'`,
-        };
       }
       return {
         eventType: "ISSUE_ASSIGNED",
-        triggerContext: eventData.assigneeTrigger
+        triggerContext: `issue assigned to '${eventData.assigneeTrigger}'`,
-          ? `issue assigned to '${eventData.assigneeTrigger}'`
-          : `issue assigned event`,
       };
 
     case "pull_request":
@@ -451,7 +418,6 @@ ${
 }
 <claude_comment_id>${context.claudeCommentId}</claude_comment_id>
 <trigger_username>${context.triggerUsername ?? "Unknown"}</trigger_username>
-<trigger_display_name>${githubData.triggerDisplayName ?? context.triggerUsername ?? "Unknown"}</trigger_display_name>
 <trigger_phrase>${context.triggerPhrase}</trigger_phrase>
 ${
   (eventData.eventName === "issue_comment" ||
@@ -498,7 +464,6 @@ Follow these steps:
    - Analyze the pre-fetched data provided above.
    - For ISSUE_CREATED: Read the issue body to find the request after the trigger phrase.
    - For ISSUE_ASSIGNED: Read the entire issue body to understand the task.
-   - For ISSUE_LABELED: Read the entire issue body to understand the task.
 ${eventData.eventName === "issue_comment" || eventData.eventName === "pull_request_review_comment" || eventData.eventName === "pull_request_review" ? `   - For comment/review events: Your instructions are in the <trigger_comment> tag above.` : ""}
 ${context.directPrompt ? `   - DIRECT INSTRUCTION: A direct instruction was provided and is shown in the <direct_prompt> tag above. This is not from any GitHub comment but a direct instruction to execute.` : ""}
    - IMPORTANT: Only the comment/issue containing '${context.triggerPhrase}' has your instructions.
@@ -538,14 +503,12 @@ ${context.directPrompt ? `   - DIRECT INSTRUCTION: A direct instruction was prov
           ? `
       - Push directly using mcp__github_file_ops__commit_files to the existing branch (works for both new and existing files).
       - Use mcp__github_file_ops__commit_files to commit files atomically in a single commit (supports single or multiple files).
-      - When pushing changes with this tool and the trigger user is not "Unknown", include a Co-authored-by trailer in the commit message.
+      - When pushing changes with this tool and TRIGGER_USERNAME is not "Unknown", include a "Co-authored-by: ${context.triggerUsername} <${context.triggerUsername}@users.noreply.github.com>" line in the commit message.`
-      - Use: "Co-authored-by: ${githubData.triggerDisplayName ?? context.triggerUsername} <${context.triggerUsername}@users.noreply.github.com>"`
           : `
       - You are already on the correct branch (${eventData.claudeBranch || "the PR branch"}). Do not create a new branch.
       - Push changes directly to the current branch using mcp__github_file_ops__commit_files (works for both new and existing files)
       - Use mcp__github_file_ops__commit_files to commit files atomically in a single commit (supports single or multiple files).
-      - When pushing changes and the trigger user is not "Unknown", include a Co-authored-by trailer in the commit message.
+      - When pushing changes and TRIGGER_USERNAME is not "Unknown", include a "Co-authored-by: ${context.triggerUsername} <${context.triggerUsername}@users.noreply.github.com>" line in the commit message.
-      - Use: "Co-authored-by: ${githubData.triggerDisplayName ?? context.triggerUsername} <${context.triggerUsername}@users.noreply.github.com>"
       ${
         eventData.claudeBranch
           ? `- Provide a URL to create a PR manually in this format:
@@ -658,9 +621,7 @@ export async function createPrompt(
       claudeBranch,
     );
 
-    await mkdir(`${process.env.RUNNER_TEMP}/claude-prompts`, {
+    await mkdir("/tmp/claude-prompts", { recursive: true });
-      recursive: true,
-    });
 
     // Generate the prompt
     const promptContent = generatePrompt(preparedContext, githubData);
@@ -671,22 +632,15 @@ export async function createPrompt(
     console.log("=======================");
 
     // Write the prompt file
-    await writeFile(
+    await writeFile("/tmp/claude-prompts/claude-prompt.txt", promptContent);
-      `${process.env.RUNNER_TEMP}/claude-prompts/claude-prompt.txt`,
-      promptContent,
-    );
 
     // Set allowed tools
-    const hasActionsReadPermission =
-      context.inputs.additionalPermissions.get("actions") === "read" &&
-      context.isPR;
     const allAllowedTools = buildAllowedToolsString(
-      context.inputs.allowedTools,
+      preparedContext.allowedTools,
-      hasActionsReadPermission,
     );
     const allDisallowedTools = buildDisallowedToolsString(
-      context.inputs.disallowedTools,
+      preparedContext.disallowedTools,
-      context.inputs.allowedTools,
+      preparedContext.allowedTools,
     );
 
     core.exportVariable("ALLOWED_TOOLS", allAllowedTools);

src/create-prompt/types.ts

@@ -65,17 +65,7 @@ type IssueAssignedEvent = {
   issueNumber: string;
   baseBranch: string;
   claudeBranch: string;
-  assigneeTrigger?: string;
+  assigneeTrigger: string;
-};
-
-type IssueLabeledEvent = {
-  eventName: "issues";
-  eventAction: "labeled";
-  isPR: false;
-  issueNumber: string;
-  baseBranch: string;
-  claudeBranch: string;
-  labelTrigger: string;
 };
 
 type PullRequestEvent = {
@@ -95,7 +85,6 @@ export type EventData =
   | IssueCommentEvent
   | IssueOpenedEvent
   | IssueAssignedEvent
-  | IssueLabeledEvent
   | PullRequestEvent;
 
 // Combined type with separate eventData field

src/entrypoints/format-turns.ts

@@ -1,461 +0,0 @@
-#!/usr/bin/env bun
-
-import { readFileSync, existsSync } from "fs";
-import { exit } from "process";
-
-export interface ToolUse {
-  type: string;
-  name?: string;
-  input?: Record<string, any>;
-  id?: string;
-}
-
-export interface ToolResult {
-  type: string;
-  tool_use_id?: string;
-  content?: any;
-  is_error?: boolean;
-}
-
-export interface ContentItem {
-  type: string;
-  text?: string;
-  tool_use_id?: string;
-  content?: any;
-  is_error?: boolean;
-  name?: string;
-  input?: Record<string, any>;
-  id?: string;
-}
-
-export interface Message {
-  content: ContentItem[];
-  usage?: {
-    input_tokens?: number;
-    output_tokens?: number;
-  };
-}
-
-export interface Turn {
-  type: string;
-  subtype?: string;
-  message?: Message;
-  tools?: any[];
-  cost_usd?: number;
-  duration_ms?: number;
-  result?: string;
-}
-
-export interface GroupedContent {
-  type: string;
-  tools_count?: number;
-  data?: Turn;
-  text_parts?: string[];
-  tool_calls?: { tool_use: ToolUse; tool_result?: ToolResult }[];
-  usage?: Record<string, number>;
-}
-
-export function detectContentType(content: any): string {
-  const contentStr = String(content).trim();
-
-  // Check for JSON
-  if (contentStr.startsWith("{") && contentStr.endsWith("}")) {
-    try {
-      JSON.parse(contentStr);
-      return "json";
-    } catch {
-      // Fall through
-    }
-  }
-
-  if (contentStr.startsWith("[") && contentStr.endsWith("]")) {
-    try {
-      JSON.parse(contentStr);
-      return "json";
-    } catch {
-      // Fall through
-    }
-  }
-
-  // Check for code-like content
-  const codeKeywords = [
-    "def ",
-    "class ",
-    "import ",
-    "from ",
-    "function ",
-    "const ",
-    "let ",
-    "var ",
-  ];
-  if (codeKeywords.some((keyword) => contentStr.includes(keyword))) {
-    if (
-      contentStr.includes("def ") ||
-      contentStr.includes("import ") ||
-      contentStr.includes("from ")
-    ) {
-      return "python";
-    } else if (
-      ["function ", "const ", "let ", "var ", "=>"].some((js) =>
-        contentStr.includes(js),
-      )
-    ) {
-      return "javascript";
-    } else {
-      return "python"; // default for code
-    }
-  }
-
-  // Check for shell/bash output
-  const shellIndicators = ["ls -", "cd ", "mkdir ", "rm ", "$ ", "# "];
-  if (
-    contentStr.startsWith("/") ||
-    contentStr.includes("Error:") ||
-    contentStr.startsWith("total ") ||
-    shellIndicators.some((indicator) => contentStr.includes(indicator))
-  ) {
-    return "bash";
-  }
-
-  // Check for diff format
-  if (
-    contentStr.startsWith("@@") ||
-    contentStr.includes("+++ ") ||
-    contentStr.includes("--- ")
-  ) {
-    return "diff";
-  }
-
-  // Check for HTML/XML
-  if (contentStr.startsWith("<") && contentStr.endsWith(">")) {
-    return "html";
-  }
-
-  // Check for markdown
-  const mdIndicators = ["# ", "## ", "### ", "- ", "* ", "```"];
-  if (mdIndicators.some((indicator) => contentStr.includes(indicator))) {
-    return "markdown";
-  }
-
-  // Default to plain text
-  return "text";
-}
-
-export function formatResultContent(content: any): string {
-  if (!content) {
-    return "*(No output)*\n\n";
-  }
-
-  let contentStr: string;
-
-  // Check if content is a list with "type": "text" structure
-  try {
-    let parsedContent: any;
-    if (typeof content === "string") {
-      parsedContent = JSON.parse(content);
-    } else {
-      parsedContent = content;
-    }
-
-    if (
-      Array.isArray(parsedContent) &&
-      parsedContent.length > 0 &&
-      typeof parsedContent[0] === "object" &&
-      parsedContent[0]?.type === "text"
-    ) {
-      // Extract the text field from the first item
-      contentStr = parsedContent[0]?.text || "";
-    } else {
-      contentStr = String(content).trim();
-    }
-  } catch {
-    contentStr = String(content).trim();
-  }
-
-  // Truncate very long results
-  if (contentStr.length > 3000) {
-    contentStr = contentStr.substring(0, 2997) + "...";
-  }
-
-  // Detect content type
-  const contentType = detectContentType(contentStr);
-
-  // Handle JSON content specially - pretty print it
-  if (contentType === "json") {
-    try {
-      // Try to parse and pretty print JSON
-      const parsed = JSON.parse(contentStr);
-      contentStr = JSON.stringify(parsed, null, 2);
-    } catch {
-      // Keep original if parsing fails
-    }
-  }
-
-  // Format with appropriate syntax highlighting
-  if (
-    contentType === "text" &&
-    contentStr.length < 100 &&
-    !contentStr.includes("\n")
-  ) {
-    // Short text results don't need code blocks
-    return `**→** ${contentStr}\n\n`;
-  } else {
-    return `**Result:**\n\`\`\`${contentType}\n${contentStr}\n\`\`\`\n\n`;
-  }
-}
-
-export function formatToolWithResult(
-  toolUse: ToolUse,
-  toolResult?: ToolResult,
-): string {
-  const toolName = toolUse.name || "unknown_tool";
-  const toolInput = toolUse.input || {};
-
-  let result = `### 🔧 \`${toolName}\`\n\n`;
-
-  // Add parameters if they exist and are not empty
-  if (Object.keys(toolInput).length > 0) {
-    result += "**Parameters:**\n```json\n";
-    result += JSON.stringify(toolInput, null, 2);
-    result += "\n```\n\n";
-  }
-
-  // Add result if available
-  if (toolResult) {
-    const content = toolResult.content || "";
-    const isError = toolResult.is_error || false;
-
-    if (isError) {
-      result += `❌ **Error:** \`${content}\`\n\n`;
-    } else {
-      result += formatResultContent(content);
-    }
-  }
-
-  return result;
-}
-
-export function groupTurnsNaturally(data: Turn[]): GroupedContent[] {
-  const groupedContent: GroupedContent[] = [];
-  const toolResultsMap = new Map<string, ToolResult>();
-
-  // First pass: collect all tool results by tool_use_id
-  for (const turn of data) {
-    if (turn.type === "user") {
-      const content = turn.message?.content || [];
-      for (const item of content) {
-        if (item.type === "tool_result" && item.tool_use_id) {
-          toolResultsMap.set(item.tool_use_id, {
-            type: item.type,
-            tool_use_id: item.tool_use_id,
-            content: item.content,
-            is_error: item.is_error,
-          });
-        }
-      }
-    }
-  }
-
-  // Second pass: process turns and group naturally
-  for (const turn of data) {
-    const turnType = turn.type || "unknown";
-
-    if (turnType === "system") {
-      const subtype = turn.subtype || "";
-      if (subtype === "init") {
-        const tools = turn.tools || [];
-        groupedContent.push({
-          type: "system_init",
-          tools_count: tools.length,
-        });
-      } else {
-        groupedContent.push({
-          type: "system_other",
-          data: turn,
-        });
-      }
-    } else if (turnType === "assistant") {
-      const message = turn.message || { content: [] };
-      const content = message.content || [];
-      const usage = message.usage || {};
-
-      // Process content items
-      const textParts: string[] = [];
-      const toolCalls: { tool_use: ToolUse; tool_result?: ToolResult }[] = [];
-
-      for (const item of content) {
-        const itemType = item.type || "";
-
-        if (itemType === "text") {
-          textParts.push(item.text || "");
-        } else if (itemType === "tool_use") {
-          const toolUseId = item.id;
-          const toolResult = toolUseId
-            ? toolResultsMap.get(toolUseId)
-            : undefined;
-          toolCalls.push({
-            tool_use: {
-              type: item.type,
-              name: item.name,
-              input: item.input,
-              id: item.id,
-            },
-            tool_result: toolResult,
-          });
-        }
-      }
-
-      if (textParts.length > 0 || toolCalls.length > 0) {
-        groupedContent.push({
-          type: "assistant_action",
-          text_parts: textParts,
-          tool_calls: toolCalls,
-          usage: usage,
-        });
-      }
-    } else if (turnType === "user") {
-      // Handle user messages that aren't tool results
-      const message = turn.message || { content: [] };
-      const content = message.content || [];
-      const textParts: string[] = [];
-
-      for (const item of content) {
-        if (item.type === "text") {
-          textParts.push(item.text || "");
-        }
-      }
-
-      if (textParts.length > 0) {
-        groupedContent.push({
-          type: "user_message",
-          text_parts: textParts,
-        });
-      }
-    } else if (turnType === "result") {
-      groupedContent.push({
-        type: "final_result",
-        data: turn,
-      });
-    }
-  }
-
-  return groupedContent;
-}
-
-export function formatGroupedContent(groupedContent: GroupedContent[]): string {
-  let markdown = "## Claude Code Report\n\n";
-
-  for (const item of groupedContent) {
-    const itemType = item.type;
-
-    if (itemType === "system_init") {
-      markdown += `## 🚀 System Initialization\n\n**Available Tools:** ${item.tools_count} tools loaded\n\n---\n\n`;
-    } else if (itemType === "system_other") {
-      markdown += `## ⚙️ System Message\n\n${JSON.stringify(item.data, null, 2)}\n\n---\n\n`;
-    } else if (itemType === "assistant_action") {
-      // Add text content first (if any) - no header needed
-      for (const text of item.text_parts || []) {
-        if (text.trim()) {
-          markdown += `${text}\n\n`;
-        }
-      }
-
-      // Add tool calls with their results
-      for (const toolCall of item.tool_calls || []) {
-        markdown += formatToolWithResult(
-          toolCall.tool_use,
-          toolCall.tool_result,
-        );
-      }
-
-      // Add usage info if available
-      const usage = item.usage || {};
-      if (Object.keys(usage).length > 0) {
-        const inputTokens = usage.input_tokens || 0;
-        const outputTokens = usage.output_tokens || 0;
-        markdown += `*Token usage: ${inputTokens} input, ${outputTokens} output*\n\n`;
-      }
-
-      // Only add separator if this section had content
-      if (
-        (item.text_parts && item.text_parts.length > 0) ||
-        (item.tool_calls && item.tool_calls.length > 0)
-      ) {
-        markdown += "---\n\n";
-      }
-    } else if (itemType === "user_message") {
-      markdown += "## 👤 User\n\n";
-      for (const text of item.text_parts || []) {
-        if (text.trim()) {
-          markdown += `${text}\n\n`;
-        }
-      }
-      markdown += "---\n\n";
-    } else if (itemType === "final_result") {
-      const data = item.data || {};
-      const cost = (data as any).cost_usd || 0;
-      const duration = (data as any).duration_ms || 0;
-      const resultText = (data as any).result || "";
-
-      markdown += "## ✅ Final Result\n\n";
-      if (resultText) {
-        markdown += `${resultText}\n\n`;
-      }
-      markdown += `**Cost:** $${cost.toFixed(4)} | **Duration:** ${(duration / 1000).toFixed(1)}s\n\n`;
-    }
-  }
-
-  return markdown;
-}
-
-export function formatTurnsFromData(data: Turn[]): string {
-  // Group turns naturally
-  const groupedContent = groupTurnsNaturally(data);
-
-  // Generate markdown
-  const markdown = formatGroupedContent(groupedContent);
-
-  return markdown;
-}
-
-function main(): void {
-  // Get the JSON file path from command line arguments
-  const args = process.argv.slice(2);
-  if (args.length === 0) {
-    console.error("Usage: format-turns.ts <json-file>");
-    exit(1);
-  }
-
-  const jsonFile = args[0];
-  if (!jsonFile) {
-    console.error("Error: No JSON file provided");
-    exit(1);
-  }
-
-  if (!existsSync(jsonFile)) {
-    console.error(`Error: ${jsonFile} not found`);
-    exit(1);
-  }
-
-  try {
-    // Read the JSON file
-    const fileContent = readFileSync(jsonFile, "utf-8");
-    const data: Turn[] = JSON.parse(fileContent);
-
-    // Group turns naturally
-    const groupedContent = groupTurnsNaturally(data);
-
-    // Generate markdown
-    const markdown = formatGroupedContent(groupedContent);
-
-    // Print to stdout (so it can be captured by shell)
-    console.log(markdown);
-  } catch (error) {
-    console.error(`Error processing file: ${error}`);
-    exit(1);
-  }
-}
-
-if (import.meta.main) {
-  main();
-}

src/entrypoints/prepare.ts

@@ -59,7 +59,6 @@ async function run() {
       repository: `${context.repository.owner}/${context.repository.repo}`,
       prNumber: context.entityNumber.toString(),
       isPR: context.isPR,
-      triggerUsername: context.actor,
     });
 
     // Step 8: Setup branch
@@ -93,8 +92,6 @@ async function run() {
       branch: branchInfo.currentBranch,
       additionalMcpConfig,
       claudeCommentId: commentId.toString(),
-      allowedTools: context.inputs.allowedTools,
-      context,
     });
     core.setOutput("mcp_config", mcpConfig);
   } catch (error) {

src/github/api/queries/github.ts

@@ -104,11 +104,3 @@ export const ISSUE_QUERY = `
     }
   }
 `;
-
-export const USER_QUERY = `
-  query($login: String!) {
-    user(login: $login) {
-      name
-    }
-  }
-`;

src/github/context.ts

@@ -1,7 +1,6 @@
 import * as github from "@actions/github";
 import type {
   IssuesEvent,
-  IssuesAssignedEvent,
   IssueCommentEvent,
   PullRequestEvent,
   PullRequestReviewEvent,
@@ -29,15 +28,11 @@ export type ParsedGitHubContext = {
   inputs: {
     triggerPhrase: string;
     assigneeTrigger: string;
-    labelTrigger: string;
+    allowedTools: string;
-    allowedTools: string[];
+    disallowedTools: string;
-    disallowedTools: string[];
     customInstructions: string;
     directPrompt: string;
     baseBranch?: string;
-    branchPrefix: string;
-    useStickyComment: boolean;
-    additionalPermissions: Map<string, string>;
   };
 };
 
@@ -57,17 +52,11 @@ export function parseGitHubContext(): ParsedGitHubContext {
     inputs: {
       triggerPhrase: process.env.TRIGGER_PHRASE ?? "@claude",
       assigneeTrigger: process.env.ASSIGNEE_TRIGGER ?? "",
-      labelTrigger: process.env.LABEL_TRIGGER ?? "",
+      allowedTools: process.env.ALLOWED_TOOLS ?? "",
-      allowedTools: parseMultilineInput(process.env.ALLOWED_TOOLS ?? ""),
+      disallowedTools: process.env.DISALLOWED_TOOLS ?? "",
-      disallowedTools: parseMultilineInput(process.env.DISALLOWED_TOOLS ?? ""),
       customInstructions: process.env.CUSTOM_INSTRUCTIONS ?? "",
       directPrompt: process.env.DIRECT_PROMPT ?? "",
       baseBranch: process.env.BASE_BRANCH,
-      branchPrefix: process.env.BRANCH_PREFIX ?? "claude/",
-      useStickyComment: process.env.USE_STICKY_COMMENT === "true",
-      additionalPermissions: parseAdditionalPermissions(
-        process.env.ADDITIONAL_PERMISSIONS ?? "",
-      ),
     },
   };
 
@@ -121,33 +110,6 @@ export function parseGitHubContext(): ParsedGitHubContext {
   }
 }
 
-export function parseMultilineInput(s: string): string[] {
-  return s
-    .split(/,|[\n\r]+/)
-    .map((tool) => tool.replace(/#.+$/, ""))
-    .map((tool) => tool.trim())
-    .filter((tool) => tool.length > 0);
-}
-
-export function parseAdditionalPermissions(s: string): Map<string, string> {
-  const permissions = new Map<string, string>();
-  if (!s || !s.trim()) {
-    return permissions;
-  }
-
-  const lines = s.trim().split("\n");
-  for (const line of lines) {
-    const trimmedLine = line.trim();
-    if (trimmedLine) {
-      const [key, value] = trimmedLine.split(":").map((part) => part.trim());
-      if (key && value) {
-        permissions.set(key, value);
-      }
-    }
-  }
-  return permissions;
-}
-
 export function isIssuesEvent(
   context: ParsedGitHubContext,
 ): context is ParsedGitHubContext & { payload: IssuesEvent } {
@@ -177,9 +139,3 @@ export function isPullRequestReviewCommentEvent(
 ): context is ParsedGitHubContext & { payload: PullRequestReviewCommentEvent } {
   return context.eventName === "pull_request_review_comment";
 }
-
-export function isIssuesAssignedEvent(
-  context: ParsedGitHubContext,
-): context is ParsedGitHubContext & { payload: IssuesAssignedEvent } {
-  return isIssuesEvent(context) && context.eventAction === "assigned";
-}

src/github/data/fetcher.ts

@@ -1,24 +1,23 @@
 import { execSync } from "child_process";
-import type { Octokits } from "../api/client";
-import { ISSUE_QUERY, PR_QUERY, USER_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;
   repository: string;
   prNumber: string;
   isPR: boolean;
-  triggerUsername?: string;
 };
 
 export type GitHubFileWithSHA = GitHubFile & {
@@ -32,7 +31,6 @@ export type FetchDataResult = {
   changedFilesWithSHA: GitHubFileWithSHA[];
   reviewData: { nodes: GitHubReview[] } | null;
   imageUrlMap: Map<string, string>;
-  triggerDisplayName?: string | null;
 };
 
 export async function fetchGitHubData({
@@ -40,7 +38,6 @@ export async function fetchGitHubData({
   repository,
   prNumber,
   isPR,
-  triggerUsername,
 }: FetchDataParams): Promise<FetchDataResult> {
   const [owner, repo] = repository.split("/");
   if (!owner || !repo) {
@@ -104,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}"`, {
@@ -194,12 +183,6 @@ export async function fetchGitHubData({
     allComments,
   );
 
-  // Fetch trigger user display name if username is provided
-  let triggerDisplayName: string | null | undefined;
-  if (triggerUsername) {
-    triggerDisplayName = await fetchUserDisplayName(octokits, triggerUsername);
-  }
-
   return {
     contextData,
     comments,
@@ -207,27 +190,5 @@ export async function fetchGitHubData({
     changedFilesWithSHA,
     reviewData,
     imageUrlMap,
-    triggerDisplayName,
   };
 }
-
-export type UserQueryResponse = {
-  user: {
-    name: string | null;
-  };
-};
-
-export async function fetchUserDisplayName(
-  octokits: Octokits,
-  login: string,
-): Promise<string | null> {
-  try {
-    const result = await octokits.graphql<UserQueryResponse>(USER_QUERY, {
-      login,
-    });
-    return result.user.name;
-  } catch (error) {
-    console.warn(`Failed to fetch user display name for ${login}:`, error);
-    return null;
-  }
-}

src/github/operations/branch.ts

@@ -26,7 +26,7 @@ export async function setupBranch(
 ): Promise<BranchInfo> {
   const { owner, repo } = context.repository;
   const entityNumber = context.entityNumber;
-  const { baseBranch, branchPrefix } = context.inputs;
+  const { baseBranch } = context.inputs;
   const isPR = context.isPR;
 
   if (isPR) {
@@ -45,16 +45,9 @@ export async function setupBranch(
 
       const branchName = prData.headRefName;
 
-      // Determine optimal fetch depth based on PR commit count, with a minimum of 20
+      // Execute git commands to checkout PR branch (shallow fetch for performance)
-      const commitCount = prData.commits.totalCount;
+      // Fetch the branch with a depth of 20 to avoid fetching too much history, while still allowing for some context
-      const fetchDepth = Math.max(commitCount, 20);
+      await $`git fetch origin --depth=20 ${branchName}`;
-
-      console.log(
-        `PR #${entityNumber}: ${commitCount} commits, using fetch depth ${fetchDepth}`,
-      );
-
-      // Execute git commands to checkout PR branch (dynamic depth based on PR size)
-      await $`git fetch origin --depth=${fetchDepth} ${branchName}`;
       await $`git checkout ${branchName}`;
 
       console.log(`Successfully checked out PR branch for PR #${entityNumber}`);
@@ -97,7 +90,7 @@ export async function setupBranch(
     .split("T")
     .join("_");
 
-  const newBranch = `${branchPrefix}${entityType}-${entityNumber}-${timestamp}`;
+  const newBranch = `claude/${entityType}-${entityNumber}-${timestamp}`;
 
   try {
     // Get the SHA of the source branch

src/github/operations/comments/create-initial.ts

@@ -9,7 +9,6 @@ import { appendFileSync } from "fs";
 import { createJobRunLink, createCommentBody } from "./common";
 import {
   isPullRequestReviewCommentEvent,
-  isPullRequestEvent,
   type ParsedGitHubContext,
 } from "../../context";
 import type { Octokit } from "@octokit/rest";
@@ -26,44 +25,8 @@ export async function createInitialComment(
   try {
     let response;
 
-    if (
+    // Only use createReplyForReviewComment if it's a PR review comment AND we have a comment_id
-      context.inputs.useStickyComment &&
+    if (isPullRequestReviewCommentEvent(context)) {
-      context.isPR &&
-      isPullRequestEvent(context)
-    ) {
-      const comments = await octokit.rest.issues.listComments({
-        owner,
-        repo,
-        issue_number: context.entityNumber,
-      });
-      console.log("users");
-      comments.data.forEach((comment) => {
-        console.log(comment.user);
-      });
-
-      const existingComment = comments.data.find(
-        (comment) =>
-          comment.user?.login.indexOf("claude[bot]") !== -1 ||
-          comment.body === initialBody,
-      );
-      if (existingComment) {
-        response = await octokit.rest.issues.updateComment({
-          owner,
-          repo,
-          comment_id: existingComment.id,
-          body: initialBody,
-        });
-      } else {
-        // Create new comment if no existing one found
-        response = await octokit.rest.issues.createComment({
-          owner,
-          repo,
-          issue_number: context.entityNumber,
-          body: initialBody,
-        });
-      }
-    } else if (isPullRequestReviewCommentEvent(context)) {
-      // Only use createReplyForReviewComment if it's a PR review comment AND we have a comment_id
       response = await octokit.rest.pulls.createReplyForReviewComment({
         owner,
         repo,

src/github/token.ts

@@ -1,7 +1,47 @@
 #!/usr/bin/env bun
 
 import * as core from "@actions/core";
-import { retryWithBackoff } from "../utils/retry";
+
+type RetryOptions = {
+  maxAttempts?: number;
+  initialDelayMs?: number;
+  maxDelayMs?: number;
+  backoffFactor?: number;
+};
+
+async function retryWithBackoff<T>(
+  operation: () => Promise<T>,
+  options: RetryOptions = {},
+): Promise<T> {
+  const {
+    maxAttempts = 3,
+    initialDelayMs = 5000,
+    maxDelayMs = 20000,
+    backoffFactor = 2,
+  } = options;
+
+  let delayMs = initialDelayMs;
+  let lastError: Error | undefined;
+
+  for (let attempt = 1; attempt <= maxAttempts; attempt++) {
+    try {
+      console.log(`Attempt ${attempt} of ${maxAttempts}...`);
+      return await operation();
+    } catch (error) {
+      lastError = error instanceof Error ? error : new Error(String(error));
+      console.error(`Attempt ${attempt} failed:`, lastError.message);
+
+      if (attempt < maxAttempts) {
+        console.log(`Retrying in ${delayMs / 1000} seconds...`);
+        await new Promise((resolve) => setTimeout(resolve, delayMs));
+        delayMs = Math.min(delayMs * backoffFactor, maxDelayMs);
+      }
+    }
+  }
+
+  console.error(`Operation failed after ${maxAttempts} attempts`);
+  throw lastError;
+}
 
 async function getOidcToken(): Promise<string> {
   try {

src/github/types.ts

@@ -1,7 +1,6 @@
 // Types for GitHub GraphQL query responses
 export type GitHubAuthor = {
   login: string;
-  name?: string;
 };
 
 export type GitHubComment = {

src/github/validation/trigger.ts

@@ -3,7 +3,6 @@
 import * as core from "@actions/core";
 import {
   isIssuesEvent,
-  isIssuesAssignedEvent,
   isIssueCommentEvent,
   isPullRequestEvent,
   isPullRequestReviewEvent,
@@ -13,7 +12,7 @@ import type { ParsedGitHubContext } from "../context";
 
 export function checkContainsTrigger(context: ParsedGitHubContext): boolean {
   const {
-    inputs: { assigneeTrigger, labelTrigger, triggerPhrase, directPrompt },
+    inputs: { assigneeTrigger, triggerPhrase, directPrompt },
   } = context;
 
   // If direct prompt is provided, always trigger
@@ -23,10 +22,10 @@ export function checkContainsTrigger(context: ParsedGitHubContext): boolean {
   }
 
   // Check for assignee trigger
-  if (isIssuesAssignedEvent(context)) {
+  if (isIssuesEvent(context) && context.eventAction === "assigned") {
     // Remove @ symbol from assignee_trigger if present
     let triggerUser = assigneeTrigger.replace(/^@/, "");
-    const assigneeUsername = context.payload.assignee?.login || "";
+    const assigneeUsername = context.payload.issue.assignee?.login || "";
 
     if (triggerUser && assigneeUsername === triggerUser) {
       console.log(`Issue assigned to trigger user '${triggerUser}'`);
@@ -34,16 +33,6 @@ export function checkContainsTrigger(context: ParsedGitHubContext): boolean {
     }
   }
 
-  // Check for label trigger
-  if (isIssuesEvent(context) && context.eventAction === "labeled") {
-    const labelName = (context.payload as any).label?.name || "";
-
-    if (labelTrigger && labelName === labelTrigger) {
-      console.log(`Issue labeled with trigger label '${labelTrigger}'`);
-      return true;
-    }
-  }
-
   // Check for issue body and title trigger on issue creation
   if (isIssuesEvent(context) && context.eventAction === "opened") {
     const issueBody = context.payload.issue.body || "";

src/mcp/github-actions-server.ts

@@ -1,275 +0,0 @@
-#!/usr/bin/env node
-
-import { McpServer } from "@modelcontextprotocol/sdk/server/mcp.js";
-import { StdioServerTransport } from "@modelcontextprotocol/sdk/server/stdio.js";
-import { z } from "zod";
-import { mkdir, writeFile } from "fs/promises";
-import { Octokit } from "@octokit/rest";
-
-const REPO_OWNER = process.env.REPO_OWNER;
-const REPO_NAME = process.env.REPO_NAME;
-const PR_NUMBER = process.env.PR_NUMBER;
-const GITHUB_TOKEN = process.env.GITHUB_TOKEN;
-const RUNNER_TEMP = process.env.RUNNER_TEMP || "/tmp";
-
-if (!REPO_OWNER || !REPO_NAME || !PR_NUMBER || !GITHUB_TOKEN) {
-  console.error(
-    "[GitHub CI Server] Error: REPO_OWNER, REPO_NAME, PR_NUMBER, and GITHUB_TOKEN environment variables are required",
-  );
-  process.exit(1);
-}
-
-const server = new McpServer({
-  name: "GitHub CI Server",
-  version: "0.0.1",
-});
-
-console.error("[GitHub CI Server] MCP Server instance created");
-
-server.tool(
-  "get_ci_status",
-  "Get CI status summary for this PR",
-  {
-    status: z
-      .enum([
-        "completed",
-        "action_required",
-        "cancelled",
-        "failure",
-        "neutral",
-        "skipped",
-        "stale",
-        "success",
-        "timed_out",
-        "in_progress",
-        "queued",
-        "requested",
-        "waiting",
-        "pending",
-      ])
-      .optional()
-      .describe("Filter workflow runs by status"),
-  },
-  async ({ status }) => {
-    try {
-      const client = new Octokit({
-        auth: GITHUB_TOKEN,
-      });
-
-      // Get the PR to find the head SHA
-      const { data: prData } = await client.pulls.get({
-        owner: REPO_OWNER!,
-        repo: REPO_NAME!,
-        pull_number: parseInt(PR_NUMBER!, 10),
-      });
-      const headSha = prData.head.sha;
-
-      const { data: runsData } = await client.actions.listWorkflowRunsForRepo({
-        owner: REPO_OWNER!,
-        repo: REPO_NAME!,
-        head_sha: headSha,
-        ...(status && { status }),
-      });
-
-      // Process runs to create summary
-      const runs = runsData.workflow_runs || [];
-      const summary = {
-        total_runs: runs.length,
-        failed: 0,
-        passed: 0,
-        pending: 0,
-      };
-
-      const processedRuns = runs.map((run: any) => {
-        // Update summary counts
-        if (run.status === "completed") {
-          if (run.conclusion === "success") {
-            summary.passed++;
-          } else if (run.conclusion === "failure") {
-            summary.failed++;
-          }
-        } else {
-          summary.pending++;
-        }
-
-        return {
-          id: run.id,
-          name: run.name,
-          status: run.status,
-          conclusion: run.conclusion,
-          html_url: run.html_url,
-          created_at: run.created_at,
-        };
-      });
-
-      const result = {
-        summary,
-        runs: processedRuns,
-      };
-
-      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,
-      };
-    }
-  },
-);
-
-server.tool(
-  "get_workflow_run_details",
-  "Get job and step details for a workflow run",
-  {
-    run_id: z.number().describe("The workflow run ID"),
-  },
-  async ({ run_id }) => {
-    try {
-      const client = new Octokit({
-        auth: GITHUB_TOKEN,
-      });
-
-      // Get jobs for this workflow run
-      const { data: jobsData } = await client.actions.listJobsForWorkflowRun({
-        owner: REPO_OWNER!,
-        repo: REPO_NAME!,
-        run_id,
-      });
-
-      const processedJobs = jobsData.jobs.map((job: any) => {
-        // Extract failed steps
-        const failedSteps = (job.steps || [])
-          .filter((step: any) => step.conclusion === "failure")
-          .map((step: any) => ({
-            name: step.name,
-            number: step.number,
-          }));
-
-        return {
-          id: job.id,
-          name: job.name,
-          conclusion: job.conclusion,
-          html_url: job.html_url,
-          failed_steps: failedSteps,
-        };
-      });
-
-      const result = {
-        jobs: processedJobs,
-      };
-
-      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,
-      };
-    }
-  },
-);
-
-server.tool(
-  "download_job_log",
-  "Download job logs to disk",
-  {
-    job_id: z.number().describe("The job ID"),
-  },
-  async ({ job_id }) => {
-    try {
-      const client = new Octokit({
-        auth: GITHUB_TOKEN,
-      });
-
-      const response = await client.actions.downloadJobLogsForWorkflowRun({
-        owner: REPO_OWNER!,
-        repo: REPO_NAME!,
-        job_id,
-      });
-
-      const logsText = response.data as unknown as string;
-
-      const logsDir = `${RUNNER_TEMP}/github-ci-logs`;
-      await mkdir(logsDir, { recursive: true });
-
-      const logPath = `${logsDir}/job-${job_id}.log`;
-      await writeFile(logPath, logsText, "utf-8");
-
-      const result = {
-        path: logPath,
-        size_bytes: Buffer.byteLength(logsText, "utf-8"),
-      };
-
-      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() {
-  try {
-    const transport = new StdioServerTransport();
-
-    await server.connect(transport);
-
-    process.on("exit", () => {
-      server.close();
-    });
-  } catch (error) {
-    throw error;
-  }
-}
-
-runServer().catch(() => {
-  process.exit(1);
-});

src/mcp/github-file-ops-server.ts

@@ -9,7 +9,6 @@ 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";
-import { retryWithBackoff } from "../utils/retry";
 
 type GitHubRef = {
   object: {
@@ -126,58 +125,13 @@ server.tool(
             ? filePath
             : join(REPO_DIR, filePath);
 
-          // Check if file is binary (images, etc.)
+          const content = await readFile(fullPath, "utf-8");
-          const isBinaryFile =
+          return {
-            /\.(png|jpg|jpeg|gif|webp|ico|pdf|zip|tar|gz|exe|bin|woff|woff2|ttf|eot)$/i.test(
+            path: filePath,
-              filePath,
+            mode: "100644",
-            );
+            type: "blob",
-
+            content: content,
-          if (isBinaryFile) {
+          };
-            // For binary files, create a blob first using the Blobs API
-            const binaryContent = await readFile(fullPath);
-
-            // Create blob using Blobs API (supports encoding parameter)
-            const blobUrl = `${GITHUB_API_URL}/repos/${owner}/${repo}/git/blobs`;
-            const blobResponse = await fetch(blobUrl, {
-              method: "POST",
-              headers: {
-                Accept: "application/vnd.github+json",
-                Authorization: `Bearer ${githubToken}`,
-                "X-GitHub-Api-Version": "2022-11-28",
-                "Content-Type": "application/json",
-              },
-              body: JSON.stringify({
-                content: binaryContent.toString("base64"),
-                encoding: "base64",
-              }),
-            });
-
-            if (!blobResponse.ok) {
-              const errorText = await blobResponse.text();
-              throw new Error(
-                `Failed to create blob for ${filePath}: ${blobResponse.status} - ${errorText}`,
-              );
-            }
-
-            const blobData = (await blobResponse.json()) as { sha: string };
-
-            // Return tree entry with blob SHA
-            return {
-              path: filePath,
-              mode: "100644",
-              type: "blob",
-              sha: blobData.sha,
-            };
-          } else {
-            // For text files, include content directly in tree
-            const content = await readFile(fullPath, "utf-8");
-            return {
-              path: filePath,
-              mode: "100644",
-              type: "blob",
-              content: content,
-            };
-          }
         }),
       );
 
@@ -234,50 +188,26 @@ server.tool(
 
       // 6. Update the reference to point to the new commit
       const updateRefUrl = `${GITHUB_API_URL}/repos/${owner}/${repo}/git/refs/heads/${branch}`;
-
+      const updateRefResponse = await fetch(updateRefUrl, {
-      // We're seeing intermittent 403 "Resource not accessible by integration" errors
+        method: "PATCH",
-      // on certain repos when updating git references. These appear to be transient
+        headers: {
-      // GitHub API issues that succeed on retry.
+          Accept: "application/vnd.github+json",
-      await retryWithBackoff(
+          Authorization: `Bearer ${githubToken}`,
-        async () => {
+          "X-GitHub-Api-Version": "2022-11-28",
-          const updateRefResponse = await fetch(updateRefUrl, {
+          "Content-Type": "application/json",
-            method: "PATCH",
-            headers: {
-              Accept: "application/vnd.github+json",
-              Authorization: `Bearer ${githubToken}`,
-              "X-GitHub-Api-Version": "2022-11-28",
-              "Content-Type": "application/json",
-            },
-            body: JSON.stringify({
-              sha: newCommitData.sha,
-              force: false,
-            }),
-          });
-
-          if (!updateRefResponse.ok) {
-            const errorText = await updateRefResponse.text();
-            const error = new Error(
-              `Failed to update reference: ${updateRefResponse.status} - ${errorText}`,
-            );
-
-            // Only retry on 403 errors - these are the intermittent failures we're targeting
-            if (updateRefResponse.status === 403) {
-              console.log("Received 403 error, will retry...");
-              throw error;
-            }
-
-            // For non-403 errors, fail immediately without retry
-            console.error("Non-retryable error:", updateRefResponse.status);
-            throw error;
-          }
         },
-        {
+        body: JSON.stringify({
-          maxAttempts: 3,
+          sha: newCommitData.sha,
-          initialDelayMs: 1000, // Start with 1 second delay
+          force: false,
-          maxDelayMs: 5000, // Max 5 seconds delay
+        }),
-          backoffFactor: 2, // Double the delay each time
+      });
-        },
+
-      );
+      if (!updateRefResponse.ok) {
+        const errorText = await updateRefResponse.text();
+        throw new Error(
+          `Failed to update reference: ${updateRefResponse.status} - ${errorText}`,
+        );
+      }
 
       const simplifiedResult = {
         commit: {
@@ -452,50 +382,26 @@ server.tool(
 
       // 6. Update the reference to point to the new commit
       const updateRefUrl = `${GITHUB_API_URL}/repos/${owner}/${repo}/git/refs/heads/${branch}`;
-
+      const updateRefResponse = await fetch(updateRefUrl, {
-      // We're seeing intermittent 403 "Resource not accessible by integration" errors
+        method: "PATCH",
-      // on certain repos when updating git references. These appear to be transient
+        headers: {
-      // GitHub API issues that succeed on retry.
+          Accept: "application/vnd.github+json",
-      await retryWithBackoff(
+          Authorization: `Bearer ${githubToken}`,
-        async () => {
+          "X-GitHub-Api-Version": "2022-11-28",
-          const updateRefResponse = await fetch(updateRefUrl, {
+          "Content-Type": "application/json",
-            method: "PATCH",
-            headers: {
-              Accept: "application/vnd.github+json",
-              Authorization: `Bearer ${githubToken}`,
-              "X-GitHub-Api-Version": "2022-11-28",
-              "Content-Type": "application/json",
-            },
-            body: JSON.stringify({
-              sha: newCommitData.sha,
-              force: false,
-            }),
-          });
-
-          if (!updateRefResponse.ok) {
-            const errorText = await updateRefResponse.text();
-            const error = new Error(
-              `Failed to update reference: ${updateRefResponse.status} - ${errorText}`,
-            );
-
-            // Only retry on 403 errors - these are the intermittent failures we're targeting
-            if (updateRefResponse.status === 403) {
-              console.log("Received 403 error, will retry...");
-              throw error;
-            }
-
-            // For non-403 errors, fail immediately without retry
-            console.error("Non-retryable error:", updateRefResponse.status);
-            throw error;
-          }
         },
-        {
+        body: JSON.stringify({
-          maxAttempts: 3,
+          sha: newCommitData.sha,
-          initialDelayMs: 1000, // Start with 1 second delay
+          force: false,
-          maxDelayMs: 5000, // Max 5 seconds delay
+        }),
-          backoffFactor: 2, // Double the delay each time
+      });
-        },
+
-      );
+      if (!updateRefResponse.ok) {
+        const errorText = await updateRefResponse.text();
+        throw new Error(
+          `Failed to update reference: ${updateRefResponse.status} - ${errorText}`,
+        );
+      }
 
       const simplifiedResult = {
         commit: {
@@ -560,7 +466,6 @@ server.tool(
 
       const octokit = new Octokit({
         auth: githubToken,
-        baseUrl: GITHUB_API_URL,
       });
 
       const isPullRequestReviewComment =

src/mcp/install-mcp-server.ts

@@ -1,7 +1,4 @@
 import * as core from "@actions/core";
-import { GITHUB_API_URL } from "../github/api/config";
-import type { ParsedGitHubContext } from "../github/context";
-import { Octokit } from "@octokit/rest";
 
 type PrepareConfigParams = {
   githubToken: string;
@@ -10,42 +7,8 @@ type PrepareConfigParams = {
   branch: string;
   additionalMcpConfig?: string;
   claudeCommentId?: string;
-  allowedTools: string[];
-  context: ParsedGitHubContext;
 };
 
-async function checkActionsReadPermission(
-  token: string,
-  owner: string,
-  repo: string,
-): Promise<boolean> {
-  try {
-    const client = new Octokit({ auth: token });
-
-    // Try to list workflow runs - this requires actions:read
-    // We use per_page=1 to minimize the response size
-    await client.actions.listWorkflowRunsForRepo({
-      owner,
-      repo,
-      per_page: 1,
-    });
-
-    return true;
-  } catch (error: any) {
-    // Check if it's a permission error
-    if (
-      error.status === 403 &&
-      error.message?.includes("Resource not accessible")
-    ) {
-      return false;
-    }
-
-    // For other errors (network issues, etc), log but don't fail
-    core.debug(`Failed to check actions permission: ${error.message}`);
-    return false;
-  }
-}
-
 export async function prepareMcpConfig(
   params: PrepareConfigParams,
 ): Promise<string> {
@@ -56,18 +19,24 @@ export async function prepareMcpConfig(
     branch,
     additionalMcpConfig,
     claudeCommentId,
-    allowedTools,
-    context,
   } = params;
   try {
-    const allowedToolsList = allowedTools || [];
+    const baseMcpConfig = {
-
-    const hasGitHubMcpTools = allowedToolsList.some((tool) =>
-      tool.startsWith("mcp__github__"),
-    );
-
-    const baseMcpConfig: { mcpServers: Record<string, unknown> } = {
       mcpServers: {
+        github: {
+          command: "docker",
+          args: [
+            "run",
+            "-i",
+            "--rm",
+            "-e",
+            "GITHUB_PERSONAL_ACCESS_TOKEN",
+            "ghcr.io/anthropics/github-mcp-server:sha-7382253",
+          ],
+          env: {
+            GITHUB_PERSONAL_ACCESS_TOKEN: githubToken,
+          },
+        },
         github_file_ops: {
           command: "bun",
           args: [
@@ -83,65 +52,11 @@ export async function prepareMcpConfig(
             ...(claudeCommentId && { CLAUDE_COMMENT_ID: claudeCommentId }),
             GITHUB_EVENT_NAME: process.env.GITHUB_EVENT_NAME || "",
             IS_PR: process.env.IS_PR || "false",
-            GITHUB_API_URL: GITHUB_API_URL,
           },
         },
       },
     };
 
-    // Only add CI server if we have actions:read permission and we're in a PR context
-    const hasActionsReadPermission =
-      context.inputs.additionalPermissions.get("actions") === "read";
-
-    if (context.isPR && hasActionsReadPermission) {
-      // Verify the token actually has actions:read permission
-      const actuallyHasPermission = await checkActionsReadPermission(
-        process.env.ACTIONS_TOKEN || "",
-        owner,
-        repo,
-      );
-
-      if (!actuallyHasPermission) {
-        core.warning(
-          "The github_ci MCP server requires 'actions: read' permission. " +
-            "Please ensure your GitHub token has this permission. " +
-            "See: https://docs.github.com/en/actions/security-guides/automatic-token-authentication#permissions-for-the-github_token",
-        );
-      }
-      baseMcpConfig.mcpServers.github_ci = {
-        command: "bun",
-        args: [
-          "run",
-          `${process.env.GITHUB_ACTION_PATH}/src/mcp/github-actions-server.ts`,
-        ],
-        env: {
-          // Use workflow github token, not app token
-          GITHUB_TOKEN: process.env.ACTIONS_TOKEN,
-          REPO_OWNER: owner,
-          REPO_NAME: repo,
-          PR_NUMBER: context.entityNumber.toString(),
-          RUNNER_TEMP: process.env.RUNNER_TEMP || "/tmp",
-        },
-      };
-    }
-
-    if (hasGitHubMcpTools) {
-      baseMcpConfig.mcpServers.github = {
-        command: "docker",
-        args: [
-          "run",
-          "-i",
-          "--rm",
-          "-e",
-          "GITHUB_PERSONAL_ACCESS_TOKEN",
-          "ghcr.io/github/github-mcp-server:sha-6d69797", // https://github.com/github/github-mcp-server/releases/tag/v0.5.0
-        ],
-        env: {
-          GITHUB_PERSONAL_ACCESS_TOKEN: githubToken,
-        },
-      };
-    }
-
     // Merge with additional MCP config if provided
     if (additionalMcpConfig && additionalMcpConfig.trim()) {
       try {

src/utils/retry.ts

@@ -1,40 +0,0 @@
-export type RetryOptions = {
-  maxAttempts?: number;
-  initialDelayMs?: number;
-  maxDelayMs?: number;
-  backoffFactor?: number;
-};
-
-export async function retryWithBackoff<T>(
-  operation: () => Promise<T>,
-  options: RetryOptions = {},
-): Promise<T> {
-  const {
-    maxAttempts = 3,
-    initialDelayMs = 5000,
-    maxDelayMs = 20000,
-    backoffFactor = 2,
-  } = options;
-
-  let delayMs = initialDelayMs;
-  let lastError: Error | undefined;
-
-  for (let attempt = 1; attempt <= maxAttempts; attempt++) {
-    try {
-      console.log(`Attempt ${attempt} of ${maxAttempts}...`);
-      return await operation();
-    } catch (error) {
-      lastError = error instanceof Error ? error : new Error(String(error));
-      console.error(`Attempt ${attempt} failed:`, lastError.message);
-
-      if (attempt < maxAttempts) {
-        console.log(`Retrying in ${delayMs / 1000} seconds...`);
-        await new Promise((resolve) => setTimeout(resolve, delayMs));
-        delayMs = Math.min(delayMs * backoffFactor, maxDelayMs);
-      }
-    }
-  }
-
-  console.error(`Operation failed after ${maxAttempts} attempts`);
-  throw lastError;
-}

test/create-prompt.test.ts

@@ -226,33 +226,6 @@ describe("generatePrompt", () => {
     );
   });
 
-  test("should generate prompt for issue labeled event", () => {
-    const envVars: PreparedContext = {
-      repository: "owner/repo",
-      claudeCommentId: "12345",
-      triggerPhrase: "@claude",
-      eventData: {
-        eventName: "issues",
-        eventAction: "labeled",
-        isPR: false,
-        issueNumber: "888",
-        baseBranch: "main",
-        claudeBranch: "claude/issue-888-20240101_120000",
-        labelTrigger: "claude-task",
-      },
-    };
-
-    const prompt = generatePrompt(envVars, mockGitHubData);
-
-    expect(prompt).toContain("<event_type>ISSUE_LABELED</event_type>");
-    expect(prompt).toContain(
-      "<trigger_context>issue labeled with 'claude-task'</trigger_context>",
-    );
-    expect(prompt).toContain(
-      "[Create a PR](https://github.com/owner/repo/compare/main",
-    );
-  });
-
   test("should include direct prompt when provided", () => {
     const envVars: PreparedContext = {
       repository: "owner/repo",
@@ -343,7 +316,7 @@ describe("generatePrompt", () => {
 
     expect(prompt).toContain("<trigger_username>johndoe</trigger_username>");
     expect(prompt).toContain(
-      'Use: "Co-authored-by: johndoe <johndoe@users.noreply.github.com>"',
+      "Co-authored-by: johndoe <johndoe@users.noreply.github.com>",
     );
   });
 
@@ -641,51 +614,6 @@ describe("getEventTypeAndContext", () => {
     expect(result.eventType).toBe("ISSUE_ASSIGNED");
     expect(result.triggerContext).toBe("issue assigned to 'claude-bot'");
   });
-
-  test("should return correct type and context for issue labeled", () => {
-    const envVars: PreparedContext = {
-      repository: "owner/repo",
-      claudeCommentId: "12345",
-      triggerPhrase: "@claude",
-      eventData: {
-        eventName: "issues",
-        eventAction: "labeled",
-        isPR: false,
-        issueNumber: "888",
-        baseBranch: "main",
-        claudeBranch: "claude/issue-888-20240101_120000",
-        labelTrigger: "claude-task",
-      },
-    };
-
-    const result = getEventTypeAndContext(envVars);
-
-    expect(result.eventType).toBe("ISSUE_LABELED");
-    expect(result.triggerContext).toBe("issue labeled with 'claude-task'");
-  });
-
-  test("should return correct type and context for issue assigned without assigneeTrigger", () => {
-    const envVars: PreparedContext = {
-      repository: "owner/repo",
-      claudeCommentId: "12345",
-      triggerPhrase: "@claude",
-      directPrompt: "Please assess this issue",
-      eventData: {
-        eventName: "issues",
-        eventAction: "assigned",
-        isPR: false,
-        issueNumber: "999",
-        baseBranch: "main",
-        claudeBranch: "claude/issue-999-20240101_120000",
-        // No assigneeTrigger when using directPrompt
-      },
-    };
-
-    const result = getEventTypeAndContext(envVars);
-
-    expect(result.eventType).toBe("ISSUE_ASSIGNED");
-    expect(result.triggerContext).toBe("issue assigned event");
-  });
 });
 
 describe("buildAllowedToolsString", () => {
@@ -724,7 +652,7 @@ describe("buildAllowedToolsString", () => {
   });
 
   test("should append custom tools when provided", () => {
-    const customTools = ["Tool1", "Tool2", "Tool3"];
+    const customTools = "Tool1,Tool2,Tool3";
     const result = buildAllowedToolsString(customTools);
 
     // Base tools should be present
@@ -743,36 +671,6 @@ describe("buildAllowedToolsString", () => {
     expect(basePlusCustom).toContain("Tool2");
     expect(basePlusCustom).toContain("Tool3");
   });
-
-  test("should include GitHub Actions tools when includeActionsTools is true", () => {
-    const result = buildAllowedToolsString([], true);
-
-    // Base tools should be present
-    expect(result).toContain("Edit");
-    expect(result).toContain("Glob");
-
-    // GitHub Actions tools should be included
-    expect(result).toContain("mcp__github_ci__get_ci_status");
-    expect(result).toContain("mcp__github_ci__get_workflow_run_details");
-    expect(result).toContain("mcp__github_ci__download_job_log");
-  });
-
-  test("should include both custom and Actions tools when both provided", () => {
-    const customTools = ["Tool1", "Tool2"];
-    const result = buildAllowedToolsString(customTools, true);
-
-    // Base tools should be present
-    expect(result).toContain("Edit");
-
-    // Custom tools should be included
-    expect(result).toContain("Tool1");
-    expect(result).toContain("Tool2");
-
-    // GitHub Actions tools should be included
-    expect(result).toContain("mcp__github_ci__get_ci_status");
-    expect(result).toContain("mcp__github_ci__get_workflow_run_details");
-    expect(result).toContain("mcp__github_ci__download_job_log");
-  });
 });
 
 describe("buildDisallowedToolsString", () => {
@@ -785,7 +683,7 @@ describe("buildDisallowedToolsString", () => {
   });
 
   test("should append custom disallowed tools when provided", () => {
-    const customDisallowedTools = ["BadTool1", "BadTool2"];
+    const customDisallowedTools = "BadTool1,BadTool2";
     const result = buildDisallowedToolsString(customDisallowedTools);
 
     // Base disallowed tools should be present
@@ -803,8 +701,8 @@ describe("buildDisallowedToolsString", () => {
   });
 
   test("should remove hardcoded disallowed tools if they are in allowed tools", () => {
-    const customDisallowedTools = ["BadTool1", "BadTool2"];
+    const customDisallowedTools = "BadTool1,BadTool2";
-    const allowedTools = ["WebSearch", "SomeOtherTool"];
+    const allowedTools = "WebSearch,SomeOtherTool";
     const result = buildDisallowedToolsString(
       customDisallowedTools,
       allowedTools,
@@ -822,7 +720,7 @@ describe("buildDisallowedToolsString", () => {
   });
 
   test("should remove all hardcoded disallowed tools if they are all in allowed tools", () => {
-    const allowedTools = ["WebSearch", "WebFetch", "SomeOtherTool"];
+    const allowedTools = "WebSearch,WebFetch,SomeOtherTool";
     const result = buildDisallowedToolsString(undefined, allowedTools);
 
     // Both hardcoded disallowed tools should be removed
@@ -834,8 +732,8 @@ describe("buildDisallowedToolsString", () => {
   });
 
   test("should handle custom disallowed tools when all hardcoded tools are overridden", () => {
-    const customDisallowedTools = ["BadTool1", "BadTool2"];
+    const customDisallowedTools = "BadTool1,BadTool2";
-    const allowedTools = ["WebSearch", "WebFetch"];
+    const allowedTools = "WebSearch,WebFetch";
     const result = buildDisallowedToolsString(
       customDisallowedTools,
       allowedTools,

test/fixtures/sample-turns-expected-output.md

@@ -1,95 +0,0 @@
-## Claude Code Report
-
-## 🚀 System Initialization
-
-**Available Tools:** 8 tools loaded
-
----
-
-I'll help you with this task. Let me start by examining the file to understand what needs to be changed.
-
-### 🔧 `Read`
-
-**Parameters:**
-```json
-{
-  "file_path": "/path/to/sample/file.py"
-}
-```
-
-**Result:**
-```python
-def example_function():
-    print("Debug message")  # This should be removed
-    return "Hello World"
-
-if __name__ == "__main__":
-    result = example_function()
-    print(result)
-```
-
-*Token usage: 100 input, 75 output*
-
----
-
-I can see the debug print statement that needs to be removed. Let me fix this by editing the file.
-
-### 🔧 `Edit`
-
-**Parameters:**
-```json
-{
-  "file_path": "/path/to/sample/file.py",
-  "old_string": "def example_function():\n    print(\"Debug message\")  # This should be removed\n    return \"Hello World\"",
-  "new_string": "def example_function():\n    return \"Hello World\""
-}
-```
-
-**→** File successfully edited. The debug print statement has been removed.
-
-*Token usage: 200 input, 50 output*
-
----
-
-Perfect! I've successfully removed the debug print statement from the function. Let me now create a pull request comment to document this change.
-
-### 🔧 `mcp__github__add_pull_request_review_comment`
-
-**Parameters:**
-```json
-{
-  "owner": "example-org",
-  "repo": "example-repo",
-  "pull_number": 123,
-  "body": "Removed debug print statement as requested.",
-  "commit_id": "abc123def456",
-  "path": "sample/file.py",
-  "line": 2
-}
-```
-
-**→** Successfully posted review comment to PR #123
-
-*Token usage: 150 input, 80 output*
-
----
-
-Great! I've successfully completed the requested task:
-
-1. ✅ Located the debug print statement in the file
-2. ✅ Removed the print statement while preserving the function logic
-3. ✅ Added a review comment documenting the change
-
-The debug print statement has been removed as requested by the reviewers.
-
-*Token usage: 180 input, 60 output*
-
----
-
-## ✅ Final Result
-
-Successfully removed debug print statement from file and added review comment to document the change.
-
-**Cost:** $0.0347 | **Duration:** 18.8s
-
-

test/fixtures/sample-turns.json

@@ -1,196 +0,0 @@
-[
-  {
-    "type": "system",
-    "subtype": "init",
-    "session_id": "sample-session-id",
-    "tools": [
-      "Task",
-      "Bash",
-      "Read",
-      "Edit",
-      "Write",
-      "mcp__github__get_file_contents",
-      "mcp__github__create_or_update_file",
-      "mcp__github__add_pull_request_review_comment"
-    ],
-    "mcp_servers": [
-      {
-        "name": "github",
-        "status": "connected"
-      }
-    ]
-  },
-  {
-    "type": "assistant",
-    "message": {
-      "id": "msg_sample123",
-      "type": "message",
-      "role": "assistant",
-      "model": "claude-test-model",
-      "content": [
-        {
-          "type": "text",
-          "text": "I'll help you with this task. Let me start by examining the file to understand what needs to be changed."
-        },
-        {
-          "type": "tool_use",
-          "id": "tool_call_1",
-          "name": "Read",
-          "input": {
-            "file_path": "/path/to/sample/file.py"
-          }
-        }
-      ],
-      "stop_reason": "tool_use",
-      "stop_sequence": null,
-      "usage": {
-        "input_tokens": 100,
-        "cache_creation_input_tokens": 0,
-        "cache_read_input_tokens": 50,
-        "output_tokens": 75
-      }
-    },
-    "session_id": "sample-session-id"
-  },
-  {
-    "type": "user",
-    "message": {
-      "content": [
-        {
-          "type": "tool_result",
-          "tool_use_id": "tool_call_1",
-          "content": "def example_function():\n    print(\"Debug message\")  # This should be removed\n    return \"Hello World\"\n\nif __name__ == \"__main__\":\n    result = example_function()\n    print(result)",
-          "is_error": false
-        }
-      ]
-    }
-  },
-  {
-    "type": "assistant",
-    "message": {
-      "id": "msg_sample124",
-      "type": "message",
-      "role": "assistant",
-      "model": "claude-test-model",
-      "content": [
-        {
-          "type": "text",
-          "text": "I can see the debug print statement that needs to be removed. Let me fix this by editing the file."
-        },
-        {
-          "type": "tool_use",
-          "id": "tool_call_2",
-          "name": "Edit",
-          "input": {
-            "file_path": "/path/to/sample/file.py",
-            "old_string": "def example_function():\n    print(\"Debug message\")  # This should be removed\n    return \"Hello World\"",
-            "new_string": "def example_function():\n    return \"Hello World\""
-          }
-        }
-      ],
-      "stop_reason": "tool_use",
-      "stop_sequence": null,
-      "usage": {
-        "input_tokens": 200,
-        "cache_creation_input_tokens": 0,
-        "cache_read_input_tokens": 100,
-        "output_tokens": 50
-      }
-    },
-    "session_id": "sample-session-id"
-  },
-  {
-    "type": "user",
-    "message": {
-      "content": [
-        {
-          "type": "tool_result",
-          "tool_use_id": "tool_call_2",
-          "content": "File successfully edited. The debug print statement has been removed.",
-          "is_error": false
-        }
-      ]
-    }
-  },
-  {
-    "type": "assistant",
-    "message": {
-      "id": "msg_sample125",
-      "type": "message",
-      "role": "assistant",
-      "model": "claude-test-model",
-      "content": [
-        {
-          "type": "text",
-          "text": "Perfect! I've successfully removed the debug print statement from the function. Let me now create a pull request comment to document this change."
-        },
-        {
-          "type": "tool_use",
-          "id": "tool_call_3",
-          "name": "mcp__github__add_pull_request_review_comment",
-          "input": {
-            "owner": "example-org",
-            "repo": "example-repo",
-            "pull_number": 123,
-            "body": "Removed debug print statement as requested.",
-            "commit_id": "abc123def456",
-            "path": "sample/file.py",
-            "line": 2
-          }
-        }
-      ],
-      "stop_reason": "tool_use",
-      "stop_sequence": null,
-      "usage": {
-        "input_tokens": 150,
-        "cache_creation_input_tokens": 0,
-        "cache_read_input_tokens": 75,
-        "output_tokens": 80
-      }
-    },
-    "session_id": "sample-session-id"
-  },
-  {
-    "type": "user",
-    "message": {
-      "content": [
-        {
-          "type": "tool_result",
-          "tool_use_id": "tool_call_3",
-          "content": "Successfully posted review comment to PR #123",
-          "is_error": false
-        }
-      ]
-    }
-  },
-  {
-    "type": "assistant",
-    "message": {
-      "id": "msg_sample126",
-      "type": "message",
-      "role": "assistant",
-      "model": "claude-test-model",
-      "content": [
-        {
-          "type": "text",
-          "text": "Great! I've successfully completed the requested task:\n\n1. ✅ Located the debug print statement in the file\n2. ✅ Removed the print statement while preserving the function logic\n3. ✅ Added a review comment documenting the change\n\nThe debug print statement has been removed as requested by the reviewers."
-        }
-      ],
-      "stop_reason": "end_turn",
-      "stop_sequence": null,
-      "usage": {
-        "input_tokens": 180,
-        "cache_creation_input_tokens": 0,
-        "cache_read_input_tokens": 90,
-        "output_tokens": 60
-      }
-    },
-    "session_id": "sample-session-id"
-  },
-  {
-    "type": "result",
-    "cost_usd": 0.0347,
-    "duration_ms": 18750,
-    "result": "Successfully removed debug print statement from file and added review comment to document the change."
-  }
-]

test/format-turns.test.ts

@@ -1,439 +0,0 @@
-import { expect, test, describe } from "bun:test";
-import { readFileSync } from "fs";
-import { join } from "path";
-import {
-  formatTurnsFromData,
-  groupTurnsNaturally,
-  formatGroupedContent,
-  detectContentType,
-  formatResultContent,
-  formatToolWithResult,
-  type Turn,
-  type ToolUse,
-  type ToolResult,
-} from "../src/entrypoints/format-turns";
-
-describe("detectContentType", () => {
-  test("detects JSON objects", () => {
-    expect(detectContentType('{"key": "value"}')).toBe("json");
-    expect(detectContentType('{"number": 42}')).toBe("json");
-  });
-
-  test("detects JSON arrays", () => {
-    expect(detectContentType("[1, 2, 3]")).toBe("json");
-    expect(detectContentType('["a", "b"]')).toBe("json");
-  });
-
-  test("detects Python code", () => {
-    expect(detectContentType("def hello():\n    pass")).toBe("python");
-    expect(detectContentType("import os")).toBe("python");
-    expect(detectContentType("from math import pi")).toBe("python");
-  });
-
-  test("detects JavaScript code", () => {
-    expect(detectContentType("function test() {}")).toBe("javascript");
-    expect(detectContentType("const x = 5")).toBe("javascript");
-    expect(detectContentType("let y = 10")).toBe("javascript");
-    expect(detectContentType("const fn = () => console.log()")).toBe(
-      "javascript",
-    );
-  });
-
-  test("detects bash/shell content", () => {
-    expect(detectContentType("/usr/bin/test")).toBe("bash");
-    expect(detectContentType("Error: command not found")).toBe("bash");
-    expect(detectContentType("ls -la")).toBe("bash");
-    expect(detectContentType("$ echo hello")).toBe("bash");
-  });
-
-  test("detects diff format", () => {
-    expect(detectContentType("@@ -1,3 +1,3 @@")).toBe("diff");
-    expect(detectContentType("+++ file.txt")).toBe("diff");
-    expect(detectContentType("--- file.txt")).toBe("diff");
-  });
-
-  test("detects HTML/XML", () => {
-    expect(detectContentType("<div>hello</div>")).toBe("html");
-    expect(detectContentType("<xml>content</xml>")).toBe("html");
-  });
-
-  test("detects markdown", () => {
-    expect(detectContentType("- List item")).toBe("markdown");
-    expect(detectContentType("* List item")).toBe("markdown");
-    expect(detectContentType("```code```")).toBe("markdown");
-  });
-
-  test("defaults to text", () => {
-    expect(detectContentType("plain text")).toBe("text");
-    expect(detectContentType("just some words")).toBe("text");
-  });
-});
-
-describe("formatResultContent", () => {
-  test("handles empty content", () => {
-    expect(formatResultContent("")).toBe("*(No output)*\n\n");
-    expect(formatResultContent(null)).toBe("*(No output)*\n\n");
-    expect(formatResultContent(undefined)).toBe("*(No output)*\n\n");
-  });
-
-  test("formats short text without code blocks", () => {
-    const result = formatResultContent("success");
-    expect(result).toBe("**→** success\n\n");
-  });
-
-  test("formats long text with code blocks", () => {
-    const longText =
-      "This is a longer piece of text that should be formatted in a code block because it exceeds the short text threshold";
-    const result = formatResultContent(longText);
-    expect(result).toContain("**Result:**");
-    expect(result).toContain("```text");
-    expect(result).toContain(longText);
-  });
-
-  test("pretty prints JSON content", () => {
-    const jsonContent = '{"key": "value", "number": 42}';
-    const result = formatResultContent(jsonContent);
-    expect(result).toContain("```json");
-    expect(result).toContain('"key": "value"');
-    expect(result).toContain('"number": 42');
-  });
-
-  test("truncates very long content", () => {
-    const veryLongContent = "A".repeat(4000);
-    const result = formatResultContent(veryLongContent);
-    expect(result).toContain("...");
-    // Should not contain the full long content
-    expect(result.length).toBeLessThan(veryLongContent.length);
-  });
-
-  test("handles type:text structure", () => {
-    const structuredContent = [{ type: "text", text: "Hello world" }];
-    const result = formatResultContent(JSON.stringify(structuredContent));
-    expect(result).toBe("**→** Hello world\n\n");
-  });
-});
-
-describe("formatToolWithResult", () => {
-  test("formats tool with parameters and result", () => {
-    const toolUse: ToolUse = {
-      type: "tool_use",
-      name: "read_file",
-      input: { file_path: "/path/to/file.txt" },
-      id: "tool_123",
-    };
-
-    const toolResult: ToolResult = {
-      type: "tool_result",
-      tool_use_id: "tool_123",
-      content: "File content here",
-      is_error: false,
-    };
-
-    const result = formatToolWithResult(toolUse, toolResult);
-
-    expect(result).toContain("### 🔧 `read_file`");
-    expect(result).toContain("**Parameters:**");
-    expect(result).toContain('"file_path": "/path/to/file.txt"');
-    expect(result).toContain("**→** File content here");
-  });
-
-  test("formats tool with error result", () => {
-    const toolUse: ToolUse = {
-      type: "tool_use",
-      name: "failing_tool",
-      input: { param: "value" },
-    };
-
-    const toolResult: ToolResult = {
-      type: "tool_result",
-      content: "Permission denied",
-      is_error: true,
-    };
-
-    const result = formatToolWithResult(toolUse, toolResult);
-
-    expect(result).toContain("### 🔧 `failing_tool`");
-    expect(result).toContain("❌ **Error:** `Permission denied`");
-  });
-
-  test("formats tool without parameters", () => {
-    const toolUse: ToolUse = {
-      type: "tool_use",
-      name: "simple_tool",
-    };
-
-    const result = formatToolWithResult(toolUse);
-
-    expect(result).toContain("### 🔧 `simple_tool`");
-    expect(result).not.toContain("**Parameters:**");
-  });
-
-  test("handles unknown tool name", () => {
-    const toolUse: ToolUse = {
-      type: "tool_use",
-    };
-
-    const result = formatToolWithResult(toolUse);
-
-    expect(result).toContain("### 🔧 `unknown_tool`");
-  });
-});
-
-describe("groupTurnsNaturally", () => {
-  test("groups system initialization", () => {
-    const data: Turn[] = [
-      {
-        type: "system",
-        subtype: "init",
-        tools: [{ name: "tool1" }, { name: "tool2" }],
-      },
-    ];
-
-    const result = groupTurnsNaturally(data);
-
-    expect(result).toHaveLength(1);
-    expect(result[0]?.type).toBe("system_init");
-    expect(result[0]?.tools_count).toBe(2);
-  });
-
-  test("groups assistant actions with tool calls", () => {
-    const data: Turn[] = [
-      {
-        type: "assistant",
-        message: {
-          content: [
-            { type: "text", text: "I'll help you" },
-            {
-              type: "tool_use",
-              id: "tool_123",
-              name: "read_file",
-              input: { file_path: "/test.txt" },
-            },
-          ],
-          usage: { input_tokens: 100, output_tokens: 50 },
-        },
-      },
-      {
-        type: "user",
-        message: {
-          content: [
-            {
-              type: "tool_result",
-              tool_use_id: "tool_123",
-              content: "file content",
-              is_error: false,
-            },
-          ],
-        },
-      },
-    ];
-
-    const result = groupTurnsNaturally(data);
-
-    expect(result).toHaveLength(1);
-    expect(result[0]?.type).toBe("assistant_action");
-    expect(result[0]?.text_parts).toEqual(["I'll help you"]);
-    expect(result[0]?.tool_calls).toHaveLength(1);
-    expect(result[0]?.tool_calls?.[0]?.tool_use.name).toBe("read_file");
-    expect(result[0]?.tool_calls?.[0]?.tool_result?.content).toBe(
-      "file content",
-    );
-    expect(result[0]?.usage).toEqual({ input_tokens: 100, output_tokens: 50 });
-  });
-
-  test("groups user messages", () => {
-    const data: Turn[] = [
-      {
-        type: "user",
-        message: {
-          content: [{ type: "text", text: "Please help me" }],
-        },
-      },
-    ];
-
-    const result = groupTurnsNaturally(data);
-
-    expect(result).toHaveLength(1);
-    expect(result[0]?.type).toBe("user_message");
-    expect(result[0]?.text_parts).toEqual(["Please help me"]);
-  });
-
-  test("groups final results", () => {
-    const data: Turn[] = [
-      {
-        type: "result",
-        cost_usd: 0.1234,
-        duration_ms: 5000,
-        result: "Task completed",
-      },
-    ];
-
-    const result = groupTurnsNaturally(data);
-
-    expect(result).toHaveLength(1);
-    expect(result[0]?.type).toBe("final_result");
-    expect(result[0]?.data).toEqual(data[0]!);
-  });
-});
-
-describe("formatGroupedContent", () => {
-  test("formats system initialization", () => {
-    const groupedContent = [
-      {
-        type: "system_init",
-        tools_count: 3,
-      },
-    ];
-
-    const result = formatGroupedContent(groupedContent);
-
-    expect(result).toContain("## Claude Code Report");
-    expect(result).toContain("## 🚀 System Initialization");
-    expect(result).toContain("**Available Tools:** 3 tools loaded");
-  });
-
-  test("formats assistant actions", () => {
-    const groupedContent = [
-      {
-        type: "assistant_action",
-        text_parts: ["I'll help you with that"],
-        tool_calls: [
-          {
-            tool_use: {
-              type: "tool_use",
-              name: "test_tool",
-              input: { param: "value" },
-            },
-            tool_result: {
-              type: "tool_result",
-              content: "result",
-              is_error: false,
-            },
-          },
-        ],
-        usage: { input_tokens: 100, output_tokens: 50 },
-      },
-    ];
-
-    const result = formatGroupedContent(groupedContent);
-
-    expect(result).toContain("I'll help you with that");
-    expect(result).toContain("### 🔧 `test_tool`");
-    expect(result).toContain("*Token usage: 100 input, 50 output*");
-  });
-
-  test("formats user messages", () => {
-    const groupedContent = [
-      {
-        type: "user_message",
-        text_parts: ["Help me please"],
-      },
-    ];
-
-    const result = formatGroupedContent(groupedContent);
-
-    expect(result).toContain("## 👤 User");
-    expect(result).toContain("Help me please");
-  });
-
-  test("formats final results", () => {
-    const groupedContent = [
-      {
-        type: "final_result",
-        data: {
-          type: "result",
-          cost_usd: 0.1234,
-          duration_ms: 5678,
-          result: "Success!",
-        } as Turn,
-      },
-    ];
-
-    const result = formatGroupedContent(groupedContent);
-
-    expect(result).toContain("## ✅ Final Result");
-    expect(result).toContain("Success!");
-    expect(result).toContain("**Cost:** $0.1234");
-    expect(result).toContain("**Duration:** 5.7s");
-  });
-});
-
-describe("formatTurnsFromData", () => {
-  test("handles empty data", () => {
-    const result = formatTurnsFromData([]);
-    expect(result).toBe("## Claude Code Report\n\n");
-  });
-
-  test("formats complete conversation", () => {
-    const data: Turn[] = [
-      {
-        type: "system",
-        subtype: "init",
-        tools: [{ name: "tool1" }],
-      },
-      {
-        type: "assistant",
-        message: {
-          content: [
-            { type: "text", text: "I'll help you" },
-            {
-              type: "tool_use",
-              id: "tool_123",
-              name: "read_file",
-              input: { file_path: "/test.txt" },
-            },
-          ],
-        },
-      },
-      {
-        type: "user",
-        message: {
-          content: [
-            {
-              type: "tool_result",
-              tool_use_id: "tool_123",
-              content: "file content",
-              is_error: false,
-            },
-          ],
-        },
-      },
-      {
-        type: "result",
-        cost_usd: 0.05,
-        duration_ms: 2000,
-        result: "Done",
-      },
-    ];
-
-    const result = formatTurnsFromData(data);
-
-    expect(result).toContain("## Claude Code Report");
-    expect(result).toContain("## 🚀 System Initialization");
-    expect(result).toContain("I'll help you");
-    expect(result).toContain("### 🔧 `read_file`");
-    expect(result).toContain("## ✅ Final Result");
-    expect(result).toContain("Done");
-  });
-});
-
-describe("integration tests", () => {
-  test("formats real conversation data correctly", () => {
-    // Load the sample JSON data
-    const jsonPath = join(__dirname, "fixtures", "sample-turns.json");
-    const expectedPath = join(
-      __dirname,
-      "fixtures",
-      "sample-turns-expected-output.md",
-    );
-
-    const jsonData = JSON.parse(readFileSync(jsonPath, "utf-8"));
-    const expectedOutput = readFileSync(expectedPath, "utf-8").trim();
-
-    // Format the data using our function
-    const actualOutput = formatTurnsFromData(jsonData).trim();
-
-    // Compare the outputs
-    expect(actualOutput).toBe(expectedOutput);
-  });
-});

test/github/context.test.ts

@@ -1,115 +0,0 @@
-import { describe, it, expect } from "bun:test";
-import {
-  parseMultilineInput,
-  parseAdditionalPermissions,
-} from "../../src/github/context";
-
-describe("parseMultilineInput", () => {
-  it("should parse a comma-separated string", () => {
-    const input = `Bash(bun install),Bash(bun test:*),Bash(bun typecheck)`;
-    const result = parseMultilineInput(input);
-    expect(result).toEqual([
-      "Bash(bun install)",
-      "Bash(bun test:*)",
-      "Bash(bun typecheck)",
-    ]);
-  });
-
-  it("should parse multiline string", () => {
-    const input = `Bash(bun install)
-Bash(bun test:*)
-Bash(bun typecheck)`;
-    const result = parseMultilineInput(input);
-    expect(result).toEqual([
-      "Bash(bun install)",
-      "Bash(bun test:*)",
-      "Bash(bun typecheck)",
-    ]);
-  });
-
-  it("should parse comma-separated multiline line", () => {
-    const input = `Bash(bun install),Bash(bun test:*)
-Bash(bun typecheck)`;
-    const result = parseMultilineInput(input);
-    expect(result).toEqual([
-      "Bash(bun install)",
-      "Bash(bun test:*)",
-      "Bash(bun typecheck)",
-    ]);
-  });
-
-  it("should ignore comments", () => {
-    const input = `Bash(bun install),
-Bash(bun test:*) # For testing
-# For type checking
-Bash(bun typecheck)
-`;
-    const result = parseMultilineInput(input);
-    expect(result).toEqual([
-      "Bash(bun install)",
-      "Bash(bun test:*)",
-      "Bash(bun typecheck)",
-    ]);
-  });
-
-  it("should parse an empty string", () => {
-    const input = "";
-    const result = parseMultilineInput(input);
-    expect(result).toEqual([]);
-  });
-});
-
-describe("parseAdditionalPermissions", () => {
-  it("should parse single permission", () => {
-    const input = "actions: read";
-    const result = parseAdditionalPermissions(input);
-    expect(result.get("actions")).toBe("read");
-    expect(result.size).toBe(1);
-  });
-
-  it("should parse multiple permissions", () => {
-    const input = `actions: read
-packages: write
-contents: read`;
-    const result = parseAdditionalPermissions(input);
-    expect(result.get("actions")).toBe("read");
-    expect(result.get("packages")).toBe("write");
-    expect(result.get("contents")).toBe("read");
-    expect(result.size).toBe(3);
-  });
-
-  it("should handle empty string", () => {
-    const input = "";
-    const result = parseAdditionalPermissions(input);
-    expect(result.size).toBe(0);
-  });
-
-  it("should handle whitespace and empty lines", () => {
-    const input = `
-    actions: read
-
-    packages: write
-    `;
-    const result = parseAdditionalPermissions(input);
-    expect(result.get("actions")).toBe("read");
-    expect(result.get("packages")).toBe("write");
-    expect(result.size).toBe(2);
-  });
-
-  it("should ignore lines without colon separator", () => {
-    const input = `actions: read
-invalid line
-packages: write`;
-    const result = parseAdditionalPermissions(input);
-    expect(result.get("actions")).toBe("read");
-    expect(result.get("packages")).toBe("write");
-    expect(result.size).toBe(2);
-  });
-
-  it("should trim whitespace around keys and values", () => {
-    const input = "  actions  :  read  ";
-    const result = parseAdditionalPermissions(input);
-    expect(result.get("actions")).toBe("read");
-    expect(result.size).toBe(1);
-  });
-});

test/install-mcp-server.test.ts

@@ -1,7 +1,6 @@
 import { describe, test, expect, beforeEach, afterEach, spyOn } from "bun:test";
 import { prepareMcpConfig } from "../src/mcp/install-mcp-server";
 import * as core from "@actions/core";
-import type { ParsedGitHubContext } from "../src/github/context";
 
 describe("prepareMcpConfig", () => {
   let consoleInfoSpy: any;
@@ -9,41 +8,6 @@ describe("prepareMcpConfig", () => {
   let setFailedSpy: any;
   let processExitSpy: any;
 
-  // Create a mock context for tests
-  const mockContext: ParsedGitHubContext = {
-    runId: "test-run-id",
-    eventName: "issue_comment",
-    eventAction: "created",
-    repository: {
-      owner: "test-owner",
-      repo: "test-repo",
-      full_name: "test-owner/test-repo",
-    },
-    actor: "test-actor",
-    payload: {} as any,
-    entityNumber: 123,
-    isPR: false,
-    inputs: {
-      triggerPhrase: "@claude",
-      assigneeTrigger: "",
-      labelTrigger: "",
-      allowedTools: [],
-      disallowedTools: [],
-      customInstructions: "",
-      directPrompt: "",
-      branchPrefix: "",
-      useStickyComment: false,
-      additionalPermissions: new Map(),
-    },
-  };
-
-  const mockPRContext: ParsedGitHubContext = {
-    ...mockContext,
-    eventName: "pull_request",
-    isPR: true,
-    entityNumber: 456,
-  };
-
   beforeEach(() => {
     consoleInfoSpy = spyOn(core, "info").mockImplementation(() => {});
     consoleWarningSpy = spyOn(core, "warning").mockImplementation(() => {});
@@ -51,11 +15,6 @@ describe("prepareMcpConfig", () => {
     processExitSpy = spyOn(process, "exit").mockImplementation(() => {
       throw new Error("Process exit");
     });
-
-    // Set up required environment variables
-    if (!process.env.GITHUB_ACTION_PATH) {
-      process.env.GITHUB_ACTION_PATH = "/test/action/path";
-    }
   });
 
   afterEach(() => {
@@ -65,20 +24,21 @@ describe("prepareMcpConfig", () => {
     processExitSpy.mockRestore();
   });
 
-  test("should return base config when no additional config is provided and no allowed_tools", async () => {
+  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",
-      allowedTools: [],
-      context: mockContext,
     });
 
     const parsed = JSON.parse(result);
     expect(parsed.mcpServers).toBeDefined();
-    expect(parsed.mcpServers.github).not.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",
     );
@@ -89,63 +49,6 @@ describe("prepareMcpConfig", () => {
     );
   });
 
-  test("should include github MCP server when mcp__github__ tools are allowed", async () => {
-    const result = await prepareMcpConfig({
-      githubToken: "test-token",
-      owner: "test-owner",
-      repo: "test-repo",
-      branch: "test-branch",
-      allowedTools: [
-        "mcp__github__create_issue",
-        "mcp__github_file_ops__commit_files",
-      ],
-      context: mockContext,
-    });
-
-    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",
-    );
-  });
-
-  test("should not include github MCP server when only file_ops tools are allowed", async () => {
-    const result = await prepareMcpConfig({
-      githubToken: "test-token",
-      owner: "test-owner",
-      repo: "test-repo",
-      branch: "test-branch",
-      allowedTools: [
-        "mcp__github_file_ops__commit_files",
-        "mcp__github_file_ops__update_claude_comment",
-      ],
-      context: mockContext,
-    });
-
-    const parsed = JSON.parse(result);
-    expect(parsed.mcpServers).toBeDefined();
-    expect(parsed.mcpServers.github).not.toBeDefined();
-    expect(parsed.mcpServers.github_file_ops).toBeDefined();
-  });
-
-  test("should include file_ops server even when no GitHub tools are allowed", async () => {
-    const result = await prepareMcpConfig({
-      githubToken: "test-token",
-      owner: "test-owner",
-      repo: "test-repo",
-      branch: "test-branch",
-      allowedTools: ["Edit", "Read", "Write"],
-      context: mockContext,
-    });
-
-    const parsed = JSON.parse(result);
-    expect(parsed.mcpServers).toBeDefined();
-    expect(parsed.mcpServers.github).not.toBeDefined();
-    expect(parsed.mcpServers.github_file_ops).toBeDefined();
-  });
-
   test("should return base config when additional config is empty string", async () => {
     const result = await prepareMcpConfig({
       githubToken: "test-token",
@@ -153,13 +56,11 @@ describe("prepareMcpConfig", () => {
       repo: "test-repo",
       branch: "test-branch",
       additionalMcpConfig: "",
-      allowedTools: [],
-      context: mockContext,
     });
 
     const parsed = JSON.parse(result);
     expect(parsed.mcpServers).toBeDefined();
-    expect(parsed.mcpServers.github).not.toBeDefined();
+    expect(parsed.mcpServers.github).toBeDefined();
     expect(parsed.mcpServers.github_file_ops).toBeDefined();
     expect(consoleWarningSpy).not.toHaveBeenCalled();
   });
@@ -171,13 +72,11 @@ describe("prepareMcpConfig", () => {
       repo: "test-repo",
       branch: "test-branch",
       additionalMcpConfig: "   \n\t  ",
-      allowedTools: [],
-      context: mockContext,
     });
 
     const parsed = JSON.parse(result);
     expect(parsed.mcpServers).toBeDefined();
-    expect(parsed.mcpServers.github).not.toBeDefined();
+    expect(parsed.mcpServers.github).toBeDefined();
     expect(parsed.mcpServers.github_file_ops).toBeDefined();
     expect(consoleWarningSpy).not.toHaveBeenCalled();
   });
@@ -201,11 +100,6 @@ describe("prepareMcpConfig", () => {
       repo: "test-repo",
       branch: "test-branch",
       additionalMcpConfig: additionalConfig,
-      allowedTools: [
-        "mcp__github__create_issue",
-        "mcp__github_file_ops__commit_files",
-      ],
-      context: mockContext,
     });
 
     const parsed = JSON.parse(result);
@@ -239,11 +133,6 @@ describe("prepareMcpConfig", () => {
       repo: "test-repo",
       branch: "test-branch",
       additionalMcpConfig: additionalConfig,
-      allowedTools: [
-        "mcp__github__create_issue",
-        "mcp__github_file_ops__commit_files",
-      ],
-      context: mockContext,
     });
 
     const parsed = JSON.parse(result);
@@ -280,14 +169,12 @@ describe("prepareMcpConfig", () => {
       repo: "test-repo",
       branch: "test-branch",
       additionalMcpConfig: additionalConfig,
-      allowedTools: [],
-      context: mockContext,
     });
 
     const parsed = JSON.parse(result);
     expect(parsed.customProperty).toBe("custom-value");
     expect(parsed.anotherProperty).toEqual({ nested: "value" });
-    expect(parsed.mcpServers.github).not.toBeDefined();
+    expect(parsed.mcpServers.github).toBeDefined();
     expect(parsed.mcpServers.custom_server).toBeDefined();
   });
 
@@ -300,15 +187,13 @@ describe("prepareMcpConfig", () => {
       repo: "test-repo",
       branch: "test-branch",
       additionalMcpConfig: invalidJson,
-      allowedTools: [],
-      context: mockContext,
     });
 
     const parsed = JSON.parse(result);
     expect(consoleWarningSpy).toHaveBeenCalledWith(
       expect.stringContaining("Failed to parse additional MCP config:"),
     );
-    expect(parsed.mcpServers.github).not.toBeDefined();
+    expect(parsed.mcpServers.github).toBeDefined();
     expect(parsed.mcpServers.github_file_ops).toBeDefined();
   });
 
@@ -321,8 +206,6 @@ describe("prepareMcpConfig", () => {
       repo: "test-repo",
       branch: "test-branch",
       additionalMcpConfig: nonObjectJson,
-      allowedTools: [],
-      context: mockContext,
     });
 
     const parsed = JSON.parse(result);
@@ -332,7 +215,7 @@ describe("prepareMcpConfig", () => {
     expect(consoleWarningSpy).toHaveBeenCalledWith(
       expect.stringContaining("MCP config must be a valid JSON object"),
     );
-    expect(parsed.mcpServers.github).not.toBeDefined();
+    expect(parsed.mcpServers.github).toBeDefined();
     expect(parsed.mcpServers.github_file_ops).toBeDefined();
   });
 
@@ -345,8 +228,6 @@ describe("prepareMcpConfig", () => {
       repo: "test-repo",
       branch: "test-branch",
       additionalMcpConfig: nullJson,
-      allowedTools: [],
-      context: mockContext,
     });
 
     const parsed = JSON.parse(result);
@@ -356,7 +237,7 @@ describe("prepareMcpConfig", () => {
     expect(consoleWarningSpy).toHaveBeenCalledWith(
       expect.stringContaining("MCP config must be a valid JSON object"),
     );
-    expect(parsed.mcpServers.github).not.toBeDefined();
+    expect(parsed.mcpServers.github).toBeDefined();
     expect(parsed.mcpServers.github_file_ops).toBeDefined();
   });
 
@@ -369,8 +250,6 @@ describe("prepareMcpConfig", () => {
       repo: "test-repo",
       branch: "test-branch",
       additionalMcpConfig: arrayJson,
-      allowedTools: [],
-      context: mockContext,
     });
 
     const parsed = JSON.parse(result);
@@ -379,7 +258,7 @@ describe("prepareMcpConfig", () => {
     expect(consoleInfoSpy).toHaveBeenCalledWith(
       "Merging additional MCP server configuration with built-in servers",
     );
-    expect(parsed.mcpServers.github).not.toBeDefined();
+    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);
@@ -416,14 +295,12 @@ describe("prepareMcpConfig", () => {
       repo: "test-repo",
       branch: "test-branch",
       additionalMcpConfig: additionalConfig,
-      allowedTools: [],
-      context: mockContext,
     });
 
     const parsed = JSON.parse(result);
     expect(parsed.mcpServers.server1).toBeDefined();
     expect(parsed.mcpServers.server2).toBeDefined();
-    expect(parsed.mcpServers.github).not.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");
@@ -438,8 +315,6 @@ describe("prepareMcpConfig", () => {
       owner: "test-owner",
       repo: "test-repo",
       branch: "test-branch",
-      allowedTools: [],
-      context: mockContext,
     });
 
     const parsed = JSON.parse(result);
@@ -459,8 +334,6 @@ describe("prepareMcpConfig", () => {
       owner: "test-owner",
       repo: "test-repo",
       branch: "test-branch",
-      allowedTools: [],
-      context: mockContext,
     });
 
     const parsed = JSON.parse(result);
@@ -468,132 +341,4 @@ describe("prepareMcpConfig", () => {
 
     process.env.GITHUB_WORKSPACE = oldEnv;
   });
-
-  test("should include github_ci server when context.isPR is true and actions:read permission is granted", async () => {
-    const oldEnv = process.env.ACTIONS_TOKEN;
-    process.env.ACTIONS_TOKEN = "workflow-token";
-
-    const contextWithPermissions = {
-      ...mockPRContext,
-      inputs: {
-        ...mockPRContext.inputs,
-        additionalPermissions: new Map([["actions", "read"]]),
-      },
-    };
-
-    const result = await prepareMcpConfig({
-      githubToken: "test-token",
-      owner: "test-owner",
-      repo: "test-repo",
-      branch: "test-branch",
-      allowedTools: [],
-      context: contextWithPermissions,
-    });
-
-    const parsed = JSON.parse(result);
-    expect(parsed.mcpServers.github_ci).toBeDefined();
-    expect(parsed.mcpServers.github_ci.env.GITHUB_TOKEN).toBe("workflow-token");
-    expect(parsed.mcpServers.github_ci.env.PR_NUMBER).toBe("456");
-    expect(parsed.mcpServers.github_file_ops).toBeDefined();
-
-    process.env.ACTIONS_TOKEN = oldEnv;
-  });
-
-  test("should not include github_ci server when context.isPR is false", async () => {
-    const result = await prepareMcpConfig({
-      githubToken: "test-token",
-      owner: "test-owner",
-      repo: "test-repo",
-      branch: "test-branch",
-      allowedTools: [],
-      context: mockContext,
-    });
-
-    const parsed = JSON.parse(result);
-    expect(parsed.mcpServers.github_ci).not.toBeDefined();
-    expect(parsed.mcpServers.github_file_ops).toBeDefined();
-  });
-
-  test("should not include github_ci server when actions:read permission is not granted", async () => {
-    const oldTokenEnv = process.env.ACTIONS_TOKEN;
-    process.env.ACTIONS_TOKEN = "workflow-token";
-
-    const result = await prepareMcpConfig({
-      githubToken: "test-token",
-      owner: "test-owner",
-      repo: "test-repo",
-      branch: "test-branch",
-      allowedTools: [],
-      context: mockPRContext,
-    });
-
-    const parsed = JSON.parse(result);
-    expect(parsed.mcpServers.github_ci).not.toBeDefined();
-    expect(parsed.mcpServers.github_file_ops).toBeDefined();
-
-    process.env.ACTIONS_TOKEN = oldTokenEnv;
-  });
-
-  test("should parse additional_permissions with multiple lines correctly", async () => {
-    const oldTokenEnv = process.env.ACTIONS_TOKEN;
-    process.env.ACTIONS_TOKEN = "workflow-token";
-
-    const contextWithPermissions = {
-      ...mockPRContext,
-      inputs: {
-        ...mockPRContext.inputs,
-        additionalPermissions: new Map([
-          ["actions", "read"],
-          ["future", "permission"],
-        ]),
-      },
-    };
-
-    const result = await prepareMcpConfig({
-      githubToken: "test-token",
-      owner: "test-owner",
-      repo: "test-repo",
-      branch: "test-branch",
-      allowedTools: [],
-      context: contextWithPermissions,
-    });
-
-    const parsed = JSON.parse(result);
-    expect(parsed.mcpServers.github_ci).toBeDefined();
-    expect(parsed.mcpServers.github_ci.env.GITHUB_TOKEN).toBe("workflow-token");
-
-    process.env.ACTIONS_TOKEN = oldTokenEnv;
-  });
-
-  test("should warn when actions:read is requested but token lacks permission", async () => {
-    const oldTokenEnv = process.env.ACTIONS_TOKEN;
-    process.env.ACTIONS_TOKEN = "invalid-token";
-
-    const contextWithPermissions = {
-      ...mockPRContext,
-      inputs: {
-        ...mockPRContext.inputs,
-        additionalPermissions: new Map([["actions", "read"]]),
-      },
-    };
-
-    const result = await prepareMcpConfig({
-      githubToken: "test-token",
-      owner: "test-owner",
-      repo: "test-repo",
-      branch: "test-branch",
-      allowedTools: [],
-      context: contextWithPermissions,
-    });
-
-    const parsed = JSON.parse(result);
-    expect(parsed.mcpServers.github_ci).toBeDefined();
-    expect(consoleWarningSpy).toHaveBeenCalledWith(
-      expect.stringContaining(
-        "The github_ci MCP server requires 'actions: read' permission",
-      ),
-    );
-
-    process.env.ACTIONS_TOKEN = oldTokenEnv;
-  });
 });

test/mockContext.ts

@@ -10,18 +10,14 @@ import type {
 const defaultInputs = {
   triggerPhrase: "/claude",
   assigneeTrigger: "",
-  labelTrigger: "",
   anthropicModel: "claude-3-7-sonnet-20250219",
-  allowedTools: [] as string[],
+  allowedTools: "",
-  disallowedTools: [] as string[],
+  disallowedTools: "",
   customInstructions: "",
   directPrompt: "",
   useBedrock: false,
   useVertex: false,
   timeoutMinutes: 30,
-  branchPrefix: "claude/",
-  useStickyComment: false,
-  additionalPermissions: new Map<string, string>(),
 };
 
 const defaultRepository = {
@@ -95,12 +91,6 @@ export const mockIssueAssignedContext: ParsedGitHubContext = {
   actor: "admin-user",
   payload: {
     action: "assigned",
-    assignee: {
-      login: "claude-bot",
-      id: 11111,
-      avatar_url: "https://avatars.githubusercontent.com/u/11111",
-      html_url: "https://github.com/claude-bot",
-    },
     issue: {
       number: 123,
       title: "Feature: Add dark mode support",
@@ -132,46 +122,6 @@ export const mockIssueAssignedContext: ParsedGitHubContext = {
   inputs: { ...defaultInputs, assigneeTrigger: "@claude-bot" },
 };
 
-export const mockIssueLabeledContext: ParsedGitHubContext = {
-  runId: "1234567890",
-  eventName: "issues",
-  eventAction: "labeled",
-  repository: defaultRepository,
-  actor: "admin-user",
-  payload: {
-    action: "labeled",
-    issue: {
-      number: 1234,
-      title: "Enhancement: Improve search functionality",
-      body: "The current search is too slow and needs optimization",
-      user: {
-        login: "alice-wonder",
-        id: 54321,
-        avatar_url: "https://avatars.githubusercontent.com/u/54321",
-        html_url: "https://github.com/alice-wonder",
-      },
-      assignee: null,
-    },
-    label: {
-      id: 987654321,
-      name: "claude-task",
-      color: "f29513",
-      description: "Label for Claude AI interactions",
-    },
-    repository: {
-      name: "test-repo",
-      full_name: "test-owner/test-repo",
-      private: false,
-      owner: {
-        login: "test-owner",
-      },
-    },
-  } as IssuesEvent,
-  entityNumber: 1234,
-  isPR: false,
-  inputs: { ...defaultInputs, labelTrigger: "claude-task" },
-};
-
 // Issue comment on issue event
 export const mockIssueCommentContext: ParsedGitHubContext = {
   runId: "1234567890",

test/permissions.test.ts

@@ -62,14 +62,10 @@ describe("checkWritePermissions", () => {
     inputs: {
       triggerPhrase: "@claude",
       assigneeTrigger: "",
-      labelTrigger: "",
+      allowedTools: "",
-      allowedTools: [],
+      disallowedTools: "",
-      disallowedTools: [],
       customInstructions: "",
       directPrompt: "",
-      branchPrefix: "claude/",
-      useStickyComment: false,
-      additionalPermissions: new Map(),
     },
   });
 

test/prepare-context.test.ts

@@ -219,55 +219,6 @@ describe("parseEnvVarsWithContext", () => {
         ),
       ).toThrow("BASE_BRANCH is required for issues event");
     });
-
-    test("should allow issue assigned event with direct_prompt and no assigneeTrigger", () => {
-      const contextWithDirectPrompt = createMockContext({
-        ...mockIssueAssignedContext,
-        inputs: {
-          ...mockIssueAssignedContext.inputs,
-          assigneeTrigger: "", // No assignee trigger
-          directPrompt: "Please assess this issue", // But direct prompt is provided
-        },
-      });
-
-      const result = prepareContext(
-        contextWithDirectPrompt,
-        "12345",
-        "main",
-        "claude/issue-123-20240101_120000",
-      );
-
-      expect(result.eventData.eventName).toBe("issues");
-      expect(result.eventData.isPR).toBe(false);
-      expect(result.directPrompt).toBe("Please assess this issue");
-      if (
-        result.eventData.eventName === "issues" &&
-        result.eventData.eventAction === "assigned"
-      ) {
-        expect(result.eventData.issueNumber).toBe("123");
-        expect(result.eventData.assigneeTrigger).toBeUndefined();
-      }
-    });
-
-    test("should throw error when neither assigneeTrigger nor directPrompt provided for issue assigned event", () => {
-      const contextWithoutTriggers = createMockContext({
-        ...mockIssueAssignedContext,
-        inputs: {
-          ...mockIssueAssignedContext.inputs,
-          assigneeTrigger: "", // No assignee trigger
-          directPrompt: "", // No direct prompt
-        },
-      });
-
-      expect(() =>
-        prepareContext(
-          contextWithoutTriggers,
-          "12345",
-          "main",
-          "claude/issue-123-20240101_120000",
-        ),
-      ).toThrow("ASSIGNEE_TRIGGER is required for issue assigned event");
-    });
   });
 
   describe("optional fields", () => {
@@ -291,7 +242,7 @@ describe("parseEnvVarsWithContext", () => {
         ...mockPullRequestCommentContext,
         inputs: {
           ...mockPullRequestCommentContext.inputs,
-          allowedTools: ["Tool1", "Tool2"],
+          allowedTools: "Tool1,Tool2",
         },
       });
       const result = prepareContext(contextWithAllowedTools, "12345");

test/trigger-validation.test.ts

@@ -6,7 +6,6 @@ import { describe, it, expect } from "bun:test";
 import {
   createMockContext,
   mockIssueAssignedContext,
-  mockIssueLabeledContext,
   mockIssueCommentContext,
   mockIssueOpenedContext,
   mockPullRequestReviewContext,
@@ -30,14 +29,10 @@ describe("checkContainsTrigger", () => {
         inputs: {
           triggerPhrase: "/claude",
           assigneeTrigger: "",
-          labelTrigger: "",
           directPrompt: "Fix the bug in the login form",
-          allowedTools: [],
+          allowedTools: "",
-          disallowedTools: [],
+          disallowedTools: "",
           customInstructions: "",
-          branchPrefix: "claude/",
-          useStickyComment: false,
-          additionalPermissions: new Map(),
         },
       });
       expect(checkContainsTrigger(context)).toBe(true);
@@ -60,14 +55,10 @@ describe("checkContainsTrigger", () => {
         inputs: {
           triggerPhrase: "/claude",
           assigneeTrigger: "",
-          labelTrigger: "",
           directPrompt: "",
-          allowedTools: [],
+          allowedTools: "",
-          disallowedTools: [],
+          disallowedTools: "",
           customInstructions: "",
-          branchPrefix: "claude/",
-          useStickyComment: false,
-          additionalPermissions: new Map(),
         },
       });
       expect(checkContainsTrigger(context)).toBe(false);
@@ -96,11 +87,6 @@ describe("checkContainsTrigger", () => {
         ...mockIssueAssignedContext,
         payload: {
           ...mockIssueAssignedContext.payload,
-          assignee: {
-            ...(mockIssueAssignedContext.payload as IssuesAssignedEvent)
-              .assignee,
-            login: "otherUser",
-          },
           issue: {
             ...(mockIssueAssignedContext.payload as IssuesAssignedEvent).issue,
             assignee: {
@@ -116,39 +102,6 @@ describe("checkContainsTrigger", () => {
     });
   });
 
-  describe("label trigger", () => {
-    it("should return true when issue is labeled with the trigger label", () => {
-      const context = mockIssueLabeledContext;
-      expect(checkContainsTrigger(context)).toBe(true);
-    });
-
-    it("should return false when issue is labeled with a different label", () => {
-      const context = {
-        ...mockIssueLabeledContext,
-        payload: {
-          ...mockIssueLabeledContext.payload,
-          label: {
-            ...(mockIssueLabeledContext.payload as any).label,
-            name: "bug",
-          },
-        },
-      } as ParsedGitHubContext;
-      expect(checkContainsTrigger(context)).toBe(false);
-    });
-
-    it("should return false for non-labeled events", () => {
-      const context = {
-        ...mockIssueLabeledContext,
-        eventAction: "opened",
-        payload: {
-          ...mockIssueLabeledContext.payload,
-          action: "opened",
-        },
-      } as ParsedGitHubContext;
-      expect(checkContainsTrigger(context)).toBe(false);
-    });
-  });
-
   describe("issue body and title trigger", () => {
     it("should return true when issue body contains trigger phrase", () => {
       const context = mockIssueOpenedContext;
@@ -274,14 +227,10 @@ describe("checkContainsTrigger", () => {
         inputs: {
           triggerPhrase: "@claude",
           assigneeTrigger: "",
-          labelTrigger: "",
           directPrompt: "",
-          allowedTools: [],
+          allowedTools: "",
-          disallowedTools: [],
+          disallowedTools: "",
           customInstructions: "",
-          branchPrefix: "claude/",
-          useStickyComment: false,
-          additionalPermissions: new Map(),
         },
       });
       expect(checkContainsTrigger(context)).toBe(true);
@@ -305,14 +254,10 @@ describe("checkContainsTrigger", () => {
         inputs: {
           triggerPhrase: "@claude",
           assigneeTrigger: "",
-          labelTrigger: "",
           directPrompt: "",
-          allowedTools: [],
+          allowedTools: "",
-          disallowedTools: [],
+          disallowedTools: "",
           customInstructions: "",
-          branchPrefix: "claude/",
-          useStickyComment: false,
-          additionalPermissions: new Map(),
         },
       });
       expect(checkContainsTrigger(context)).toBe(true);
@@ -336,14 +281,10 @@ describe("checkContainsTrigger", () => {
         inputs: {
           triggerPhrase: "@claude",
           assigneeTrigger: "",
-          labelTrigger: "",
           directPrompt: "",
-          allowedTools: [],
+          allowedTools: "",
-          disallowedTools: [],
+          disallowedTools: "",
           customInstructions: "",
-          branchPrefix: "claude/",
-          useStickyComment: false,
-          additionalPermissions: new Map(),
         },
       });
       expect(checkContainsTrigger(context)).toBe(false);