wherobots-python-sdk 0.3.0__tar.gz → 0.4.0__tar.gz

{wherobots_python_sdk-0.3.0 → wherobots_python_sdk-0.4.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: wherobots-python-sdk
-Version: 0.3.0
+Version: 0.4.0
 Summary: Python SDK for Wherobots (currently covers the Jobs REST API)
 Author-email: Wherobots <support@wherobots.com>
 License-Expression: Apache-2.0
@@ -23,13 +23,6 @@ Description-Content-Type: text/markdown
 License-File: LICENSE
 Requires-Dist: requests>=2.33.0
 Requires-Dist: boto3>=1.34.0
-Provides-Extra: dev
-Requires-Dist: pytest>=7.4.0; extra == "dev"
-Requires-Dist: pytest-cov>=4.1.0; extra == "dev"
-Requires-Dist: pytest-mock>=3.11.1; extra == "dev"
-Requires-Dist: ruff>=0.6.0; extra == "dev"
-Requires-Dist: mypy>=1.5.0; extra == "dev"
-Requires-Dist: types-requests>=2.31; extra == "dev"
 Dynamic: license-file
 
 # Wherobots Python SDK
@@ -67,11 +60,11 @@ The simplest way to run a local script — only an API key is needed:
 from wherobots import WherobotsJob
 
 job = WherobotsJob(
-    script="my_analysis.py",       # local file — auto-uploaded via presigned URL
+    script="my_analysis.py",  # local file — auto-uploaded via presigned URL
     name="analysis-job-001",
     runtime="tiny",
     region="aws-us-west-2",
-    api_key="wbk_...",             # or set WHEROBOTS_API_KEY env var
+    api_key="wbk_...",  # or set WHEROBOTS_API_KEY env var
 )
 
 run_id = job.submit()
@@ -135,6 +128,7 @@ The API key can also be set via the `WHEROBOTS_API_KEY` environment variable.
 | `WHEROBOTS_API_BASE_URL`          | API base URL                    | `https://api.cloud.wherobots.com`    |
 | `WHEROBOTS_VERSION`               | Wherobots version               | `latest`                             |
 | `WHEROBOTS_REQUEST_TIMEOUT_SECONDS`| HTTP request timeout (seconds) | `30`                                 |
+| `WHEROBOTS_CLIENT_CHAIN`           | Upstream client-attribution chain (see below) | *(none)*              |
 
 ### WherobotsConfig
 
@@ -164,6 +158,35 @@ config = WherobotsConfig(
 | `base_url`                 | `str`          | `"https://api.cloud.wherobots.com"` |
 | `version`                  | `str`          | `"latest"`                          |
 | `request_timeout_seconds`  | `int`          | `30`                                |
+| `client_chain`             | `str \| None`  | `None`                              |
+
+### Client attribution
+
+Every request the SDK sends carries an advisory `X-Wherobots-Client` header so
+Wherobots can attribute workloads to the client that started them:
+
+```
+X-Wherobots-Client: client=python-sdk;ver=0.3.0;plat=darwin
+```
+
+The header is an ordered, comma-separated chain of hops — leftmost is the
+origin client, and each component appends its own hop on the right. If your
+tool embeds this SDK, pass your own hop as `client_chain` and the SDK appends
+itself after it:
+
+```python
+job = WherobotsJob(
+    script="job.py",
+    name="my-job-name",
+    client_chain="client=airflow;ver=1.2.0",
+)
+# X-Wherobots-Client: client=airflow;ver=1.2.0, client=python-sdk;ver=0.3.0;plat=linux
+```
+
+`client_chain` also works on `WherobotsJob.from_run_id(...)` and
+`WherobotsConfig`, or can be set once via `WHEROBOTS_CLIENT_CHAIN`. The header
+is advisory only: it is used for attribution and analytics and never affects
+authentication or authorization.
 
 ## SDK Reference
 
@@ -192,6 +215,7 @@ WherobotsJob(
     base_url: str | None = None,             # Override API URL
     request_timeout_seconds: int | None = None,  # HTTP timeout
     config: WherobotsConfig | None = None,   # Full config object
+    client_chain: str | None = None,         # Upstream X-Wherobots-Client chain
 )
 ```
 
@@ -318,7 +342,7 @@ environment.
 
 ```python
 job = WherobotsJob(
-    script="my_job.py",                  # local script auto-upload still works
+    script="my_job.py",  # local script auto-upload still works
     name="job-with-local-wheel",
     dependencies=[
         # Local wheel — uploaded to managed storage at submit() time
@@ -368,7 +392,7 @@ with FilesAPI.from_config(WherobotsConfig.from_env()) as files:
     )
 
 job = WherobotsJob(
-    script=script_uri,                              # already-uploaded URI
+    script=script_uri,  # already-uploaded URI
     name="job-in-integration",
     dependencies=[WherobotsJob.add_file_dependency(wheel_uri)],
 )
@@ -628,10 +652,8 @@ Base URL: `https://api.cloud.wherobots.com`
 ```bash
 git clone https://github.com/wherobots/wherobots-python-sdk.git
 cd wherobots-python-sdk
-python -m venv .venv
-source .venv/bin/activate
-pip install -e ".[dev]"
-pre-commit install
+uv sync --group dev
+uv run pre-commit install
 ```
 
 ### Quality checks
@@ -639,10 +661,27 @@ pre-commit install
 ```bash
 make lint        # ruff check + ruff format --check
 make format      # auto-fix lint issues and reformat
-make typecheck   # mypy (strict)
-make check       # lint + typecheck + tests + build
+make typecheck   # pyrefly (strict config)
+make rails       # import-linter + no-suppression + ast-grep
+make check       # lint + typecheck + rails + tests + build
 ```
 
+### Suppression baseline
+
+Pre-existing debt is grandfathered in `.slop-suppressions.json` (CODEOWNED): the inline
+suppressions already in the tree, plus the `# noqa: CODE` / `# pyrefly: ignore` comments that
+`ruff check --add-noqa` and `pyrefly check --suppress-errors` wrote for pre-existing violations.
+The suppressions are functional at the tool level; the keystone
+(`.slop-repellant/check_no_suppression.py`) fails any net-new one.
+
+- **Prune** after fixing baselined code (drops stale entries):
+  - ruff's `RUF100` auto-removes stale noqas via the pre-commit hook's `--fix`
+  - `uv run pyrefly suppress --remove-unused`
+  - then `uv run python .slop-repellant/check_no_suppression.py --update`
+- **Reset** (re-record from scratch): delete `.slop-suppressions.json`, then run the same `--update`.
+- Shrinking the baseline is always welcome; **growing it requires CODEOWNER sign-off** — fix the
+  code instead of widening the baseline.
+
 ### Releasing
 
 See [`RELEASING.md`](RELEASING.md) for the release flow (version

{wherobots_python_sdk-0.3.0 → wherobots_python_sdk-0.4.0}/README.md

@@ -33,11 +33,11 @@ The simplest way to run a local script — only an API key is needed:
 from wherobots import WherobotsJob
 
 job = WherobotsJob(
-    script="my_analysis.py",       # local file — auto-uploaded via presigned URL
+    script="my_analysis.py",  # local file — auto-uploaded via presigned URL
     name="analysis-job-001",
     runtime="tiny",
     region="aws-us-west-2",
-    api_key="wbk_...",             # or set WHEROBOTS_API_KEY env var
+    api_key="wbk_...",  # or set WHEROBOTS_API_KEY env var
 )
 
 run_id = job.submit()
@@ -101,6 +101,7 @@ The API key can also be set via the `WHEROBOTS_API_KEY` environment variable.
 | `WHEROBOTS_API_BASE_URL`          | API base URL                    | `https://api.cloud.wherobots.com`    |
 | `WHEROBOTS_VERSION`               | Wherobots version               | `latest`                             |
 | `WHEROBOTS_REQUEST_TIMEOUT_SECONDS`| HTTP request timeout (seconds) | `30`                                 |
+| `WHEROBOTS_CLIENT_CHAIN`           | Upstream client-attribution chain (see below) | *(none)*              |
 
 ### WherobotsConfig
 
@@ -130,6 +131,35 @@ config = WherobotsConfig(
 | `base_url`                 | `str`          | `"https://api.cloud.wherobots.com"` |
 | `version`                  | `str`          | `"latest"`                          |
 | `request_timeout_seconds`  | `int`          | `30`                                |
+| `client_chain`             | `str \| None`  | `None`                              |
+
+### Client attribution
+
+Every request the SDK sends carries an advisory `X-Wherobots-Client` header so
+Wherobots can attribute workloads to the client that started them:
+
+```
+X-Wherobots-Client: client=python-sdk;ver=0.3.0;plat=darwin
+```
+
+The header is an ordered, comma-separated chain of hops — leftmost is the
+origin client, and each component appends its own hop on the right. If your
+tool embeds this SDK, pass your own hop as `client_chain` and the SDK appends
+itself after it:
+
+```python
+job = WherobotsJob(
+    script="job.py",
+    name="my-job-name",
+    client_chain="client=airflow;ver=1.2.0",
+)
+# X-Wherobots-Client: client=airflow;ver=1.2.0, client=python-sdk;ver=0.3.0;plat=linux
+```
+
+`client_chain` also works on `WherobotsJob.from_run_id(...)` and
+`WherobotsConfig`, or can be set once via `WHEROBOTS_CLIENT_CHAIN`. The header
+is advisory only: it is used for attribution and analytics and never affects
+authentication or authorization.
 
 ## SDK Reference
 
@@ -158,6 +188,7 @@ WherobotsJob(
     base_url: str | None = None,             # Override API URL
     request_timeout_seconds: int | None = None,  # HTTP timeout
     config: WherobotsConfig | None = None,   # Full config object
+    client_chain: str | None = None,         # Upstream X-Wherobots-Client chain
 )
 ```
 
@@ -284,7 +315,7 @@ environment.
 
 ```python
 job = WherobotsJob(
-    script="my_job.py",                  # local script auto-upload still works
+    script="my_job.py",  # local script auto-upload still works
     name="job-with-local-wheel",
     dependencies=[
         # Local wheel — uploaded to managed storage at submit() time
@@ -334,7 +365,7 @@ with FilesAPI.from_config(WherobotsConfig.from_env()) as files:
     )
 
 job = WherobotsJob(
-    script=script_uri,                              # already-uploaded URI
+    script=script_uri,  # already-uploaded URI
     name="job-in-integration",
     dependencies=[WherobotsJob.add_file_dependency(wheel_uri)],
 )
@@ -594,10 +625,8 @@ Base URL: `https://api.cloud.wherobots.com`
 ```bash
 git clone https://github.com/wherobots/wherobots-python-sdk.git
 cd wherobots-python-sdk
-python -m venv .venv
-source .venv/bin/activate
-pip install -e ".[dev]"
-pre-commit install
+uv sync --group dev
+uv run pre-commit install
 ```
 
 ### Quality checks
@@ -605,10 +634,27 @@ pre-commit install
 ```bash
 make lint        # ruff check + ruff format --check
 make format      # auto-fix lint issues and reformat
-make typecheck   # mypy (strict)
-make check       # lint + typecheck + tests + build
+make typecheck   # pyrefly (strict config)
+make rails       # import-linter + no-suppression + ast-grep
+make check       # lint + typecheck + rails + tests + build
 ```
 
+### Suppression baseline
+
+Pre-existing debt is grandfathered in `.slop-suppressions.json` (CODEOWNED): the inline
+suppressions already in the tree, plus the `# noqa: CODE` / `# pyrefly: ignore` comments that
+`ruff check --add-noqa` and `pyrefly check --suppress-errors` wrote for pre-existing violations.
+The suppressions are functional at the tool level; the keystone
+(`.slop-repellant/check_no_suppression.py`) fails any net-new one.
+
+- **Prune** after fixing baselined code (drops stale entries):
+  - ruff's `RUF100` auto-removes stale noqas via the pre-commit hook's `--fix`
+  - `uv run pyrefly suppress --remove-unused`
+  - then `uv run python .slop-repellant/check_no_suppression.py --update`
+- **Reset** (re-record from scratch): delete `.slop-suppressions.json`, then run the same `--update`.
+- Shrinking the baseline is always welcome; **growing it requires CODEOWNER sign-off** — fix the
+  code instead of widening the baseline.
+
 ### Releasing
 
 See [`RELEASING.md`](RELEASING.md) for the release flow (version

{wherobots_python_sdk-0.3.0 → wherobots_python_sdk-0.4.0}/pyproject.toml

@@ -35,13 +35,19 @@ Repository = "https://github.com/wherobots/wherobots-python-sdk"
 Issues = "https://github.com/wherobots/wherobots-python-sdk/issues"
 Documentation = "https://docs.wherobots.com"
 
-[project.optional-dependencies]
+# PEP 735 dev dependencies — installed with `uv sync --group dev`.
+[dependency-groups]
 dev = [
-    "pytest>=7.4.0",
+    "pytest>=8.0",
     "pytest-cov>=4.1.0",
     "pytest-mock>=3.11.1",
-    "ruff>=0.6.0",
-    "mypy>=1.5.0",
+    "ruff>=0.15,<0.17",
+    "pyrefly>=0.16",
+    "import-linter>=2.1",
+    "pre-commit>=3.8",
+    "hypothesis>=6.100",
+    "cosmic-ray>=8.4",
+    "ast-grep-cli>=0.39",
     "types-requests>=2.31",
 ]
 
@@ -54,44 +60,49 @@ wherobots = ["py.typed"]
 [tool.setuptools.dynamic]
 version = {attr = "wherobots.__version__.__version__"}
 
+# SLOP REPELLANT ruff rail: line-length 100, py310 target, select = ["ALL"]
+# plus the typing.Any/typing.cast ban all come from the base config.
+# NOTE: `lint.select`/`lint.ignore` would REPLACE the base's — tune only via
+# the extend-* keys.
 [tool.ruff]
-line-length = 100
-target-version = "py310"
-extend-exclude = ["env", ".venv", "venv", "build", "dist"]
+extend = ".slop-repellant/ruff-base.toml"
+# .slop-repellant/ is vendored, kit-MANAGED tooling (re-rendered on
+# slop-repellant-update) — linting it locally would fork the managed files.
+extend-exclude = ["env", ".venv", "venv", "build", "dist", ".slop-repellant"]
 
 [tool.ruff.lint]
-select = ["E", "F", "I", "UP", "B", "SIM", "RUF"]
 # SIM117 merges nested with-statements — often hurts readability when
 # the context managers are logically distinct. Disable.
-ignore = ["SIM117"]
+# EM101/EM102/TRY003 (exception-message-as-literal trio): style position,
+# not debt — we keep messages inline at the raise site. Disable globally.
+# PLR0917 (too-many-positional-arguments): newly enabled in ruff 0.16's ALL
+# set. It is the positional-only variant of PLR0913 (too-many-arguments),
+# which the few wide signatures here already carry an accepted `# noqa:
+# PLR0913` for — so PLR0917 is redundant with a complaint the codebase has
+# already signed off on. Ignored globally rather than making those args
+# keyword-only, which would be a semver-breaking public-API change.
+extend-ignore = ["SIM117", "EM101", "EM102", "TRY003", "PLR0917"]
 
-[tool.ruff.lint.per-file-ignores]
+[tool.ruff.lint.extend-per-file-ignores]
 # Tests may assert freely, re-raise without chaining, shadow builtins
 # for fixtures, and use mock call patterns that look like unused
-# locals.
-"tests/**/*.py" = ["B011", "B904", "F841", "RUF059", "RUF013"]
+# locals. N803/N806 fire on @patch-injected MockSession-style names and
+# PLC0415 on deliberate deferred imports — idiomatic in tests, not debt.
+# (Extends the base's tests/** ignore set.)
+"tests/**/*.py" = ["B011", "B904", "F841", "RUF059", "RUF013", "N803", "N806", "PLC0415"]
+# e2e/ holds standalone CLI scripts: printing is their job (T201), and
+# they are intentionally not a package (INP001) with exec bits (EXE001).
+"e2e/**/*.py" = ["T201", "INP001", "EXE001"]
 # Package __init__ groups exports by category with comment headers;
 # don't let ruff re-sort them alphabetically.
 "wherobots/__init__.py" = ["RUF022"]
 
-[tool.mypy]
-python_version = "3.10"
-strict = true
-files = ["wherobots"]
-warn_unused_ignores = true
-show_error_codes = true
-
-[[tool.mypy.overrides]]
-# Test code is not held to strict typing — productionize incrementally.
-module = "tests.*"
-ignore_errors = true
-
-[[tool.mypy.overrides]]
-# boto3 / botocore ship runtime types in some versions but the
-# pre-commit mypy env doesn't install them. Declare the policy once
-# here rather than scattering per-import ``# type: ignore`` comments.
-module = ["boto3", "boto3.*", "botocore", "botocore.*"]
-ignore_missing_imports = true
+[tool.pyrefly]
+project-includes = ["wherobots"]
+check-unannotated-defs = true
+strict-callable-subtyping = true
+permissive-ignores = false
+ignore-errors-in-generated-code = false
 
 [tool.pytest.ini_options]
 testpaths = ["tests"]

{wherobots_python_sdk-0.3.0 → wherobots_python_sdk-0.4.0}/tests/test_api.py

@@ -49,7 +49,7 @@ def _make_response(json_data, status_code=200):
 
 
 # =========================================================================== #
-# list()
+# list()  # noqa: ERA001
 # =========================================================================== #
 
 
@@ -120,7 +120,7 @@ class TestList:
 
 
 # =========================================================================== #
-# create()
+# create()  # noqa: ERA001
 # =========================================================================== #
 
 
@@ -177,7 +177,7 @@ class TestCreate:
 
 
 # =========================================================================== #
-# get()
+# get()  # noqa: ERA001
 # =========================================================================== #
 
 
@@ -197,7 +197,7 @@ class TestGet:
 
 
 # =========================================================================== #
-# cancel()
+# cancel()  # noqa: ERA001
 # =========================================================================== #
 
 
@@ -213,7 +213,7 @@ class TestCancel:
 
 
 # =========================================================================== #
-# get_logs()
+# get_logs()  # noqa: ERA001
 # =========================================================================== #
 
 
@@ -241,7 +241,7 @@ class TestGetLogs:
 
 
 # =========================================================================== #
-# get_metrics()
+# get_metrics()  # noqa: ERA001
 # =========================================================================== #
 
 

{wherobots_python_sdk-0.3.0 → wherobots_python_sdk-0.4.0}/tests/test_base_client.py

@@ -8,6 +8,12 @@ import requests
 from wherobots.api.base import BaseClient
 from wherobots.config import WherobotsConfig
 from wherobots.exceptions import WherobotsAPIError, WherobotsConfigError
+from wherobots.utils.client_header import (
+    CLIENT_HEADER,
+    build_hop,
+    resolve_platform,
+    resolve_version,
+)
 
 # ---------------------------------------------------------------------------
 # Fixtures
@@ -34,6 +40,118 @@ class TestAuthHeaders:
         assert "Authorization" not in headers
 
 
+# ---------------------------------------------------------------------------
+# Client attribution (X-Wherobots-Client)
+# ---------------------------------------------------------------------------
+
+
+class TestClientAttributionHeader:
+    @patch("wherobots.api.base.requests.Session")
+    def test_header_set_on_the_session(self, MockSession, config):
+        session = MockSession.return_value
+        session.headers = {}
+        session.mount = MagicMock()
+
+        BaseClient(config)
+
+        assert session.headers[CLIENT_HEADER] == build_hop()
+        assert session.headers[CLIENT_HEADER] == (
+            f"client=python-sdk;ver={resolve_version()};plat={resolve_platform()}"
+        )
+
+    @patch("wherobots.api.base.requests.Session")
+    def test_header_rides_along_with_every_request(self, MockSession, config):
+        # The header lives on the session rather than on individual calls,
+        # so it is attached to whatever requests the client later makes.
+        session = MockSession.return_value
+        session.headers = {}
+        session.mount = MagicMock()
+
+        mock_response = MagicMock()
+        mock_response.raise_for_status.return_value = None
+        session.request.return_value = mock_response
+
+        client = BaseClient(config)
+        client.get("/runs")
+        client.post("/runs", json_body={"name": "test"})
+
+        assert session.request.call_count == 2
+        assert session.headers[CLIENT_HEADER] == build_hop()
+
+    @patch("wherobots.api.base.requests.Session")
+    def test_upstream_chain_is_appended_to_not_replaced(self, MockSession):
+        cfg = WherobotsConfig(
+            api_key="test-key-123",
+            base_url="https://api.test.wherobots.com",
+            client_chain="client=airflow;ver=1.2.0",
+        )
+        session = MockSession.return_value
+        session.headers = {}
+        session.mount = MagicMock()
+
+        BaseClient(cfg)
+
+        assert session.headers[CLIENT_HEADER] == f"client=airflow;ver=1.2.0, {build_hop()}"
+
+    @patch("wherobots.api.base.requests.Session")
+    def test_attribution_never_displaces_auth(self, MockSession):
+        # A CRLF is the only thing in a chain that could ever *become* another
+        # header, so that is what this has to carry: a plain comma-separated
+        # "X-API-Key: nope" is just more text inside one header value and could
+        # not displace auth no matter how the chain were handled.
+        cfg = WherobotsConfig(
+            api_key="test-key-123",
+            base_url="https://api.test.wherobots.com",
+            client_chain="client=spoofed\r\nX-API-Key: nope",
+        )
+        session = MockSession.return_value
+        session.headers = {}
+        session.mount = MagicMock()
+
+        BaseClient(cfg)
+
+        assert session.headers["X-API-Key"] == "test-key-123"
+        assert "\r" not in session.headers[CLIENT_HEADER]
+        assert "\n" not in session.headers[CLIENT_HEADER]
+
+    def test_a_hostile_upstream_chain_still_yields_a_sendable_request(self):
+        """The chain is caller-supplied, so it must not be able to break a call.
+
+        `requests` refuses to send a header value containing a control
+        character, raising `InvalidHeader` — a `requests` exception, not a
+        `WherobotsJobError`. Before the chain was scrubbed, an upstream that
+        forwarded a CR or LF broke every request this SDK made and leaked a
+        foreign exception type doing it.
+
+        Non-ASCII text gets past `requests` entirely and then fails one layer
+        lower, where `http.client.putheader` latin-1 encodes, so both are
+        exercised here and the latin-1 encode is asserted explicitly.
+
+        This asserts against real header preparation rather than a mocked
+        session, because the mock is exactly what hid the defect.
+        """
+        cfg = WherobotsConfig(
+            api_key="test-key-123",
+            base_url="https://api.test.wherobots.com",
+            client_chain="client=evil\r\nX-Injected: yes, client=日本語",
+        )
+
+        with patch("wherobots.api.base.requests.Session") as MockSession:
+            session = MockSession.return_value
+            session.headers = {}
+            session.mount = MagicMock()
+            BaseClient(cfg)
+
+        prepared = requests.Request(
+            "GET", "https://api.test.wherobots.com/runs", headers=session.headers
+        ).prepare()
+
+        assert "X-Injected" not in prepared.headers
+        assert prepared.headers[CLIENT_HEADER].startswith("client=evil")
+        # What http.client actually does with it.
+        prepared.headers[CLIENT_HEADER].encode("latin-1")
+
+
 # ---------------------------------------------------------------------------
 # Config validation
 # ---------------------------------------------------------------------------

{wherobots_python_sdk-0.3.0 → wherobots_python_sdk-0.4.0}/tests/test_client.py

@@ -821,3 +821,35 @@ class TestScriptAutoUpload:
         assert uri == "s3://managed/scripts/job.py"
         mock_files.upload_managed_file.assert_called_once_with("job.py", subdir="scripts")
         mock_files.upload_dependency.assert_not_called()
+
+
+# ── Client attribution chain plumbing ──────────────────────────────────
+
+
+class TestClientChainPlumbing:
+    """``client_chain`` reaches the config every construction path uses."""
+
+    def test_constructor_threads_client_chain_into_config(self, mock_env):
+        job = WherobotsJob(
+            script="s3://bucket/main.py",
+            name="chain-job-name",
+            runtime="tiny",
+            client_chain="client=airflow;ver=1.2.0",
+        )
+
+        assert job._config.client_chain == "client=airflow;ver=1.2.0"
+
+    def test_constructor_defaults_to_no_upstream_chain(self, mock_env, monkeypatch):
+        monkeypatch.delenv("WHEROBOTS_CLIENT_CHAIN", raising=False)
+
+        job = WherobotsJob(script="s3://bucket/main.py", name="chain-job-name", runtime="tiny")
+
+        assert job._config.client_chain is None
+
+    def test_from_run_id_threads_client_chain_into_config(self, mock_env, mock_run_view):
+        with patch("wherobots.client.RunsAPI") as runs_cls:
+            runs_cls.from_config.return_value = _make_mock_api(run_view=mock_run_view)
+
+            job = WherobotsJob.from_run_id("run-123", client_chain="client=airflow;ver=1.2.0")
+
+            assert job._config.client_chain == "client=airflow;ver=1.2.0"