cfengine 0.16.1__tar.gz → 0.16.3__tar.gz

{cfengine-0.16.1 → cfengine-0.16.3}/PKG-INFO

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

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: cfengine
-Version: 0.16.1
+Version: 0.16.3
 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.1 → cfengine-0.16.3}/src/cfengine_cli/docs.py

@@ -279,6 +279,7 @@ def _process_markdown_code_blocks(
             for cb in parsed_markdowns["files"][origin_path]["code-blocks"][i:]:
                 cb["first_line"] += offset
                 cb["last_line"] += offset
+            offset = 0
 
             language = _translate_language(code_block["language"])
             snippet_number = i + 1

{cfengine-0.16.1 → cfengine-0.16.3}/src/cfengine_cli/format.py

@@ -711,6 +711,10 @@ 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."""
     prev = child.prev_named_sibling
+    # Empty comments preceding this child will be dropped — look past them
+    # so we evaluate against the real prior content.
+    while prev and prev.type == "comment" and _is_empty_comment(prev):
+        prev = prev.prev_named_sibling
     if not prev:
         return False
 
@@ -743,12 +747,6 @@ def _needs_blank_line_before(child: Node, indent: int, line_length: int) -> bool
     if child.type == "comment":
         if _is_empty_comment(child):
             return False
-        # Empty comments preceding this one will be dropped — look past them
-        # so we treat the comment as following the real prior content.
-        while prev and prev.type == "comment" and _is_empty_comment(prev):
-            prev = prev.prev_named_sibling
-        if prev is None:
-            return False
         # Top-level comment after a complete block — visually separates them
         if prev.type in BLOCK_TYPES:
             return True
@@ -783,12 +781,27 @@ def _needs_blank_line_before(child: Node, indent: int, line_length: int) -> bool
 
 
 def _is_empty_comment(node: Node) -> bool:
-    """Check if a bare '#' comment should be dropped (not between other comments)."""
+    """Check if a bare '#' comment should be dropped.
+
+    A run of bare '#' between two non-empty comments collapses to a
+    single '#' — the first in the run is kept, the rest are dropped.
+    Bare '#' not flanked by real comments is always dropped."""
     if text(node).strip() != "#":
         return False
     prev = node.prev_named_sibling
+    # Not the first in a run of '#' — a kept '#' precedes us, drop.
+    if prev and prev.type == "comment" and text(prev).strip() == "#":
+        return True
+    # First in a run — kept only if a non-empty comment sits on each side
+    # (forward search walks past the rest of the run to find one).
+    if not (prev and prev.type == "comment"):
+        return True
     nxt = node.next_named_sibling
-    return not (prev and prev.type == "comment" and nxt and nxt.type == "comment")
+    while nxt and nxt.type == "comment" and text(nxt).strip() == "#":
+        nxt = nxt.next_named_sibling
+    if not (nxt and nxt.type == "comment"):
+        return True
+    return False
 
 
 def _skip_comments(sibling: Node | None, direction: str = "next") -> Node | None:
@@ -809,6 +822,10 @@ def _comment_indent(node: Node, indent: int) -> int:
     # content at the end of the previous sibling subtree, so that comments
     # appended after a class-guarded block visually belong to that block.
     if nearest is None:
+        # A comment between a promiser and its terminating ';' is a promise
+        # continuation — indent it past the promiser.
+        if node.parent and node.parent.type in {"promise", "half_promise"}:
+            return indent + 2
         prev = _skip_comments(node.prev_named_sibling, "prev")
         if prev and prev.type in INDENTED_TYPES:
             return _trailing_comment_indent(prev, indent)
@@ -892,6 +909,10 @@ def _autoformat(
     if node.type in {",", ";"}:
         if previous and previous.type == "macro":
             fmt.print(node, indent + 2)
+        elif previous and previous.type == "comment":
+            # A '#' comment runs to end of line, so a following ';'/',' can't
+            # share that line — put it on its own continuation line.
+            fmt.print(node, indent + 2)
         else:
             fmt.print_same_line(node)
     elif node.type == "comment":

{cfengine-0.16.1 → cfengine-0.16.3}/tests/format/004_comments.expected.cf

@@ -151,3 +151,16 @@ bundle agent stigs
   vars:
     "foo" string => "bar";
 }
+
+bundle agent example_comment_before_semicolon
+{
+  files:
+    "affected object"
+      # ...details....
+      ;
+
+  processes:
+    "affected object"
+      # ...details....
+      ;
+}

{cfengine-0.16.1 → cfengine-0.16.3}/tests/format/004_comments.input.cf

@@ -143,3 +143,14 @@ bundle agent stigs
   vars:
     "foo" string => "bar";
 }
+bundle agent example_comment_before_semicolon
+{
+  files:
+    "affected object"
+    # ...details....
+    ;
+  processes:
+    "affected object"
+    # ...details....
+    ;
+}

{cfengine-0.16.1 → cfengine-0.16.3}/tests/format/006_remove_empty_comments.expected.cf

@@ -1,3 +1,6 @@
+# explanation
+#
+# one more line
 bundle agent main
 {
   # comment
@@ -22,3 +25,21 @@ bundle agent c
   reports:
     "hello";
 }
+
+bundle agent d
+{
+  reports:
+    "hello";
+
+    any::
+      "world";
+}
+
+bundle agent e
+{
+  reports:
+    "hello";
+
+    any::
+      "world";
+}

{cfengine-0.16.1 → cfengine-0.16.3}/tests/format/006_remove_empty_comments.input.cf

@@ -1,3 +1,18 @@
+#
+#
+#
+#
+#
+# explanation
+#
+#
+#
+#
+# one more line
+#
+#
+#
+#
 bundle agent main
 {
 #
@@ -28,3 +43,38 @@ bundle agent c
 reports:
 "hello";
 }
+#
+bundle agent d
+{
+#
+reports:
+#
+"hello";
+#
+any::
+#
+"world";
+}
+
+#
+#
+#
+bundle agent e
+{
+#
+#
+#
+reports:
+#
+#
+#
+"hello";
+#
+#
+#
+any::
+#
+#
+#
+"world";
+}