Skip to content

feat: Git 操作を grape に移行 - #114

Merged
version-1 merged 3 commits into
mainfrom
feat/use-grape-for-git-operations
Aug 2, 2026
Merged

feat: Git 操作を grape に移行#114
version-1 merged 3 commits into
mainfrom
feat/use-grape-for-git-operations

Conversation

@version-1

Copy link
Copy Markdown
Owner

Git 操作を grape に移行

Summary

エージェントが行う push と rebase を grape のポリシー管理下へ移行し、Codex / Claude の手順と権限設定を統一します。

主な変更点:

  • safe-git-push を廃止し、push を grape push へ統一
  • 通常の rebase を grape rebase へ統一し、Git 直接操作を競合処理に限定
  • grape-usage external skill を Codex / Claude の配布対象へ追加

やったこと

  • AGENTS、CLAUDE、PR 作成、Gardener、beautify-commit の手順を grape ベースへ更新
  • Codex と Claude で grape * を許可し、Claude の包括的な Git 許可を安全なサブコマンド列挙へ変更
  • interactive rebase は grape が対応するまで実行しない方針を明記
  • safe-git-push の実装、許可設定、文書、eval の参照を削除
  • grape-usage の Git tree hash を固定して両環境への配布を設定

動作確認

  • make deploy-dry-run
  • GOMODCACHE=/tmp/gomodcache GOCACHE=/tmp/gocache GOTOOLCHAIN=local go test ./...
  • JSON 構文検査と git diff --check

レビュー & 動作確認 チェックリスト

  • grape 権限 - Codex / Claude で grape * が許可されていること
  • rebase 制約 - Git 直接 rebase が競合処理3操作だけに限定されていること
  • external skill - grape-usage が Codex / Claude の両方へ配布されること

推奨テスト計画:

  1. make deploy-dry-rungrape-usage の hash 検証と配布先を確認する
  2. Codex / Claude の許可設定で grape と Git rebase の対象範囲を確認する
  3. skill、eval、文書に safe-git-push の参照が残っていないことを確認する

その他気になることや相談ごと

git rebase -i は grape が対応するまで利用しません。実環境への make deploy はこの PR では実行していません。

safe-git-push を廃止し、push と rebase の手順・権限設定を grape に統一する。grape-usage external skill を Codex と Claude の配布対象へ追加した。\n\n検証: make deploy-dry-run\n検証: GOMODCACHE=/tmp/gomodcache GOCACHE=/tmp/gocache GOTOOLCHAIN=local go test ./...
Claude の競合処理許可を打ち消す ask ルールを除去し、履歴整理後の push も grape push --force-with-lease に統一する。\n\n検証: jq empty claude/settings.json\n検証: git diff --check
包括的な Bash(git *) 許可を安全なサブコマンド列挙へ置き換え、直接 rebase は競合処理の3操作だけを許可する。\n\n検証: jq empty claude/settings.json\n検証: git diff --check
@version-1
version-1 merged commit 61044b1 into main Aug 2, 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