Commit Graph

12 Commits

Author SHA1 Message Date
toyota 9af7f62adb refactor: remove program-oriented review sections from PR review prompt
Remove Code Quality, Security, Testing, and Performance sections
as this repository is a skill marketplace, not a program repository.
Focus solely on Skill Quality review using /claude-skill-review skill.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2026-02-05 14:36:59 +09:00
toyota 3dc0385a1c refactor: simplify PR review prompt by using claude-skill-review skill
Replace inline skill quality checks with /claude-skill-review skill
invocation. This reduces prompt size and leverages progressive
disclosure pattern for skill documentation.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2026-02-05 14:21:11 +09:00
toyota 073bfbfeb4 feat: add GitHub MCP inline comment support for PR reviews
GitHub MCPを利用してインラインコメント機能を追加:

変更内容:
- allowedToolsに`mcp__github_inline_comment__create_inline_comment`を追加
- プロンプトにインラインコメント投稿の指示を追加
- 具体的なファイルと行番号が特定できる問題はインラインコメントで指摘
- 全体レビューコメントではインラインコメントへの参照のみ記載

効果:
- レビューの可読性向上(問題箇所が明確)
- 全体レビューコメントの簡潔化
- コードレビューの効率化

参考:
https://github.com/anthropics/claude-code-action/blob/main/docs/solutions.md#automatic-pr-code-review

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
2025-11-11 23:31:32 +09:00
toyota 97f0d07b43 fix: address review feedback from automated PR review
レビュー指摘事項に対応:

Major問題の修正:
- max-turnsを20に変更 (.github/workflows/pr-review-vertex-ai.yml:341)
- フックパスを修正 (frontend_nextjs_development/hooks/biome-format/hooks.json:9)
  ${CLAUDE_PLUGIN_ROOT}/hooks/biome_format.sh
  → ${CLAUDE_PLUGIN_ROOT}/hooks/biome-format/biome_format.sh

Minor問題の修正:
- Biomeフックにエラーハンドリングを追加 (biome_format.sh:62-77)
  - npx実行の成功/失敗を判定
  - 失敗時はエラーメッセージを出力してexit 1

テスト実施:
- PR #9での動作確認完了
- レビュー投稿成功確認(Changes Requested)
- テスト結果をPR本文に追記

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
2025-11-11 23:12:26 +09:00
toyota 5555474855 fix: use --allowedTools in claude_args instead of additional_permissions
claude_argsで--allowedToolsを使用する正しい方法に修正:
- additional_permissionsパラメータを削除
- claude_argsに--allowedTools "Bash(gh:*)"を追加
- ghコマンド全般の実行を許可

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
2025-11-11 22:54:24 +09:00
toyota a556f2c199 fix: add gh command permission to additional_permissions
Claude Code Actionでghコマンドが権限エラーで拒否される問題を修正:
- additional_permissions に "Bash(gh:*)" を追加
- これによりgh pr view, gh pr review, gh issue createが実行可能に

Permission denials:
- gh pr view (PR情報取得)
- gh pr diff (差分取得)
- gh pr review (レビュー投稿)
- gh issue create (Issue作成)

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
2025-11-11 22:48:08 +09:00
toyota 39cf93c99a fix: remove unsupported --timeout option from claude_args
Claude Codeが--timeoutオプションをサポートしていないため削除:
- エラー: "unknown option '--timeout'"
- --max-turns 10のみでタイムアウトを制御

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
2025-11-11 22:33:38 +09:00
toyota 4804736eb4 fix: add timeout and paths-ignore to prevent workflow hanging
GitHub Actionsが1時間以上実行される問題を修正:
- タイムアウトを10分(600秒)に設定
- max-turnsを20から10に削減
- ワークフロー自体の変更をレビュー対象から除外
- ドキュメントファイルもレビュー対象から除外

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
2025-11-11 17:18:26 +09:00
toyota 06c4a25a41 refactor: embed SKILL.md logic into workflow and remove skill directory
SKILL.mdの内容をGitHub Actionsワークフローのプロンプトに直接埋め込み、
skillディレクトリを削除しました。これにより、GitHub Actions環境で
Skillツールの権限エラーを回避できます。

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
2025-11-11 16:08:15 +09:00
toyota 7dddab0c03 feat: sync root workflow with updated skill implementation
🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
2025-11-11 15:55:39 +09:00
toyota a31b84c0a1 ci: configure Vertex AI environment variables for PR review
🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
2025-11-11 15:18:58 +09:00
toyota f386605cdd ci: add Vertex AI automated PR review workflow 2025-11-11 15:10:01 +09:00