Necessary tweaks to the current formatting (#1033)

Author: Janther

package-lock.json

@@ -1,6 +1,6 @@
 {
   "name": "prettier-plugin-solidity",
-  "version": "1.3.1",
+  "version": "1.4.0",
   "description": "A Prettier Plugin for automatically formatting your Solidity code.",
   "type": "module",
   "main": "./src/index.js",
@@ -104,7 +104,7 @@
     "lines-and-columns": "^2.0.3",
     "prettier": "^3.1.1",
     "proxyquire": "^2.1.3",
-    "solc": "^0.8.25",
+    "solc": "^0.8.26",
     "webpack": "^5.88.2",
     "webpack-cli": "^5.1.4"
   },

package.json

@@ -8,15 +8,19 @@ import {
 const { hardline } = doc.builders;
 
 export const AssemblyBlock = {
-  print: ({ node, options, path, print }) => [
-    '{',
-    printSeparatedItem(
-      [
-        printPreservingEmptyLines(path, 'operations', options, print),
-        printComments(node, path, options)
-      ],
-      { firstSeparator: hardline, grouped: false }
-    ),
-    '}'
-  ]
+  print: ({ node, options, path, print }) =>
+    // if block is empty, just return the pair of braces
+    node.operations.length === 0 && !node.comments
+      ? '{}'
+      : [
+          '{',
+          printSeparatedItem(
+            [
+              printPreservingEmptyLines(path, 'operations', options, print),
+              printComments(node, path, options)
+            ],
+            { firstSeparator: hardline, grouped: false }
+          ),
+          '}'
+        ]
 };

src/nodes/AssemblyBlock.js

@@ -1,7 +1,12 @@
+import { doc } from 'prettier';
+
+const { hardline } = doc.builders;
+
 export const AssemblyStackAssignment = {
   print: ({ node, path, print }) => [
     path.call(print, 'expression'),
-    ' =: ',
+    hardline,
+    '=: ',
     node.name
   ]
 };

src/nodes/AssemblyStackAssignment.js

@@ -3,7 +3,6 @@ import { printSeparatedItem } from '../common/printer-helpers.js';
 
 const { group, hardline, ifBreak, indent, line, softline } = doc.builders;
 
-let groupIndex = 0;
 const experimentalTernaries = (node, path, print, options) => {
   const parent = path.getParentNode();
   const isNested = parent.type === 'Conditional';
@@ -30,11 +29,9 @@ const experimentalTernaries = (node, path, print, options) => {
 
   const conditionAndTrueExpressionGroup = group(
     [conditionDoc, trueExpressionDoc],
-    { id: `Conditional.trueExpressionDoc-${groupIndex}` }
+    { id: Symbol('Conditional.trueExpression') }
   );
 
-  groupIndex += 1;
-
   // For the odd case of `tabWidth` of 1 or 0 we initiate `fillTab` as a single
   // space.
   let fillTab = ' ';

src/nodes/Conditional.js

@@ -1,27 +1,6 @@
-import { doc } from 'prettier';
-import { printComments } from '../common/printer-helpers.js';
-
-const { hardline } = doc.builders;
-
 export const ExpressionStatement = {
-  print: ({ node, options, path, print }) => {
-    const parts = [];
-
-    const parent = path.getParentNode();
-
-    if (parent.type === 'IfStatement') {
-      if (node.comments?.length) {
-        const comments = printComments(node, path, options);
-        if (comments?.length) {
-          parts.push(comments);
-          parts.push(hardline);
-        }
-      }
-    }
-
-    parts.push(path.call(print, 'expression'));
-    parts.push(node.omitSemicolon ? '' : ';');
-
-    return parts;
-  }
+  print: ({ node, path, print }) => [
+    path.call(print, 'expression'),
+    node.omitSemicolon ? '' : ';'
+  ]
 };

src/nodes/ExpressionStatement.js

@@ -24,7 +24,6 @@ const printArguments = (path, print) =>
     lastSeparator: [softline, ')']
   });
 
-let groupIndex = 0;
 export const FunctionCall = {
   print: ({ node, path, print, options }) => {
     let expressionDoc = path.call(print, 'expression');
@@ -42,11 +41,9 @@ export const FunctionCall = {
     // arguments accordingly.
     if (expressionDoc.label === 'MemberAccessChain') {
       expressionDoc = group(expressionDoc.contents, {
-        id: `FunctionCall.expression-${groupIndex}`
+        id: Symbol('FunctionCall.expression')
       });
 
-      groupIndex += 1;
-
       argumentsDoc = indentIfBreak(argumentsDoc, {
         groupId: expressionDoc.id
       });

src/nodes/FunctionCall.js

@@ -24,16 +24,19 @@ const stateMutability = (node) =>
     : '';
 
 export const FunctionTypeName = {
-  print: ({ node, path, print }) => [
-    'function(',
-    printSeparatedList(path.map(print, 'parameterTypes')),
-    ')',
-    indent(
-      group([
-        visibility(node),
-        stateMutability(node),
-        returnTypes(node, path, print)
-      ])
-    )
-  ]
+  print: ({ node, path, print }) =>
+    group([
+      'function(',
+      printSeparatedList(path.map(print, 'parameterTypes'), {
+        grouped: false
+      }),
+      ')',
+      indent(
+        group([
+          visibility(node),
+          stateMutability(node),
+          returnTypes(node, path, print)
+        ])
+      )
+    ])
 };

src/nodes/FunctionTypeName.js

@@ -1,11 +1,11 @@
 import { doc } from 'prettier';
 import { printString } from '../common/util.js';
 
-const { join, line } = doc.builders;
+const { join, hardline } = doc.builders;
 
 export const HexLiteral = {
   print: ({ node, options }) => {
     const list = node.parts.map((part) => `hex${printString(part, options)}`);
-    return join(line, list);
+    return join(hardline, list);
   }
 };

src/nodes/HexLiteral.js

@@ -1,8 +1,5 @@
 import { doc } from 'prettier';
-import {
-  printComments,
-  printSeparatedItem
-} from '../common/printer-helpers.js';
+import { printSeparatedItem } from '../common/printer-helpers.js';
 
 const { group, hardline, indent, line } = doc.builders;
 
@@ -22,10 +19,9 @@ const printFalseBody = (node, path, print) =>
     ? [' ', path.call(print, 'falseBody')]
     : group(indent([line, path.call(print, 'falseBody')]));
 
-const printElse = (node, path, print, commentsBetweenIfAndElse) => {
+const printElse = (node, path, print) => {
   if (node.falseBody) {
-    const elseOnSameLine =
-      node.trueBody.type === 'Block' && commentsBetweenIfAndElse.length === 0;
+    const elseOnSameLine = node.trueBody.type === 'Block';
     return [
       elseOnSameLine ? ' ' : hardline,
       'else',
@@ -36,22 +32,11 @@ const printElse = (node, path, print, commentsBetweenIfAndElse) => {
 };
 
 export const IfStatement = {
-  print: ({ node, options, path, print }) => {
-    const comments = node.comments || [];
-    const commentsBetweenIfAndElse = comments.filter(
-      (comment) => !comment.leading && !comment.trailing
-    );
-
-    const parts = [];
-
-    parts.push('if (', printSeparatedItem(path.call(print, 'condition')), ')');
-    parts.push(printTrueBody(node, path, print));
-    if (commentsBetweenIfAndElse.length && node.falseBody) {
-      parts.push(hardline);
-      parts.push(printComments(node, path, options));
-    }
-    parts.push(printElse(node, path, print, commentsBetweenIfAndElse));
-
-    return parts;
-  }
+  print: ({ node, path, print }) => [
+    'if (',
+    printSeparatedItem(path.call(print, 'condition')),
+    ')',
+    printTrueBody(node, path, print),
+    printElse(node, path, print)
+  ]
 };