chore(api): Stop calling project-release endpoints (#2991)

### Description
We should instead use the organization-release endpoints. The
project-release endpoints have been kept around for backwards
compatibility with ancient self-hosted versions, which we are dropping
support for.

### Issues
- Resolves #2840 - Resolves
[CLI-186](https://linear.app/getsentry/issue/CLI-186/remove-calls-to-projectsreleases-endpoints)

Author: szokeasaurusrex

src/api/mod.rs

@@ -27,7 +27,6 @@ use chrono::Duration;
 use chrono::{DateTime, FixedOffset, Utc};
 use clap::ArgMatches;
 use flate2::write::GzEncoder;
-use if_chain::if_chain;
 use lazy_static::lazy_static;
 use log::{debug, info, warn};
 use parking_lot::Mutex;
@@ -460,21 +459,9 @@ impl AuthenticatedApi<'_> {
 
     /// Creates a new release.
     pub fn new_release(&self, org: &str, release: &NewRelease) -> ApiResult<ReleaseInfo> {
-        // for single project releases use the legacy endpoint that is project bound.
-        // This means we can support both old and new servers.
-        if release.projects.len() == 1 {
-            let path = format!(
-                "/projects/{}/{}/releases/",
-                PathArg(org),
-                PathArg(&release.projects[0])
-            );
-            self.post(&path, release)?
-                .convert_rnf(ApiErrorKind::ProjectNotFound)
-        } else {
-            let path = format!("/organizations/{}/releases/", PathArg(org));
-            self.post(&path, release)?
-                .convert_rnf(ApiErrorKind::OrganizationNotFound)
-        }
+        let path = format!("/organizations/{}/releases/", PathArg(org));
+        self.post(&path, release)?
+            .convert_rnf(ApiErrorKind::OrganizationNotFound)
     }
 
     /// Updates a release.
@@ -484,29 +471,20 @@ impl AuthenticatedApi<'_> {
         version: &str,
         release: &UpdatedRelease,
     ) -> ApiResult<ReleaseInfo> {
-        if_chain! {
-            if let Some(ref projects) = release.projects;
-            if projects.len() == 1;
-            then {
-                let path = format!("/projects/{}/{}/releases/{}/",
-                    PathArg(org),
-                    PathArg(&projects[0]),
-                    PathArg(version)
-                );
-                self.put(&path, release)?.convert_rnf(ApiErrorKind::ReleaseNotFound)
-            } else {
-                if release.version.is_some() {
-                    let path = format!("/organizations/{}/releases/",
-                                    PathArg(org));
-                    return self.post(&path, release)?.convert_rnf(ApiErrorKind::ReleaseNotFound)
-                }
-
-                let path = format!("/organizations/{}/releases/{}/",
-                                PathArg(org),
-                                PathArg(version));
-                self.put(&path, release)?.convert_rnf(ApiErrorKind::ReleaseNotFound)
-            }
+        if release.version.is_some() {
+            let path = format!("/organizations/{}/releases/", PathArg(org));
+            return self
+                .post(&path, release)?
+                .convert_rnf(ApiErrorKind::ReleaseNotFound);
         }
+
+        let path = format!(
+            "/organizations/{}/releases/{}/",
+            PathArg(org),
+            PathArg(version)
+        );
+        self.put(&path, release)?
+            .convert_rnf(ApiErrorKind::ReleaseNotFound)
     }
 
     /// Sets release commits
@@ -530,28 +508,14 @@ impl AuthenticatedApi<'_> {
     }
 
     /// Deletes an already existing release.  Returns `true` if it was deleted
-    /// or `false` if not.  The project is needed to support the old deletion
-    /// API.
-    pub fn delete_release(
-        &self,
-        org: &str,
-        project: Option<&str>,
-        version: &str,
-    ) -> ApiResult<bool> {
-        let resp = if let Some(project) = project {
-            self.delete(&format!(
-                "/projects/{}/{}/releases/{}/",
-                PathArg(org),
-                PathArg(project),
-                PathArg(version)
-            ))?
-        } else {
-            self.delete(&format!(
-                "/organizations/{}/releases/{}/",
-                PathArg(org),
-                PathArg(version)
-            ))?
-        };
+    /// or `false` if not.
+    pub fn delete_release(&self, org: &str, version: &str) -> ApiResult<bool> {
+        let resp = self.delete(&format!(
+            "/organizations/{}/releases/{}/",
+            PathArg(org),
+            PathArg(version)
+        ))?;
+
         if resp.status() == 404 {
             Ok(false)
         } else {
@@ -561,26 +525,12 @@ impl AuthenticatedApi<'_> {
 
     /// Looks up a release and returns it.  If it does not exist `None`
     /// will be returned.
-    pub fn get_release(
-        &self,
-        org: &str,
-        project: Option<&str>,
-        version: &str,
-    ) -> ApiResult<Option<ReleaseInfo>> {
-        let path = if let Some(project) = project {
-            format!(
-                "/projects/{}/{}/releases/{}/",
-                PathArg(org),
-                PathArg(project),
-                PathArg(version)
-            )
-        } else {
-            format!(
-                "/organizations/{}/releases/{}/",
-                PathArg(org),
-                PathArg(version)
-            )
-        };
+    pub fn get_release(&self, org: &str, version: &str) -> ApiResult<Option<ReleaseInfo>> {
+        let path = format!(
+            "/organizations/{}/releases/{}/",
+            PathArg(org),
+            PathArg(version)
+        );
         let resp = self.get(&path)?;
         if resp.status() == 404 {
             Ok(None)
@@ -591,40 +541,24 @@ impl AuthenticatedApi<'_> {
 
     /// Returns a list of releases for a given project.  This is currently a
     /// capped list by what the server deems an acceptable default limit.
-    pub fn list_releases(&self, org: &str, project: Option<&str>) -> ApiResult<Vec<ReleaseInfo>> {
-        if let Some(project) = project {
-            let path = format!("/projects/{}/{}/releases/", PathArg(org), PathArg(project));
-            self.get(&path)?
-                .convert_rnf::<Vec<ReleaseInfo>>(ApiErrorKind::ProjectNotFound)
-        } else {
-            let path = format!("/organizations/{}/releases/", PathArg(org));
-            self.get(&path)?
-                .convert_rnf::<Vec<ReleaseInfo>>(ApiErrorKind::OrganizationNotFound)
-        }
+    pub fn list_releases(&self, org: &str) -> ApiResult<Vec<ReleaseInfo>> {
+        let path = format!("/organizations/{}/releases/", PathArg(org));
+        self.get(&path)?
+            .convert_rnf::<Vec<ReleaseInfo>>(ApiErrorKind::OrganizationNotFound)
     }
 
     /// Looks up a release commits and returns it.  If it does not exist `None`
     /// will be returned.
     pub fn get_release_commits(
         &self,
         org: &str,
-        project: Option<&str>,
         version: &str,
     ) -> ApiResult<Option<Vec<ReleaseCommit>>> {
-        let path = if let Some(project) = project {
-            format!(
-                "/projects/{}/{}/releases/{}/commits/",
-                PathArg(org),
-                PathArg(project),
-                PathArg(version)
-            )
-        } else {
-            format!(
-                "/organizations/{}/releases/{}/commits/",
-                PathArg(org),
-                PathArg(version)
-            )
-        };
+        let path = format!(
+            "/organizations/{}/releases/{}/commits/",
+            PathArg(org),
+            PathArg(version)
+        );
         let resp = self.get(&path)?;
         if resp.status() == 404 {
             Ok(None)

src/commands/releases/delete.rs

@@ -17,13 +17,11 @@ pub fn execute(matches: &ArgMatches) -> Result<()> {
     let api = Api::current();
     #[expect(clippy::unwrap_used, reason = "legacy code")]
     let version = matches.get_one::<String>("version").unwrap();
-    let project = config.get_project(matches).ok();
 
-    if api.authenticated()?.delete_release(
-        &config.get_org(matches)?,
-        project.as_deref(),
-        version,
-    )? {
+    if api
+        .authenticated()?
+        .delete_release(&config.get_org(matches)?, version)?
+    {
         println!("Deleted release {version}!");
     } else {
         println!("Did nothing. Release with this version ({version}) does not exist.");

src/commands/releases/info.rs

@@ -36,8 +36,7 @@ pub fn execute(matches: &ArgMatches) -> Result<()> {
     let version = matches.get_one::<String>("version").unwrap();
     let config = Config::current();
     let org = config.get_org(matches)?;
-    let project = config.get_project(matches).ok();
-    let release = authenticated_api.get_release(&org, project.as_deref(), version)?;
+    let release = authenticated_api.get_release(&org, version)?;
 
     if is_quiet_mode() {
         if release.is_none() {
@@ -85,9 +84,7 @@ pub fn execute(matches: &ArgMatches) -> Result<()> {
         }
 
         if matches.get_flag("show_commits") {
-            if let Ok(Some(commits)) =
-                authenticated_api.get_release_commits(&org, project.as_deref(), version)
-            {
+            if let Ok(Some(commits)) = authenticated_api.get_release_commits(&org, version) {
                 if !commits.is_empty() {
                     data_row.add(
                         commits

src/commands/releases/list.rs

@@ -43,10 +43,9 @@ pub fn make_command(command: Command) -> Command {
 pub fn execute(matches: &ArgMatches) -> Result<()> {
     let config = Config::current();
     let api = Api::current();
-    let project = config.get_project(matches).ok();
     let releases = api
         .authenticated()?
-        .list_releases(&config.get_org(matches)?, project.as_deref())?;
+        .list_releases(&config.get_org(matches)?)?;
 
     if matches.get_flag("raw") {
         let versions = releases

tests/integration/releases/delete.rs

@@ -6,7 +6,7 @@ fn successfully_deletes() {
         .mock_endpoint(
             MockEndpointBuilder::new(
                 "DELETE",
-                "/api/0/projects/wat-org/wat-project/releases/wat-release/",
+                "/api/0/organizations/wat-org/releases/wat-release/",
             )
             .with_status(204),
         )
@@ -20,7 +20,7 @@ fn allows_for_release_to_start_with_hyphen() {
         .mock_endpoint(
             MockEndpointBuilder::new(
                 "DELETE",
-                "/api/0/projects/wat-org/wat-project/releases/-hyphenated-release/",
+                "/api/0/organizations/wat-org/releases/-hyphenated-release/",
             )
             .with_status(204),
         )
@@ -32,11 +32,8 @@ fn allows_for_release_to_start_with_hyphen() {
 fn informs_about_nonexisting_releases() {
     TestManager::new()
         .mock_endpoint(
-            MockEndpointBuilder::new(
-                "DELETE",
-                "/api/0/projects/wat-org/wat-project/releases/whoops/",
-            )
-            .with_status(404),
+            MockEndpointBuilder::new("DELETE", "/api/0/organizations/wat-org/releases/whoops/")
+                .with_status(404),
         )
         .register_trycmd_test("releases/releases-delete-nonexisting.trycmd")
         .with_default_token();
@@ -48,7 +45,7 @@ fn doesnt_allow_to_delete_active_releases() {
         .mock_endpoint(
             MockEndpointBuilder::new(
                 "DELETE",
-                "/api/0/projects/wat-org/wat-project/releases/wat-release/",
+                "/api/0/organizations/wat-org/releases/wat-release/",
             )
             .with_status(400)
             .with_response_file("releases/delete-active-release.json"),

tests/integration/releases/finalize.rs

@@ -6,11 +6,8 @@ use serde_json::json;
 fn successfully_creates_a_release() {
     TestManager::new()
         .mock_endpoint(
-            MockEndpointBuilder::new(
-                "PUT",
-                "/api/0/projects/wat-org/wat-project/releases/wat-release/",
-            )
-            .with_response_file("releases/get-release.json"),
+            MockEndpointBuilder::new("PUT", "/api/0/organizations/wat-org/releases/wat-release/")
+                .with_response_file("releases/get-release.json"),
         )
         .register_trycmd_test("releases/releases-finalize.trycmd")
         .with_default_token();
@@ -22,7 +19,7 @@ fn allows_for_release_to_start_with_hyphen() {
         .mock_endpoint(
             MockEndpointBuilder::new(
                 "PUT",
-                "/api/0/projects/wat-org/wat-project/releases/-hyphenated-release/",
+                "/api/0/organizations/wat-org/releases/-hyphenated-release/",
             )
             .with_response_file("releases/get-release.json"),
         )
@@ -34,15 +31,12 @@ fn allows_for_release_to_start_with_hyphen() {
 fn release_with_custom_dates() {
     TestManager::new()
         .mock_endpoint(
-            MockEndpointBuilder::new(
-                "PUT",
-                "/api/0/projects/wat-org/wat-project/releases/wat-release/",
-            )
-            .with_response_file("releases/get-release.json")
-            .with_matcher(Matcher::PartialJson(json!({
-                "projects": ["wat-project"],
-                "dateReleased": "2015-05-15T00:00:00Z"
-            }))),
+            MockEndpointBuilder::new("PUT", "/api/0/organizations/wat-org/releases/wat-release/")
+                .with_response_file("releases/get-release.json")
+                .with_matcher(Matcher::PartialJson(json!({
+                    "projects": ["wat-project"],
+                    "dateReleased": "2015-05-15T00:00:00Z"
+                }))),
         )
         .register_trycmd_test("releases/releases-finalize-dates.trycmd")
         .with_default_token();