remove unnecessary changes

This commit is contained in:
km-anthropic
2025-07-29 11:01:38 -07:00
parent 3402c5355d
commit 3ebb5202a2
2 changed files with 11 additions and 19 deletions

View File

@@ -34,16 +34,6 @@ const BASE_ALLOWED_TOOLS = [
]; ];
const DISALLOWED_TOOLS = ["WebSearch", "WebFetch"]; const DISALLOWED_TOOLS = ["WebSearch", "WebFetch"];
function getEntityNumberXml(eventData: EventData): string {
if (eventData.isPR && "prNumber" in eventData) {
return `<pr_number>${eventData.prNumber}</pr_number>`;
}
if ("issueNumber" in eventData) {
return `<issue_number>${eventData.issueNumber}</issue_number>`;
}
return "";
}
export function buildAllowedToolsString( export function buildAllowedToolsString(
customAllowedTools?: string[], customAllowedTools?: string[],
includeActionsTools: boolean = false, includeActionsTools: boolean = false,
@@ -135,10 +125,8 @@ export function prepareContext(
const isPR = context.isPR; const isPR = context.isPR;
// Get PR/Issue number from entityNumber // Get PR/Issue number from entityNumber
const prNumber = const prNumber = isPR ? context.entityNumber.toString() : undefined;
isPR && context.entityNumber ? context.entityNumber.toString() : undefined; const issueNumber = !isPR ? context.entityNumber.toString() : undefined;
const issueNumber =
!isPR && context.entityNumber ? context.entityNumber.toString() : undefined;
// Extract trigger username and comment data based on event type // Extract trigger username and comment data based on event type
let triggerUsername: string | undefined; let triggerUsername: string | undefined;
@@ -551,7 +539,6 @@ export function generatePrompt(
); );
} }
const { eventData } = context;
const { const {
contextData, contextData,
comments, comments,
@@ -559,10 +546,11 @@ export function generatePrompt(
reviewData, reviewData,
imageUrlMap, imageUrlMap,
} = githubData; } = githubData;
const { eventData } = context;
const { eventType, triggerContext } = getEventTypeAndContext(context); const { eventType, triggerContext } = getEventTypeAndContext(context);
const formattedContext = formatContext(contextData, eventData.isPR!); const formattedContext = formatContext(contextData, eventData.isPR);
const formattedComments = formatComments(comments, imageUrlMap); const formattedComments = formatComments(comments, imageUrlMap);
const formattedReviewComments = eventData.isPR const formattedReviewComments = eventData.isPR
? formatReviewComments(reviewData, imageUrlMap) ? formatReviewComments(reviewData, imageUrlMap)
@@ -611,10 +599,14 @@ ${eventData.isPR ? formattedChangedFiles || "No files changed" : ""}
<is_pr>${eventData.isPR ? "true" : "false"}</is_pr> <is_pr>${eventData.isPR ? "true" : "false"}</is_pr>
<trigger_context>${triggerContext}</trigger_context> <trigger_context>${triggerContext}</trigger_context>
<repository>${context.repository}</repository> <repository>${context.repository}</repository>
${getEntityNumberXml(eventData)} ${
eventData.isPR
? `<pr_number>${eventData.prNumber}</pr_number>`
: `<issue_number>${eventData.issueNumber ?? ""}</issue_number>`
}
<claude_comment_id>${context.claudeCommentId}</claude_comment_id> <claude_comment_id>${context.claudeCommentId}</claude_comment_id>
<trigger_username>${context.triggerUsername ?? "Unknown"}</trigger_username> <trigger_username>${context.triggerUsername ?? "Unknown"}</trigger_username>
<trigger_display_name>${githubData?.triggerDisplayName ?? context.triggerUsername ?? "Unknown"}</trigger_display_name> <trigger_display_name>${githubData.triggerDisplayName ?? context.triggerUsername ?? "Unknown"}</trigger_display_name>
<trigger_phrase>${context.triggerPhrase}</trigger_phrase> <trigger_phrase>${context.triggerPhrase}</trigger_phrase>
${ ${
(eventData.eventName === "issue_comment" || (eventData.eventName === "issue_comment" ||

View File

@@ -1,6 +1,6 @@
export type CommonFields = { export type CommonFields = {
repository: string; repository: string;
claudeCommentId?: string; claudeCommentId: string;
triggerPhrase: string; triggerPhrase: string;
triggerUsername?: string; triggerUsername?: string;
customInstructions?: string; customInstructions?: string;