Skip to content

refactor: centralize stdout writing#49

Merged
steipete merged 1 commit intomainfrom
refactor/stdout-writer
Feb 16, 2026
Merged

refactor: centralize stdout writing#49
steipete merged 1 commit intomainfrom
refactor/stdout-writer

Conversation

@steipete
Copy link
Owner

@steipete steipete commented Feb 16, 2026

Summary

  • add StdoutWriter to centralize serialized stdout writes
  • route command/help/RPC/watch output through StdoutWriter
  • remove watch-specific stdio flush plumbing and use FileHandle-based line writes
  • keep JSONLines.encode helper while delegating line output to unified writer
  • reduce test noise by capturing stdout in command/router tests

Validation

  • make format
  • make lint
  • make test

@steipete steipete merged commit 1b27e3b into main Feb 16, 2026
2 checks passed
@steipete
Copy link
Owner Author

Landed via temp rebase onto main.

  • Gate: make lint && make test && make build
  • Land commit: afabfc8
  • Merge commit: 1b27e3b

Thanks @steipete!

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.

1 participant