Fix UndefinedPropertyFetch and MixedAssignment psalm errors

Author: p123-stack

src/Formatter/Specialised/BoltOGMTranslator.php

@@ -104,6 +104,7 @@ private function makeFromBoltNode(BoltNode $node): Node
 
         /** @var string|null $elementId */
         if (property_exists($node, 'element_id')) {
+            /** @var string|null $elementIdValue */
             $elementIdValue = $node->element_id ?? null;
             $elementId = is_string($elementIdValue) ? $elementIdValue : (string) $node->id;
         } else {
@@ -183,22 +184,23 @@ private function makeFromBoltRelationship(BoltRelationship $rel): Relationship
 
         /** @var string|null $elementId */
         if (property_exists($rel, 'element_id')) {
+            /** @var string|null $elementIdValue */
             $elementIdValue = $rel->element_id ?? null;
             $elementId = is_string($elementIdValue) ? $elementIdValue : (string) $rel->id;
         } else {
             $elementId = (string) $rel->id;
         }
 
-        // For Neo4j 5+, try to get actual element IDs if available
-        // For Neo4j 4, fallback to string representation of integer IDs
         if (property_exists($rel, 'startNodeElementId')) {
+            /** @var string|null $startNodeElementIdValue */
             $startNodeElementIdValue = $rel->startNodeElementId ?? null;
             $startNodeElementId = is_string($startNodeElementIdValue) ? $startNodeElementIdValue : (string) $rel->startNodeId;
         } else {
             $startNodeElementId = (string) $rel->startNodeId;
         }
 
         if (property_exists($rel, 'endNodeElementId')) {
+            /** @var string|null $endNodeElementIdValue */
             $endNodeElementIdValue = $rel->endNodeElementId ?? null;
             $endNodeElementId = is_string($endNodeElementIdValue) ? $endNodeElementIdValue : (string) $rel->endNodeId;
         } else {
@@ -231,6 +233,7 @@ private function makeFromBoltUnboundRelationship(BoltUnboundRelationship $rel):
 
         /** @var string|null $elementId */
         if (property_exists($rel, 'element_id')) {
+            /** @var string|null $elementIdValue */
             $elementIdValue = $rel->element_id ?? null;
             $elementId = is_string($elementIdValue) ? $elementIdValue : (string) $rel->id;
         } else {

tests/Unit/NoAuthTest.php

@@ -36,6 +36,7 @@ protected function setUp(): void
         $logger = $this->createMock(Neo4jLogger::class);
         $this->auth = new NoAuth($logger);
     }
+
     public function testAuthenticateBoltSuccessV5(): void
     {
         $userAgent = 'neo4j-client/1.0';