Skip to content

Downgrade engine default message to trace level#3687

Merged
pelikhan merged 2 commits into
mainfrom
copilot/downgrade-note-to-trace
Nov 12, 2025
Merged

Downgrade engine default message to trace level#3687
pelikhan merged 2 commits into
mainfrom
copilot/downgrade-note-to-trace

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Nov 12, 2025

The info message "NOTE: No 'engine:' setting found, defaulting to: copilot" was displayed with --verbose flag, creating unnecessary noise in verbose output.

Changes

  • Changed message level: Replaced console.FormatInfoMessage() with logger's Printf() method in pkg/workflow/compiler.go
  • Removed verbose gate: Message now controlled by DEBUG environment variable instead of --verbose flag

Behavior

Before:

./gh-aw compile workflow.md --verbose
# ℹ NOTE: No 'engine:' setting found, defaulting to: copilot

After:

# Normal or verbose mode - no message
./gh-aw compile workflow.md --verbose

# Debug mode - trace message with timing
DEBUG=workflow:compiler ./gh-aw compile workflow.md
# workflow:compiler No 'engine:' setting found, defaulting to: copilot +108µs

Message now follows the repository's logger pattern, appearing only when DEBUG=workflow:compiler or DEBUG=* is set.

Original prompt

Downgrade this message "ℹ NOTE: No 'engine:' setting found, defaulting to: copilot" to a trace message.


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Change the message "No 'engine:' setting found, defaulting to: copilot"
from an info-level message (displayed with --verbose flag) to a trace-level
message (displayed only when DEBUG=workflow:compiler or DEBUG=* is set).

This reduces noise in verbose output while still making the information
available for debugging purposes.

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot AI changed the title [WIP] Downgrade note message to trace level Downgrade engine default message to trace level Nov 12, 2025
Copilot AI requested a review from pelikhan November 12, 2025 05:24
@pelikhan pelikhan marked this pull request as ready for review November 12, 2025 05:34
Copilot AI review requested due to automatic review settings November 12, 2025 05:34
@pelikhan pelikhan merged commit 129e4f6 into main Nov 12, 2025
4 checks passed
@pelikhan pelikhan deleted the copilot/downgrade-note-to-trace branch November 12, 2025 05:34
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR downgrades the engine default message from an info-level console message to a trace-level logger message. The message previously appeared when running gh aw compile with --verbose flag, but now only appears when the DEBUG=workflow:compiler environment variable is set. This reduces noise in verbose output while maintaining the ability to trace default engine selection during debugging.

Key Changes

  • Replaced styled info message (console.FormatInfoMessage) with logger trace message (log.Printf)
  • Removed verbose flag gate, making message controlled by DEBUG environment variable
  • Message now follows repository's standard logger pattern with namespace, timing information, and color-coding

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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.

3 participants