chore(compartment-mapper): fix type of internal function

[boneskull]

This addresses the @ts-ignore found in makeFauxModuleExportsNamespace(), which was renamed from makeFeauxModuleExportsNamespace(), because "féaux" translates to "loyal supporters" and not "fake" or "false" like was presumably intended.

I am not so bold to rename it makeErsatzModuleExportsNamespace(), despite "ersatz" being a wholly superior word to either of the above alternatives.

Anyway, this adds a proper type signature and fixes the return type; the importHook prop of the returned value did not adhere to the definition of ImportHook. Likewise, StaticModuleType demands an exports array.

Given this apparently worked just fine before I changed it, one could argue the types ought to be loosened somewhat.

---

• No, I didn't make an issue
• I happened upon the code in question, so I fixed it due to my personal compulsions
• No security concerns
• No scaling concerns
• Modifies a private API
• I have low confidence that @kriskowal did not name this function intentionally. If he did, it went over my head.

[kriskowal]

Please consider makeFauxBespokeErsatzModuleExportsNamespace. And, don’t let your guard down. I do occasionally leave language trust-falls. One time, someone (rightly) corrected “treble” to “triple” in one of my design docs and @peats-bond did me a solid: “resolved: won’t fix”.