← 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

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.

Suggested action: —

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
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.
qwen3.6-35b-a3b-mtp-q6 — — Merged. Added .debug and .verbose methods to the Emitter class. Internal pause events now route through .debug instead of requiring TRACE mode, aligning with the updated specification.

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 Merged a pull request adding a .trace method to the Emitter class for displaying trace information. Approved by two reviewers, passed CI checks, and introduced 86 lines of code across four files.
77%
#90 Refactor Emitter to comply with the new verbosity levels behaviour (CRAFT-1136). craft-cli merged Merged PR refactoring the Emitter for new verbosity levels (CRAFT-1136). Approved, passed CI, and modified four files. The conceptual changes are part of a larger series, with test updates and future spec adjustments noted.
77%
#69 Add a 'pause' method to the Emitter (CRAFT-907). craft-cli merged Merged adding a 'pause' method to the Emitter, which completely frees the terminal and stops all interactions and log retrieval. Includes test examples, approved by one reviewer, and modifies four files.
75%