style(runtime): format this branch's runtime changes with clang-format

The repo formats staged C/C++/ObjC sources through lint-staged, but the
`clang-format` on PATH here resolved to depot_tools' wrapper, which needs
a Chromium checkout and exits non-zero without one. The hook failed
without aborting the commit, so everything this branch added went in
unformatted while main's sources stayed formatted.

Normalized with clang-format 20.1.8 (Homebrew LLVM), verified to
reproduce byte-identical output on all 30 runtime files #412 touched, so
this matches the style the V8 upgrade was written in.

Scoped with `git clang-format origin/main` to the lines this branch
actually changed: main's own pre-existing drift in files like
ConcurrentQueue.cpp is left alone rather than swept into this branch's
diff. Formatting only — no behavior change.

Author: NathanWalker

NativeScript/runtime/ConcurrentQueue.cpp

@@ -41,17 +41,18 @@ std::vector<std::shared_ptr<worker::Message>> ConcurrentQueue::PopAll() {
 }
 
 bool ConcurrentQueue::IsEmpty() {
-    std::unique_lock<std::mutex> mlock(this->mutex_);
-    return this->messagesQueue_.empty();
+  std::unique_lock<std::mutex> mlock(this->mutex_);
+  return this->messagesQueue_.empty();
 }
 
 void ConcurrentQueue::Signal() {
-    // Mirrors Push()'s validity handling instead of SignalAndWakeUp()'s
-    // assert: a retry racing Terminate() must be a silent no-op.
-    if (this->runLoopTasksSource_ == nullptr || !CFRunLoopSourceIsValid(this->runLoopTasksSource_)) {
-        return;
-    }
-    this->SignalAndWakeUp();
+  // Mirrors Push()'s validity handling instead of SignalAndWakeUp()'s
+  // assert: a retry racing Terminate() must be a silent no-op.
+  if (this->runLoopTasksSource_ == nullptr ||
+      !CFRunLoopSourceIsValid(this->runLoopTasksSource_)) {
+    return;
+  }
+  this->SignalAndWakeUp();
 }
 
 void ConcurrentQueue::SignalAndWakeUp() {

NativeScript/runtime/DevFlags.mm

@@ -1,11 +1,11 @@
 #import <Foundation/Foundation.h>
 
+#include <mutex>
+#include <vector>
 #include "DevFlags.h"
 #include "Helpers.h"
 #include "Runtime.h"
 #include "RuntimeConfig.h"
-#include <vector>
-#include <mutex>
 
 namespace tns {
 
@@ -34,8 +34,7 @@ bool IsHttpFetchUrlLogEnabled() {
       }
     }
     if (IsScriptLoadingLogEnabled()) {
-      Log(@"[http-loader] fetch-url-log=%s",
-          s_enabled ? "enabled" : "disabled");
+      Log(@"[http-loader] fetch-url-log=%s", s_enabled ? "enabled" : "disabled");
     }
   });
   return s_enabled;
@@ -57,8 +56,7 @@ bool IsHttpFetchUrlLogEnabled() {
 // "https://cdn.example.com.attacker.com/x.js" or
 // "https://cdn.example.com:9999/x.js". To allow a specific port, include it in
 // the allowlist entry (deny-by-default for anything not explicitly listed).
-static bool RemoteUrlMatchesAllowlistEntry(const std::string& url,
-                                           const std::string& entry) {
+static bool RemoteUrlMatchesAllowlistEntry(const std::string& url, const std::string& entry) {
   if (entry.empty()) return false;
   if (url.size() < entry.size()) return false;
   if (url.compare(0, entry.size(), entry) != 0) return false;
@@ -126,14 +124,14 @@ bool IsRemoteUrlAllowed(const std::string& url) {
   if (s_remoteModuleAllowlist.empty()) {
     return true;
   }
-  
+
   // Check if URL matches any allowlist entry on a URL-component boundary.
   for (const std::string& entry : s_remoteModuleAllowlist) {
     if (RemoteUrlMatchesAllowlistEntry(url, entry)) {
       return true;
     }
   }
-  
+
   return false;
 }
 

NativeScript/runtime/HMRSupport.h

@@ -44,7 +44,8 @@ namespace tns {
 // module under exactly one URL and never varies it for freshness.
 std::string CanonicalizeHttpUrlKey(const std::string& url);
 
-// Minimal text fetch for HTTP ESM loader. Returns true on 2xx with non-empty body.
+// Minimal text fetch for HTTP ESM loader. Returns true on 2xx with non-empty
+// body.
 // - out: response body
 // - contentType: Content-Type header if present
 // - status: HTTP status code
@@ -109,8 +110,7 @@ void MarkUrlsForCacheBust(const std::vector<std::string>& urls);
 // `outFetchedCount` (optional) receives the number of distinct URLs
 // fetched. `outElapsedMs` (optional) receives wall-clock time.
 bool KickstartHmrPrefetchUrlsSync(const std::vector<std::string>& urls,
-                                  int maxConcurrent,
-                                  double timeoutSeconds,
+                                  int maxConcurrent, double timeoutSeconds,
                                   size_t* outFetchedCount,
                                   uint64_t* outElapsedMs);
 
@@ -131,8 +131,8 @@ void CleanupHMRGlobals();
 // Mirror a globally-installed value onto `globalThis.<name>` so
 // `globalThis.<name>` lookups resolve when the runtime installs the
 // canonical value on the realm's global object.
-void MirrorGlobalOnGlobalThis(v8::Isolate* isolate, v8::Local<v8::Context> context,
-                              const char* name);
+void MirrorGlobalOnGlobalThis(v8::Isolate* isolate,
+                              v8::Local<v8::Context> context, const char* name);
 
 // ─────────────────────────────────────────────────────────────
 // Dev host namespace installer
@@ -147,12 +147,13 @@ void MirrorGlobalOnGlobalThis(v8::Isolate* isolate, v8::Local<v8::Context> conte
 //   - configureRuntime(config)        (import map + volatile patterns)
 //   - invalidateModules(urls)         (registry + cache eviction)
 //   - kickstartPrefetch(urls, opts?)  (parallel HTTP prewarm, list mode)
-//   - seedModuleBodies(entries)       (batch prewarm seeding from the boot archive)
+//   - seedModuleBodies(entries)       (batch prewarm seeding from the boot
+//   archive)
 //   - getLoadedModuleUrls()           (registry introspection)
 //   - setDevBootComplete(value?)      (boot-complete signal)
 //   - terminateAllWorkers()           (main isolate only; see Worker.h)
 //   - canonicalizeHttpUrlKey(url)     (debug builds only; test diagnostic)
-void InitializeHmrDevGlobals(v8::Isolate* isolate, v8::Local<v8::Context> context,
-                             bool isWorker);
+void InitializeHmrDevGlobals(v8::Isolate* isolate,
+                             v8::Local<v8::Context> context, bool isWorker);
 
 } // namespace tns