fix(npm): better error is version is specified after subpath

cli/node/mod.rs

@@ -447,7 +447,20 @@ pub fn node_resolve_npm_reference(
     NodeModuleKind::Esm,
   )
   .with_context(|| {
-    format!("Error resolving package config for '{}'.", reference)
+    let mut msg =
+      format!("Error resolving package config for '{}'.", reference);
+
+    if let Some(sub_path) = &reference.sub_path {
+      if let Some(at_index) = sub_path.rfind('@') {
+        let (sub_path, version) = sub_path.split_at(at_index);
+        msg = format!(
+          "{} Did you mean to write '{}{}/{}'?",
+          msg, reference.req, version, sub_path
+        );
+      }
+    }
+
+    msg
   })?;
 
   let url = ModuleSpecifier::from_file_path(resolved_path).unwrap();

cli/tests/integration/npm_tests.rs

@@ -188,6 +188,14 @@ itest!(require_json {
   http_server: true,
 });
 
+itest!(error_version_after_subpath {
+  args: "run --unstable -A --quiet npm/error_version_after_subpath/main.js",
+  output: "npm/error_version_after_subpath/main.out",
+  envs: env_vars(),
+  http_server: true,
+  exit_code: 1,
+});
+
 #[test]
 fn parallel_downloading() {
   let (out, _err) = util::run_and_collect_output_with_args(

cli/tests/testdata/npm/error_version_after_subpath/main.js

@@ -0,0 +1 @@
+import "npm:react-dom/server@18.2.0";

cli/tests/testdata/npm/error_version_after_subpath/main.out

@@ -0,0 +1,5 @@
+error: Could not resolve 'react-dom/server@18.2.0'.
+
+Caused by:
+    0: Error resolving package config for 'react-dom/server@18.2.0'. Did you mean to write 'react-dom@18.2.0/server'?
+    1: [ERR_PACKAGE_PATH_NOT_EXPORTED] Package subpath './server@18.2.0' is not defined by "exports" in [WILDCARD]

ext/node/errors.rs

@@ -81,23 +81,36 @@ pub fn err_invalid_package_target(
 }
 
 pub fn err_package_path_not_exported(
-  pkg_path: String,
+  mut pkg_path: String,
   subpath: String,
   maybe_referrer: Option<String>,
 ) -> AnyError {
   let mut msg = "[ERR_PACKAGE_PATH_NOT_EXPORTED]".to_string();
 
+  #[cfg(windows)]
+  {
+    if !pkg_path.ends_with('\\') {
+      pkg_path.push('\\');
+    }
+  }
+  #[cfg(not(windows))]
+  {
+    if !pkg_path.ends_with('/') {
+      pkg_path.push('/');
+    }
+  }
+
   if subpath == "." {
     msg = format!(
-      "{} No \"exports\" main defined in {}package.json",
+      "{} No \"exports\" main defined in '{}package.json'",
       msg, pkg_path
     );
   } else {
-    msg = format!("{} Package subpath \'{}\' is not defined by \"exports\" in {}package.json", msg, subpath, pkg_path);
+    msg = format!("{} Package subpath '{}' is not defined by \"exports\" in '{}package.json'", msg, subpath, pkg_path);
   };
 
   if let Some(referrer) = maybe_referrer {
-    msg = format!("{} imported from {}", msg, referrer);
+    msg = format!("{} imported from '{}'", msg, referrer);
   }
 
   generic_error(msg)