fix: branch info parse error on checkout to tag,hash

info_test.go

@@ -35,7 +35,13 @@ func TestNewRemoteInfo(t *testing.T) {
 }
 
 func TestParseBranchLine_simple(t *testing.T) {
-	info, err := gitw.ParseBranchLine("* fea/new_br001", false)
+	info, err := gitw.ParseBranchLine("* ", false)
+	assert.Error(t, err)
+
+	info, err = gitw.ParseBranchLine("* (HEAD)", false)
+	assert.Error(t, err)
+
+	info, err = gitw.ParseBranchLine("* fea/new_br001", false)
 	assert.NoError(t, err)
 
 	assert.True(t, info.Current)
@@ -81,6 +87,9 @@ func TestParseBranchLine_verbose(t *testing.T) {
 	assert.Equal(t, "my_new_br", info.Short)
 	assert.Equal(t, "6fb8dcd", info.Hash)
 	assert.Equal(t, "the message 003", info.HashMsg)
+
+	info, err = gitw.ParseBranchLine("* （头指针在 v0.2.3 分离） 3c08adf chore: update readme add branch info docs", true)
+	assert.Error(t, err)
 }
 
 func TestBranchInfo_parse_simple(t *testing.T) {

info_util.go

@@ -74,18 +74,27 @@ var ErrInvalidBrLine = errorx.Raw("invalid git branch line text")
 // 	False - only branch name
 //	True  - get by `git br -v --all`
 //	        format: * BRANCH_NAME  COMMIT_ID  COMMIT_MSG
-func ParseBranchLine(line string, verbose bool) (info *BranchInfo, err error) {
-	info = &BranchInfo{}
+func ParseBranchLine(line string, verbose bool) (*BranchInfo, error) {
+	info := &BranchInfo{}
 	line = strings.TrimSpace(line)
 
 	if strings.HasPrefix(line, "*") {
 		info.Current = true
 		line = strings.Trim(line, "*\t ")
 	}
 
+	if line == "" {
+		return nil, ErrInvalidBrLine
+	}
+
+	// at tag head. eg: `* （头指针在 v0.2.3 分离） 3c08adf chore: update readme add branch info docs`
+	if strings.HasPrefix(line, "(") || strings.HasPrefix(line, "（") {
+		return nil, ErrInvalidBrLine
+	}
+
 	if !verbose {
 		info.SetName(line)
-		return
+		return info, nil
 	}
 
 	// parse name
@@ -103,7 +112,7 @@ func ParseBranchLine(line string, verbose bool) (info *BranchInfo, err error) {
 	}
 
 	info.Hash, info.HashMsg = nodes[0], nodes[1]
-	return
+	return info, nil
 }
 
 // eg: