api 6.0.0 → 6.1.1

package/dist/index.js

@@ -70,7 +70,6 @@ var Sdk = /** @class */ (function () {
          * Create dynamic accessors for every operation with a defined operation ID. If an operation
          * does not have an operation ID it can be accessed by its `.method('/path')` accessor instead.
          *
-         * @param spec
          */
         function loadOperations(spec) {
             return Object.entries(spec.getPaths())

package/dist/packageInfo.d.ts

@@ -1,2 +1,2 @@
 export declare const PACKAGE_NAME = "api";
-export declare const PACKAGE_VERSION = "6.0.0";
+export declare const PACKAGE_VERSION = "6.1.1";

package/dist/packageInfo.js

@@ -3,4 +3,4 @@ exports.__esModule = true;
 exports.PACKAGE_VERSION = exports.PACKAGE_NAME = void 0;
 // This file is automatically updated by the build script.
 exports.PACKAGE_NAME = 'api';
-exports.PACKAGE_VERSION = '6.0.0';
+exports.PACKAGE_VERSION = '6.1.1';

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "api",
-  "version": "6.0.0",
+  "version": "6.1.1",
   "description": "Magical SDK generation from an OpenAPI definition 🪄",
   "main": "./dist/index.js",
   "types": "./dist/index.d.ts",
@@ -10,10 +10,11 @@
   "scripts": {
     "build": "tsc",
     "debug:bin": "node -r ts-node/register src/bin.ts",
+    "lint:types": "tsc --noEmit",
     "prebuild": "rm -rf dist/; npm run version",
     "prepack": "npm run build",
-    "test": "nyc mocha $(find test -name '*.test.ts' -not -path '*/smoketest.test.ts')",
-    "test:smoke": "npx mocha test/cli/codegen/languages/typescript/smoketest.test.ts",
+    "test": "vitest --coverage",
+    "test:smoke": "vitest --config=vitest-smoketest.config.ts ",
     "version": "node -p \"'// This file is automatically updated by the build script.\\nexport const PACKAGE_NAME = \\'' + require('./package.json').name + '\\';\\nexport const PACKAGE_VERSION = \\'' + require('./package.json').version + '\\';'\" > src/packageInfo.ts; git add src/packageInfo.ts"
   },
   "repository": {
@@ -63,7 +64,6 @@
     "node-abort-controller": "^3.1.1",
     "oas": "^20.4.0",
     "ora": "^5.4.1",
-    "prettier": "^2.8.3",
     "prompts": "^2.4.2",
     "remove-undefined-objects": "^2.0.2",
     "semver": "^7.3.8",
@@ -74,7 +74,6 @@
   "devDependencies": {
     "@readme/oas-examples": "^5.9.0",
     "@types/caseless": "^0.12.2",
-    "@types/chai": "^4.3.4",
     "@types/find-cache-dir": "^3.2.1",
     "@types/js-yaml": "^4.0.5",
     "@types/lodash.camelcase": "^4.3.7",
@@ -82,31 +81,19 @@
     "@types/lodash.merge": "^4.6.7",
     "@types/lodash.setwith": "^4.3.7",
     "@types/lodash.startcase": "^4.4.7",
-    "@types/mocha": "^10.0.1",
     "@types/prettier": "^2.7.2",
     "@types/prompts": "^2.4.2",
     "@types/semver": "^7.3.13",
-    "@types/sinon-chai": "^3.2.9",
     "@types/ssri": "^7.1.1",
     "@types/validate-npm-package-name": "^4.0.0",
-    "chai": "^4.3.7",
+    "@vitest/coverage-v8": "^0.34.1",
     "fetch-mock": "^9.11.0",
-    "mocha": "^10.1.0",
-    "mock-require": "^3.0.3",
-    "nyc": "^15.1.0",
     "oas-normalize": "^8.3.2",
-    "sinon": "^15.0.0",
-    "sinon-chai": "^3.7.0",
-    "type-fest": "^3.5.4",
+    "type-fest": "^4.2.0",
     "typescript": "^4.9.5",
-    "unique-temp-dir": "^1.0.0"
+    "unique-temp-dir": "^1.0.0",
+    "vitest": "^0.34.1"
   },
   "prettier": "@readme/eslint-config/prettier",
-  "nyc": {
-    "exclude": [
-      "dist/",
-      "test/"
-    ]
-  },
-  "gitHead": "9759db4c067baf3928ad3a9d76b4965658955d78"
+  "gitHead": "e440a28841e753895bfb5315b4d31f679824fa39"
 }

package/src/cache.ts

@@ -16,8 +16,11 @@ type CacheStore = Record<
   string,
   {
     hash: string;
-    path?: string; // Deprecated in v4.5.0 in favor of `hash`.
     original: string | OASDocument;
+    /**
+     * @deprecated Deprecated in v4.5.0 in favor of `hash`.
+     */
+    path?: string;
     title?: string;
     version?: string;
   }

package/src/cli/codegen/index.ts

@@ -9,7 +9,7 @@ export default function codegen(
   language: SupportedLanguages,
   spec: Oas,
   specPath: string,
-  identifier: string
+  identifier: string,
 ): CodeGeneratorLanguage {
   switch (language) {
     case 'js':

package/src/cli/codegen/language.ts

@@ -49,7 +49,7 @@ export default abstract class CodeGeneratorLanguage {
      */
     if (JSON.stringify(spec.api).includes('"$ref":"#/')) {
       throw new Error(
-        'Sorry, this library does not yet support generating an SDK for an OpenAPI definition that contains circular references.'
+        'Sorry, this library does not yet support generating an SDK for an OpenAPI definition that contains circular references.',
       );
     }
   }

package/src/cli/codegen/languages/typescript/util.ts

@@ -1,7 +1,6 @@
 import camelCase from 'lodash.camelcase';
 import deburr from 'lodash.deburr';
 import startCase from 'lodash.startcase';
-import { format as prettier } from 'prettier';
 
 /**
  * This is a mix of reserved JS words and keywords in TypeScript that might be reserved or
@@ -100,14 +99,6 @@ const RESERVED_WORDS = [
   'window',
 ];
 
-export function formatter(content: string) {
-  return prettier(content, {
-    parser: 'typescript',
-    printWidth: 100,
-    singleQuote: true,
-  });
-}
-
 /**
  * @see {@link https://www.30secondsofcode.org/js/s/word-wrap}
  */

package/src/cli/codegen/languages/typescript.ts

@@ -3,7 +3,13 @@ import type { InstallerOptions } from '../language';
 import type Oas from 'oas';
 import type { Operation } from 'oas';
 import type { HttpMethods, SchemaObject } from 'oas/dist/rmoas.types';
-import type { ClassDeclaration, JSDocStructure, OptionalKind, ParameterDeclarationStructure } from 'ts-morph';
+import type {
+  ClassDeclaration,
+  JSDocStructure,
+  JSDocTagStructure,
+  OptionalKind,
+  ParameterDeclarationStructure,
+} from 'ts-morph';
 import type { PackageJson } from 'type-fest';
 
 import fs from 'fs';
@@ -17,19 +23,25 @@ import { IndentationText, Project, QuoteKind, ScriptTarget, VariableDeclarationK
 import logger from '../../logger';
 import CodeGeneratorLanguage from '../language';
 
-import { docblockEscape, formatter, generateTypeName, wordWrap } from './typescript/util';
+import { docblockEscape, generateTypeName, wordWrap } from './typescript/util';
 
 export interface TSGeneratorOptions {
-  outputJS?: boolean;
   compilerTarget?: 'cjs' | 'esm';
+  outputJS?: boolean;
 }
 
 interface OperationTypeHousing {
+  operation: Operation;
   types: {
     params?: false | Record<'body' | 'formData' | 'metadata', string>;
-    responses?: Record<string, string>;
+    responses?: Record<
+      string | number,
+      {
+        description?: string;
+        type: string;
+      }
+    >;
   };
-  operation: Operation;
 }
 
 export default class TSGenerator extends CodeGeneratorLanguage {
@@ -60,7 +72,7 @@ export default class TSGenerator extends CodeGeneratorLanguage {
   usesHTTPMethodRangeInterface = false;
 
   constructor(spec: Oas, specPath: string, identifier: string, opts: TSGeneratorOptions = {}) {
-    const options: { outputJS: boolean; compilerTarget: 'cjs' | 'esm' } = {
+    const options: { compilerTarget: 'cjs' | 'esm'; outputJS: boolean } = {
       outputJS: false,
       compilerTarget: 'cjs',
       ...opts,
@@ -233,7 +245,7 @@ export default class TSGenerator extends CodeGeneratorLanguage {
             return {};
           }
 
-          let code = formatter(sourceFile.text);
+          let code = sourceFile.text;
           if (file === 'index.js' && this.compilerTarget === 'cjs') {
             /**
              * There's an annoying quirk with `ts-morph` where if we're exporting a default export
@@ -259,7 +271,7 @@ export default class TSGenerator extends CodeGeneratorLanguage {
 
     return [
       ...this.project.getSourceFiles().map(sourceFile => ({
-        [sourceFile.getBaseName()]: formatter(sourceFile.getFullText()),
+        [sourceFile.getBaseName()]: sourceFile.getFullText(),
       })),
 
       // Because we're returning the raw source files for TS generation we also need to separately
@@ -269,7 +281,7 @@ export default class TSGenerator extends CodeGeneratorLanguage {
         .emitToMemory({ emitOnlyDtsFiles: true })
         .getFiles()
         .map(sourceFile => ({
-          [path.basename(sourceFile.filePath)]: formatter(sourceFile.text),
+          [path.basename(sourceFile.filePath)]: sourceFile.text,
         })),
     ].reduce((prev, next) => Object.assign(prev, next));
   }
@@ -328,7 +340,7 @@ export default class TSGenerator extends CodeGeneratorLanguage {
               {
                 tagName: 'param',
                 text: wordWrap(
-                  'config.timeout Override the default `fetch` request timeout of 30 seconds. This number should be represented in milliseconds.'
+                  'config.timeout Override the default `fetch` request timeout of 30 seconds. This number should be represented in milliseconds.',
                 ),
               },
             ],
@@ -358,7 +370,7 @@ sdk.auth('username', 'password');
 sdk.auth('myBearerToken');
 
 @example <caption>API Keys</caption>
-sdk.auth('myApiKey');`)
+sdk.auth('myApiKey');`),
               ),
             tags: [
               { tagName: 'see', text: '{@link https://spec.openapis.org/oas/v3.0.3#fixed-fields-22}' },
@@ -391,7 +403,7 @@ sdk.server('https://{region}.api.example.com/{basePath}', {
 });
 
 @example <caption>Fully qualified server URL</caption>
-sdk.server('https://eu.api.example.com/v14');`)
+sdk.server('https://eu.api.example.com/v14');`),
               ),
             tags: [
               { tagName: 'param', text: 'url Server URL' },
@@ -502,6 +514,20 @@ sdk.server('https://eu.api.example.com/v14');`)
     return sourceFile;
   }
 
+  /**
+   * Add a new JSDoc `@tag` to an existing docblock.
+   *
+   */
+  static addTagToDocblock(docblock: OptionalKind<JSDocStructure>, tag: OptionalKind<JSDocTagStructure>) {
+    const tags = docblock.tags ?? [];
+    tags.push(tag);
+
+    return {
+      ...docblock,
+      tags,
+    };
+  }
+
   /**
    * Create operation accessors on the SDK.
    *
@@ -510,9 +536,9 @@ sdk.server('https://eu.api.example.com/v14');`)
     operation: Operation,
     operationId: string,
     paramTypes?: OperationTypeHousing['types']['params'],
-    responseTypes?: OperationTypeHousing['types']['responses']
+    responseTypes?: OperationTypeHousing['types']['responses'],
   ) {
-    const docblock: OptionalKind<JSDocStructure> = {};
+    let docblock: OptionalKind<JSDocStructure> = {};
     const summary = operation.getSummary();
     const description = operation.getDescription();
     if (summary || description) {
@@ -531,7 +557,10 @@ sdk.server('https://eu.api.example.com/v14');`)
       };
 
       if (summary && description) {
-        docblock.tags = [{ tagName: 'summary', text: docblockEscape(wordWrap(summary)) }];
+        docblock = TSGenerator.addTagToDocblock(docblock, {
+          tagName: 'summary',
+          text: docblockEscape(wordWrap(summary)),
+        });
       }
     }
 
@@ -568,8 +597,8 @@ sdk.server('https://eu.api.example.com/v14');`)
 
     let returnType = 'Promise<FetchResponse<number, unknown>>';
     if (responseTypes) {
-      returnType = `Promise<${Object.entries(responseTypes)
-        .map(([status, responseType]) => {
+      const returnTypes = Object.entries(responseTypes)
+        .map(([status, { description: responseDescription, type: responseType }]) => {
           if (status.toLowerCase() === 'default') {
             return `FetchResponse<number, ${responseType}>`;
           } else if (status.length === 3 && status.toUpperCase().endsWith('XX')) {
@@ -580,19 +609,54 @@ sdk.server('https://eu.api.example.com/v14');`)
               return `FetchResponse<number, ${responseType}>`;
             }
 
+            if (Number(statusPrefix) >= 4) {
+              docblock = TSGenerator.addTagToDocblock(docblock, {
+                tagName: 'throws',
+                text: `FetchError<${status}, ${responseType}>${
+                  responseDescription ? docblockEscape(wordWrap(` ${responseDescription}`)) : ''
+                }`,
+              });
+
+              return false;
+            }
+
             this.usesHTTPMethodRangeInterface = true;
             return `FetchResponse<HTTPMethodRange<${statusPrefix}00, ${statusPrefix}99>, ${responseType}>`;
           }
 
+          // 400 and 500 status code families are thrown as exceptions so adding them as a possible
+          // return type isn't valid.
+          if (Number(status) >= 400) {
+            docblock = TSGenerator.addTagToDocblock(docblock, {
+              tagName: 'throws',
+              text: `FetchError<${status}, ${responseType}>${
+                responseDescription ? docblockEscape(wordWrap(` ${responseDescription}`)) : ''
+              }`,
+            });
+
+            return false;
+          }
+
           return `FetchResponse<${status}, ${responseType}>`;
         })
-        .join(' | ')}>`;
+        .filter(Boolean)
+        .join(' | ');
+
+      // If all of our documented responses are for error status codes then all we can document for
+      // anything else that might happen is `unknown`.
+      returnType = `Promise<${returnTypes.length ? returnTypes : 'FetchResponse<number, unknown>'}>`;
     }
 
+    const shouldAddAltTypedOverloads = Object.keys(parameters).length === 2 && hasOptionalBody && !hasOptionalMetadata;
     const operationIdAccessor = this.sdk.addMethod({
       name: operationId,
       returnType,
-      docs: Object.keys(docblock).length ? [docblock] : null,
+
+      // If we're going to be creating typed method overloads for optional body an metadata handling
+      // we should only add a docblock to the first overload we create because IDE Intellisense will
+      // always use that and adding a docblock to all three will bloat the SDK with unused and
+      // unsurfaced method documentation.
+      docs: shouldAddAltTypedOverloads ? null : Object.keys(docblock).length ? [docblock] : null,
       statements: writer => {
         /**
          * @example return this.core.fetch('/pet/findByStatus', 'get', body, metadata);
@@ -612,7 +676,7 @@ sdk.server('https://eu.api.example.com/v14');`)
             }
 
             fetchStmt.write(arg.name);
-            if (totalParams > 1 && i !== totalParams) {
+            if (i !== totalParams - 1) {
               fetchStmt.write(', ');
             }
           });
@@ -626,10 +690,6 @@ sdk.server('https://eu.api.example.com/v14');`)
     // If we have both body and metadata parameters but only body is optional we need to create
     // a couple function overloads as Typescript doesn't let us have an optional method parameter
     // come before one that's required.
-    //
-    // None of these accessor overloads will receive a docblock because the original will have
-    // that covered.
-    const shouldAddAltTypedOverloads = Object.keys(parameters).length === 2 && hasOptionalBody && !hasOptionalMetadata;
     if (shouldAddAltTypedOverloads) {
       // Create an overload that has both `body` and `metadata` parameters as required.
       operationIdAccessor.addOverload({
@@ -645,7 +705,6 @@ sdk.server('https://eu.api.example.com/v14');`)
       operationIdAccessor.addOverload({
         parameters: [{ ...parameters.metadata }],
         returnType,
-        docs: Object.keys(docblock).length ? [docblock] : null,
       });
 
       // Create an overload that has both `body` and `metadata` parameters as optional. Even though
@@ -807,7 +866,7 @@ sdk.server('https://eu.api.example.com/v14');`)
       .reduce((prev, next) => Object.assign(prev, next));
 
     const res = Object.entries(schemas)
-      .map(([status, { schema }]) => {
+      .map(([status, { description, schema }]) => {
         let typeName;
 
         if (typeof schema === 'string' && schema.startsWith('::convert::')) {
@@ -826,7 +885,10 @@ sdk.server('https://eu.api.example.com/v14');`)
         return {
           // Types are prefixed with `types.` because that's how we're importing them from
           // `types.d.ts`.
-          [status]: `types.${typeName}`,
+          [status]: {
+            type: `types.${typeName}`,
+            description,
+          },
         };
       })
       .reduce((prev, next) => Object.assign(prev, next), {});

package/src/cli/commands/install.ts

@@ -4,7 +4,6 @@ import { Command, Option } from 'commander';
 import figures from 'figures';
 import Oas from 'oas';
 import ora from 'ora';
-import validateNPMPackageName from 'validate-npm-package-name';
 
 import Fetcher from '../../fetcher';
 import codegen from '../codegen';
@@ -18,16 +17,17 @@ cmd
   .name('install')
   .description('install an API SDK into your codebase')
   .argument('<uri>', 'an API to install')
+  .option('-i, --identifier <identifier>', 'API identifier (eg. `@api/petstore`)')
   .addOption(
     new Option('-l, --lang <language>', 'SDK language').choices([
       'js', // User generally wants JS, we'll prompt if they want CJS or ESM files.
       'js-cjs',
       'js-esm',
       'ts',
-    ])
+    ]),
   )
   .addOption(new Option('-y, --yes', 'Automatically answer "yes" to any prompts printed'))
-  .action(async (uri: string, options: { lang: string; yes?: boolean }) => {
+  .action(async (uri: string, options: { identifier?: string; lang: string; yes?: boolean }) => {
     let language: SupportedLanguages;
     if (options.lang) {
       language = options.lang as SupportedLanguages;
@@ -69,7 +69,12 @@ cmd
     }
 
     let identifier;
-    if (Fetcher.isAPIRegistryUUID(uri)) {
+    if (options.identifier) {
+      // `Storage.isIdentifierValid` will throw an exception if an identifier is invalid.
+      if (Storage.isIdentifierValid(options.identifier)) {
+        identifier = options.identifier;
+      }
+    } else if (Fetcher.isAPIRegistryUUID(uri)) {
       identifier = Fetcher.getProjectPrefixFromRegistryUUID(uri);
     } else {
       ({ value: identifier } = await promptTerminal({
@@ -82,19 +87,11 @@ cmd
             return false;
           }
 
-          // Is this identifier already in storage?
-          if (Storage.isInLockFile({ identifier: value })) {
-            return `"${value}" is already taken in your \`.api/\` directory. Please enter another identifier.`;
-          }
-
-          const isValidForNPM = validateNPMPackageName(`@api/${value}`);
-          if (!isValidForNPM.validForNewPackages) {
-            // `prompts` doesn't support surfacing multiple errors in a `validate` call so we can
-            // only surface the first to the user.
-            return isValidForNPM.errors[0];
+          try {
+            return Storage.isIdentifierValid(value, true);
+          } catch (err) {
+            return err.message;
           }
-
-          return true;
         },
       }));
     }
@@ -191,7 +188,7 @@ cmd
 Examples:
   $ api install @developers/v2.0#nysezql0wwo236
   $ api install https://raw.githubusercontent.com/readmeio/oas-examples/main/3.0/json/petstore-simple.json
-  $ api install ./petstore.json`
+  $ api install ./petstore.json`,
   );
 
 export default cmd;

package/src/cli/lib/prompt.ts

@@ -9,7 +9,7 @@ import prompts from 'prompts';
  */
 export default async function promptTerminal<T extends string = string>(
   question: prompts.PromptObject<T>,
-  options?: prompts.Options
+  options?: prompts.Options,
 ) {
   const enableTerminalCursor = () => {
     process.stdout.write('\x1B[?25h');

package/src/cli/storage.ts

@@ -5,6 +5,7 @@ import path from 'path';
 
 import makeDir from 'make-dir';
 import ssri from 'ssri';
+import validateNPMPackageName from 'validate-npm-package-name';
 
 import Fetcher from '../fetcher';
 import { PACKAGE_VERSION } from '../packageInfo';
@@ -105,6 +106,22 @@ export default class Storage {
     return Storage.lockfile;
   }
 
+  static isIdentifierValid(identifier: string, prefixWithAPINamespace?: boolean) {
+    // Is this identifier already in storage?
+    if (Storage.isInLockFile({ identifier })) {
+      throw new Error(`"${identifier}" is already taken in your \`.api/\` directory. Please try another identifier.`);
+    }
+
+    const isValidForNPM = validateNPMPackageName(prefixWithAPINamespace ? `@api/${identifier}` : identifier);
+    if (!isValidForNPM.validForNewPackages) {
+      // `prompts` doesn't support surfacing multiple errors in a `validate` call so we can only
+      // surface the first to the user.
+      throw new Error(`Identifier cannot be used for an NPM package: ${isValidForNPM.errors[0]}`);
+    }
+
+    return true;
+  }
+
   static isInLockFile(search: { identifier?: string; source?: string }) {
     // Because this method may run before we initialize a new storage object we should make sure
     // that we have a storage directory present.
@@ -207,7 +224,6 @@ export default class Storage {
    *         ├── openapi.json
    *         └── package.json
    *
-   * @param spec
    */
   save(spec: OASDocument) {
     if (!this.identifier) {
@@ -254,12 +270,13 @@ export default class Storage {
 }
 
 export interface Lockfile {
+  apis: LockfileAPI[];
+
   /**
    * The `api.json` schema version. This will only ever change if we introduce breaking changes to
    * this store.
    */
   version: '1.0';
-  apis: LockfileAPI[];
 }
 
 export interface LockfileAPI {
@@ -272,13 +289,11 @@ export interface LockfileAPI {
   identifier: string;
 
   /**
-   * The original source that was used to generate the SDK with.
+   * The version of `api` that was used to install this SDK.
    *
-   * @example https://raw.githubusercontent.com/readmeio/oas-examples/main/3.0/json/petstore-simple.json
-   * @example ./petstore.json
-   * @example @developers/v2.0#nysezql0wwo236
+   * @example 5.0.0
    */
-  source: string;
+  installerVersion: string;
 
   /**
    * An integrity hash that will be used to determine on `npx api update` calls if the API has
@@ -289,9 +304,11 @@ export interface LockfileAPI {
   integrity: string;
 
   /**
-   * The version of `api` that was used to install this SDK.
+   * The original source that was used to generate the SDK with.
    *
-   * @example 5.0.0
+   * @example https://raw.githubusercontent.com/readmeio/oas-examples/main/3.0/json/petstore-simple.json
+   * @example ./petstore.json
+   * @example @developers/v2.0#nysezql0wwo236
    */
-  installerVersion: string;
+  source: string;
 }

package/src/core/errors/fetchError.ts

@@ -1,9 +1,9 @@
-class FetchError extends Error {
+class FetchError<Status = number, Data = unknown> extends Error {
   /** HTTP Status */
-  status: number;
+  status: Status;
 
   /** The content of the response. */
-  data: unknown;
+  data: Data;
 
   /** The Headers of the response. */
   headers: Headers;
@@ -11,7 +11,7 @@ class FetchError extends Error {
   /** The raw `Response` object. */
   res: Response;
 
-  constructor(status: number, data: unknown, headers: Headers, res: Response) {
+  constructor(status: Status, data: Data, headers: Headers, res: Response) {
     super(res.statusText);
 
     this.name = 'FetchError';

package/src/core/getJSONSchemaDefaults.ts

@@ -12,7 +12,6 @@ import traverse from 'json-schema-traverse';
  *
  * @todo This is a good candidate to be moved into a core `oas` library method.
  * @see {@link https://github.com/mdornseif/json-schema-default}
- * @param jsonSchemas
  */
 export default function getJSONSchemaDefaults(jsonSchemas: SchemaWrapper[]) {
   return jsonSchemas
@@ -27,7 +26,7 @@ export default function getJSONSchemaDefaults(jsonSchemas: SchemaWrapper[]) {
           parentPointer: string,
           parentKeyword: string,
           parentSchema: SchemaObject,
-          indexProperty: string
+          indexProperty: string,
         ) => {
           if (!pointer.startsWith('/properties/')) {
             return;
@@ -59,7 +58,7 @@ export default function getJSONSchemaDefaults(jsonSchemas: SchemaWrapper[]) {
               }
             }
           }
-        }
+        },
       );
 
       if (!Object.keys(defaults).length) {

package/src/core/index.ts

@@ -26,9 +26,9 @@ export interface ConfigOptions {
 
 export interface FetchResponse<status, data> {
   data: data;
-  status: status;
   headers: Headers;
   res: Response;
+  status: status;
 }
 
 // https://stackoverflow.com/a/39495173
@@ -128,7 +128,12 @@ export default class APICore {
           const parsed = await parseResponse(res);
 
           if (res.status >= 400 && res.status <= 599) {
-            throw new FetchError(parsed.status, parsed.data, parsed.headers, parsed.res);
+            throw new FetchError<typeof parsed.status, typeof parsed.data>(
+              parsed.status,
+              parsed.data,
+              parsed.headers,
+              parsed.res,
+            );
           }
 
           return parsed;