cel-expr · copybara-service · Jul 8, 2025 · Jul 8, 2025
diff --git a/policy/src/main/java/dev/cel/policy/CelPolicy.java b/policy/src/main/java/dev/cel/policy/CelPolicy.java
@@ -234,6 +234,8 @@ public abstract static class Variable {

    public abstract Optional<ValueString> description();

+    public abstract Optional<ValueString> displayName();
+
    /** Builder for {@link Variable}. */
    @AutoValue.Builder
    public abstract static class Builder implements RequiredFieldsChecker {
@@ -244,12 +246,16 @@ public abstract static class Builder implements RequiredFieldsChecker {

      abstract Optional<ValueString> description();

+      abstract Optional<ValueString> displayName();
+
      public abstract Builder setName(ValueString name);

      public abstract Builder setExpression(ValueString expression);

      public abstract Builder setDescription(ValueString description);

+      public abstract Builder setDisplayName(ValueString displayName);
+
      @Override
      public ImmutableList<RequiredField> requiredFields() {
        return ImmutableList.of(

diff --git a/policy/src/main/java/dev/cel/policy/CelPolicyYamlParser.java b/policy/src/main/java/dev/cel/policy/CelPolicyYamlParser.java
@@ -293,6 +293,9 @@ public CelPolicy.Variable parseVariable(
          case "description":
            builder.setDescription(ctx.newValueString(valueNode));
            break;
+          case "display_name":
+            builder.setDisplayName(ctx.newValueString(valueNode));
+            break;
          default:
            tagVisitor.visitVariableTag(ctx, keyId, keyName, valueNode, policyBuilder, builder);
            break;

diff --git a/policy/src/test/java/dev/cel/policy/CelPolicyYamlParserTest.java b/policy/src/test/java/dev/cel/policy/CelPolicyYamlParserTest.java
@@ -65,6 +65,22 @@ public void parseYamlPolicy_withDescription() throws Exception {
        .hasValue(ValueString.of(10, "this is a description of the variable"));
  }

+  @Test
+  public void parseYamlPolicy_withDisplayName() throws Exception {
+    String policySource =
+        "rule:\n"
+            + "  variables:\n"
+            + "  - name: 'variable_with_description'\n"
+            + "    display_name: 'Display Name'\n"
+            + "    expression: 'true'";
+
+    CelPolicy policy = POLICY_PARSER.parse(policySource);
+
+    assertThat(policy.rule().variables()).hasSize(1);
+    assertThat(Iterables.getOnlyElement(policy.rule().variables()).displayName())
+        .hasValue(ValueString.of(10, "Display Name"));
+  }
+
  @Test
  public void parseYamlPolicy_withExplanation() throws Exception {
    String policySource =