extractors: replace desktop file ids with paths
Metadata
Current evaluation
Merged PR #1879 as 75a919b2c, replacing desktop file IDs with paths and moving ID translation into the AppStream extractor.
Resolution: closed resolved
Reason: PR #1879 was merged as commit 75a919b2c. Extractors now use desktop file paths, while AppStream IDs remain in metadata for translation by the AppStream extractor.
Evaluated by: mr-cal-bot
No scores available.
Issue body
- [x] Have you followed the [guidelines for contributing](https://github.com/snapcore/snapcraft/blob/master/CONTRIBUTING.md)?
- [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
- [x] If this is a bugfix. Have you checked that there is a bug report open for the issue you are trying to fix on [bug reports](https://bugs.launchpad.net/snapcraft)?
- [x] If this is a new feature. Have you discussed the design on the [forum](https://forum.snapcraft.io)?
- [x] Have you successfully run `./runtests.sh static`?
- [x] Have you successfully run `./runtests.sh unit`?
-----
@kyrofa and I discussed about moving the task of translating from desktop file id to desktop file path to the appstream extractor. This opens the door to save in the metadata the contents of the file, instead of the path. But we haven't yet decided if that next step would be good.
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #1879 as 75a919b2c, replacing desktop file IDs with paths and moving ID translation into the AppStream extractor. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #1879 as 75a919b2c: extractors now use desktop file paths, with ID translation moved to the appstream extractor. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged after review and conflict resolution. Replaces desktop file IDs with paths in extractors, shifting translation logic to the appstream extractor. Approved by reviewer and passed CI checks. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Successfully merged. Replaced desktop file IDs with paths in extractors, moving translation logic to the appstream extractor to enable future metadata storage. Conflicts were resolved during review. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #1853 extractors: support appstream icon and desktop | snapcraft | merged | Merged as commit ce3209468: AppStream extractors now adopt icon and desktop metadata (icon paths, desktop file IDs) from .metainfo.xml. Closes issue #1828. |