cel-expr · copybara-service · Jan 31, 2025 · Jan 31, 2025
diff --git a/parser/BUILD.bazel b/parser/BUILD.bazel
@@ -24,6 +24,11 @@ java_library(
    exports = ["//parser/src/main/java/dev/cel/parser:parser_builder"],
 )

+java_library(
+    name = "unparser_visitor",
+    exports = ["//parser/src/main/java/dev/cel/parser:unparser_visitor"],
+)
+
 java_library(
    name = "macro",
    exports = ["//parser/src/main/java/dev/cel/parser:macro"],

diff --git a/parser/src/main/java/dev/cel/parser/BUILD.bazel b/parser/src/main/java/dev/cel/parser/BUILD.bazel
@@ -94,14 +94,12 @@ java_library(
    deps = [
        ":operator",
        "//:auto_value",
-        "//common",
        "//common:compiler_common",
        "//common:source_location",
        "//common/ast",
        "//common/ast:expr_factory",
        "@maven//:com_google_errorprone_error_prone_annotations",
        "@maven//:com_google_guava_guava",
-        "@maven//:org_jspecify_jspecify",
    ],
 )


diff --git a/parser/src/main/java/dev/cel/parser/CelUnparserVisitor.java b/parser/src/main/java/dev/cel/parser/CelUnparserVisitor.java
@@ -319,7 +319,7 @@ private void visitIndex(CelCall expr, String op) {
    stringBuilder.append(RIGHT_BRACKET);
  }

-  private void visitMaybeNested(CelExpr expr, boolean nested) {
+  protected void visitMaybeNested(CelExpr expr, boolean nested) {
    if (nested) {
      stringBuilder.append(LEFT_PAREN);
    }
@@ -329,7 +329,7 @@ private void visitMaybeNested(CelExpr expr, boolean nested) {
    }
  }

-  private boolean isBinaryOrTernaryOperator(CelExpr expr) {
+  protected boolean isBinaryOrTernaryOperator(CelExpr expr) {
    if (!isComplexOperator(expr)) {
      return false;
    }