From 29caec557f522dccf05b588e9488d4ab50f9e797 Mon Sep 17 00:00:00 2001 From: Nelson Osacky Date: Wed, 22 Apr 2026 14:25:50 +0200 Subject: [PATCH] docs(snapshots): Correct ImageMetadata CLI-managed fields list The doc comment listed `image_file_name` as a CLI-managed override, but no such field is written by the CLI. Only `width` and `height` are set by `ImageMetadata::new`. The backend derives `image_file_name` from the manifest map key, not from a metadata field. Co-Authored-By: Claude Opus 4.7 (1M context) --- src/api/data_types/snapshots.rs | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/api/data_types/snapshots.rs b/src/api/data_types/snapshots.rs index e07e4e9f93..ffc6a8e6bb 100644 --- a/src/api/data_types/snapshots.rs +++ b/src/api/data_types/snapshots.rs @@ -40,8 +40,8 @@ pub struct SnapshotsManifest<'a> { // Keep in sync with https://github.com/getsentry/sentry/blob/master/src/sentry/preprod/snapshots/manifest.py /// Metadata for a single image in a snapshot manifest. /// -/// CLI-managed fields (`image_file_name`, `width`, `height`) override any -/// identically named fields provided by user sidecar metadata. +/// CLI-managed fields (`width`, `height`) override any identically named +/// fields provided by user sidecar metadata. #[derive(Debug, Serialize)] pub struct ImageMetadata { #[serde(flatten)]