Skip to content

Probe: capture copilot-review event payload#1593

Closed
brendancol wants to merge 1 commit into
mainfrom
probe-copilot-review
Closed

Probe: capture copilot-review event payload#1593
brendancol wants to merge 1 commit into
mainfrom
probe-copilot-review

Conversation

@brendancol
Copy link
Copy Markdown
Contributor

Summary

Test plan

  • Inspect the debug-event job logs and record the printed assoc value
  • Use that to fix the if: gate in a follow-up, then revert this PR and remove the debug job

Adds a top-of-file comment so this PR touches the workflow and triggers
the unconditional debug-event job. Revert after reading the logs.
@github-actions github-actions Bot added the performance PR touches performance-sensitive code label May 11, 2026
@brendancol
Copy link
Copy Markdown
Contributor Author

Superseded by #1594 — debug output captured (assoc=CONTRIBUTOR), follow-up fix opened.

@brendancol brendancol closed this May 11, 2026
@brendancol brendancol deleted the probe-copilot-review branch May 11, 2026 15:06
brendancol added a commit that referenced this pull request May 11, 2026
The pull_request_target event payload reports author_association as
CONTRIBUTOR for org members on same-repo branches (confirmed via the
debug-event job added in #1592 and fired by #1593), so the previous
gate filtered out every maintainer PR.

Switch to head.repo.full_name == base.repo.full_name. Same-repo PRs
require push access to the base repo, which is the maintainer signal
we actually want. Fork PRs (where the maintainer cannot push to base)
remain excluded.

Also removes the now-unneeded debug-event job.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

performance PR touches performance-sensitive code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant