Compare commits

...

4 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
GitHub Actions
ef8c0a650e chore: bump Claude Code version to 1.0.93 2025-08-27 22:27:45 +00:00
GitHub Actions
dd49718216 chore: bump Claude Code version to 1.0.94 2025-08-27 21:53:29 +00:00
4 changed files with 32 additions and 4 deletions

View File

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

View File

@@ -24,11 +24,16 @@ export const tagMode: Mode = {
description: "Traditional implementation mode triggered by @claude mentions",
shouldTrigger(context) {
console.log("tagMode.shouldTrigger called");
// Tag mode only handles entity events
if (!isEntityContext(context)) {
console.log("Not entity context, returning 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) {

View File

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

View File

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