fix: add proper error messages (#738)
Metadata
Current evaluation
Merged fix adding clearer error messages for invalid source paths and network failures. NetworkRequestError now includes the source URL. Approved by two reviewers and passed CI, resolving issue #738.
Suggested action: —
No scores available.
Issue body
- [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
-----
Fixes #738
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged fix adding clearer error messages for invalid source paths and network failures. NetworkRequestError now includes the source URL. Approved by two reviewers and passed CI, resolving issue #738. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged fix for #738 that improves error messages for invalid source paths and network failures. The NetworkRequestError now includes the source URL when available, providing clearer debugging guidance. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #322 sources: improve missing local source error message | craft-parts | merged | Merged. Updates error handling to display a clear source not found message instead of a file copy error when a local source is missing. Approved by two reviewers, passed CI, and changed 17 lines across two files. | |
| #1533 fix(sources): handle streaming request errors | craft-parts | merged | Merged. Wraps streaming file download request exceptions into NetworkRequestError to surface proper source-facing errors. Approved by two reviewers and merged after resolving transient CI failures and adding a changelog entry. Closes #1396. |