@putout/printer 8.23.0 → 8.25.0

package/ChangeLog

@@ -1,3 +1,13 @@
+2024.05.01, v8.25.0
+
+feature:
+- 095a130 @putout/printer: ImportDeclaration: maxPropertiesLengthInOneLine
+
+2024.04.30, v8.24.0
+
+feature:
+- dd056f5 @putout/printer: semantics: add maxPropertiesLengthInOneLine to align with eslint-plugin-putout: long-properties-destructuring (https://github.com/coderaiser/putout/tree/master/packages/eslint-plugin-putout/lib/long-properties-destructuring#readme)
+
 2024.04.30, v8.23.0
 
 feature:

package/README.md

@@ -98,6 +98,7 @@ print(ast, {
         maxElementsInOneLine: 3,
         maxVariablesInOneLine: 4,
         maxPropertiesInOneLine: 2,
+        maxPropertiesLengthInOneLine: 15,
         trailingComma: true,
         encodeSingleQuote: true,
         encodeDoubleQuote: false,
@@ -172,6 +173,7 @@ Options used to configure logic of output, similar to ESLint rules:
 - ✅ `maxElementsInOneLine` - count of `ArrayExpression` and `ArrayPattern` elements placed in one line.
 - ✅ `maxVariablesInOneLine` - count of `VariableDeclarators` in one line.
 - ✅ `maxPropertiesInOneLine` - count of `ObjectProperties` in one line.
+- ✅ `maxPropertiesLengthInOneLine` - maximum length of `Object Property`, when violated splits event if `maxPropertiesInOneLine` satisfies;
 - ✅ `roundBraces` - to output braces in a  single argument arrow function expressions: `(a) => {}` or not `a => {}`.
 
 ## Visitors API

package/lib/tokenize/expressions/object-pattern/{max-properties-in-one-line.js → more-then-max-properties-in-one-line.js}

@@ -1,6 +1,6 @@
 'use strict';
 
-module.exports.checkMaxPropertiesInOneLine = (path, {maxPropertiesInOneLine}) => {
+module.exports.moreThenMaxPropertiesInOneLine = (path, {maxPropertiesInOneLine}) => {
     const {parentPath} = path;
     
     if (parentPath.isObjectProperty())

package/lib/tokenize/expressions/object-pattern/more-then-max-properties-length-in-one-line.js

@@ -0,0 +1,19 @@
+'use strict';
+
+const {isIdentifier} = require('@putout/babel').types;
+
+module.exports.moreThenMaxPropertiesLengthInOneLine = (path, {maxPropertiesLengthInOneLine}) => {
+    const {properties} = path.node;
+    
+    for (const {key} of properties) {
+        if (!isIdentifier(key))
+            continue;
+        
+        const {name} = key;
+        
+        if (name.length >= maxPropertiesLengthInOneLine)
+            return true;
+    }
+    
+    return false;
+};

package/lib/tokenize/expressions/object-pattern/object-pattern.js

@@ -14,11 +14,12 @@ const {
     exists,
 } = require('../../is');
 
-const {checkMaxPropertiesInOneLine} = require('./max-properties-in-one-line');
+const {moreThenMaxPropertiesInOneLine} = require('./more-then-max-properties-in-one-line');
+
 const {maybeTypeAnnotation} = require('../../maybe/maybe-type-annotation');
+const {moreThenMaxPropertiesLengthInOneLine} = require('./more-then-max-properties-length-in-one-line');
 
 const isTwoLevelsDeep = ({parentPath}) => parentPath.parentPath.parentPath.isObjectProperty();
-
 const isOneParentProperty = ({parentPath}) => parentPath.parentPath.node.properties?.length === 1;
 
 function isIndent(path) {
@@ -27,8 +28,11 @@ function isIndent(path) {
 
 module.exports.ObjectPattern = {
     print: maybeTypeAnnotation((path, {print, maybe}, semantics) => {
-        const {maxPropertiesInOneLine} = semantics;
         const shouldIndent = isIndent(path);
+        const {
+            maxPropertiesInOneLine,
+            maxPropertiesLengthInOneLine,
+        } = semantics;
         
         maybe.indent.inc(shouldIndent);
         print('{');
@@ -38,6 +42,7 @@ module.exports.ObjectPattern = {
         
         const is = shouldAddNewline(path, {
             maxPropertiesInOneLine,
+            maxPropertiesLengthInOneLine,
         });
         
         const hasObject = n && hasObjectPattern(properties);
@@ -150,12 +155,24 @@ function hasObjectPattern(properties) {
 const ONE_LINE = false;
 const COUPLE_LINES = true;
 
-function shouldAddNewline(path, {maxPropertiesInOneLine}) {
+function shouldAddNewline(path, semantics) {
     const {parentPath} = path;
     const properties = path.get('properties');
     const n = properties.length - 1;
     
-    if (checkMaxPropertiesInOneLine(path, {maxPropertiesInOneLine}))
+    const {
+        maxPropertiesInOneLine,
+        maxPropertiesLengthInOneLine,
+    } = semantics;
+    
+    const moreLength = moreThenMaxPropertiesLengthInOneLine(path, {
+        maxPropertiesLengthInOneLine,
+    });
+    const moreCount = moreThenMaxPropertiesInOneLine(path, {
+        maxPropertiesInOneLine,
+    });
+    
+    if (moreCount && !moreLength)
         return ONE_LINE;
     
     if (!isForOf(path) && !path.parentPath.isFunction() && n && checkLength(properties))

package/lib/tokenize/overrides.js

@@ -30,6 +30,7 @@ function initSemantics(semantics = {}) {
     return {
         comments: true,
         maxPropertiesInOneLine: 2,
+        maxPropertiesLengthInOneLine: 15,
         maxSpecifiersInOneLine: 2,
         maxElementsInOneLine: 5,
         maxVariablesInOneLine: 4,

package/lib/tokenize/statements/import-declaration/import-declaration.js

@@ -12,7 +12,15 @@ const {
 
 module.exports.ImportAttribute = ImportAttribute;
 module.exports.ImportDeclaration = {
-    print(path, {print, maybe, write, traverse, indent}, options) {
+    print(path, printer, semantics) {
+        const {
+            print,
+            maybe,
+            write,
+            traverse,
+            indent,
+        } = printer;
+        
         const {phase} = path.node;
         const isType = path.node.importKind === 'type';
         const specifiers = path.get('specifiers');
@@ -44,7 +52,7 @@ module.exports.ImportDeclaration = {
             traverse(spec.get('local'));
         }
         
-        const maxSpecifiersInOneLine = parseMaxSpecifiers(imports, options);
+        const maxSpecifiersInOneLine = parseMaxSpecifiers(imports, semantics);
         const importsCount = imports.length - 1;
         
         for (const [index, spec] of imports.entries()) {
@@ -75,9 +83,9 @@ module.exports.ImportDeclaration = {
                 maybe.write(n, ',');
                 
                 const last = index === n;
-                const penult = index === n - 1;
+                const penulty = index === n - 1;
                 
-                maybe.write.newline(penult && defaults.length);
+                maybe.write.newline(penulty && defaults.length);
                 maybe.write.newline(last);
             }
             
@@ -110,12 +118,18 @@ module.exports.ImportDeclaration = {
     },
 };
 
-function parseMaxSpecifiers(imports, options) {
-    const {maxSpecifiersInOneLine} = options;
+function parseMaxSpecifiers(imports, semantics) {
+    const {
+        maxSpecifiersInOneLine,
+        maxPropertiesLengthInOneLine,
+    } = semantics;
     
     for (const {node} of imports) {
         if (node.imported.name !== node.local.name)
             return 1;
+        
+        if (node.imported.name.length >= maxPropertiesLengthInOneLine)
+            return 1;
     }
     
     return maxSpecifiersInOneLine;

package/package.json

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