[PR #378] Handle git aliases in hook processing #434

New issue

Closed

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

kerem commented

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

Owner

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

State: closed
Merged: No

Summary

resolve git aliases before dispatching hooks so aliased commands still trigger hook logic
expand aliases using git config (ignoring shell aliases) and prepend alias args to original args
add tests covering alias expansion and argument passthrough

Rationale

Commands invoked via git aliases (e.g., , ) were bypassing git-ai hooks. Resolving aliases to their underlying commands ensures hooks still run.

Changes

resolve alias definitions and rewrite parsed command/args prior to hook dispatch
ignore shell-style aliases
add unit tests for alias expansion behavior

Test Plan

unit tests added for alias expansion

Closes #377

**Original Pull Request:** https://github.com/git-ai-project/git-ai/pull/378 **State:** closed **Merged:** No --- ## Summary - resolve git aliases before dispatching hooks so aliased commands still trigger hook logic - expand aliases using git config (ignoring shell aliases) and prepend alias args to original args - add tests covering alias expansion and argument passthrough ## Rationale Commands invoked via git aliases (e.g., , ) were bypassing git-ai hooks. Resolving aliases to their underlying commands ensures hooks still run. ## Changes - resolve alias definitions and rewrite parsed command/args prior to hook dispatch - ignore shell-style aliases - add unit tests for alias expansion behavior ## Test Plan - unit tests added for alias expansion Closes #377  --- <a href="https://app.devin.ai/review/git-ai-project/git-ai/pull/378" 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>