report: add report versioning

This commit adds a version to the diagnostic report feature.

PR-URL: #28121
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>

Author: cjihrig

doc/api/report.md

@@ -22,6 +22,7 @@ is provided below for reference.
 ```json
 {
   "header": {
+    "reportVersion": 1,
     "event": "exception",
     "trigger": "Exception",
     "filename": "report.20181221.005011.8974.0.001.json",

src/node_report.cc

@@ -24,6 +24,7 @@
 extern char** environ;
 #endif
 
+constexpr int NODE_REPORT_VERSION = 1;
 constexpr int NANOS_PER_SEC = 1000 * 1000 * 1000;
 constexpr double SEC_PER_MICROS = 1e-6;
 
@@ -172,7 +173,7 @@ static void WriteNodeReport(Isolate* isolate,
   JSONWriter writer(out);
   writer.json_start();
   writer.json_objectstart("header");
-
+  writer.json_keyvalue("reportVersion", NODE_REPORT_VERSION);
   writer.json_keyvalue("event", message);
   writer.json_keyvalue("trigger", trigger);
   if (!filename.empty())

test/common/report.js

@@ -63,8 +63,10 @@ function _validateContent(data) {
                         'nodejsVersion', 'wordSize', 'arch', 'platform',
                         'componentVersions', 'release', 'osName', 'osRelease',
                         'osVersion', 'osMachine', 'cpus', 'host',
-                        'glibcVersionRuntime', 'glibcVersionCompiler', 'cwd'];
+                        'glibcVersionRuntime', 'glibcVersionCompiler', 'cwd',
+                        'reportVersion'];
   checkForUnknownFields(header, headerFields);
+  assert.strictEqual(header.reportVersion, 1);  // Increment as needed.
   assert.strictEqual(typeof header.event, 'string');
   assert.strictEqual(typeof header.trigger, 'string');
   assert(typeof header.filename === 'string' || header.filename === null);