Merge pull request #4 from SAP-samples/feature/tray-lifecycle

feat(tray): add tray lifecycle management and CLI commands

Author: jung-thomas

CLAUDE.md

@@ -122,6 +122,10 @@ On every command invocation (except `update` and dev builds), a background gorou
 
 `internal/service/` provides a `Scheduler` interface with platform implementations behind build tags: `scheduler_windows.go` (Task Scheduler via `schtasks`), `scheduler_darwin.go` (launchd plist), `scheduler_linux.go` (systemd user timer). Each shells out to OS tools — no CGO, no new dependencies. `service.New(cacheDir)` returns the platform-appropriate implementation. Scheduler logs to `~/.cache/sap-devs/daemon.log`. Interval is configured via `config.Service.Interval` (default 6h).
 
+### Tray Companion (Experimental)
+
+`internal/trayctl/` manages an optional GUI tray binary (`sap-devs-tray`) downloaded from GitHub Releases. `Manager` handles download, SHA256 checksum verification (via `tray-checksums.txt`), extraction (tar.gz/zip), start/stop (process management), and version-matched updates during `sap-devs update`. `autostart.go` provides cross-platform login startup registration: Windows registry (`HKCU\...\Run`), macOS LaunchAgent plist, Linux XDG `.desktop` file. The tray binary is stored at `~/.cache/sap-devs/bin/sap-devs-tray`. Config key: `config.Tray.Autostart`.
+
 ### CLI Commands
 
 | Command | Purpose |
@@ -150,6 +154,7 @@ On every command invocation (except `update` and dev builds), a background gorou
 | `update` | Self-update the binary |
 | `init` | First-time setup wizard |
 | `service install/uninstall/status` | Manage OS-native background scheduler (systemd/launchd/Task Scheduler) |
+| `tray install/uninstall/start/stop/status` | Download and manage optional GUI tray companion (Wails v3, experimental) |
 
 ### Project Detection & Health Check
 

cmd/tray.go

@@ -0,0 +1,181 @@
+package cmd
+
+import (
+	"bufio"
+	"fmt"
+	"os"
+	"strings"
+
+	"github.com/SAP-samples/sap-devs-cli/internal/config"
+	"github.com/SAP-samples/sap-devs-cli/internal/credentials"
+	"github.com/SAP-samples/sap-devs-cli/internal/trayctl"
+	"github.com/SAP-samples/sap-devs-cli/internal/xdg"
+	"github.com/spf13/cobra"
+)
+
+var trayCmd = &cobra.Command{
+	Use:   "tray",
+	Short: "Manage the optional GUI tray companion",
+	Long: `Manage the sap-devs system tray companion — an optional graphical dashboard
+that shows sync status, active profile, and injected tools at a glance.
+
+Note: The tray companion uses Wails v3 (currently in alpha).
+This is an optional enhancement — all CLI features work without it.`,
+}
+
+var trayInstallCmd = &cobra.Command{
+	Use:   "install",
+	Short: "Download and install the tray companion binary",
+	RunE: func(cmd *cobra.Command, args []string) error {
+		paths, err := xdg.New()
+		if err != nil {
+			return err
+		}
+
+		mgr := &trayctl.Manager{
+			CacheDir: paths.CacheDir,
+			Token:    credentials.Resolve(paths.ConfigDir),
+			Version:  Version,
+			RepoURL:  repoURL,
+		}
+
+		out := cmd.OutOrStdout()
+		fmt.Fprintln(out, "Downloading sap-devs-tray...")
+		if err := mgr.Install(); err != nil {
+			return err
+		}
+
+		fmt.Fprintln(out, "Verifying...")
+		if err := mgr.Verify(); err != nil {
+			return fmt.Errorf("verification failed: %w", err)
+		}
+
+		fmt.Fprintln(out, "Tray companion installed successfully.")
+		fmt.Fprintln(out)
+		fmt.Fprintln(out, "Note: The sap-devs tray companion uses Wails v3 (currently in alpha).")
+		fmt.Fprintln(out, "This is an optional enhancement — all CLI features work without it.")
+		fmt.Fprintln(out, "If you encounter issues, run `sap-devs tray uninstall` to remove it.")
+		fmt.Fprintln(out)
+
+		fmt.Fprint(out, "Start tray automatically on login? [Y/n] ")
+		reader := bufio.NewReader(os.Stdin)
+		answer, _ := reader.ReadString('\n')
+		answer = strings.TrimSpace(strings.ToLower(answer))
+		if answer == "" || answer == "y" || answer == "yes" {
+			if err := mgr.RegisterAutostart(); err != nil {
+				fmt.Fprintf(out, "Warning: could not register autostart: %v\n", err)
+			} else {
+				fmt.Fprintln(out, "Autostart registered.")
+				cfg, _ := config.Load(paths.ConfigDir)
+				cfg.Tray.Autostart = true
+				_ = cfg.Save(paths.ConfigDir)
+			}
+		}
+
+		return nil
+	},
+}
+
+var trayUninstallCmd = &cobra.Command{
+	Use:   "uninstall",
+	Short: "Remove the tray companion",
+	RunE: func(cmd *cobra.Command, args []string) error {
+		paths, err := xdg.New()
+		if err != nil {
+			return err
+		}
+		mgr := &trayctl.Manager{CacheDir: paths.CacheDir, Version: Version, RepoURL: repoURL}
+
+		if err := mgr.UnregisterAutostart(); err != nil {
+			fmt.Fprintf(cmd.ErrOrStderr(), "Warning: could not remove autostart entry: %v\n", err)
+		}
+		if err := mgr.Uninstall(); err != nil {
+			return err
+		}
+
+		cfg, _ := config.Load(paths.ConfigDir)
+		cfg.Tray.Autostart = false
+		_ = cfg.Save(paths.ConfigDir)
+
+		fmt.Fprintln(cmd.OutOrStdout(), "Tray companion uninstalled.")
+		return nil
+	},
+}
+
+var trayStartCmd = &cobra.Command{
+	Use:   "start",
+	Short: "Launch the tray companion",
+	RunE: func(cmd *cobra.Command, args []string) error {
+		paths, err := xdg.New()
+		if err != nil {
+			return err
+		}
+		mgr := &trayctl.Manager{CacheDir: paths.CacheDir}
+		if err := mgr.Start(); err != nil {
+			return err
+		}
+		fmt.Fprintln(cmd.OutOrStdout(), "Tray companion started.")
+		return nil
+	},
+}
+
+var trayStopCmd = &cobra.Command{
+	Use:   "stop",
+	Short: "Stop the running tray companion",
+	RunE: func(cmd *cobra.Command, args []string) error {
+		paths, err := xdg.New()
+		if err != nil {
+			return err
+		}
+		mgr := &trayctl.Manager{CacheDir: paths.CacheDir}
+		if err := mgr.Stop(); err != nil {
+			return err
+		}
+		fmt.Fprintln(cmd.OutOrStdout(), "Tray companion stopped.")
+		return nil
+	},
+}
+
+var trayStatusCmd = &cobra.Command{
+	Use:   "status",
+	Short: "Show tray companion status",
+	RunE: func(cmd *cobra.Command, args []string) error {
+		paths, err := xdg.New()
+		if err != nil {
+			return err
+		}
+		mgr := &trayctl.Manager{CacheDir: paths.CacheDir, Version: Version}
+		out := cmd.OutOrStdout()
+
+		if !mgr.IsInstalled() {
+			fmt.Fprintln(out, "Tray: not installed")
+			fmt.Fprintln(out, "Run `sap-devs tray install` to download the tray companion.")
+			return nil
+		}
+
+		running := "stopped"
+		if mgr.IsRunning() {
+			running = "running"
+		}
+
+		cfg, _ := config.Load(paths.ConfigDir)
+		autostart := "disabled"
+		if cfg.Tray.Autostart {
+			autostart = "enabled"
+		}
+
+		fmt.Fprintf(out, "Tray:      installed (%s)\n", running)
+		fmt.Fprintf(out, "Autostart: %s\n", autostart)
+		fmt.Fprintf(out, "Binary:    %s\n", mgr.BinaryPath())
+		return nil
+	},
+}
+
+func init() {
+	trayCmd.AddCommand(trayInstallCmd)
+	trayCmd.AddCommand(trayUninstallCmd)
+	trayCmd.AddCommand(trayStartCmd)
+	trayCmd.AddCommand(trayStopCmd)
+	trayCmd.AddCommand(trayStatusCmd)
+	rootCmd.AddCommand(trayCmd)
+}

cmd/update.go

@@ -7,6 +7,7 @@ import (
 	"github.com/spf13/cobra"
 	"github.com/SAP-samples/sap-devs-cli/internal/credentials"
 	"github.com/SAP-samples/sap-devs-cli/internal/i18n"
+	"github.com/SAP-samples/sap-devs-cli/internal/trayctl"
 	"github.com/SAP-samples/sap-devs-cli/internal/update"
 	"github.com/SAP-samples/sap-devs-cli/internal/xdg"
 )
@@ -49,6 +50,17 @@ var updateCmd = &cobra.Command{
 		}
 
 		fmt.Fprintln(cmd.OutOrStdout(), i18n.Tf(i18n.ActiveLang, "update.done", map[string]any{"TagName": rel.TagName}))
+
+		mgr := &trayctl.Manager{CacheDir: paths.CacheDir, Version: rel.Version, Token: token, RepoURL: repoURL}
+		if mgr.IsInstalled() {
+			fmt.Fprintln(cmd.OutOrStdout(), "Updating tray companion...")
+			if err := mgr.Install(); err != nil {
+				fmt.Fprintf(cmd.ErrOrStderr(), "Warning: tray update failed: %v\n", err)
+			} else {
+				fmt.Fprintln(cmd.OutOrStdout(), "Tray companion updated.")
+			}
+		}
+
 		return nil
 	},
 }

internal/config/config.go

@@ -19,6 +19,7 @@ type Config struct {
 	Tutorial        TutorialConfig `yaml:"tutorial,omitempty"`
 	ExperienceLevel string         `yaml:"experience_level,omitempty"`
 	Service         ServiceConfig  `yaml:"service,omitempty"`
+	Tray            TrayConfig     `yaml:"tray,omitempty"`
 }
 
 // SyncConfig controls per-category TTLs for background content refresh.
@@ -60,6 +61,11 @@ type ServiceConfig struct {
 	Interval time.Duration `yaml:"interval"`
 }
 
+// TrayConfig controls the optional GUI tray companion.
+type TrayConfig struct {
+	Autostart bool `yaml:"autostart,omitempty"`
+}
+
 func (e EventsConfig) EffectiveLocalRadius() int {
 	if e.LocalRadius > 0 {
 		return e.LocalRadius

internal/config/config_test.go

@@ -159,3 +159,21 @@ func TestServiceConfig_RoundTrip(t *testing.T) {
 	require.NoError(t, err)
 	assert.Equal(t, 12*time.Hour, loaded.Service.Interval)
 }
+
+func TestTrayConfig_Defaults(t *testing.T) {
+	dir := t.TempDir()
+	cfg, err := config.Load(dir)
+	require.NoError(t, err)
+	assert.False(t, cfg.Tray.Autostart)
+}
+
+func TestTrayConfig_RoundTrip(t *testing.T) {
+	dir := t.TempDir()
+	cfg := config.Default()
+	cfg.Tray.Autostart = true
+	require.NoError(t, cfg.Save(dir))
+
+	loaded, err := config.Load(dir)
+	require.NoError(t, err)
+	assert.True(t, loaded.Tray.Autostart)
+}

internal/trayctl/autostart.go

@@ -0,0 +1,122 @@
+package trayctl
+
+import (
+	"fmt"
+	"os"
+	"os/exec"
+	"path/filepath"
+	"runtime"
+)
+
+func autostartEntryName() string {
+	switch runtime.GOOS {
+	case "windows":
+		return "sap-devs-tray"
+	case "darwin":
+		return "com.sap-devs.tray"
+	default:
+		return "sap-devs-tray.desktop"
+	}
+}
+
+func (m *Manager) RegisterAutostart() error {
+	binaryPath := m.BinaryPath()
+	switch runtime.GOOS {
+	case "windows":
+		return registerWindowsAutostart(binaryPath)
+	case "darwin":
+		return registerDarwinAutostart(binaryPath)
+	case "linux":
+		return registerLinuxAutostart(binaryPath)
+	default:
+		return fmt.Errorf("autostart not supported on %s", runtime.GOOS)
+	}
+}
+
+func (m *Manager) UnregisterAutostart() error {
+	switch runtime.GOOS {
+	case "windows":
+		return unregisterWindowsAutostart()
+	case "darwin":
+		return unregisterDarwinAutostart()
+	case "linux":
+		return unregisterLinuxAutostart()
+	default:
+		return nil
+	}
+}
+
+func registerWindowsAutostart(binaryPath string) error {
+	cmd := exec.Command("reg", "add",
+		`HKCU\Software\Microsoft\Windows\CurrentVersion\Run`,
+		"/v", "sap-devs-tray",
+		"/t", "REG_SZ",
+		"/d", binaryPath,
+		"/f",
+	)
+	return cmd.Run()
+}
+
+func unregisterWindowsAutostart() error {
+	cmd := exec.Command("reg", "delete",
+		`HKCU\Software\Microsoft\Windows\CurrentVersion\Run`,
+		"/v", "sap-devs-tray",
+		"/f",
+	)
+	return cmd.Run()
+}
+
+func registerDarwinAutostart(binaryPath string) error {
+	plist := fmt.Sprintf(`<?xml version="1.0" encoding="UTF-8"?>
+<!DOCTYPE plist PUBLIC "-//Apple//DTD PLIST 1.0//EN" "http://www.apple.com/DTDs/PropertyList-1.0.dtd">
+<plist version="1.0">
+<dict>
+	<key>Label</key>
+	<string>com.sap-devs.tray</string>
+	<key>ProgramArguments</key>
+	<array>
+		<string>%s</string>
+	</array>
+	<key>RunAtLoad</key>
+	<true/>
+</dict>
+</plist>`, binaryPath)
+
+	home, _ := os.UserHomeDir()
+	path := filepath.Join(home, "Library", "LaunchAgents", "com.sap-devs.tray.plist")
+	if err := os.MkdirAll(filepath.Dir(path), 0755); err != nil {
+		return err
+	}
+	return os.WriteFile(path, []byte(plist), 0644)
+}
+
+func unregisterDarwinAutostart() error {
+	home, _ := os.UserHomeDir()
+	path := filepath.Join(home, "Library", "LaunchAgents", "com.sap-devs.tray.plist")
+	_ = exec.Command("launchctl", "unload", path).Run()
+	return os.Remove(path)
+}
+
+func registerLinuxAutostart(binaryPath string) error {
+	entry := fmt.Sprintf(`[Desktop Entry]
+Type=Application
+Name=sap-devs Tray
+Exec=%s
+Terminal=false
+StartupNotify=false
+X-GNOME-Autostart-enabled=true
+`, binaryPath)
+
+	home, _ := os.UserHomeDir()
+	dir := filepath.Join(home, ".config", "autostart")
+	if err := os.MkdirAll(dir, 0755); err != nil {
+		return err
+	}
+	return os.WriteFile(filepath.Join(dir, "sap-devs-tray.desktop"), []byte(entry), 0644)
+}
+
+func unregisterLinuxAutostart() error {
+	home, _ := os.UserHomeDir()
+	path := filepath.Join(home, ".config", "autostart", "sap-devs-tray.desktop")
+	return os.Remove(path)
+}