fix(runtime): entered context in Runtime entry points

Added Context::Scope to Runtime::RunMainScript, Runtime::RunModule, and Runtime::RunScript to ensure the runtime's context is explicitly entered before delegating to ModuleInternal.

While the context is already entered once in Runtime::Init and remains current for the runtime's lifetime, this change adds a defensive context scope at each entry point for consistency and future-proofing.

Author: adrian-niculescu

NativeScript/runtime/Runtime.mm

@@ -22,15 +22,15 @@
 #include "DisposerPHV.h"
 #include "IsolateWrapper.h"
 
+#include <mutex>
 #include <unordered_map>
+#include "DevFlags.h"
+#include "HMRSupport.h"
 #include "ModuleBinding.hpp"
 #include "ModuleInternalCallbacks.h"
 #include "URLImpl.h"
 #include "URLPatternImpl.h"
 #include "URLSearchParamsImpl.h"
-#include <mutex>
-#include "HMRSupport.h"
-#include "DevFlags.h"
 
 #define STRINGIZE(x) #x
 #define STRINGIZE_VALUE_OF(x) STRINGIZE(x)
@@ -128,7 +128,7 @@ static void InitializeImportMetaObject(Local<Context> context, Local<Module> mod
 std::atomic<int> Runtime::nextIsolateId{0};
 SimpleAllocator allocator_;
 NSDictionary* AppPackageJson = nil;
-static std::unordered_map<std::string, id> AppConfigCache; // generic cache for app config values
+static std::unordered_map<std::string, id> AppConfigCache;  // generic cache for app config values
 static std::mutex AppConfigCacheMutex;
 
 // Global flag to track when JavaScript errors occur during execution
@@ -301,8 +301,8 @@ void DisposeIsolateWhenPossible(Isolate* isolate) {
   DefineDrainMicrotaskMethod(isolate, globalTemplate);
   // queueMicrotask(callback) per spec
   {
-    Local<FunctionTemplate> qmtTemplate = FunctionTemplate::New(
-        isolate, [](const FunctionCallbackInfo<Value>& info) {
+    Local<FunctionTemplate> qmtTemplate =
+        FunctionTemplate::New(isolate, [](const FunctionCallbackInfo<Value>& info) {
           auto* isolate = info.GetIsolate();
           if (info.Length() < 1 || !info[0]->IsFunction()) {
             isolate->ThrowException(Exception::TypeError(
@@ -434,13 +434,23 @@ void DisposeIsolateWhenPossible(Isolate* isolate) {
   v8::Locker locker(isolate);
   Isolate::Scope isolate_scope(isolate);
   HandleScope handle_scope(isolate);
+
+  auto cache = Caches::Get(isolate);
+  auto context = cache->GetContext();
+  Context::Scope context_scope(context);
+
   this->moduleInternal_->RunModule(isolate, "./");
 }
 
 void Runtime::RunModule(const std::string moduleName) {
   Isolate* isolate = this->GetIsolate();
   Isolate::Scope isolate_scope(isolate);
   HandleScope handle_scope(isolate);
+
+  auto cache = Caches::Get(isolate);
+  auto context = cache->GetContext();
+  Context::Scope context_scope(context);
+
   this->moduleInternal_->RunModule(isolate, moduleName);
 }
 
@@ -449,6 +459,11 @@ void DisposeIsolateWhenPossible(Isolate* isolate) {
   v8::Locker locker(isolate);
   Isolate::Scope isolate_scope(isolate);
   HandleScope handle_scope(isolate);
+
+  auto cache = Caches::Get(isolate);
+  auto context = cache->GetContext();
+  Context::Scope context_scope(context);
+
   this->moduleInternal_->RunScript(isolate, script);
 }
 
@@ -488,7 +503,8 @@ void DisposeIsolateWhenPossible(Isolate* isolate) {
     result = AppPackageJson[nsKey];
   }
 
-  // Store in cache (can cache nil as NSNull to differentiate presence if desired; for now, cache as-is)
+  // Store in cache (can cache nil as NSNull to differentiate presence if desired; for now, cache
+  // as-is)
   {
     std::lock_guard<std::mutex> lock(AppConfigCacheMutex);
     AppConfigCache[key] = result;