fix: minor changes on update

cmd/server/server.go

@@ -232,6 +232,7 @@ func runServer() {
 	meta.Register(api, mods)
 	meta.RegisterHealth(api, sessions)
 	meta.RegisterWhoami(api, sessions, roles, mods)
+	meta.ConfigPath = configPath
 	meta.RegisterUpdate(api)
 
 	auth.RegisterLogin(api, sessions, auditStore, cfg.SecureCookie())

internal/meta/update.go

@@ -6,11 +6,16 @@ import (
 	"os/exec"
 	"syscall"
 
+	"nadir/internal/config"
 	"nadir/internal/oscmd"
 
 	"github.com/danielgtaylor/huma/v2"
 )
 
+// ConfigPath is set at startup so the update handler can re-load config and
+// surface release_repo / parse errors to the caller instead of only stderr.
+var ConfigPath string
+
 // RegisterUpdate wires POST /api/meta/update. It runs the equivalent of
 // `sudo nadir update` in a detached session and returns 202 immediately; the
 // systemctl restart that ends the updater drops in-flight connections, so the
@@ -28,9 +33,18 @@ func RegisterUpdate(api huma.API) {
 		Description:   "Equivalent to running `sudo nadir update` on the host: queries server.release_repo for the latest release, downloads the binary matching the host's architecture, atomically replaces the running binary, and restarts the systemd unit. Returns 202 immediately; the service restart drops in-flight connections, so poll /api/health to confirm the new version is up. Requires the wildcard admin role.",
 		Tags:          []string{"Meta"},
 		Metadata:      map[string]any{"module": "meta", "permission": "root"},
-		Errors:        []int{401, 403, 500},
+		Errors:        []int{400, 401, 403, 500},
 		DefaultStatus: 202,
 	}, func(ctx context.Context, _ *struct{}) (*oscmd.StatusOutput, error) {
+		if ConfigPath != "" {
+			cfg, err := config.Load(ConfigPath)
+			if err != nil {
+				return nil, huma.Error500InternalServerError("config load failed", err)
+			}
+			if cfg.Server.ReleaseRepo == "" {
+				return nil, huma.Error400BadRequest("server.release_repo not set in " + ConfigPath)
+			}
+		}
 		exe, err := os.Executable()
 		if err != nil {
 			return nil, huma.Error500InternalServerError("could not resolve own binary path", err)

internal/modules/system/info.go

@@ -159,9 +159,18 @@ func cpuInfo() CPUInfo {
 	c := CPUInfo{Model: cpuModel(string(data)), LogicalCPUs: runtime.NumCPU()}
 	c.MinMHz, c.MaxMHz, c.CurrentMHz = cpuFreqMHz("/sys/devices/system/cpu")
 	// ponytail: cpufreq sysfs is absent on many VMs and stock Ubuntu server
-	// kernels; fall back to /proc/cpuinfo "cpu MHz" so CurrentMHz isn't 0.
+	// kernels; fall back to /proc/cpuinfo "cpu MHz" — VMs have a fixed clock,
-	if c.CurrentMHz == 0 {
+	// so min == max == cur is the honest answer.
-		c.CurrentMHz = cpuinfoMaxMHz(string(data))
+	if mhz := cpuinfoMaxMHz(string(data)); mhz > 0 {
+		if c.CurrentMHz == 0 {
+			c.CurrentMHz = mhz
+		}
+		if c.MaxMHz == 0 {
+			c.MaxMHz = mhz
+		}
+		if c.MinMHz == 0 {
+			c.MinMHz = mhz
+		}
 	}
 	return c
 }