Revert review Code Triggered by labeled event

[theMickster]

🎟️ Tracking

📔 Objective

Refactor pull request trigger types to those supported by Anthropic's Claude Code

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 7.04%. Comparing base (32241c1) to head (ea3d726).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files

@@          Coverage Diff          @@
##            main    #250   +/-   ##
=====================================
  Coverage   7.04%   7.04%           
=====================================
  Files         49      49           
  Lines       1818    1818           
  Branches      99      99           
=====================================
  Hits         128     128           
  Misses      1685    1685           
  Partials       5       5           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[claude]

⚠️ IMPORTANT: Revert removes ready_for_review trigger, creating gap in review workflow

Details and fix

This revert changes the trigger from the original [opened, synchronize, reopened, ready_for_review] to [opened, synchronize, reopened], removing the ready_for_review event.

Impact:

• Draft PRs marked "ready for review" will not trigger the code review workflow
• Reviews will only run when new commits are pushed (synchronize)
• Creates a gap where PRs can bypass automated review

Execution path demonstrating the issue:

1. Developer creates draft PR → no review (correct, draft PRs shouldn't be reviewed)
2. Developer marks PR as "ready for review" → no review (MISSING - should trigger)
3. Only when next commit is pushed will review workflow run

Recommended fix:

Suggested change

	types: [opened, synchronize, reopened]
	types: [opened, synchronize, reopened, ready_for_review]

This restores the original behavior while correctly reverting the labeled trigger that was causing issues.

Reference: GitHub Actions pull_request events

[claude]

Overall Assessment: REQUEST CHANGES

This PR reverts the workflow trigger from [opened, labeled] back to [opened, synchronize, reopened]. However, the revert inadvertently removes the ready_for_review trigger that was present in the original configuration before PR #249.

Code Review Details

• ⚠️ IMPORTANT: Revert removes ready_for_review trigger, creating gap in review workflow
  • .github/workflows/review-code.yml:5

---

🤖 Generated with Claude Code

Review powered by Bitwarden Code Review Agent • Feedback