einja-inc avatar

task-commit

コミット・プッシュを実行するSkill。docs/einja/steering/commit-rules.mdのルールに従い、分割コミットを実施。直接呼び出し可能(確認あり)、task-exec経由で

by einja-inc|Open Source

task-commit Skill: コミット・プッシュ実行エンジン

役割

変更をコミット・プッシュします。docs/einja/steering/commit-rules.md のコミットルールに厳格に従い、適切な粒度でコミットを分割します。

パラメータ

パラメータ必須説明
planFileいいえコミット対象のPlanファイルパス。指定された場合、そのPlanに関連する変更のみをコミット対象とし、Plan選択のAskUserQuestionをスキップする

呼び出し例:

  • Plan指定あり: einja-task-commit planFile=docs/plans/202603/20260313-feature-auth.plan.md
  • Plan指定なし: einja-task-commit(従来通りの動作)

Sandbox注意事項

すべての git / pnpm コマンドは dangerouslyDisableSandbox: true で実行すること。 sandbox が有効な環境では git の unlink・rename 等がブロックされるため、Bash tool 呼び出し時に必ずこのフラグを設定する。

実行手順(7ステップ)

ステップ1: 最新化(必要な場合のみ)

⚠️ 重要: 毎回 git pull --rebase を実行するのは非効率。リモートに新しいコミットがある場合のみ実行する。

手順

# 1. リモートの最新情報を取得(ローカルは変更しない)
git fetch origin

# 2. リモートとの差分を確認
git rev-list HEAD..origin/main --count
  • 差分が0の場合: 最新なので何もしない(次のステップへ)
  • 差分がある場合: 以下を実行
# ローカル変更を一時退避
git stash

# リモートの変更を取り込み
git pull --rebase

# ローカル変更を復元
git stash pop

コンフリクト発生時

  1. einja-conflict-resolver Skill の手順に従って解消
  2. 解消できない場合は以下を出力して終了:
## 🚀 コミット・プッシュ

### ステータス: ❌ FAILURE

**エラー**: git pull --rebase でコンフリクトが発生し、解消できませんでした。

手動でコンフリクトを解決してください。

\`\`\`
[コンフリクト詳細]
\`\`\`

ステップ2: 変更確認

  1. git status で変更ファイルを確認
  2. git diff で差分内容を確認
  3. 変更がない場合: 以下を出力して正常終了
## 🚀 コミット・プッシュ完了

### ステータス: ✅ SUCCESS(変更なし)

コミット対象の変更がありませんでした。

ステップ3: コミット分割方針の決定

⚠️ 重要: docs/einja/steering/commit-rules.md の「コミットの分割方針」を必ず参照すること。

3-A: コミット対象Planの決定

以下の優先順位で、コミット対象のPlanを決定する:

条件動作AskUserQuestion
planFile パラメータ指定あり指定されたPlanに関連する変更のみをコミット対象とする不要
planFile 指定なし+変更に関連するPlan 1つ自動でそのPlanを使用不要
planFile 指定なし+変更に関連するPlan 複数AskUserQuestionでPlan選択を確認必要(下記参照)
planFile 指定なし+関連PlanなしPlan分割なし。通常の分割基準(3-C)で判定不要(3-Dで分割案確認)

Plan特定方法: git diff --stat の変更ファイル一覧と、docs/plans/ 配下のPlanファイル内容を突合し、関連するPlanを特定する。

複数Plan時のAskUserQuestion(3-A該当時のみ)
AskUserQuestion:
  question: |
    変更が複数のPlanに跨っています。どのPlanをコミット対象としますか?

    検出されたPlan:
    - docs/plans/202603/20260313-feature-auth.plan.md(関連ファイル: 5個)
    - docs/plans/202603/20260313-fix-logging.plan.md(関連ファイル: 3個)
  header: "コミット対象Plan選択"
  options:
    - label: "すべてのPlanを含める"
      description: "全Planの変更をPlan単位で分割コミットする"
    - label: "docs/plans/202603/20260313-feature-auth.plan.md"
      description: "このPlanの関連変更のみコミット(他は次回)"
    - label: "docs/plans/202603/20260313-fix-logging.plan.md"
      description: "このPlanの関連変更のみコミット(他は次回)"
  multiSelect: true

3-B: Planタスク単位の分割

前提: 3-AでPlanが特定された場合のみ実行。Planなしの場合は3-Cへスキップ。

コミット対象Planが決定後、Planの「タスク概要」セクションに記載されたタスクに対応する実装ファイルセット単位でコミットを分割する。

分割手順
  1. Planファイルの「タスク概要」セクションを読み、タスク一覧を把握する
  2. git diff --stat の変更ファイル一覧を各タスクにマッピングする
  3. タスク単位で別々のコミットを作成する(1タスク = 1コミットが基本)
  4. 複数タスクにまたがる共通変更は、最も関連の強いタスクのコミットに含める
  5. Planファイル自体は実装コミットに含める(Planと実装を別コミットにしない)
  6. Planに紐づかない独立した変更がある場合は別コミットにまとめる
マッピングルール
状況対応
1タスクの変更ファイルが明確そのタスク名をコミットメッセージに反映
複数タスクが同一ファイルを変更変更内容の主目的で判断し、1つのタスクに帰属させる
Planファイル(docs/plans/実装コミットに含める(Planと実装を分けない)
タスクに対応しない変更(設定変更等)別コミット(chore: or docs:)にまとめる
タスクが細かすぎる(1-2行の変更)関連タスクとまとめて1コミットにしてよい
PR分割の提案

1つのPlanに関連する変更が 30ファイル以上 または 1000行以上 の場合、コミット分割ではなくPR自体を分けることをユーザーに提案する。機械的変更(リネーム、フォーマッタ適用等)のみの場合は例外として分割不要。

提案時のAskUserQuestion例:

Plan「{plan名}」の変更が{N}ファイル / {M}行あります。
レビュー負荷が高いため、PRを分割することを推奨します。

選択肢:
- PR分割: Plan単位でブランチ・PRを分ける
- 1PRにまとめる: 全変更を1つのPRでコミット

3-C: 通常の分割基準

Planありの場合: 3-Bのタスク単位分割が基本。以下の基準は補助的に適用する。 Planなしの場合: 以下の基準で分割を検討する。

  1. 異なる目的や種類の変更

    • ソースコードの変更とドキュメントの更新
    • 機能追加と設定ファイルの変更
    • リファクタリングとバグ修正
  2. 異なるコンポーネントやモジュールの変更

    • 複数のマイクロサービスやコンポーネントの変更
    • フロントエンドとバックエンドの変更
  3. レビューの容易さを考慮した分割

    • 大規模な変更は小さな論理的なまとまりに分割
    • テストコードの追加・更新は実装とは別のコミット

3-D: コミット分割案の確認

最終的なコミット分割案をAskUserQuestionツールで確認する。

⚠️ 重要: question パラメータに具体的なコミット分割内容を必ず記載すること。「コミット分割案を確認してください」のような抽象的な質問は禁止

AskUserQuestion:
  question: |
    以下のコミット分割案で実行してよろしいですか?

    【コミット1】feat: ユーザー認証機能の追加
    対象: src/auth/login.ts, src/auth/logout.ts, src/auth/middleware.ts

    【コミット2】test: 認証機能のテスト追加
    対象: src/auth/__tests__/login.test.ts, src/auth/__tests__/logout.test.ts
  header: "分割案"
  options:
    - label: "承認"
      description: "この分割案でコミットを実行"
    - label: "1コミットにまとめる"
      description: "すべての変更を1つのコミットにまとめる"

question記載必須項目:

  • 各コミットのメッセージ(プレフィックス付き)
  • 各コミットの対象ファイル一覧

ユーザーが「Other」で修正指示を出した場合は、その指示に従って分割案を修正し、再度確認を取る。


ステップ4: 品質チェック

einja-task-exec Skill経由での呼び出しの場合

einja-task-exec Skill経由でQA合格後に呼び出されるため、品質チェック(lint/typecheck/test/build)はスキップします。

QAフェーズで既に実行済みのため、重複実行は不要です。

直接呼び出しの場合

コミット前に pnpm prepush を実行して以下のチェックを行います:

  1. lint: lint-staged による変更ファイルの lint チェック
  2. typecheck: TypeScript の型チェック
  3. test: 変更ファイルに関連するテストのみ実行(vitest related --run)

pnpm prepush が失敗した場合、pnpm install してリトライする(コマンド例参照)。リトライも失敗した場合はエラー内容を報告して終了。


ステップ5: コミット実行

合意した分割方針に従って順次コミットを実行。

コミットメッセージ形式

docs/einja/steering/commit-rules.md に従い:

  • プレフィックス: feat:, fix:, docs:, style:, refactor:, test:, chore:
  • 言語: 日本語
  • 形式: 1行目に概要、2行目以降に詳細

コミットコマンド

einja-task-exec Skill経由での呼び出しの場合(QA済み):

git add src/auth/login.ts src/auth/logout.ts && git commit -m "$(cat <<'EOF'
feat: ユーザー認証機能の追加

- JWT認証の実装
- ログイン・ログアウトエンドポイントの追加
- 認証ミドルウェアの実装
EOF
)"

直接呼び出しの場合(prepush実行):

git add src/auth/login.ts src/auth/logout.ts && (pnpm prepush || (pnpm install && pnpm prepush)) && git commit -m "$(cat <<'EOF'
feat: ユーザー認証機能の追加

- JWT認証の実装
- ログイン・ログアウトエンドポイントの追加
- 認証ミドルウェアの実装
EOF
)"

ステップ6: プッシュ実行

  1. git push を実行
  2. 結果を出力

ステップ7: CI確認

プッシュ完了後、_einja-ci-check インナーSkillの手順に従ってCI確認・自動修正を実行する。

スキップ条件:

  • PRが存在しないかつmainブランチ直プッシュでもない(CI自体が起動しない場合のみ)

実行方法:

  1. .claude/skills/_einja-ci-check/SKILL.md の手順に従う
  2. パラメータはデフォルト値を使用(maxRetries: 2, timeout: 300

出力形式

成功時

## 🚀 コミット・プッシュ完了

### コミットサマリー
- **コミット数**: {count}個
- **変更ファイル数**: {files}個

### コミット一覧
| # | メッセージ | ファイル数 |
|---|----------|----------|
| 1 | feat: ユーザー認証機能の追加 | 3 |
| 2 | test: 認証機能のテスト追加 | 2 |

### プッシュ結果
\`\`\`
[git push の出力]
\`\`\`

### ステータス: ✅ SUCCESS

失敗時

## 🚀 コミット・プッシュ

### ステータス: ❌ FAILURE

**エラー**: [エラーの種類]

\`\`\`
[エラー詳細]
\`\`\`

[推奨される対処方法]

エラーハンドリング

エラー種別対処
git pull コンフリクトeinja-conflict-resolver Skill の手順に従って解消を試行
コンフリクト解消失敗報告して終了、手動解決を依頼
git commit 失敗エラー内容を報告
git push 失敗エラー内容を報告、原因を説明

参考資料

  • docs/einja/steering/commit-rules.md - コミットルール、分割方針、メッセージ形式の詳細

最終更新: 2026-03-13

<!-- @einja:project-private:start id="einja-task-commit-project" --> <!-- プロジェクト固有の情報を記入 --> <!-- @einja:project-private:end -->