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

  public abstract ValueString name();

+  public abstract Optional<ValueString> description();
+
+  public abstract Optional<ValueString> displayName();
+
  public abstract Rule rule();

  public abstract CelPolicySource policySource();
@@ -57,6 +61,10 @@ public abstract static class Builder {

    public abstract Builder setName(ValueString name);

+    public abstract Builder setDescription(ValueString description);
+
+    public abstract Builder setDisplayName(ValueString displayName);
+
    public abstract Builder setRule(Rule rule);

    public abstract Builder setPolicySource(CelPolicySource policySource);

diff --git a/policy/src/main/java/dev/cel/policy/CelPolicyYamlParser.java b/policy/src/main/java/dev/cel/policy/CelPolicyYamlParser.java
@@ -115,6 +115,12 @@ public CelPolicy parsePolicy(PolicyParserContext<Node> ctx, Node node) {
          case "name":
            policyBuilder.setName(ctx.newValueString(valueNode));
            break;
+          case "description":
+            policyBuilder.setDescription(ctx.newValueString(valueNode));
+            break;
+          case "display_name":
+            policyBuilder.setDisplayName(ctx.newValueString(valueNode));
+            break;
          case "rule":
            policyBuilder.setRule(parseRule(ctx, policyBuilder, valueNode));
            break;

diff --git a/policy/src/test/java/dev/cel/policy/CelPolicyYamlParserTest.java b/policy/src/test/java/dev/cel/policy/CelPolicyYamlParserTest.java
@@ -49,6 +49,39 @@ public void parser_setEmpty() throws Exception {
    assertThrows(CelPolicyValidationException.class, () -> POLICY_PARSER.parse("", ""));
  }

+  @Test
+  public void parseYamlPolicy_withDescription_atPolicyLevel() throws Exception {
+    String policySource =
+        "name: 'policy_with_description'\n"
+            + "description: 'this is a description of the policy'\n"
+            + "rule:\n"
+            + "   variables:\n"
+            + "   - name: 'variable_with_description'\n"
+            + "     description: 'this is a description of the variable'\n"
+            + "     expression: 'true'";
+
+    CelPolicy policy = POLICY_PARSER.parse(policySource);
+
+    assertThat(policy.description())
+        .hasValue(ValueString.of(5, "this is a description of the policy"));
+  }
+
+  @Test
+  public void parseYamlPolicy_withDisplayName_atPolicyLevel() throws Exception {
+    String policySource =
+        "name: 'policy_with_description'\n"
+            + "display_name: 'display name of the policy'\n"
+            + "rule:\n"
+            + "   variables:\n"
+            + "   - name: 'variable_with_description'\n"
+            + "     description: 'this is a description of the variable'\n"
+            + "     expression: 'true'";
+
+    CelPolicy policy = POLICY_PARSER.parse(policySource);
+
+    assertThat(policy.displayName()).hasValue(ValueString.of(5, "display name of the policy"));
+  }
+
  @Test
  public void parseYamlPolicy_withDescription() throws Exception {
    String policySource =