Merge pull request #5 from bvieira/prepare-commit

Validate commit message action

Makefile

@@ -50,9 +50,10 @@ release:
 	make build
 	@zip -j bin/git-sv_$(VERSION)_$(BUILDOS)_$(BUILDARCH).zip bin/$(BUILDOS)_$(BUILDARCH)/$(BIN)
 
-## release-all: prepare linux, darwin and windows binary for release
+## release-all: prepare linux, darwin and windows binary for release (requires sv4git)
 release-all:
 	@rm -rf bin
-	BUILDOS=linux make release
-	BUILDOS=darwin make release
-	BUILDOS=windows make release
+	
+	VERSION=$(shell git sv nv) BUILDOS=linux make release
+	VERSION=$(shell git sv nv) BUILDOS=darwin make release
+	VERSION=$(shell git sv nv) BUILDOS=windows make release

README.md

@@ -13,7 +13,7 @@ download the latest release and add the binary on your path
 you can config using the environment variables
 
 | Variable                       | description                                                                                                            | default                                                      |
-| --------- | ----------| ----------|
+| ------------------------------ | ---------------------------------------------------------------------------------------------------------------------- | ------------------------------------------------------------ |
 | MAJOR_VERSION_TYPES            | types used to bump major version                                                                                       |                                                              |
 | MINOR_VERSION_TYPES            | types used to bump minor version                                                                                       | feat                                                         |
 | PATCH_VERSION_TYPES            | types used to bump patch version                                                                                       | build,ci,docs,fix,perf,refactor,style,test                   |
@@ -22,6 +22,10 @@ you can config using the environment variables
 | ISSUEID_PREFIXES               | list of prefixes that will be used to identify an issue id                                                             | jira:,JIRA:,Jira:                                            |
 | TAG_PATTERN                    | tag version pattern                                                                                                    | %d.%d.%d                                                     |
 | RELEASE_NOTES_TAGS             | release notes headers for each visible type                                                                            | fix:Bug Fixes,feat:Features                                  |
+| VALIDATE_MESSAGE_SKIP_BRANCHES | ignore branches from this list on validate commit message                                                              | master,develop                                               |
+| COMMIT_MESSAGE_TYPES           | list of valid commit types for commit message                                                                          | build,ci,chore,docs,feat,fix,perf,refactor,revert,style,test |
+| ISSUE_KEY_NAME                 | metadata key name used on validate commit message hook to enhance footer, if blank footer will not be added            | jira                                                         |
+| BRANCH_ISSUE_REGEX             | regex to extract issue id from branch name, must have 3 groups (prefix, id, posfix), if blank footer will not be added | ^([a-z]+\\/)?([A-Z]+-[0-9]+)(-.*)?                           |
 
 ### Running
 
@@ -56,15 +60,36 @@ git-sv rn -h
 ##### Available commands
 
 | Variable                     | description                                                   |    has options     |
-| --------- | ---------- | :----------: |
+| ---------------------------- | ------------------------------------------------------------- | :----------------: |
 | current-version, cv          | get last released version from git                            |        :x:         |
 | next-version, nv             | generate the next version based on git commit messages        |        :x:         |
 | commit-log, cl               | list all commit logs since last version as jsons              | :heavy_check_mark: |
 | release-notes, rn            | generate release notes                                        | :heavy_check_mark: |
 | changelog, cgl               | generate changelog                                            | :heavy_check_mark: |
 | tag, tg                      | generate tag with version based on git commit messages        |        :x:         |
+| validate-commit-message, vcm | use as prepare-commit-message hook to validate commit message | :heavy_check_mark: |
 | help, h                      | Shows a list of commands or help for one command              |        :x:         |
 
+##### Use validate-commit-message as prepare-commit-msg hook
+
+Configure your .git/hooks/prepare-commit-msg
+
+```bash
+#!/bin/sh
+
+COMMIT_MSG_FILE=$1
+COMMIT_SOURCE=$2
+SHA1=$3
+
+git sv vcm --path "$(pwd)" --file $COMMIT_MSG_FILE --source $COMMIT_SOURCE
+```
+
+tip: you can configure a directory as your global git templates using the command below, check [git config docs](https://git-scm.com/docs/git-config#Documentation/git-config.txt-inittemplateDir) for more information!
+
+```bash
+git config --global init.templatedir '<YOUR TEMPLATE DIR>'
+```
+
 ## Development
 
 ### Makefile
@@ -78,7 +103,7 @@ make
 #### Make configs
 
 | Variable   | description            |
-| --------- | ----------|
+| ---------- | ---------------------- |
 | BUILDOS    | build OS               |
 | BUILDARCH  | build arch             |
 | ECHOFLAGS  | flags used on echo     |
@@ -86,7 +111,7 @@ make
 | BUILDFLAGS | flags used on build    |
 
 | Parameters | description                         |
-| --------- | ----------|
+| ---------- | ----------------------------------- |
 | args       | parameters that will be used on run |
 
 ```bash

cmd/git-sv/config.go

@@ -16,6 +16,10 @@ type Config struct {
 	IssueIDPrefixes             []string          `envconfig:"ISSUEID_PREFIXES" default:"jira:,JIRA:,Jira:"`
 	TagPattern                  string            `envconfig:"TAG_PATTERN" default:"%d.%d.%d"`
 	ReleaseNotesTags            map[string]string `envconfig:"RELEASE_NOTES_TAGS" default:"fix:Bug Fixes,feat:Features"`
+	ValidateMessageSkipBranches []string          `envconfig:"VALIDATE_MESSAGE_SKIP_BRANCHES" default:"master,develop"`
+	CommitMessageTypes          []string          `envconfig:"COMMIT_MESSAGE_TYPES" default:"build,ci,chore,docs,feat,fix,perf,refactor,revert,style,test"`
+	IssueKeyName                string            `envconfig:"ISSUE_KEY_NAME" default:"jira"`
+	BranchIssueRegex            string            `envconfig:"BRANCH_ISSUE_REGEX" default:"^([a-z]+\\/)?([A-Z]+-[0-9]+)(-.*)?"`
 }
 
 func loadConfig() Config {

cmd/git-sv/handlers.go

@@ -3,6 +3,8 @@ package main
 import (
 	"encoding/json"
 	"fmt"
+	"io/ioutil"
+	"os"
 	"sort"
 	"sv4git/sv"
 	"time"
@@ -228,3 +230,58 @@ func changelogHandler(git sv.Git, semverProcessor sv.SemVerCommitsProcessor, rnP
 		return nil
 	}
 }
+
+func validateCommitMessageHandler(git sv.Git, validateMessageProcessor sv.ValidateMessageProcessor) func(c *cli.Context) error {
+	return func(c *cli.Context) error {
+		branch := git.Branch()
+		if validateMessageProcessor.SkipBranch(branch) {
+			warn("commit message validation skipped, branch in ignore list...")
+			return nil
+		}
+
+		filepath := fmt.Sprintf("%s/%s", c.String("path"), c.String("file"))
+
+		commitMessage, err := readFile(filepath)
+		if err != nil {
+			return fmt.Errorf("failed to read commit message, error: %s", err.Error())
+		}
+
+		if err := validateMessageProcessor.Validate(commitMessage); err != nil {
+			return fmt.Errorf("invalid commit message, error: %s", err.Error())
+		}
+
+		msg, err := validateMessageProcessor.Enhance(branch, commitMessage)
+		if err != nil {
+			warn("could not enhance commit message, %s", err.Error())
+			return nil
+		}
+		if msg == "" {
+			return nil
+		}
+
+		if err := appendOnFile(msg, filepath); err != nil {
+			return fmt.Errorf("failed to append meta-informations on footer, error: %s", err.Error())
+		}
+
+		return nil
+	}
+}
+
+func readFile(filepath string) (string, error) {
+	f, err := ioutil.ReadFile(filepath)
+	if err != nil {
+		return "", err
+	}
+	return string(f), nil
+}
+
+func appendOnFile(message, filepath string) error {
+	f, err := os.OpenFile(filepath, os.O_APPEND|os.O_WRONLY, 0644)
+	if err != nil {
+		return err
+	}
+	defer f.Close()
+
+	_, err = f.WriteString(message)
+	return err
+}

cmd/git-sv/log.go

@@ -0,0 +1,7 @@
+package main
+
+import "fmt"
+
+func warn(format string, values ...interface{}) {
+	fmt.Printf("WARN: "+format+"\n", values...)
+}

cmd/git-sv/main.go

@@ -18,6 +18,7 @@ func main() {
 	semverProcessor := sv.NewSemVerCommitsProcessor(cfg.IncludeUnknownTypeAsPatch, cfg.MajorVersionTypes, cfg.MinorVersionTypes, cfg.PatchVersionTypes)
 	releasenotesProcessor := sv.NewReleaseNoteProcessor(cfg.ReleaseNotesTags)
 	outputFormatter := sv.NewOutputFormatter()
+	validateMessageProcessor := sv.NewValidateMessageProcessor(cfg.ValidateMessageSkipBranches, cfg.CommitMessageTypes, cfg.IssueKeyName, cfg.BranchIssueRegex)
 
 	app := cli.NewApp()
 	app.Name = "sv"
@@ -66,6 +67,17 @@ func main() {
 			Usage:   "generate tag with version based on git commit messages",
 			Action:  tagHandler(git, semverProcessor),
 		},
+		{
+			Name:    "validate-commit-message",
+			Aliases: []string{"vcm"},
+			Usage:   "use as prepare-commit-message hook to validate message",
+			Action:  validateCommitMessageHandler(git, validateMessageProcessor),
+			Flags: []cli.Flag{
+				&cli.StringFlag{Name: "path", Required: true, Usage: "git working directory"},
+				&cli.StringFlag{Name: "file", Required: true, Usage: "name of the file that contains the commit log message"},
+				&cli.StringFlag{Name: "source", Required: true, Usage: "source of the commit message"},
+			},
+		},
 	}
 
 	apperr := app.Run(os.Args)

go.mod

@@ -6,5 +6,5 @@ require (
 	github.com/Masterminds/semver v1.5.0
 	github.com/cpuguy83/go-md2man/v2 v2.0.0 // indirect
 	github.com/kelseyhightower/envconfig v1.4.0
-	github.com/urfave/cli/v2 v2.1.1
+	github.com/urfave/cli/v2 v2.2.0
 )

go.sum

@@ -13,7 +13,7 @@ github.com/russross/blackfriday/v2 v2.0.1 h1:lPqVAte+HuHNfhJ/0LC98ESWRz8afy9tM/0
 github.com/russross/blackfriday/v2 v2.0.1/go.mod h1:+Rmxgy9KzJVeS9/2gXHxylqXiyQDYRxCVz55jmeOWTM=
 github.com/shurcooL/sanitized_anchor_name v1.0.0 h1:PdmoCO6wvbs+7yrJyMORt4/BmY5IYyJwS/kOiWx8mHo=
 github.com/shurcooL/sanitized_anchor_name v1.0.0/go.mod h1:1NzhyTcUVG4SuEtjjoZeVRXNmyL/1OwPU0+IJeTBvfc=
-github.com/urfave/cli/v2 v2.1.1 h1:Qt8FeAtxE/vfdrLmR3rxR6JRE0RoVmbXu8+6kZtYU4k=
-github.com/urfave/cli/v2 v2.1.1/go.mod h1:SE9GqnLQmjVa0iPEY0f1w3ygNIYcIJ0OKPMoW2caLfQ=
+github.com/urfave/cli/v2 v2.2.0 h1:JTTnM6wKzdA0Jqodd966MVj4vWbbquZykeX1sKbe2C4=
+github.com/urfave/cli/v2 v2.2.0/go.mod h1:SE9GqnLQmjVa0iPEY0f1w3ygNIYcIJ0OKPMoW2caLfQ=
 gopkg.in/check.v1 v0.0.0-20161208181325-20d25e280405/go.mod h1:Co6ibVJAznAaIkqp8huTwlJQCZ016jof/cbN4VW5Yz0=
 gopkg.in/yaml.v2 v2.2.2/go.mod h1:hI93XBmqTisBFMUTm0b8Fm+jr3Dg1NNxqwp+5A1VGuI=

sv/git.go

@@ -28,6 +28,7 @@ type Git interface {
 	Log(initialTag, endTag string) ([]GitCommitLog, error)
 	Tag(version semver.Version) error
 	Tags() ([]GitTag, error)
+	Branch() string
 }
 
 // GitCommitLog description of a single commit log
@@ -115,6 +116,16 @@ func (g GitImpl) Tags() ([]GitTag, error) {
 	return parseTagsOutput(string(out))
 }
 
+// Branch get git branch
+func (GitImpl) Branch() string {
+	cmd := exec.Command("git", "symbolic-ref", "--short", "HEAD")
+	out, err := cmd.CombinedOutput()
+	if err != nil {
+		return ""
+	}
+	return strings.TrimSpace(strings.Trim(string(out), "\n"))
+}
+
 func parseTagsOutput(input string) ([]GitTag, error) {
 	scanner := bufio.NewScanner(strings.NewReader(input))
 	var result []GitTag

sv/git_test.go

@@ -13,8 +13,8 @@ func Test_parseTagsOutput(t *testing.T) {
 		want    []GitTag
 		wantErr bool
 	}{
-		{"with date", "2020-05-01 18:00:00 -0300#1.0.0", []GitTag{GitTag{Name: "1.0.0", Date: date("2020-05-01 18:00:00 -0300")}}, false},
-		{"without date", "#1.0.0", []GitTag{GitTag{Name: "1.0.0", Date: time.Time{}}}, false},
+		{"with date", "2020-05-01 18:00:00 -0300#1.0.0", []GitTag{{Name: "1.0.0", Date: date("2020-05-01 18:00:00 -0300")}}, false},
+		{"without date", "#1.0.0", []GitTag{{Name: "1.0.0", Date: time.Time{}}}, false},
 	}
 	for _, tt := range tests {
 		t.Run(tt.name, func(t *testing.T) {

sv/validatemessage.go

@@ -0,0 +1,108 @@
+package sv
+
+import (
+	"bufio"
+	"fmt"
+	"regexp"
+	"strings"
+)
+
+// ValidateMessageProcessor interface.
+type ValidateMessageProcessor interface {
+	SkipBranch(branch string) bool
+	Validate(message string) error
+	Enhance(branch string, message string) (string, error)
+}
+
+// NewValidateMessageProcessor ValidateMessageProcessorImpl constructor
+func NewValidateMessageProcessor(skipBranches, supportedTypes []string, issueKeyName, branchIssueRegex string) *ValidateMessageProcessorImpl {
+	return &ValidateMessageProcessorImpl{
+		skipBranches:     skipBranches,
+		supportedTypes:   supportedTypes,
+		issueKeyName:     issueKeyName,
+		branchIssueRegex: branchIssueRegex,
+	}
+}
+
+// ValidateMessageProcessorImpl process validate message hook.
+type ValidateMessageProcessorImpl struct {
+	skipBranches     []string
+	supportedTypes   []string
+	issueKeyName     string
+	branchIssueRegex string
+}
+
+// SkipBranch check if branch should be ignored.
+func (p ValidateMessageProcessorImpl) SkipBranch(branch string) bool {
+	return contains(branch, p.skipBranches)
+}
+
+// Validate commit message.
+func (p ValidateMessageProcessorImpl) Validate(message string) error {
+	valid, err := regexp.MatchString("^("+strings.Join(p.supportedTypes, "|")+")(\\(.+\\))?!?: .*$", firstLine(message))
+	if err != nil {
+		return err
+	}
+	if !valid {
+		return fmt.Errorf("message should contain type: %v, and should be valid according with conventional commits", p.supportedTypes)
+	}
+	return nil
+}
+
+// Enhance add metadata on commit message.
+func (p ValidateMessageProcessorImpl) Enhance(branch string, message string) (string, error) {
+	if p.branchIssueRegex == "" || p.issueKeyName == "" || hasIssueID(message, p.issueKeyName) {
+		return "", nil //enhance disabled
+	}
+
+	r, err := regexp.Compile(p.branchIssueRegex)
+	if err != nil {
+		return "", fmt.Errorf("could not compile issue regex: %s, error: %v", p.branchIssueRegex, err.Error())
+	}
+
+	groups := r.FindStringSubmatch(branch)
+	if len(groups) != 4 {
+		return "", fmt.Errorf("could not find issue id group with configured regex")
+	}
+
+	footer := fmt.Sprintf("%s: %s", p.issueKeyName, groups[2])
+
+	if !hasFooter(message) {
+		return "\n" + footer, nil
+	}
+
+	return footer, nil
+}
+
+func hasFooter(message string) bool {
+	r := regexp.MustCompile("^[a-zA-Z-]+: .*|^[a-zA-Z-]+ #.*|^BREAKING CHANGE: .*")
+
+	scanner := bufio.NewScanner(strings.NewReader(message))
+	lines := 0
+	for scanner.Scan() {
+		if lines > 0 && r.MatchString(scanner.Text()) {
+			return true
+		}
+		lines++
+	}
+
+	return false
+}
+
+func hasIssueID(message, issueKeyName string) bool {
+	r := regexp.MustCompile(fmt.Sprintf("(?m)^%s: .+$", issueKeyName))
+	return r.MatchString(message)
+}
+
+func contains(value string, content []string) bool {
+	for _, v := range content {
+		if value == v {
+			return true
+		}
+	}
+	return false
+}
+
+func firstLine(value string) string {
+	return strings.Split(value, "\n")[0]
+}

sv/validatemessage_test.go

@@ -0,0 +1,179 @@
+package sv
+
+import (
+	"testing"
+)
+
+var issueRegex = "^([a-z]+\\/)?([A-Z]+-[0-9]+)(-.*)?"
+
+// messages samples start
+var fullMessage = `fix: correct minor typos in code
+
+see the issue for details
+
+on typos fixed.
+
+Reviewed-by: Z
+Refs #133`
+var fullMessageWithJira = `fix: correct minor typos in code
+
+see the issue for details
+
+on typos fixed.
+
+Reviewed-by: Z
+Refs #133
+jira: JIRA-456`
+var fullMessageRefs = `fix: correct minor typos in code
+
+see the issue for details
+
+on typos fixed.
+
+Refs #133`
+var subjectAndBodyMessage = `fix: correct minor typos in code
+
+see the issue for details
+
+on typos fixed.`
+var subjectAndFooterMessage = `refactor!: drop support for Node 6
+
+BREAKING CHANGE: refactor to use JavaScript features not available in Node 6.`
+
+// multiline samples end
+
+func TestValidateMessageProcessorImpl_Validate(t *testing.T) {
+	p := NewValidateMessageProcessor([]string{"develop", "master"}, []string{"feat", "fix"}, "jira", issueRegex)
+
+	tests := []struct {
+		name    string
+		message string
+		wantErr bool
+	}{
+		{"single line valid message", "feat: add something", false},
+		{"single line valid message with scope", "feat(scope): add something", false},
+		{"single line invalid type message", "something: add something", true},
+		{"single line invalid type message", "feat?: add something", true},
+
+		{"multi line valid message", `feat: add something
+		
+		team: x`, false},
+
+		{"multi line invalid message", `feat add something
+		
+		team: x`, true},
+
+		{"support ! for breaking change", "feat!: add something", false},
+		{"support ! with scope for breaking change", "feat(scope)!: add something", false},
+	}
+	for _, tt := range tests {
+		t.Run(tt.name, func(t *testing.T) {
+			if err := p.Validate(tt.message); (err != nil) != tt.wantErr {
+				t.Errorf("ValidateMessageProcessorImpl.Validate() error = %v, wantErr %v", err, tt.wantErr)
+			}
+		})
+	}
+}
+
+func TestValidateMessageProcessorImpl_Enhance(t *testing.T) {
+	p := NewValidateMessageProcessor([]string{"develop", "master"}, []string{"feat", "fix"}, "jira", issueRegex)
+
+	tests := []struct {
+		name    string
+		branch  string
+		message string
+		want    string
+		wantErr bool
+	}{
+		{"issue on branch name", "JIRA-123", "fix: fix something", "\njira: JIRA-123", false},
+		{"issue on branch name with description", "JIRA-123-some-description", "fix: fix something", "\njira: JIRA-123", false},
+		{"issue on branch name with prefix", "feature/JIRA-123", "fix: fix something", "\njira: JIRA-123", false},
+		{"with footer", "JIRA-123", fullMessage, "jira: JIRA-123", false},
+		{"with issue on footer", "JIRA-123", fullMessageWithJira, "", false},
+		{"issue on branch name with prefix and description", "feature/JIRA-123-some-description", "fix: fix something", "\njira: JIRA-123", false},
+		{"no issue on branch name", "branch", "fix: fix something", "", true},
+		{"unexpected branch name", "feature /JIRA-123", "fix: fix something", "", true},
+	}
+	for _, tt := range tests {
+		t.Run(tt.name, func(t *testing.T) {
+			got, err := p.Enhance(tt.branch, tt.message)
+			if (err != nil) != tt.wantErr {
+				t.Errorf("ValidateMessageProcessorImpl.Enhance() error = %v, wantErr %v", err, tt.wantErr)
+				return
+			}
+			if got != tt.want {
+				t.Errorf("ValidateMessageProcessorImpl.Enhance() = %v, want %v", got, tt.want)
+			}
+		})
+	}
+}
+
+func Test_firstLine(t *testing.T) {
+	tests := []struct {
+		name  string
+		value string
+		want  string
+	}{
+		{"empty string", "", ""},
+
+		{"single line string", "single line", "single line"},
+
+		{"multi line string", `first line
+		last line`, "first line"},
+	}
+	for _, tt := range tests {
+		t.Run(tt.name, func(t *testing.T) {
+			if got := firstLine(tt.value); got != tt.want {
+				t.Errorf("firstLine() = %v, want %v", got, tt.want)
+			}
+		})
+	}
+}
+
+func Test_hasIssueID(t *testing.T) {
+	tests := []struct {
+		name         string
+		message      string
+		issueKeyName string
+		want         bool
+	}{
+		{"single line without issue", "feat: something", "jira", false},
+		{"multi line without issue", `feat: something
+		
+yay`, "jira", false},
+		{"multi line without jira issue", `feat: something
+		
+jira1: JIRA-123`, "jira", false},
+		{"multi line with issue", `feat: something
+		
+jira: JIRA-123`, "jira", true},
+	}
+	for _, tt := range tests {
+		t.Run(tt.name, func(t *testing.T) {
+			if got := hasIssueID(tt.message, tt.issueKeyName); got != tt.want {
+				t.Errorf("hasIssueID() = %v, want %v", got, tt.want)
+			}
+		})
+	}
+}
+
+func Test_hasFooter(t *testing.T) {
+	tests := []struct {
+		name    string
+		message string
+		want    bool
+	}{
+		{"simple message", "feat: add something", false},
+		{"full messsage", fullMessage, true},
+		{"full messsage with refs", fullMessageRefs, true},
+		{"subject and footer message", subjectAndFooterMessage, true},
+		{"subject and body message", subjectAndBodyMessage, false},
+	}
+	for _, tt := range tests {
+		t.Run(tt.name, func(t *testing.T) {
+			if got := hasFooter(tt.message); got != tt.want {
+				t.Errorf("hasFooter() = %v, want %v", got, tt.want)
+			}
+		})
+	}
+}