Apply suggestions from code review

Co-authored-by: Boris Zbarsky <[email protected]>

src/darwin/Framework/CHIP/MTRCommissioningParameters.h

@@ -100,7 +100,7 @@ MTR_AVAILABLE(ios(16.1), macos(13.0), watchos(9.1), tvos(16.1))
  * Read device type information from all endpoints during commissioning.
  * Defaults to NO.
  */
-@property (nonatomic, assign) BOOL readEndpointInformation;
+@property (nonatomic, assign) BOOL readEndpointInformation MTR_NEWLY_AVAILABLE;
 
 @end
 

src/darwin/Framework/CHIP/MTRDeviceControllerDelegateBridge.mm

@@ -150,7 +150,7 @@ - (MTREndpointInfo *)rootEndpoint
 {
     MTRDeviceController * strongController = mController;
     id<MTRDeviceControllerDelegate> strongDelegate = mDelegate;
-    VerifyOrReturn(strongDelegate && mQueue);
+    VerifyOrReturn(strongDelegate && mQueue && strongController);
 
     // TODO: These checks are pointless since currently mController == mDelegate
     BOOL wantCommissioneeInfo = [strongDelegate respondsToSelector:@selector(controller:readCommissioneeInfo:)];

src/darwin/Framework/CHIP/MTREndpointInfo.mm

@@ -118,7 +118,7 @@ + (BOOL)populateChildrenForEndpoints:(NSDictionary<NSNumber *, MTREndpointInfo *
     }
 
     // Now iterate over the endpoints in reverse topological order, i.e. bottom up. This means
-    // that we will visit children before parents, so the first time we see and endpoint in a
+    // that we will visit children before parents, so the first time we see an endpoint in a
     // PartsList we can assign it as a child of the endpoint we're processing, and we can be sure
     // that this is the closest parent, not some higher ancestor using the Full-Family Pattern.
     NSMutableArray<MTREndpointInfo *> * children = [[NSMutableArray alloc] init];