Skip to content

fix: suppress stdout log spam when daemon is already running#219

Merged
kianwoon merged 1 commit intomainfrom
fix/silent-already-running-start
Apr 9, 2026
Merged

fix: suppress stdout log spam when daemon is already running#219
kianwoon merged 1 commit intomainfrom
fix/silent-already-running-start

Conversation

@kianwoon
Copy link
Copy Markdown
Owner

@kianwoon kianwoon commented Apr 9, 2026

Summary

  • Add alreadyRunning flag to DaemonStartResult — set when startDaemon() detects the daemon is already running
  • CLI handler (index.ts) now suppresses all output when alreadyRunning is true, exiting silently with code 0
  • Eliminates the 32K+ "already running" messages that accumulate in stdout.log from launchd KeepAlive re-invocations

Test plan

  • TypeScript compiles cleanly (npx tsc --noEmit)
  • All 22 daemon tests pass
  • node dist/index.js start when already running produces no output, exits 0
  • node dist/index.js reload still works normally
  • Build succeeds (npm run build)

Closes #217

@kianwoon kianwoon merged commit 95d18d0 into main Apr 9, 2026
5 checks passed
@kianwoon kianwoon deleted the fix/silent-already-running-start branch April 9, 2026 04:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

bug: 32K+ "already running" log spam in stdout.log

1 participant