Remove false-positive @odata.bind warning from _lowercase_keys

The runtime warning fired on legitimate lowercase system navigation
properties (ownerid, parentaccountid, transactioncurrencyid) that have
no underscore or are correctly all-lowercase. The real fix (preserving
@odata.bind key casing) is already in place — the SDK should not
second-guess what the caller passes.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: suyask-msft

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

@@ -373,7 +373,7 @@ except ValidationError as e:
 - Check filter/expand parameters use correct case
 - Verify column names exist and are spelled correctly
 - Ensure custom columns include customization prefix
-- For `@odata.bind` errors ("undeclared property"): the navigation property name before `@odata.bind` is case-sensitive and must match the entity's `$metadata` exactly (e.g., `new_CustomerId@odata.bind` for custom lookups, `parentaccountid@odata.bind` for system lookups). The SDK preserves `@odata.bind` key casing and emits a warning if it detects likely-wrong lowercase casing on custom lookups.
+- For `@odata.bind` errors ("undeclared property"): the navigation property name before `@odata.bind` is case-sensitive and must match the entity's `$metadata` exactly (e.g., `new_CustomerId@odata.bind` for custom lookups, `parentaccountid@odata.bind` for system lookups). The SDK preserves `@odata.bind` key casing.
 
 ## Best Practices
 

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

@@ -373,7 +373,7 @@ except ValidationError as e:
 - Check filter/expand parameters use correct case
 - Verify column names exist and are spelled correctly
 - Ensure custom columns include customization prefix
-- For `@odata.bind` errors ("undeclared property"): the navigation property name before `@odata.bind` is case-sensitive and must match the entity's `$metadata` exactly (e.g., `new_CustomerId@odata.bind` for custom lookups, `parentaccountid@odata.bind` for system lookups). The SDK preserves `@odata.bind` key casing and emits a warning if it detects likely-wrong lowercase casing on custom lookups.
+- For `@odata.bind` errors ("undeclared property"): the navigation property name before `@odata.bind` is case-sensitive and must match the entity's `$metadata` exactly (e.g., `new_CustomerId@odata.bind` for custom lookups, `parentaccountid@odata.bind` for system lookups). The SDK preserves `@odata.bind` key casing.
 
 ## Best Practices
 

src/PowerPlatform/Dataverse/data/_odata.py

@@ -13,7 +13,6 @@
 import re
 import json
 import uuid
-import warnings
 from datetime import datetime, timezone
 import importlib.resources as ir
 from contextlib import contextmanager
@@ -107,22 +106,6 @@ def _lowercase_keys(record: Dict[str, Any]) -> Dict[str, Any]:
         """
         if not isinstance(record, dict):
             return record
-        for k in record:
-            if isinstance(k, str) and "@odata.bind" in k:
-                nav_prop = k.split("@odata.bind")[0]
-                if nav_prop and nav_prop == nav_prop.lower() and "_" in nav_prop:
-                    # Likely already-lowercased navigation property name.
-                    # Custom lookup navigation properties use PascalCase
-                    # (e.g. new_CustomerId), not lowercase LogicalName.
-                    warnings.warn(
-                        f"@odata.bind key '{k}' appears to use a lowercase "
-                        f"navigation property name. Navigation property names "
-                        f"are case-sensitive and must match the entity's "
-                        f"$metadata (e.g. 'new_CustomerId@odata.bind', not "
-                        f"'new_customerid@odata.bind'). This will likely "
-                        f"cause a 400 error.",
-                        stacklevel=4,
-                    )
         return {k.lower() if isinstance(k, str) and "@odata." not in k else k: v for k, v in record.items()}
 
     @staticmethod

tests/unit/data/test_odata_internal.py

@@ -354,43 +354,6 @@ def test_odata_bind_keys_preserve_case(self):
         self.assertIn("new_CustomerId@odata.bind", payload)
         self.assertNotIn("new_customerid@odata.bind", payload)
 
-    def test_odata_bind_lowercase_warns(self):
-        """Lowercase @odata.bind nav property emits a warning."""
-        import warnings
-
-        with warnings.catch_warnings(record=True) as w:
-            warnings.simplefilter("always")
-            self.od._upsert(
-                "accounts",
-                "account",
-                {"accountnumber": "ACC-001"},
-                {
-                    "name": "Contoso",
-                    "new_customerid@odata.bind": "/contacts(00000000-0000-0000-0000-000000000001)",
-                },
-            )
-            odata_warnings = [x for x in w if "@odata.bind" in str(x.message)]
-            self.assertEqual(len(odata_warnings), 1)
-            self.assertIn("case-sensitive", str(odata_warnings[0].message))
-
-    def test_odata_bind_pascalcase_no_warning(self):
-        """PascalCase @odata.bind nav property does NOT emit a warning."""
-        import warnings
-
-        with warnings.catch_warnings(record=True) as w:
-            warnings.simplefilter("always")
-            self.od._upsert(
-                "accounts",
-                "account",
-                {"accountnumber": "ACC-001"},
-                {
-                    "name": "Contoso",
-                    "new_CustomerId@odata.bind": "/contacts(00000000-0000-0000-0000-000000000001)",
-                },
-            )
-            odata_warnings = [x for x in w if "@odata.bind" in str(x.message)]
-            self.assertEqual(len(odata_warnings), 0)
-
     def test_convert_labels_skips_odata_keys(self):
         """_convert_labels_to_ints should skip @odata.bind keys (no metadata lookup)."""
         # Patch _optionset_map to track calls