Skip to content

fix(ci): skip automated Claude review for Dependabot PRs - #33

Merged
Jamie-BitFlight merged 5 commits into
mainfrom
repair/dependabot-review
Nov 1, 2026
Merged

Jamie-BitFlight merged 5 commits into
mainfrom
repair/dependabot-review

Conversation

@Jamie-BitFlight

@Jamie-BitFlight Jamie-BitFlight commented Oct 11, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • Skip automated Claude reviews for Dependabot-authored pull requests without changing human-authored review behavior.
  • Use the PR author rather than the triggering actor so human reruns do not spend review tokens on dependency updates.
  • Replace the previous bot allow-list and Dependabot credential guidance; no new credentials or permissions are needed.
  • Dependency updates remain covered by CI, lint, unit tests, and Jira E2E tests.

Validation

  • Parsed workflow YAML and checked the Dependabot/human author and rerun-actor condition matrix.
  • git diff --check passed.
  • Exact head a9b4342 passes CI (typecheck, lint, build and dist parity), Unit Tests, Code Linting, Jira E2E and the unchanged human-review path. Dependabot skip semantics were checked with the author/rerun matrix; no bot-triggered run was created.

This supersedes the previous approach of enabling paid automated review for Dependabot. No merge or branch-protection changes are included.

@coderabbitai

coderabbitai Bot commented Oct 11, 2026 •

Copy link
Copy Markdown

Important

  • Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 8e1d292d-473a-4e40-aadd-60432d061413
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

Share

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 11, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-11T01:15:20.198915Z c6299e5 New commits
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 52b89ed7fd

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: c6299e5bcf

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread .github/workflows/README.md Resolved
@Jamie-BitFlight

Copy link
Copy Markdown
Contributor Author

why would we waste tokens on checking dependabot with claude code reviews

@Jamie-BitFlight Jamie-BitFlight changed the title fix(ci): allow Dependabot to receive automated review fix(ci): skip automated Claude review for Dependabot PRs Oct 30, 2026
Copilot AI balanced review requested due to automatic review settings November 1, 2026 09:20

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 Approval recommended

There are no active review issues for the change excluding Dependabot-authored pull requests from automated Claude review.

No open issues 🎉

What changed in this PR

This PR skips automated Claude review for Dependabot-authored pull requests while preserving existing review behavior for human-authored PRs.

Changes:

  • Adds a PR-author condition so Dependabot reviews remain skipped even on human-triggered reruns.
  • Clarifies Claude secret requirements and documents continued CI coverage for dependency updates.
File Description
.github/​workflows/​README.md Documents Dependabot review skipping and existing Claude secret requirements.
.github/​workflows/​claude-code-review.yml Excludes Dependabot-authored PRs from the automated review job.

Review effort: Balanced

@Jamie-BitFlight
Jamie-BitFlight merged commit c7534dc into main Nov 1, 2026
13 checks passed
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