From 1fae65b3841271fb50ae848bdf124d944a2f8c5e Mon Sep 17 00:00:00 2001 From: "claude[bot]" <209825114+claude[bot]@users.noreply.github.com> Date: Wed, 11 Jun 2025 13:53:34 +0000 Subject: [PATCH 1/4] fix: add baseUrl to Octokit initialization in update_claude_comment Fixes Bad credentials error on GitHub Enterprise Server by passing GITHUB_API_URL as baseUrl when initializing Octokit, consistent with other Octokit instances in the codebase. Fixes #156 Related to #107 Co-authored-by: ashwin-ant --- src/mcp/github-file-ops-server.ts | 1 + 1 file changed, 1 insertion(+) diff --git a/src/mcp/github-file-ops-server.ts b/src/mcp/github-file-ops-server.ts index a34f11595..9a769af1a 100644 --- a/src/mcp/github-file-ops-server.ts +++ b/src/mcp/github-file-ops-server.ts @@ -466,6 +466,7 @@ server.tool( const octokit = new Octokit({ auth: githubToken, + baseUrl: GITHUB_API_URL, }); const isPullRequestReviewComment = From 4f6cefd1fe07329d9884f9f659177494fb256732 Mon Sep 17 00:00:00 2001 From: "claude[bot]" <209825114+claude[bot]@users.noreply.github.com> Date: Wed, 11 Jun 2025 13:58:19 +0000 Subject: [PATCH 2/4] fix: pass GITHUB_API_URL as env var to MCP server Update the MCP server initialization to pass GITHUB_API_URL as an environment variable, allowing it to work correctly with GitHub Enterprise Server instances. Co-authored-by: ashwin-ant --- src/mcp/github-file-ops-server.ts | 2 +- src/mcp/install-mcp-server.ts | 1 + 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/src/mcp/github-file-ops-server.ts b/src/mcp/github-file-ops-server.ts index 9a769af1a..57a60a00d 100644 --- a/src/mcp/github-file-ops-server.ts +++ b/src/mcp/github-file-ops-server.ts @@ -6,7 +6,6 @@ import { z } from "zod"; import { readFile } from "fs/promises"; import { join } from "path"; import fetch from "node-fetch"; -import { GITHUB_API_URL } from "../github/api/config"; import { Octokit } from "@octokit/rest"; import { updateClaudeComment } from "../github/operations/comments/update-claude-comment"; @@ -40,6 +39,7 @@ const REPO_OWNER = process.env.REPO_OWNER; const REPO_NAME = process.env.REPO_NAME; const BRANCH_NAME = process.env.BRANCH_NAME; const REPO_DIR = process.env.REPO_DIR || process.cwd(); +const GITHUB_API_URL = process.env.GITHUB_API_URL || "https://api.github.com"; if (!REPO_OWNER || !REPO_NAME || !BRANCH_NAME) { console.error( diff --git a/src/mcp/install-mcp-server.ts b/src/mcp/install-mcp-server.ts index 0eba6af54..7172bc5c7 100644 --- a/src/mcp/install-mcp-server.ts +++ b/src/mcp/install-mcp-server.ts @@ -46,6 +46,7 @@ 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: process.env.GITHUB_API_URL || "https://api.github.com", }, }, }, From 22e1d39f4770b4e89253f49165b1d1e0df066505 Mon Sep 17 00:00:00 2001 From: "claude[bot]" <209825114+claude[bot]@users.noreply.github.com> Date: Wed, 11 Jun 2025 14:10:26 +0000 Subject: [PATCH 3/4] fix: import GITHUB_API_URL from config in install-mcp-server Use the centralized GITHUB_API_URL constant from src/github/api/config.ts instead of reading directly from process.env when passing environment variables to the MCP server. This ensures consistency with how the API URL is handled throughout the codebase. Co-authored-by: ashwin-ant --- src/mcp/install-mcp-server.ts | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/mcp/install-mcp-server.ts b/src/mcp/install-mcp-server.ts index 7172bc5c7..0fa543621 100644 --- a/src/mcp/install-mcp-server.ts +++ b/src/mcp/install-mcp-server.ts @@ -1,4 +1,5 @@ import * as core from "@actions/core"; +import { GITHUB_API_URL } from "../github/api/config"; type PrepareConfigParams = { githubToken: string; @@ -46,7 +47,7 @@ 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: process.env.GITHUB_API_URL || "https://api.github.com", + GITHUB_API_URL: GITHUB_API_URL, }, }, }, From 3373efd9b1469047afab79d2cafeb89be58d5cb3 Mon Sep 17 00:00:00 2001 From: Ashwin Bhat Date: Wed, 11 Jun 2025 10:20:25 -0400 Subject: [PATCH 4/4] fix --- src/mcp/github-file-ops-server.ts | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/mcp/github-file-ops-server.ts b/src/mcp/github-file-ops-server.ts index 57a60a00d..9a769af1a 100644 --- a/src/mcp/github-file-ops-server.ts +++ b/src/mcp/github-file-ops-server.ts @@ -6,6 +6,7 @@ import { z } from "zod"; import { readFile } from "fs/promises"; import { join } from "path"; import fetch from "node-fetch"; +import { GITHUB_API_URL } from "../github/api/config"; import { Octokit } from "@octokit/rest"; import { updateClaudeComment } from "../github/operations/comments/update-claude-comment"; @@ -39,7 +40,6 @@ const REPO_OWNER = process.env.REPO_OWNER; const REPO_NAME = process.env.REPO_NAME; const BRANCH_NAME = process.env.BRANCH_NAME; const REPO_DIR = process.env.REPO_DIR || process.cwd(); -const GITHUB_API_URL = process.env.GITHUB_API_URL || "https://api.github.com"; if (!REPO_OWNER || !REPO_NAME || !BRANCH_NAME) { console.error(