feat(models): add SnapInfo model for snapd response
Metadata
Current evaluation
Merged PR #908 as commit e996996, adding a Pydantic SnapInfo model, optional publisher handling, and tests for snapd responses; fixes #783.
Resolution: closed resolved
Reason: Merged as commit e996996. The change introduced SnapInfo and SnapPublisher models, handled missing publisher data, added tests, and resolved issue #783.
Evaluated by: mr-cal-bot
No scores available.
Issue body
Converted Snap Info to a Pydantic model and added test cases.
Fixes: #783
---
- [x] Have you followed the guidelines for contributing?
- [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
- [ ] Have you successfully run `make lint && make test`?
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #908 as commit e996996, adding a Pydantic SnapInfo model, optional publisher handling, and tests for snapd responses; fixes #783. | |
| qwen/qwen3.6-35b-a3b | — | closed resolved | Converted snapd response handling to a Pydantic SnapInfo model with optional publisher support. Merged to resolve #783. |
Update history
| Date | Change |
|---|---|
| created |
Related work
-
Related To:
canonical/craft-providers#783
(confidence 100%)
The request to model snap information with Pydantic was explicitly fixed by PR #908.
-
Related To:
canonical/craft-providers#939
(confidence 95%)
Follow-up PR #939 added the enclosing SnapdResponse model as commit 2702b5a.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #939 feat(models): add SnapdResponse model for snapd Response | craft-providers | merged | Merged #939 as 2702b5a, adding a Pydantic SnapdResponse model and tests to fix #784. The reported macOS CI failure was confirmed as unrelated flakiness. | |
| #784 Snap revision should use a Pydantic model | craft-providers | closed | Fixed by merged PR #908 (e996996), adding Pydantic SnapdResponse and SnapInfo models for validated snapd API responses. | |
| #783 Snap info should use a Pydantic model | craft-providers | closed | Snapd snap metadata now uses a Pydantic SnapInfo model, merged in PR #908 at commit e996996. |