Only report exceeding node count once.

PiperOrigin-RevId: 955002671

Author: jnthntatum

parser/internal/pratt_parser_worker.cc

@@ -198,8 +198,9 @@ void ParserWorker::SynchronizeOnDelimiter() {
 
 int64_t ParserWorker::NextId(int32_t position) {
   int64_t id = next_id_++;
-  if (id > options_.expression_node_limit) {
+  if (id > options_.expression_node_limit && !node_limit_exceeded_) {
     ReportError(position, "expression node limit exceeded");
+    node_limit_exceeded_ = true;
   }
   if (position >= 0) {
     positions_.insert({id, position});
@@ -209,9 +210,7 @@ int64_t ParserWorker::NextId(int32_t position) {
 
 int64_t ParserWorker::NextId() { return NextId(-1); }
 
-bool ParserWorker::NodeLimitExceeded() {
-  return next_id_ > options_.expression_node_limit;
-}
+bool ParserWorker::NodeLimitExceeded() { return node_limit_exceeded_; }
 
 int64_t ParserWorker::CopyId(int64_t id) {
   if (id == 0) {

parser/internal/pratt_parser_worker.h

@@ -98,6 +98,7 @@ class ParserWorker {
   Token peek_token_;
   int recursion_depth_ = 0;
   int64_t next_id_ = 1;
+  bool node_limit_exceeded_ = false;
   absl::flat_hash_map<int64_t, int32_t> positions_;
   std::vector<cel::ParseIssue>* absl_nullable parse_issues_;
   int error_count_ = 0;