Merge pull request #4 from channel3-ai/release-please--branches--main--changes--next

ahs53 · web-flow · commit dd6ab66e4a45 · 2025-09-16T13:20:11.000-07:00
release: 2.2.0
diff --git a/.release-please-manifest.json b/.release-please-manifest.json
@@ -1,3 +1,3 @@
 {
-  ".": "2.1.0"
+  ".": "2.2.0"
 }
diff --git a/.stats.yml b/.stats.yml
@@ -1,4 +1,4 @@
 configured_endpoints: 6
 openapi_spec_url: https://storage.googleapis.com/stainless-sdk-openapi-specs/channel3%2Fpublic-sdk-c870f6f12fcf671675af9dfddb9b6e4fd968bc75717a50d53d5626180ecbdcaf.yml
 openapi_spec_hash: 3ac13f44aeb96508ab9073b88a9f633d
-config_hash: 8e18f4f27ef0200cac8c5ee682f53ab8
+config_hash: 1a71e5acc94d5679abff73aafac92746
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,5 +1,13 @@
 # Changelog
 
+## 2.2.0 (2025-09-16)
+
+Full Changelog: [v2.1.0...v2.2.0](https://github.com/channel3-ai/sdk-python/compare/v2.1.0...v2.2.0)
+
+### Features
+
+* **api:** manual updates ([79f6922](https://github.com/channel3-ai/sdk-python/commit/79f69227ff70ac778731b6dac5f1297a66800893))
+
 ## 2.1.0 (2025-09-16)
 
 Full Changelog: [v2.0.0...v2.1.0](https://github.com/channel3-ai/sdk-python/compare/v2.0.0...v2.1.0)
diff --git a/README.md b/README.md
@@ -30,8 +30,6 @@ from channel3_sdk import Channel3
 
 client = Channel3(
     api_key=os.environ.get("CHANNEL3_API_KEY"),  # This is the default and can be omitted
-    # defaults to "production".
-    environment="development",
 )
 
 response = client.search.perform()
@@ -53,8 +51,6 @@ from channel3_sdk import AsyncChannel3
 
 client = AsyncChannel3(
     api_key=os.environ.get("CHANNEL3_API_KEY"),  # This is the default and can be omitted
-    # defaults to "production".
-    environment="development",
 )
 
 
diff --git a/pyproject.toml b/pyproject.toml
@@ -1,6 +1,6 @@
 [project]
 name = "channel3_sdk"
-version = "2.1.0"
+version = "2.2.0"
 description = "The official Python library for the channel3 API"
 dynamic = ["readme"]
 license = "Apache-2.0"
diff --git a/src/channel3_sdk/__init__.py b/src/channel3_sdk/__init__.py
@@ -6,7 +6,6 @@
 from ._types import NOT_GIVEN, Omit, NoneType, NotGiven, Transport, ProxiesTypes
 from ._utils import file_from_path
 from ._client import (
-    ENVIRONMENTS,
     Client,
     Stream,
     Timeout,
@@ -72,7 +71,6 @@
     "AsyncStream",
     "Channel3",
     "AsyncChannel3",
-    "ENVIRONMENTS",
     "file_from_path",
     "BaseModel",
     "DEFAULT_TIMEOUT",
diff --git a/src/channel3_sdk/_client.py b/src/channel3_sdk/_client.py
@@ -3,8 +3,8 @@
 from __future__ import annotations
 
 import os
-from typing import Any, Dict, Union, Mapping, cast
-from typing_extensions import Self, Literal, override
+from typing import Any, Union, Mapping
+from typing_extensions import Self, override
 
 import httpx
 
@@ -41,7 +41,6 @@
 )
 
 __all__ = [
-    "ENVIRONMENTS",
     "Timeout",
     "Transport",
     "ProxiesTypes",
@@ -52,11 +51,6 @@
     "AsyncClient",
 ]
 
-ENVIRONMENTS: Dict[str, str] = {
-    "production": "https://api.trychannel3.com",
-    "development": "https://localhost:8000",
-}
-
 
 class Channel3(SyncAPIClient):
     search: search.SearchResource
@@ -69,14 +63,11 @@ class Channel3(SyncAPIClient):
     # client options
     api_key: str
 
-    _environment: Literal["production", "development"] | NotGiven
-
     def __init__(
         self,
         *,
         api_key: str | None = None,
-        environment: Literal["production", "development"] | NotGiven = NOT_GIVEN,
-        base_url: str | httpx.URL | None | NotGiven = NOT_GIVEN,
+        base_url: str | httpx.URL | None = None,
         timeout: Union[float, Timeout, None, NotGiven] = NOT_GIVEN,
         max_retries: int = DEFAULT_MAX_RETRIES,
         default_headers: Mapping[str, str] | None = None,
@@ -107,31 +98,10 @@ def __init__(
             )
         self.api_key = api_key
 
-        self._environment = environment
-
-        base_url_env = os.environ.get("CHANNEL3_BASE_URL")
-        if is_given(base_url) and base_url is not None:
-            # cast required because mypy doesn't understand the type narrowing
-            base_url = cast("str | httpx.URL", base_url)  # pyright: ignore[reportUnnecessaryCast]
-        elif is_given(environment):
-            if base_url_env and base_url is not None:
-                raise ValueError(
-                    "Ambiguous URL; The `CHANNEL3_BASE_URL` env var and the `environment` argument are given. If you want to use the environment, you must pass base_url=None",
-                )
-
-            try:
-                base_url = ENVIRONMENTS[environment]
-            except KeyError as exc:
-                raise ValueError(f"Unknown environment: {environment}") from exc
-        elif base_url_env is not None:
-            base_url = base_url_env
-        else:
-            self._environment = environment = "production"
-
-            try:
-                base_url = ENVIRONMENTS[environment]
-            except KeyError as exc:
-                raise ValueError(f"Unknown environment: {environment}") from exc
+        if base_url is None:
+            base_url = os.environ.get("CHANNEL3_BASE_URL")
+        if base_url is None:
+            base_url = f"https://api.trychannel3.com"
 
         super().__init__(
             version=__version__,
@@ -175,7 +145,6 @@ def copy(
         self,
         *,
         api_key: str | None = None,
-        environment: Literal["production", "development"] | None = None,
         base_url: str | httpx.URL | None = None,
         timeout: float | Timeout | None | NotGiven = NOT_GIVEN,
         http_client: httpx.Client | None = None,
@@ -211,7 +180,6 @@ def copy(
         return self.__class__(
             api_key=api_key or self.api_key,
             base_url=base_url or self.base_url,
-            environment=environment or self._environment,
             timeout=self.timeout if isinstance(timeout, NotGiven) else timeout,
             http_client=http_client,
             max_retries=max_retries if is_given(max_retries) else self.max_retries,
@@ -288,14 +256,11 @@ class AsyncChannel3(AsyncAPIClient):
     # client options
     api_key: str
 
-    _environment: Literal["production", "development"] | NotGiven
-
     def __init__(
         self,
         *,
         api_key: str | None = None,
-        environment: Literal["production", "development"] | NotGiven = NOT_GIVEN,
-        base_url: str | httpx.URL | None | NotGiven = NOT_GIVEN,
+        base_url: str | httpx.URL | None = None,
         timeout: Union[float, Timeout, None, NotGiven] = NOT_GIVEN,
         max_retries: int = DEFAULT_MAX_RETRIES,
         default_headers: Mapping[str, str] | None = None,
@@ -326,31 +291,10 @@ def __init__(
             )
         self.api_key = api_key
 
-        self._environment = environment
-
-        base_url_env = os.environ.get("CHANNEL3_BASE_URL")
-        if is_given(base_url) and base_url is not None:
-            # cast required because mypy doesn't understand the type narrowing
-            base_url = cast("str | httpx.URL", base_url)  # pyright: ignore[reportUnnecessaryCast]
-        elif is_given(environment):
-            if base_url_env and base_url is not None:
-                raise ValueError(
-                    "Ambiguous URL; The `CHANNEL3_BASE_URL` env var and the `environment` argument are given. If you want to use the environment, you must pass base_url=None",
-                )
-
-            try:
-                base_url = ENVIRONMENTS[environment]
-            except KeyError as exc:
-                raise ValueError(f"Unknown environment: {environment}") from exc
-        elif base_url_env is not None:
-            base_url = base_url_env
-        else:
-            self._environment = environment = "production"
-
-            try:
-                base_url = ENVIRONMENTS[environment]
-            except KeyError as exc:
-                raise ValueError(f"Unknown environment: {environment}") from exc
+        if base_url is None:
+            base_url = os.environ.get("CHANNEL3_BASE_URL")
+        if base_url is None:
+            base_url = f"https://api.trychannel3.com"
 
         super().__init__(
             version=__version__,
@@ -394,7 +338,6 @@ def copy(
         self,
         *,
         api_key: str | None = None,
-        environment: Literal["production", "development"] | None = None,
         base_url: str | httpx.URL | None = None,
         timeout: float | Timeout | None | NotGiven = NOT_GIVEN,
         http_client: httpx.AsyncClient | None = None,
@@ -430,7 +373,6 @@ def copy(
         return self.__class__(
             api_key=api_key or self.api_key,
             base_url=base_url or self.base_url,
-            environment=environment or self._environment,
             timeout=self.timeout if isinstance(timeout, NotGiven) else timeout,
             http_client=http_client,
             max_retries=max_retries if is_given(max_retries) else self.max_retries,
diff --git a/src/channel3_sdk/_version.py b/src/channel3_sdk/_version.py
@@ -1,4 +1,4 @@
 # File generated from our OpenAPI spec by Stainless. See CONTRIBUTING.md for details.
 
 __title__ = "channel3_sdk"
-__version__ = "2.1.0"  # x-release-please-version
+__version__ = "2.2.0"  # x-release-please-version
diff --git a/tests/test_client.py b/tests/test_client.py
@@ -560,16 +560,6 @@ def test_base_url_env(self) -> None:
             client = Channel3(api_key=api_key, _strict_response_validation=True)
             assert client.base_url == "http://localhost:5000/from/env/"
 
-        # explicit environment arg requires explicitness
-        with update_env(CHANNEL3_BASE_URL="http://localhost:5000/from/env"):
-            with pytest.raises(ValueError, match=r"you must pass base_url=None"):
-                Channel3(api_key=api_key, _strict_response_validation=True, environment="production")
-
-            client = Channel3(
-                base_url=None, api_key=api_key, _strict_response_validation=True, environment="production"
-            )
-            assert str(client.base_url).startswith("https://api.trychannel3.com")
-
     @pytest.mark.parametrize(
         "client",
         [
@@ -1373,16 +1363,6 @@ def test_base_url_env(self) -> None:
             client = AsyncChannel3(api_key=api_key, _strict_response_validation=True)
             assert client.base_url == "http://localhost:5000/from/env/"
 
-        # explicit environment arg requires explicitness
-        with update_env(CHANNEL3_BASE_URL="http://localhost:5000/from/env"):
-            with pytest.raises(ValueError, match=r"you must pass base_url=None"):
-                AsyncChannel3(api_key=api_key, _strict_response_validation=True, environment="production")
-
-            client = AsyncChannel3(
-                base_url=None, api_key=api_key, _strict_response_validation=True, environment="production"
-            )
-            assert str(client.base_url).startswith("https://api.trychannel3.com")
-
     @pytest.mark.parametrize(
         "client",
         [

Original file line number	Diff line number	Diff line change
`@@ -1,3 +1,3 @@`
`1`	`1`	`{`
`2`		`- ".": "2.1.0"`
	`2`	`+ ".": "2.2.0"`
`3`	`3`	`}`
Original file line number	Diff line number	Diff line change
`@@ -30,8 +30,6 @@ from channel3_sdk import Channel3`
`30`	`30`
`31`	`31`	`client = Channel3(`
`32`	`32`	`api_key=os.environ.get("CHANNEL3_API_KEY"), # This is the default and can be omitted`
`33`		`- # defaults to "production".`
`34`		`- environment="development",`
`35`	`33`	`)`
`36`	`34`
`37`	`35`	`response = client.search.perform()`
`@@ -53,8 +51,6 @@ from channel3_sdk import AsyncChannel3`
`53`	`51`
`54`	`52`	`client = AsyncChannel3(`
`55`	`53`	`api_key=os.environ.get("CHANNEL3_API_KEY"), # This is the default and can be omitted`
`56`		`- # defaults to "production".`
`57`		`- environment="development",`
`58`	`54`	`)`
`59`	`55`
`60`	`56`