@knime/hub-features 1.15.3 → 1.15.4

package/CHANGELOG.md

@@ -1,5 +1,13 @@
 # @knime/hub-features
 
+## 1.15.4
+
+### Patch Changes
+
+- e44c108: fix rfcErrors parser and return undefined on error
+- Updated dependencies [e44c108]
+  - @knime/components@1.41.8
+
 ## 1.15.3
 
 ### Patch Changes

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@knime/hub-features",
-  "version": "1.15.3",
+  "version": "1.15.4",
   "description": "Vue components & composables for shared hub features",
   "repository": {
     "type": "git",
@@ -36,7 +36,7 @@
     "lodash-es": "4.17.21",
     "ofetch": "^1.4.1",
     "typescript": "^5.8.3",
-    "@knime/components": "1.41.7",
+    "@knime/components": "1.41.8",
     "@knime/styles": "1.14.3",
     "@knime/utils": "1.7.1"
   },

package/src/components/versions/composables/useVersionsApi.ts

@@ -1,4 +1,4 @@
-import { FetchError, type FetchOptions } from "ofetch";
+import { type FetchOptions } from "ofetch";
 
 import { type HubAvatarData, rfcErrors } from "@knime/hub-features";
 import { VERSION_DEFAULT_LIMIT } from "@knime/hub-features/versions";
@@ -37,11 +37,7 @@ export const useVersionsApi = ({
 
       return await $ofetch(path, { ...defaults, ...fetchOptions });
     } catch (error) {
-      if (error instanceof FetchError) {
-        throw rfcErrors.tryParse(error);
-      }
-
-      throw error;
+      throw rfcErrors.tryParse(error) ?? error;
     }
   };
 

package/src/rfcErrors/index.ts

@@ -35,21 +35,28 @@ const toToast = ({
 };
 
 /**
- * Try to parse an ofetch's `FetchError` to an `RFCError`. When parsing is not possible
- * it returns the same `FetchError` given unchanged.
+ * Parse an ofetch's `FetchError` to an `RFCError`.
+ * When parsing is not possible it returns undefined.
  */
-const tryParse = (error: FetchError): RFCError | FetchError => {
+const tryParse = (error: unknown): RFCError | undefined => {
+  if (!(error instanceof FetchError)) {
+    return undefined;
+  }
+
   if (!error.response) {
-    return error;
+    return undefined;
   }
 
   const responseDate = error.response.headers.get("date")
     ? new Date(error.response.headers.get("date")!)
     : undefined;
 
+  const title =
+    typeof error.data === "string" ? error.data : error.data?.title ?? "";
+
   const rfcErrorData: RFCErrorData = {
-    title: error.data.title as string,
-    details: error.data.details as string[] | undefined,
+    title: title as string,
+    details: error.data?.details as string[] | undefined,
     status: error.statusCode as number,
     date: responseDate,
     requestId: error.response.headers.get("x-request-id") ?? undefined,

package/src/useDownloadArtifact/useDownloadArtifact.ts

@@ -284,10 +284,7 @@ export const useDownloadArtifact = (
     } catch (error: unknown) {
       // here only errors thrown by the initial requests are caught;
       // errors occurring while polling will be handled in pollDownloadItem
-      if (error instanceof FetchError) {
-        throw rfcErrors.tryParse(error);
-      }
-      throw error;
+      throw rfcErrors.tryParse(error) ?? error;
     }
   };
 

package/src/useFileUpload/useFileUpload.ts

@@ -287,11 +287,7 @@ export const useFileUpload = (options: UseFileUploadOptions = {}) => {
 
         useUploadManagerResult.start(parentId, uploadPayload);
       } catch (error) {
-        if (error instanceof FetchError) {
-          throw rfcErrors.tryParse(error);
-        }
-
-        throw error;
+        throw rfcErrors.tryParse(error) ?? error;
       } finally {
         // errors can only be thrown in the prepareUpload call, useUploadManagerResult.start does its own error handling
         prepareQueueSize.value -= enqueableFiles.length;