From 97879238799386205ea9a2df9ba701bf161ca05d Mon Sep 17 00:00:00 2001
From: nathaninline <mail@nathaninline.com>
Date: Sat, 25 Jul 2026 11:37:42 +0200
Subject: [PATCH] macOS : service en mode utilisateur quand aucun LaunchDaemon
 n'est installe

Jean.app tourne sans root : launchctl load echouait sur un plist absent et
l'UI restait bloquee sur << service arrete >>. On bascule alors sur le meme
mecanisme que Windows (jean serve detache + fichier PID), avec setsid pour
survivre a la fermeture de l'app et un kill par groupe pour emporter
llama-server. Ajoute aussi l'action 'status', qui manquait cote darwin.
---
 internal/jean/sys_service_darwin.go | 167 ++++++++++++++++++++++++++++
 1 file changed, 167 insertions(+)

diff --git a/internal/jean/sys_service_darwin.go b/internal/jean/sys_service_darwin.go
index 1a2f7ea..c2d046b 100644
--- a/internal/jean/sys_service_darwin.go
+++ b/internal/jean/sys_service_darwin.go
@@ -7,7 +7,9 @@ import (
 	"os"
 	"os/exec"
 	"path/filepath"
+	"strconv"
 	"strings"
+	"syscall"
 	"time"
 )
 
@@ -34,6 +36,14 @@ func launchdLogPath() string { return filepath.Join(JeanHome(), serviceName()+".
 func serviceAction(action string) error {
 	svc := serviceName()
 	plist := launchdPlistPath(svc)
+	// Pas de LaunchDaemon installé — cas normal d'un Mac de bureau, où Jean tourne
+	// via Jean.app sans jamais passer par `sudo jean install` : on gère alors le
+	// service comme sous Windows, avec un `jean serve` détaché suivi par un fichier
+	// PID. Aucun droit root requis. Sans ça, `start` échouait sur un plist absent
+	// et l'UI affichait éternellement « service arrêté ».
+	if _, err := os.Stat(plist); err != nil {
+		return userSvcAction(action)
+	}
 	run := func(args ...string) error {
 		bin, a := "launchctl", args
 		if os.Geteuid() != 0 {
@@ -61,6 +71,15 @@ func serviceAction(action string) error {
 			return err
 		}
 		return checkStarted(svc)
+	case "status":
+		if serviceIsActive() {
+			fmt.Printf("%s %s: actif (launchd)\n", green("[ok]"), svc)
+		} else {
+			fmt.Printf("%s %s: arrêté\n", yellow("[info]"), svc)
+		}
+		fmt.Printf("  logs   : %s\n", launchdLogPath())
+		fmt.Printf("  config : %s\n", confPath())
+		return nil
 	}
 	return fmt.Errorf("action inconnue: %s", action)
 }
@@ -94,10 +113,158 @@ func serviceLogs() error {
 
 // serviceIsActive : le service est chargé ET possède un PID courant. `launchctl
 // list <label>` renvoie un dict incluant la clé "PID" uniquement quand il tourne.
+// Sans LaunchDaemon installé, on lit le fichier PID du mode utilisateur.
 func serviceIsActive() bool {
+	if _, err := os.Stat(launchdPlistPath(serviceName())); err != nil {
+		pid := readServicePID()
+		return pid > 0 && processAlive(pid)
+	}
 	out, err := exec.Command("launchctl", "list", launchdLabel(serviceName())).Output()
 	if err != nil {
 		return false
 	}
 	return strings.Contains(string(out), "\"PID\"")
 }
+
+// ---------------------------------------------------------------------------
+// Mode utilisateur (sans launchd) — même approche que Windows : `jean serve` est
+// lancé détaché, son PID va dans JEAN_HOME/<svc>.pid et sa sortie dans
+// JEAN_HOME/<svc>.log. Setsid le détache de notre session pour qu'il survive à
+// la fermeture de Jean.app, et permet de tuer tout le groupe (llama-server
+// enfant compris) d'un seul signal.
+
+func pidFilePath() string { return filepath.Join(JeanHome(), serviceName()+".pid") }
+func logFilePath() string { return filepath.Join(JeanHome(), serviceName()+".log") }
+
+func userSvcAction(action string) error {
+	switch action {
+	case "start":
+		return userSvcStart()
+	case "stop":
+		return userSvcStop(true)
+	case "restart":
+		_ = userSvcStop(false)
+		time.Sleep(500 * time.Millisecond)
+		return userSvcStart()
+	case "status":
+		pid := readServicePID()
+		if pid > 0 && processAlive(pid) {
+			fmt.Printf("%s %s: actif (PID %d)\n", green("[ok]"), serviceName(), pid)
+		} else {
+			fmt.Printf("%s %s: arrêté\n", yellow("[info]"), serviceName())
+		}
+		fmt.Printf("  logs   : %s\n", logFilePath())
+		fmt.Printf("  config : %s\n", confPath())
+		return nil
+	case "enable", "disable":
+		fmt.Printf("%s '%s' réclame un service système : lance %s pour installer le LaunchDaemon.\n",
+			yellow("[info]"), action, bold("sudo jean install"))
+		return nil
+	}
+	return fmt.Errorf("action inconnue: %s", action)
+}
+
+func userSvcStart() error {
+	if pid := readServicePID(); pid > 0 && processAlive(pid) {
+		fmt.Printf("%s déjà démarré (PID %d)\n", yellow("[info]"), pid)
+		return nil
+	}
+	self, err := os.Executable()
+	if err != nil {
+		return err
+	}
+	if err := os.MkdirAll(JeanHome(), 0o755); err != nil {
+		return err
+	}
+	logf, err := os.OpenFile(logFilePath(), os.O_CREATE|os.O_WRONLY|os.O_APPEND, 0o644)
+	if err != nil {
+		return fmt.Errorf("ouverture du log %s: %w", logFilePath(), err)
+	}
+	defer logf.Close()
+
+	cmd := exec.Command(self, "serve")
+	cmd.Stdout, cmd.Stderr = logf, logf
+	cmd.SysProcAttr = &syscall.SysProcAttr{Setsid: true}
+	if err := cmd.Start(); err != nil {
+		return fmt.Errorf("démarrage de 'jean serve': %w", err)
+	}
+	pid := cmd.Process.Pid
+	if err := os.WriteFile(pidFilePath(), []byte(strconv.Itoa(pid)), 0o644); err != nil {
+		return fmt.Errorf("écriture du PID: %w", err)
+	}
+	_ = cmd.Process.Release()
+	return userCheckStarted(pid)
+}
+
+func userCheckStarted(pid int) error {
+	time.Sleep(2 * time.Second)
+	if processAlive(pid) {
+		fmt.Printf("%s %s: démarré (PID %d)\n", green("[ok]"), serviceName(), pid)
+		fmt.Printf("       logs: %s  (jean logs pour suivre)\n", dim(logFilePath()))
+		return nil
+	}
+	fmt.Printf("%s %s: le processus s'est arrêté — derniers logs :\n", red("[ERREUR]"), serviceName())
+	fmt.Println("------------------------------------------------")
+	fmt.Print(tailFile(logFilePath(), 20))
+	fmt.Println("------------------------------------------------")
+	fmt.Printf("→ jean logs   pour plus de détails\n→ jean edit   pour corriger config.env\n")
+	_ = os.Remove(pidFilePath())
+	return fmt.Errorf("service %s non démarré", serviceName())
+}
+
+func userSvcStop(verbose bool) error {
+	pid := readServicePID()
+	if pid <= 0 || !processAlive(pid) {
+		_ = os.Remove(pidFilePath())
+		if verbose {
+			fmt.Println(yellow("[info]") + " aucun service en cours d'exécution")
+		}
+		return nil
+	}
+	// Setsid a fait de l'enfant un chef de groupe : le PID négatif vise le groupe
+	// entier, donc llama-server s'arrête avec lui.
+	if err := syscall.Kill(-pid, syscall.SIGTERM); err != nil {
+		_ = syscall.Kill(pid, syscall.SIGTERM)
+	}
+	for i := 0; i < 40 && processAlive(pid); i++ {
+		time.Sleep(100 * time.Millisecond)
+	}
+	if processAlive(pid) {
+		_ = syscall.Kill(-pid, syscall.SIGKILL)
+	}
+	_ = os.Remove(pidFilePath())
+	if verbose {
+		fmt.Println(green("[ok]") + " arrêté")
+	}
+	return nil
+}
+
+func readServicePID() int {
+	b, err := os.ReadFile(pidFilePath())
+	if err != nil {
+		return 0
+	}
+	pid, _ := strconv.Atoi(strings.TrimSpace(string(b)))
+	return pid
+}
+
+// processAlive : le signal 0 ne tue rien, il teste juste l'existence du process.
+func processAlive(pid int) bool {
+	if pid <= 0 {
+		return false
+	}
+	return syscall.Kill(pid, 0) == nil
+}
+
+// tailFile renvoie les n dernières lignes d'un fichier (best-effort).
+func tailFile(path string, n int) string {
+	b, err := os.ReadFile(path)
+	if err != nil {
+		return ""
+	}
+	lines := strings.Split(strings.TrimRight(string(b), "\n"), "\n")
+	if len(lines) > n {
+		lines = lines[len(lines)-n:]
+	}
+	return strings.Join(lines, "\n") + "\n"
+}