Address review comments, fix CI to use right SDK name.

Author: bzbarsky-apple

.github/workflows/darwin.yaml

@@ -58,7 +58,7 @@ jobs:
               working-directory: src/darwin/Framework
               # Disable availability annotations, since we are not building a system
               # Matter.framework.
-              run: xcodebuild -target "Matter" -sdk tvos -configuration Debug GCC_PREPROCESSOR_DEFINITIONS='${inherited} MTR_NO_AVAILABILITY=1'
+              run: xcodebuild -target "Matter" -sdk appletvos -configuration Debug GCC_PREPROCESSOR_DEFINITIONS='${inherited} MTR_NO_AVAILABILITY=1'
             - name: Run iOS Build Debug
               working-directory: src/darwin/Framework
               # Disable availability annotations, since we are not building a system

src/platform/Darwin/DnssdContexts.cpp

@@ -519,14 +519,20 @@ void ResolveContext::DispatchSuccess()
     bool needDelete = MdnsContexts::GetInstance().RemoveWithoutDeleting(this);
 
 #if TARGET_OS_TV
-    // On tvOS, priotize results from en0, en1, ir0 in that order, if those
+    // On tvOS, prioritize results from en0, en1, ir0 in that order, if those
     // interfaces are present, since those will generally have more up-to-date
     // information.
     static const unsigned int priorityInterfaceIndices[] = {
         if_nametoindex("en0"),
         if_nametoindex("en1"),
         if_nametoindex("ir0"),
     };
+#else
+    // Elsewhere prioritize "lo0" over other interfaces.
+    static const unsigned int priorityInterfaceIndices[] = {
+        if_nametoindex("lo0"),
+    };
+#endif // TARGET_OS_TV
 
     for (auto interfaceIndex : priorityInterfaceIndices)
     {
@@ -539,7 +545,6 @@ void ResolveContext::DispatchSuccess()
             return;
         }
     }
-#endif // TARGET_OS_TV
 
     for (auto & interface : interfaces)
     {