cfengine 0.16.0__tar.gz → 0.16.1__tar.gz

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

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

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

@@ -743,18 +743,23 @@ 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
         parent = child.parent
-        # Trailing comment at the end of a bundle body lands deeply
-        # indented (aligned with the deepest attribute); insert a blank
-        # line so it doesn't run into the preceding section.
+        # Comment at bundle_block_body level after a bundle_section —
+        # either a trailing comment for the previous section or a leading
+        # comment for the next one. Either way, visually separate it.
         if (
             prev.type == "bundle_section"
             and parent
             and parent.type == "bundle_block_body"
-            and _skip_comments(child.next_named_sibling, "next") is None
         ):
             return True
         if parent and parent.type in {"bundle_section", "class_guarded_promises"}:

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

@@ -144,3 +144,10 @@ body package_method cgcacgcac
     # bar
     package_changes => "bulk";
 }
+
+# Example here:
+bundle agent stigs
+{
+  vars:
+    "foo" string => "bar";
+}

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

@@ -133,3 +133,13 @@ package_changes => "bulk";
 # bar
 package_changes => "bulk";
 }
+
+#
+# Example here:
+#
+
+bundle agent stigs
+{
+  vars:
+    "foo" string => "bar";
+}

{cfengine-0.16.0 → cfengine-0.16.1}/tests/format/005_bundle_comments.expected.cf

@@ -104,3 +104,27 @@ bundle agent trailing_example_3
 
       # Trailing
 }
+
+bundle agent before_promise_types
+{
+  # Reports promises:
+  reports:
+    "Hello, world!";
+
+  # Classes:
+  classes:
+    "myclass" if => "foo";
+
+  # Variables:
+  # Multi line
+  vars:
+    "myvar"
+      string => "bar",
+      if => "foo";
+
+  # More reports:
+  reports:
+    "Lorem ipsum";
+
+    # Trailing comment
+}

{cfengine-0.16.0 → cfengine-0.16.1}/tests/format/005_bundle_comments.input.cf

@@ -101,3 +101,23 @@ if => "bar";
 
 # Trailing
 }
+
+bundle agent before_promise_types
+{
+# Reports promises:
+reports:
+"Hello, world!";
+# Classes:
+classes:
+"myclass" if => "foo";
+# Variables:
+# Multi line
+vars:
+"myvar"
+string => "bar",
+if => "foo";
+# More reports:
+reports:
+"Lorem ipsum";
+# Trailing comment
+}

{cfengine-0.16.0 → cfengine-0.16.1}/tests/format/011_promises.expected.cf

@@ -505,6 +505,7 @@ bundle common services_autorun
     services_autorun|services_autorun_inputs|services_autorun_bundles::
       "inputs" slist => { "$(sys.local_libdir)/autorun.cf" };
       "bundles" slist => { "autorun" };
+
   # run loaded bundles
   reports:
     DEBUG|DEBUG_services_autorun::