fixup! Use emplace instead of indexing into _nodeAddons

Author: kraenhansen

packages/react-native-node-api-modules/cpp/CxxNodeApiHostModule.cpp

@@ -19,16 +19,17 @@ CxxNodeApiHostModule::requireNodeAddon(jsi::Runtime &rt,
   if (1 == count && args[0].isString()) {
     return thisModule.requireNodeAddon(rt, args[0].asString(rt));
   }
+  // TODO: Throw a meaningful error
   return jsi::Value::undefined();
 }
 
 jsi::Value CxxNodeApiHostModule::requireNodeAddon(jsi::Runtime &rt,
                                                   const jsi::String path) {
   const std::string pathStr = path.utf8(rt);
 
-  auto [it, inserted] = nodeAddons_.emplace(pathStr);
+  auto [it, inserted] = nodeAddons_.emplace(pathStr, NodeAddon());
   NodeAddon &addon = it->second;
-  
+
   // Check if this module has been loaded already, if not then load it...
   if (inserted) {
     if (!loadNodeAddon(addon, pathStr)) {

packages/react-native-node-api-modules/cpp/CxxNodeApiHostModule.hpp

@@ -35,4 +35,4 @@ class JSI_EXPORT CxxNodeApiHostModule : public facebook::react::TurboModule {
   bool initializeNodeModule(facebook::jsi::Runtime &rt, NodeAddon &addon);
 };
 
-} // namespace callstack::nodeapihost
+} // namespace callstack::nodeapihost