cli: fix help command legacy fallback
Metadata
Current evaluation
Merged fix for the CLI help command legacy fallback. Handles craft-cli's ProvideHelpException to standardize help output. Approved, passed CI, and achieved 100% diff coverage before merging into the 7.0 branch.
Suggested action: —
No scores available.
Issue body
Handle craft-cli's `ProvideHelpException` error to display the same
message displayed when invoked using the `--help` option.
Signed-off-by: Claudio Matsuoka <claudio.matsuoka@canonical.com>
- [x] Have you followed the [guidelines for contributing](https://github.com/snapcore/snapcraft/blob/master/CONTRIBUTING.md)?
- [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
- [ ] Have you successfully run `./runtests.sh static`?
- [ ] Have you successfully run `./runtests.sh tests/unit`?
-----
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged fix for the CLI help command legacy fallback. Handles craft-cli's ProvideHelpException to standardize help output. Approved, passed CI, and achieved 100% diff coverage before merging into the 7.0 branch. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged into snapcraft/7.0. Fixes legacy help command fallback by handling craft-cli's ProvideHelpException to ensure consistent help output. Code coverage increased by 0.28%. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged into snapcraft/7.0. Resolves the CLI help command legacy fallback by handling craft-cli's ProvideHelpException to display consistent help messages. Achieved 100% diff coverage. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #3735 commands: help for all legacy commands | snapcraft | merged | Merged implementation adding help text for all legacy commands via passthrough. Approved, passed CI, and resolves CRAFT-986 through CRAFT-991. Minor coverage decrease noted but accepted. | |
| #1845 cli: implement help <command> | snapcraft | merged | Implemented snapcraft help <command> CLI functionality. Merged after passing CI checks and receiving reviewer approval. Closes issue #1699. | |
| #2429 cli: fix usage string in help command | snapcraft | merged | Merged a one-line fix that corrected an incorrect usage string in the CLI help command. The change was approved, passed CI checks, and resolved the reported bug. | |
| #1112 cli: add supported profile arguments to help | charmcraft | merged | Merged a single-line update adding supported profile arguments to the CLI help text. Approved by one reviewer with zero unresolved comments and passing CI. Resolves CRAFT-1733. | |
| #84 fix(cli): enable per-command help queries | craft-application | merged | Merged to main, enabling per-command help queries in the CLI and resolving issue #76. Approved by two reviewers, passed CI checks, and achieved 100% diff coverage across three files. | |
| #1789 cli: improve the help command | snapcraft | merged | Improved the CLI help command to match --help parity and expose hidden topics and plugin help. Merged after reviewer approval and passing CI, modifying three files with 62 additions and 11 deletions. | |
| #3626 cli: integrate craft-cli and add version command (CRAFT-730) | snapcraft | merged | Merged after integrating craft-cli and adding a version command, falling back to the legacy implementation for other functionality. Passed CI, approved, and merged. Help output remains handled by the legacy system until fully updated. | |
| #4419 cli: catch craft-provider errors | snapcraft | merged | Merged a cherry-picked bugfix to catch craft-provider errors in the CLI. Approved, passed CI, and integrated into the hotfix/7.5 branch after resolving initial dependencies. | |
| #4295 cli: catch craft-provider errors | snapcraft | merged | Merged after approval, adding CLI error handling for craft-provider failures. Resolves #4262 and CRAFT-1898. Passed CI checks and achieved 100% diff coverage across two modified files. |