← Back to issue list

Added the .debug and .verbose methods to Emitter (CRAFT-1137).

View original Github issue

Metadata

Project
craft-cli
Number
#92
Type
pull request
State
merged
Author
facundobatista
Labels
—
Created
Updated
Closed

Current evaluation

Merged PR #92 added Emitter.debug() and Emitter.verbose() for the new verbosity spec and moved pause start/end messages to debug output, visible only in TRACE mode.

Resolution: closed resolved

Reason: PR #92 was merged in commit 40fd946; implementation commit 8035c05 added the new Emitter methods and updated pause reporting.

Evaluated by: mr-cal-bot

No scores available.

Issue body

These two behave ok according to the new Spec. Also I moved the "pause started" and "pause ended" to be signaled using internal .debug (now that it's available) as before they will only appear in the logs if TRACE mode.

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Merged PR #92 added Emitter.debug() and Emitter.verbose() for the new verbosity spec and moved pause start/end messages to debug output, visible only in TRACE mode.
stealth/space-bunny-alpha — closed resolved Merged PR #92 added Emitter.debug() and .verbose() for the new spec and moved pause start/end signals to internal debug logging in commit 40fd946.
stealth/space-bunny-alpha — closed resolved Merged PR #92 added Emitter.debug() and Emitter.verbose() and moved pause start/end events to internal debug logging via merge commit 40fd946.
qwen/qwen3.6-35b-a3b — — Added .debug and .verbose methods to the Emitter class per the new specification. Refactored pause start and end signals to use internal .debug. Approved, passed CI, and merged with 221 additions and 8 deletions across four files.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#6 Added a .trace method to Emitter to show trace information (CRAFT-97). craft-cli merged PR #6 by maintainer facundobatista added Emitter.trace() for trace-level output (CRAFT-97). Approved by 2 reviewers and merged into main via commit be12175 (impl 729b1ee); the method still ships today in craft_cli/messages.py.
74%
#90 Refactor Emitter to comply with the new verbosity levels behaviour (CRAFT-1136). craft-cli merged Merged PR #90 as 89b9eaa (implementation 3526a7a), refactoring Emitter toward CRAFT-1136's new verbosity behavior as part of a broader PR series.
73%
#88 Changed NORMAL to BRIEF and added a DEBUG verbosity level (still not used) (CRAFT-1134) craft-cli merged Merged PR #88 in ffa95e8, renaming EmitterMode.NORMAL to BRIEF and adding an initially unused DEBUG level.
70%