storeapi: improve candid interaction errors (CRAFT-135)
Metadata
Current evaluation
Merged into master after reviewer approval and passing CI. Introduces distinct error classes and messages for token handling in the store API to improve Candid interaction error reporting.
Suggested action: —
No scores available.
Issue body
Provide distinct error classes and messages when dealing with tokens.
Signed-off-by: Sergio Schvezov <sergio.schvezov@canonical.com>
- [ ] Have you followed the [guidelines for contributing](https://github.com/snapcore/snapcraft/blob/master/CONTRIBUTING.md)?
- [ ] 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 into master after reviewer approval and passing CI. Introduces distinct error classes and messages for token handling in the store API to improve Candid interaction error reporting. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged into master, the update improves Candid interaction errors by introducing distinct error classes and messages for token handling. Integrated as PR #3562 despite a minor coverage decrease. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged into master. Introduces distinct error classes and messages for token handling in the store API to improve Candid interaction error reporting. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #2116 storeapi: handle 5xx error codes for all store endpoints | snapcraft | merged | Merged robust 5xx error handling across all store API endpoints. Fixes Launchpad bug #1768370 by generalizing release-specific error handling. Approved by two reviewers and passed CI checks. | |
| #3667 storeapi: fix typo | snapcraft | merged | Merged a single-line typo correction in the storeapi module. Approved by a reviewer, passed CI, and increased test coverage by 0.07%. | |
| #3692 legacy storeapi: use craft-store | snapcraft | merged | Merged PR replaces the legacy storeapi HTTP client with craft-store, removes obsolete tests and logout calls, and adapts the codebase to craft-store API changes. Approved and passed CI. | |
| #3550 storeapi: log responses and registration error code | snapcraft | merged | Merged. Adds debug logging for store API responses and improves registration error code output. The minimal change (+16/-5 lines, 4 files) passed CI, achieved 100% coverage, and was approved for backporting. |