Fix: correctly promote push events to Linear/Jira context and refine interaction detection

Co-authored-by: Junie <[email protected]>

Author: mashan555

src/github/context.ts

@@ -265,10 +265,12 @@ export function extractJunieWorkflowContext(tokenOwner: TokenOwner): JunieExecut
             if (process.env.LINEAR_API_TOKEN && process.env.ALL_INPUTS) {
                 try {
                     const allInputs = JSON.parse(process.env.ALL_INPUTS);
-                    if (allInputs.action === LINEAR_EVENT_ACTION) {
+                    if (allInputs.action === LINEAR_EVENT_ACTION || (allInputs.issue_id && allInputs.issue_title)) {
+                        console.log("✓ Detected Linear issue data in push event inputs - promoting to Linear event context");
                         return extractLinearEventData({ inputs: allInputs } as any, commonFields);
                     }
-                    if (allInputs.action === JIRA_EVENT_ACTION) {
+                    if (allInputs.action === JIRA_EVENT_ACTION || (allInputs.issue_key && allInputs.issue_summary)) {
+                        console.log("✓ Detected Jira issue data in push event inputs - promoting to Jira event context");
                         return extractJiraEventData({ inputs: allInputs } as any, commonFields);
                     }
                 } catch (e) {
@@ -578,6 +580,11 @@ export function isMinorFixEvent(context: JunieExecutionContext) {
 export function isTriggeredByUserInteraction(
     context: JunieExecutionContext,
 ): context is UserInitiatedEventContext {
+    // Exclude push events from "user interaction" because they don't have trigger phrases usually
+    // and we want to avoid unnecessary permission checks on every push
+    if (context.eventName === "push") {
+        return false;
+    }
     return USER_TRIGGERED_EVENTS.includes(context.eventName as UserTriggeredEventName);
 }
 

src/github/junie/prepare-junie.ts

@@ -110,13 +110,14 @@ export async function initializeJunieExecution({
 
 async function shouldHandle(context: JunieExecutionContext, octokit: Octokits): Promise<boolean> {
     console.log(`Checking if Junie should handle event: ${context.eventName}`);
+    
+    // Check for Linear/Jira first, regardless of event type (support push event triggers for tests)
     if (isLinearWorkflowDispatchEvent(context)) {
         console.log("✓ Linear task detected - handling task");
         try {
             getLinearClient(); // Just to verify token early
         } catch (e) {
             console.warn(`Linear client initialization failed: ${e instanceof Error ? e.message : String(e)}`);
-            // We don't return false here because we still want to try running the action
         }
         return true;
     }