← Back to issue list

feat: error on default command

View original Github issue

Metadata

Project
craft-cli
Number
#422
Type
pull request
State
merged
Author
mr-cal
Labels
Created
Updated
Closed

Current evaluation

Merged. Adds opt-in allow_default_command to Dispatcher for default command support without breaking existing apps. Approved by three reviewers, passed CI, and unblocks downstream snapcraft.

Suggested action:

No scores available.

Issue body

- [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 `make lint && make test`? --- Adds an `allow_default_command` parameter to the Dispatcher to allow using a default command. This is an opt-in flag, so there are no behavioral changes for existing apps. Tested downstream in https://github.com/canonical/snapcraft/pull/6074 Unblocks https://github.com/canonical/snapcraft/issues/5673 (SNAPCRAFT-1194)

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b Merged. Adds opt-in allow_default_command to Dispatcher for default command support without breaking existing apps. Approved by three reviewers, passed CI, and unblocks downstream snapcraft.
qwen3.6-35b-a3b-mtp-q6 Merged. Introduces an opt-in allow_default_command parameter to the Dispatcher, enabling default command execution without altering existing app behavior. Successfully unblocks downstream snapcraft issue #5673.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#362 feat(dispatcher): add a warning for the default command craft-cli merged Merged implementation that emits a warning when the default command is used. Approved by two reviewers, passed all CI checks, and unblocks related Snapcraft tracking issues.
79%
#281 feat(app): provide a way to customize the Dispatcher craft-application merged Merged a feature to customize the Dispatcher, allowing Snapcraft to set a default command without supporting it at the craft-application level. Approved by three reviewers, passed CI checks, and updated one file with 14 additions and 6 deletions.
76%
#361 Remove the default command craft-cli open Maintainer task to remove the Dispatcher's default_command/allow_default_command support (added for Snapcraft compat). Feature still present in code; removal gated on Snapcraft 10 (core28) release. No labels, 0 comments.
76%
#58 Support for a default command in Dispatcher (CRAFT-760). craft-cli merged Merged after approval, adding default command support to the Dispatcher (CRAFT-760). Includes documentation and modifies three files with 79 additions and 25 deletions.
71%