如何修复Git环境pre-commit-msg钩子异常？代码是否有误？

你的pre-commit-msg钩子脚本存在几个明确的错误，导致无论提交消息是否合规都会触发错误，下面逐一分析并给出修复方案：

核心错误点 #

1. 关键变量未赋值
   你把获取提交消息的代码commit_message=$(cat .git/COMMIT_EDITMSG)注释掉了，导致$commit_message变量是空值。空字符串自然无法匹配任何正则模式，所以每次提交都会触发错误退出。

2. 正则表达式错误

   • bash的正则匹配不需要用/包裹模式，你写的/([A-Z]+[-][\d]+\/\S+)/里的首尾/是多余的，会被当作模式的一部分，导致匹配失败。
   • 没有使用^锚定字符串开头，原正则只要提交消息包含符合规则的片段就会通过，但你的需求是提交消息必须以JIRA工单前缀开头，缺失锚定的话，不符合规则的提交消息如果中间带有类似片段也会误判通过（当然现在因为变量为空，这个问题还没显现）。
   • 正则里的[-]可以简化为-，[\d]在bash的扩展正则中用[0-9]能保证更好的兼容性。

修复后的完整脚本 #

#!/bin/bash
# 只取提交消息的第一行（通常是提交标题，符合钩子检查的场景）
commit_message=$(head -1 .git/COMMIT_EDITMSG)
commit_error="提交消息格式错误！请以JIRA工单前缀开头，示例：
ABC-123/ticket-task 修复了XX问题
ABC-234/feat-user-login 新增登录功能"

# 正则规则：以大写字母+连字符+数字+斜杠+非空白任务标识开头
if [[ ! $commit_message =~ ^[A-Z]+-[0-9]+\/\S+ ]]; then
    echo >&2 "$commit_error"
    exit 1
fi

优化说明 #

• 改用head -1获取提交消息第一行：实际场景中提交消息通常是多行，标题行才需要符合前缀规则，这样更贴合实际使用习惯。
• 优化错误提示：把正确的示例写得更清晰，帮助用户快速修正提交消息。
• 正则调整为^[A-Z]+-[0-9]+\/\S+：明确锚定开头，确保提交消息必须以JIRA工单（如ABC-123）+斜杠+任务标识（如ticket-task/feat-xxx）开头，完全匹配你的需求。

验证方法 #

1. 确保脚本有可执行权限：chmod +x .git/hooks/pre-commit-msg
2. 测试合规提交消息：git commit -m "ABC-123/ticket-task 新增功能"，应该能正常提交。
3. 测试不合规提交消息：git commit -m "新增功能"，应该触发错误提示并终止提交。

内容的提问来源于stack exchange，提问作者axel