[PR #579] Fix Windows repo hook binary self-heal refresh #585

New issue

Closed

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

kerem commented

2026-03-02 04:14:02 +03:00

Owner

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

State: closed
Merged: Yes

Summary\n- prefer the current runtime binary over saved repo state when resolving managed hook source binaries\n- add Windows-only stale detection for managed hook copies using mtime, with content fallback for ambiguous timestamps\n- defer Windows hook replacement when a hook binary is locked/running, while preserving hard-fail behavior for non-lock errors\n- add Windows tests for mtime refresh and lock/defer retry, plus a resolver precedence test\n\n## Testing\n- cargo fmt -- --check\n- cargo clippy\n- cargo test --lib resolve_repo_hook_binary_path_prefers_runtime_binary_over_saved_external_binary\n- cargo test --lib ensure_hook_entry_install_is_idempotent\n

**Original Pull Request:** https://github.com/git-ai-project/git-ai/pull/579 **State:** closed **Merged:** Yes --- ## Summary\n- prefer the current runtime binary over saved repo state when resolving managed hook source binaries\n- add Windows-only stale detection for managed hook copies using mtime, with content fallback for ambiguous timestamps\n- defer Windows hook replacement when a hook binary is locked/running, while preserving hard-fail behavior for non-lock errors\n- add Windows tests for mtime refresh and lock/defer retry, plus a resolver precedence test\n\n## Testing\n- cargo fmt -- --check\n- cargo clippy\n- cargo test --lib resolve_repo_hook_binary_path_prefers_runtime_binary_over_saved_external_binary\n- cargo test --lib ensure_hook_entry_install_is_idempotent\n  --- <a href="https://app.devin.ai/review/git-ai-project/git-ai/pull/579" 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>