Cleanup branch (#27)

* getters are removed, all the @apis are removed

Author: 123kiran17

src/Authentication/AuthenticateInterface.php

@@ -4,9 +4,6 @@
 
 use Psr\Http\Message\RequestInterface;
 
-/**
- *  @api
- */
 interface AuthenticateInterface
 {
     public function getHeader(): string;

src/Authentication/BasicAuthentication.php

@@ -4,10 +4,7 @@
 
 use Psr\Http\Message\RequestInterface;
 
-/**
- *  @api
- */
-class BasicAuthentication implements AuthenticateInterface
+final class BasicAuthentication implements AuthenticateInterface
 {
     private string $username;
     private string $password;

src/Authentication/BearerAuthentication.php

@@ -4,10 +4,7 @@
 
 use Psr\Http\Message\RequestInterface;
 
-/**
- *  @api
- */
-class BearerAuthentication implements AuthenticateInterface
+final class BearerAuthentication implements AuthenticateInterface
 {
     public function __construct(private string $token)
     {

src/Authentication/NoAuth.php

@@ -4,10 +4,7 @@
 
 use Psr\Http\Message\RequestInterface;
 
-/**
- *  @api
- */
-class NoAuth implements AuthenticateInterface
+final class NoAuth implements AuthenticateInterface
 {
     #[\Override]
     public function getHeader(): string

src/Exception/Neo4jException.php

@@ -4,15 +4,12 @@
 
 use Exception;
 
-/**
- * @api
- */
-class Neo4jException extends Exception
+final class Neo4jException extends Exception
 {
-    private readonly string $errorCode;
-    private readonly ?string $errorType;
-    private readonly ?string $errorSubType;
-    private readonly ?string $errorName;
+    public readonly string $errorCode;
+    public readonly ?string $errorType;
+    public readonly ?string $errorSubType;
+    public readonly ?string $errorName;
 
     public function __construct(
         array       $errorDetails = [],
@@ -31,7 +28,6 @@ public function __construct(
 
     /**
      * Create a Neo4jException instance from a Neo4j error response array.
-     *
      * @param array $response The error response from Neo4j.
      * @param \Throwable|null $exception Optional previous exception for chaining.
      * @return self
@@ -44,23 +40,4 @@ public static function fromNeo4jResponse(array $response, ?\Throwable $exception
         return new self($errorDetails, previous: $exception);
     }
 
-    public function getErrorCode(): string
-    {
-        return $this->errorCode;
-    }
-
-    public function getType(): ?string
-    {
-        return $this->errorType;
-    }
-
-    public function getSubType(): ?string
-    {
-        return $this->errorSubType;
-    }
-
-    public function getName(): ?string
-    {
-        return $this->errorName;
-    }
 }

src/Neo4jQueryAPI.php

@@ -22,9 +22,6 @@ public function __construct(
     ) {
     }
 
-    /**
-     * @api
-     */
     public static function login(string $address, AuthenticateInterface $auth = null): self
     {
         $client = Psr18ClientDiscovery::find();

src/Neo4jRequestFactory.php

@@ -8,10 +8,7 @@
 use Psr\Http\Message\RequestInterface;
 use Psr\Http\Message\StreamFactoryInterface;
 
-/**
- * @api
- */
-class Neo4jRequestFactory
+final class Neo4jRequestFactory
 {
     public function __construct(
         private RequestFactoryInterface $psr17Factory,

src/OGM.php

@@ -8,10 +8,7 @@
 use Neo4j\QueryAPI\Objects\Path;
 use InvalidArgumentException;
 
-/**
- *  @api
- */
-class OGM
+final class OGM
 {
     /**
      * @param array<array-key, mixed> $data
@@ -57,44 +54,18 @@ private function mapNode(array $nodeData): Node
     {
         return new Node(
             labels: $nodeData['_labels'] ?? [],
-            properties: $this->mapProperties($nodeData['_properties'] ?? []) // ✅ Fix: Ensure properties exist
+            properties: $this->mapProperties($nodeData['_properties'] ?? [])
         );
     }
 
     private function mapRelationship(array $relationshipData): Relationship
     {
         return new Relationship(
-            type: $relationshipData['_type'] ?? 'UNKNOWN',  // ✅ Fix: Default to 'UNKNOWN'
+            type: $relationshipData['_type'] ?? 'UNKNOWN',
             properties: $this->mapProperties($relationshipData['_properties'] ?? [])
         );
     }
 
-
-    public static function parseWKT(string $wkt): Point
-    {
-        $sridPos = strpos($wkt, ';');
-        if ($sridPos === false) {
-            throw new \InvalidArgumentException("Invalid WKT format: missing ';'");
-        }
-        $sridPart = substr($wkt, 0, $sridPos);
-        $srid = (int)str_replace('SRID=', '', $sridPart);
-
-        $pointPos = strpos($wkt, 'POINT');
-        if ($pointPos === false) {
-            throw new \InvalidArgumentException("Invalid WKT format: missing 'POINT'");
-        }
-        $pointPart = substr($wkt, $pointPos + 6);
-
-        $pointPart = str_replace('Z', '', $pointPart);
-        $pointPart = trim($pointPart, ' ()');
-        $coordinates = explode(' ', $pointPart);
-
-        [$x, $y, $z] = array_pad(array_map('floatval', $coordinates), 3, 0.0);
-
-        return new Point($x, $y, $z, $srid);
-    }
-
-
     private function mapPath(array $pathData): Path
     {
         $nodes = [];

src/Objects/Authentication.php

@@ -7,10 +7,7 @@
 use Neo4j\QueryAPI\Authentication\BearerAuthentication;
 use Neo4j\QueryAPI\Authentication\NoAuth;
 
-/**
- *  @api
- */
-class Authentication
+final class Authentication
 {
     public static function basic(string $username, string $password): AuthenticateInterface
     {

src/Objects/Bookmarks.php

@@ -4,10 +4,7 @@
 
 use JsonSerializable;
 
-/**
- * @api
- */
-class Bookmarks implements \Countable, JsonSerializable
+final class Bookmarks implements \Countable, JsonSerializable
 {
     public function __construct(private array $bookmarks)
     {