feat(webhook): add @bot retry command + conversation framework #65

Merged
rcsheets merged 2 commits from feat/retry-command into main 2026-05-30 06:45:23 +00:00
Owner

Lets users say @bot retry on a PR with a failed review instead of
removing/re-adding the bot as reviewer. Introduces a small command
registry and a bot-conversation context (chronological thread of
bot comments + human mentions, plus the most recent primary event)
so future deterministic commands can drop in without re-deriving
state.

Co-Authored-By: Claude Opus 4.7 (1M context) noreply@anthropic.com

Lets users say @bot retry on a PR with a failed review instead of removing/re-adding the bot as reviewer. Introduces a small command registry and a bot-conversation context (chronological thread of bot comments + human mentions, plus the most recent primary event) so future deterministic commands can drop in without re-deriving state. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
feat(webhook): add @bot retry command + conversation framework
All checks were successful
ci / check (pull_request) Successful in 44s
6104c0d91b
Lets users say @bot retry on a PR with a failed review instead of
removing/re-adding the bot as reviewer. Introduces a small command
registry and a bot-conversation context (chronological thread of
bot comments + human mentions, plus the most recent primary event)
so future deterministic commands can drop in without re-deriving
state.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Collaborator

Automated review by pr-reviewer v0.28.4 | Full Review | Mistral Small | tracking id r-1a824f-ca7aad
This is an AI-generated review and may contain mistakes.

Status: Failed


Review failed. Tracking id r-1a824f-ca7aad — see logs for details.

*Automated review by [pr-reviewer](https://git.brooktrails.org/brooktrails/pr-reviewer) v0.28.4 | Full Review | Mistral Small | tracking id `r-1a824f-ca7aad`* *This is an AI-generated review and may contain mistakes.* **Status:** ❌ Failed --- Review failed. Tracking id `r-1a824f-ca7aad` — see logs for details.
Collaborator

Automated review by pr-reviewer v0.28.4 | Full Review | Mistral Small | tracking id r-1a8312-6cc534
This is an AI-generated review and may contain mistakes.

Status: Failed


Review failed. Tracking id r-1a8312-6cc534 — see logs for details.

*Automated review by [pr-reviewer](https://git.brooktrails.org/brooktrails/pr-reviewer) v0.28.4 | Full Review | Mistral Small | tracking id `r-1a8312-6cc534`* *This is an AI-generated review and may contain mistakes.* **Status:** ❌ Failed --- Review failed. Tracking id `r-1a8312-6cc534` — see logs for details.
Collaborator

Automated review by pr-reviewer v0.28.4 | Full Review | Mistral Small | tracking id r-1a83dc-d5e507
This is an AI-generated review and may contain mistakes.

Status: Failed


Review failed. Tracking id r-1a83dc-d5e507 — see logs for details.

*Automated review by [pr-reviewer](https://git.brooktrails.org/brooktrails/pr-reviewer) v0.28.4 | Full Review | Mistral Small | tracking id `r-1a83dc-d5e507`* *This is an AI-generated review and may contain mistakes.* **Status:** ❌ Failed --- Review failed. Tracking id `r-1a83dc-d5e507` — see logs for details.
fix(reviewer): surface backend error details in pod logs
All checks were successful
ci / check (pull_request) Successful in 44s
dd50b4eea3
Empty-body 502s from ingress proxies collapsed into "vllm 502: " or
"anthropic 502: " with no further context. The runner also never
logged primary-config errors — they were only visible on the
dashboard via review_events.error.

httpStatusError now appends url=, server=, content-type=, and
x-request-id= when present, replaces an empty body with <empty body>,
and caps long bodies. recordConfigResult logs primary failures at
error level and shadows at warn level so operators can correlate a
failed review with the upstream cause in the pod log.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Collaborator

Automated review by pr-reviewer v0.28.4 | Safety Check | Mistral Small | tracking id r-1a874c-68d68e
This is an AI-generated review and may contain mistakes.

Status: Failed


Review failed. Tracking id r-1a874c-68d68e — see logs for details.

*Automated review by [pr-reviewer](https://git.brooktrails.org/brooktrails/pr-reviewer) v0.28.4 | Safety Check | Mistral Small | tracking id `r-1a874c-68d68e`* *This is an AI-generated review and may contain mistakes.* **Status:** ❌ Failed --- Review failed. Tracking id `r-1a874c-68d68e` — see logs for details.
rcsheets deleted branch feat/retry-command 2026-05-30 06:45:23 +00:00
Sign in to join this conversation.
No reviewers
No labels
No milestone
No project
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
brooktrails/pr-reviewer!65
No description provided.