refactor(runtime): drop the require() optional-module placeholder

Author: NathanWalker

test-app/runtime/src/main/cpp/ModuleInternal.cpp

@@ -86,16 +86,6 @@ static std::string PromiseRejectionMessage(Isolate* isolate, Local<Promise> prom
     return errorMessage;
 }
 
-// Helper function to check if a module name looks like an optional external module
-bool ModuleInternal::IsLikelyOptionalModule(const std::string& moduleName) {
-    // Check if it's a bare module name (no path separators) that could be an npm package
-    if (moduleName.find('/') == std::string::npos && moduleName.find('\\') == std::string::npos &&
-        moduleName[0] != '.' && moduleName[0] != '~' && moduleName[0] != '/') {
-        return true;
-    }
-    return false;
-}
-
 // A package-style specifier: neither a path nor a scheme, so it may be claimed
 // by a registry rather than resolved on disk.
 static bool IsBareSpecifier(const std::string& specifier) {

test-app/runtime/src/main/cpp/ModuleInternal.h

@@ -38,7 +38,6 @@ class ModuleInternal {
         static void CheckFileExists(v8::Isolate* isolate, const std::string& path, const std::string& baseDir);
 
         // Helper functions for ES module support
-        static bool IsLikelyOptionalModule(const std::string& moduleName);
         static bool IsESModule(const std::string& path);
         static v8::Local<v8::Value> LoadESModule(v8::Isolate* isolate, const std::string& path);