refactor(v3): remove legacy api accessors

Author: andrepimpao

docs/v3-architecture-plan.md

@@ -200,31 +200,21 @@ Start with a minimal context API. Add richer access only when implementation nee
 Current public methods:
 
 - `request`
-- `getBaseUrl`
-- `setBaseUrl`
-- `getQueryDefault`
-- `addQueryDefault`
-- `removeQueryDefault`
-- `getHeaderDefault`
-- `addHeaderDefault`
-- `removeHeaderDefault`
 - `getClientBuilder`
 - `setClientBuilder`
-- `getCacheBuilder`
-- `setCacheBuilder`
 - `getLoggerBuilder`
 - `setLoggerBuilder`
-- `getAuthentication`
-- `setAuthentication`
+- `plugins`
+- `cache`
+- `config`
 - `addPreRequestListener`
 - `addPostRequestListener`
 - `addResponseContentsListener`
-- `buildPath`
 
 Observations:
 
 - The current `Api` class is easy to extend but exposes low-level request execution directly.
-- SDK packages currently use `request` and `buildPath` from resources.
+- SDK packages currently use `request` from resources.
 - Defaults are global to the API instance, which makes resource-level fluent options awkward.
 - JSON decoding and error handling are implemented through listeners.
 - Plugin configuration is automatic inside `request`, which can duplicate responsibilities and makes request flow harder to reason about.

src/Api.php

@@ -133,25 +133,21 @@ protected function resource(string $class): Resource
 
     protected function baseUrl(?string $baseUrl): static
     {
-        $this->setBaseUrl($baseUrl);
+        $this->baseUrl = $baseUrl;
 
         return $this;
     }
 
     protected function queryDefaults(array $query): static
     {
-        foreach ($query as $name => $value) {
-            $this->addQueryDefault($name, $value);
-        }
+        $this->queryDefaults = array_merge($this->queryDefaults, $query);
 
         return $this;
     }
 
     protected function headerDefaults(array $headers): static
     {
-        foreach ($headers as $name => $value) {
-            $this->addHeaderDefault($name, $value);
-        }
+        $this->headerDefaults = array_merge($this->headerDefaults, $headers);
 
         return $this;
     }
@@ -257,56 +253,6 @@ private function buildPlugins(): array
         return $plugins->all();
     }
 
-    public function getBaseUrl(): ?string
-    {
-        return $this->baseUrl;
-    }
-
-    public function setBaseUrl(?string $baseUrl): self
-    {
-        $this->baseUrl = $baseUrl;
-
-        return $this;
-    }
-
-    public function getQueryDefault(string $name): mixed
-    {
-        return $this->queryDefaults[$name] ?? null;
-    }
-
-    public function addQueryDefault(string $name, mixed $value): self
-    {
-        $this->queryDefaults[$name] = $value;
-
-        return $this;
-    }
-
-    public function removeQueryDefault(string $name): self
-    {
-        unset($this->queryDefaults[$name]);
-
-        return $this;
-    }
-
-    public function getHeaderDefault(string $name): mixed
-    {
-        return $this->headerDefaults[$name] ?? null;
-    }
-
-    public function addHeaderDefault(string $name, mixed $value): self
-    {
-        $this->headerDefaults[$name] = $value;
-
-        return $this;
-    }
-
-    public function removeHeaderDefault(string $name): self
-    {
-        unset($this->headerDefaults[$name]);
-
-        return $this;
-    }
-
     public function getClientBuilder(): ?ClientBuilder
     {
         return $this->clientBuilder;
@@ -352,7 +298,7 @@ public function addResponseContentsListener(callable $listener, int $priority =
         return $this;
     }
 
-    public function buildPath(string $path, array $parameters): string
+    private function buildPath(string $path, array $parameters): string
     {
         foreach ($parameters as $parameter => $value) {
             $path = str_replace(