printing undefined as empty string so most of the Ternary operators could be removed.

Author: Janther

src/slang-nodes/AddressType.ts

@@ -1,5 +1,6 @@
 import { NonterminalKind } from '@nomicfoundation/slang/kinds/index.js';
 import { getNodeMetadata } from '../slang-utils/metadata.js';
+import { joinExisting } from '../slang-utils/join-existing.js';
 
 import type * as ast from '@nomicfoundation/slang/ast';
 import type { Doc } from 'prettier';
@@ -24,6 +25,6 @@ export class AddressType implements SlangNode {
   }
 
   print(): Doc {
-    return ['address', this.payableKeyword ? ' payable' : ''];
+    return joinExisting(' ', ['address', this.payableKeyword]);
   }
 }

src/slang-nodes/ArrayTypeName.ts

@@ -39,11 +39,6 @@ export class ArrayTypeName implements SlangNode {
   }
 
   print(path: AstPath<ArrayTypeName>, print: PrintFunction): Doc {
-    return [
-      path.call(print, 'operand'),
-      '[',
-      this.index ? path.call(print, 'index') : '',
-      ']'
-    ];
+    return [path.call(print, 'operand'), '[', path.call(print, 'index'), ']'];
   }
 }

src/slang-nodes/AssemblyStatement.ts

@@ -1,5 +1,6 @@
 import { NonterminalKind } from '@nomicfoundation/slang/kinds/index.js';
 import { getNodeMetadata, updateMetadata } from '../slang-utils/metadata.js';
+import { joinExisting } from '../slang-utils/join-existing.js';
 import { StringLiteral } from './StringLiteral.js';
 import { AssemblyFlagsDeclaration } from './AssemblyFlagsDeclaration.js';
 import { YulBlock } from './YulBlock.js';
@@ -48,11 +49,11 @@ export class AssemblyStatement implements SlangNode {
   }
 
   print(path: AstPath<AssemblyStatement>, print: PrintFunction): Doc {
-    return [
-      'assembly ',
-      this.label ? [path.call(print, 'label'), ' '] : '',
-      this.flags ? [path.call(print, 'flags'), ' '] : '',
+    return joinExisting(' ', [
+      'assembly',
+      path.call(print, 'label'),
+      path.call(print, 'flags'),
       path.call(print, 'body')
-    ];
+    ]);
   }
 }

src/slang-nodes/CatchClause.ts

@@ -41,10 +41,6 @@ export class CatchClause implements SlangNode {
   }
 
   print(path: AstPath<CatchClause>, print: PrintFunction): Doc {
-    return [
-      'catch ',
-      this.error ? path.call(print, 'error') : '',
-      path.call(print, 'body')
-    ];
+    return ['catch ', path.call(print, 'error'), path.call(print, 'body')];
   }
 }

src/slang-nodes/CatchClauseError.ts

@@ -49,7 +49,7 @@ export class CatchClauseError implements SlangNode {
 
   print(path: AstPath<CatchClauseError>, print: PrintFunction): Doc {
     return [
-      this.name ? path.call(print, 'name') : '',
+      path.call(print, 'name'),
       group(path.call(print, 'parameters')),
       ' '
     ];

src/slang-nodes/DecimalNumberExpression.ts

@@ -1,5 +1,6 @@
 import { NonterminalKind } from '@nomicfoundation/slang/kinds/index.js';
 import { getNodeMetadata, updateMetadata } from '../slang-utils/metadata.js';
+import { joinExisting } from '../slang-utils/join-existing.js';
 import { NumberUnit } from './NumberUnit.js';
 
 import type * as ast from '@nomicfoundation/slang/ast';
@@ -33,6 +34,6 @@ export class DecimalNumberExpression implements SlangNode {
   }
 
   print(path: AstPath<DecimalNumberExpression>, print: PrintFunction): Doc {
-    return [this.literal, this.unit ? [' ', path.call(print, 'unit')] : ''];
+    return joinExisting(' ', [this.literal, path.call(print, 'unit')]);
   }
 }

src/slang-nodes/ErrorParameter.ts

@@ -1,5 +1,6 @@
 import { NonterminalKind } from '@nomicfoundation/slang/kinds/index.js';
 import { getNodeMetadata, updateMetadata } from '../slang-utils/metadata.js';
+import { joinExisting } from '../slang-utils/join-existing.js';
 import { TypeName } from './TypeName.js';
 import { Identifier } from './Identifier.js';
 
@@ -39,9 +40,9 @@ export class ErrorParameter implements SlangNode {
   }
 
   print(path: AstPath<ErrorParameter>, print: PrintFunction): Doc {
-    return [
+    return joinExisting(' ', [
       path.call(print, 'typeName'),
-      this.name ? [' ', path.call(print, 'name')] : ''
-    ];
+      path.call(print, 'name')
+    ]);
   }
 }

src/slang-nodes/EventParameter.ts

@@ -1,5 +1,6 @@
 import { NonterminalKind } from '@nomicfoundation/slang/kinds/index.js';
 import { getNodeMetadata, updateMetadata } from '../slang-utils/metadata.js';
+import { joinExisting } from '../slang-utils/join-existing.js';
 import { TypeName } from './TypeName.js';
 import { Identifier } from './Identifier.js';
 
@@ -42,10 +43,10 @@ export class EventParameter implements SlangNode {
   }
 
   print(path: AstPath<EventParameter>, print: PrintFunction): Doc {
-    return [
+    return joinExisting(' ', [
       path.call(print, 'typeName'),
-      this.indexedKeyword ? ' indexed' : '',
-      this.name ? [' ', path.call(print, 'name')] : ''
-    ];
+      this.indexedKeyword,
+      path.call(print, 'name')
+    ]);
   }
 }

src/slang-nodes/ForStatement.ts

@@ -1,5 +1,5 @@
-import { doc } from 'prettier';
 import { NonterminalKind } from '@nomicfoundation/slang/kinds/index.js';
+import { doc } from 'prettier';
 import { printSeparatedList } from '../slang-printers/print-separated-list.js';
 import { getNodeMetadata, updateMetadata } from '../slang-utils/metadata.js';
 import { ForStatementInitialization } from './ForStatementInitialization.js';
@@ -68,7 +68,7 @@ export class ForStatement implements SlangNode {
   print(path: AstPath<ForStatement>, print: PrintFunction): Doc {
     const initialization = path.call(print, 'initialization');
     const condition = path.call(print, 'condition');
-    const iterator = this.iterator ? path.call(print, 'iterator') : '';
+    const iterator = path.call(print, 'iterator');
 
     return [
       'for (',

src/slang-nodes/HexNumberExpression.ts

@@ -1,5 +1,6 @@
 import { NonterminalKind } from '@nomicfoundation/slang/kinds/index.js';
 import { getNodeMetadata, updateMetadata } from '../slang-utils/metadata.js';
+import { joinExisting } from '../slang-utils/join-existing.js';
 import { NumberUnit } from './NumberUnit.js';
 
 import type * as ast from '@nomicfoundation/slang/ast';
@@ -33,6 +34,6 @@ export class HexNumberExpression implements SlangNode {
   }
 
   print(path: AstPath<HexNumberExpression>, print: PrintFunction): Doc {
-    return [this.literal, this.unit ? [' ', path.call(print, 'unit')] : ''];
+    return joinExisting(' ', [this.literal, path.call(print, 'unit')]);
   }
 }