Skip to content

fix(snapshots): Preserve subdirectory structure in manifest keys#3269

Merged
NicoHinderling merged 2 commits intomasterfrom
fix/snapshot-preserve-relative-paths
Apr 15, 2026
Merged

fix(snapshots): Preserve subdirectory structure in manifest keys#3269
NicoHinderling merged 2 commits intomasterfrom
fix/snapshot-preserve-relative-paths

Conversation

@NicoHinderling
Copy link
Copy Markdown
Contributor

Use full relative paths (e.g. settings/profile.png) instead of bare
filenames (e.g. profile.png) as manifest keys when uploading build
snapshots.

Previously, upload_images extracted only the filename from each image's
relative path to use as the manifest key. This meant images in different
subdirectories with the same filename would collide — only the first
encountered would be uploaded, and the rest silently skipped with a
warning.

Now the manifest key includes the full relative path from the upload
directory, preserving the subdirectory structure and eliminating false
collisions.

NicoHinderling and others added 2 commits April 15, 2026 12:58
Use full relative paths (e.g. settings/profile.png) instead of bare
filenames (e.g. profile.png) as manifest keys when uploading build
snapshots. Previously, images in different subdirectories with the
same filename would collide and only the first would be uploaded.

Co-Authored-By: Claude <noreply@anthropic.com>
Co-Authored-By: Claude <noreply@anthropic.com>
@NicoHinderling NicoHinderling force-pushed the fix/snapshot-preserve-relative-paths branch from 2c1f889 to 1486e9c Compare April 15, 2026 19:59
@NicoHinderling NicoHinderling marked this pull request as ready for review April 15, 2026 19:59
@NicoHinderling NicoHinderling requested review from a team as code owners April 15, 2026 19:59
@NicoHinderling NicoHinderling merged commit cdcc4a2 into master Apr 15, 2026
26 checks passed
@NicoHinderling NicoHinderling deleted the fix/snapshot-preserve-relative-paths branch April 15, 2026 20:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants