fixup! Add error message handling to JSON parser

Author: himself65

src/json_parser.cc

@@ -1,5 +1,6 @@
 #include "json_parser.h"
 #include "debug_utils.h"
+#include <cstdio>
 
 namespace node {
 
@@ -12,20 +13,25 @@ bool JSONParser::Parse(const std::string& content) {
   size_t json_length = json_content_.size();
   json_content_.append(simdjson::SIMDJSON_PADDING, ' ');
 
-  simdjson::padded_string_view json_view(json_content_.data(),
-                                         json_length,
-                                         json_content_.size());
+  simdjson::padded_string_view json_view(
+      json_content_.data(), json_length, json_content_.size());
 
   simdjson::ondemand::document document;
   simdjson::error_code error = parser_.iterate(json_view).get(document);
 
   if (error != simdjson::SUCCESS) {
+    error_message_ = simdjson::error_message(error);
+    // Print error to stderr to match V8 behavior
+    std::fprintf(stderr, "JSON Parse error: %s\n", error_message_.c_str());
     return false;
   }
 
   simdjson::ondemand::object obj;
   error = document.get_object().get(obj);
   if (error != simdjson::SUCCESS) {
+    error_message_ = simdjson::error_message(error);
+    // Print error to stderr to match V8 behavior
+    std::fprintf(stderr, "JSON Parse error: %s\n", error_message_.c_str());
     return false;
   }
 
@@ -39,9 +45,10 @@ std::optional<std::string> JSONParser::GetTopLevelStringField(
     return {};
   }
 
-  simdjson::padded_string_view json_view(json_content_.data(),
-                                         json_content_.size() - simdjson::SIMDJSON_PADDING,
-                                         json_content_.size());
+  simdjson::padded_string_view json_view(
+      json_content_.data(),
+      json_content_.size() - simdjson::SIMDJSON_PADDING,
+      json_content_.size());
 
   simdjson::ondemand::document document;
   simdjson::error_code error = parser_.iterate(json_view).get(document);
@@ -69,9 +76,10 @@ std::optional<bool> JSONParser::GetTopLevelBoolField(std::string_view field) {
     return {};
   }
 
-  simdjson::padded_string_view json_view(json_content_.data(),
-                                         json_content_.size() - simdjson::SIMDJSON_PADDING,
-                                         json_content_.size());
+  simdjson::padded_string_view json_view(
+      json_content_.data(),
+      json_content_.size() - simdjson::SIMDJSON_PADDING,
+      json_content_.size());
 
   simdjson::ondemand::document document;
   simdjson::error_code error = parser_.iterate(json_view).get(document);
@@ -106,9 +114,10 @@ std::optional<JSONParser::StringDict> JSONParser::GetTopLevelStringDict(
     return std::nullopt;
   }
 
-  simdjson::padded_string_view json_view(json_content_.data(),
-                                         json_content_.size() - simdjson::SIMDJSON_PADDING,
-                                         json_content_.size());
+  simdjson::padded_string_view json_view(
+      json_content_.data(),
+      json_content_.size() - simdjson::SIMDJSON_PADDING,
+      json_content_.size());
 
   simdjson::ondemand::document document;
   simdjson::error_code error = parser_.iterate(json_view).get(document);

src/json_parser.h

@@ -22,10 +22,12 @@ class JSONParser {
   std::optional<std::string> GetTopLevelStringField(std::string_view field);
   std::optional<bool> GetTopLevelBoolField(std::string_view field);
   std::optional<StringDict> GetTopLevelStringDict(std::string_view field);
+  std::string GetErrorMessage() const { return error_message_; }
 
  private:
   simdjson::ondemand::parser parser_;
   std::string json_content_;
+  std::string error_message_;
   bool parsed_ = false;
 };
 }  // namespace node

test/parallel/test-single-executable-blob-config-errors.js

@@ -52,7 +52,8 @@ const assert = require('assert');
     });
   const stderr = child.stderr.toString();
   assert.strictEqual(child.status, 1);
-  assert.match(stderr, /SyntaxError: Expected ':' after property name/);
+  // The error message changed from V8's parser to simdjson
+  assert.match(stderr, /JSON Parse error: INCOMPLETE_ARRAY_OR_OBJECT/);
   assert(
     stderr.includes(
       `Cannot parse JSON from ${config}`