Better organize the code.

Not sure if this is overkill, but it sure is cleaner. :)

Reviewed the patch, looks optimized, suggestions
if (!empty($this->getNid())) {
if ($this->getAgreeNode()) {

Instead can be used:
if ((!empty($this->getNid()) && ($this->getAgreeNode())) {

#3 suggested code optimized fixed and recreate patch.

Actually, the getAgreeNode function already checks for nid:

public function getAgreeNode() {  
  if (!$this->getNid()) {  
    return NULL; 
  }  
  return $this->entityTypeManager->getStorage('node')->load($this->getNid());

But it needs an else statement.

Actually, I like this better than the else statement:

  /**
   * {@inheritdoc}
   */
  public function getAgreeNode() {
    $node = NULL;
    if (!empty($this->getNid())) {
      $node = $this->entityTypeManager->getStorage('node')->load($this->getNid());
    }
    return $node;
  }

Might be a good idea to create a MR instead for better comparison?