Stop indirect call warning for 'require.cache' for CommonJS modules

[huonw]

This follows/expands 4e503ba (#560) to suppress the warning about an indirect call to 'require' for require.cache, in addition to require.main, when building CommonJS modules. This is because manipulating/reading the cache shouldn't affect the overall bundling (AIUI).

I've validated that this suppresses the warning for the file from #812:

curl 'https://raw.githubusercontent.com/aws/aws-cdk/c6256992902fc4237ceb9f965e970e2c2ef00777/packages/%40aws-cdk/core/lib/private/runtime-info.ts' > runtime-info.ts
./esbuild --format=cjs runtime-info.ts

Output before:

 > runtime-info.ts: warning: Indirect calls to "require" will not be bundled (surround with a try/catch to silence this warning)
    16 │   for (const fileName of Object.keys(require.cache)) {
       ╵                                      ~~~~~~~

 > runtime-info.ts: warning: Indirect calls to "require" will not be bundled (surround with a try/catch to silence this warning)
    69 │   const mod = require.cache[fileName];
       ╵               ~~~~~~~

var __create = Object.create;
var __defProp = Object.defineProperty;
var __getProtoOf = Object.getPrototypeOf;
...

Output after:

var __create = Object.create;
var __defProp = Object.defineProperty;
var __getProtoOf = Object.getPrototypeOf;
...

(I'm not sure if I've updated the CHANGELOG correctly.)

Fixes #812

[UncleBill]

-				// These shouldn't warn
+				// These should warn

This should be a typo?