Skip to content

v1.2.0: πŸ‘€ reaction on requests + rename comment trigger to /orca-code-review - #6

Merged
ZhenghuaBao merged 2 commits into
mainfrom
feat/eyes-reaction
Jul 21, 2026
Merged

v1.2.0: πŸ‘€ reaction on requests + rename comment trigger to /orca-code-review#6
ZhenghuaBao merged 2 commits into
mainfrom
feat/eyes-reaction

Conversation

@ZhenghuaBao

@ZhenghuaBao ZhenghuaBao commented Jul 21, 2026

Copy link
Copy Markdown
Contributor

Orca-Code-Review β€” push 1

Severity Count
P0 0
P1 0
P2 0
P3 0

Tier: STRONG (final pass) β€” pass

βœ… no blocking findings

Summary

  • Add πŸ‘€ GitHub reaction on the PR body (and on the trigger comment for /orca-code-review) so requesters see an immediate acknowledgement that the bot noticed the trigger. Static one-shot; failures non-fatal.
  • Rename the comment trigger from /orcarouter-review to /orca-code-review to match the shipped brand.

Compatibility

  • New reaction step: purely additive. If the reactions API fails, the step logs a warning and the review still runs (never blocks).
  • Trigger rename: the action code does NOT depend on the string β€” the startsWith(...) gate lives in the consumer's workflow. Existing workflows on /orcarouter-review keep working (their local string still matches their own if:). Consumers who copy the updated example workflow will now use /orca-code-review.

Test plan

  • feat/eyes-reaction verified end-to-end on ZhenghuaBao/Code-Review-Test PR chore: rename example workflow to orca-code-review.yml (v1.2.1)Β #7:
    • πŸ‘€ posted on PR body and on the trigger comment (github-actions[bot]).
    • New /orca-code-review trigger recognised (job started, no more skipped in 1s).
    • Reactions API POST is idempotent β€” re-runs (synchronize) don't duplicate.

πŸ€– Generated with Claude Code

ZhenghuaBao and others added 2 commits July 21, 2026 15:00
Static one-shot reaction so the requester sees the bot noticed the
trigger. On every event, react on the PR body (issue reactions
endpoint β€” PRs are issues). For issue_comment triggers
(/orcarouter-review), also react on the trigger comment so a
maintainer sees direct feedback on THEIR comment. Uses gh api with
inputs.github-token (issues: write + pull-requests: write already
granted in the example workflow).

Reaction outages are non-fatal: the review must never be blocked by a
UI-affordance failure. Placed after Resolve PR refs (PR number
available) and before any skip gate, so even a skipped run still
visibly acknowledges receipt. Reactions API POST is idempotent, so
re-runs (synchronize) don't duplicate.

Co-Authored-By: Claude Opus 4 <noreply@anthropic.com>
Matches the shipped brand (Orca-Code-Review). Legacy /orcarouter-review
was left over from the pre-rebrand naming and confused new consumers
about what command actually re-runs the review.

Applies the rename in the example workflow's issue_comment `if:` gate,
the SECURITY.md threat-model section, the action's input-description
prose, the README, and one comment in the settings unit test. No
runtime code depends on the exact string outside the workflow's `if:`
gate itself, so no gate-logic change is needed.

Consumers with the previous workflow copy must update the trigger
string in their `.github/workflows/*.yml` to /orca-code-review.

Co-Authored-By: Claude Opus 4 <noreply@anthropic.com>
@ZhenghuaBao
ZhenghuaBao merged commit 317031e into main Jul 21, 2026
1 check passed
@ZhenghuaBao
ZhenghuaBao deleted the feat/eyes-reaction branch July 21, 2026 07:57

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

πŸ’‘ Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 2e644f6b60

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with πŸ‘.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread action.yml
`⏸️ **Automatic review is off for this PR** (${process.env.REASON}).\n\n` +
`The check **passes** β€” this never blocks your merge. A maintainer can still ` +
`run a review on demand by commenting \`/orcarouter-review\`, or change the ` +
`run a review on demand by commenting \`/orca-code-review\`, or change the ` +

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Preserve the old trigger in skip notices

For repos that upgrade the floating @v1 action but keep the previously copied workflow gate, issue_comment still only matches /orcarouter-review; this action cannot update that consumer-owned if:. When a settings skip happens (auto_review=false, on_demand, draft, or author allowlist), this PR comment now tells maintainers to use /orca-code-review, which those existing workflows will ignore, so the advertised on-demand escape hatch does not work. Consider mentioning both triggers or making the displayed trigger configurable.

Useful? React with πŸ‘Β / πŸ‘Ž.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant