Address Copilot review round 4: unused imports, type annotations, docs

- Remove unused imports in test_batch_scenarios.py (json, _RecordCreate,
  _RecordDelete, _RecordUpdate, _QuerySql, _CRLF, BatchRequest)
- Remove unused imports in test_batch_dataframe.py (patch, BatchRecordOperations)
- Remove unused import in test_batch_edge_cases.py (patch)
- Fix BatchRequest._items type annotation: list -> List[Any]
- Fix example GUID: nonexistent-guid -> 00000000-0000-0000-0000-000000000000
- Fix SKILL.md docs: entity_ids includes creates AND updates, not just creates

Author: Saurabh Badenkal

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

@@ -407,7 +407,7 @@ print(f"Succeeded: {len(result.succeeded)}, Failed: {len(result.failed)}")
 - `result.succeeded` -- responses with 2xx status codes
 - `result.failed` -- responses with non-2xx status codes
 - `result.has_errors` -- True if any response failed
-- `result.entity_ids` -- GUIDs from successful create operations
+- `result.entity_ids` -- GUIDs from OData-EntityId headers (creates and updates)
 
 **Batch limitations:**
 - Maximum 1000 operations per batch

examples/advanced/batch.py

@@ -166,7 +166,7 @@
 print("\n[INFO] Example 6: Continue on error")
 
 batch = client.batch.new()
-batch.records.get("account", "nonexistent-guid-1111-1111-111111111111")
+batch.records.get("account", "00000000-0000-0000-0000-000000000000")
 batch.query.sql("SELECT TOP 1 name FROM account")
 
 result = batch.execute(continue_on_error=True)

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

@@ -407,7 +407,7 @@ print(f"Succeeded: {len(result.succeeded)}, Failed: {len(result.failed)}")
 - `result.succeeded` -- responses with 2xx status codes
 - `result.failed` -- responses with non-2xx status codes
 - `result.has_errors` -- True if any response failed
-- `result.entity_ids` -- GUIDs from successful create operations
+- `result.entity_ids` -- GUIDs from OData-EntityId headers (creates and updates)
 
 **Batch limitations:**
 - Maximum 1000 operations per batch

src/PowerPlatform/Dataverse/operations/batch.py

@@ -756,7 +756,7 @@ class BatchRequest:
 
     def __init__(self, client: "DataverseClient") -> None:
         self._client = client
-        self._items: list = []
+        self._items: List[Any] = []
         self._content_id_counter: List[int] = [1]  # shared across all changesets
         self.records = BatchRecordOperations(self)
         self.tables = BatchTableOperations(self)

tests/unit/data/test_batch_edge_cases.py

@@ -10,7 +10,7 @@
 
 import json
 import unittest
-from unittest.mock import MagicMock, patch
+from unittest.mock import MagicMock
 
 from PowerPlatform.Dataverse.data._batch import (
     _BatchClient,

tests/unit/test_batch_dataframe.py

@@ -4,13 +4,12 @@
 """Unit tests for BatchDataFrameOperations."""
 
 import unittest
-from unittest.mock import MagicMock, patch
+from unittest.mock import MagicMock
 
 import pandas as pd
 
 from PowerPlatform.Dataverse.operations.batch import (
     BatchDataFrameOperations,
-    BatchRecordOperations,
     BatchRequest,
 )
 from PowerPlatform.Dataverse.data._batch import _RecordCreate, _RecordUpdate, _RecordDelete

tests/unit/test_batch_scenarios.py

@@ -9,24 +9,17 @@
 These tests serve as executable documentation for SDK consumers.
 """
 
-import json
 import unittest
 from unittest.mock import MagicMock
 
 from PowerPlatform.Dataverse.data._batch import (
     _BatchClient,
     _ChangeSet,
-    _RecordCreate,
-    _RecordDelete,
     _RecordGet,
-    _RecordUpdate,
-    _QuerySql,
-    _CRLF,
 )
 from PowerPlatform.Dataverse.core.errors import ValidationError
 from PowerPlatform.Dataverse.data._raw_request import _RawRequest
 from PowerPlatform.Dataverse.models.batch import BatchItemResponse, BatchResult
-from PowerPlatform.Dataverse.operations.batch import BatchRequest
 
 
 def _make_od():