Skip private members with decorators

README.md

@@ -8,6 +8,7 @@ A TypeScript custom transformer which minify names of private class members.
 For now it just renames private members with prepending some prefix to name.
 For example, if you have `privateMember`, then after transformation the name will be `_private_privateMember`.
 After that you can use terser/uglify with mangle options to minify that members.
+Note, that private class members with decorators won't be prefixed and further minified.
 
 ## Caution!!!
 

src/properties-minifier.ts

@@ -190,6 +190,11 @@ function isPrivateNonStaticClassMember(symbol: ts.Symbol | undefined): boolean {
 	}
 
 	return symbol.declarations.some((x: ts.Declaration) => {
-		return (isClassMember(x) || isConstructorParameter(x)) && isPrivateNonStatic(x);
+		// terser / uglify property minifiers aren't able to handle decorators
+		return (isClassMember(x) && !hasDecorators(x) || isConstructorParameter(x)) && isPrivateNonStatic(x);
 	});
 }
+
+function hasDecorators(node: ts.Node): boolean {
+	return !!node.decorators;
+}

tests/functional-test-cases.ts

@@ -75,6 +75,7 @@ describe('Functional tests', () => {
 				rootNames: [testCase.inputFileName],
 				options: {
 					target: ts.ScriptTarget.ES5,
+					experimentalDecorators: true,
 				},
 			});
 

tests/test-cases/private-with-decorators/input.ts

@@ -0,0 +1,17 @@
+export class Class {
+	public publicField: number = 123;
+	//@ts-ignore
+	@decorator private privateField: string = 'string-value';
+
+	public constructor() {
+		this.privateMethod(this.privateField);
+		this.privateMethod(this.publicField);
+
+		this['privateMethod'](this.privateField);
+	}
+
+	//@ts-ignore
+	@decorator private privateMethod(a: string | number): void { }
+}
+
+function decorator(target: any, propertyKey: string): void {}

tests/test-cases/private-with-decorators/output.js

@@ -0,0 +1,29 @@
+"use strict";
+var __decorate = (this && this.__decorate) || function (decorators, target, key, desc) {
+    var c = arguments.length, r = c < 3 ? target : desc === null ? desc = Object.getOwnPropertyDescriptor(target, key) : desc, d;
+    if (typeof Reflect === "object" && typeof Reflect.decorate === "function") r = Reflect.decorate(decorators, target, key, desc);
+    else for (var i = decorators.length - 1; i >= 0; i--) if (d = decorators[i]) r = (c < 3 ? d(r) : c > 3 ? d(target, key, r) : d(target, key)) || r;
+    return c > 3 && r && Object.defineProperty(target, key, r), r;
+};
+Object.defineProperty(exports, "__esModule", { value: true });
+var Class = /** @class */ (function () {
+    function Class() {
+        this.publicField = 123;
+        //@ts-ignore
+        this.privateField = 'string-value';
+        this.privateMethod(this.privateField);
+        this.privateMethod(this.publicField);
+        this['privateMethod'](this.privateField);
+    }
+    //@ts-ignore
+    Class.prototype.privateMethod = function (a) { };
+    __decorate([
+        decorator
+    ], Class.prototype, "privateField", void 0);
+    __decorate([
+        decorator
+    ], Class.prototype, "privateMethod", null);
+    return Class;
+}());
+exports.Class = Class;
+function decorator(target, propertyKey) { }