craftctl: fix client error handling
Metadata
Current evaluation
Merged PR #193 (commit e9787a0c; merge 9ea70b26) makes craftctl return control API errors to clients instead of raising server-side exceptions.
Resolution: closed resolved
Reason: PR #193 was approved and merged, resolving client error handling through commit e9787a0c and merge commit 9ea70b26.
Evaluated by: mr-cal-bot
No scores available.
Issue body
Errors arising from usage of the control API should be sent back
to the client instead of triggering a server-side exception.
Signed-off-by: Claudio Matsuoka <claudio.matsuoka@canonical.com>
- [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
-----
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #193 (commit e9787a0c; merge 9ea70b26) makes craftctl return control API errors to clients instead of raising server-side exceptions. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #193 fixed craftctl to return control API errors to clients instead of triggering server-side exceptions (commit e9787a0c). | |
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #193 fixed craftctl to return control API errors to the client instead of triggering server-side exceptions (commit e9787a0c). | |
| stealth/space-bunny-alpha | — | closed resolved | Merged as 9ea70b26 (implementation e9787a0c), fixing craftctl to return control API errors to clients instead of raising server-side exceptions. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #193 updated craftctl to return control API errors to clients instead of causing server-side exceptions. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged fix for craftctl client error handling. Control API errors now return to the client instead of causing server-side exceptions. Approved by one reviewer, passed CI, and merged after a commit message update. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged. Control API errors in craftctl now return to the client instead of causing server-side exceptions. Author force-pushed once to correct the commit message before final merge. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #185 craftctl: update to comply with specification | craft-parts | merged | Merged in commit ac87cd89 via merge 8139ef30; craftctl’s default, set, and get commands were updated to match the specification, including validation and consumption safeguards. |