refactor(complex-types): avoid passing isPropertyBlacklisted to every functions

Author: so1ve

packages/complex-types/src/core/printer.ts

@@ -3,6 +3,8 @@ import ts from "typescript";
 import { escapeQuotes } from "./utils";
 
 export class Printer {
+	private isPropertyBlacklisted: (prop: string) => boolean = () => false;
+
 	constructor(private checker: ts.TypeChecker) {}
 
 	private typeToString(type: ts.Type): string {
@@ -16,14 +18,11 @@ export class Printer {
 	private printUnionOrIntersection(
 		type: ts.UnionOrIntersectionType,
 		separator: string,
-		isPropertyBlacklisted: (prop: string) => boolean,
 		inner: boolean,
 	): string {
 		return [
 			...new Set(
-				type.types
-					.map((t) => this.printType(t, isPropertyBlacklisted, inner))
-					.filter(Boolean),
+				type.types.map((t) => this.printType(t, inner)).filter(Boolean),
 			),
 		].join(separator);
 	}
@@ -32,17 +31,13 @@ export class Printer {
 		return !!(symbol.flags & ts.SymbolFlags.Optional);
 	}
 
-	private printConditionType(
-		type: ts.ConditionalType,
-		isPropertyBlacklisted: (prop: string) => boolean,
-		inner: boolean,
-	): string {
+	private printConditionType(type: ts.ConditionalType, inner: boolean): string {
 		const decl = type.root.node;
 		const { trueType, falseType } = decl;
 		const trueTypeNode = this.checker.getTypeAtLocation(trueType);
 		const falseTypeNode = this.checker.getTypeAtLocation(falseType);
 
-		return `${this.printType(trueTypeNode, isPropertyBlacklisted, inner)} | ${this.printType(falseTypeNode, isPropertyBlacklisted, inner)}`;
+		return `${this.printType(trueTypeNode, inner)} | ${this.printType(falseTypeNode, inner)}`;
 	}
 
 	private printPrimitiveType(type: ts.Type): string {
@@ -65,25 +60,11 @@ export class Printer {
 		return "";
 	}
 
-	private printType(
-		type: ts.Type,
-		isPropertyBlacklisted: (prop: string) => boolean,
-		inner = false,
-	): string {
+	private printType(type: ts.Type, inner = false): string {
 		if (type.isUnion()) {
-			return this.printUnionOrIntersection(
-				type,
-				" | ",
-				isPropertyBlacklisted,
-				inner,
-			);
+			return this.printUnionOrIntersection(type, " | ", inner);
 		} else if (type.isIntersection()) {
-			return this.printUnionOrIntersection(
-				type,
-				" & ",
-				isPropertyBlacklisted,
-				inner,
-			);
+			return this.printUnionOrIntersection(type, " & ", inner);
 		}
 		if (this.checker.isArrayType(type)) {
 			return "Array";
@@ -108,12 +89,12 @@ export class Printer {
 			for (const prop of properties) {
 				const propType = this.checker.getTypeOfSymbol(prop);
 				const name = prop.getName();
-				if (isPropertyBlacklisted(name)) {
+				if (this.isPropertyBlacklisted(name)) {
 					continue;
 				}
 
 				props[name] = {
-					value: this.printType(propType, isPropertyBlacklisted, true),
+					value: this.printType(propType, true),
 					isOptional: this.isSymbolOptional(prop),
 				};
 			}
@@ -139,11 +120,7 @@ export class Printer {
 		} else if (type.flags & ts.TypeFlags.Undefined) {
 			return "";
 		} else if (type.flags & ts.TypeFlags.Conditional) {
-			return this.printConditionType(
-				type as ts.ConditionalType,
-				isPropertyBlacklisted,
-				inner,
-			);
+			return this.printConditionType(type as ts.ConditionalType, inner);
 		} else if (type.isTypeParameter()) {
 			const symbol = type.getSymbol();
 			const decl = symbol?.declarations?.[0];
@@ -156,7 +133,7 @@ export class Printer {
 			}
 			const refType = this.checker.getTypeAtLocation(ref);
 
-			return this.printType(refType, isPropertyBlacklisted, inner);
+			return this.printType(refType, inner);
 		}
 
 		return this.typeToString(type);
@@ -166,9 +143,10 @@ export class Printer {
 		node: ts.Node,
 		isPropertyBlacklisted: (prop: string) => boolean,
 	): string {
+		this.isPropertyBlacklisted = isPropertyBlacklisted;
 		const type = this.checker.getTypeAtLocation(node);
 
-		return this.printType(type, isPropertyBlacklisted);
+		return this.printType(type);
 	}
 
 	private printEventsByCallSignatures(