← Back to issue list

Provide a way to merge other log file into current one

View original Github issue

Metadata

Project
craft-cli
Number
#121
Type
issue
State
open
Author
facundobatista
Labels
—
Created
Updated
Closed
—

Current evaluation

Feature request to merge external log files (e.g. subprocess output) into the current log. Delivered as Emitter.append_to_log() in craft-cli 3.0.0 via commit 839986b (#331); issue was left open and is now obsolete.

Suggested action: close resolved

Reason: The requested capability exists in the codebase: craft_cli/messages.py:837 defines `Emitter.append_to_log(file, prefix=':: ')` with the docstring "Dump the contents of an external log file into the emitter log", added by commit 839986b "feat(emitter): add a log dumping function (#331)". docs/changelog.rst confirms it shipped under "New features - Add an ``append_to_log`` method to the emitter, which reads from a file and dumps it directly into the log" in release 3.0.0 (2025-03-27), and it is covered by the integration test `test_append_to_log` (tests/integration/test_messages_integration.py:1874). No maintainer comments or labels exist on the issue, so the only reason it is still open is that it was never closed after the implementing PR merged.

Evaluated by: mr-cal-bot

Impact: 30 Quick Win: 1 Complexity: 15 Confidence: 96 Actionability: 5

Issue body

Applications using the library sometime need to merge other logs into the current one (for example, the collected output of a subprocess run). The library should provide a method like `merge_logs_from` to merge lines from other file into the current one.

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha
Impact: 30
Quick Win: 1
Complexity: 15
Confidence: 96
Actionability: 5
close resolved Feature request to merge external log files (e.g. subprocess output) into the current log. Delivered as Emitter.append_to_log() in craft-cli 3.0.0 via commit 839986b (#331); issue was left open and is now obsolete.
qwen/qwen3.6-35b-a3b
Impact: 30
Quick Win: 14
Complexity: 25
Confidence: 70
Actionability: 60
needs triage Feature request to add a merge_logs_from method for merging external log files (e.g., subprocess output) into the current log. Open, unlabelled, no comments.
qwen/qwen3.8-27b
Impact: 35
Quick Win: 19
Staleness: 60
Complexity: 45
Confidence: 75
Support Request: 0
needs triage Feature request from maintainer to add a merge_logs_from method to Emitter for merging external log files (e.g. subprocess output) into the current log. Unlabelled, no comments, no implementation found in codebase after ~4 years.
qwen/qwen3.6-35b-a3b
Staleness: 90
Complexity: 20
Confidence: 85
Support Request: 10
needs triage Feature request to add a merge_logs_from method for merging external log files into the current log. Untriaged, no comments, inactive for over a year.
qwen3.6-35b-a3b-mtp-q6
Staleness: 85
Complexity: 20
Confidence: 80
Support Request: 10
needs triage Feature request to add a merge_logs_from method for combining external log files into the current log stream. Unlabelled, zero comments, and inactive for over a year, awaiting project assessment.

Update history

No update history recorded yet.

Related work

  • Likely Fixed By: canonical/craft-cli#331 (confidence 93%)

    PR #331, commit 839986b 'feat(emitter): add a log dumping function', added Emitter.append_to_log and the 3.0.0 changelog entry; this is the direct implementation of the requested `merge_logs_from` capability under a different name.

Related issues

No related issues found above the similarity threshold.