Add debug logging to checkContainsTrigger for test debugging

Added console.log statements to trace execution flow in trigger validation:
- Log context and inputs at function entry
- Log issue content and regex pattern matching
- Log final return value
- Add logging to tagMode.shouldTrigger to trace delegation

These logs help debug the 'shouldTrigger delegates to checkContainsTrigger' test.
This commit is contained in:
Ashwin Bhat
2025-08-27 17:02:16 -07:00
parent ef8c0a650e
commit 4ebee54a00
3 changed files with 31 additions and 4 deletions

View File

@@ -12,10 +12,23 @@ import {
import type { ParsedGitHubContext } from "../context"; import type { ParsedGitHubContext } from "../context";
export function checkContainsTrigger(context: ParsedGitHubContext): boolean { export function checkContainsTrigger(context: ParsedGitHubContext): boolean {
console.log("checkContainsTrigger called with context:", {
eventName: context.eventName,
eventAction: context.eventAction,
inputs: context.inputs,
});
const { const {
inputs: { assigneeTrigger, labelTrigger, triggerPhrase, prompt }, inputs: { assigneeTrigger, labelTrigger, triggerPhrase, prompt },
} = context; } = context;
console.log("Extracted inputs:", {
assigneeTrigger,
labelTrigger,
triggerPhrase,
prompt,
});
// If prompt is provided, always trigger // If prompt is provided, always trigger
if (prompt) { if (prompt) {
console.log(`Prompt provided, triggering action`); console.log(`Prompt provided, triggering action`);
@@ -46,15 +59,21 @@ export function checkContainsTrigger(context: ParsedGitHubContext): boolean {
// Check for issue body and title trigger on issue creation // Check for issue body and title trigger on issue creation
if (isIssuesEvent(context) && context.eventAction === "opened") { if (isIssuesEvent(context) && context.eventAction === "opened") {
console.log("Checking issue opened trigger");
const issueBody = context.payload.issue.body || ""; const issueBody = context.payload.issue.body || "";
const issueTitle = context.payload.issue.title || ""; const issueTitle = context.payload.issue.title || "";
console.log("Issue content:", { issueBody, issueTitle });
// Check for exact match with word boundaries or punctuation // Check for exact match with word boundaries or punctuation
const regex = new RegExp( const regex = new RegExp(
`(^|\\s)${escapeRegExp(triggerPhrase)}([\\s.,!?;:]|$)`, `(^|\\s)${escapeRegExp(triggerPhrase)}([\\s.,!?;:]|$)`,
); );
console.log("Regex pattern:", regex.toString());
// Check in body // Check in body
if (regex.test(issueBody)) { const bodyMatch = regex.test(issueBody);
console.log("Body match result:", bodyMatch);
if (bodyMatch) {
console.log( console.log(
`Issue body contains exact trigger phrase '${triggerPhrase}'`, `Issue body contains exact trigger phrase '${triggerPhrase}'`,
); );
@@ -62,7 +81,9 @@ export function checkContainsTrigger(context: ParsedGitHubContext): boolean {
} }
// Check in title // Check in title
if (regex.test(issueTitle)) { const titleMatch = regex.test(issueTitle);
console.log("Title match result:", titleMatch);
if (titleMatch) {
console.log( console.log(
`Issue title contains exact trigger phrase '${triggerPhrase}'`, `Issue title contains exact trigger phrase '${triggerPhrase}'`,
); );
@@ -133,6 +154,7 @@ export function checkContainsTrigger(context: ParsedGitHubContext): boolean {
} }
console.log(`No trigger was met for ${triggerPhrase}`); console.log(`No trigger was met for ${triggerPhrase}`);
console.log("Returning false from checkContainsTrigger");
return false; return false;
} }

View File

@@ -24,11 +24,16 @@ export const tagMode: Mode = {
description: "Traditional implementation mode triggered by @claude mentions", description: "Traditional implementation mode triggered by @claude mentions",
shouldTrigger(context) { shouldTrigger(context) {
console.log("tagMode.shouldTrigger called");
// Tag mode only handles entity events // Tag mode only handles entity events
if (!isEntityContext(context)) { if (!isEntityContext(context)) {
console.log("Not entity context, returning false");
return false; return false;
} }
return checkContainsTrigger(context); console.log("Is entity context, calling checkContainsTrigger");
const result = checkContainsTrigger(context);
console.log("checkContainsTrigger returned:", result);
return result;
}, },
prepareContext(context, data) { prepareContext(context, data) {

View File

@@ -22,7 +22,7 @@ describe("Tag Mode", () => {
expect(tagMode.shouldCreateTrackingComment()).toBe(true); expect(tagMode.shouldCreateTrackingComment()).toBe(true);
}); });
test("shouldTrigger delegates to checkContainsTrigger", () => { test.only("shouldTrigger delegates to checkContainsTrigger", () => {
const contextWithTrigger = createMockContext({ const contextWithTrigger = createMockContext({
eventName: "issue_comment", eventName: "issue_comment",
isPR: false, isPR: false,