Skip to content

feat: Git extension stage 1 — bundled extensions/git with hooks on all core commands #1941

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
mnriem merged 11 commits into from
Apr 7, 2026
Merged

feat: Git extension stage 1 — bundled extensions/git with hooks on all core commands #1941

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
a084d09
feat: add git extension with hooks on all core commands
mnriem Apr 6, 2026
2a6b79f
fix: set git identity env vars in extension tests for CI runners
mnriem Apr 6, 2026
9166afe
fix: address PR review comments
mnriem Apr 6, 2026
174e909
fix: address second round of PR review comments
mnriem Apr 6, 2026
7fd87f9
fix: address third round of PR review comments
mnriem Apr 6, 2026
995b255
fix: address fourth round of PR review comments
mnriem Apr 6, 2026
34d0dd5
fix: add --allow-empty to git commit in initialize-repo scripts
mnriem Apr 7, 2026
bd6186a
fix: resolve display names to bundled extensions before catalog download
mnriem Apr 7, 2026
2f352cd
fix: handle before_/after_ prefixes in auto-commit message derivation
mnriem Apr 7, 2026
3fd5144
fix: use portable grep -qw for word boundary in create-new-feature.sh
mnriem Apr 7, 2026
c870b62
fix: validate hook values, numeric --number, and PS warning routing
mnriem Apr 7, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
fix: handle before_/after_ prefixes in auto-commit message derivation 
- Strip both before_ and after_ prefixes when deriving command name
  (fixes misleading 'Auto-commit after before_plan' messages)
- Include phase (before/after) in default commit messages
- Clarify README config example is an override, not default behavior
  • Loading branch information
@mnriem
mnriem committed Apr 7, 2026
commit 2f352cdf5c7ca935ee05c83c019d457c04c56768
1 change: 1 addition & 0 deletions extensions/git/README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -57,6 +57,7 @@ branch_numbering: sequential
init_commit_message: "[Spec Kit] Initial commit"

# Auto-commit per command (all disabled by default)
# Example: enable auto-commit after specify
auto_commit:
default: false
after_specify:
Expand Down
9 changes: 5 additions & 4 deletions extensions/git/scripts/bash/auto-commit.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -124,16 +124,17 @@ if git diff --quiet HEAD 2>/dev/null && git diff --cached --quiet 2>/dev/null &&
fi

# Derive a human-readable command name from the event
# e.g., after_specify -> specify, after_plan -> plan
_command_name=$(echo "$EVENT_NAME" | sed 's/^after_//')
# e.g., after_specify -> specify, before_plan -> plan
_command_name=$(echo "$EVENT_NAME" | sed 's/^after_//' | sed 's/^before_//')
_phase=$(echo "$EVENT_NAME" | grep -q '^before_' && echo 'before' || echo 'after')

# Use custom message if configured, otherwise default
if [ -z "$_commit_msg" ]; then
_commit_msg="[Spec Kit] Auto-commit after ${_command_name}"
_commit_msg="[Spec Kit] Auto-commit ${_phase} ${_command_name}"
fi
Comment thread
mnriem marked this conversation as resolved.

# Stage and commit
_git_out=$(git add . 2>&1) || { echo "[specify] Error: git add failed: $_git_out" >&2; exit 1; }
_git_out=$(git commit -q -m "$_commit_msg" 2>&1) || { echo "[specify] Error: git commit failed: $_git_out" >&2; exit 1; }

echo "✓ Changes committed after ${_command_name}" >&2
echo "✓ Changes committed ${_phase} ${_command_name}" >&2
7 changes: 4 additions & 3 deletions extensions/git/scripts/powershell/auto-commit.ps1
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -127,11 +127,12 @@ if ($d1 -eq 0 -and $d2 -eq 0 -and -not $untracked) {
}

# Derive a human-readable command name from the event
$commandName = $EventName -replace '^after_', ''
$commandName = $EventName -replace '^after_', '' -replace '^before_', ''
$phase = if ($EventName -match '^before_') { 'before' } else { 'after' }

# Use custom message if configured, otherwise default
if (-not $commitMsg) {
$commitMsg = "[Spec Kit] Auto-commit after $commandName"
$commitMsg = "[Spec Kit] Auto-commit $phase $commandName"
}
Comment thread
mnriem marked this conversation as resolved.

# Stage and commit
Expand All @@ -145,4 +146,4 @@ try {
exit 1
}

Write-Host "✓ Changes committed after $commandName"
Write-Host "✓ Changes committed $phase $commandName"
Loading