Full audit: fix Learn docstrings, add __all__, fix param doc, add AttributeOf test

Critical fixes:
- tables.py: Learn-incompatible ':class:\list\ of :class:\str\' -> 'list[str]'
- _odata.py, _relationships.py, _error_codes.py: add __all__ = [] per SKILL.md

Important fixes:
- _odata.py: _delete_multiple param 'logical_name' -> 'table_schema_name'
- _odata.py: _lowercase_list docstring removed misleading \ mention
- _odata.py: _query_sql guardrails comment clarified (defense-in-depth)

New test:
- test_excludes_attribute_of_columns: verifies computed display-name
  columns (with AttributeOf set) are filtered from sql_columns()

771 unit tests passing.

Author: Saurabh Badenkal

src/PowerPlatform/Dataverse/core/_error_codes.py

@@ -9,6 +9,8 @@
 SQL parsing errors, and metadata operation errors.
 """
 
+__all__ = []
+
 # HTTP subcode constants
 HTTP_400 = "http_400"
 HTTP_401 = "http_401"

src/PowerPlatform/Dataverse/data/_odata.py

@@ -5,6 +5,8 @@
 
 from __future__ import annotations
 
+__all__ = []
+
 from typing import Any, Dict, Optional, List, Union, Iterable, Callable
 from enum import Enum
 from dataclasses import dataclass, field
@@ -116,7 +118,7 @@ def _lowercase_keys(record: Dict[str, Any]) -> Dict[str, Any]:
     def _lowercase_list(items: Optional[List[str]]) -> Optional[List[str]]:
         """Convert all strings in a list to lowercase for case-insensitive column names.
 
-        Used for $select, $orderby, $expand parameters where column names must be lowercase.
+        Used for $select and $orderby parameters where column names must be lowercase.
         """
         if not items:
             return items
@@ -556,8 +558,8 @@ def _delete_multiple(
     ) -> Optional[str]:
         """Delete many records by GUID list via the ``BulkDelete`` action.
 
-        :param logical_name: Logical (singular) entity name.
-        :type logical_name: ``str``
+        :param table_schema_name: Schema name of the table.
+        :type table_schema_name: ``str``
         :param ids: GUIDs of records to delete.
         :type ids: ``list[str]``
 

src/PowerPlatform/Dataverse/data/_relationships.py

@@ -9,6 +9,8 @@
 
 from __future__ import annotations
 
+__all__ = []
+
 import re
 from typing import Any, Dict, List, Optional
 

src/PowerPlatform/Dataverse/operations/tables.py

@@ -825,7 +825,7 @@ def list_table_relationships(
         :type filter: :class:`str` or None
         :param select: Optional list of property names to project via
             ``$select``.  Values are passed as-is (PascalCase).
-        :type select: :class:`list` of :class:`str` or None
+        :type select: list[str] or None
 
         :return: Combined list of one-to-many, many-to-one, and many-to-many
             relationship metadata dictionaries.

tests/unit/test_query_operations.py

@@ -491,6 +491,32 @@ def test_empty_table(self):
         cols = self.client.query.sql_columns("account")
         self.assertEqual(cols, [])
 
+    def test_excludes_attribute_of_columns(self):
+        """Columns with AttributeOf set (computed display names) should be excluded."""
+        self._mock_columns(
+            [
+                {
+                    "LogicalName": "name",
+                    "AttributeType": "String",
+                    "IsPrimaryId": False,
+                    "IsPrimaryName": True,
+                    "DisplayName": {},
+                },
+                {
+                    "LogicalName": "createdbyname",
+                    "AttributeType": "String",
+                    "IsPrimaryId": False,
+                    "IsPrimaryName": False,
+                    "DisplayName": {},
+                    "AttributeOf": "createdby",
+                },
+            ]
+        )
+        cols = self.client.query.sql_columns("account")
+        names = [c["name"] for c in cols]
+        self.assertIn("name", names)
+        self.assertNotIn("createdbyname", names)
+
 
 class TestSqlSelect(unittest.TestCase):
     """Tests for client.query.sql_select()."""