Skip to content

Simplify AI PR Review workflow by removing manual trigger#205

Merged
DiamondDagger590 merged 1 commit into
recodefrom
claude/cleanup-duplicate-workflows-FoxNr
Mar 13, 2026
Merged

Simplify AI PR Review workflow by removing manual trigger#205
DiamondDagger590 merged 1 commit into
recodefrom
claude/cleanup-duplicate-workflows-FoxNr

Conversation

@DiamondDagger590
Copy link
Copy Markdown
Owner

@DiamondDagger590 DiamondDagger590 commented Mar 13, 2026

Summary

This PR simplifies the AI PR Review workflow configuration by removing the two-workflow setup and consolidating into a single workflow that runs directly on pull requests.

Key Changes

  • Removed .github/workflows/pr-review-run.yml - The secondary workflow that was triggered via workflow_run event is no longer needed
  • Modified .github/workflows/pr-review.yml - Removed the workflow_dispatch trigger, keeping only the pull_request event trigger

Implementation Details

The previous setup used a two-workflow pattern where:

  1. A pull_request workflow would prepare review inputs and upload them as artifacts
  2. A separate workflow_run workflow would download those artifacts and run the actual review

This change consolidates the logic into a single workflow that triggers directly on pull request events. The workflow_dispatch option (manual trigger) has been removed as it's no longer needed with this simplified approach.

This reduces complexity while maintaining the core functionality of running AI-powered PR reviews on pull request creation and updates.

https://claude.ai/code/session_012q77afgp6s3GbRQGCjKFr1

Summary by CodeRabbit

  • Chores
    • Removed automated AI-powered PR review workflow
    • Disabled manual trigger option for PR reviews

…m pr-review.yml

pr-review-run.yml is superseded by ai-persona-review.yml, which handles
persona reviews triggered by workflow_run from pr-review.yml and also
supports manual dispatch via workflow_dispatch. Removing the redundant
file eliminates duplicate review runs on every PR.

The workflow_dispatch trigger added to pr-review.yml in PR #204 is not
needed there since manual invocation is already covered by
ai-persona-review.yml.
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Mar 13, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: 9ab3fb79-08ac-4e9a-b8ca-cebc20423d59

📥 Commits

Reviewing files that changed from the base of the PR and between 898bd5b and c349353.

📒 Files selected for processing (2)
  • .github/workflows/pr-review-run.yml
  • .github/workflows/pr-review.yml
💤 Files with no reviewable changes (2)
  • .github/workflows/pr-review.yml
  • .github/workflows/pr-review-run.yml

Walkthrough

The PR removes GitHub Actions workflows that implement automated AI PR reviews. The pr-review-run.yml file is completely deleted (113 lines), and the workflow_dispatch trigger is removed from pr-review.yml, leaving only pull_request event triggers.

Changes

Cohort / File(s) Summary
GitHub Actions Workflows
.github/workflows/pr-review-run.yml, .github/workflows/pr-review.yml
Deletion of complete AI PR review pipeline workflow and removal of manual dispatch trigger from PR review workflow, disabling automated review functionality.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

Possibly related PRs

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main change: removal of the workflow_dispatch manual trigger from pr-review.yml and deletion of the duplicate pr-review-run.yml workflow to consolidate the AI PR Review process.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch claude/cleanup-duplicate-workflows-FoxNr
📝 Coding Plan
  • Generate coding plan for human review comments

Comment @coderabbitai help to get the list of available commands and usage tips.

Tip

CodeRabbit can scan for known vulnerabilities in your dependencies using OSV Scanner.

OSV Scanner will automatically detect and report security vulnerabilities in your project's dependencies. No additional configuration is required.

@DiamondDagger590 DiamondDagger590 merged commit 7ce40cb into recode Mar 13, 2026
3 checks passed
@DiamondDagger590 DiamondDagger590 deleted the claude/cleanup-duplicate-workflows-FoxNr branch March 13, 2026 20:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants