fix(packaging): AppStream URLs + Sentry debug upload for Session 07 - #521
Merged
Conversation
AppImage packaging failed because appstreamcli could not reach the legacy mberrys/Loop-pdf homepage, bugtracker, and help URLs (404). Update metainfo to the canonical studio-berry/loop repository. Co-authored-by: michael berry <mberrys@users.noreply.github.com>
mberrys
marked this pull request as ready for review
September 2, 2026 07:16
Windows MSI packaging failed because sentry-cli uploaded PDBs to berry-studios/loop-pdf, which does not exist on de.sentry.io. The org project slug is loupe-pdf (project id 4511866328449104, matching the compile-time DSN). Co-authored-by: michael berry <mberrys@users.noreply.github.com>
cursor
Bot
changed the title
…tract sentry-cli accepts project id 4511866328449104 (same as compile-time DSN) instead of the legacy slug, avoiding the banned product token in tracked source while still uploading to the correct berry-studios EU project. Co-authored-by: michael berry <mberrys@users.noreply.github.com>
No Loop-named project slug exists on de.sentry.io; loop-pdf is absent and the legacy slug is policy-banned. Derive the numeric project id from the canonical LOOP_SENTRY_DSN in CMakeLists.txt instead of hardcoding slugs in workflows or tracked defaults. Co-authored-by: michael berry <mberrys@users.noreply.github.com>
mberrys
enabled auto-merge
September 2, 2026 08:04
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Session 07 packaging fixes on one branch for merge into
stable.Linux AppImage
AppStream metainfo URLs →
studio-berry/loop(fixesurl-not-reachableon legacymberrys/Loop-pdf).Windows MSI — Sentry debug upload
loop-pdfslug does not exist onde.sentry.io. The real project is id4511866328449104(matches compile-time DSN inCMakeLists.txt).sentry-cli debug-files uploadaccepts that numeric project id. Workflows and script default now use4511866328449104instead of any legacy slug, satisfyingcheck_loop_identity(banned legacy product token) while uploading to the correctberry-studiosEU project.Verification (local)
python3 scripts/ci/check_loop_identity.py— passpython3 -m unittest discover -s scripts/ci -p 'test_*.py'— pass (223 tests)python3 scripts/ci/check_supply_chain_pins.py— passappstreamcli validate --pedanticon appdata.xml — passScope
Minimal packaging/config only. Not merged; parent handles merge + package re-dispatch.