remove unnecessary changes from PR

- Revert update-comment-link.ts changes (agent mode doesn't use this)
- Revert create-initial.ts changes (agent mode doesn't create comments)
- Remove unused default-branch.ts file
- Revert install-mcp-server.ts changes (agent mode uses minimal MCP)

These files are only used by tag mode for entity-based events, not needed
for workflow_dispatch/schedule support via agent mode.
This commit is contained in:
km-anthropic
2025-07-29 11:06:24 -07:00
parent 3ebb5202a2
commit b85d457dc6
4 changed files with 12 additions and 35 deletions

View File

@@ -24,13 +24,6 @@ async function run() {
const context = parseGitHubContext(); const context = parseGitHubContext();
const { owner, repo } = context.repository; const { owner, repo } = context.repository;
// This script is only called for entity-based events
if (!context.entityNumber) {
throw new Error("update-comment-link requires an entityNumber");
}
const entityNumber = context.entityNumber;
const octokit = createOctokit(githubToken); const octokit = createOctokit(githubToken);
const serverUrl = GITHUB_SERVER_URL; const serverUrl = GITHUB_SERVER_URL;
@@ -80,7 +73,7 @@ async function run() {
const { data: pr } = await octokit.rest.pulls.get({ const { data: pr } = await octokit.rest.pulls.get({
owner, owner,
repo, repo,
pull_number: entityNumber, pull_number: context.entityNumber,
}); });
console.log(`PR state: ${pr.state}`); console.log(`PR state: ${pr.state}`);
console.log(`PR comments count: ${pr.comments}`); console.log(`PR comments count: ${pr.comments}`);

View File

@@ -22,12 +22,6 @@ export async function createInitialComment(
) { ) {
const { owner, repo } = context.repository; const { owner, repo } = context.repository;
// Entity events should always have entityNumber
if (!context.entityNumber) {
throw new Error("createInitialComment requires an entityNumber");
}
const entityNumber = context.entityNumber;
const jobRunLink = createJobRunLink(owner, repo, context.runId); const jobRunLink = createJobRunLink(owner, repo, context.runId);
const initialBody = createCommentBody(jobRunLink); const initialBody = createCommentBody(jobRunLink);
@@ -42,7 +36,7 @@ export async function createInitialComment(
const comments = await octokit.rest.issues.listComments({ const comments = await octokit.rest.issues.listComments({
owner, owner,
repo, repo,
issue_number: entityNumber, issue_number: context.entityNumber,
}); });
const existingComment = comments.data.find((comment) => { const existingComment = comments.data.find((comment) => {
const idMatch = comment.user?.id === CLAUDE_APP_BOT_ID; const idMatch = comment.user?.id === CLAUDE_APP_BOT_ID;
@@ -65,7 +59,7 @@ export async function createInitialComment(
response = await octokit.rest.issues.createComment({ response = await octokit.rest.issues.createComment({
owner, owner,
repo, repo,
issue_number: entityNumber, issue_number: context.entityNumber,
body: initialBody, body: initialBody,
}); });
} }
@@ -74,7 +68,7 @@ export async function createInitialComment(
response = await octokit.rest.pulls.createReplyForReviewComment({ response = await octokit.rest.pulls.createReplyForReviewComment({
owner, owner,
repo, repo,
pull_number: entityNumber, pull_number: context.entityNumber,
comment_id: context.payload.comment.id, comment_id: context.payload.comment.id,
body: initialBody, body: initialBody,
}); });
@@ -83,7 +77,7 @@ export async function createInitialComment(
response = await octokit.rest.issues.createComment({ response = await octokit.rest.issues.createComment({
owner, owner,
repo, repo,
issue_number: entityNumber, issue_number: context.entityNumber,
body: initialBody, body: initialBody,
}); });
} }
@@ -101,7 +95,7 @@ export async function createInitialComment(
const response = await octokit.rest.issues.createComment({ const response = await octokit.rest.issues.createComment({
owner, owner,
repo, repo,
issue_number: entityNumber, issue_number: context.entityNumber,
body: initialBody, body: initialBody,
}); });

View File

@@ -1,13 +0,0 @@
import type { Octokit } from "@octokit/rest";
export async function getDefaultBranch(
rest: Octokit,
owner: string,
repo: string,
): Promise<string> {
const repoResponse = await rest.repos.get({
owner,
repo,
});
return repoResponse.data.default_branch;
}

View File

@@ -1,5 +1,5 @@
import * as core from "@actions/core"; import * as core from "@actions/core";
import { GITHUB_API_URL } from "../github/api/config"; import { GITHUB_API_URL, GITHUB_SERVER_URL } from "../github/api/config";
import type { ParsedGitHubContext } from "../github/context"; import type { ParsedGitHubContext } from "../github/context";
import { Octokit } from "@octokit/rest"; import { Octokit } from "@octokit/rest";
@@ -141,7 +141,7 @@ export async function prepareMcpConfig(
GITHUB_TOKEN: process.env.ACTIONS_TOKEN, GITHUB_TOKEN: process.env.ACTIONS_TOKEN,
REPO_OWNER: owner, REPO_OWNER: owner,
REPO_NAME: repo, REPO_NAME: repo,
PR_NUMBER: context.entityNumber?.toString() || "", PR_NUMBER: context.entityNumber.toString(),
RUNNER_TEMP: process.env.RUNNER_TEMP || "/tmp", RUNNER_TEMP: process.env.RUNNER_TEMP || "/tmp",
}, },
}; };
@@ -156,10 +156,13 @@ export async function prepareMcpConfig(
"--rm", "--rm",
"-e", "-e",
"GITHUB_PERSONAL_ACCESS_TOKEN", "GITHUB_PERSONAL_ACCESS_TOKEN",
"ghcr.io/github/github-mcp-server:sha-721fd3e", // https://github.com/github/github-mcp-server/releases/tag/v0.6.0 "ghcr.io/github/github-mcp-server:sha-efef8ae", // https://github.com/github/github-mcp-server/releases/tag/v0.9.0
"-e",
"GITHUB_HOST",
], ],
env: { env: {
GITHUB_PERSONAL_ACCESS_TOKEN: githubToken, GITHUB_PERSONAL_ACCESS_TOKEN: githubToken,
GITHUB_HOST: GITHUB_SERVER_URL,
}, },
}; };
} }