Reshape update notifier plan

Author: kindermax

docs/docs/changelog.md

@@ -14,6 +14,7 @@ title: Changelog
 * `[Added]` Support `env_file` in global config and commands. File names are expanded after `env` is resolved, and values loaded from env files override values from `env`.
 * `[Changed]` Migrate the LSP YAML parser from the CGO-based tree-sitter bindings to pure-Go [`gotreesitter`](https://github.com/odvcencio/gotreesitter), removing the C toolchain requirement from normal builds and release packaging.
 * `[Refactoring]` Move CLI startup flow from `cmd/lets/main.go` into `internal/cli/cli.go`, keeping `main.go` as a thin launcher.
+* `[Added]` Show background update notifications for interactive sessions, with Homebrew-aware guidance and `LETS_CHECK_UPDATE` opt-out.
 
 ## [0.0.59](https://github.com/lets-cli/lets/releases/tag/v0.0.59)
 

docs/docs/env.md

@@ -10,6 +10,7 @@ title: Environment
 * `LETS_DEBUG` - enable debug messages
 * `LETS_CONFIG` - changes default `lets.yaml` file path (e.g. LETS_CONFIG=lets.my.yaml)
 * `LETS_CONFIG_DIR` - changes path to dir where `lets.yaml` file placed
+* `LETS_CHECK_UPDATE` - disables background update checks and notifications
 * `NO_COLOR` - disables colored output. See https://no-color.org/
 
 ### Environment variables available at command runtime

internal/cli/cli.go

@@ -3,10 +3,12 @@ package cli
 import (
 	"context"
 	"errors"
+	"fmt"
 	"os"
 	"os/signal"
 	"strings"
 	"syscall"
+	"time"
 
 	"github.com/lets-cli/lets/internal/cmd"
 	"github.com/lets-cli/lets/internal/config"
@@ -17,10 +19,18 @@ import (
 	"github.com/lets-cli/lets/internal/upgrade"
 	"github.com/lets-cli/lets/internal/upgrade/registry"
 	"github.com/lets-cli/lets/internal/workdir"
+	"github.com/mattn/go-isatty"
 	log "github.com/sirupsen/logrus"
 	"github.com/spf13/cobra"
 )
 
+const updateCheckTimeout = 3 * time.Second
+
+type updateCheckResult struct {
+	notifier *upgrade.UpdateNotifier
+	notice   *upgrade.UpdateNotice
+}
+
 func Main(version string, buildDate string) int {
 	ctx := getContext()
 
@@ -118,6 +128,9 @@ func Main(version string, buildDate string) int {
 		return 0
 	}
 
+	updateCh, cancelUpdateCheck := maybeStartUpdateCheck(ctx, version, command)
+	defer cancelUpdateCheck()
+
 	if err := rootCmd.ExecuteContext(ctx); err != nil {
 		var depErr *executor.DependencyError
 		if errors.As(err, &depErr) {
@@ -128,6 +141,8 @@ func Main(version string, buildDate string) int {
 		return getExitCode(err, 1)
 	}
 
+	printUpdateNotice(updateCh)
+
 	return 0
 }
 
@@ -167,6 +182,82 @@ func failOnConfigError(root *cobra.Command, current *cobra.Command, rootFlags *f
 	return (root.Flags().NFlag() == 0 && !rootCommands.Contains(current.Name())) && !rootFlags.help && !rootFlags.init
 }
 
+func maybeStartUpdateCheck(
+	ctx context.Context,
+	version string,
+	command *cobra.Command,
+) (<-chan updateCheckResult, context.CancelFunc) {
+	if !shouldCheckForUpdate(command.Name(), isInteractiveStderr()) {
+		log.Debugf("lets: update check disabled")
+		return nil, func() {}
+	}
+
+	log.Debugf("lets: start update check")
+
+	notifier, err := upgrade.NewUpdateNotifier(registry.NewGithubRegistry(ctx))
+	if err != nil {
+		return nil, func() {}
+	}
+
+	ch := make(chan updateCheckResult, 1)
+	checkCtx, cancel := context.WithTimeout(ctx, updateCheckTimeout)
+
+	go func() {
+		notice, err := notifier.Check(checkCtx, version)
+		if err != nil {
+			upgrade.LogUpdateCheckError(err)
+		}
+		log.Debugf("lets: update check done")
+
+		ch <- updateCheckResult{
+			notifier: notifier,
+			notice:   notice,
+		}
+	}()
+
+	return ch, cancel
+}
+
+func printUpdateNotice(updateCh <-chan updateCheckResult) {
+	if updateCh == nil {
+		return
+	}
+
+	select {
+	case result := <-updateCh:
+		if result.notice == nil {
+			return
+		}
+
+		if _, err := fmt.Fprintln(os.Stderr, result.notice.Message()); err != nil {
+			return
+		}
+
+		if err := result.notifier.MarkNotified(result.notice); err != nil {
+			upgrade.LogUpdateCheckError(err)
+		}
+	default:
+	}
+}
+
+func shouldCheckForUpdate(commandName string, interactive bool) bool {
+	if !interactive || os.Getenv("CI") != "" || os.Getenv("LETS_CHECK_UPDATE") != "" {
+		return false
+	}
+
+	switch commandName {
+	case "completion", "help", "lsp", "self":
+		return false
+	default:
+		return true
+	}
+}
+
+func isInteractiveStderr() bool {
+	fd := os.Stderr.Fd()
+	return isatty.IsTerminal(fd) || isatty.IsCygwinTerminal(fd)
+}
+
 type flags struct {
 	config  string
 	debug   int

internal/cli/cli_test.go

@@ -0,0 +1,39 @@
+package cli
+
+import "testing"
+
+func TestShouldCheckForUpdate(t *testing.T) {
+	t.Run("should allow normal interactive commands", func(t *testing.T) {
+		if !shouldCheckForUpdate("lets", true) {
+			t.Fatal("expected update check to be enabled")
+		}
+	})
+
+	t.Run("should skip non interactive sessions", func(t *testing.T) {
+		if shouldCheckForUpdate("lets", false) {
+			t.Fatal("expected non-interactive session to skip update check")
+		}
+	})
+
+	t.Run("should skip when CI is set", func(t *testing.T) {
+		t.Setenv("CI", "1")
+		if shouldCheckForUpdate("lets", true) {
+			t.Fatal("expected CI to skip update check")
+		}
+	})
+
+	t.Run("should skip when notifier disabled", func(t *testing.T) {
+		t.Setenv("LETS_CHECK_UPDATE", "1")
+		if shouldCheckForUpdate("lets", true) {
+			t.Fatal("expected opt-out env to skip update check")
+		}
+	})
+
+	t.Run("should skip internal commands", func(t *testing.T) {
+		for _, name := range []string{"completion", "help", "lsp", "self"} {
+			if shouldCheckForUpdate(name, true) {
+				t.Fatalf("expected %q to skip update check", name)
+			}
+		}
+	})
+}