Skip to content

Run the AI reviewer only on /review - #14

Merged
SimonKimDev merged 1 commit into
mainfrom
simonk/dev/manual-review
Oct 17, 2026
Merged

SimonKimDev merged 1 commit into
mainfrom
simonk/dev/manual-review

Conversation

@SimonKimDev

Copy link
Copy Markdown
Owner

The AI reviewer now runs only when you comment /review on a PR.

  • Deleted pr-review.md and its lock file (the pull_request trigger). pr-review-command.md is the only reviewer workflow now, renamed from PR re-review to PR review, and its prompt no longer assumes there was an earlier review.
  • README and the comment in shared/pr-review.md updated.

Also: .gitattributes keeps .github/workflows/**/*.md LF. gh aw compile on a Windows checkout (CRLF) warned end marker for unknown section "gpt-reviewer": gh-aw's ## agent: regexes end in [ \t]*$, which doesn't match before \r. The compiled lock then lacked GH_AW_INFO_SUB_AGENT_MODELS. The sub-agents themselves are read at runtime from the LF checkout on Linux runners, so reviews worked; this only affected compiling locally. With LF, compile has 0 warnings, and the lock matches one compiled in an LF-only checkout. Devel has the same issue.

Drop the pull_request-triggered pr-review workflow so reviews (and their
premium requests) only happen on demand. pr-review-command.md no longer
assumes an earlier review exists.

Keep gh-aw sources LF on Windows: gh aw compile doesn't recognize
"## agent:" markers that end in CRLF, so a Windows checkout compiled
without the sub-agents' model list (GH_AW_INFO_SUB_AGENT_MODELS) and
warned about an orphan end marker. Runs on GitHub weren't affected.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Copilot-Session: 90315f97-7c82-4bc1-9bf1-2f8460782554
@SimonKimDev
SimonKimDev marked this pull request as ready for review October 17, 2026 10:20
@SimonKimDev
SimonKimDev merged commit bedc78c into main Oct 17, 2026
2 checks passed
@SimonKimDev
SimonKimDev deleted the simonk/dev/manual-review branch October 17, 2026 10:20
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