feat: enhance team protection settings with security features

[keito4]

Summary

• デフォルトのマージ方法を squash から merge commit のみに変更
• ブランチ保護に CODEOWNERS レビュー必須を追加
• 自動マージを無効化（レビュー後の手動マージを必須に）
• Secret scanning と push protection を有効化
• Private vulnerability reporting を有効化
• ドキュメントを新しい設定内容に更新

Changes

スクリプト (script/setup-team-protection.sh)

設定	変更内容
デフォルトマージ方法	squash → merge
allow_auto_merge	false を追加
require_code_owner_reviews	false → true
secret_scanning	新規追加（有効化）
secret_scanning_push_protection	新規追加（有効化）
private_vulnerability_reporting	新規追加（有効化）

ドキュメント (.claude/commands/setup-team-protection.md)

• ブランチ保護ルールに「CODEOWNERSレビュー必須」「古いレビューの自動却下」を追加
• リポジトリ設定に「自動マージ無効」を追加
• セキュリティ設定を更新

Test plan

• bash script/setup-team-protection.sh --dry-run でエラーなく実行できること
• ドキュメントの内容がスクリプトの動作と一致していること

🤖 Generated with Claude Code

Summary by CodeRabbit

• New Features

  • Configurable merge method selection with multiple strategies available.
  • Enhanced security protections: Dependabot security updates, secret scanning push protection, and private vulnerability reporting.
  • Code owner reviews now required for branch protection.

• Documentation

  • Expanded with merge configuration examples and future enhancement roadmap.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Walkthrough

Updates documentation and shell script to introduce configurable merge method options (merge, squash, rebase, all), enhance security features including secret scanning push protection and private vulnerability reporting, enable CODEOWNERS review requirements for branch protection, and outline future enhancement possibilities.

Changes

Cohort / File(s)	Summary
Documentation & Configuration .claude/commands/setup-team-protection.md	Adds merge method configuration with multiple strategy options and defaults; introduces CODEOWNERS review requirement; updates security settings to include Dependabot, secret scanning push protection, and private vulnerability reporting; includes new usage script examples and "future enhancements" section outlining planned expansions.
Script Implementation script/setup-team-protection.sh	Introduces MERGE_METHOD global variable with CLI option --merge-method; adds case statement logic to dynamically derive merge commit permissions based on selected strategy; enables private vulnerability reporting and configures secret scanning via API patch; updates branch protection to require CODEOWNERS reviews; enhances status messaging and help documentation.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

• feat: add setup-team-protection command for repository security #237: Extends the same setup script with MERGE_METHOD variable, new CLI option, and updated security/branch-protection settings implementation.

Poem

🐰 Branch protection now stands tall,
With merge methods—we've got 'em all!
Squash, rebase, or merge so clean,
CODEOWNERS guard the pristine scene.
Security checks now push back threats,
Our repository's finest yet! 🌿

✨ Finishing touches

• 📝 Generate docstrings

---

📜 Recent review details

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between ff4bfa8 and d7d8af0.

📒 Files selected for processing (2)

• .claude/commands/setup-team-protection.md
• script/setup-team-protection.sh

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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

[claude]

PR Review: Team Protection設定の強化

このPRについて、コード品質、セキュリティ、ドキュメントの観点からレビューを実施しました。

---

✅ 良い点

セキュリティ強化

• Secret scanningとpush protectionの有効化は重要なセキュリティ改善です
• Private vulnerability reportingの追加により、脆弱性報告プロセスが改善されます
• CODEOWNERSレビュー必須化により、コードレビューの品質が向上します

柔軟性の向上

• マージ方法を選択可能にする設計は優れています
• デフォルトをmergeに変更することで、コミット履歴の保持が容易になります
• 自動マージ無効化により、手動承認が必須になります

コード品質

• set -euo pipefailによるエラーハンドリングは適切です
• 引数パースのロジックが明確で、拡張性があります
• DRY_RUNモードの実装により、安全なテストが可能です

---

⚠️ 改善提案

1. デフォルト値の不整合

ヘルプメッセージ(line 20)ではdefault: squashですが、コード(line 38)ではMERGE_METHOD=mergeとなっています。
推奨: ヘルプメッセージをdefault: mergeに修正してください。

2. evalの使用

script/setup-team-protection.sh:282でevalを使用しています。
推奨: セキュリティリスクを避けるため、evalを使わずに直接gh apiコマンドを実行する方式に変更してください。

3. ドキュメントの不整合

.claude/commands/setup-team-protection.md:88ではrequire_code_owner_reviews=falseとなっていますが、実際のスクリプト(line 200)ではtrueとなっています。
推奨: ドキュメント例をtrueに更新してください。

4. Test Planの未完了

PR本文のTest planが未チェックです。
推奨: マージ前にこれらのテストを実施し、チェックを完了させてください。

---

📊 テストカバレッジ

このスクリプトには自動テストがありません。
推奨: --dry-runモードでの実行結果を検証するテストケースの追加を検討してください。

---

🎯 総合評価

判定: 条件付き承認（軽微な修正後にマージ可能）

必須（P0）

1. ヘルプメッセージのデフォルト値を修正
2. ドキュメント例のrequire_code_owner_reviewsをtrueに修正
3. Test planの実施とチェック完了

推奨（P1）

4. evalを使わない実装に変更
5. Secret scanningエラーメッセージの改善

将来的な改善（P2）

6. 自動テストの追加

🤖 Generated by Claude Code

[github-actions]

🎉 This PR is included in version 1.44.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀