Non-cyclic modules are not waited for

[Jamesernator]

In InnerModuleEvaluation Step 1. we have:

1. If module is not a Cyclic Module Record, then
  a. Perform ? module.Evaluate().
  b. Return index.

However as per the changes to Abstract Module Record, .Evaluate() always returns a promise. Previously this step would've thrown the resulting error, but now it simply returns a promise which may reject. As such we need to handle rejections.

This will affect JSON modules which are not cyclic module records:

// works even if "./config.json" has a parsing error
import config from "./config.json" assert { type: "json" };

console.log(config);

[Jamesernator]

Realised that the implication is larger than just errors, basically the result of .Evaluate() is not awaited at all. This means if it takes a long time to resolve a cyclic module might execute before .Evaluate() is even finished. As such the value might not even be ready.