feat: better versioning

Author: pd93

cmd/release/main.go

@@ -67,6 +67,10 @@ func release() error {
 		return err
 	}
 
+	if err := setVersionFile("internal/version/version.txt", version); err != nil {
+		return err
+	}
+
 	if err := setJSONVersion("package.json", version); err != nil {
 		return err
 	}
@@ -144,6 +148,10 @@ func changelog(version *semver.Version) error {
 	return os.WriteFile(changelogTarget, []byte(changelog), 0o644)
 }
 
+func setVersionFile(fileName string, version *semver.Version) error {
+	return os.WriteFile(fileName, []byte(version.String()), 0o644)
+}
+
 func setJSONVersion(fileName string, version *semver.Version) error {
 	// Read the JSON file
 	b, err := os.ReadFile(fileName)

cmd/task/task.go

@@ -17,7 +17,7 @@ import (
 	"github.com/go-task/task/v3/internal/filepathext"
 	"github.com/go-task/task/v3/internal/flags"
 	"github.com/go-task/task/v3/internal/logger"
-	ver "github.com/go-task/task/v3/internal/version"
+	"github.com/go-task/task/v3/internal/version"
 	"github.com/go-task/task/v3/taskfile"
 	"github.com/go-task/task/v3/taskfile/ast"
 )
@@ -57,7 +57,7 @@ func run() error {
 	}
 
 	if flags.Version {
-		fmt.Printf("Task version: %s\n", ver.GetVersionWithSum())
+		fmt.Println(version.GetVersionWithBuildInfo())
 		return nil
 	}
 

internal/version/version.go

@@ -1,33 +1,67 @@
 package version
 
 import (
-	"fmt"
+	_ "embed"
 	"runtime/debug"
+	"strings"
 )
 
 var (
-	version = ""
-	sum     = ""
+	//go:embed version.txt
+	version string
+	commit  string
+	dirty   bool
 )
 
 func init() {
-	info, ok := debug.ReadBuildInfo()
-	if !ok || info.Main.Version == "(devel)" || info.Main.Version == "" {
-		version = "unknown"
-	} else {
-		if version == "" {
-			version = info.Main.Version
+	version = strings.TrimSpace(version)
+	// Attempt to get build info from the Go runtime. We only use this if not
+	// built from a tagged version.
+	if info, ok := debug.ReadBuildInfo(); ok && info.Main.Version == "(devel)" {
+		commit = getCommit(info)
+		dirty = getDirty(info)
+	}
+}
+
+func getDirty(info *debug.BuildInfo) bool {
+	for _, setting := range info.Settings {
+		if setting.Key == "vcs.modified" {
+			return setting.Value == "true"
 		}
-		if sum == "" {
-			sum = info.Main.Sum
+	}
+	return false
+}
+
+func getCommit(info *debug.BuildInfo) string {
+	for _, setting := range info.Settings {
+		if setting.Key == "vcs.revision" {
+			return setting.Value[:7]
 		}
 	}
+	return ""
 }
 
+// GetVersion returns the version of Task. By default, this is retrieved from
+// the embedded version.txt file which is kept up-to-date by our release script.
+// However, it can also be overridden at build time using:
+// -ldflags="-X 'github.com/go-task/task/v3/internal/version.version=vX.X.X'".
 func GetVersion() string {
 	return version
 }
 
-func GetVersionWithSum() string {
-	return fmt.Sprintf("%s (%s)", version, sum)
+// GetVersionWithBuildInfo is the same as [GetVersion], but it also includes
+// the commit hash and dirty status if available. This will only work when built
+// within inside of a Git checkout.
+func GetVersionWithBuildInfo() string {
+	var buildInfo string
+	if commit != "" {
+		buildInfo += commit
+	}
+	if dirty {
+		buildInfo += "-dirty"
+	}
+	if buildInfo != "" {
+		return version + "-" + buildInfo
+	}
+	return version
 }

internal/version/version.txt

@@ -0,0 +1 @@
+3.42.1