refactor: use type instead of interface

src/client.test.ts

@@ -57,9 +57,9 @@ describe("creates an instance with options", () => {
 	it("simple json (GET) with generic type", async () => {
 		uri.pathname = "/file/simple.json";
 		const client = fletcher.create({ headers: { foo: "bar" } });
-		interface IFooBar {
+		type IFooBar = {
 			foo: string;
-		}
+		};
 		const actual = await client.json<IFooBar>(uri.href);
 
 		const expected: IFooBar = { foo: "bar" };

src/fletcher.d.ts

@@ -19,48 +19,48 @@ export type FetchOptions = AxiosRequestConfig;
 
 export { type ICookie, type CookieJar };
 
-export interface IProxyConfig {
+export type IProxyConfig = {
 	username?: string;
 	password?: string;
 	host: string;
 	port: number;
 	protocol?: string;
-}
+};
 
 type RequestRedirect = "follow" | "error" | "manual";
 
 type RequestData = Record<string, unknown>;
 
-interface IOnAfterRequestArgs {
+type IOnAfterRequestArgs = {
 	response: AxiosResponse;
-}
+};
 
-export interface IOnAfterRequestFn {
+export type IOnAfterRequestFn = {
 	(args: IOnAfterRequestArgs): Promise<void> | void;
-}
+};
 
-export interface IFletcherBrowserUserOptions {
+export type IFletcherBrowserUserOptions = {
 	blockedResourceTypes: boolean | string[];
 	endpoint: string;
 	onPageReady: (page: Page) => Promise<unknown>;
 	screenshot: ScreenshotOptions;
 	waitForSelector: string;
-}
+};
 
-export interface ICacheParams {
+export type ICacheParams = {
 	format: string;
 	url: string;
 	options?: Partial<IFletcherUserOptions>;
 	payload?: string;
-}
+};
 
-export interface IFletcherCacheMethods {
+export type IFletcherCacheMethods = {
 	hit: (key: string) => null | unknown;
 	write: (key: string, payload?: string) => void;
 	key: (params: ICacheParams) => string;
-}
+};
 
-export interface IFletcherUserOptions {
+export type IFletcherUserOptions = {
 	browser: Partial<IFletcherBrowserUserOptions>;
 	cache: boolean;
 	cacheMethods: Partial<IFletcherCacheMethods>;
@@ -85,9 +85,9 @@ export interface IFletcherUserOptions {
 	urlSearchParams: UrlSearchParams;
 	userAgent: string;
 	validateStatus: (statusCode: number) => boolean;
-}
+};
 
-export interface IFletcherOptions {
+export type IFletcherOptions = {
 	body?: string;
 	cache: boolean;
 	delay: number;
@@ -102,13 +102,13 @@ export interface IFletcherOptions {
 	timeout: number;
 	url: string;
 	validateStatus: (statusCode: number) => boolean;
-}
+};
 
-interface IInstanceMethod<T> {
+type IInstanceMethod<T> = {
 	(url: string, options?: Partial<IFletcherUserOptions>): Promise<T>;
-}
+};
 
-export interface IInstance {
+export type IInstance = {
 	embeddedJson: <T = unknown>(
 		url: string,
 		options?: Partial<IFletcherUserOptions>
@@ -142,12 +142,12 @@ export interface IInstance {
 		) => Promise<T>;
 		jsonld: IInstanceMethod<unknown[]>;
 	};
-}
+};
 
-export interface IResponse {
+export type IResponse = {
 	// body: Readable & Dispatcher.BodyMixin;
 	headers: AxiosResponseHeaders;
 	statusCode: number;
 	statusMessage?: string;
 	text: () => Promise<string>;
-}
+};

src/helpers/async-retry.ts

@@ -1,16 +1,16 @@
 import retrier, { type WrapOptions } from "retry";
 
-export interface IOptions extends WrapOptions {
+export type IOptions = WrapOptions & {
 	onRetry?: ((e: Error, attempt: number) => unknown) | undefined;
-}
+};
 
-interface IErrorWithBail extends Error {
+type IErrorWithBail = Error & {
 	bail?: (error: Error) => void;
-}
+};
 
-interface IAsyncFn<T> {
+type IAsyncFn<T> = {
 	(...args: unknown[]): Promise<T>;
-}
+};
 
 export function retry<T>(fn: IAsyncFn<T>, opts: IOptions): Promise<T> {
 	function run(

src/helpers/cookie-jar.ts

@@ -1,10 +1,10 @@
 /* eslint-disable class-methods-use-this */
 import Cookie from "cookie";
 
-export interface ICookie {
+export type ICookie = {
 	key: string;
 	value: string;
-}
+};
 
 export class CookieJar {
 	cookies: ICookie[] = [];

src/script.test.ts

@@ -24,9 +24,9 @@ describe("inline scripts", () => {
 
 	it("evaluates a script and returns its global scope", async () => {
 		uri.pathname = "/file/inline-script.html";
-		interface IPageData {
+		type IPageData = {
 			pageData: { foo: string };
-		}
+		};
 		const actual = await fletcher.script<IPageData>(uri.href, {
 			scriptPath: "script:nth-of-type(1)",
 		});

src/services/browser.ts

@@ -9,9 +9,9 @@ import { Cache } from "../options/cache";
 import { getJsonLd } from "../options/json-ld";
 import { getScript } from "../options/script";
 
-interface IExecutorFn<T = unknown> {
+type IExecutorFn<T = unknown> = {
 	(page: Page): Promise<T>;
-}
+};
 
 const cache = new Cache();