[PR #551] Rebase global-hooks-dual-mode onto latest main #559

New issue

Closed

opened 2026-03-02 04:13:58 +03:00 by kerem · 0 comments

kerem commented

2026-03-02 04:13:58 +03:00

Owner

Original Pull Request: https://github.com/git-ai-project/git-ai/pull/551

State: closed
Merged: Yes

Summary

Rebased the codex/global-hooks-dual-mode branch onto the latest upstream/main, incorporating 120 new upstream commits
All 32 branch commits replayed cleanly with no conflicts
No functional changes to the branch's own code; this is purely a rebase to stay current with main

Context

This branch implements dual-mode global hooks support for git-ai, allowing global Git hooks to coexist alongside wrapper mode. Key changes in this branch include:

Dual-mode hook system (git_hook_handlers.rs): 2900+ line module supporting global hooks alongside the existing wrapper mode
Hook forwarding: Selective forwarding of non-managed hooks, symlink installation, and environment isolation
Rebase workflow hardening: Suppression/restoration of hooks during rebases, cherry-pick batch detection stabilization
Performance optimization: No-op hook fast paths, repo-local hook provisioning, benchmark harness and CI gates
CI matrix expansion: Test matrix across OS and git test modes, plus performance benchmark workflow
Comprehensive test coverage: Hook forwarding integration tests, hook mode tests, stash attribution tests

Test plan

CI passes on the rebased branch
Existing hook mode tests continue to pass
Performance benchmarks show no regression from rebase

🤖 Generated with Claude Code

**Original Pull Request:** https://github.com/git-ai-project/git-ai/pull/551 **State:** closed **Merged:** Yes --- ## Summary - Rebased the `codex/global-hooks-dual-mode` branch onto the latest `upstream/main`, incorporating 120 new upstream commits - All 32 branch commits replayed cleanly with no conflicts - No functional changes to the branch's own code; this is purely a rebase to stay current with main ## Context This branch implements dual-mode global hooks support for git-ai, allowing global Git hooks to coexist alongside wrapper mode. Key changes in this branch include: - **Dual-mode hook system** (`git_hook_handlers.rs`): 2900+ line module supporting global hooks alongside the existing wrapper mode - **Hook forwarding**: Selective forwarding of non-managed hooks, symlink installation, and environment isolation - **Rebase workflow hardening**: Suppression/restoration of hooks during rebases, cherry-pick batch detection stabilization - **Performance optimization**: No-op hook fast paths, repo-local hook provisioning, benchmark harness and CI gates - **CI matrix expansion**: Test matrix across OS and git test modes, plus performance benchmark workflow - **Comprehensive test coverage**: Hook forwarding integration tests, hook mode tests, stash attribution tests ## Test plan - [ ] CI passes on the rebased branch - [ ] Existing hook mode tests continue to pass - [ ] Performance benchmarks show no regression from rebase 🤖 Generated with [Claude Code](https://claude.com/claude-code)  --- <a href="https://app.devin.ai/review/git-ai-project/git-ai/pull/551" target="_blank"> <picture> <source media="(prefers-color-scheme: dark)" srcset="https://static.devin.ai/assets/gh-open-in-devin-review-dark.svg?v=1"> <img src="https://static.devin.ai/assets/gh-open-in-devin-review-light.svg?v=1" alt="Open with Devin"> </picture> </a>