feat: allow strict/permissive fetch-service sessions
Metadata
Current evaluation
Merged after approval and CI checks, adding mandatory strict or permissive session arguments to the --use-fetch-service flag. Resolves issue #487.
Suggested action: —
No scores available.
Issue body
'--use-fetch-service' now takes a mandatory argument defining the type of session to use: "strict" or "permissive".
Fixes #487
- [ ] Have you followed the guidelines for contributing?
- [ ] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
- [ ] Have you successfully run `tox`?
-----
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged after approval and CI checks, adding mandatory strict or permissive session arguments to the --use-fetch-service flag. Resolves issue #487. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged. Adds a mandatory strict or permissive argument to the --use-fetch-service flag, resolving issue #487. The update explicitly defines session behavior for the fetch service. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #487 Allow strict fetch-service sessions on the command-line | craft-application | closed | Closed after UX discussion for --use-fetch-service=strict and --use-fetch-service=permissive CLI flags. No implementation or PR was created, indicating the feature was abandoned or deferred. | |
| #272 feat: add fetch-service version to session metadata | fetch-service | merged | Merged after approval from two reviewers and passing CI. Introduces a generator key to session metadata to explicitly track the fetch-service version, resolving issue #258. | |
| #837 feat: "support" existing fetch-service sessions | craft-application | merged | Merged incremental changes adding CRAFT_USE_EXTERNAL_FETCH_SERVICE and CRAFT_PROXY_CERT environment variables for external fetch-service sessions. Approved by reviewers and passing CI, this serves as a foundational step toward full external session management. | |
| #480 feat: support per-session inspector conf | fetch-service | merged | Merged: Added per-session inspector configuration support. Approved by two reviewers and passed all CI checks. Proceeded despite 14 unresolved comments and a known spread test failure from an external store bug. Closes FETCH-31. | |
| #836 test: use a 'strict' fetch-service session | craft-application | closed | The pull request was closed after passing all CI checks. No reviewer comments or merge activity were recorded. |