mirror of
https://github.com/chenhg5/cc-connect.git
synced 2026-07-03 12:28:10 +08:00
* feat(daemon): add CC_LOG_MAX_BACKUPS env var support (#1222) PR #1243 only addressed CC_LOG_MAX_SIZE while leaving the backup count hard-wired to one (.log.1). That still loses any post-mortem context older than one rotation, which is the same class of failure users reported on #1222. This change adds the matching knob so the post-mortem trail is configurable, with the same flag > env > default priority used for size. - daemon: add ParseLogBackups(s) (>=1, no unit suffix, error echoes input) and DefaultLogMaxBackups = 3. - daemon: extend RotatingWriter with maxBackups; rotateLocked walks the chain (delete .N, shift .(N-1) -> .N .. .1 -> .2, rename active -> .1, reopen) and a public Rotate() hook for tests/SIGHUP. - daemon: Config/Meta gain LogMaxBackups; Resolve() defaults to 3. - cmd/cc-connect: resolveLogMaxBackups + preScanLogMaxBackupsFlag + --log-max-backups flag; startup log now reports max_backups and its source. The rotating-writer setup happens before flag.Parse so the pre-scan keeps the flag effective there too. - daemon/launchd.go, daemon/systemd.go, daemon/windows.go: thread CC_LOG_MAX_BACKUPS through the service templates so a fresh install picks it up. - tests: TestParseLogBackups (19 subtests + error-echo), three new RotatingWriter tests (chain, disabled, fallback), four resolver tests + pre-scan tests in cmd/cc-connect. TestIssue1222_BackupRetention pins the new env-var behaviour as the regression test for the follow-up to #1222. * fix(daemon): silence errcheck on logrotate_test.go defer Close QA review (run 27109765660) flagged defer w.Close() in the 4 backup-related tests added by #1260. Wrap each in defer func() { _ = w.Close() }() so errcheck is satisfied without changing test semantics (temp-dir cleanup is best-effort). Verified locally: - golangci-lint --new-from-rev origin/main ./daemon/... -> 0 issues - go test -count=1 -tags no_web ./daemon/ ./cmd/cc-connect/ -> ok --------- Co-authored-by: cc-connect dev-claudecode <dev-claudecode@cc-connect.local> Co-authored-by: Claude <noreply@anthropic.com>
354 lines
9.1 KiB
Go
354 lines
9.1 KiB
Go
//go:build darwin
|
|
|
|
package daemon
|
|
|
|
import (
|
|
"encoding/xml"
|
|
"fmt"
|
|
"log/slog"
|
|
"os"
|
|
"os/exec"
|
|
"path/filepath"
|
|
"sort"
|
|
"strconv"
|
|
"strings"
|
|
"time"
|
|
)
|
|
|
|
const (
|
|
launchdLabel = "com.cc-connect.service"
|
|
)
|
|
|
|
var runLaunchctl = func(args ...string) (string, error) {
|
|
cmd := exec.Command("launchctl", args...)
|
|
out, err := cmd.CombinedOutput()
|
|
return strings.TrimSpace(string(out)), err
|
|
}
|
|
|
|
type launchdManager struct{}
|
|
|
|
// CheckLinger always returns true on macOS: launchd user agents persist
|
|
// independently of login sessions, so no "linger" warning is needed.
|
|
func CheckLinger() (enabled bool, user string) {
|
|
return true, ""
|
|
}
|
|
|
|
func newPlatformManager() (Manager, error) {
|
|
return &launchdManager{}, nil
|
|
}
|
|
|
|
func (*launchdManager) Platform() string { return "launchd" }
|
|
|
|
func (m *launchdManager) Install(cfg Config) error {
|
|
plistPath := launchdPlistPath()
|
|
|
|
if err := os.MkdirAll(filepath.Dir(plistPath), 0755); err != nil {
|
|
return fmt.Errorf("create LaunchAgents dir: %w", err)
|
|
}
|
|
if err := os.MkdirAll(filepath.Dir(cfg.LogFile), 0755); err != nil {
|
|
return fmt.Errorf("create log dir: %w", err)
|
|
}
|
|
|
|
// Unload existing service first (ignore errors) so we do not leave a stale
|
|
// job behind when switching between GUI and headless sessions.
|
|
bootoutLaunchdTargets()
|
|
|
|
plist := buildPlist(cfg)
|
|
// 0600: plist may contain captured secret values (config.toml ${ENV}
|
|
// placeholders and any EnvDiscoverer extension output). User-only
|
|
// LaunchAgents path; root can still read but that is the user's own
|
|
// machine boundary. os.WriteFile only applies perm on create, so
|
|
// Chmod afterwards is required to harden reinstalls of files that
|
|
// pre-existed at 0644 from earlier cc-connect versions.
|
|
if err := os.WriteFile(plistPath, []byte(plist), 0600); err != nil {
|
|
return fmt.Errorf("write plist: %w", err)
|
|
}
|
|
if err := os.Chmod(plistPath, 0600); err != nil {
|
|
return fmt.Errorf("chmod plist: %w", err)
|
|
}
|
|
|
|
domain := preferredLaunchdDomain()
|
|
if out, err := runLaunchctl("bootstrap", domain, plistPath); err != nil {
|
|
return fmt.Errorf("launchctl bootstrap: %s (%w)", out, err)
|
|
}
|
|
|
|
if _, err := runLaunchctl("kickstart", "-kp", launchdTarget(domain)); err != nil {
|
|
return fmt.Errorf("launchctl kickstart: %w", err)
|
|
}
|
|
return nil
|
|
}
|
|
|
|
func (m *launchdManager) Uninstall() error {
|
|
bootoutLaunchdTargets()
|
|
|
|
plistPath := launchdPlistPath()
|
|
if err := os.Remove(plistPath); err != nil && !os.IsNotExist(err) {
|
|
return fmt.Errorf("remove plist: %w", err)
|
|
}
|
|
return nil
|
|
}
|
|
|
|
func (*launchdManager) Start() error {
|
|
if _, target, _, ok := loadedLaunchdTarget(); ok {
|
|
out, err := runLaunchctl("kickstart", "-kp", target)
|
|
if err != nil {
|
|
return fmt.Errorf("start: %s (%w)", out, err)
|
|
}
|
|
return nil
|
|
}
|
|
|
|
domain := preferredLaunchdDomain()
|
|
plistPath := launchdPlistPath()
|
|
var out string
|
|
if _, err := runLaunchctl("bootstrap", domain, plistPath); err != nil {
|
|
// already bootstrapped — try kickstart
|
|
out, err = runLaunchctl("kickstart", "-kp", launchdTarget(domain))
|
|
if err != nil {
|
|
return fmt.Errorf("start: %s (%w)", out, err)
|
|
}
|
|
}
|
|
return nil
|
|
}
|
|
|
|
func (*launchdManager) Stop() error {
|
|
var lastOut string
|
|
var lastErr error
|
|
for _, target := range launchdTargets() {
|
|
out, err := runLaunchctl("bootout", target)
|
|
if err == nil {
|
|
return nil
|
|
}
|
|
lastOut = out
|
|
lastErr = err
|
|
}
|
|
if lastErr != nil {
|
|
return fmt.Errorf("stop: %s (%w)", lastOut, lastErr)
|
|
}
|
|
return nil
|
|
}
|
|
|
|
func (*launchdManager) Restart() error {
|
|
domain := preferredLaunchdDomain()
|
|
if loadedDomain, _, _, ok := loadedLaunchdTarget(); ok && domain != launchdGUIDomain() {
|
|
domain = loadedDomain
|
|
}
|
|
target := launchdTarget(domain)
|
|
bootoutLaunchdTargets()
|
|
|
|
plistPath := launchdPlistPath()
|
|
|
|
// launchd bootout is asynchronous; retry bootstrap with backoff
|
|
// to avoid "Bootstrap failed: 5" race condition.
|
|
var out string
|
|
var err error
|
|
for i := 0; i < 3; i++ {
|
|
if i > 0 {
|
|
time.Sleep(500 * time.Millisecond)
|
|
}
|
|
out, err = runLaunchctl("bootstrap", domain, plistPath)
|
|
if err == nil {
|
|
break
|
|
}
|
|
}
|
|
if err != nil {
|
|
return fmt.Errorf("restart: %s (%w)", out, err)
|
|
}
|
|
if _, err := runLaunchctl("kickstart", "-kp", target); err != nil {
|
|
return fmt.Errorf("restart kickstart: %w", err)
|
|
}
|
|
return nil
|
|
}
|
|
|
|
func (*launchdManager) Status() (*Status, error) {
|
|
st := &Status{Platform: "launchd"}
|
|
|
|
plistPath := launchdPlistPath()
|
|
if _, err := os.Stat(plistPath); err != nil {
|
|
return st, nil
|
|
}
|
|
st.Installed = true
|
|
|
|
_, _, out, ok := loadedLaunchdTarget()
|
|
if !ok {
|
|
return st, nil
|
|
}
|
|
|
|
for _, line := range strings.Split(out, "\n") {
|
|
trimmed := strings.TrimSpace(line)
|
|
if strings.HasPrefix(trimmed, "pid = ") {
|
|
if pid, err := strconv.Atoi(strings.TrimPrefix(trimmed, "pid = ")); err == nil && pid > 0 {
|
|
st.PID = pid
|
|
st.Running = true
|
|
}
|
|
}
|
|
if strings.Contains(trimmed, "state = running") {
|
|
st.Running = true
|
|
}
|
|
}
|
|
return st, nil
|
|
}
|
|
|
|
// ── helpers ─────────────────────────────────────────────────
|
|
|
|
func launchdPlistPath() string {
|
|
home, _ := os.UserHomeDir()
|
|
return filepath.Join(home, "Library", "LaunchAgents", launchdLabel+".plist")
|
|
}
|
|
|
|
func launchdUserDomain() string {
|
|
return fmt.Sprintf("user/%d", os.Getuid())
|
|
}
|
|
|
|
func launchdGUIDomain() string {
|
|
return fmt.Sprintf("gui/%d", os.Getuid())
|
|
}
|
|
|
|
func preferredLaunchdDomain() string {
|
|
guiDomain := launchdGUIDomain()
|
|
if _, err := runLaunchctl("print", guiDomain); err == nil {
|
|
return guiDomain
|
|
}
|
|
return launchdUserDomain()
|
|
}
|
|
|
|
func launchdDomains() []string {
|
|
preferred := preferredLaunchdDomain()
|
|
guiDomain := launchdGUIDomain()
|
|
userDomain := launchdUserDomain()
|
|
if preferred == guiDomain {
|
|
return []string{guiDomain, userDomain}
|
|
}
|
|
return []string{userDomain, guiDomain}
|
|
}
|
|
|
|
func launchdTarget(domain string) string {
|
|
return fmt.Sprintf("%s/%s", domain, launchdLabel)
|
|
}
|
|
|
|
func launchdTargets() []string {
|
|
domains := launchdDomains()
|
|
targets := make([]string, 0, len(domains))
|
|
for _, domain := range domains {
|
|
targets = append(targets, launchdTarget(domain))
|
|
}
|
|
return targets
|
|
}
|
|
|
|
func loadedLaunchdTarget() (string, string, string, bool) {
|
|
for _, domain := range launchdDomains() {
|
|
target := launchdTarget(domain)
|
|
out, err := runLaunchctl("print", target)
|
|
if err == nil {
|
|
return domain, target, out, true
|
|
}
|
|
}
|
|
return "", "", "", false
|
|
}
|
|
|
|
func bootoutLaunchdTargets() {
|
|
for _, target := range launchdTargets() {
|
|
_, _ = runLaunchctl("bootout", target)
|
|
}
|
|
}
|
|
|
|
// templateOwnedEnvKeys are keys the plist template renders directly; if
|
|
// they also appear in cfg.EnvExtra the template version wins.
|
|
var templateOwnedEnvKeys = map[string]struct{}{
|
|
"CC_LOG_FILE": {},
|
|
"CC_LOG_MAX_SIZE": {},
|
|
"PATH": {},
|
|
}
|
|
|
|
// renderEnvExtraPlist returns the serialized key/value pairs (without the
|
|
// surrounding <dict> wrapper) for cfg.EnvExtra, sorted by key and with
|
|
// invalid keys / empty values dropped. Both keys and values are XML-escaped.
|
|
func renderEnvExtraPlist(envExtra map[string]string) string {
|
|
if len(envExtra) == 0 {
|
|
return ""
|
|
}
|
|
keys := make([]string, 0, len(envExtra))
|
|
for k := range envExtra {
|
|
if _, owned := templateOwnedEnvKeys[k]; owned {
|
|
continue
|
|
}
|
|
if !isValidEnvName(k) {
|
|
slog.Warn("daemon: launchd: dropping invalid env name from EnvExtra",
|
|
"key", k)
|
|
continue
|
|
}
|
|
if envExtra[k] == "" {
|
|
continue
|
|
}
|
|
keys = append(keys, k)
|
|
}
|
|
sort.Strings(keys)
|
|
var b strings.Builder
|
|
for _, k := range keys {
|
|
fmt.Fprintf(&b, "\t\t<key>%s</key>\n\t\t<string>%s</string>\n",
|
|
xmlEscape(k), xmlEscape(envExtra[k]))
|
|
}
|
|
return b.String()
|
|
}
|
|
|
|
func xmlEscape(s string) string {
|
|
var b strings.Builder
|
|
_ = xml.EscapeText(&b, []byte(s))
|
|
return b.String()
|
|
}
|
|
|
|
func buildPlist(cfg Config) string {
|
|
envPATH := cfg.EnvPATH
|
|
if envPATH == "" {
|
|
envPATH = "/usr/local/bin:/usr/bin:/bin:/opt/homebrew/bin"
|
|
}
|
|
envExtra := renderEnvExtraPlist(cfg.EnvExtra)
|
|
// User-supplied paths can legitimately contain XML-special characters
|
|
// ('&', '<', '>', '"', '\''). Without escaping, `launchctl bootstrap`
|
|
// rejects the plist with a parse error and daemon install fails. The
|
|
// label is a hard-coded constant; LogMaxSize is an int; envExtra is
|
|
// escaped by renderEnvExtraPlist.
|
|
return 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>%s</string>
|
|
<key>ProgramArguments</key>
|
|
<array>
|
|
<string>%s</string>
|
|
</array>
|
|
<key>WorkingDirectory</key>
|
|
<string>%s</string>
|
|
<key>RunAtLoad</key>
|
|
<true/>
|
|
<key>LimitLoadToSessionType</key>
|
|
<array>
|
|
<string>Aqua</string>
|
|
<string>Background</string>
|
|
</array>
|
|
<key>KeepAlive</key>
|
|
<dict>
|
|
<key>SuccessfulExit</key>
|
|
<false/>
|
|
</dict>
|
|
<key>EnvironmentVariables</key>
|
|
<dict>
|
|
<key>CC_LOG_FILE</key>
|
|
<string>%s</string>
|
|
<key>CC_LOG_MAX_SIZE</key>
|
|
<string>%d</string>
|
|
<key>CC_LOG_MAX_BACKUPS</key>
|
|
<string>%d</string>
|
|
<key>PATH</key>
|
|
<string>%s</string>
|
|
%s </dict>
|
|
<key>StandardOutPath</key>
|
|
<string>/dev/null</string>
|
|
<key>StandardErrorPath</key>
|
|
<string>/dev/null</string>
|
|
</dict>
|
|
</plist>
|
|
`, launchdLabel, xmlEscape(cfg.BinaryPath), xmlEscape(cfg.WorkDir), xmlEscape(cfg.LogFile), cfg.LogMaxSize, cfg.LogMaxBackups, xmlEscape(envPATH), envExtra)
|
|
}
|
|
|