almost done

Author: lgarber-akamai

linodecli/help_pages.py

@@ -298,30 +298,34 @@ def _help_group_arguments(
     """
     Returns help page groupings for a list of POST/PUT arguments.
     """
+    args = [arg for arg in args if not arg.read_only]
     args_sorted = sorted(args, key=lambda a: a.path)
-    arg_parents = set(arg.parent for arg in args_sorted)
 
-    groups_tmp = defaultdict(list)
+    paths = {tuple(arg.path.split(".")) for arg in args_sorted}
+    path_to_args = defaultdict(list)
 
-    # Initial grouping by root parent
     for arg in args_sorted:
-        if arg.read_only:
-            continue
+        arg_path = tuple(arg.path.split("."))
 
-        group_key = arg.path.split(".")[-1]
-        if arg.parent:
-            group_key = arg.parent
+        if not arg.is_parent:
+            # Parent arguments are grouped in with their children
+            arg_path = arg_path[:-1]
 
-            if arg.path in arg_parents:
-                group_key = arg.path
+        print(arg.path, arg.is_parent, arg_path)
+        # Find first common parent
+        while len(arg_path) > 1 and arg_path not in paths:
+            arg_path = arg_path[:-1]
 
-        groups_tmp[group_key].append(arg)
+        print(arg.path, arg.is_parent, arg_path)
+        path_to_args[arg_path].append(arg)
 
     group_required = []
     groups = []
     ungrouped = []
 
-    for group in groups_tmp.values():
+    for k, group in sorted(
+        path_to_args.items(), key=lambda a: (a[0], len(a[1]))
+    ):
         # If the group has more than one element,
         # leave it as is in the result
         if len(group) > 1: