← Back to issue list

The analyze command should also work on a not-packed charm

View original Github issue

Metadata

Project
charmcraft
Number
#642
Type
issue
State
open
Author
facundobatista
Labels
good first issue
Created
Updated
Closed
—

Current evaluation

Feature request to let `charmcraft analyze` accept an unpacked charm directory (default `.`) in addition to packed charm files. Still open, labelled good first issue; maintainer says trivial via lint_directory but waiting on #1042.

Suggested action: keep open

Reason: Maintainer lengau confirmed this is 'pretty trivial' — add an is_dir() conditional and call AnalysisService.lint_directory, which already exists in charmcraft/services/analysis.py — yet the current analyse.py still only handles file paths, so the feature remains valid and unimplemented. The latest comment says to wait on #1042, though that issue appears to be about `charmcraft purge`, which is unrelated and lowers confidence.

Evaluated by: mr-cal-bot

Impact: 40 Quick Win: 22 Complexity: 25 Confidence: 65 Actionability: 75

Issue body

Currently `charmcraft analyze` only accepts an already-packed charm as the source of files to analyze. It should also accept a directory (default to `.`) where the charm files exist. IOW: - `charmcraft analyze`: will analyze charm files in current directory - `charmcraft analyze somedir`: will analyze charm files in `somedir` directory - `charmcraft analyze somefile`: current behaviour

Evaluation history

Date Model Scores Action Summary
~deepseek/deepseek-pro-latest
Impact: 40
Quick Win: 22
Complexity: 25
Confidence: 65
Actionability: 75
keep open Feature request to let `charmcraft analyze` accept an unpacked charm directory (default `.`) in addition to packed charm files. Still open, labelled good first issue; maintainer says trivial via lint_directory but waiting on #1042.
qwen/qwen3.6-35b-a3b
Impact: 35
Quick Win: 21
Complexity: 25
Confidence: 75
Actionability: 80
keep open Feature request to make `charmcraft analyze` accept directories (unpacked charms) in addition to packed charm files. Currently only works with packed charms. Open with 'good first issue' label.
qwen/qwen3.8-27b
Impact: 35
Quick Win: 26
Staleness: 58
Complexity: 25
Confidence: 80
Support Request: 10
keep open Feature request: `charmcraft analyze` should accept an unpacked charm directory (default `.`) in addition to a packed charm file. Labeled good first issue; a maintainer notes the fix is trivial but the work is blocked on #1042.
qwen/qwen3.6-35b-a3b
Staleness: 60
Complexity: 20
Confidence: 85
Support Request: 5
keep open Feature request to allow charmcraft analyze to accept directories instead of only packed charms. Currently blocked on issue 1042, with a maintainer noting the implementation is trivial. Labeled good first issue.
qwen3.6-35b-a3b-mtp-q6
Staleness: 75
Complexity: 15
Confidence: 80
Support Request: 0
keep open Feature request to allow charmcraft analyze to accept directories instead of only packed charms. Currently blocked on issue #1042, with a trivial implementation path identified.
qwen3.6-35b-a3b-mtp-q6
Staleness: 60
Complexity: 15
Confidence: 85
Support Request: 5
keep open Feature request to extend charmcraft analyze to accept directories alongside packed charms. Currently waiting on dependency issue #1042, with no recent maintainer action in ~9 months.

Update history

No update history recorded yet.

Related work

  • Blocked By: charmcraft#1042 (confidence 55%)

    Maintainer explicitly said 'We're going to wait for https://github.com/canonical/charmcraft/issues/1042 on this', but that issue is titled as a purge/cleanall command request, so the dependency is unclear.

Related issues

No related issues found above the similarity threshold.