Block a user
docs: add rule for when @impl functions earn their own @doc
docs: add rule for when @impl functions earn their own @doc
Clean consolidation. Pattern 10 now owns the full rule, cross-refs are minimal, decision test is concrete and actionable. Examples are well-chosen.
Make release asset uploads idempotent
Distinguish 404 from other errors in GetAllFilesInPath
rodin
deleted branch fix/8-10-error-handling-idempotent-upload from rodin/review-bot
2026-05-02 17:07:23 +00:00
fix: distinguish 404 in GetAllFilesInPath, make uploads idempotent
rodin
pushed to fix/8-10-error-handling-idempotent-upload at rodin/review-bot
2026-05-02 17:02:06 +00:00
rodin
pushed to fix/8-10-error-handling-idempotent-upload at rodin/review-bot
2026-05-02 16:58:43 +00:00
fix: distinguish 404 in GetAllFilesInPath, make uploads idempotent
rodin
pushed to fix/8-10-error-handling-idempotent-upload at rodin/review-bot
2026-05-02 16:50:47 +00:00
rodin
created branch fix/8-10-error-handling-idempotent-upload in rodin/review-bot
2026-05-02 16:50:47 +00:00
fix: remove worst-wins escalation logic
Escalation deadlock risk with shared tokens
Separate token per reviewer role (eliminate worst-wins)
Improve test coverage and add end-to-end integration tests in CI
fix: remove worst-wins escalation logic
Addressed both findings:
- Shared-token mode now skips ALL update logic — no PATCH, no supersede, just always POST fresh. This prevents clobbering a sibling's review body or state. The…