Skip to content

Commit 342be6c

Browse files
runningcodeclaude
andauthored
docs(snapshots): Correct ImageMetadata CLI-managed fields list (#3277)
## Summary - The doc comment on `ImageMetadata` listed `image_file_name` as a CLI-managed override, but no such field is written by the CLI. - `ImageMetadata::new` only sets `width` and `height`; the Sentry backend derives `image_file_name` from the manifest map key, not from a metadata field. - Updates the comment to match reality. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
1 parent 4430631 commit 342be6c

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

src/api/data_types/snapshots.rs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -40,8 +40,8 @@ pub struct SnapshotsManifest<'a> {
4040
// Keep in sync with https://github.com/getsentry/sentry/blob/master/src/sentry/preprod/snapshots/manifest.py
4141
/// Metadata for a single image in a snapshot manifest.
4242
///
43-
/// CLI-managed fields (`image_file_name`, `width`, `height`) override any
44-
/// identically named fields provided by user sidecar metadata.
43+
/// CLI-managed fields (`width`, `height`) override any identically named
44+
/// fields provided by user sidecar metadata.
4545
#[derive(Debug, Serialize)]
4646
pub struct ImageMetadata {
4747
#[serde(flatten)]

0 commit comments

Comments
 (0)