cleanup

Author: tpellissier

README.md

@@ -5,7 +5,7 @@ A minimal Python SDK to use Microsoft Dataverse as a database for Azure AI Found
 - Read (SQL) — Execute read-only T‑SQL via the McpExecuteSqlQuery Custom API. Returns `list[dict]`.
 - OData CRUD — Thin wrappers over Dataverse Web API (create/get/update/delete).
 - Bulk create — Pass a list of records to `create(...)` to invoke the bound `CreateMultiple` action; returns `list[str]` of GUIDs. If `@odata.type` is absent the SDK resolves the logical name from metadata (cached).
-- Bulk update — Call `update_multiple(entity_set, records)` to invoke the bound `UpdateMultiple` action; returns nothing (transactional fire-and-forget). Each record must include the real primary key attribute (e.g. `accountid`).
+- Bulk update — Call `update_multiple(entity_set, records)` to invoke the bound `UpdateMultiple` action; returns nothing. Each record must include the real primary key attribute (e.g. `accountid`).
 - Retrieve multiple (paging) — Generator-based `get_multiple(...)` that yields pages, supports `$top` and Prefer: `odata.maxpagesize` (`page_size`).
 - Metadata helpers — Create/inspect/delete simple custom tables (EntityDefinitions + Attributes).
 - Pandas helpers — Convenience DataFrame oriented wrappers for quick prototyping/notebooks.
@@ -135,7 +135,7 @@ print({"created_ids": ids})
 
 ## Bulk update (UpdateMultiple)
 
-Use `update_multiple(entity_set, records)` for a transactional batch update. The method returns `None` regardless of service response; this avoids exposing inconsistent behavior across environments that may or may not emit updated IDs.
+Use `update_multiple(entity_set, records)` for a transactional batch update. The method returns `None`.
 
 ```python
 ids = client.create("accounts", [
@@ -283,7 +283,7 @@ VS Code Tasks
 
 ## Limitations / Future Work
 - No general-purpose OData batching, upsert, or association operations yet.
-- `DeleteMultiple` not yet exposed; `UpdateMultiple` is available but returns no IDs (fire-and-forget semantics in this SDK version).
+- `DeleteMultiple` not yet exposed.
 - Minimal retry policy in library (network-error only); examples include additional backoff for transient Dataverse consistency.
 - Entity naming conventions in Dataverse: for multi-create the SDK resolves logical names from entity set metadata.
 

examples/quickstart.py

@@ -305,9 +305,8 @@ def print_line_summaries(label: str, summaries: list[dict]) -> None:
 		backoff_retry(lambda: client.update_multiple(entity_set, bulk_updates))
 		print({"bulk_update_requested": len(bulk_updates), "bulk_update_completed": True})
 		# Verify the updated count values by refetching the subset
-		# (Individual gets keep the example simple and avoid crafting a complex $filter.)
 		verification = []
-		# Small delay to reduce risk of any brief replication delay (usually unnecessary)
+		# Small delay to reduce risk of any brief replication delay
 		time.sleep(1)
 		for rid in subset:
 			rec = backoff_retry(lambda rid=rid: client.get(entity_set, rid))

src/dataverse_sdk/client.py

@@ -123,8 +123,7 @@ def update_multiple(self, entity: str, records: List[Dict[str, Any]]) -> None:
         Returns
         -------
         None
-            On success returns nothing. The underlying action does not reliably emit updated IDs across
-            environments; for predictable semantics the SDK returns None.
+            On success returns nothing.
         """
         self._get_odata().update_multiple(entity, records)
         return None

src/dataverse_sdk/odata.py

@@ -250,11 +250,7 @@ def update_multiple(self, entity_set: str, records: List[Dict[str, Any]]) -> Non
         headers = self._headers().copy()
         r = self._request("post", url, headers=headers, json=payload)
         r.raise_for_status()
-        try:
-            body = r.json() if r.text else {}
-        except ValueError:
-            body = {}
-        # Intentionally ignore response content: no stable contract for IDs across environments.
+       # Intentionally ignore response content: no stable contract for IDs across environments.
         return None
 
     def delete(self, entity_set: str, key: str) -> None: