feat: vpc splitted as discussed in comments

Author: RusselSand

README.md

@@ -19,7 +19,7 @@ Requires [uv](https://docs.astral.sh/uv/) and Python 3.11+.
 uv sync --group dev       # install with dev dependencies
 uv run ruff check src/    # lint
 uv run mypy src/          # type check
-uv run pytest -v          # test
+uv run pytest -v          # test core
 ```
 
 ## License

pyproject.toml

@@ -18,6 +18,7 @@ dependencies = [
 [dependency-groups]
 dev = [
     "pytest>=9.0.2,<10",
+    "pytest-mock>=3.14",
     "pytest-httpx>=0.36",
     "ruff>=0.15.2",
     "mypy>=1.19",
@@ -48,4 +49,5 @@ testpaths = ["tests"]
 log_cli = true
 log_cli_level = "INFO"
 log_cli_format = "%(asctime)s [%(levelname)s] %(message)s"
-log_cli_date_format = "%H:%M:%S"
+log_cli_date_format = "%H:%M:%S"
+addopts = "--import-mode=importlib --ignore=tests/acceptance"

src/sdk/core/opts.py

@@ -0,0 +1,61 @@
+"""Base classes for request options."""
+
+from __future__ import annotations
+
+from typing import Any, ClassVar
+
+from pydantic import BaseModel
+
+
+class BaseOpts(BaseModel):
+    """Base class for request body options.
+
+    Subclasses set ``_wrapper_key`` to wrap the payload in a named object
+    (e.g. ``{"vpc": {...}}``). Without it the body is returned flat.
+
+    Field-omission rules (``exclude_none=True, exclude_defaults=True``):
+
+    * ``name: str = ""`` -> omitted when caller does not set it.
+    * ``name: str | None = None`` -> ``None`` is omitted, but any other
+      value (including ``""``) is sent, allowing callers to clear
+      server-side fields.
+    """
+
+    _wrapper_key: ClassVar[str | None] = None
+
+    def to_request_body(self) -> dict[str, Any]:
+        body = self.model_dump(
+            exclude_none=True,
+            exclude_defaults=True,
+            by_alias=True,
+        )
+        if self._wrapper_key is not None:
+            return {self._wrapper_key: body}
+        return body
+
+
+class BaseQueryOpts(BaseModel):
+    """Base class for query string options.
+
+    Output is a flat ``dict[str, str]`` ready for an HTTP client's
+    ``params=`` argument. ``None`` and empty strings are dropped; numeric
+    zero and ``False`` are preserved. Nested models or collections raise
+    :class:`TypeError`.
+    """
+
+    def to_query_params(self) -> dict[str, str]:
+        raw = self.model_dump(exclude_none=True, by_alias=True)
+        params: dict[str, str] = {}
+        for key, value in raw.items():
+            if value == "":
+                continue
+            if isinstance(value, (dict, list)):
+                raise TypeError(
+                    f"Query param {key!r} has non-scalar value {value!r}; "
+                    "query strings only support scalar values."
+                )
+            if isinstance(value, bool):
+                params[key] = "true" if value else "false"
+            else:
+                params[key] = str(value)
+        return params

src/sdk/services/vpc/v1/__init__.py

@@ -0,0 +1,18 @@
+"""VPC service, API v1.
+
+Resources are exposed as submodules. Usage::
+
+    from sdk.services.vpc.v1 import vpcs
+
+    new_vpc = vpcs.create(client, vpcs.CreateVpcOpts(name="my-vpc"))
+    for v in vpcs.list(client):
+        print(v.id, v.name)
+
+Future resources (subnets, peerings, ...) are added as sibling submodules.
+"""
+
+from __future__ import annotations
+
+from . import vpcs
+
+__all__ = ["vpcs"]