cfengine 0.16.7__tar.gz → 0.18.0__tar.gz

{cfengine-0.16.7 → cfengine-0.18.0}/.github/workflows/update-syntax-description.yml

@@ -55,8 +55,9 @@ jobs:
           fi
       - name: Create Pull Request
         if: env.CHANGES_DETECTED == 'true'
-        uses: cfengine/create-pull-request@v6
+        uses: cfengine/create-pull-request@v7
         with:
+          sign-commits: true
           title: Updated syntax-description.json
           body: Automated update to syntax-description.json [the `update-syntax-description` workflow](https://github.com/cfengine/cfengine-cli/tree/main/.github/workflows/update-syntax-description.yml).
           reviewers: |

{cfengine-0.16.7 → cfengine-0.18.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: cfengine
-Version: 0.16.7
+Version: 0.18.0
 Summary: Human-oriented CLI for interacting with CFEngine policy language, projects, and other CFEngine tools
 License:                     GNU GENERAL PUBLIC LICENSE
                                Version 3, 29 June 2007

{cfengine-0.16.7 → cfengine-0.18.0}/src/cfengine.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: cfengine
-Version: 0.16.7
+Version: 0.18.0
 Summary: Human-oriented CLI for interacting with CFEngine policy language, projects, and other CFEngine tools
 License:                     GNU GENERAL PUBLIC LICENSE
                                Version 3, 29 June 2007

{cfengine-0.16.7 → cfengine-0.18.0}/src/cfengine.egg-info/SOURCES.txt

@@ -83,6 +83,8 @@ tests/format/011_macros.expected.cf
 tests/format/011_macros.input.cf
 tests/format/011_promises.expected.cf
 tests/format/011_promises.input.cf
+tests/format/012_sameline_comments.expected.cf
+tests/format/012_sameline_comments.input.cf
 tests/lint/001_hello_world.cf
 tests/lint/002_deprecations.expected.txt
 tests/lint/002_deprecations.x.cf

{cfengine-0.16.7 → cfengine-0.18.0}/src/cfengine.egg-info/scm_file_list.json

@@ -130,9 +130,11 @@
     "tests/format/004_comments.input.cf",
     "tests/format/001_hello_world.expected.cf",
     "tests/format/011_promises.input.cf",
+    "tests/format/012_sameline_comments.expected.cf",
     "tests/format/008_long_string.expected.cf",
     "tests/format/008_long_string.input.cf",
     "tests/format/002_basics.input.cf",
+    "tests/format/012_sameline_comments.input.cf",
     "tests/format/011_macros.input.cf",
     "tests/unit/__init__.py",
     "tests/unit/test_version.py",

cfengine-0.18.0/src/cfengine.egg-info/scm_version.json

@@ -0,0 +1,8 @@
+{
+  "tag": "0.18.0",
+  "distance": 0,
+  "node": "g226b1b9a975f32e3626fd1bd8cc4578e2a6f2b94",
+  "dirty": false,
+  "branch": "HEAD",
+  "node_date": "2026-07-07"
+}

{cfengine-0.16.7 → cfengine-0.18.0}/src/cfengine_cli/commands.py

@@ -52,14 +52,14 @@ def format(names, line_length, check) -> int:
     return format_paths(names, line_length, check)
 
 
-def _lint(files, strict) -> int:
+def _lint(files, strict, syntax_path) -> int:
     if not files:
-        return lint_args(["."], strict)
-    return lint_args(files, strict)
+        return lint_args(["."], strict, syntax_path)
+    return lint_args(files, strict, syntax_path)
 
 
-def lint(files, strict) -> int:
-    errors = _lint(files, strict)
+def lint(files, strict, syntax_path) -> int:
+    errors = _lint(files, strict, syntax_path)
     if errors == 0:
         print("Success, no errors found.")
     elif errors == 1:

{cfengine-0.16.7 → cfengine-0.18.0}/src/cfengine_cli/docs.py

@@ -61,15 +61,16 @@ def extract_inline_code(path, languages):
             indent = count_indent(lines[first_line])
             # Index of first line to NOT include, the line after closing triple backtick:
             last_line = child.map[1]
+            lines_to_extract = lines[first_line:last_line]
+            _remove_indentation(lines_to_extract, indent)
             yield {
                 "language": language,
                 "flags": flags,
                 "first_line": child.map[0],
                 "last_line": child.map[1],
                 "indent": indent,
-                "lines": lines[
-                    first_line:last_line
-                ],  # Includes backtick fences on both sides
+                # TODO: Dead code, the actual lines are extracted "twice" (2 places in the codebase)
+                "lines": lines_to_extract,  # Includes backtick fences on both sides
             }
 
 
@@ -97,12 +98,24 @@ def get_markdown_files(start, languages):
     return return_dict
 
 
+def _remove_indentation(snippet_lines, indent):
+    for i, line in enumerate(snippet_lines):
+        if line == "":
+            continue
+        assert line.startswith(" " * indent)
+        snippet_lines[i] = line[indent:]
+
+
 def fn_extract(origin_path, snippet_path, _language, first_line, last_line):
     try:
         with open(origin_path, "r") as f:
-            content = f.read()
-
-        code_snippet = "\n".join(content.split("\n")[first_line + 1 : last_line - 1])
+            lines = f.readlines()
+        lines = [x[0:-1] for x in lines]  # Remove newlines
+        fence = lines[first_line]
+        indent = count_indent(fence)
+        snippet_lines = lines[first_line + 1 : last_line - 1]
+        _remove_indentation(snippet_lines, indent)
+        code_snippet = "\n".join(snippet_lines)
 
         with open(snippet_path, "w") as f:
             f.write(code_snippet + "\n")
@@ -174,7 +187,7 @@ def fn_replace(origin_path, snippet_path, _language, first_line, last_line, inde
             origin_lines = f.read().split("\n")
             pretty_lines = pretty_content.split("\n")
 
-            pretty_lines = [" " * indent + x for x in pretty_lines]
+            pretty_lines = ["" if x == "" else " " * indent + x for x in pretty_lines]
 
             offset = len(pretty_lines) - len(
                 origin_lines[first_line + 1 : last_line - 1]
@@ -213,7 +226,6 @@ def fn_autoformat(
             except json.decoder.JSONDecodeError:
                 raise UserError(f"Invalid json in '{snippet_path}'")
         case "cf":
-            # Note: Dead code - Not used for CFEngine policy yet
             format_policy_file(snippet_path, 80, False)
             return False
 
@@ -434,7 +446,7 @@ def _format_docs_single_arg(path):
             formatted = True
         _process_markdown_code_blocks(
             path=path,
-            languages=["json"],  # TODO: Add cfengine3 here
+            languages=["json", "cfengine3"],
             extract=True,
             syntax_check=False,
             output_check=False,

{cfengine-0.16.7 → cfengine-0.18.0}/src/cfengine_cli/format.py

@@ -1,5 +1,6 @@
 from __future__ import annotations
 
+from enum import IntEnum
 from typing import IO
 
 import json
@@ -35,6 +36,16 @@ BLOCK_BODY_TYPES = {"bundle_block_body", "promise_block_body", "body_block_body"
 
 PROMISER_PARTS = {"promiser", "->", "stakeholder"}
 
+SAMELINE_COMMENT_SEPARATOR = " "
+
+
+class FormatResult(IntEnum):
+    """Outcome of a formatting operation."""
+
+    NO_CHANGE = 0  # File was already formatted, no reformat needed
+    NEEDS_FORMAT = 1  # Reformat needed (reported in check mode)
+    REFORMATTED = 2  # File was reformatted successfully
+
 
 def _has_direct_macro(node: Node) -> bool:
     """Check if any direct child of a node is a macro (non-recursive)."""
@@ -62,12 +73,8 @@ def _contains_list_with_comment(nodes: Node | list[Node]) -> bool:
     return _contains_list_with_comment(nodes.children)
 
 
-def format_json_file(filename: str, check: bool) -> int:
-    """Reformat a JSON file in place using cfbs pretty-printer.
-
-    Returns 0 in case of successful reformat or no reformat needed.
-    Returns 1 when check is True and reformat is needed.
-    """
+def format_json_file(filename: str, check: bool) -> FormatResult:
+    """Reformat a JSON file in place using cfbs pretty-printer."""
     assert filename.endswith(".json")
 
     if check:
@@ -76,16 +83,17 @@ def format_json_file(filename: str, check: bool) -> int:
         assert type(success) is bool
         if not success:
             print(f"JSON file '{filename}' needs reformatting")
-        return int(not success)
+            return FormatResult.NEEDS_FORMAT
+        return FormatResult.NO_CHANGE
 
     try:
         reformatted = pretty_file(filename)
         if reformatted:
             print(f"JSON file '{filename}' was reformatted")
-        return 0  # Successfully reformatted or no reformat needed
+            return FormatResult.REFORMATTED
+        return FormatResult.NO_CHANGE
     except json.decoder.JSONDecodeError as e:
-        print(f"JSON file '{filename}' invalid ({str(e)})")
-        return 1
+        raise UserError(f"JSON file '{filename}' invalid ({str(e)})")
 
 
 def text(node: Node) -> str:
@@ -241,6 +249,9 @@ def split_generic_list(
     # outside the function call, so we do not add trailing commas into function
     # call argument lists.
     value_end_indices: list[int] = []
+    # Same-line (trailing) comments are attached only after trailing commas are
+    # placed, so the comma lands before the comment instead of after it.
+    sameline_comments: dict[int, str] = {}
     for element in middle:
         if elements and element.type == ",":
             elements[-1] = elements[-1] + ","
@@ -249,7 +260,10 @@ def split_generic_list(
             elements.append(text(element))
             continue
         if element.type == "comment":
-            elements.append(" " * indent + text(element))
+            if _is_sameline_comment(element) and elements:
+                sameline_comments[len(elements) - 1] = text(element)
+            else:
+                elements.append(" " * indent + text(element))
             continue
         line = " " * indent + stringify_single_line_node(element)
         # Strict < reserves 1 char for the comma appended after this check
@@ -275,6 +289,10 @@ def split_generic_list(
             if not elements[i].lstrip().startswith("#"):
                 elements[i] = _set_trailing_comma(elements[i], trailing_comma)
                 break
+
+    # Attach same-line comments to the end of their value lines
+    for i, comment in sameline_comments.items():
+        elements[i] = elements[i] + SAMELINE_COMMENT_SEPARATOR + comment
     return elements
 
 
@@ -354,7 +372,10 @@ def _format_attribute_with_macros(node: Node, indent: int) -> list[str]:
         if child.type == "macro":
             lines.append(text(child))
         elif child.type == "comment":
-            lines.append(" " * (indent + 2) + text(child))
+            if _is_sameline_comment(child) and lines:
+                lines[-1] = lines[-1] + SAMELINE_COMMENT_SEPARATOR + text(child)
+            else:
+                lines.append(" " * (indent + 2) + text(child))
         else:
             lines.append(" " * (indent + 2) + stringify_single_line_node(child))
 
@@ -505,13 +526,19 @@ def _format_stakeholder_elements(
         return maybe_split_generic_list(middle, indent, line_length)
     # Comments present — format element-by-element to preserve them
     elements: list[str] = []
+    # Same-line (trailing) comments are attached only after the trailing comma is
+    # placed, so the comma lands before the comment instead of after it.
+    sameline_comments: dict[int, str] = {}
     for node in middle:
         if node.type == ",":
             if elements:
                 elements[-1] = elements[-1] + ","
             continue
         if node.type == "comment":
-            elements.append(" " * indent + text(node))
+            if _is_sameline_comment(node) and elements:
+                sameline_comments[len(elements) - 1] = text(node)
+            else:
+                elements.append(" " * indent + text(node))
         else:
             line = " " * indent + stringify_single_line_node(node)
             # Strict < reserves 1 char for the comma appended after this check
@@ -528,6 +555,10 @@ def _format_stakeholder_elements(
         if not elements[i].endswith(","):
             elements[i] = elements[i] + ","
         break
+
+    # Attach inline comments to the end of their value lines
+    for i, comment in sameline_comments.items():
+        elements[i] = elements[i] + SAMELINE_COMMENT_SEPARATOR + comment
     return elements
 
 
@@ -682,9 +713,13 @@ def _format_block_header(node: Node, fmt: Formatter) -> list[Node]:
     """Format a block header line and return the body's children for further processing."""
     header_parts: list[str] = []
     header_comments: list[str] = []
+    # Direct comment children of the block (excludes comments inside a
+    # parameter list) — candidates for being kept on the header line.
+    direct_comments: list[Node] = []
     for x in node.children[0:-1]:
         if x.type == "comment":
             header_comments.append(text(x))
+            direct_comments.append(x)
         elif x.type == "parameter_list":
             parts: list[str] = []
             for p in x.children:
@@ -696,6 +731,16 @@ def _format_block_header(node: Node, fmt: Formatter) -> list[Node]:
         else:
             header_parts.append(text(x))
     line = " ".join(header_parts)
+    # A lone same-line comment stays on the header line. When the header has
+    # more than one comment, keep them together on their own lines below the
+    # header instead of splitting one off onto the header line.
+    if (
+        len(header_comments) == 1
+        and len(direct_comments) == 1
+        and _is_sameline_comment(direct_comments[0])
+    ):
+        line += SAMELINE_COMMENT_SEPARATOR + header_comments[0]
+        header_comments = []
     if not fmt.empty:
         prev_sib = node.prev_named_sibling
         # Skip over preceding empty comments since they will be removed
@@ -725,6 +770,11 @@ def _format_block_header(node: Node, fmt: Formatter) -> list[Node]:
 
 def _needs_blank_line_before(child: Node, indent: int, line_length: int) -> bool:
     """Check if a blank separator line should precede this child node."""
+    # Inline (trailing) comments are appended to the preceding line, so they
+    # must never be preceded by a blank separator line.
+    if child.type == "comment" and _is_sameline_comment(child):
+        return False
+
     prev = child.prev_named_sibling
     # Empty comments preceding this child will be dropped — look past them
     # so we evaluate against the real prior content.
@@ -798,6 +848,21 @@ def _needs_blank_line_before(child: Node, indent: int, line_length: int) -> bool
 # ---------------------------------------------------------------------------
 
 
+def _is_sameline_comment(node: Node) -> bool:
+    """Check if a comment sits on the same source line as the element before it.
+
+    A trailing comment shares the row of the preceding sibling (which may be
+    a leaf like ',' or ';', or a whole promise/attribute). Such comments are
+    kept on the same output line as that element rather than moved to their
+    own line."""
+    if node.type != "comment":
+        return False
+    prev = node.prev_sibling
+    if prev is None:
+        return False
+    return prev.end_point[0] == node.start_point[0]
+
+
 def _is_empty_comment(node: Node) -> bool:
     """Check if a bare '#' comment should be dropped.
 
@@ -934,7 +999,11 @@ def _autoformat(
         else:
             fmt.print_same_line(node)
     elif node.type == "comment":
-        if not _is_empty_comment(node):
+        if _is_empty_comment(node):
+            pass
+        elif _is_sameline_comment(node) and not fmt.empty:
+            fmt.print_same_line(SAMELINE_COMMENT_SEPARATOR + text(node))
+        else:
             fmt.print(node, _comment_indent(node, indent))
     else:
         fmt.print(node, indent)
@@ -945,11 +1014,9 @@ def _autoformat(
 # ---------------------------------------------------------------------------
 
 
-def format_policy_file(filename: str, line_length: int, check: bool) -> int:
+def format_policy_file(filename: str, line_length: int, check: bool) -> FormatResult:
     """Format a .cf policy file in place, writing only if content changed.
 
-    Returns 0 in case of successful reformat or no reformat needed.
-    Returns 1 when check is True and reformat is needed.
     Raises PolicySyntaxError when the file has syntax errors."""
     assert filename.endswith(".cf")
 
@@ -968,21 +1035,28 @@ def format_policy_file(filename: str, line_length: int, check: bool) -> int:
     fmt = Formatter()
     _autoformat(root_node, fmt, line_length)
 
-    new_data = fmt.buffer + "\n"
+    new_data = fmt.buffer
+    if not new_data.endswith("\n"):
+        # TODO: Look into why formatter sometimes outputs
+        #       trailing newline and other times not.
+        new_data += "\n"
+    assert new_data.endswith("\n") and not new_data.endswith("\n\n")
+
     if new_data != original_data.decode("utf-8"):
         if check:
             print(f"Policy file '{filename}' needs reformatting")
-            return 1
+            return FormatResult.NEEDS_FORMAT
 
         with open(filename, "w") as f:
             f.write(new_data)
         print(f"Policy file '{filename}' was reformatted")
-    return 0
+        return FormatResult.REFORMATTED
+    return FormatResult.NO_CHANGE
 
 
 def format_policy_fin_fout(
     fin: IO[str], fout: IO[str], line_length: int, check: bool
-) -> int:
+) -> FormatResult:
     """Format CFEngine policy read from fin, writing the result to fout.
 
     Raises PolicySyntaxError when the input has syntax errors."""
@@ -1002,10 +1076,12 @@ def format_policy_fin_fout(
 
     new_data = fmt.buffer + "\n"
     fout.write(new_data)
-    return 0
+    if new_data != original_data.decode("utf-8"):
+        return FormatResult.REFORMATTED
+    return FormatResult.NO_CHANGE
 
 
-def _format_filename(filename: str, line_length: int, check: bool) -> int:
+def _format_filename(filename: str, line_length: int, check: bool) -> FormatResult:
     """Format a single file.
 
     Raises PolicySyntaxError for .cf files with syntax errors."""
@@ -1016,8 +1092,17 @@ def _format_filename(filename: str, line_length: int, check: bool) -> int:
     raise UserError(f"Unrecognized file format: {filename}")
 
 
-def _format_dirname(directory: str, line_length: int, check: bool) -> int:
-    ret = 0
+def _combine_results(a: FormatResult, b: FormatResult) -> FormatResult:
+    """Combine two format results. A non-NO_CHANGE result takes precedence.
+
+    Within a single format run, `check` is fixed, so NEEDS_FORMAT and
+    REFORMATTED never occur together — one always dominates NO_CHANGE."""
+    return b if b != FormatResult.NO_CHANGE else a
+
+
+def _format_dirname(directory: str, line_length: int, check: bool) -> FormatResult:
+    """Format files in directory recursively."""
+    ret = FormatResult.NO_CHANGE
     for root, dirs, files in os.walk(directory):
         # Don't recurse into hidden folders
         dirs[:] = [d for d in dirs if not d.startswith(".")]
@@ -1037,18 +1122,20 @@ def _format_dirname(directory: str, line_length: int, check: bool) -> int:
                 continue  # Test files skipped during directory traversal
             filepath = os.path.join(root, name)
             if name.endswith(".json") or name.endswith(".cf"):
-                ret |= _format_filename(filepath, line_length, check)
+                ret = _combine_results(
+                    ret, _format_filename(filepath, line_length, check)
+                )
     return ret
 
 
-def _format_paths_inner(names, line_length, check) -> int:
+def _format_paths_inner(names, line_length, check) -> FormatResult:
     if not names:
         return _format_dirname(".", line_length, check)
     if len(names) == 1 and names[0] == "-":
         # Special case, format policy file from stdin to stdout
         return format_policy_fin_fout(sys.stdin, sys.stdout, line_length, check)
 
-    ret = 0
+    ret = FormatResult.NO_CHANGE
     for name in names:
         if name == "-":
             raise UserError(
@@ -1057,14 +1144,12 @@ def _format_paths_inner(names, line_length, check) -> int:
         if not os.path.exists(name):
             raise UserError(f"{name} does not exist")
         if os.path.isfile(name):
-            ret |= _format_filename(name, line_length, check)
+            ret = _combine_results(ret, _format_filename(name, line_length, check))
             continue
         if os.path.isdir(name):
-            ret |= _format_dirname(name, line_length, check)
+            ret = _combine_results(ret, _format_dirname(name, line_length, check))
             continue
-    if check:
-        return ret
-    return 0
+    return ret
 
 
 def format_paths(names, line_length, check) -> int:
@@ -1075,7 +1160,14 @@ def format_paths(names, line_length, check) -> int:
     needed), 1 when reformatting is needed in check mode or a policy file
     has syntax errors."""
     try:
-        return _format_paths_inner(names, line_length, check)
+        r = _format_paths_inner(names, line_length, check)
+        if r == FormatResult.NEEDS_FORMAT:
+            assert check
+            # File needs reformatting
+            return 1
+        assert r in (FormatResult.NO_CHANGE, FormatResult.REFORMATTED)
+        # Success - no format needed or successfully reformatted
+        return 0
     except PolicySyntaxError as e:
         print(f"Error: {e}")
         return 1

{cfengine-0.16.7 → cfengine-0.18.0}/src/cfengine_cli/lint.py

@@ -81,9 +81,9 @@ class SyntaxData:
     BUILTIN_PROMISE_TYPES = {}
     BUILTIN_FUNCTIONS = {}
 
-    def __init__(self):
+    def __init__(self, syntax_path=None):
         """Load the bundled syntax-description.json and derive lookup dicts."""
-        self._data_dict = self._load_syntax_description()
+        self._data_dict = self._load_syntax_description(path=syntax_path)
         self._derive_syntax_dicts(self._data_dict)
 
         assert self.BUILTIN_BODY_TYPES
@@ -1100,6 +1100,7 @@ def _lint_main(
     state=None,
     snippet: Snippet | None = None,
     syntax_data=None,
+    syntax_path: str | None = None,
 ) -> int:
     """This is the main function used for linting, it does all the steps on all
     the arguments (files / folders).
@@ -1127,7 +1128,10 @@ def _lint_main(
     state.mode = Mode.SYNTAX
 
     if syntax_data is None:
-        syntax_data = SyntaxData()
+        try:
+            syntax_data = SyntaxData(syntax_path)
+        except FileNotFoundError:
+            raise UserError(f"'{syntax_path}' does not exist")
 
     filenames = _args_to_filenames(args)
 
@@ -1379,9 +1383,11 @@ def lint_single_file(file: str, strict: bool = True) -> int:
     return _lint_main([file], strict)
 
 
-def lint_args(args: Iterable[str], strict: bool = True) -> int:
+def lint_args(
+    args: Iterable[str], strict: bool = True, syntax_path: str | None = None
+) -> int:
     """Lint a list of args (files / folders)"""
-    return _lint_main(list(args), strict)
+    return _lint_main(list(args), strict, syntax_path=syntax_path)
 
 
 def lint_policy_file_snippet(

{cfengine-0.16.7 → cfengine-0.18.0}/src/cfengine_cli/main.py

@@ -59,6 +59,11 @@ def _get_arg_parser():
         default="yes",
         help="Strict mode. Default=yes, checks for undefined promise types, bundles, bodies, functions",
     )
+    lnt.add_argument(
+        "--syntax-description",
+        type=str,
+        help="Lint based on a user given syntax description",
+    )
     lnt.add_argument("files", nargs="*", help="Files to lint")
     subp.add_parser(
         "report",
@@ -188,7 +193,11 @@ def run_command_with_args(args) -> int:
     if args.command == "format":
         return commands.format(args.files, args.line_length, args.check)
     if args.command == "lint":
-        return commands.lint(args.files, (args.strict.lower() in ("y", "ye", "yes")))
+        return commands.lint(
+            args.files,
+            (args.strict.lower() in ("y", "ye", "yes")),
+            args.syntax_description,
+        )
     if args.command == "report":
         return commands.report()
     if args.command == "run":

{cfengine-0.16.7 → cfengine-0.18.0}/src/cfengine_cli/syntax-description.json

@@ -753,7 +753,7 @@
         },
         "protocol_version": {
           "attribute": "protocol_version",
-          "range": "1,classic,2,tls,3,cookie,4,filestream,latest",
+          "range": "1,classic,2,tls,3,cookie,4,filestream,5,leech2,latest",
           "status": "normal",
           "type": "option",
           "visibility": "body"
@@ -1020,7 +1020,7 @@
         },
         "protocol_version": {
           "attribute": "protocol_version",
-          "range": "1,classic,2,tls,3,cookie,4,filestream,latest",
+          "range": "1,classic,2,tls,3,cookie,4,filestream,5,leech2,latest",
           "status": "normal",
           "type": "option",
           "visibility": "body"
@@ -7368,6 +7368,13 @@
           "type": "clist",
           "visibility": "promiseType"
         },
+        "cancel": {
+          "attribute": "cancel",
+          "range": "true,false,yes,no,on,off",
+          "status": "normal",
+          "type": "option",
+          "visibility": "promiseType"
+        },
         "classes": {
           "attribute": "classes",
           "status": "normal",
@@ -7465,6 +7472,13 @@
           "type": "clist",
           "visibility": "promiseType"
         },
+        "timer_policy": {
+          "attribute": "timer_policy",
+          "range": "absolute,reset",
+          "status": "normal",
+          "type": "option",
+          "visibility": "promiseType"
+        },
         "unless": {
           "attribute": "unless",
           "range": "",
@@ -9395,7 +9409,7 @@
         },
         "signals": {
           "attribute": "signals",
-          "range": "(hup|int|trap|kill|pipe|cont|abrt|stop|quit|term|child|usr1|usr2|bus|segv|[0-9]+s?)",
+          "range": "((?i:hup|int|trap|kill|pipe|cont|abrt|stop|quit|term|child|usr1|usr2|bus|segv)|[0-9]+s?)",
           "status": "normal",
           "type": "slist",
           "visibility": "promiseType"

{cfengine-0.16.7 → cfengine-0.18.0}/tests/format/004_comments.expected.cf

@@ -52,17 +52,14 @@ bundle agent win_services
     "autostart_services"
       slist => {
         "Alerter",
-        "W32Time",
-        # Windows Time
+        "W32Time", # Windows Time
       };
 
     some_class::
       "autostart_services"
         slist => {
-          "MpsSvc",
-          # Windows Firewall
-          "W32Time",
-          # Windows Time
+          "MpsSvc", # Windows Firewall
+          "W32Time", # Windows Time
         };
 
     some_class::

{cfengine-0.16.7 → cfengine-0.18.0}/tests/format/005_bundle_comments.expected.cf

@@ -1,5 +1,4 @@
-bundle agent a
-# @brief Description of a
+bundle agent a # @brief Description of a
 {
   reports:
     "Hello, world!";