[PR #352] [MERGED] ci: consolidate Claude Code review workflows onto the OAuth token #97

New issue

Closed

opened 2026-06-08 08:29:55 +00:00 by ryangr0 · 0 comments

ryangr0 commented

2026-06-08 08:29:55 +00:00

Owner

📋 Pull Request Information

Original PR: https://github.com/webgrip/homelab-cluster/pull/352
Author: @Ryangr0
Created: 6/7/2026
Status: ✅ Merged
Merged: 6/7/2026
Merged by: @Ryangr0

Base: main ← Head: fix/consolidate-claude-workflows

📝 Commits (1)

e436694 ci: consolidate Claude Code review workflows onto the OAuth token

📊 Changes

3 files changed (+6 additions, -99 deletions)

View changed files

➖ .github/workflows/claude-code-review.yml (+0 -44)
📝 .github/workflows/claude-review.yml (+6 -5)
➖ .github/workflows/claude.yml (+0 -50)

📄 Description

Problem

The review job (custom claude-review.yml) fails on every PR with:

Either ANTHROPIC_API_KEY, CLAUDE_CODE_OAUTH_TOKEN, or workload identity federation is required

Root cause: claude-review.yml referenced secrets.ANTHROPIC_API_KEY, but the only Claude secret in the repo is CLAUDE_CODE_OAUTH_TOKEN (no ANTHROPIC_API_KEY exists). On top of that, three overlapping Claude workflows had accumulated — claude.yml and claude-code-review.yml (stock templates, OAuth token) duplicated the custom claude-review.yml, so every PR was reviewed twice and every @claude handled twice.

Change

Point claude-review.yml at the secret that actually exists: anthropic_api_key → claude_code_oauth_token: ${{ secrets.CLAUDE_CODE_OAUTH_TOKEN }} (both the review and mention jobs), and update the stale header comment.
Delete the two redundant stock workflows: claude.yml, claude-code-review.yml.

claude-review.yml remains the single Claude workflow: homelab-aware automated manifest review (Renovate-focused) on PRs + on-demand @claude on comments.

Note: this fixes the failing review check going forward. PR #351 will go green once it picks up this workflow (rebase/merge main), or it can be merged as-is — the failing bot-review job doesn't block the Flux validation checks.

🤖 Generated with Claude Code

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/webgrip/homelab-cluster/pull/352 **Author:** [@Ryangr0](https://github.com/Ryangr0) **Created:** 6/7/2026 **Status:** ✅ Merged **Merged:** 6/7/2026 **Merged by:** [@Ryangr0](https://github.com/Ryangr0) **Base:** `main` ← **Head:** `fix/consolidate-claude-workflows` --- ### 📝 Commits (1) - [`e436694`](https://github.com/webgrip/homelab-cluster/commit/e436694cba3e928474528b068b3de603ada8ca93) ci: consolidate Claude Code review workflows onto the OAuth token ### 📊 Changes **3 files changed** (+6 additions, -99 deletions) <details> <summary>View changed files</summary> ➖ `.github/workflows/claude-code-review.yml` (+0 -44) 📝 `.github/workflows/claude-review.yml` (+6 -5) ➖ `.github/workflows/claude.yml` (+0 -50) </details> ### 📄 Description ## Problem The `review` job (custom `claude-review.yml`) fails on every PR with: > Either ANTHROPIC_API_KEY, CLAUDE_CODE_OAUTH_TOKEN, or workload identity federation is required Root cause: `claude-review.yml` referenced `secrets.ANTHROPIC_API_KEY`, but the only Claude secret in the repo is `CLAUDE_CODE_OAUTH_TOKEN` (no `ANTHROPIC_API_KEY` exists). On top of that, three overlapping Claude workflows had accumulated — `claude.yml` and `claude-code-review.yml` (stock templates, OAuth token) duplicated the custom `claude-review.yml`, so every PR was reviewed twice and every `@claude` handled twice. ## Change - Point `claude-review.yml` at the secret that actually exists: `anthropic_api_key → claude_code_oauth_token: ${{ secrets.CLAUDE_CODE_OAUTH_TOKEN }}` (both the `review` and `mention` jobs), and update the stale header comment. - Delete the two redundant stock workflows: `claude.yml`, `claude-code-review.yml`. `claude-review.yml` remains the single Claude workflow: homelab-aware automated manifest review (Renovate-focused) on PRs + on-demand `@claude` on comments. > Note: this fixes the failing `review` check going forward. PR #351 will go green once it picks up this workflow (rebase/merge main), or it can be merged as-is — the failing bot-review job doesn't block the Flux validation checks. 🤖 Generated with [Claude Code](https://claude.com/claude-code) --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

ryangr0

2026-06-08 08:29:55 +00:00