fix(check-line-alignment): if no types are present, avoid allocatin…

…g extra space; fixes #891

README.md

@@ -2682,6 +2682,22 @@ const fn = ({ids}) => {}
         */
        const fn = ( lorem, sit ) => {}
 // "jsdoc/check-line-alignment": ["error"|"warn", "always"]
+
+/**
+ * Function description.
+ *
+ * @param lorem Description.
+ * @param sit   Description multi words.
+ */
+const fn = ( lorem, sit ) => {};
+
+/**
+ * Function description.
+ *
+ * @return Return description.
+ */
+const fn2 = () => {}
+// "jsdoc/check-line-alignment": ["error"|"warn", "always"]
 ````
 
 

src/alignTransform.js

@@ -79,7 +79,7 @@ const alignTransform = ({
   let intoTags = false;
   let width;
 
-  const alignTokens = (tokens) => {
+  const alignTokens = (tokens, hasNoTypes) => {
     const nothingAfter = {
       delim: false,
       name: false,
@@ -107,6 +107,11 @@ const alignTransform = ({
       }
     }
 
+    if (hasNoTypes) {
+      nothingAfter.tag = true;
+      tokens.postTag = '';
+    }
+
     // Todo: Avoid fixing alignment of blocks with multiline wrapping of type
     if (tokens.tag === '' && tokens.type) {
       return tokens;
@@ -137,7 +142,7 @@ const alignTransform = ({
     return tokens;
   };
 
-  const update = (line, index, source) => {
+  const update = (line, index, source, hasNoTypes) => {
     const tokens = {
       ...line.tokens,
     };
@@ -200,7 +205,7 @@ const alignTransform = ({
 
     return {
       ...line,
-      tokens: alignTokens(tokens),
+      tokens: alignTokens(tokens, hasNoTypes),
     };
   };
 
@@ -211,11 +216,16 @@ const alignTransform = ({
     width = source.reduce(getWidth(tags), {
       ...zeroWidth,
     });
+    const hasNoTypes = fields.tags.every(({
+      type,
+    }) => {
+      return !type;
+    });
 
     return rewireSource({
       ...fields,
       source: source.map((line, index) => {
-        return update(line, index, source);
+        return update(line, index, source, hasNoTypes);
       }),
     });
   };

test/rules/assertions/checkLineAlignment.js

@@ -1556,5 +1556,26 @@ export default {
         'always',
       ],
     },
+    {
+      code: `
+      /**
+       * Function description.
+       *
+       * @param lorem Description.
+       * @param sit   Description multi words.
+       */
+      const fn = ( lorem, sit ) => {};
+
+      /**
+       * Function description.
+       *
+       * @return Return description.
+       */
+      const fn2 = () => {}
+      `,
+      options: [
+        'always',
+      ],
+    },
   ],
 };