add iologger for debugging purposes

[juruen]

Context

This PR introduces an IOLogger struct that logs all stdin and stdout interactions between the client and the server.

This feature is particularly useful for debugging, as it provides visibility into the exact input and output exchanged between the client and the server.

By default, logging is disabled but can be enabled by setting the enable-command-logging flag to true. I considered enabling it only when the log level is set to debug, but decided that having a separate toggle would be preferable due to potential privacy concerns.

[Copilot]

Pull Request Overview

A PR to add an IOLogger for logging stdin and stdout between the client and server for debugging purposes.

• Introduces an IOLogger struct with Read and Write methods that log data.
• Adds tests in pkg/log/io_test.go to verify logged IO behavior.
• Updates the server startup in cmd/server/main.go to conditionally wrap standard IO with the IOLogger based on a new command flag.

Reviewed Changes

Copilot reviewed 3 out of 4 changed files in this pull request and generated 1 comment.

File	Description
pkg/log/io.go	Introduces the IOLogger implementation and its logging behavior on Read and Write.
pkg/log/io_test.go	Provides tests to verify logged input and output functionality.
cmd/server/main.go	Modifies server initialization to retrieve and pass a command logging flag and binds a new flag.

Files not reviewed (1)

• go.mod: Language not supported

Tip: Copilot only keeps its highest confidence comments to reduce noise and keep you focused. Learn more

[SamMorrowDrums]

I can see how this would save a lot of pain when iterating on things.