From e63ee73fea8d4e9a3fbefdec7533349e688b366a Mon Sep 17 00:00:00 2001
From: Oliver Browne <oliver@posthog.com>
Date: Fri, 17 Oct 2025 17:34:55 +0300
Subject: [PATCH] fix(cli): add back upload arguments, warn on them (#39888)

---
 cli/CHANGELOG.md             |  4 ++++
 cli/Cargo.lock               | 16 ++++++++--------
 cli/Cargo.toml               |  2 +-
 cli/src/sourcemaps/mod.rs    |  2 ++
 cli/src/sourcemaps/upload.rs | 16 +++++++++++++++-
 5 files changed, 30 insertions(+), 10 deletions(-)

diff --git a/cli/CHANGELOG.md b/cli/CHANGELOG.md
index 0d5e4d8c10..ec866abf99 100644
--- a/cli/CHANGELOG.md
+++ b/cli/CHANGELOG.md
@@ -1,5 +1,9 @@
 # posthog-cli
 
+# 0.5.1
+
+- Attempts to reduce impact of previous breaking changes - re-adds `--project` and `--version` arguments to sourcemap upload command, marking them as no longer used
+
 # 0.5.0
 
 - Sourcemap injection, upload and process commands made retriable. Significant improvement to release creation.
diff --git a/cli/Cargo.lock b/cli/Cargo.lock
index 2df138600e..afaf413d6f 100644
--- a/cli/Cargo.lock
+++ b/cli/Cargo.lock
@@ -951,7 +951,7 @@ dependencies = [
  "http 1.3.1",
  "hyper 1.7.0",
  "hyper-util",
- "rustls 0.23.32",
+ "rustls 0.23.33",
  "rustls-pki-types",
  "tokio",
  "tokio-rustls 0.26.4",
@@ -1517,7 +1517,7 @@ checksum = "8b870d8c151b6f2fb93e84a13146138f05d02ed11c7e7c54f8826aaaf7c9f184"
 
 [[package]]
 name = "posthog-cli"
-version = "0.5.0"
+version = "0.5.1"
 dependencies = [
  "anyhow",
  "chrono",
@@ -1611,7 +1611,7 @@ dependencies = [
  "quinn-proto",
  "quinn-udp",
  "rustc-hash",
- "rustls 0.23.32",
+ "rustls 0.23.33",
  "socket2 0.6.1",
  "thiserror 2.0.17",
  "tokio",
@@ -1631,7 +1631,7 @@ dependencies = [
  "rand",
  "ring",
  "rustc-hash",
- "rustls 0.23.32",
+ "rustls 0.23.33",
  "rustls-pki-types",
  "slab",
  "thiserror 2.0.17",
@@ -1858,7 +1858,7 @@ dependencies = [
  "percent-encoding",
  "pin-project-lite",
  "quinn",
- "rustls 0.23.32",
+ "rustls 0.23.33",
  "rustls-pki-types",
  "serde",
  "serde_json",
@@ -1942,9 +1942,9 @@ dependencies = [
 
 [[package]]
 name = "rustls"
-version = "0.23.32"
+version = "0.23.33"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "cd3c25631629d034ce7cd9940adc9d45762d46de2b0f57193c4443b92c6d4d40"
+checksum = "751e04a496ca00bb97a5e043158d23d66b5aabf2e1d5aa2a0aaebb1aafe6f82c"
 dependencies = [
  "once_cell",
  "ring",
@@ -2476,7 +2476,7 @@ version = "0.26.4"
 source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "1729aa945f29d91ba541258c8df89027d5792d85a8841fb65e8bf0f4ede4ef61"
 dependencies = [
- "rustls 0.23.32",
+ "rustls 0.23.33",
  "tokio",
 ]
 
diff --git a/cli/Cargo.toml b/cli/Cargo.toml
index 9d3c2ef736..15f32380c9 100644
--- a/cli/Cargo.toml
+++ b/cli/Cargo.toml
@@ -1,6 +1,6 @@
 [package]
 name = "posthog-cli"
-version = "0.5.0"
+version = "0.5.1"
 authors = [
     "David <david@posthog.com>",
     "Olly <oliver@posthog.com>",
diff --git a/cli/src/sourcemaps/mod.rs b/cli/src/sourcemaps/mod.rs
index 2d415b63f7..0ded594fdd 100644
--- a/cli/src/sourcemaps/mod.rs
+++ b/cli/src/sourcemaps/mod.rs
@@ -71,6 +71,8 @@ impl From<ProcessArgs> for (InjectArgs, UploadArgs) {
             delete_after: args.delete_after,
             skip_ssl_verification: args.skip_ssl_verification,
             batch_size: args.batch_size,
+            project: None,
+            version: None,
         };
 
         (inject_args, upload_args)
diff --git a/cli/src/sourcemaps/upload.rs b/cli/src/sourcemaps/upload.rs
index 4246845f5c..a8c93212d3 100644
--- a/cli/src/sourcemaps/upload.rs
+++ b/cli/src/sourcemaps/upload.rs
@@ -1,7 +1,7 @@
 use std::path::PathBuf;
 
 use anyhow::{Context, Ok, Result};
-use tracing::info;
+use tracing::{info, warn};
 
 use crate::api::symbol_sets::{upload, SymbolSetUpload};
 use crate::invocation_context::context;
@@ -31,6 +31,14 @@ pub struct UploadArgs {
     /// The maximum number of chunks to upload in a single batch
     #[arg(long, default_value = "50")]
     pub batch_size: usize,
+
+    /// DEPRECATED: Does nothing. Set project during `inject` instead
+    #[arg(long)]
+    pub project: Option<String>,
+
+    /// DEPRECATED: Does nothing. Set version during `inject` instead
+    #[arg(long)]
+    pub version: Option<String>,
 }
 
 pub fn upload_cmd(args: UploadArgs) -> Result<()> {
@@ -40,8 +48,14 @@ pub fn upload_cmd(args: UploadArgs) -> Result<()> {
         delete_after,
         skip_ssl_verification: _,
         batch_size,
+        project: p,
+        version: v,
     } = args;
 
+    if p.is_some() || v.is_some() {
+        warn!("`--project` and `--version` are deprecated and do nothing. Set project and version during `inject` instead.");
+    }
+
     context().capture_command_invoked("sourcemap_upload");
 
     let pairs = read_pairs(&directory, &ignore)?;