Merge branch 'main' into users/zhaodongwang/dataFrameExtensionClaude

zhaodongwang-msft · web-flow · commit 7a6fb2a81e97 · 2026-02-12T14:45:19.000-08:00
diff --git a/.claude/skills/dataverse-sdk-dev/SKILL.md b/.claude/skills/dataverse-sdk-dev/SKILL.md
@@ -26,4 +26,4 @@ This skill provides guidance for developers working on the PowerPlatform Dataver
 8. **No noqa comments** - Do not add `# noqa: BLE001` or similar linter suppression comments
 9. **Document public APIs** - Add Sphinx-style docstrings with examples for public methods
 10. **Define __all__ in module files, not __init__.py** - Use `__all__` to control exports in the actual module file (e.g., errors.py), not in `__init__.py`.
-11. **black formatting** - Use `black` for code formatting
+11. **Run black before committing** - Always run `python -m black <changed files>` before committing. CI will reject unformatted code. Config is in `pyproject.toml` under `[tool.black]`. And follow black formatting in general when coding.
diff --git a/examples/advanced/file_upload.py b/examples/advanced/file_upload.py
@@ -68,6 +68,7 @@ def log(call: str):
 _FILE_HASH_CACHE = {}
 ATTRIBUTE_VISIBILITY_DELAYS = (0, 3, 10, 20, 35, 50, 70, 90, 120)
 
+
 def file_sha256(path: Path):  # returns (hex_digest, size_bytes)
     try:
         m = _FILE_HASH_CACHE.get(path)
@@ -166,19 +167,15 @@ def backoff(op, *, delays=(0, 2, 5, 10, 20, 20)):
             result = op()
             if attempts > 1:
                 retry_count = attempts - 1
-                print(
-                    f"   [INFO] Backoff succeeded after {retry_count} retry(s); waited {total_delay}s total."
-                )
+                print(f"   [INFO] Backoff succeeded after {retry_count} retry(s); waited {total_delay}s total.")
             return result
         except Exception as ex:  # noqa: BLE001
             last = ex
             continue
     if last:
         if attempts:
             retry_count = max(attempts - 1, 0)
-            print(
-                f"   [WARN] Backoff exhausted after {retry_count} retry(s); waited {total_delay}s total."
-            )
+            print(f"   [WARN] Backoff exhausted after {retry_count} retry(s); waited {total_delay}s total.")
         raise last
 
 
@@ -337,9 +334,7 @@ def get_dataset_info(file_path: Path):
         )
         print({"chunk_upload_completed": True})
         odata = client._get_odata()
-        dl_url_chunk = (
-            f"{odata.api}/{entity_set}({record_id})/{chunk_file_attr_schema.lower()}/$value"  # raw entity_set for download
-        )
+        dl_url_chunk = f"{odata.api}/{entity_set}({record_id})/{chunk_file_attr_schema.lower()}/$value"  # raw entity_set for download
         resp_chunk = backoff(lambda: odata._request("get", dl_url_chunk))
         content_chunk = resp_chunk.content or b""
         import hashlib  # noqa: WPS433
diff --git a/examples/advanced/walkthrough.py b/examples/advanced/walkthrough.py
@@ -52,19 +52,15 @@ def backoff(op, *, delays=(0, 2, 5, 10, 20, 20)):
             result = op()
             if attempts > 1:
                 retry_count = attempts - 1
-                print(
-                    f"   [INFO] Backoff succeeded after {retry_count} retry(s); waited {total_delay}s total."
-                )
+                print(f"   [INFO] Backoff succeeded after {retry_count} retry(s); waited {total_delay}s total.")
             return result
         except Exception as ex:  # noqa: BLE001
             last = ex
             continue
     if last:
         if attempts:
             retry_count = max(attempts - 1, 0)
-            print(
-                f"   [WARN] Backoff exhausted after {retry_count} retry(s); waited {total_delay}s total."
-            )
+            print(f"   [WARN] Backoff exhausted after {retry_count} retry(s); waited {total_delay}s total.")
         raise last
 
 
@@ -339,7 +335,7 @@ def main():
         print(f"[OK] Deleted table: {table_name}")
     except Exception as ex:  # noqa: BLE001
         code = getattr(getattr(ex, "response", None), "status_code", None)
-        if (isinstance(ex, (requests.exceptions.HTTPError, MetadataError)) and code == 404):
+        if isinstance(ex, (requests.exceptions.HTTPError, MetadataError)) and code == 404:
             print(f"[OK] Table removed: {table_name}")
         else:
             raise
diff --git a/examples/basic/functional_testing.py b/examples/basic/functional_testing.py
@@ -91,22 +91,16 @@ def wait_for_table_metadata(
                 odata._entity_set_from_schema_name(table_schema_name)
 
                 if attempt > 1:
-                    print(
-                        f"   [OK] Table metadata available after {attempt} attempts."
-                    )
+                    print(f"   [OK] Table metadata available after {attempt} attempts.")
                 return info
         except Exception:
             pass
 
         if attempt < retries:
-            print(
-                f"   Waiting for table metadata to publish (attempt {attempt}/{retries})..."
-            )
+            print(f"   Waiting for table metadata to publish (attempt {attempt}/{retries})...")
             time.sleep(delay_seconds)
 
-    raise RuntimeError(
-        "Table metadata did not become available in time. Please retry later."
-    )
+    raise RuntimeError("Table metadata did not become available in time. Please retry later.")
 
 
 def ensure_test_table(client: DataverseClient) -> Dict[str, Any]:
@@ -229,9 +223,7 @@ def test_read_record(client: DataverseClient, table_info: Dict[str, Any], record
                 break
             except HttpError as err:
                 if getattr(err, "status_code", None) == 404 and attempt < retries:
-                    print(
-                        f"   Record not queryable yet (attempt {attempt}/{retries}). Retrying in {delay_seconds}s..."
-                    )
+                    print(f"   Record not queryable yet (attempt {attempt}/{retries}). Retrying in {delay_seconds}s...")
                     time.sleep(delay_seconds)
                     continue
                 raise
@@ -301,9 +293,7 @@ def test_query_records(client: DataverseClient, table_info: Dict[str, Any]) -> N
                 break
             except HttpError as err:
                 if getattr(err, "status_code", None) == 404 and attempt < retries:
-                    print(
-                        f"   Query retry {attempt}/{retries} after metadata 404 ({err}). Waiting {delay_seconds}s..."
-                    )
+                    print(f"   Query retry {attempt}/{retries} after metadata 404 ({err}). Waiting {delay_seconds}s...")
                     time.sleep(delay_seconds)
                     continue
                 raise
@@ -373,9 +363,7 @@ def cleanup_test_data(client: DataverseClient, table_info: Dict[str, Any], recor
                     print("[OK] Test table deleted successfully (404 reported).")
                     break
                 if attempt < retries:
-                    print(
-                        f"   Table delete retry {attempt}/{retries} after error ({err}). Waiting {delay_seconds}s..."
-                    )
+                    print(f"   Table delete retry {attempt}/{retries} after error ({err}). Waiting {delay_seconds}s...")
                     time.sleep(delay_seconds)
                     continue
                 print(f"[WARN] Failed to delete test table: {err}")
diff --git a/examples/basic/installation_example.py b/examples/basic/installation_example.py
@@ -173,8 +173,7 @@ def show_usage_examples():
     print("\nUsage Examples")
     print("=" * 50)
 
-    print(
-        """
+    print("""
 Basic Setup:
 ```python
 from PowerPlatform.Dataverse.client import DataverseClient
@@ -244,8 +243,7 @@ def show_usage_examples():
 tables = client.list_tables()
 print(f"Found {len(tables)} tables")
 ```
-"""
-    )
+""")
 
 
 def interactive_test():
diff --git a/src/PowerPlatform/Dataverse/claude_skill/dataverse-sdk-dev/SKILL.md b/src/PowerPlatform/Dataverse/claude_skill/dataverse-sdk-dev/SKILL.md
@@ -26,4 +26,4 @@ This skill provides guidance for developers working on the PowerPlatform Dataver
 8. **No noqa comments** - Do not add `# noqa: BLE001` or similar linter suppression comments
 9. **Document public APIs** - Add Sphinx-style docstrings with examples for public methods
 10. **Define __all__ in module files, not __init__.py** - Use `__all__` to control exports in the actual module file (e.g., errors.py), not in `__init__.py`.
-11. **black formatting** - Use `black` for code formatting
+11. **Run black before committing** - Always run `python -m black <changed files>` before committing. CI will reject unformatted code. Config is in `pyproject.toml` under `[tool.black]`. And follow black formatting in general when coding.
diff --git a/src/PowerPlatform/Dataverse/client.py b/src/PowerPlatform/Dataverse/client.py
@@ -737,19 +737,19 @@ def delete_table(self, table_schema_name: str) -> None:
         with self._scoped_odata() as od:
             od._delete_table(table_schema_name)
 
-    def list_tables(self) -> list[str]:
+    def list_tables(self) -> list[dict[str, Any]]:
         """
-        List all custom tables in the Dataverse environment.
+        List all non-private tables in the Dataverse environment.
 
-        :return: List of custom table names.
-        :rtype: :class:`list` of :class:`str`
+        :return: List of EntityDefinition metadata dictionaries.
+        :rtype: :class:`list` of :class:`dict`
 
         Example:
-            List all custom tables::
+            List all non-private tables and print their logical names::
 
                 tables = client.list_tables()
                 for table in tables:
-                    print(table)
+                    print(table["LogicalName"])
         """
         with self._scoped_odata() as od:
             return od._list_tables()
