fix(npm): don't fail if conditional exports don't contains types (#16651)

If resolving types for an npm package, we didn't find "types" entry in 
the conditional exports declaration we were falling-through to regular 
resolution, instead of short-circuiting and giving up on resolving
types, which might lead to unwarranted errors.

Closes https://github.com/denoland/deno/issues/16649

cli/node/mod.rs

@@ -688,6 +688,8 @@ fn package_config_resolve(
         legacy_main_resolve(&package_config, referrer_kind, conditions)
       {
         return Ok(Some(path));
+      } else {
+        return Ok(None);
       }
     }
     return package_exports_resolve(

cli/tests/integration/npm_tests.rs

@@ -263,6 +263,14 @@ itest!(types_ambient_module_import_map {
   exit_code: 1,
 });
 
+itest!(no_types_in_conditional_exports {
+  args: "run --check --unstable npm/no_types_in_conditional_exports/main.ts",
+  output: "npm/no_types_in_conditional_exports/main.out",
+  exit_code: 0,
+  envs: env_vars(),
+  http_server: true,
+});
+
 #[test]
 fn parallel_downloading() {
   let (out, _err) = util::run_and_collect_output_with_args(

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

@@ -0,0 +1,5 @@
+Download http://localhost:4545/npm/registry/@denotest/no-types-in-conditional-exports
+Download http://localhost:4545/npm/registry/@denotest/no-types-in-conditional-exports/1.0.0.tgz
+Check [WILDCARD]npm/no_types_in_conditional_exports/main.ts
+[WILDCARD]
+{ foo: "bar" }

cli/tests/testdata/npm/no_types_in_conditional_exports/main.ts

@@ -0,0 +1,2 @@
+import foo from "npm:@denotest/no-types-in-conditional-exports@1.0.0";
+console.log(foo);

cli/tests/testdata/npm/registry/@denotest/no-types-in-conditional-exports/1.0.0/lib/foo-esm.js

@@ -0,0 +1,3 @@
+export default {
+    "foo": "bar"
+};

cli/tests/testdata/npm/registry/@denotest/no-types-in-conditional-exports/1.0.0/lib/foo.js

@@ -0,0 +1,3 @@
+module.exports = {
+    "foo": "bar"
+};

cli/tests/testdata/npm/registry/@denotest/no-types-in-conditional-exports/1.0.0/package.json

@@ -0,0 +1,14 @@
+{
+  "name": "@denotest/no-types-in-conditional-exports",
+  "version": "1.0.0",
+  "main": "./lib/foo.js",
+  "module": "./lib/foo-esm.js",
+  "exports": {
+    ".": {
+      "require": "./lib/foo.js",
+      "import": "./lib/foo-esm.js"
+    },
+    "./*": "./*"
+  },
+  "type": "module"
+}