Skip to content

create-pr スキルに PR の言語ルールとテンプレート運用を追加 - #6

Merged
leohara merged 1 commit into
mainfrom
chore/using-git-worktrees-rules
Apr 8, 2026
Merged

leohara merged 1 commit into
mainfrom
chore/using-git-worktrees-rules

Conversation

@leohara

@leohara leohara commented Apr 8, 2026

Copy link
Copy Markdown
Owner

概要

  • create-pr スキルに、commit message は英語、PR タイトルは日本語、PR 本文はテンプレート準拠という既定ルールを追加しました。
  • .github/pull_request_template.md を新規追加し、特定機能に依存しない汎用的な PR テンプレートを整備しました。
  • 対象領域:
    • src/ / app/ / lib/ などのアプリケーションロジック
    • API / IPC / バックエンド / 外部連携
    • UI / 画面表示 / 状態管理 / UX
    • データ保存 / DB / ファイル操作 / 権限 / バリデーション
    • tests/ / docs/ / CI / 開発者向け設定

背景

  • create-pr 実行時の commit message、PR タイトル、PR 本文の運用ルールを毎回手動で揃える必要がありました。
  • スキル側に既定値とテンプレート運用を持たせることで、PR 作成フローを安定化したい意図があります。
  • 関連する issue / 仕様 / 会話ログ / 障害報告
    • 会話ログ: create-pr に commit message は英語、PR タイトルは日本語、PR 本文はテンプレート準拠の指示を追加する依頼

変更内容

  • .codex/skills/create-pr/SKILL.md に repository defaults を追加し、言語ルールとテンプレート利用ルールを明文化しました。
  • .codex/skills/create-pr/agents/openai.yaml の default prompt に同じ既定ルールを反映しました。
  • .github/pull_request_template.md を追加し、対象領域・背景・変更内容・影響範囲・検証・レビュー観点・残課題を含む汎用テンプレートを定義しました。
  • どの機能・画面・API・ジョブ・フローに影響するか
    • Codex の create-pr スキルによる commit / push / PR 作成フロー
  • レビュワーに特に見てほしい観点
    • 明示的なユーザー指示がある場合に既定言語ルールより優先される構成になっているか
    • テンプレートが特定アプリ構成に寄りすぎず、別リポジトリでも再利用しやすいか
    • verification summary の書き方が現在のスキル運用と整合しているか

影響範囲

  • ユーザー影響
    • create-pr 利用時の commit message / PR タイトル / PR 本文の出力規約が揃います。
  • 開発者影響
    • .github/pull_request_template.md が新規追加され、今後の PR 本文のベースになります。

検証

  • 実施した確認内容
    • create-pr スキル差分、agent prompt 反映、PR テンプレート追加を確認
  • 実行したコマンドと結果
    • git diff --check: pass
    • pnpm run lint: pass
    • pnpm run type-check: pass
    • pnpm run test: pass (No test files found, exit code 0)
  • 未実施の確認があれば理由
    • pnpm run build: package.json に build script が定義されていないため未実施

レビュー観点

  • SKILL.md と agent default prompt の指示内容が矛盾していないか
  • PR テンプレートの粒度と項目名が汎用利用に十分か
  • build 未定義のリポジトリでこの検証 summary の扱いに問題がないか

残課題

  • 今回の PR で扱わなかった項目
    • code-change-verification 側のデフォルト plan に build 未定義時の扱いを組み込むこと
  • 必要なら次の対応内容
    • スキル群全体で PR 本文テンプレートの利用規約をさらに共通化すること

@leohara
leohara merged commit aa34beb into main Apr 8, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant