avoid calling coerce multiple times and only do it once at the beginning of the parse

Author: Janther

src/slang-nodes/ContractDefinition.ts

@@ -1,5 +1,5 @@
 import { doc } from 'prettier';
-import { coerce, satisfies } from 'semver';
+import { satisfies } from 'semver';
 import { NonterminalKind } from '@nomicfoundation/slang/cst';
 import { getNodeMetadata, updateMetadata } from '../slang-utils/metadata.js';
 import { Identifier } from './Identifier.js';
@@ -44,11 +44,10 @@ export class ContractDefinition implements SlangNode {
     this.cleanModifierInvocationArguments(options);
   }
 
-  cleanModifierInvocationArguments(options: ParserOptions<AstNode>): void {
+  cleanModifierInvocationArguments({ compiler }: ParserOptions<AstNode>): void {
     // Older versions of Solidity defined a constructor as a function having
     // the same name as the contract.
-    const compiler = coerce(options.compiler);
-    if (compiler && !satisfies(compiler, '>=0.5.0')) {
+    if (!satisfies(compiler, '>=0.5.0')) {
       for (const { variant: member } of this.members.items) {
         if (
           member.kind === NonterminalKind.FunctionDefinition &&

src/slang-nodes/FunctionDefinition.ts

@@ -1,4 +1,4 @@
-import { coerce, satisfies } from 'semver';
+import { satisfies } from 'semver';
 import { NonterminalKind } from '@nomicfoundation/slang/cst';
 import { printFunction } from '../slang-printers/print-function.js';
 import { getNodeMetadata, updateMetadata } from '../slang-utils/metadata.js';
@@ -54,8 +54,7 @@ export class FunctionDefinition implements SlangNode {
 
     // Older versions of Solidity defined a constructor as a function having
     // the same name as the contract.
-    const compiler = coerce(options.compiler);
-    if (compiler && satisfies(compiler, '>=0.5.0')) {
+    if (satisfies(options.compiler, '>=0.5.0')) {
       this.cleanModifierInvocationArguments();
     }
   }

src/slang-nodes/ImportDeconstructionSymbols.ts

@@ -1,5 +1,5 @@
 import { doc } from 'prettier';
-import { coerce, satisfies } from 'semver';
+import { satisfies } from 'semver';
 import { NonterminalKind } from '@nomicfoundation/slang/cst';
 import { printSeparatedList } from '../slang-printers/print-separated-list.js';
 import { getNodeMetadata, updateMetadata } from '../slang-utils/metadata.js';
@@ -38,22 +38,22 @@ export class ImportDeconstructionSymbols implements SlangNode {
   print(
     path: AstPath<ImportDeconstructionSymbols>,
     print: PrintFunction,
-    options: ParserOptions<AstNode>
+    { compiler, bracketSpacing }: ParserOptions<AstNode>
   ): Doc {
-    const compiler = coerce(options.compiler);
+    const items = path.map(print, 'items');
     return printSeparatedList(
-      path.map(print, 'items'),
-      compiler && satisfies(compiler, '>=0.7.4') && this.items.length > 1
+      items,
+      satisfies(compiler, '>=0.7.4') && items.length > 1
         ? {
             // if the compiler exists and is greater than or equal to 0.7.4 we will
             // split the ImportDirective.
-            firstSeparator: options.bracketSpacing ? line : softline,
+            firstSeparator: bracketSpacing ? line : softline,
             separator: [',', line]
           }
         : {
             // if the compiler is not given or is lower than 0.7.4 we will not
             // split the ImportDirective.
-            firstSeparator: options.bracketSpacing ? ' ' : '',
+            firstSeparator: bracketSpacing ? ' ' : '',
             separator: ', '
           }
     );