ci: remove broken self-built BugBot PR Review workflow (Cursor Bugbot covers it)#270
Merged
Conversation
The self-built 'BugBot PR Review' check (.github/workflows/claude-code-review.yml, anthropics/claude-code-action) has red-failed on every recent PR. After the ANTHROPIC_API_KEY was fixed and the action SHA was bumped in #268, it STILL crashes identically with an internal 'directory mismatch for directory .../post-buffered-inline-comments.ts ... indicates a bug' — the bug is present at every action version tried (1dc994ee and 51ea8ea7), it produces no useful review ('No buffered inline comments'), and the recurring spurious red obscures the real merge gates on every PR. It is redundant with the hosted Cursor Bugbot (cursor[bot]), which is the documented substantive AI reviewer and works. Delete the broken workflow. Unaffected: claude.yml (interactive @claude) and claude-code-fix.yml; the real merge gates (check/docker/security/e2e/GitGuardian). Docs: CLAUDE.md CI bullet corrected (was 'now WORKS' — false); KNOWN_ISSUES KI-027 marked resolved-by-removal. https://claude.ai/code/session_01VzJJTUcvzZgS9jN8aap9iv
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
CI: remove the broken self-built BugBot PR Review workflow
Follow-through on quieting the BugBot noise — the gentle fix didn't hold, so this is the option we lined up as the fallback ("drop the job, Cursor Bugbot is the real reviewer").
Why removal, not another patch
The self-built
BugBot PR Reviewcheck has red-failed on every recent PR. The history:directory mismatch … post-buffered-inline-comments.ts … indicates a bugcrash.anthropics/claude-code-actionSHA to fix it — and Phase 49: Vetup horse-upserted producer emit (+ placeholder animal-to-vetup workflow) #269 (the first PR after that merged) proved the identical crash persists at the new SHA too. The bug is version-independent, it produces no useful review ("No buffered inline comments"), and the recurring red obscures the real gates on every PR.It's redundant with the hosted Cursor Bugbot (
cursor[bot]), which is the documented substantive AI reviewer and works (it's caught real findings throughout — e.g. #252/#253/#256/#264). So the broken workflow is deleted.Change
.github/workflows/claude-code-review.yml.CLAUDE.mdCI bullet corrected (it claimed "self-built BugBot now WORKS" — false).KNOWN_ISSUESKI-027 marked resolved-by-removal.Unaffected
claude.yml(interactive @claude) andclaude-code-fix.ymlremain. Real merge gates —check/docker/security/e2e/ GitGuardian — untouched. Cursor Bugbot still reviews every PR.Note: this PR itself will show the BugBot red one last time (the deletion only takes effect once merged); after merge, no future PR runs it.
Final merge left to Richard / Freddie.
https://claude.ai/code/session_01VzJJTUcvzZgS9jN8aap9iv
Generated by Claude Code