feat: add fetch-service integration
Metadata
Current evaluation
Merged in PR #456, adding fetch-service lifecycle, sessions, certificates, build-instance configuration, and environment integration across 18 files.
Resolution: closed resolved
Reason: PR #456 was merged after approval from two reviewers, delivering the previously reviewed fetch-service integration into craft application builds.
Evaluated by: mr-cal-bot
No scores available.
Issue body
This PR brings in all the work that has been done in the past few months to integrate the fetch-service into a craft tool's build. Each commit has already been individually reviewed.
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged in PR #456, adding fetch-service lifecycle, sessions, certificates, build-instance configuration, and environment integration across 18 files. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged fetch-service integration into the craft tool build in canonical/craft-application#456, adding 1,445 lines across 18 files. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged fetch-service integration into the craft tool build. Changes span 18 files with 1445 additions and 32 deletions. Approved by two reviewers after individual commit reviews. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #272 feat: add fetch-service version to session metadata | fetch-service | merged | Merged PR #272 added a generator field with the fetch-service version to session metadata, resolving fetch-service#258; merged as commit a0e826f. | |
| #837 feat: "support" existing fetch-service sessions | craft-application | merged | Merged in craft-application#837 as d03d8b43, adding external fetch-service and proxy-certificate environment support as groundwork; the PR explicitly delivered only partial packing support. | |
| #1665 feat: add fetch-libs command | charmcraft | merged | Merged PR #1665 added Charmcraft's fetch-libs command; implementation landed in commit 38ea17d6. | |
| #496 feat: mark the fetch-service integration as experimental | craft-application | merged | Merged as 327ff944, hiding --enable-fetch-service from pack help and warning when the experimental integration is enabled; fixes #488. | |
| #903 docs: reference for the fetch-service integration | craft-application | merged | Merged documentation for the fetch-service integration; resolving merge commit 149fb22d landed PR #903. | |
| #416 feat: add a configuration service | craft-application | merged | Merged PR #416 as commit 4d7bc64d, adding prioritized configuration lookup through app/CRAFT_* environment variables, custom handlers, snap settings, and defaults. | |
| #371 feat: add session-aware logging support | fetch-service | merged | Merged PR #371 at commit ffb88ea, adding session-aware logs that prefix session events with session IDs and resolve fetch-service#135. | |
| #34 feat: fetch service snap | fetch-service | merged | PR #34 merged as commit 40e5088, adding the initial snapcraft.yaml plus snap/local assets (fetch-service.sh wrapper, config) so the fetch-service charm can ship as a snap. |