Refactor $select parameter handling in _ODataClient and normalize empty select list in TableOperations

Author: maksii

src/PowerPlatform/Dataverse/data/_odata.py

@@ -1519,10 +1519,12 @@ def _get_table_metadata(
         params: Dict[str, str] = {}
         if select is not None and isinstance(select, str):
             raise TypeError("select must be a list of property names, not a bare string")
+        base_fields = {"MetadataId", "LogicalName", "SchemaName", "EntitySetName"}
         if select:
-            base_fields = {"MetadataId", "LogicalName", "SchemaName", "EntitySetName"}
             merged = list(base_fields | set(select))
-            params["$select"] = ",".join(merged)
+        else:
+            merged = list(base_fields)
+        params["$select"] = ",".join(merged)
         expand_parts: List[str] = []
         if include_attributes:
             expand_parts.append("Attributes")

src/PowerPlatform/Dataverse/operations/tables.py

@@ -208,6 +208,11 @@ def get(
             # Extended with relationships
             info = client.tables.get("account", include_relationships=True)
         """
+        # Normalize empty list to None so callers passing select=[] get the
+        # lightweight path instead of an expensive full-entity-definition fetch.
+        if select is not None and len(select) == 0:
+            select = None
+
         # When no extra parameters are passed, use the original lightweight lookup.
         # This ensures backward compatibility -- existing callers get identical behavior.
         if not include_columns and not include_relationships and select is None: