Refactor Emitter to comply with the new verbosity levels behaviour (CRAFT-1136).
Metadata
Current evaluation
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.
Suggested action: —
No scores available.
Issue body
Note that Emitter final behaviour (as described in the Spec) will only be achieved after more several PRs, I added XXX notes for changes that will happen in the short term future.
This branch is for sure the most "conceptual" one in the "verbosity levels" series. The changes in the code are not many, but the tests are seriously affected (and will continue to mutate when other PRs land). My recommendation is to analyze the changes in the code using [the Spec itself](https://docs.google.com/document/d/1pJDT-NE3jhkfVvDw6ba0Z3adnaQnGRi9slC9jIOqrZ4/edit) for better reasoning..
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | 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. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged refactoring of the Emitter class to align with new verbosity level specifications. The conceptual changes significantly impact tests and establish the baseline for subsequent updates in the verbosity levels series. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #92 Added the .debug and .verbose methods to Emitter (CRAFT-1137). | craft-cli | merged | 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. | |
| #94 Refactor how the logger Handler behaves for the new verbosity levels (CRAFT-1138). | craft-cli | merged | Merged refactoring of the logger Handler to support new verbosity levels. Approved by one reviewer with zero unresolved comments. The change modified four files, adding 114 lines and removing 43. | |
| #801 Adapted project to the new Craft CLI verbosity levels (CRAFT-1143). | charmcraft | merged | Merged after approvals and CI. Updates logging to debug, switches to progress/permanent messages, simplifies startup via --verbosity, initializes emit in BRIEF mode, and refreshes dependencies to match new Craft CLI verbosity levels. | |
| #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. |