Skip to content

fix: イベントトリガーをコスト最適な設定に修正 - #661

Merged
keito4 merged 1 commit into
mainfrom
fix/revert-claude-action-event-triggers
Apr 2, 2026
Merged

fix: イベントトリガーをコスト最適な設定に修正#661
keito4 merged 1 commit into
mainfrom
fix/revert-claude-action-event-triggers

Conversation

@keito4

@keito4 keito4 commented Apr 2, 2026

Copy link
Copy Markdown
Owner

概要

  • claude.yml から assigned イベントを除外(コスト最適化)
  • claude-code-review.yml から synchronize を除外(push毎のレビュー実行防止)
  • repo-maintenance の推奨値に reopened を追加して整合性を確保

変更内容

ファイル 変更
.github/workflows/claude.yml issues: [opened, assigned][opened]
.github/workflows/claude-code-review.yml [opened, synchronize, ready_for_review, reopened][opened, ready_for_review, reopened]
.claude/commands/repo-maintenance.md 推奨値を [opened, ready_for_review, reopened] に更新

テスト

  • ✅ pre-commit フック: Format, Lint, Test 通過
  • ✅ 最新の main ブランチとマージ済み

🤖 Generated with Claude Code

Summary by CodeRabbit

Chores

  • Optimized GitHub workflow automation triggers to refine code review and issue tracking processes.
  • Updated workflow configuration guidance to reflect trigger event changes.

claude.yml から assigned イベントを除外し opened のみに戻す。
claude-code-review.yml から synchronize を除外しpush毎のレビュー実行を防止。
repo-maintenance の推奨値に reopened を追加。

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

coderabbitai Bot commented Apr 2, 2026

Copy link
Copy Markdown

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 22deeeb5-630f-428f-9fbf-ba191c847fd9

📥 Commits

Reviewing files that changed from the base of the PR and between 6eae8c1 and 341cc7f.

📒 Files selected for processing (3)
  • .claude/commands/repo-maintenance.md
  • .github/workflows/claude-code-review.yml
  • .github/workflows/claude.yml

📝 Walkthrough

Walkthrough

GitHub Actions workflow triggers are narrowed: claude-code-review.yml removes synchronize from pull request event types, claude.yml restricts issue events to opened only, and documentation is updated to reflect the expanded exclusion filter recommendation.

Changes

Cohort / File(s) Summary
GitHub Actions Workflow Triggers
.github/workflows/claude-code-review.yml, .github/workflows/claude.yml
Narrowed trigger conditions: removed synchronize event from PR triggers in claude-code-review.yml, and restricted issue triggers to opened events only in claude.yml.
Documentation
.claude/commands/repo-maintenance.md
Updated recommended exclusion filter for synchronize trigger from [opened, ready_for_review] to [opened, ready_for_review, reopened].

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related issues

Possibly related PRs

Suggested labels

released

Poem

🐰 With triggers now tightened and workflows refined,
We prune the excess events from the bind,
Each workflow now waits for just the right call,
Configuration perfected—less noise, more control! ✨

🚥 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 in Japanese describes optimizing event triggers for cost efficiency, which directly aligns with the PR's core objective of narrowing workflow triggers to reduce unnecessary workflow executions and associated costs.
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
  • Commit unit tests in branch fix/revert-claude-action-event-triggers

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

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

@keito4

keito4 commented Apr 2, 2026

Copy link
Copy Markdown
Owner Author

🔍 AI Code Review (Local Hook)

Models: Codex (default) / Gemini (default)

🤖 Codex Review

指摘事項(重大度順)

  1. PR更新時にClaudeレビューが再実行されなくなる
    • 影響ファイル/行: .github/workflows/claude-code-review.yml:3-5
    • 内容: pull_requesttypes から synchronize を外したため、PRに新しいコミットがpushされてもレビューが走りません。初回レビュー後に修正コミットが積まれる運用だと、最新コードへのレビューが抜ける可能性が高いです(DX/品質リスク)。
    • 対応案: synchronize を戻す、または手動再実行の明確な運用(workflow_dispatch 追加など)を用意。

総合判定
patch is incorrect
理由: PR更新時の自動レビューが失われ、最新コードに対するレビュー保証が崩れるため。
信頼度: 0.62


⚠️ 修正が必要です

上記のレビューで問題が指摘されています。修正してからマージしてください。


🤖 Generated by post_pr_ai_review.py hook

@keito4
keito4 merged commit efb2bfa into main Apr 2, 2026
4 of 6 checks passed
@keito4
keito4 deleted the fix/revert-claude-action-event-triggers branch April 2, 2026 14:29
@github-actions

github-actions Bot commented Apr 2, 2026

Copy link
Copy Markdown
Contributor

🎉 This PR is included in version 1.108.2 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

released リリース済み

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant