ref(sourcemaps): Fix unnecessary_wraps lint for SourceMapsProcessor::add (#2367)

Ref #2357

Author: szokeasaurusrex

src/commands/react_native/appcenter.rs

@@ -171,7 +171,7 @@ pub fn execute(matches: &ArgMatches) -> Result<()> {
                    ext == OsStr::new("bundle");
                 then {
                     let url = format!("~/{filename}");
-                    processor.add(&url, ReleaseFileSearch::collect_file(entry.path())?)?;
+                    processor.add(&url, ReleaseFileSearch::collect_file(entry.path())?);
                 }
             }
         }

src/commands/react_native/gradle.rs

@@ -95,11 +95,11 @@ pub fn execute(matches: &ArgMatches) -> Result<()> {
     processor.add(
         &bundle_url,
         ReleaseFileSearch::collect_file(bundle_path.clone())?,
-    )?;
+    );
     processor.add(
         &sourcemap_url,
         ReleaseFileSearch::collect_file(sourcemap_path)?,
-    )?;
+    );
 
     if let Ok(ram_bundle) = RamBundle::parse_unbundle_from_path(&bundle_path) {
         debug!("File RAM bundle found, extracting its contents...");

src/commands/react_native/xcode.rs

@@ -323,11 +323,11 @@ pub fn execute(matches: &ArgMatches) -> Result<()> {
     info!("  sourcemap path: {}", sourcemap_path.display());
 
     let mut processor = SourceMapProcessor::new();
-    processor.add(&bundle_url, ReleaseFileSearch::collect_file(bundle_path)?)?;
+    processor.add(&bundle_url, ReleaseFileSearch::collect_file(bundle_path)?);
     processor.add(
         &sourcemap_url,
         ReleaseFileSearch::collect_file(sourcemap_path)?,
-    )?;
+    );
     processor.rewrite(&[base.parent().unwrap().to_str().unwrap()])?;
     processor.add_sourcemap_references()?;
     processor.add_debug_id_references()?;

src/commands/sourcemaps/inject.rs

@@ -103,7 +103,7 @@ pub fn execute(matches: &ArgMatches) -> Result<()> {
             .collect_files()?;
         for source in sources {
             let url = path_as_url(&source.path);
-            processor.add(&url, source)?;
+            processor.add(&url, source);
         }
     }
 

src/commands/sourcemaps/upload.rs

@@ -302,11 +302,11 @@ fn process_sources_from_bundle(
     processor.add(
         &bundle_url,
         ReleaseFileSearch::collect_file(bundle_path.clone())?,
-    )?;
+    );
     processor.add(
         &sourcemap_url,
         ReleaseFileSearch::collect_file(sourcemap_path)?,
-    )?;
+    );
 
     if let Ok(ram_bundle) = sourcemap::ram_bundle::RamBundle::parse_unbundle_from_path(&bundle_path)
     {
@@ -394,7 +394,7 @@ fn process_sources_from_paths(
         for source in sources {
             let local_path = source.path.strip_prefix(base_path).unwrap();
             let url = format!("{}/{}{}", url_prefix, path_as_url(local_path), url_suffix);
-            processor.add(&url, source)?;
+            processor.add(&url, source);
         }
     }
 

src/utils/sourcemaps.rs

@@ -254,10 +254,8 @@ impl SourceMapProcessor {
     }
 
     /// Adds a new file for processing.
-    #[expect(clippy::unnecessary_wraps)]
-    pub fn add(&mut self, url: &str, file: ReleaseFileMatch) -> Result<()> {
+    pub fn add(&mut self, url: &str, file: ReleaseFileMatch) {
         self.pending_sources.insert((url.to_string(), file));
-        Ok(())
     }
 
     fn flush_pending_sources(&mut self) {