[PR #529] [MERGED] Fix corehooks hook-contract regressions and stale INITIAL handling #546

New issue

Closed

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

kerem commented

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

Owner

📋 Pull Request Information

Original PR: https://github.com/git-ai-project/git-ai/pull/529
Author: @svarlamov
Created: 2/16/2026
Status: ✅ Merged
Merged: 2/16/2026
Merged by: @svarlamov

Base: feat/corehooks ← Head: codex/corehooks-simplify-perf-round2

📝 Commits (3)

7ca03ec Optimize corehooks hot paths and simplify hook script checks
f6008ce Add aggressive no-AI hook prefilters for corehooks hot path
95df4e0 Fix Windows hook contract regressions in corehooks shims

📊 Changes

4 files changed (+416 additions, -110 deletions)

View changed files

📝 src/commands/core_hook_trampoline.rs (+35 -16)
📝 src/commands/core_hooks.rs (+296 -85)
📝 src/commands/install_hooks.rs (+50 -8)
📝 src/git/repo_storage.rs (+35 -1)

📄 Description

Summary

restore relevant ref detection coverage for reference-transaction prefilter (HEAD and refs/heads/*)
include INITIAL in non-empty working-log detection and hook prefilter dispatch checks
remove stale INITIAL file when writing empty initial attributions
update install-hooks script freshness checks for new prefilter signatures
add regression tests for INITIAL-only logs and ref prefilter coverage

Validation

cargo fmt -- --check
RUSTFLAGS='-D warnings' cargo clippy
cargo test -- --test-threads=8

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/git-ai-project/git-ai/pull/529 **Author:** [@svarlamov](https://github.com/svarlamov) **Created:** 2/16/2026 **Status:** ✅ Merged **Merged:** 2/16/2026 **Merged by:** [@svarlamov](https://github.com/svarlamov) **Base:** `feat/corehooks` ← **Head:** `codex/corehooks-simplify-perf-round2` --- ### 📝 Commits (3) - [`7ca03ec`](https://github.com/git-ai-project/git-ai/commit/7ca03eccfa91895b08d9f5047973006547736a51) Optimize corehooks hot paths and simplify hook script checks - [`f6008ce`](https://github.com/git-ai-project/git-ai/commit/f6008ce146ebfdef705b3268e166a9f9ff741927) Add aggressive no-AI hook prefilters for corehooks hot path - [`95df4e0`](https://github.com/git-ai-project/git-ai/commit/95df4e01ffafb7f28608570b3a6444351f61a9c1) Fix Windows hook contract regressions in corehooks shims ### 📊 Changes **4 files changed** (+416 additions, -110 deletions) <details> <summary>View changed files</summary> 📝 `src/commands/core_hook_trampoline.rs` (+35 -16) 📝 `src/commands/core_hooks.rs` (+296 -85) 📝 `src/commands/install_hooks.rs` (+50 -8) 📝 `src/git/repo_storage.rs` (+35 -1) </details> ### 📄 Description ## Summary - restore relevant ref detection coverage for `reference-transaction` prefilter (`HEAD` and `refs/heads/*`) - include `INITIAL` in non-empty working-log detection and hook prefilter dispatch checks - remove stale `INITIAL` file when writing empty initial attributions - update install-hooks script freshness checks for new prefilter signatures - add regression tests for INITIAL-only logs and ref prefilter coverage ## Validation - `cargo fmt -- --check` - `RUSTFLAGS='-D warnings' cargo clippy` - `cargo test -- --test-threads=8`  --- <a href="https://app.devin.ai/review/git-ai-project/git-ai/pull/529" 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>  --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>