Use kit daemon lifecycle primitives

[mariusvniekerk]

Summary

• replace roborev's custom daemon endpoint/runtime/start plumbing with go.kenn.io/kit daemon primitives
• drop legacy daemon runtime compatibility for addr/port/daemon.json and require kit-style ping responses
• keep roborev-specific server, worker, shutdown, and API behavior while delegating shared lifecycle concerns to the kit dependency

Validation

• go fmt ./cmd/roborev ./internal/daemon
• go vet ./...
• go test ./...
• pre-commit golangci-lint hook

[roborev-ci]

roborev: Fail

Changes need one medium contract-sync fix before merge.

Medium

• internal/daemon_client/client.gen.go:425: PingInfo now includes the required ok field and /api/ping returns it, but the checked-in generated client and openapi-3.0.json still omit it. This leaves the daemon API contract and generated client out of sync.
  • Fix: Regenerate and commit the OpenAPI spec and daemon client after the PingInfo shape change.

---

Review type: | Agent: codex | Job: 19339

[roborev-ci]

roborev: Combined Review (ac4a839)

No issues found.

---

Synthesized from 2 reviews (agents: codex | types: default, security)

[roborev-ci]

roborev: Fail

Likely needs one medium fix: update the Nix vendorHash for the changed Go dependency.

Medium

• flake.nix:23: go.mod now references a new go.kenn.io/kit pseudo-version, but vendorHash was not updated. Nix buildGoModule vendoring will likely fail with a fixed-output hash mismatch. Regenerate the Nix vendor hash after the final go.mod/go.sum changes and update vendorHash.

---

Review type: | Agent: codex | Job: 19381

[roborev-ci]

roborev: Combined Review (1a5c2fc)

No issues found.

---

Synthesized from 2 reviews (agents: codex | types: default, security)

[roborev-ci]

roborev: Combined Review (f43e28c)

No issues found.

---

Synthesized from 2 reviews (agents: codex | types: default, security)