e621-client 1.1.2 → 1.1.4

package/dist/client/E621ApiClient.js

@@ -112,16 +112,20 @@ class E621ApiClient {
             if (!this.isBrowser) {
                 headers.addUserAgent(this.client);
             }
-            const response = yield fetch(this.buildUrl(route, params), {
+            const url = this.buildUrl(route, params);
+            const response = yield fetch(url, {
                 method: "GET",
                 headers: headers.addAuthorization(this.credentials).build(),
             });
             if (response.ok) {
-                return response.json().catch(utils_1.logErrorAndReturnNull);
+                return response.json();
             }
-            else {
+            else if (response.status === 404) {
                 return null;
             }
+            else {
+                throw new Error(`Request to ${route} failed with status code ${response.status}`);
+            }
         });
     }
     buildUrl(route, params = {}) {

package/dist/test.js

@@ -16,8 +16,8 @@ function test() {
             isBrowser: false,
             client: "EstoBox (by Tommus621 on e621)",
         });
-        const user = yield client.getUserById(918032);
-        console.log(user);
+        const favs = yield client.getFavorites({ userId: "964986", page: "2" });
+        console.log(favs.length);
     });
 }
 test().finally();

package/dist/utils.d.ts

@@ -1,3 +1,2 @@
 export declare function ensureArray<T>(value: T | T[]): T[];
 export declare function translateOptions<Options extends Record<string, string>>(options: Options, table: Partial<Options>): Record<string, string>;
-export declare function logErrorAndReturnNull(error: unknown): null;

package/dist/utils.js

@@ -1,6 +1,6 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.logErrorAndReturnNull = exports.translateOptions = exports.ensureArray = void 0;
+exports.translateOptions = exports.ensureArray = void 0;
 function ensureArray(value) {
     if (Array.isArray(value)) {
         return value;
@@ -17,8 +17,3 @@ function translateOptions(options, table) {
     return params;
 }
 exports.translateOptions = translateOptions;
-function logErrorAndReturnNull(error) {
-    console.error(error);
-    return null;
-}
-exports.logErrorAndReturnNull = logErrorAndReturnNull;

package/package.json

@@ -1,11 +1,11 @@
 {
   "name": "e621-client",
-  "version": "1.1.2",
+  "version": "1.1.4",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",
   "scripts": {
     "build": "tsc",
-    "publish": "npm run build & npm publish"
+    "prepublishOnly": "tsc"
   },
   "keywords": [],
   "author": "Tommus",

package/src/client/E621ApiClient.ts

@@ -5,7 +5,7 @@ import {
   GetPostResponse,
   GetPostsResponse,
 } from "../types/responses";
-import { ensureArray, logErrorAndReturnNull, translateOptions } from "../utils";
+import { ensureArray, translateOptions } from "../utils";
 import { E621ApiHeaders } from "./E621ApiHeaders";
 
 export interface E621ApiOptions {
@@ -149,15 +149,20 @@ export class E621ApiClient {
       headers.addUserAgent(this.client);
     }
 
-    const response = await fetch(this.buildUrl(route, params), {
+    const url = this.buildUrl(route, params);
+    const response = await fetch(url, {
       method: "GET",
       headers: headers.addAuthorization(this.credentials).build(),
     });
 
     if (response.ok) {
-      return response.json().catch(logErrorAndReturnNull);
-    } else {
+      return response.json();
+    } else if (response.status === 404) {
       return null;
+    } else {
+      throw new Error(
+        `Request to ${route} failed with status code ${response.status}`,
+      );
     }
   }
 

package/src/utils.ts

@@ -17,8 +17,3 @@ export function translateOptions<Options extends Record<string, string>>(
 
   return params;
 }
-
-export function logErrorAndReturnNull(error: unknown) {
-  console.error(error);
-  return null;
-}