@putout/printer 2.95.0 → 2.96.0

package/ChangeLog

@@ -1,3 +1,8 @@
+2023.08.03, v2.96.0
+
+feature:
+- 205200b @putout/printer: UpdateExpression: parens (coderaiser/minify#112)
+
 2023.08.02, v2.95.0
 
 feature:

package/lib/tokenize/expressions/assignment-expression.js

@@ -1,15 +1,16 @@
 'use strict';
 
 const {isObjectPattern} = require('@babel/types');
+const {isParens} = require('./unary-expression/parens');
 
 module.exports.AssignmentExpression = {
     condition: (path) => {
-        const {left, extra} = path.node;
+        const {left} = path.node;
         
         if (isObjectPattern(left))
             return true;
         
-        if (extra?.parenthesized)
+        if (isParens(path))
             return true;
         
         return path.parentPath.isLogicalExpression();

package/lib/tokenize/expressions/binary-expression/binary-expression.js

@@ -6,12 +6,11 @@ const {
 } = require('./concatanate');
 
 const {maybeSpace} = require('./maybe-space');
+const {isParens} = require('../unary-expression/parens');
 
 const BinaryExpression = {
     condition(path) {
-        const parens = path.node.extra?.parenthesized;
-        
-        if (parens)
+        if (isParens(path))
             return true;
         
         return path.parentPath.isAwaitExpression();

package/lib/tokenize/expressions/conditional-expression.js

@@ -1,6 +1,6 @@
 'use strict';
 
-const isParens = (path) => path.node.extra?.parenthesized;
+const {isParens} = require('./unary-expression/parens');
 
 module.exports.ConditionalExpression = {
     condition: isParens,

package/lib/tokenize/expressions/index.js

@@ -1,7 +1,7 @@
 'use strict';
 
 const functions = require('./function/functions');
-const unaryExpressions = require('./unary-expressions');
+const unaryExpressions = require('./unary-expression/unary-expressions');
 const memberExpressions = require('./member-expression/member-expressions');
 
 const {

package/lib/tokenize/expressions/unary-expression/parens.js

@@ -0,0 +1,13 @@
+'use strict';
+
+const isParens = (path) => path.node.extra?.parenthesized;
+
+module.exports.isParens = isParens;
+
+module.exports.maybeParenOpen = (path, {maybe}) => {
+    maybe.write(isParens(path), '(');
+};
+
+module.exports.maybeParenClose = (path, {maybe}) => {
+    maybe.write(isParens(path), ')');
+};

package/lib/tokenize/expressions/{unary-expressions.js → unary-expression/unary-expressions.js}

@@ -1,6 +1,10 @@
 'use strict';
 
-const {isLast, isNext} = require('../is');
+const {isLast, isNext} = require('../../is');
+const {
+    maybeParenOpen,
+    maybeParenClose,
+} = require('./parens');
 
 module.exports.UnaryExpression = unaryExpression;
 module.exports.UpdateExpression = unaryExpression;
@@ -39,14 +43,17 @@ function printUnary(path, name, {print}) {
 
 const isWord = (a) => /^(delete|typeof|void|throw)$/.test(a);
 
-function unaryExpression(path, {maybe, traverse}) {
+function unaryExpression(path, printer) {
+    const {maybe, traverse} = printer;
     const {prefix, operator} = path.node;
-    
     const argPath = path.get('argument');
     
+    maybeParenOpen(path, printer);
+    
     maybe.print(prefix, operator);
     maybe.print(isWord(operator), ' ');
-    
     traverse(argPath);
     maybe.print(!prefix, operator);
+    
+    maybeParenClose(path, printer);
 }

package/lib/tokenize/literals/identifier.js

@@ -0,0 +1,35 @@
+'use strict';
+
+const {
+    maybeParenOpen,
+    maybeParenClose,
+} = require('../expressions/unary-expression/parens');
+
+const {maybeDecorators} = require('../maybe-get');
+
+module.exports.Identifier = (path, printer) => {
+    const {
+        write,
+        maybe,
+        traverse,
+        print,
+    } = printer;
+    
+    const {node} = path;
+    const {name, optional} = node;
+    
+    const typeAnnotation = path.get('typeAnnotation');
+    
+    maybeParenOpen(path, printer);
+    
+    for (const decorator of maybeDecorators(path)) {
+        traverse(decorator);
+        print(' ');
+    }
+    
+    write(name);
+    maybe.write(optional, '?');
+    traverse(typeAnnotation);
+    
+    maybeParenClose(path, printer);
+};

package/lib/tokenize/literals/index.js

@@ -1,9 +1,11 @@
 'use strict';
 
 const {TemplateLiteral} = require('./template-literal');
-const {maybeDecorators} = require('../maybe-get');
+
+const {Identifier} = require('./identifier');
 
 module.exports = {
+    Identifier,
     TemplateLiteral,
     BigIntLiteral(path, {write}) {
         write(path.node.raw);
@@ -39,23 +41,6 @@ module.exports = {
         
         write(raw || `'${value}'`);
     },
-    Identifier(path, {write, maybe, traverse, print}) {
-        const {node} = path;
-        const {name, optional} = node;
-        const parenthesized = node.extra?.parenthesized;
-        const typeAnnotation = path.get('typeAnnotation');
-        
-        maybe.write(parenthesized, '(');
-        for (const decorator of maybeDecorators(path)) {
-            traverse(decorator);
-            print(' ');
-        }
-        
-        write(name);
-        maybe.write(optional, '?');
-        traverse(typeAnnotation);
-        maybe.write(parenthesized, ')');
-    },
     RegExpLiteral(path, {print}) {
         print(path.node.raw);
     },

package/lib/tokenize/typescript/index.js

@@ -30,6 +30,11 @@ const {TSCallSignatureDeclaration} = require('./function/ts-call-signature-decla
 const {TSConstructSignatureDeclaration} = require('./function/ts-construct-signature-declaration');
 const {TSMethodSignature} = require('./function/ts-method-signature');
 
+const {
+    maybeParenOpen,
+    maybeParenClose,
+} = require('../expressions/unary-expression/parens');
+
 module.exports = {
     TSAsExpression,
     TSTypeLiteral,
@@ -119,11 +124,13 @@ module.exports = {
     TSBooleanKeyword(path, {write}) {
         write('boolean');
     },
-    TSUnionType(path, {traverse, write, maybe}) {
+    TSUnionType(path, printer) {
+        const {traverse, write} = printer;
+        
         const types = path.get('types');
         const n = types.length - 1;
         
-        maybe.write(path.node.extra?.parenthesized, '(');
+        maybeParenOpen(path, printer);
         
         for (const [i, type] of types.entries()) {
             traverse(type);
@@ -135,7 +142,7 @@ module.exports = {
             }
         }
         
-        maybe.write(path.node.extra?.parenthesized, ')');
+        maybeParenClose(path, printer);
     },
     TSNumberKeyword(path, {write}) {
         write('number');

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@putout/printer",
-  "version": "2.95.0",
+  "version": "2.96.0",
   "type": "commonjs",
   "author": "coderaiser <mnemonic.enemy@gmail.com> (https://github.com/coderaiser)",
   "description": "Simplest possible opinionated Babel AST printer for 🐊Putout",