Feature/focus_modes (#14)

* missing deps only

* added focus modes and further improved test coverage

* fixed bug in test

Author: Joschi3

Readme.md

@@ -67,6 +67,8 @@ options:
   --skip-rosdep-key-validation  Check if rosdeps are valid.
   --compare-with-cmake          Check if all CMake dependencies are in package.xml.
   --auto-fill-missing-deps      Automatically fill missing dependencies in package.xml.
+  --missing-deps-only           Only report missing dependencies (implies --check-only).
+  --ignore-formatting-errors    Skip formatting-only checks (implies --check-only).
 ```
 Example with verbose logging:
 ```
@@ -143,3 +145,8 @@ This will:
 - Exit non-zero if any problems are found
 → **No files will be modified**
 - if rosdep is not available in the CI environment use the `--skip-rosdep-key-validation` flag
+
+### 🎯 Focused modes
+
+- `--missing-deps-only`: Skip all formatting and schema checks and only report missing dependencies (including launch/test dependencies).
+- `--ignore-formatting-errors`: Run all critical checks while ignoring pure formatting issues like indentation or ordering. This mode is check-only and will not rewrite files.

package_xml_validation/helpers/pkg_xml_formatter.py

@@ -198,14 +198,14 @@ def check_element_occurrences(self, root, xml_file):
             count = len(root.findall(elem))
             if count < min_occurrences:
                 self.logger.info(
-                    f"Error: Element '{elem}' in {xml_file} has fewer than {min_occurrences} occurrences."
+                    f"Error: Element <{elem}> in {xml_file} has fewer than {min_occurrences} occurrences."
                 )
                 incorrect_occurrences = True
                 if self.check_only:
                     return False
             elif max_occurrences is not None and count > max_occurrences:
                 self.logger.info(
-                    f"Error: Element '{elem}' in {xml_file} has more than {max_occurrences} occurrences."
+                    f"Error: Element <{elem}> in {xml_file} has more than {max_occurrences} occurrences."
                 )
                 incorrect_occurrences = True
                 if self.check_only:
@@ -219,12 +219,29 @@ def check_element_occurrences(self, root, xml_file):
 
         # Correct the occurrences
         for elem, min_occurrence, max_occurrences in ELEMENTS:
-            count = len(root.findall(elem))
+            elements = root.findall(elem)
+            count = len(elements)
             if max_occurrences is not None and count > max_occurrences:
-                for i in range(count - max_occurrences):
-                    root.remove(root.find(elem))
+                reference = (
+                    ET.tostring(elements[0], with_tail=False)
+                    if len(elements) > 0
+                    else None
+                )
+                if reference and all(
+                    ET.tostring(elm, with_tail=False) == reference
+                    for elm in elements[1:]
+                ):
+                    for extra in elements[max_occurrences:]:
+                        root.remove(extra)
+                    self.logger.info(
+                        f"Removed identical duplicate element <{elem}> from {xml_file}."
+                    )
+                else:
+                    self.logger.warning(
+                        f"Multiple <{elem}> entries differ. Please resolve manually. There are {count} occurrences in {xml_file} but there should be no more than {max_occurrences}."
+                    )
             if count < min_occurrence:
-                self.logger.info("Please add the missing element: ", elem)
+                self.logger.warning(f"Please add the missing element: <{elem}>")
         return False
 
     def check_element_order(self, root, xml_file):
@@ -431,7 +448,7 @@ def check_for_non_existing_tags(self, root, xml_file):
                 non_existing_tags.append(elem.tag)
         if non_existing_tags:
             self.logger.error(
-                f"Non-existing tags found in {xml_file}: {', '.join(non_existing_tags)}"
+                f"Unknown tags found in {xml_file}: {', '.join(non_existing_tags)}"
             )
             return False
         return True