refactor: replace type-aliases with interfaces

Author: raveclassic

src/fileReader.ts

@@ -1,8 +1,6 @@
 import * as yaml from 'js-yaml';
 
-export type JSON = {
-	[key: string]: unknown;
-};
+export interface JSON extends Record<string, unknown> {}
 export type FileReader = (buffer: Buffer) => JSON;
 
 export const fromJSON: FileReader = buffer => JSON.parse(buffer.toString());

src/fs.ts

@@ -1,23 +1,23 @@
 import * as fs from 'fs-extra';
 import * as path from 'path';
 
-export type File = {
+export interface File {
 	type: 'FILE';
 	name: string;
 	content: string;
-};
+}
 
 export const file = (name: string, content: string): File => ({
 	type: 'FILE',
 	name,
 	content,
 });
 
-export type Directory = {
+export interface Directory {
 	type: 'DIRECTORY';
 	name: string;
 	content: FSEntity[];
-};
+}
 
 export const directory = (name: string, content: FSEntity[]): Directory => ({
 	type: 'DIRECTORY',
@@ -27,10 +27,10 @@ export const directory = (name: string, content: FSEntity[]): Directory => ({
 
 export type FSEntity = File | Directory;
 
-export type BufferWithName = {
+export interface BufferWithName {
 	buffer: Buffer;
 	fileName: string;
-};
+}
 
 export const write = async (destination: string, entity: FSEntity): Promise<void> => {
 	switch (entity.type) {

src/index.ts

@@ -15,7 +15,7 @@ import { PathReporter } from 'io-ts/lib/PathReporter';
 
 const log = console.log.bind(console, '[SWAGGER-CODEGEN-TS]:');
 
-export type GenerateOptions = {
+export interface GenerateOptions {
 	/**
 	 * Paths to spec files
 	 */
@@ -37,7 +37,7 @@ export type GenerateOptions = {
 	 * @param buffer - File Buffer
 	 */
 	fileReader: FileReader;
-};
+}
 
 const cwd = process.cwd();
 const resolvePath = (p: string) => (path.isAbsolute(p) ? p : path.resolve(cwd, p));

src/language/typescript.ts

@@ -58,26 +58,28 @@ const concatIf = <A>(condition: boolean, as: A[], a: A[]): A[] => concatIfL(cond
 const unless = (condition: boolean, a: string): string => (condition ? '' : a);
 const when = (condition: boolean, a: string): string => (condition ? a : '');
 
-type Dependency = {
+interface Dependency {
 	name: string;
 	path: string;
-};
-type SerializedType = {
+}
+
+interface SerializedType {
 	type: string;
 	io: string;
 	dependencies: Dependency[];
 	refs: string[];
-};
+}
+
 const serializedType = (type: string, io: string, dependencies: Dependency[], refs: string[]): SerializedType => ({
 	type,
 	io,
 	dependencies,
 	refs,
 });
 
-type SerializedParameter = SerializedType & {
+interface SerializedParameter extends SerializedType {
 	isRequired: boolean;
-};
+}
 const serializedParameter = (
 	type: string,
 	io: string,
@@ -91,9 +93,9 @@ const serializedParameter = (
 	dependencies,
 	refs,
 });
-type SerializedPathParameter = SerializedParameter & {
+interface SerializedPathParameter extends SerializedParameter {
 	name: string;
-};
+}
 const serializedPathParameter = (
 	name: string,
 	type: string,
@@ -217,9 +219,9 @@ const serializePathGroup = (name: string, group: Record<string, PathItemObject>,
 		`
 			${dependencies}
 		
-			export type ${groupName} = {
+			export interface ${groupName} {
 				${serialized.type}
-			};
+			}
 			
 			export const ${decapitalize(groupName)} = asks((e: { apiClient: APIClient }): ${groupName} => ({
 				${serialized.io}
@@ -771,17 +773,17 @@ const client = `
 	import { report } from '../utils/utils';
 	import { left } from 'fp-ts/lib/Either';
 
-	export type APIRequest = {
+	export interface APIRequest {
 		url: string;
 		query?: object;
 		body?: unknown;
 	};
 
-	export type FullAPIRequest = APIRequest & {
+	export interface FullAPIRequest extends APIRequest {
 		method: 'GET' | 'POST' | 'PUT' | 'DELETE' | 'PATCH' | 'HEAD' | 'OPTIONS';
 	};
 	
-	export type APIClient = {
+	export interface APIClient {
 		readonly request: (request: FullAPIRequest) => LiveData<Error, mixed>;
 	};