cfengine 0.17.0__tar.gz → 0.18.0__tar.gz

{cfengine-0.17.0 → 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.17.0 → cfengine-0.18.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: cfengine
-Version: 0.17.0
+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.17.0 → cfengine-0.18.0}/src/cfengine.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: cfengine
-Version: 0.17.0
+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.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.17.0 → 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.17.0 → 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.17.0 → 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
@@ -38,6 +39,14 @@ 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)."""
     return any(child.type == "macro" for child in node.children)
@@ -64,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:
@@ -78,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:
@@ -1008,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")
 
@@ -1031,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."""
@@ -1065,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."""
@@ -1079,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(".")]
@@ -1100,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(
@@ -1120,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:
@@ -1138,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.17.0 → 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.17.0 → 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.17.0 → cfengine-0.18.0}/src/cfengine_cli/syntax-description.json

@@ -9409,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.17.0/src/cfengine.egg-info/scm_version.json

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