Support printing errors as JSON given --json (#524)

Signed-off-by: Juan Cruz Viotti <jv@jviotti.com>

Author: jviotti

docs/bundle.markdown

@@ -5,7 +5,7 @@ Bundling
 jsonschema bundle <schema.json|.yaml>
   [--http/-h] [--verbose/-v] [--resolve/-r <schemas-or-directories> ...]
   [--extension/-e <extension>] [--ignore/-i <schemas-or-directories>]
-  [--without-id/-w] [--default-dialect/-d <uri>]
+  [--without-id/-w] [--default-dialect/-d <uri>] [--json/-j]
 ```
 
 A schema may contain references to remote schemas outside the scope of the

docs/compile.markdown

@@ -8,7 +8,7 @@ Compiling
 jsonschema compile <schema.json|.yaml> [--http/-h] [--verbose/-v]
   [--resolve/-r <schemas-or-directories> ...] [--extension/-e <extension>]
   [--ignore/-i <schemas-or-directories>] [--fast/-f] [--default-dialect/-d <uri>]
-  [--minify/-m]
+  [--minify/-m] [--json/-j]
 ```
 
 The `validate` command will first compile the schema into an optimised

docs/decode.markdown

@@ -3,7 +3,7 @@ Decode
 
 ```sh
 jsonschema decode <output.binpack> <output.json|.jsonl>
-  [--default-dialect/-d <uri>]
+  [--default-dialect/-d <uri>] [--json/-j]
 ```
 
 This command decodes a JSON document using [JSON

docs/encode.markdown

@@ -3,7 +3,7 @@ Encode
 
 ```sh
 jsonschema encode <document.json|.jsonl> <output.binpack>
-  [--default-dialect/-d <uri>]
+  [--default-dialect/-d <uri>] [--json/-j]
 ```
 
 This command encodes a JSON document using [JSON

docs/format.markdown

@@ -4,8 +4,9 @@ Formatting
 ```sh
 jsonschema fmt [schemas-or-directories...]
   [--check/-c] [--verbose/-v] [--resolve/-r <schemas-or-directories> ...]
-  [--extension/-e <extension>] [--ignore/-i <schemas-or-directories>] 
+  [--extension/-e <extension>] [--ignore/-i <schemas-or-directories>]
   [--keep-ordering/-k] [--indentation/-n <spaces>] [--default-dialect/-d <uri>]
+  [--json/-j]
 ```
 
 Schemas are code. As such, they are expected follow consistent stylistic

docs/test.markdown

@@ -8,7 +8,7 @@ Testing
 jsonschema test [schemas-or-directories...]
   [--http/-h] [--verbose/-v] [--resolve/-r <schemas-or-directories> ...]
   [--extension/-e <extension>] [--ignore/-i <schemas-or-directories>]
-  [--default-dialect/-d <uri>]
+  [--default-dialect/-d <uri>] [--json/-j]
 ```
 
 Schemas are code. As such, you should run an automated unit testing suite

src/command_lint.cc

@@ -202,29 +202,34 @@ auto sourcemeta::jsonschema::lint(const sourcemeta::core::Options &options)
 
       auto copy = entry.second;
 
-      const auto wrapper_result = sourcemeta::jsonschema::try_catch([&]() {
-        try {
-          bundle.apply(
-              copy, sourcemeta::core::schema_official_walker, custom_resolver,
-              get_lint_callback(errors_array, entry, output_json), dialect,
-              sourcemeta::core::URI::from_path(entry.first).recompose());
-          return EXIT_SUCCESS;
-        } catch (const sourcemeta::core::SchemaTransformRuleProcessedTwiceError
-                     &error) {
-          throw LintAutoFixError{error.what(), entry.first, error.location()};
-        } catch (const sourcemeta::core::SchemaBrokenReferenceError &error) {
-          throw LintAutoFixError{
-              "Could not autofix the schema without breaking its internal "
-              "references",
-              entry.first, error.location()};
-        } catch (const sourcemeta::core::SchemaUnknownBaseDialectError &) {
-          throw FileError<sourcemeta::core::SchemaUnknownBaseDialectError>(
-              entry.first);
-        } catch (const sourcemeta::core::SchemaResolutionError &error) {
-          throw FileError<sourcemeta::core::SchemaResolutionError>(entry.first,
-                                                                   error);
-        }
-      });
+      const auto wrapper_result =
+          sourcemeta::jsonschema::try_catch(options, [&]() {
+            try {
+              bundle.apply(
+                  copy, sourcemeta::core::schema_official_walker,
+                  custom_resolver,
+                  get_lint_callback(errors_array, entry, output_json), dialect,
+                  sourcemeta::core::URI::from_path(entry.first).recompose());
+              return EXIT_SUCCESS;
+            } catch (
+                const sourcemeta::core::SchemaTransformRuleProcessedTwiceError
+                    &error) {
+              throw LintAutoFixError{error.what(), entry.first,
+                                     error.location()};
+            } catch (
+                const sourcemeta::core::SchemaBrokenReferenceError &error) {
+              throw LintAutoFixError{
+                  "Could not autofix the schema without breaking its internal "
+                  "references",
+                  entry.first, error.location()};
+            } catch (const sourcemeta::core::SchemaUnknownBaseDialectError &) {
+              throw FileError<sourcemeta::core::SchemaUnknownBaseDialectError>(
+                  entry.first);
+            } catch (const sourcemeta::core::SchemaResolutionError &error) {
+              throw FileError<sourcemeta::core::SchemaResolutionError>(
+                  entry.first, error);
+            }
+          });
 
       if (wrapper_result == EXIT_SUCCESS) {
         if (copy != entry.second) {
@@ -246,27 +251,28 @@ auto sourcemeta::jsonschema::lint(const sourcemeta::core::Options &options)
           resolver(options, options.contains("http"), dialect, configuration)};
       LOG_VERBOSE(options) << "Linting: " << entry.first.string() << "\n";
 
-      const auto wrapper_result = sourcemeta::jsonschema::try_catch([&]() {
-        try {
-          const auto subresult = bundle.check(
-              entry.second, sourcemeta::core::schema_official_walker,
-              custom_resolver,
-              get_lint_callback(errors_array, entry, output_json), dialect,
-              sourcemeta::core::URI::from_path(entry.first).recompose());
-          scores.emplace_back(subresult.second);
-          if (subresult.first) {
-            return EXIT_SUCCESS;
-          } else {
-            return EXIT_FAILURE;
-          }
-        } catch (const sourcemeta::core::SchemaUnknownBaseDialectError &) {
-          throw FileError<sourcemeta::core::SchemaUnknownBaseDialectError>(
-              entry.first);
-        } catch (const sourcemeta::core::SchemaResolutionError &error) {
-          throw FileError<sourcemeta::core::SchemaResolutionError>(entry.first,
-                                                                   error);
-        }
-      });
+      const auto wrapper_result =
+          sourcemeta::jsonschema::try_catch(options, [&]() {
+            try {
+              const auto subresult = bundle.check(
+                  entry.second, sourcemeta::core::schema_official_walker,
+                  custom_resolver,
+                  get_lint_callback(errors_array, entry, output_json), dialect,
+                  sourcemeta::core::URI::from_path(entry.first).recompose());
+              scores.emplace_back(subresult.second);
+              if (subresult.first) {
+                return EXIT_SUCCESS;
+              } else {
+                return EXIT_FAILURE;
+              }
+            } catch (const sourcemeta::core::SchemaUnknownBaseDialectError &) {
+              throw FileError<sourcemeta::core::SchemaUnknownBaseDialectError>(
+                  entry.first);
+            } catch (const sourcemeta::core::SchemaResolutionError &error) {
+              throw FileError<sourcemeta::core::SchemaResolutionError>(
+                  entry.first, error);
+            }
+          });
 
       if (wrapper_result != EXIT_SUCCESS) {
         result = false;