Merge pull request #69 from palantir/zka/surface-version-already-exists

Resurface `Version already exists` message
palantir · Feb 16, 2024 · f78d825 · f78d825
2 parents 70f597a + 11ed160
commit f78d825
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 1 deletion.
diff --git a/packages/cli/changelog/@unreleased/pr-69.v2.yml b/packages/cli/changelog/@unreleased/pr-69.v2.yml
@@ -0,0 +1,5 @@
+type: fix
+fix:
+  description: Resurface `Version Already Exists` message
+  links:
+  - https://github.com/palantir/osdk-ts/pull/69
diff --git a/packages/cli/src/net/createFetch.mts b/packages/cli/src/net/createFetch.mts
@@ -63,7 +63,12 @@ function handleFetchError(e: unknown): Promise<Response> {
     tip = "Check your token has the required scopes for this operation";
   }
 
-  throw new ExitProcessError(1, e.message, tip);
+  let message = e.message;
+  if (e.errorName === "Artifacts:ArtifactAlreadyExists") {
+    message = "Version already exists";
+  }
+
+  throw new ExitProcessError(1, message, tip);
 }
 
 function createRequestLoggingFetch(