fix: enable the reporting of parsing related problems when running deno lint (#24332)

.dprint.json

@@ -41,6 +41,7 @@
     "tests/registry/",
     "tests/specs/fmt",
     "tests/specs/lint/bom",
+    "tests/specs/lint/syntax_error_reporting",
     "tests/specs/publish/no_check_surfaces_syntax_error",
     "tests/testdata/byte_order_mark.ts",
     "tests/testdata/encoding",

cli/tools/lint/mod.rs

@@ -646,7 +646,12 @@ fn handle_lint_result(
   let mut reporter = reporter_lock.lock();
 
   match result {
-    Ok((_source, mut file_diagnostics)) => {
+    Ok((source, mut file_diagnostics)) => {
+      if !source.diagnostics().is_empty() {
+        for parse_diagnostic in source.diagnostics() {
+          log::warn!("{}: {}", colors::yellow("warn"), parse_diagnostic);
+        }
+      }
       file_diagnostics.sort_by(|a, b| match a.specifier.cmp(&b.specifier) {
         std::cmp::Ordering::Equal => a.range.start.cmp(&b.range.start),
         file_order => file_order,

tests/specs/lint/syntax_error_reporting/__test__.jsonc

@@ -0,0 +1,5 @@
+{
+  "args": "lint script.ts",
+  "output": "lint.out",
+  "exitCode": 1
+}

tests/specs/lint/syntax_error_reporting/lint.out

@@ -0,0 +1,16 @@
+warn: Unterminated string constant at [WILDCARD]script.ts:1:13
+
+  const foo = 'bar
+              ~~~~
+error[no-unused-vars]: `foo` is never used
+ --> [WILDCARD]script.ts:1:7
+  | 
+1 | const foo = 'bar
+  |       ^^^
+  = hint: If this is intentional, prefix it with an underscore like `_foo`
+
+  docs: https://lint.deno.land/rules/no-unused-vars
+
+
+Found 1 problem
+Checked 1 file

tests/specs/lint/syntax_error_reporting/script.ts

@@ -0,0 +1 @@
+const foo = 'bar