feat(mobile-app): Add release notes option

Author: noahsmartin

src/api/data_types/chunking/mobile_app.rs

@@ -10,6 +10,7 @@ pub struct ChunkedMobileAppRequest<'a> {
     pub chunks: &'a [Digest],
     #[serde(skip_serializing_if = "Option::is_none")]
     pub build_configuration: Option<&'a str>,
+    pub release_notes: Option<&'a str>,
     // VCS fields
     #[serde(skip_serializing_if = "Option::is_none")]
     pub head_sha: Option<&'a str>,

src/api/mod.rs

@@ -1035,10 +1035,7 @@ impl<'a> AuthenticatedApi<'a> {
         &self,
         org: &str,
         project: &str,
-        checksum: Digest,
-        chunks: &[Digest],
-        build_configuration: Option<&str>,
-        vcs_info: &VcsInfo<'_>,
+        request: &ChunkedMobileAppRequest<'_>,
     ) -> ApiResult<AssembleMobileAppResponse> {
         let url = format!(
             "/projects/{}/{}/files/preprodartifacts/assemble/",
@@ -1047,19 +1044,7 @@ impl<'a> AuthenticatedApi<'a> {
         );
 
         self.request(Method::Post, &url)?
-            .with_json_body(&ChunkedMobileAppRequest {
-                checksum,
-                chunks,
-                build_configuration,
-                head_sha: vcs_info.head_sha,
-                base_sha: vcs_info.base_sha,
-                provider: vcs_info.vcs_provider,
-                head_repo_name: vcs_info.head_repo_name,
-                base_repo_name: vcs_info.base_repo_name,
-                head_ref: vcs_info.head_ref,
-                base_ref: vcs_info.base_ref,
-                pr_number: vcs_info.pr_number,
-            })?
+            .with_json_body(&request)?
             .send()?
             .convert_rnf(ApiErrorKind::ProjectNotFound)
     }

src/commands/mobile_app/upload.rs

@@ -10,7 +10,10 @@ use symbolic::common::ByteView;
 use zip::write::SimpleFileOptions;
 use zip::{DateTime, ZipWriter};
 
-use crate::api::{Api, AuthenticatedApi, ChunkUploadCapability, ChunkedFileState, VcsInfo};
+use crate::api::{
+    Api, AuthenticatedApi, ChunkUploadCapability, ChunkedFileState, ChunkedMobileAppRequest,
+    VcsInfo,
+};
 use crate::config::Config;
 use crate::utils::args::ArgExt as _;
 use crate::utils::chunks::{upload_chunks, Chunk};
@@ -91,6 +94,11 @@ pub fn make_command(command: Command) -> Command {
                 .long("build-configuration")
                 .help("The build configuration to use for the upload. If not provided, the current version will be used.")
         )
+        .arg(
+            Arg::new("release_notes")
+                .long("release-notes")
+                .help("The release notes to use for the upload.")
+        )
 }
 
 pub fn execute(matches: &ArgMatches) -> Result<()> {
@@ -145,6 +153,7 @@ pub fn execute(matches: &ArgMatches) -> Result<()> {
     let pr_number = matches.get_one::<u32>("pr_number");
 
     let build_configuration = matches.get_one("build_configuration").map(String::as_str);
+    let release_notes = matches.get_one("release_notes").map(String::as_str);
 
     let api = Api::current();
     let authenticated_api = api.authenticated()?;
@@ -217,6 +226,7 @@ pub fn execute(matches: &ArgMatches) -> Result<()> {
             &org,
             &project,
             build_configuration,
+            release_notes,
             &vcs_info,
         ) {
             Ok(artifact_url) => {
@@ -376,6 +386,7 @@ fn upload_file(
     org: &str,
     project: &str,
     build_configuration: Option<&str>,
+    release_notes: Option<&str>,
     vcs_info: &VcsInfo<'_>,
 ) -> Result<String> {
     const SELF_HOSTED_ERROR_HINT: &str = "If you are using a self-hosted Sentry server, \
@@ -436,10 +447,20 @@ fn upload_file(
         let response = api.assemble_mobile_app(
             org,
             project,
-            checksum,
-            &checksums,
-            build_configuration,
-            vcs_info,
+            &ChunkedMobileAppRequest {
+                checksum,
+                chunks: &checksums,
+                build_configuration,
+                release_notes,
+                head_sha: vcs_info.head_sha,
+                base_sha: vcs_info.base_sha,
+                provider: vcs_info.vcs_provider,
+                head_repo_name: vcs_info.head_repo_name,
+                base_repo_name: vcs_info.base_repo_name,
+                head_ref: vcs_info.head_ref,
+                base_ref: vcs_info.base_ref,
+                pr_number: vcs_info.pr_number,
+            },
         )?;
         chunks.retain(|Chunk((digest, _))| response.missing_chunks.contains(digest));
 

tests/integration/_cases/mobile_app/mobile_app-upload-help-macos.trycmd

@@ -51,6 +51,8 @@ Options:
       --build-configuration <build_configuration>
           The build configuration to use for the upload. If not provided, the current version will
           be used.
+      --release-notes <release_notes>
+          The release notes to use for the upload.
   -h, --help
           Print help
 

tests/integration/_cases/mobile_app/mobile_app-upload-help-not-macos.trycmd

@@ -49,6 +49,8 @@ Options:
       --build-configuration <build_configuration>
           The build configuration to use for the upload. If not provided, the current version will
           be used.
+      --release-notes <release_notes>
+          The release notes to use for the upload.
   -h, --help
           Print help