🐛 fix(crux): /v1/health reports tmux and returns 503 when it is missing

changelog.d/fixed-crux-health-tmux.md

@@ -0,0 +1 @@
+- Crux `GET /v1/health` now reports a `tmux` field (where tmux resolves on PATH) and answers 503, with the full health body, when tmux is missing: every agent runs in tmux, so a server without it cannot serve traffic and was reporting healthy ([hivecommons/crux#137](https://github.com/hivecommons/crux/issues/137)). Existing fields are unchanged; probes that treated any 200 as healthy now see the failure.

src/crux/api/api.go

@@ -105,6 +105,9 @@ type Health struct {
 	// that restarted the server sees the uptime reset.
 	StartedAt     time.Time `json:"started_at"`
 	UptimeSeconds float64   `json:"uptime_seconds"`
+	// Tmux is where tmux resolves on PATH, "" when it does not. Every agent
+	// runs inside tmux, so the route answers 503 (with this body) when empty.
+	Tmux string `json:"tmux"`
 }
 
 // HiveCallback is the wire form of the callback client's status.

src/crux/serve/families_test.go

@@ -884,6 +884,7 @@ func TestServeWithoutHostOrBackground(t *testing.T) {
 	if len(rt.Background()) != 0 {
 		t.Fatal("background loops started while disabled")
 	}
+	t.Setenv("PATH", fakeTmuxDir(t))
 	var h api.Health
 	newFamServer(t).expect(http.MethodGet, api.PathHealth, nil, http.StatusOK, &h)
 	if len(h.Background) != 0 {

src/crux/serve/health_test.go

@@ -0,0 +1,42 @@
+package serve
+
+import (
+	"net/http"
+	"os"
+	"os/exec"
+	"path/filepath"
+	"testing"
+
+	"github.com/hivecommons/hive/crux/api"
+)
+
+// fakeTmuxDir returns a directory holding an executable named tmux, for a
+// PATH that resolves it without needing a real install.
+func fakeTmuxDir(t *testing.T) string {
+	t.Helper()
+	dir := t.TempDir()
+	if err := os.WriteFile(filepath.Join(dir, "tmux"), []byte("#!/bin/sh\n"), 0o755); err != nil {
+		t.Fatal(err)
+	}
+	return dir
+}
+
+func TestHealthTmuxMissingIs503(t *testing.T) {
+	t.Setenv("PATH", t.TempDir())
+	var h api.Health
+	newFamServer(t).expect(http.MethodGet, api.PathHealth, nil, http.StatusServiceUnavailable, &h)
+	if h.Tmux != "" || h.Version != api.Version {
+		t.Fatalf("health: %+v", h)
+	}
+}
+
+func TestHealthTmuxPresentIs200(t *testing.T) {
+	if _, err := exec.LookPath("tmux"); err != nil {
+		t.Skip("tmux not installed")
+	}
+	var h api.Health
+	newFamServer(t).expect(http.MethodGet, api.PathHealth, nil, http.StatusOK, &h)
+	if h.Tmux == "" {
+		t.Fatalf("health: %+v", h)
+	}
+}

src/crux/serve/serve.go

@@ -34,6 +34,7 @@ import (
 	"github.com/hivecommons/hive/crux/internal/privatedir"
 
 	"github.com/hivecommons/hive/crux"
+	"github.com/hivecommons/hive/crux/agent/tmux"
 	"github.com/hivecommons/hive/crux/api"
 	"github.com/hivecommons/hive/crux/checkout"
 	"github.com/hivecommons/hive/crux/config"
@@ -388,7 +389,13 @@ func (s *Server) handleHealth(w http.ResponseWriter, _ *http.Request) {
 		hc := s.HiveCallback()
 		h.HiveCallback = &hc
 	}
-	s.writeJSON(w, http.StatusOK, h)
+	status := http.StatusOK
+	if path, err := tmux.LookPath(); err != nil {
+		status = http.StatusServiceUnavailable
+	} else {
+		h.Tmux = path
+	}
+	s.writeJSON(w, status, h)
 }
 
 func (s *Server) handleList(w http.ResponseWriter, _ *http.Request) {