feat: add recursive do option

.golangci.yml

@@ -18,6 +18,13 @@ linters-settings:
     rules:
       - name: unused-parameter
         disabled: true
+  unused:
+    field-writes-are-uses: false
+    post-statements-are-reads: true
+    exported-fields-are-used: false
+    parameters-are-used: true
+    local-variables-are-used: false
+    generated-is-used: false
 
 issues:
   exclude:

internal/config/actions.go

@@ -0,0 +1,59 @@
+package config
+
+type Action struct {
+	Name   string `json:"name,omitempty"   mapstructure:"name"   toml:"name,omitempty"   yaml:",omitempty"`
+	Run    string `json:"run,omitempty"    mapstructure:"run"    toml:"run,omitempty"    yaml:",omitempty"`
+	Script string `json:"script,omitempty" mapstructure:"script" toml:"script,omitempty" yaml:",omitempty"`
+	Runner string `json:"runner,omitempty" mapstructure:"runner" toml:"runner,omitempty" yaml:",omitempty"`
+
+	Glob     string `json:"glob,omitempty"      mapstructure:"glob"      toml:"glob,omitempty"      yaml:",omitempty"`
+	Root     string `json:"root,omitempty"      mapstructure:"root"      toml:"root,omitempty"      yaml:",omitempty"`
+	Files    string `json:"files,omitempty"     mapstructure:"files"     toml:"files,omitempty"     yaml:",omitempty"`
+	FailText string `json:"fail_text,omitempty" mapstructure:"fail_text" toml:"fail_text,omitempty" yaml:"fail_text,omitempty"`
+
+	Tags      []string `json:"tags,omitempty"       mapstructure:"tags"       toml:"tags,omitempty"       yaml:",omitempty"`
+	FileTypes []string `json:"file_types,omitempty" mapstructure:"file_types" toml:"file_types,omitempty" yaml:"file_types,omitempty"`
+
+	Env map[string]string `json:"env,omitempty" mapstructure:"env" toml:"env,omitempty" yaml:",omitempty"`
+
+	Interactive bool `json:"interactive,omitempty" mapstructure:"interactive" toml:"interactive,omitempty" yaml:",omitempty"`
+	UseStdin    bool `json:"use_stdin,omitempty"   mapstructure:"use_stdin"   toml:"use_stdin,omitempty"   yaml:",omitempty"`
+	StageFixed  bool `json:"stage_fixed,omitempty" mapstructure:"stage_fixed" toml:"stage_fixed,omitempty" yaml:"stage_fixed,omitempty"`
+
+	Exclude interface{} `json:"exclude,omitempty" mapstructure:"exclude" toml:"exclude,omitempty"     yaml:",omitempty"`
+	Skip    interface{} `json:"skip,omitempty"    mapstructure:"skip"    toml:"skip,omitempty,inline" yaml:",omitempty"`
+	Only    interface{} `json:"only,omitempty"    mapstructure:"only"    toml:"only,omitempty,inline" yaml:",omitempty"`
+
+	Group *Group `json:"group,omitempty" mapstructure:"group" toml:"group,omitempty" yaml:",omitempty"`
+}
+
+type Group struct {
+	Name     string `json:"name,omitempty"     mapstructure:"name"     toml:"name,omitempty"     yaml:",omitempty"`
+	Root     string `json:"root,omitempty"     mapstructure:"root"     toml:"root,omitempty"     yaml:",omitempty"`
+	Parallel bool   `json:"parallel,omitempty" mapstructure:"parallel" toml:"parallel,omitempty" yaml:",omitempty"`
+	Piped    bool   `json:"piped,omitempty"    mapstructure:"piped"    toml:"piped,omitempty"    yaml:",omitempty"`
+	Glob     string `json:"glob,omitempty"     mapstructure:"glob"     toml:"glob,omitempty"     yaml:",omitempty"`
+	Actions       []*Action  `json:"actions,omitempty"       mapstructure:"actions"       toml:"actions,omitempty"       yaml:",omitempty"`
+}
+
+func (action *Action) PrintableName(id string) string {
+	if len(action.Name) != 0 {
+		return action.Name
+	}
+	if len(action.Run) != 0 {
+		return action.Run
+	}
+	if len(action.Script) != 0 {
+		return action.Script
+	}
+
+	return "[" + id + "]"
+}
+
+func (g *Group) PrintableName(id string) string {
+	if len(g.Name) != 0 {
+		return g.Name
+	}
+
+	return "[" + id + "]"
+}

internal/config/command.go

@@ -5,12 +5,9 @@ import (
 	"strings"
 
 	"github.com/spf13/viper"
-
-	"github.com/evilmartians/lefthook/internal/git"
-	"github.com/evilmartians/lefthook/internal/system"
 )
 
-var errFilesIncompatible = errors.New("One of your runners contains incompatible file types")
+var ErrFilesIncompatible = errors.New("One of your runners contains incompatible file types")
 
 type Command struct {
 	Run   string `json:"run"             mapstructure:"run"   toml:"run"             yaml:"run"`
@@ -38,19 +35,6 @@ type commandRunReplace struct {
 	Run string `mapstructure:"run"`
 }
 
-func (c Command) Validate() error {
-	if !isRunnerFilesCompatible(c.Run) {
-		return errFilesIncompatible
-	}
-
-	return nil
-}
-
-func (c Command) DoSkip(state func() git.State) bool {
-	skipChecker := NewSkipChecker(system.Cmd)
-	return skipChecker.check(state, c.Skip, c.Only)
-}
-
 func (c Command) ExecutionPriority() int {
 	return c.Priority
 }

internal/config/config.go

@@ -11,8 +11,6 @@ import (
 	"github.com/mitchellh/mapstructure"
 	toml "github.com/pelletier/go-toml/v2"
 	"gopkg.in/yaml.v3"
-
-	"github.com/evilmartians/lefthook/internal/version"
 )
 
 type DumpFormat int
@@ -45,10 +43,6 @@ type Config struct {
 	Hooks map[string]*Hook `mapstructure:"-"`
 }
 
-func (c *Config) Validate() error {
-	return version.CheckCovered(c.MinVersion)
-}
-
 func (c *Config) Md5() (checksum string, err error) {
 	configBytes := new(bytes.Buffer)
 

internal/config/files.go

@@ -9,9 +9,6 @@ const (
 	SubPushFiles   string = "{push_files}"
 )
 
-func isRunnerFilesCompatible(runner string) bool {
-	if strings.Contains(runner, SubStagedFiles) && strings.Contains(runner, SubPushFiles) {
-		return false
-	}
-	return true
+func IsRunFilesCompatible(run string) bool {
+	return !(strings.Contains(run, SubStagedFiles) && strings.Contains(run, SubPushFiles))
 }

internal/config/hook.go

@@ -1,51 +1,34 @@
 package config
 
 import (
-	"errors"
 	"os"
 	"strings"
 
 	"github.com/spf13/viper"
-
-	"github.com/evilmartians/lefthook/internal/git"
-	"github.com/evilmartians/lefthook/internal/system"
 )
 
 const CMD = "{cmd}"
 
-var errPipedAndParallelSet = errors.New("conflicting options 'piped' and 'parallel' are set to 'true', remove one of this option from hook group")
-
 type Hook struct {
-	// Should be unmarshalled with `mapstructure:"commands"`
-	// But replacing '{cmd}' is still an issue
-	// Unmarshalling it manually, so omit auto unmarshalling
-	Commands map[string]*Command `json:"commands,omitempty" mapstructure:"-" toml:"commands,omitempty" yaml:",omitempty"`
-
-	// Should be unmarshalled with `mapstructure:"scripts"`
-	// But parsing keys with dots in it is still an issue: https://github.com/spf13/viper/issues/324
-	// Unmarshalling it manually, so omit auto unmarshalling
-	Scripts map[string]*Script `json:"scripts,omitempty" mapstructure:"-" toml:"scripts,omitempty" yaml:",omitempty"`
-
-	Files       string      `json:"files,omitempty"        mapstructure:"files"        toml:"files,omitempty"        yaml:",omitempty"`
 	Parallel    bool        `json:"parallel,omitempty"     mapstructure:"parallel"     toml:"parallel,omitempty"     yaml:",omitempty"`
 	Piped       bool        `json:"piped,omitempty"        mapstructure:"piped"        toml:"piped,omitempty"        yaml:",omitempty"`
 	Follow      bool        `json:"follow,omitempty"       mapstructure:"follow"       toml:"follow,omitempty"       yaml:",omitempty"`
+	Files       string      `json:"files,omitempty"        mapstructure:"files"        toml:"files,omitempty"        yaml:",omitempty"`
 	ExcludeTags []string    `json:"exclude_tags,omitempty" mapstructure:"exclude_tags" toml:"exclude_tags,omitempty" yaml:"exclude_tags,omitempty"`
 	Skip        interface{} `json:"skip,omitempty"         mapstructure:"skip"         toml:"skip,omitempty,inline"  yaml:",omitempty"`
 	Only        interface{} `json:"only,omitempty"         mapstructure:"only"         toml:"only,omitempty,inline"  yaml:",omitempty"`
-}
 
-func (h *Hook) Validate() error {
-	if h.Parallel && h.Piped {
-		return errPipedAndParallelSet
-	}
+	Actions []*Action `json:"actions,omitempty" mapstructure:"actions" toml:"actions,omitempty" yaml:",omitempty"`
 
-	return nil
-}
+	// Should be unmarshalled with `mapstructure:"commands"`
+	// But replacing '{cmd}' is still an issue
+	// Unmarshalling it manually, so omit auto unmarshalling
+	Commands map[string]*Command `json:"commands,omitempty" mapstructure:"-" toml:"commands,omitempty" yaml:",omitempty"`
 
-func (h *Hook) DoSkip(state func() git.State) bool {
-	skipChecker := NewSkipChecker(system.Cmd)
-	return skipChecker.check(state, h.Skip, h.Only)
+	// Should be unmarshalled with `mapstructure:"scripts"`
+	// But parsing keys with dots in it is still an issue: https://github.com/spf13/viper/issues/324
+	// Unmarshalling it manually, so omit auto unmarshalling
+	Scripts map[string]*Script `json:"scripts,omitempty" mapstructure:"-" toml:"scripts,omitempty" yaml:",omitempty"`
 }
 
 func unmarshalHooks(base, extra *viper.Viper) (*Hook, error) {