refactor(build): Extract version metadata writing to shared function

Extract the CLI version metadata generation logic to a shared
`write_version_metadata` function to ensure consistency across both
directory and file upload paths.

This also fixes a compression inconsistency where the metadata file
was using Deflated in normalize.rs but Stored in upload.rs. Now both
paths consistently use Deflated compression for the metadata file.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>

Author: runningcode

src/commands/build/upload.rs

@@ -14,11 +14,12 @@ use crate::api::{
     Api, AuthenticatedApi, ChunkUploadCapability, ChunkedBuildRequest, ChunkedFileState, VcsInfo,
 };
 use crate::config::Config;
-use crate::constants::VERSION;
 use crate::utils::args::ArgExt as _;
 #[cfg(all(target_os = "macos", target_arch = "aarch64"))]
 use crate::utils::build::{handle_asset_catalogs, ipa_to_xcarchive, is_apple_app, is_ipa_file};
-use crate::utils::build::{is_aab_file, is_apk_file, is_zip_file, normalize_directory};
+use crate::utils::build::{
+    is_aab_file, is_apk_file, is_zip_file, normalize_directory, write_version_metadata,
+};
 use crate::utils::chunks::{upload_chunks, Chunk};
 use crate::utils::fs::get_sha1_checksums;
 use crate::utils::fs::TempDir;
@@ -456,8 +457,7 @@ fn normalize_file(path: &Path, bytes: &[u8]) -> Result<TempFile> {
     zip.start_file(file_name, options)?;
     zip.write_all(bytes)?;
 
-    zip.start_file(".sentry-cli-metadata.txt", options)?;
-    writeln!(zip, "sentry-cli-version: {VERSION}")?;
+    write_version_metadata(&mut zip)?;
 
     zip.finish()?;
     debug!("Successfully created normalized zip for file");

src/utils/build/mod.rs

@@ -5,7 +5,7 @@ mod validation;
 
 #[cfg(all(target_os = "macos", target_arch = "aarch64"))]
 pub use self::apple::{handle_asset_catalogs, ipa_to_xcarchive};
-pub use self::normalize::normalize_directory;
+pub use self::normalize::{normalize_directory, write_version_metadata};
 pub use self::validation::{is_aab_file, is_apk_file, is_zip_file};
 #[cfg(all(target_os = "macos", target_arch = "aarch64"))]
 pub use self::validation::{is_apple_app, is_ipa_file};

src/utils/build/normalize.rs

@@ -77,6 +77,23 @@ fn add_entries_to_zip(
     Ok(file_count)
 }
 
+/// Creates file options with consistent compression and timestamp settings for metadata files.
+/// Uses Deflated compression to match other files in the archive.
+fn metadata_file_options() -> SimpleFileOptions {
+    SimpleFileOptions::default()
+        .compression_method(zip::CompressionMethod::Deflated)
+        .last_modified_time(DateTime::default())
+}
+
+/// Writes the CLI version metadata to a zip archive.
+pub fn write_version_metadata<W: std::io::Write + std::io::Seek>(
+    zip: &mut ZipWriter<W>,
+) -> Result<()> {
+    zip.start_file(".sentry-cli-metadata.txt", metadata_file_options())?;
+    writeln!(zip, "sentry-cli-version: {VERSION}")?;
+    Ok(())
+}
+
 // For XCArchive directories, we'll zip the entire directory
 // It's important to not change the contents of the directory or the size
 // analysis will be wrong and the code signature will break.
@@ -109,11 +126,7 @@ pub fn normalize_directory(path: &Path, parsed_assets_path: &Path) -> Result<Tem
         )?;
     }
 
-    let options = SimpleFileOptions::default()
-        .compression_method(zip::CompressionMethod::Deflated)
-        .last_modified_time(DateTime::default());
-    zip.start_file(".sentry-cli-metadata.txt", options)?;
-    writeln!(zip, "sentry-cli-version: {VERSION}")?;
+    write_version_metadata(&mut zip)?;
 
     zip.finish()?;
     debug!("Successfully created normalized zip for directory with {file_count} files");