snaps: eliminate extra call to query snap info (CRAFT-479)
Metadata
Current evaluation
Merged optimization reusing the initial snap info response to eliminate duplicate snapd queries. Approved by one reviewer, passed CI, and achieved 100% diff coverage with a three-line change.
Suggested action: —
No scores available.
Issue body
Re-use first response to prevent Snapcraft from asking snapd
to reach out to the Snap Store twice in a row for the same
data.
Signed-off-by: Chris Patterson <chris.patterson@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 optimization reusing the initial snap info response to eliminate duplicate snapd queries. Approved by one reviewer, passed CI, and achieved 100% diff coverage with a three-line change. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged into master. Optimizes snap info retrieval by reusing the initial response, eliminating redundant snapd calls to the Snap Store. Achieved 100% diff coverage. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged into master. Reuses the initial snap info response to eliminate redundant snapd queries to the Snap Store. Achieved 100% diff coverage. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #5689 fix: remove unnecessary snapd_snap flag (#5316) | snapcraft | merged | PR #5316 merged, removing the unnecessary snapd_snap flag across two files and deleting 14 lines. The change received three approvals, passed all CI checks, and was successfully integrated. | |
| #4 snap: introduce initial snap packaging (CRAFT-578) | rockcraft | merged | Merged initial snap packaging implementation (CRAFT-578). Approved by one reviewer, passed CI checks, and added 86 lines across three files. |