← Back to issue list

messages: allow to set the terminal window title

View original Github issue

Metadata

Project
craft-cli
Number
#160
Type
pull request
State
open
Author
sergio-costas
Labels
—
Created
Updated
Closed
—

Current evaluation

Adds an emitter API to set the terminal window title via ANSI escape codes, for #159. Idle 1103 days with 4 unresolved review comments; messages.py was refactored and tests reorganized since, so it needs rework rather than a rebase.

Suggested action: close stale

Reason: The PR has had no activity for 1103 days, review status is still "pending (0 reviewers)" with 4 unresolved review comments, and the branch has diverged substantially from the current tree: messages.py was refactored (Printer and reset_terminal_style now live in craft_cli/printer.py) and the unit test layout changed, with tests/unit/test_messages.py no longer existing. Reviewer tigarmo's substantive concern that "the last set title persists after Snapcraft exits... we should clear that up, ideally restoring the original title" was acknowledged with "I'll try it" but there is no evidence it was ever addressed; the final comment only argues the Konsole issue is a Konsole bug (bugs.kde.org/show_bug.cgi?id=471353). This now requires reimplementation, not a merge.

Evaluated by: mr-cal-bot

Impact: 30 Quick Win: 2 Complexity: 45 Confidence: 72 Actionability: 12

Issue body

This patch allows to set the text in the title of a terminal window. The idea is to allow snapcraft to set in the title which step is currently doing. This is a first proposal for https://github.com/canonical/craft-cli/issues/159 - [x] Have you followed the guidelines for contributing? - [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? - [x] Have you successfully run `tox`?

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha
Impact: 30
Quick Win: 2
Complexity: 45
Confidence: 72
Actionability: 12
close stale Adds an emitter API to set the terminal window title via ANSI escape codes, for #159. Idle 1103 days with 4 unresolved review comments; messages.py was refactored and tests reorganized since, so it needs rework rather than a rebase.
qwen/qwen3.6-35b-a3b
Impact: 25
Quick Win: 3
Complexity: 30
Confidence: 75
Actionability: 15
close stale Adds terminal window title setting capability for craft-cli. Has unresolved reviewer concerns about Konsole compatibility and title persistence. Inactive for ~3 years with no CI checks.
qwen/qwen3.8-27b
Impact: 25
Quick Win: 16
Staleness: 85
Complexity: 35
Confidence: 75
close stale Adds terminal window title setting via ANSI escape codes to show build progress. Last active 2.9 years ago; codebase has since undergone major refactoring (starbase merge, new emitter API). Feature never merged; issue #159 still open but untriaged.
qwen/qwen3.6-35b-a3b
Staleness: 95
Complexity: 15
Confidence: 90
close stale Adds terminal window title setting via ANSI escape codes. Inactive for over two years with unresolved review comments on terminal compatibility and title persistence.
qwen3.6-35b-a3b-mtp-q6
Staleness: 95
Complexity: 15
Confidence: 85
close stale Adds terminal window title support via ANSI escape codes for snapcraft step tracking. Currently inactive for nearly three years with unresolved compatibility concerns and no maintainer response.

Update history

No update history recorded yet.

Related work

  • Related To: canonical/craft-cli#159 (confidence 92%)

    The feature request this PR proposes a solution for; still open, and its summary records that the PR "is stalled with unresolved reviewer concerns".

Related issues

No related issues found above the similarity threshold.