chore: Rename and clean up internal datamodel service classes. (#4590)

## Context
Some internal classes name were strictly bound to Commands where they
could also be used for Query. We rename and make them usable by both.

## Change Summary
Renamed all internal datamodel classes so that they are generic.
'help' is removed which was earlier providing dynamic help for legacy
datamodel objects.

## Impact
No user side impact is anticipated.

Key Changes:

1. Renamed PyCommandArguments to PyArguments and all related
CommandArguments classes to just Arguments
2. Extended PyQuery to support argument instances similar to PyCommand
3. Removed duplicate help() methods from multiple classes

---------

Co-authored-by: pyansys-ci-bot <92810346+pyansys-ci-bot@users.noreply.github.com>

Author: prmukherj

devel/undocumented_fields/undocumented_fields.txt

@@ -200,7 +200,7 @@
     - path
     - rules
     - service
-  Class: PyCommandArgumentsSubItem
+  Class: PyArgumentsSubItem
     - getAttribValue
     - getState
     - setState

doc/changelog.d/4590.maintenance.md

@@ -0,0 +1 @@
+Rename and clean up internal datamodel service classes.

src/ansys/fluent/core/codegen/datamodelgen.py

@@ -37,7 +37,7 @@
     meshing_utility_examples,
 )
 from ansys.fluent.core.services.datamodel_se import (
-    PySingletonCommandArgumentsSubItem,
+    PyArgumentsSingletonSubItem,
     arg_class_by_type,
 )
 from ansys.fluent.core.utils.fix_doc import escape_wildcards
@@ -357,7 +357,7 @@ def _write_arg_class(self, f: FileIO, arg_info, indent: str):
         for line in arg_doc.splitlines():
             f.write(f"{indent}    {escape_wildcards(line)}\n")
         f.write(f'{indent}    """\n\n')
-        if arg_class == PySingletonCommandArgumentsSubItem:
+        if arg_class == PyArgumentsSingletonSubItem:
             f.write(
                 f"{indent}    def __init__(self, parent, attr, service, rules, path):\n"
             )
@@ -492,56 +492,55 @@ def _write_static_info(self, name: str, info: Any, f: FileIO, level: int = 0):
                         info["queries"][k]["queryinfo"],
                         file,
                     )
-        for k in commands:
-            f.write(f"{indent}    class {k}(PyCommand):\n")
-            f.write(f'{indent}        """\n')
-            command_static_info = info["commands"][k]
-            f.write(
-                _build_command_query_docstring(
-                    k, command_static_info, f"{indent}        ", True
+
+        def _write_static_command_and_query_info(
+            actions, class_name: str, st_info_key: tuple[str], is_command: bool
+        ):
+            for k in actions:
+                f.write(f"{indent}    class {k}({class_name}):\n")
+                f.write(f'{indent}        """\n')
+                actions_static_info = info[st_info_key[0]][k]
+                f.write(
+                    _build_command_query_docstring(
+                        k, actions_static_info, f"{indent}        ", is_command
+                    )
                 )
-            )
-            f.write(f'{indent}        """\n')
-            f.write(
-                f"{indent}        class _{k}CommandArguments(PyCommandArguments):\n"
-            )
-            f.write(
-                f"{indent}            def __init__(self, service, rules, command, path, id):\n"
-            )
-            f.write(
-                f"{indent}                super().__init__(service, rules, command, path, id)\n"
-            )
-            args_info = command_static_info["commandinfo"].get("args", [])
-            for arg_info in args_info:
-                arg_name = arg_info["name"]
-                py_name = _convert_to_py_name(arg_name)
+                f.write(f'{indent}        """\n')
+                f.write(f"{indent}        class _{k}Arguments(PyArguments):\n")
                 f.write(
-                    f'{indent}                self.{py_name} = self._{py_name}(self, "{arg_name}", service, rules, path)\n'
+                    f"{indent}            def __init__(self, service, rules, command, path, id):\n"
                 )
-            f.write("\n")
-            for arg_info in args_info:
-                self._write_arg_class(f, arg_info, f"{indent}            ")
+                f.write(
+                    f"{indent}                super().__init__(service, rules, command, path, id)\n"
+                )
+                args_info = actions_static_info[st_info_key[1]].get("args", [])
+                for arg_info in args_info:
+                    arg_name = arg_info["name"]
+                    py_name = _convert_to_py_name(arg_name)
+                    f.write(
+                        f'{indent}                self.{py_name} = self._{py_name}(self, "{arg_name}", service, rules, path)\n'
+                    )
+                f.write("\n")
+                for arg_info in args_info:
+                    self._write_arg_class(f, arg_info, f"{indent}            ")
 
-            f.write(
-                f"{indent}        def create_instance(self) -> _{k}CommandArguments:\n"
-            )
-            f.write(f"{indent}            args = self._get_create_instance_args()\n")
-            f.write(f"{indent}            if args is not None:\n")
-            f.write(
-                f"{indent}                return self._{k}CommandArguments(*args)\n\n"
-            )
-            api_tree[k] = "Command"
-        for k in queries:
-            f.write(f"{indent}    class {k}(PyQuery):\n")
-            f.write(f'{indent}        """\n')
-            f.write(
-                _build_command_query_docstring(
-                    k, info["queries"][k], f"{indent}        ", False
+                f.write(
+                    f"{indent}        def create_instance(self) -> _{k}Arguments:\n"
                 )
-            )
-            f.write(f'{indent}        """\n')
-            f.write(f"{indent}        pass\n\n")
-            api_tree[k] = "Query"
+                f.write(
+                    f"{indent}            args = self._get_create_instance_args()\n"
+                )
+                f.write(f"{indent}            if args is not None:\n")
+                f.write(f"{indent}                return self._{k}Arguments(*args)\n\n")
+                api_tree[k] = st_info_key[2]
+
+        _write_static_command_and_query_info(
+            commands, "PyCommand", ("commands", "commandinfo", "Command"), True
+        )
+        _write_static_command_and_query_info(
+            queries, "PyQuery", ("queries", "queryinfo", "Query"), False
+        )
+
         return api_tree
 
     def write_static_info(self) -> None:
@@ -566,12 +565,12 @@ def write_static_info(self) -> None:
                 f.write("    PyNamedObjectContainer,\n")
                 f.write("    PyCommand,\n")
                 f.write("    PyQuery,\n")
-                f.write("    PyCommandArguments,\n")
-                f.write("    PyTextualCommandArgumentsSubItem,\n")
-                f.write("    PyNumericalCommandArgumentsSubItem,\n")
-                f.write("    PyDictionaryCommandArgumentsSubItem,\n")
-                f.write("    PyParameterCommandArgumentsSubItem,\n")
-                f.write("    PySingletonCommandArgumentsSubItem\n")
+                f.write("    PyArguments,\n")
+                f.write("    PyArgumentsTextualSubItem,\n")
+                f.write("    PyArgumentsNumericalSubItem,\n")
+                f.write("    PyArgumentsDictionarySubItem,\n")
+                f.write("    PyArgumentsParameterSubItem,\n")
+                f.write("    PyArgumentsSingletonSubItem\n")
                 f.write(")\n\n\n")
                 api_tree_val = {
                     name: self._write_static_info("Root", info.static_info, f)