← Back to issue list

fix(PackageService): normalize paths returned by the pack method

View original Github issue

Metadata

Project
craft-application
Number
#767
Type
pull request
State
merged
Author
cmatsuoka
Labels
—
Created
Updated
Closed

Current evaluation

Merged #767 in commit 5a0fca83, making callers normalize PackageService.pack results and discard invalid paths.

Resolution: closed resolved

Reason: The approved PR was merged with all CI checks passing. Commit 5a0fca83 implements path normalization in the caller instead of requiring applications to return resolved paths.

Evaluated by: mr-cal-bot

No scores available.

Issue body

The `pack` method is implemented in the application's `PackageService` and returns a list of packed files to the command runner. Instead of relying on the application returning resolved filesystem paths, let the caller normalize them and discard invalid results in the process. Signed-off-by: Claudio Matsuoka <claudio.matsuoka@canonical.com> - [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`? - [ ] Have you added an entry to the changelog (`docs/reference/changelog.rst`)? ---

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Merged #767 in commit 5a0fca83, making callers normalize PackageService.pack results and discard invalid paths.
stealth/space-bunny-alpha — closed resolved Merged path-normalization fix: PackageService.pack callers now normalize returned file paths and discard invalid results instead of requiring application-resolved paths.
qwen/qwen3.6-35b-a3b — — Merged fix normalizing paths returned by PackageService.pack. The caller now handles path normalization and discards invalid results instead of relying on resolved filesystem paths. Approved by two reviewers with all CI checks passing.

Update history

No update history recorded yet.

Related issues

No related issues found above the similarity threshold.