rename OperationContext field to user_agent_context

Author: arorashivam96

src/PowerPlatform/Dataverse/core/config.py

@@ -32,17 +32,17 @@ class OperationContext:
     (e.g. ``"app=myapp/1.0;agent=claude-code"``).  Free-form text, email
     addresses, and other potentially sensitive strings are rejected.
 
-    :param operation_context: Attribution string in ``key=value;key=value`` format.
-    :type operation_context: :class:`str`
+    :param user_agent_context: Attribution string in ``key=value;key=value`` format.
+    :type user_agent_context: :class:`str`
 
     :raises ValueError: If the string is empty, contains control characters, or
         does not match the required ``key=value`` format.
     """
 
-    operation_context: str
+    user_agent_context: str
 
     def __post_init__(self) -> None:
-        val = self.operation_context
+        val = self.user_agent_context
         if not val:
             raise ValueError("operation_context must not be empty.")
         if any(c in val for c in "\r\n\x00"):

src/PowerPlatform/Dataverse/data/_odata.py

@@ -207,7 +207,7 @@ def __init__(
             logger=self._http_logger,
         )
         ctx_obj = self.config.operation_context
-        self._operation_context = ctx_obj.operation_context if ctx_obj else None
+        self._operation_context = ctx_obj.user_agent_context if ctx_obj else None
         self._logical_to_entityset_cache: dict[str, str] = {}
         # Cache: normalized table_schema_name (lowercase) -> primary id attribute (e.g. accountid)
         self._logical_primaryid_cache: dict[str, str] = {}

tests/unit/test_operation_context.py

@@ -17,41 +17,41 @@ class TestOperationContextValidation(unittest.TestCase):
     """Tests for OperationContext format validation and PII rejection."""
 
     def test_valid_single_pair(self):
-        ctx = OperationContext(operation_context="app=test/1.0")
-        self.assertEqual(ctx.operation_context, "app=test/1.0")
+        ctx = OperationContext(user_agent_context="app=test/1.0")
+        self.assertEqual(ctx.user_agent_context, "app=test/1.0")
 
     def test_valid_multiple_pairs(self):
-        ctx = OperationContext(operation_context="app=test/1.0;skill=dv-data;agent=claude-code")
-        self.assertEqual(ctx.operation_context, "app=test/1.0;skill=dv-data;agent=claude-code")
+        ctx = OperationContext(user_agent_context="app=test/1.0;skill=dv-data;agent=claude-code")
+        self.assertEqual(ctx.user_agent_context, "app=test/1.0;skill=dv-data;agent=claude-code")
 
     def test_valid_with_dots_slashes_hyphens(self):
-        ctx = OperationContext(operation_context="app=dataverse-skills/1.2.1")
-        self.assertEqual(ctx.operation_context, "app=dataverse-skills/1.2.1")
+        ctx = OperationContext(user_agent_context="app=dataverse-skills/1.2.1")
+        self.assertEqual(ctx.user_agent_context, "app=dataverse-skills/1.2.1")
 
     def test_reject_empty(self):
         with self.assertRaises(ValueError):
-            OperationContext(operation_context="")
+            OperationContext(user_agent_context="")
 
     def test_reject_email(self):
         with self.assertRaises(ValueError):
-            OperationContext(operation_context="myname@email.com")
+            OperationContext(user_agent_context="myname@email.com")
 
     def test_reject_freeform_text(self):
         with self.assertRaises(ValueError):
-            OperationContext(operation_context="my bank password is 1234")
+            OperationContext(user_agent_context="my bank password is 1234")
 
     def test_reject_control_chars(self):
         for bad in ["has\rnewline", "has\nnewline", "has\x00null"]:
             with self.assertRaises(ValueError):
-                OperationContext(operation_context=bad)
+                OperationContext(user_agent_context=bad)
 
     def test_reject_spaces(self):
         with self.assertRaises(ValueError):
-            OperationContext(operation_context="app=my app")
+            OperationContext(user_agent_context="app=my app")
 
     def test_reject_no_equals(self):
         with self.assertRaises(ValueError):
-            OperationContext(operation_context="justaplainstring")
+            OperationContext(user_agent_context="justaplainstring")
 
 
 class TestOperationContextConfig(unittest.TestCase):
@@ -62,9 +62,9 @@ def test_default_is_none(self):
         self.assertIsNone(config.operation_context)
 
     def test_explicit_value(self):
-        ctx = OperationContext(operation_context="app=test/1.0;agent=claude-code")
+        ctx = OperationContext(user_agent_context="app=test/1.0;agent=claude-code")
         config = DataverseConfig(operation_context=ctx)
-        self.assertEqual(config.operation_context.operation_context, "app=test/1.0;agent=claude-code")
+        self.assertEqual(config.operation_context.user_agent_context, "app=test/1.0;agent=claude-code")
 
     def test_default_constructor_is_none(self):
         config = DataverseConfig()
@@ -79,14 +79,14 @@ def setUp(self):
         self.base_url = "https://example.crm.dynamics.com"
 
     def test_kwarg_sets_config(self):
-        ctx = OperationContext(operation_context="app=test/1.0;skill=dv-data;agent=claude-code")
+        ctx = OperationContext(user_agent_context="app=test/1.0;skill=dv-data;agent=claude-code")
         client = DataverseClient(
             self.base_url,
             self.mock_credential,
             context=ctx,
         )
         self.assertEqual(
-            client._config.operation_context.operation_context,
+            client._config.operation_context.user_agent_context,
             "app=test/1.0;skill=dv-data;agent=claude-code",
         )
 
@@ -95,22 +95,22 @@ def test_no_kwarg_leaves_config_default(self):
         self.assertIsNone(client._config.operation_context)
 
     def test_config_and_context_raises(self):
-        ctx = OperationContext(operation_context="app=test/1.0")
+        ctx = OperationContext(user_agent_context="app=test/1.0")
         config = DataverseConfig(operation_context=ctx)
         with self.assertRaises(ValueError):
             DataverseClient(
                 self.base_url,
                 self.mock_credential,
                 config=config,
-                context=OperationContext(operation_context="app=other/2.0"),
+                context=OperationContext(user_agent_context="app=other/2.0"),
             )
 
     def test_config_alone_works(self):
-        ctx = OperationContext(operation_context="app=test/1.0;agent=copilot")
+        ctx = OperationContext(user_agent_context="app=test/1.0;agent=copilot")
         config = DataverseConfig(operation_context=ctx)
         client = DataverseClient(self.base_url, self.mock_credential, config=config)
         self.assertEqual(
-            client._config.operation_context.operation_context,
+            client._config.operation_context.user_agent_context,
             "app=test/1.0;agent=copilot",
         )
 
@@ -132,7 +132,7 @@ def test_default_user_agent_unchanged(self):
 
     def test_operation_context_appended(self):
         ctx_str = "app=dataverse-skills/1.2.1;skill=dv-data;agent=claude-code"
-        ctx = OperationContext(operation_context=ctx_str)
+        ctx = OperationContext(user_agent_context=ctx_str)
         config = DataverseConfig(operation_context=ctx)
         odata = _ODataClient(self.dummy_auth, self.base_url, config=config)
         headers = odata._headers()
@@ -146,4 +146,4 @@ def test_none_context_no_parentheses(self):
 
     def test_empty_string_rejected_at_creation(self):
         with self.assertRaises(ValueError):
-            OperationContext(operation_context="")
+            OperationContext(user_agent_context="")