fix: GenAI Client(evals) - Map legacy RubricMetric.GROUNDEDNESS to grounding_v1

PiperOrigin-RevId: 919752952

Author: jsondai

agentplatform/_genai/_evals_metric_loaders.py

@@ -355,6 +355,23 @@ def FINAL_RESPONSE_QUALITY(self) -> LazyLoadedPrebuiltMetric:
     def HALLUCINATION(self) -> LazyLoadedPrebuiltMetric:
         return self.__getattr__("HALLUCINATION")
 
+    @property
+    def GROUNDING(self) -> LazyLoadedPrebuiltMetric:  # pylint: disable=invalid-name
+        return self.__getattr__("GROUNDING", version="v1")
+
+    @property
+    def GROUNDEDNESS(self) -> LazyLoadedPrebuiltMetric:  # pylint: disable=invalid-name
+        logger.warning(
+            "RubricMetric.GROUNDEDNESS is a deprecated alias and now maps to"
+            " RubricMetric.GROUNDING (grounding_v1). Note that the input"
+            " contract changed: legacy GROUNDEDNESS scored 'response' against"
+            " 'prompt'; grounding_v1 scores 'response' sentence-by-sentence"
+            " against an additional 'context' field. Add a 'context' field to"
+            " your dataset, otherwise scores will silently collapse to 0."
+            " Update your code to use RubricMetric.GROUNDING directly."
+        )
+        return self.__getattr__("GROUNDING", version="v1")
+
     @property
     def TOOL_USE_QUALITY(self) -> LazyLoadedPrebuiltMetric:
         return self.__getattr__("TOOL_USE_QUALITY")

tests/unit/agentplatform/genai/test_evals.py

@@ -6355,6 +6355,37 @@ def test_metric_to_yaml_file_raises_importerror_if_yaml_is_none(self):
             metric_obj.to_yaml_file("/fake/path/error.yaml")
 
 
+class TestPrebuiltMetricLoaderGroundedness:
+    """Unit tests for legacy RubricMetric.GROUNDEDNESS alias to grounding_v1."""
+
+    def test_grounding_resolves_to_grounding_v1(self):
+        lazy_metric = agentplatform_genai_types.RubricMetric.GROUNDING
+        assert lazy_metric.name == "GROUNDING"
+        assert lazy_metric._get_api_metric_spec_name() == "grounding_v1"
+
+    def test_groundedness_aliases_grounding_v1(self):
+        lazy_metric = agentplatform_genai_types.RubricMetric.GROUNDEDNESS
+        assert lazy_metric.name == "GROUNDING"
+        assert lazy_metric._get_api_metric_spec_name() == "grounding_v1"
+
+    def test_groundedness_logs_field_difference_warning(self, caplog):
+        loader_logger = (
+            "agentplatform._genai._evals_metric_loaders"
+        )
+        with caplog.at_level("WARNING", logger=loader_logger):
+            _ = agentplatform_genai_types.RubricMetric.GROUNDEDNESS
+        messages = [r.getMessage() for r in caplog.records if r.name == loader_logger]
+        assert any("GROUNDEDNESS" in m for m in messages)
+        assert any("grounding_v1" in m for m in messages)
+        assert any("context" in m for m in messages)
+
+    def test_groundedness_resolve_returns_grounding_v1_metric(self):
+        lazy_metric = agentplatform_genai_types.RubricMetric.GROUNDEDNESS
+        resolved = lazy_metric.resolve(api_client=mock.MagicMock())
+        assert isinstance(resolved, agentplatform_genai_types.Metric)
+        assert resolved.name == "grounding_v1"
+
+
 class TestMergeResponseDatasets:
     """Unit tests for the merge_response_datasets_into_canonical_format function."""