Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Oct 31, 2025

CLI tools should output plain text to stdout, not structured logs. This change removes log/slog in favor of fmt.Fprintf(os.Stdout, ...) for informational messages.

Changes:

  • Removed log/slog import
  • Replaced 4 slog.Info() calls with fmt.Fprintf(os.Stdout, ...)

Before:

slog.Info("Including memory file", "path", path)

After:

fmt.Fprintf(os.Stdout, "Including memory file: %s\n", path)

Messages now print as readable text instead of structured log entries.

Original prompt

This is a CLI tool, lets just print to stdout, don't use slog.


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Copilot AI self-assigned this Oct 31, 2025
Co-authored-by: alexec <1142830+alexec@users.noreply.github.com>
Copilot AI changed the title [WIP] Remove slog usage and print to stdout in CLI tool Replace slog with stdout for CLI output Oct 31, 2025
Copilot AI requested a review from alexec October 31, 2025 23:14
@alexec alexec marked this pull request as ready for review October 31, 2025 23:17
@alexec alexec merged commit 55bc26d into main Oct 31, 2025
2 checks passed
@alexec alexec deleted the copilot/print-to-stdout branch October 31, 2025 23:17
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.

2 participants