Remove ParserFormatter's constructor.

All state held in that class is apparently now moved to the
ParserCombinator which uses the serialized output

Possibly change all methods in the class to be class/static methods
instead in the future.

muppet/format.py

@@ -98,7 +98,7 @@ def parse_puppet(source: str, file: str) -> str:
     ast = build_ast(puppet_parser(source))
     # logger.info("ast: %a", ast)
     # From the ast, build a parser combinator parser.
-    parser = ParserFormatter(source, file=file).serialize(ast)
+    parser = ParserFormatter().serialize(ast)
     # logger.warning("parser: %a", parser)
     # Run the generatefd parser, giving us a list of match objects
     match_objects = ParserCombinator(source, file).get(parser)

muppet/puppet/format/parser.py

@@ -201,25 +201,7 @@ class rich_char(ParseDirective):
 
 
 class ParserFormatter(Serializer[ParseDirective]):
-    """
-    Reserialize AST by highlighting the original source code.
-
-    :param source:
-        The original source code. *Must* be the exact same source as
-        used to construct the corresponinding Puppet ast object.
-    :param seek:
-        Current parsing position in the string.
-
-        TODO make this private.
-    """
-
-    # parser: ParserCombinator
-
-    # def __init__(self, source: str, file: Optional[str] = None):
-    def __init__(self, source: str, file: str):
-        # self.parser = ParserCombinator(source=source, file=file)
-        self.file = file
-        pass
+    """Reserialize AST by highlighting the original source code."""
 
     def declaration_parameter(self, item: PuppetDeclarationParameter) -> ParseDirective:
         """Build parser for the given declaration parameter."""
@@ -683,7 +665,6 @@ class ParserFormatter(Serializer[ParseDirective]):
             parser = (single_quoted | double_quoted)
         else:
             parser = (raw_string | single_quoted | double_quoted)
-        # TODO should the whitespace really be here?
         return ws & tag('string', name(it.s, parser))
 
     @override

tests/test_parse_elsif.py

@@ -95,7 +95,7 @@ def parse_string(s: str, *,
     if ast:
         assert generated_ast == ast
 
-    parser = ParserFormatter(s, "s").serialize(generated_ast)
+    parser = ParserFormatter().serialize(generated_ast)
 
     print()
     pprint(parser)