From a9eb0131a1190d8584fb3b27ed0ef8e14257575d Mon Sep 17 00:00:00 2001 From: Ivan Lapa Date: Thu, 10 Sep 2026 14:57:00 +0200 Subject: [PATCH] ci: remove Claude Code review workflows MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit 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) --- .github/workflows/claude-review.yml | 106 ---------------------------- .github/workflows/claude.yml | 33 --------- 2 files changed, 139 deletions(-) delete mode 100644 .github/workflows/claude-review.yml delete mode 100644 .github/workflows/claude.yml diff --git a/.github/workflows/claude-review.yml b/.github/workflows/claude-review.yml deleted file mode 100644 index 1829200..0000000 --- a/.github/workflows/claude-review.yml +++ /dev/null @@ -1,106 +0,0 @@ -name: Claude Code Review Pipeline - -on: - pull_request: - types: [opened, synchronize, reopened] - -# Cancel previous runs for the same PR when new commits are pushed -concurrency: - group: claude-review-${{ github.event.pull_request.number }} - cancel-in-progress: true - -jobs: - # ───────────────────────────────────────────── - # Detect which parts of the codebase changed - # ───────────────────────────────────────────── - detect-changes: - runs-on: ubuntu-latest - outputs: - client: ${{ steps.filter.outputs.client }} - server: ${{ steps.filter.outputs.server }} - steps: - - uses: actions/checkout@v4 - - uses: dorny/paths-filter@v3 - id: filter - with: - filters: | - client: - - 'client/**/*.{js,jsx,ts,tsx,css}' - server: - - 'server/**/*.{js,ts}' - - # ───────────────────────────────────────────── - # React Reviewer — runs only if client/ changed - # ───────────────────────────────────────────── - review-react: - needs: detect-changes - if: needs.detect-changes.outputs.client == 'true' - runs-on: ubuntu-latest - permissions: - contents: read - pull-requests: write - steps: - - uses: actions/checkout@v4 - with: - fetch-depth: 0 - - - name: React Architecture Review - uses: anthropic/claude-code-action@v1 - with: - anthropic_api_key: ${{ secrets.ANTHROPIC_API_KEY }} - use_sticky_comment: true - prompt: | - Read and follow the agent definition at `.claude/agents/react-reviewer.md`. - Load any skills referenced in its frontmatter (e.g. `react-best-practices`). - Apply the agent's workflow, rules, and output format to review the PR diff. - Only review changed files in `client/`. Ignore server changes entirely. - - # ───────────────────────────────────────────── - # Backend Reviewer — runs only if server/ changed - # ───────────────────────────────────────────── - review-backend: - needs: detect-changes - if: needs.detect-changes.outputs.server == 'true' - runs-on: ubuntu-latest - permissions: - contents: read - pull-requests: write - steps: - - uses: actions/checkout@v4 - with: - fetch-depth: 0 - - - name: Backend Architecture Review - uses: anthropic/claude-code-action@v1 - with: - anthropic_api_key: ${{ secrets.ANTHROPIC_API_KEY }} - use_sticky_comment: true - prompt: | - Read and follow the agent definition at `.claude/agents/backend-reviewer.md`. - Load any skills referenced in its frontmatter (e.g. `express-best-practices`). - Apply the agent's workflow, rules, and output format to review the PR diff. - Only review changed files in `server/`. Ignore client changes entirely. - - # ───────────────────────────────────────────── - # Security Reviewer — always runs on all changes - # ───────────────────────────────────────────── - review-security: - runs-on: ubuntu-latest - permissions: - contents: read - pull-requests: write - steps: - - uses: actions/checkout@v4 - with: - fetch-depth: 0 - - - name: Security Review - uses: anthropic/claude-code-action@v1 - with: - anthropic_api_key: ${{ secrets.ANTHROPIC_API_KEY }} - use_sticky_comment: true - prompt: | - Read and follow the agent definition at `.claude/agents/security-reviewer.md`. - Also load the security skill at `.claude/skills/security/` if it exists. - Apply the agent's workflow, OWASP checks, and output format to review the PR diff. - Review ALL changed files for security vulnerabilities. diff --git a/.github/workflows/claude.yml b/.github/workflows/claude.yml deleted file mode 100644 index 5b6691c..0000000 --- a/.github/workflows/claude.yml +++ /dev/null @@ -1,33 +0,0 @@ -name: Claude Code Action Official - -on: - issues: - types: [opened, labeled] - issue_comment: - types: [created] - pull_request_review_comment: - types: [created] - pull_request_review: - types: [submitted] - -jobs: - claude: - if: | - (github.event_name == 'issues' && github.event.label.name == 'claude') || - (github.event_name == 'issue_comment' && contains(github.event.comment.body, '@claude')) || - (github.event_name == 'pull_request_review_comment' && contains(github.event.comment.body, '@claude')) || - (github.event_name == 'pull_request_review' && contains(github.event.review.body, '@claude')) - runs-on: ubuntu-latest - permissions: - contents: write - pull-requests: write - issues: write - steps: - - uses: actions/checkout@v4 - with: - fetch-depth: 1 - - - name: Claude Code Action Official - uses: anthropic/claude-code-action@v1 - with: - anthropic_api_key: ${{ secrets.ANTHROPIC_API_KEY }}