black formatting (also added it to SKILL.md

Author: Max Wang

.claude/skills/dataverse-sdk-dev/SKILL.md

@@ -25,4 +25,5 @@ This skill provides guidance for developers working on the PowerPlatform Dataver
 7. **Standardize output format** - Use `[INFO]`, `[WARN]`, `[ERR]`, `[OK]` prefixes for console output
 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`.
+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

src/PowerPlatform/Dataverse/claude_skill/dataverse-sdk-dev/SKILL.md

@@ -25,4 +25,5 @@ This skill provides guidance for developers working on the PowerPlatform Dataver
 7. **Standardize output format** - Use `[INFO]`, `[WARN]`, `[ERR]`, `[OK]` prefixes for console output
 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`.
+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

tests/unit/test_client_dataframe.py

@@ -189,10 +189,12 @@ def test_create_empty_dataframe(self):
 
     def test_create_converts_nan_to_none(self):
         """NaN values are converted to None in the payload."""
-        df = pd.DataFrame([
-            {"name": "Contoso", "telephone1": "555-0100"},
-            {"name": "Fabrikam", "telephone1": None},
-        ])
+        df = pd.DataFrame(
+            [
+                {"name": "Contoso", "telephone1": "555-0100"},
+                {"name": "Fabrikam", "telephone1": None},
+            ]
+        )
         self.client._odata._create_multiple.return_value = ["guid-1", "guid-2"]
         self.client._odata._entity_set_from_schema_name.return_value = "accounts"
 
@@ -277,10 +279,12 @@ def test_update_multiple_change_columns(self):
 
     def test_update_preserves_none_for_clearing_fields(self):
         """None values in update are kept as None to allow clearing fields in Dataverse."""
-        df = pd.DataFrame([
-            {"accountid": "guid-1", "name": "New Name", "telephone1": None},
-            {"accountid": "guid-2", "name": None, "telephone1": "555-0200"},
-        ])
+        df = pd.DataFrame(
+            [
+                {"accountid": "guid-1", "name": "New Name", "telephone1": None},
+                {"accountid": "guid-2", "name": None, "telephone1": "555-0200"},
+            ]
+        )
 
         self.client.update_dataframe("account", df, id_column="accountid")