Merge pull request #152 from olehermanse/main

cfengine format: Fixed issue causing no empty line before comment + promise types

Author: olehermanse

src/cfengine_cli/format.py

@@ -747,14 +747,13 @@ def _needs_blank_line_before(child: Node, indent: int, line_length: int) -> bool
         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"}:

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
+}

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
+}

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::