fix(publish): surface syntax errors when using --no-check (#24620)

.dprint.json

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

cli/tools/registry/diagnostics.rs

@@ -14,6 +14,7 @@ use deno_ast::diagnostics::DiagnosticSnippetHighlightStyle;
 use deno_ast::diagnostics::DiagnosticSourcePos;
 use deno_ast::diagnostics::DiagnosticSourceRange;
 use deno_ast::swc::common::util::take::Take;
+use deno_ast::ParseDiagnostic;
 use deno_ast::SourcePos;
 use deno_ast::SourceRange;
 use deno_ast::SourceRanged;
@@ -117,6 +118,7 @@ pub enum PublishDiagnostic {
     text_info: SourceTextInfo,
     range: SourceRange,
   },
+  SyntaxError(ParseDiagnostic),
 }
 
 impl PublishDiagnostic {
@@ -165,6 +167,7 @@ impl Diagnostic for PublishDiagnostic {
       ExcludedModule { .. } => DiagnosticLevel::Error,
       MissingConstraint { .. } => DiagnosticLevel::Error,
       BannedTripleSlashDirectives { .. } => DiagnosticLevel::Error,
+      SyntaxError { .. } => DiagnosticLevel::Error,
     }
   }
 
@@ -183,6 +186,7 @@ impl Diagnostic for PublishDiagnostic {
       BannedTripleSlashDirectives { .. } => {
         Cow::Borrowed("banned-triple-slash-directives")
       }
+      SyntaxError { .. } => Cow::Borrowed("syntax-error"),
     }
   }
 
@@ -203,6 +207,7 @@ impl Diagnostic for PublishDiagnostic {
       ExcludedModule { .. } => Cow::Borrowed("module in package's module graph was excluded from publishing"),
       MissingConstraint { specifier, .. } => Cow::Owned(format!("specifier '{}' is missing a version constraint", specifier)),
       BannedTripleSlashDirectives { .. } => Cow::Borrowed("triple slash directives that modify globals are not allowed"),
+      SyntaxError(diagnostic) => diagnostic.message(),
     }
   }
 
@@ -269,6 +274,7 @@ impl Diagnostic for PublishDiagnostic {
         source_pos: DiagnosticSourcePos::SourcePos(range.start),
         text_info: Cow::Borrowed(text_info),
       },
+      SyntaxError(diagnostic) => diagnostic.location(),
     }
   }
 
@@ -348,6 +354,7 @@ impl Diagnostic for PublishDiagnostic {
           description: Some("the triple slash directive".into()),
         }],
       }),
+      SyntaxError(diagnostic) => diagnostic.snippet(),
     }
   }
 
@@ -380,6 +387,7 @@ impl Diagnostic for PublishDiagnostic {
       BannedTripleSlashDirectives { .. } => Some(
         Cow::Borrowed("remove the triple slash directive"),
       ),
+      SyntaxError(diagnostic) => diagnostic.hint(),
     }
   }
 
@@ -407,7 +415,16 @@ impl Diagnostic for PublishDiagnostic {
           None => None,
         }
       }
-      _ => None,
+      SyntaxError(diagnostic) => diagnostic.snippet_fixed(),
+      FastCheck(_)
+      | SpecifierUnfurl(_)
+      | InvalidPath { .. }
+      | DuplicatePath { .. }
+      | UnsupportedFileType { .. }
+      | UnsupportedJsxTsx { .. }
+      | ExcludedModule { .. }
+      | MissingConstraint { .. }
+      | BannedTripleSlashDirectives { .. } => None,
     }
   }
 
@@ -456,6 +473,7 @@ impl Diagnostic for PublishDiagnostic {
         Cow::Borrowed("instead instruct the user of your package to specify these directives"),
         Cow::Borrowed("or set their 'lib' compiler option appropriately"),
       ]),
+      SyntaxError(diagnostic) => diagnostic.info(),
     }
   }
 
@@ -488,6 +506,7 @@ impl Diagnostic for PublishDiagnostic {
       BannedTripleSlashDirectives { .. } => Some(Cow::Borrowed(
         "https://jsr.io/go/banned-triple-slash-directives",
       )),
+      SyntaxError(diagnostic) => diagnostic.docs_url(),
     }
   }
 }

cli/tools/registry/graph.rs

@@ -147,6 +147,13 @@ impl GraphDiagnosticsCollector {
       let parsed_source = self
         .parsed_source_cache
         .get_parsed_source_from_js_module(module)?;
+
+      // surface syntax errors
+      for diagnostic in parsed_source.diagnostics() {
+        diagnostics_collector
+          .push(PublishDiagnostic::SyntaxError(diagnostic.clone()));
+      }
+
       check_for_banned_triple_slash_directives(
         &parsed_source,
         diagnostics_collector,

tests/specs/npm/lifecycle_scripts/no_deno_json.out

@@ -1,6 +1,7 @@
 [UNORDERED_START]
 Download http://localhost:4260/@denotest/lifecycle-scripts-cjs
 Download http://localhost:4260/@denotest/bin
+[UNORDERED_START]
 Download http://localhost:4260/@denotest/lifecycle-scripts-cjs/1.0.0.tgz
 Download http://localhost:4260/@denotest/bin/1.0.0.tgz
 Initialize @denotest/lifecycle-scripts-cjs@1.0.0

tests/specs/publish/no_check_surfaces_syntax_error/__test__.jsonc

@@ -0,0 +1,5 @@
+{
+  "args": "publish --dry-run --no-check",
+  "output": "publish.out",
+  "exitCode": 1
+}

tests/specs/publish/no_check_surfaces_syntax_error/deno.json

@@ -0,0 +1,5 @@
+{
+  "name": "@scope/pkg",
+  "version": "1.0.0",
+  "exports": "./mod.ts"
+}

tests/specs/publish/no_check_surfaces_syntax_error/mod.ts

@@ -0,0 +1 @@
++

tests/specs/publish/no_check_surfaces_syntax_error/publish.out

@@ -0,0 +1,8 @@
+Checking for slow types in the public API...
+error[syntax-error]: Expression expected
+ --> [WILDLINE]mod.ts:1:1
+  | 
+1 | +
+  | ^
+
+error: Found 1 problem