Skip to content

ci: remove Claude Code review workflows - #27

Merged
burnjohn merged 1 commit into
mainfrom
chore/remove-claude-ci
Sep 10, 2026
Merged

burnjohn merged 1 commit into
mainfrom
chore/remove-claude-ci

Conversation

@burnjohn

Copy link
Copy Markdown
Owner

DevDigest's exported workflow (.github/workflows/devdigest.yml, merged in #26) now handles automated PR review, so the two Claude Code review workflows are redundant — every PR was being reviewed twice.

Changes

  • Delete .github/workflows/claude-review.yml
  • Delete .github/workflows/claude.yml

Deliberately kept

The agent definitions and skills under .claude/ (agents/*-reviewer.md, skills/security/) stay — they are still used when running Claude Code locally. Nothing outside the two deleted files references them from CI.

Also drops the need for the ANTHROPIC_API_KEY repository secret; OPENROUTER_API_KEY (used by DevDigest) is now the only one CI needs.

Note

This PR doubles as the first real end-to-end test of the DevDigest reviewer running in CI.

🤖 Generated with Claude Code

DevDigest's exported workflow (.github/workflows/devdigest.yml, merged in
#26) now covers automated PR review, so the two Claude Code review
workflows are redundant — every PR was getting reviewed twice.

Removes the workflows only. The agent definitions and skills under
.claude/ stay: they are still used when running Claude Code locally, and
nothing else references the deleted files.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>

@github-actions github-actions 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.

General Reviewer — Approved ✅

No findings. Looks good.

@greptile-apps

greptile-apps Bot commented Sep 10, 2026

Copy link
Copy Markdown

RetriggerConfidence Score: 4/5

The interactive issue and @claude assistance regression should be fixed or explicitly confirmed as intentionally retired before merging.

Findings

  1. P1 Interactive Claude triggers are removed ▶

Summary

  • Removes the client, server, and security review pipeline backed by ANTHROPIC_API_KEY.
  • Also removes the event-driven Claude workflow for issue labels and @claude interactions.
  • Leaves local Claude agent definitions and skills unchanged.

Reviews (1) · Last reviewed commit: "ci: remove Claude Code review workflows"

@burnjohn
burnjohn merged commit 5bd7216 into main Sep 10, 2026
2 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.

1 participant