[WIP] Review context handling in push_to_pull_request_branch (#19604)

* Initial plan

* Fix: handle undefined context in push_to_pull_request_branch

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>

Author: Copilot

actions/setup/js/push_to_pull_request_branch.cjs

@@ -192,7 +192,7 @@ async function main(config = {}) {
     // Compute the target branch name based on target configuration
     let pullNumber;
     if (target === "triggering") {
-      pullNumber = context.payload?.pull_request?.number || context.payload?.issue?.number;
+      pullNumber = typeof context !== "undefined" ? context.payload?.pull_request?.number || context.payload?.issue?.number : undefined;
 
       if (!pullNumber) {
         return { success: false, error: 'push-to-pull-request-branch with target "triggering" requires pull request context' };

actions/setup/js/push_to_pull_request_branch.test.cjs

@@ -334,6 +334,19 @@ index 0000000..abc1234
       expect(result.error).toContain("pull request context");
     });
 
+    it('should fail gracefully with target "triggering" when context is undefined (MCP daemon mode)', async () => {
+      delete global.context;
+
+      const patchPath = createPatchFile();
+
+      const module = await loadModule();
+      const handler = await module.main({ target: "triggering" });
+      const result = await handler({ patch_path: patchPath }, {});
+
+      expect(result.success).toBe(false);
+      expect(result.error).toContain("pull request context");
+    });
+
     it('should handle schedule event with target "*" and explicit PR number', async () => {
       mockContext.eventName = "schedule";
       delete mockContext.payload.pull_request;