fix(dart-symbol-map): Make --url global argument available

The --url argument was not recognized by the dart-symbol-map command
because it was missing from the derive parser schema. Added the url
field to the SentryCLI struct to allow global --url usage. Also
updated the legacy parser to mark --url as global for consistency
with other global arguments.

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

Author: szokeasaurusrex

src/commands/derive_parser.rs

@@ -18,6 +18,10 @@ pub(super) struct SentryCLI {
     #[arg(help = "Use the given Sentry auth token")]
     pub(super) auth_token: Option<AuthToken>,
 
+    #[arg(global = true, long, value_name = "URL")]
+    #[arg(help = "Fully qualified URL to the Sentry server.{n}[default: https://sentry.io/]")]
+    pub(super) url: Option<String>,
+
     #[arg(global=true, ignore_case=true, value_parser=["trace", "debug", "info", "warn", "error"])]
     #[arg(long, help = "Set the log output verbosity")]
     pub(super) log_level: Option<String>,

src/commands/mod.rs

@@ -154,7 +154,7 @@ fn app() -> Command {
         .max_term_width(100)
         .subcommand_required(true)
         .arg_required_else_help(true)
-        .arg(Arg::new("url").value_name("URL").long("url").help(
+        .arg(Arg::new("url").value_name("URL").long("url").global(true).help(
             "Fully qualified URL to the Sentry server.{n}\
              [default: https://sentry.io/]",
         ))

tests/integration/_responses/dart_symbol_map/assemble-success.json

@@ -0,0 +1,21 @@
+{
+  "08978d44d46a9c3c0dab8f7aabc46c68": {
+    "state": "ok",
+    "detail": null,
+    "missingChunks": [],
+    "dif": {
+      "id": "1",
+      "uuid": "00000000-0000-0000-0000-000000000000",
+      "debugId": "00000000-0000-0000-0000-000000000000",
+      "objectName": "dartsymbolmap.json",
+      "cpuName": "any",
+      "headers": {
+        "Content-Type": "application/octet-stream"
+      },
+      "size": 1,
+      "sha1": "08978d44d46a9c3c0dab8f7aabc46c68",
+      "dateCreated": "1776-07-04T12:00:00.000Z",
+      "data": {}
+    }
+  }
+}

tests/integration/upload_dart_symbol_map.rs

@@ -102,3 +102,71 @@ fn command_upload_dart_symbol_map_invalid_mapping() {
         .with_default_token()
         .run_and_assert(AssertCommand::Failure);
 }
+
+#[test]
+fn command_upload_dart_symbol_map_with_custom_url() {
+    // Test that --url global argument is recognized and used by dart-symbol-map command.
+    // This verifies the fix for issue #3106.
+    let call_count = AtomicU8::new(0);
+
+    let manager = TestManager::new()
+        .mock_endpoint(
+            MockEndpointBuilder::new("GET", "/api/0/organizations/wat-org/chunk-upload/")
+                .with_response_file("dart_symbol_map/get-chunk-upload.json"),
+        )
+        .mock_endpoint(MockEndpointBuilder::new(
+            "POST",
+            "/api/0/organizations/wat-org/chunk-upload/",
+        ))
+        .mock_endpoint(
+            MockEndpointBuilder::new(
+                "POST",
+                "/api/0/projects/wat-org/wat-project/files/difs/assemble/",
+            )
+            .with_response_fn(move |request| {
+                let body = request.body().expect("body should be readable");
+                let body_json: serde_json::Value = serde_json::from_slice(body)
+                    .expect("request body should be valid JSON");
+
+                let (checksum, _obj) = body_json
+                    .as_object()
+                    .and_then(|m| m.iter().next())
+                    .map(|(k, v)| (k.clone(), v.clone()))
+                    .expect("assemble request must contain at least one object");
+
+                match call_count.fetch_add(1, Ordering::Relaxed) {
+                    0 => format!(
+                        "{{\"{checksum}\":{{\"state\":\"not_found\",\"missingChunks\":[\"{checksum}\"]}}}}"
+                    )
+                    .into(),
+                    1 => format!(
+                        "{{\"{checksum}\":{{\"state\":\"created\",\"missingChunks\":[]}}}}"
+                    )
+                    .into(),
+                    2 => format!(
+                        "{{\"{checksum}\":{{\"state\":\"ok\",\"detail\":null,\"missingChunks\":[],\"dif\":{{\"id\":\"1\",\"uuid\":\"00000000-0000-0000-0000-000000000000\",\"debugId\":\"00000000-0000-0000-0000-000000000000\",\"objectName\":\"dartsymbolmap.json\",\"cpuName\":\"any\",\"headers\":{{\"Content-Type\":\"application/octet-stream\"}},\"size\":1,\"sha1\":\"{checksum}\",\"dateCreated\":\"1776-07-04T12:00:00.000Z\",\"data\":{{}}}}}}}}"
+                    )
+                    .into(),
+                    n => panic!(
+                        "Only 3 calls to the assemble endpoint expected, but there were {}.",
+                        n + 1
+                    ),
+                }
+            })
+            .expect(3),
+        );
+
+    let server_url = manager.server_url();
+
+    manager
+        .assert_cmd([
+            "--url",
+            &server_url,
+            "dart-symbol-map",
+            "upload",
+            "tests/integration/_fixtures/dart_symbol_map/dartsymbolmap.json",
+            "tests/integration/_fixtures/Sentry.Samples.Console.Basic.pdb",
+        ])
+        .with_default_token()
+        .run_and_assert(AssertCommand::Success);
+}