feat(sourcemaps): Remove use-artifact-bundle flag

Remove the no-op --use-artifact-bundle CLI flag, drop the JS option,
and document the breaking change.

Closes #2352
Closes [CLI-20](https://linear.app/getsentry/issue/CLI-20/remove-use-artifact-bundle-option)

Author: cursoragent

CHANGELOG.md

@@ -23,6 +23,7 @@ we should rename this section to "Unreleased" -->
 - Removed the `--started` argument from the `sentry-cli releases finalize` command ([#XXXX](TODO)). This argument is a no-op, so any users using it should simply stop using it.
 - In the JS API, the `SentryCli.execute` method's `live` parameter now only takes boolean values ([#2971](https://github.com/getsentry/sentry-cli/pull/2971)). Setting `live` to `true` now behaves like `'rejectOnError'` did previously, with a zero exit status resolving the returned promise with `"success (live mode)"` and a non-zero status rejecting the promise with an error message.
 - In the JS API, the `option` parameter to `Releases.uploadSourceMaps` no longer takes a `live` property ([#2971](https://github.com/getsentry/sentry-cli/pull/2971)). We now always execute the command with `live` set to `true`.
+- Removed the `--use-artifact-bundle` flag from `sentry-cli sourcemaps upload`, along with the corresponding `useArtifactBundle` JavaScript option ([#XXXX](TODO)). The flag was a no-op that only emitted a deprecation warning.
 
 #### Node.js Wrapper Breakages
 

lib/releases/options/uploadSourcemaps.js

@@ -55,8 +55,4 @@ module.exports = {
     param: '--ext',
     type: 'array',
   },
-  useArtifactBundle: {
-    param: '--use-artifact-bundle',
-    type: 'boolean',
-  },
 };

lib/types.ts

@@ -137,14 +137,6 @@ export interface SentryCliUploadSourceMapsOptions {
    * Usually your build number.
    */
   dist?: string;
-  /**
-   * Force use of new Artifact Bundles upload, that enables use of Debug ID for Source Maps discovery,
-   * even when the Sentry server does not declare support for it.
-   *
-   * @deprecated This option is deprecated and will be removed in the next major version. Sentry CLI
-   *     should always respect what the server says it supports.
-   */
-  useArtifactBundle?: boolean;
 }
 
 /**

src/commands/sourcemaps/upload.rs

@@ -1,6 +1,5 @@
 #![expect(clippy::unwrap_used, reason = "contains legacy code which uses unwrap")]
 
-use std::env;
 use std::path::PathBuf;
 use std::time::Duration;
 
@@ -223,17 +222,6 @@ pub fn make_command(command: Command) -> Command {
                      uploaded.",
                 ),
         )
-        .arg(
-            Arg::new("use_artifact_bundle")
-                .long("use-artifact-bundle")
-                .action(ArgAction::SetTrue)
-                .help(
-                    "[DEPRECATED] Force artifact bundles to be used for upload, even when not \
-                    supported by the server. This option has always only been intended for \
-                    internal use, and it is now officially deprecated.",
-                )
-                .hide(true),
-        )
         // Legacy flag that has no effect, left hidden for backward compatibility
         .arg(
             Arg::new("rewrite")
@@ -431,13 +419,6 @@ pub fn execute(matches: &ArgMatches) -> Result<()> {
     let mut processor = SourceMapProcessor::new();
     let chunk_upload_options = api.authenticated()?.get_chunk_upload_options(&org)?;
 
-    if matches.get_flag("use_artifact_bundle")
-        || env::var("SENTRY_FORCE_ARTIFACT_BUNDLES").ok().as_deref() == Some("1")
-    {
-        log::warn!("The --use-artifact-bundle option and the SENTRY_FORCE_ARTIFACT_BUNDLES environment variable \
-                    are both deprecated, and both will be removed in the next major version.");
-    }
-
     if matches.contains_id("bundle") && matches.contains_id("bundle_sourcemap") {
         process_sources_from_bundle(matches, &mut processor)?;
     } else {