Shift away session decode from constructor

Author: chr-hertel

src/Server/Session/Session.php

@@ -22,35 +22,27 @@
 class Session implements SessionInterface
 {
     /**
-     * @param array<string, mixed> $data Stores all session data.
-     *                                   Keys are snake_case by convention for MCP-specific data.
-     *
      * Official keys are:
      * - initialized: bool
      * - client_info: array|null
      * - protocol_version: string|null
      * - log_level: string|null
+     *
+     * @var array<string, mixed>
      */
+    private array $data;
+
     public function __construct(
-        protected SessionStoreInterface $store,
-        protected Uuid $id = new UuidV4(),
-        protected array $data = [],
+        private SessionStoreInterface $store,
+        private Uuid $id = new UuidV4(),
     ) {
-        if ($rawData = $this->store->read($this->id)) {
-            $this->data = json_decode($rawData, true) ?? [];
-        }
     }
 
     public function getId(): Uuid
     {
         return $this->id;
     }
 
-    public function getStore(): SessionStoreInterface
-    {
-        return $this->store;
-    }
-
     public function save(): bool
     {
         return $this->store->write($this->id, json_encode($this->data, \JSON_THROW_ON_ERROR));
@@ -59,7 +51,7 @@ public function save(): bool
     public function get(string $key, mixed $default = null): mixed
     {
         $key = explode('.', $key);
-        $data = $this->data;
+        $data = $this->readData();
 
         foreach ($key as $segment) {
             if (\is_array($data) && \array_key_exists($segment, $data)) {
@@ -75,6 +67,7 @@ public function get(string $key, mixed $default = null): mixed
     public function set(string $key, mixed $value, bool $overwrite = true): void
     {
         $segments = explode('.', $key);
+        $this->readData();
         $data = &$this->data;
 
         while (\count($segments) > 1) {
@@ -94,7 +87,7 @@ public function set(string $key, mixed $value, bool $overwrite = true): void
     public function has(string $key): bool
     {
         $key = explode('.', $key);
-        $data = $this->data;
+        $data = $this->readData();
 
         foreach ($key as $segment) {
             if (\is_array($data) && \array_key_exists($segment, $data)) {
@@ -112,6 +105,7 @@ public function has(string $key): bool
     public function forget(string $key): void
     {
         $segments = explode('.', $key);
+        $this->readData();
         $data = &$this->data;
 
         while (\count($segments) > 1) {
@@ -143,7 +137,7 @@ public function pull(string $key, mixed $default = null): mixed
 
     public function all(): array
     {
-        return $this->data;
+        return $this->readData();
     }
 
     public function hydrate(array $attributes): void
@@ -156,4 +150,22 @@ public function jsonSerialize(): array
     {
         return $this->all();
     }
+
+    /**
+     * @return array<string, mixed>
+     */
+    private function readData(): array
+    {
+        if (isset($this->data)) {
+            return $this->data;
+        }
+
+        $rawData = $this->store->read($this->id);
+
+        if (false === $rawData) {
+            return $this->data = [];
+        }
+
+        return $this->data = json_decode($rawData, true, flags: \JSON_THROW_ON_ERROR);
+    }
 }