Skip unsupported pull_request_review events in activation reaction/status conditions

[Copilot]

Activation steps were attempting to add the eyes reaction and run-start comment on pull_request_review events. Those paths are unsupported by add_reaction.cjs / add_workflow_run_comment.cjs, causing validation failures in workflows such as mattpocock-skills-reviewer.

• Condition builder change

  • Updated /pkg/workflow/expression_builder.go so reaction/status-comment condition generation no longer emits pull_request_review branches.
  • Kept existing fork-guard semantics for supported pull_request events.

• Centralized dispatch parity

  • Removed aw_context.event_type == 'pull_request_review' from the generated workflow_dispatch source-event disjunction for these same activation paths.

• Behavioral test updates

  • Updated /pkg/workflow/expressions_test.go to assert:
    • pull_request_review is excluded for both reaction and status-comment conditions.
    • supported events (issues, guarded pull_request, discussion) remain present.
  • Renamed/refined test cases for clearer intent.

if includePullRequests {
    terms = append(terms, BuildEventTypeEquals("pull_request_review_comment"))
    terms = append(terms, BuildEquals(eventExpr, BuildStringLiteral("pull_request")))
    // pull_request_review intentionally excluded
}

[Copilot]

Pull request overview

This PR prevents activation workflows from attempting reaction/status-comment behaviors on unsupported pull_request_review events by removing pull_request_review branches from the generated native and workflow_dispatch-source event conditions, and updating unit tests accordingly.

Changes:

• Removed pull_request_review from reaction/status-comment condition generation (native event and centralized workflow_dispatch source-event disjunction).
• Preserved fork-guard semantics for supported pull_request activation paths.
• Updated/renamed expression builder tests to assert pull_request_review is excluded while supported events remain included.

Show a summary per file

File	Description
pkg/workflow/expression_builder.go	Stops emitting pull_request_review branches in reaction/status-comment and workflow_dispatch source-event conditions.
pkg/workflow/expressions_test.go	Updates tests to assert pull_request_review is excluded and supported events remain present.

Copilot's findings

Tip

Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

• Files reviewed: 2/2 changed files
• Comments generated: 1

[pelikhan]

@copilot review all comments, lint go

[Copilot]

@copilot review all comments, lint go

Done. I addressed the pending review feedback by clarifying the fork-guard comment in pkg/workflow/expression_builder.go to cover both reactions and status-comments in commit 875f0ca, and re-ran Go lint (make golint) successfully.