address comments

Author: yunhanw-google

examples/chip-tool/commands/clusters/ClusterCommand.h

@@ -61,7 +61,6 @@ class ClusterCommand : public InteractionModelCommands, public ModelCommand, pub
     {
         ReturnErrorOnFailure(InteractionModelCommands::SendCommand(device, endpointId, clusterId, commandId, value));
         mPeerNodeId    = chip::ScopedNodeId(device->GetDeviceId(), device->GetSecureSession().Value()->GetFabricIndex());
-        mCheckInNodeId = chip::ScopedNodeId(value.checkInNodeID, device->GetSecureSession().Value()->GetFabricIndex());
         return CHIP_NO_ERROR;
     }
 
@@ -264,10 +263,21 @@ class ClusterCommand : public InteractionModelCommands, public ModelCommand, pub
 private:
     chip::ClusterId mClusterId;
     chip::CommandId mCommandId;
+    // The scoped node ID to which RegisterClient and unregisterClient command will be sent 
     chip::ScopedNodeId mPeerNodeId;
+    // The scoped node ID to which a Check-In message will be sent
     chip::ScopedNodeId mCheckInNodeId;
+
+    // Used to determine if a particular client has an active subscription for the given entry. 
+    // The MonitoredSubject, when it is a NodeID, MAY be the same as the CheckInNodeID. 
+    // The MonitoredSubject gives the registering client the flexibility of having a different 
+    // CheckInNodeID from the MonitoredSubject. 
     uint64_t mMonitoredSubject                                     = static_cast<uint64_t>(0);
+
+    // Client type of the client registering
     chip::app::Clusters::IcdManagement::ClientTypeEnum mClientType = chip::app::Clusters::IcdManagement::ClientTypeEnum::kPermanent;
+    
+    // Shared secret between the client and the ICD to encrypt the Check-In message.
     uint8_t mICDSymmetricKey[chip::Crypto::kAES_CCM128_Key_Length];
 
     CHIP_ERROR mError = CHIP_NO_ERROR;