Luijaa/web-template(PR事前チェック)

プルリクエストを作成・更新する前に、make check、シークレットスキャン、ドキュメント、差分レビューなど、PR前の全チェックリストを実行します。PR作成前や作業の準備ができているか確認された際に使用します。

web-template(PR事前チェック) とは?

このスキルは、プルリクエストを作成または更新する前に、プロジェクトの品質基準を自動的に満たしていることを確認するための包括的なチェックリストを提供します。リポジトリの構造やスタックに応じて、`make check`、`make e2e`、`make dast` などのターゲットを条件付きで実行し、結果をテーブルで報告します。また、シークレットの漏洩、不要なログ、依存関係のライセンス、コミットメッセージやPR説明の規約準拠、UIの国際化対応など、12の詳細なチェック項目を網羅しています。Claude Code、Cursor、Codex などのAIエージェント向けに設計されており、各チェックの結果(合格/不合格)と根拠を明確に示すことで、開発者が安心してPRを発行できるように支援します。

対応✓Claude Code~Codex CLI~Cursor
npx skills add https://github.com/Luijaa/web-template/tree/HEAD/.agents/skills/pre-pr-check

Installed? Explore more 生産性&コラボレーション skills: steipete/gemini, steipete/gh-issues, steipete/skill-creator · View all 6 →

お気に入りのAIに質問する

このエージェントスキルを事前に読み込んだ状態で新しいチャットを開きます。

ドキュメント

PR事前チェック (Pre-PR check)

すべての項目を実行し、合格/不合格を表形式で報告します。不合格の場合はその根拠(コマンドと簡潔な結果)も示します。

  1. make check — すべてのターゲット(docs-check、lint + SAST、test、build、license)が合格していること。ただし、以下の2つのケースは例外:

    • ドキュメントのみのPR(docs(design): …、ADR、docs):make docs-check が合格すれば十分。
    • スタックが未選択のリポジトリ(ターゲットが not configured と表示 — ADR-0001 が未承認):そのターゲットは N/A(未設定) と報告する。「合格」とは扱わず、また停止理由にもしない。レシピを exit 0 で合格させるように変更してはいけない。

    また、make e2e はユーザーに見える部分を変更した場合(e2e テストを追加・変更)、make dast はミドルウェア、ヘッダー、Cookie、エラーページ、新しいルートに触れた場合に実行する。

  2. git diff origin/main...HEAD --stat — 変更されたファイルが作業対象のみであることを確認する。ビルド出力、.env、個人ファイルなどが含まれていないこと。

  3. 一時ログの消し忘れ: git diff origin/main...HEAD | grep -n '\[DBG-' — 何も見つからないこと(fix-bug の項目3)。

  4. 差分内のシークレットを検出: git diff origin/main...HEAD | grep -nEi '(api[_-]?key|secret|token|password|BEGIN .*PRIVATE KEY)' — 見つかった場合は停止し報告する。

  5. 新しい環境変数はすべて .env.example に追加されていること。

  6. 追加・アップグレードされた依存関係(ロックファイル、go.mod、Dockerfile の FROM の差分を確認):バージョンとライセンスが docs/dependencies.md と一致し、PR説明に記載されていること。

  7. 新しいマイグレーション(存在する場合):実行可能でロールバック可能であること。

  8. ドキュメント:README、docs、ADR(docs/adr/)が変更に応じて更新されていること。ドメインの新しい用語は CONTEXT.md に記載され、コード内の名前が CONTEXT.md と一致していること。

  9. review スキルに従ってレビューする(Claude Code:サブエージェント reviewer;他のエージェント:.agents/skills/review/SKILL.md を読み、自身の差分を確認する) — 「修正必須」の項目をすべて修正してから先に進む。

  10. コミットメッセージは Conventional Commits 形式であり、ブランチ名は main ではないこと。

  11. PR説明には Closes #<issue> を含めること(issue がない場合は先に作成する)。さらに ## CHANGELOG セクションが必要(カテゴリ + 1行の日本語(ユーザー視点)または -)。CHANGELOG.md 自体は編集しないこと(docs/release.md 参照)。

  12. UI に触れる場合:ハードコードされた文字列がないこと、すべての言語のキーが揃っていること、色の直値がないこと、th / en × light / dark の両方で見た目が正常であること(docs/ui.md 参照)。

いずれかの項目が不合格の場合、PRを開いてはいけません。先に修正するか、何がブロックしているのかを報告してください。

Individual skills in this repo

This repo contains 4 individual skills — each has its own dedicated page.

関連スキル