We currently have separate Actions for initial PR-open vs comment-triggered LLM reviews using Claude.
There's some repeated prompt configuration between them, but we're also missing a safe+smarter way to enable an initial review when a PR is first marked as ready for review, and it seems our comment-triggering mechanism is having trouble being the first LLM commenter (which is the only option for PRs from before the Claude review Actions were created, and also for any PRs that are initially opened as drafts).
I suspect we can consolidate the two Actions into a single one, and guard it with a check that allows a single automated run when first opened, reopened, or marked as ready for review, and adjusts the prompt context to be conditional on whether there were previous review comments (in general, and specifically from Claude).
We currently have separate Actions for initial PR-open vs comment-triggered LLM reviews using Claude.
There's some repeated prompt configuration between them, but we're also missing a safe+smarter way to enable an initial review when a PR is first marked as ready for review, and it seems our comment-triggering mechanism is having trouble being the first LLM commenter (which is the only option for PRs from before the Claude review Actions were created, and also for any PRs that are initially opened as drafts).
I suspect we can consolidate the two Actions into a single one, and guard it with a check that allows a single automated run when first opened, reopened, or marked as ready for review, and adjusts the prompt context to be conditional on whether there were previous review comments (in general, and specifically from Claude).