chore: add nix.gni fallback for non-Nix builds and fix compiler warnings

CMakeLists.txt previously required a nix.gni file (generated by the Nix
shell hook) containing platform-specific GN args. Builds outside Nix
failed with "file STRINGS file nix.gni cannot be read". Now falls back
to sensible defaults (is_clang=true, use_lld=false,
clang_use_chrome_plugins=false) when the file doesn't exist.

Also add a default UknownError path for ErrorFactory::DOMExceptionNameToString

Author: kevindhanna

CMakeLists.txt

@@ -185,7 +185,18 @@ set_target_properties(libc++abi PROPERTIES IMPORTED_OBJECTS_DEBUG "${libc++abi_o
 # M114: branch-heads/5735
 set(WEBRTC_REVISION branch-heads/5735)
 
-file(STRINGS nix.gni NIX_GN_GEN_ARGS)
+# nix.gni is generated by the Nix shell hook (shell.nix) with platform-specific
+# GN args like clang_base_path and mac_sdk_path. For non-Nix builds, create an
+# empty file or one with: is_clang=true\nuse_lld=false\nclang_use_chrome_plugins=false
+if(EXISTS "${CMAKE_SOURCE_DIR}/nix.gni")
+  file(STRINGS nix.gni NIX_GN_GEN_ARGS)
+else()
+  set(NIX_GN_GEN_ARGS
+    "is_clang=true"
+    "use_lld=false"
+    "clang_use_chrome_plugins=false"
+  )
+endif()
 list(APPEND GN_GEN_ARGS
   rtc_build_examples=false
   rtc_use_x11=false

src/node/error_factory.cc

@@ -94,6 +94,8 @@ node_webrtc::ErrorFactory::DOMExceptionNameToString(DOMExceptionName name) {
     return "NetworkError";
   case kOperationError:
     return "OperationError";
+  default:
+    return "UnknownError";
   }
 }