Compare commits

...

2 Commits

Author SHA1 Message Date
Ashwin Bhat
851ffa401e tmp 2025-08-27 17:09:55 -07:00
Ashwin Bhat
4ebee54a00 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.
2025-08-27 17:05:29 -07:00
4 changed files with 32 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

@@ -72,7 +72,7 @@ export const createMockAutomationContext = (
const mergedInputs = overrides.inputs const mergedInputs = overrides.inputs
? { ...defaultInputs, ...overrides.inputs } ? { ...defaultInputs, ...overrides.inputs }
: defaultInputs; : { ...defaultInputs };
return { ...baseContext, ...overrides, inputs: mergedInputs }; return { ...baseContext, ...overrides, inputs: mergedInputs };
}; };

View File

@@ -23,6 +23,7 @@ describe("Tag Mode", () => {
}); });
test("shouldTrigger delegates to checkContainsTrigger", () => { test("shouldTrigger delegates to checkContainsTrigger", () => {
console.log("enter test");
const contextWithTrigger = createMockContext({ const contextWithTrigger = createMockContext({
eventName: "issue_comment", eventName: "issue_comment",
isPR: false, isPR: false,