Remove PyPI publish workflow, fix CI issues

- Remove publish-pypi.yml (private fork, no PyPI publishing)
- Fix mypy type ignore in _compat.py line 29
- Fix mock.patch strings in tests (orb -> orb_sdk)
- Fix error message strings in tests
- Update apply-customizations.sh with all fixes

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <[email protected]>

Author: srir

.github/workflows/publish-pypi.yml

@@ -24,6 +24,15 @@ find tests -name "*.py" -exec sed -i 's/from orb$/from orb_sdk/g' {} +
 find tests -name "*.py" -exec sed -i 's/import orb\([^_]\)/import orb_sdk\1/g' {} +
 find tests -name "*.py" -exec sed -i 's/import orb$/import orb_sdk/g' {} +
 
+# Update mock.patch strings and error messages in tests
+echo "Updating mock.patch and error message strings in tests/"
+find tests -name "*.py" -exec sed -i "s/\"orb\._/\"orb_sdk._/g" {} +
+find tests -name "*.py" -exec sed -i "s/'orb\._/'orb_sdk._/g" {} +
+
+# Add type ignore to _compat.py line 29 if missing
+echo "Adding type ignore comment to _compat.py"
+sed -i 's/return tp is Union or tp is types.UnionType$/return tp is Union or tp is types.UnionType  # type: ignore[comparison-overlap]/' src/orb_sdk/_utils/_compat.py
+
 # Update pyproject.toml
 echo "Updating pyproject.toml"
 sed -i 's/name = "orb-billing"/name = "orb_sdk"/' pyproject.toml

scripts/apply-customizations.sh

@@ -26,7 +26,7 @@ def is_union(tp: Optional[Type[Any]]) -> bool:
     else:
         import types
 
-        return tp is Union or tp is types.UnionType
+        return tp is Union or tp is types.UnionType  # type: ignore[comparison-overlap]
 
 
 def is_typeddict(tp: Type[Any]) -> bool:

src/orb_sdk/_utils/_compat.py

@@ -752,7 +752,7 @@ def test_parse_retry_after_header(
         calculated = client._calculate_retry_timeout(remaining_retries, options, headers)
         assert calculated == pytest.approx(timeout, 0.5 * 0.875)  # pyright: ignore[reportUnknownMemberType]
 
-    @mock.patch("orb._base_client.BaseClient._calculate_retry_timeout", _low_retry_timeout)
+    @mock.patch("orb_sdk._base_client.BaseClient._calculate_retry_timeout", _low_retry_timeout)
     @pytest.mark.respx(base_url=base_url)
     def test_retrying_timeout_errors_doesnt_leak(self, respx_mock: MockRouter, client: Orb) -> None:
         respx_mock.post("/customers").mock(side_effect=httpx.TimeoutException("Test timeout error"))
@@ -762,7 +762,7 @@ def test_retrying_timeout_errors_doesnt_leak(self, respx_mock: MockRouter, clien
 
         assert _get_open_connections(client) == 0
 
-    @mock.patch("orb._base_client.BaseClient._calculate_retry_timeout", _low_retry_timeout)
+    @mock.patch("orb_sdk._base_client.BaseClient._calculate_retry_timeout", _low_retry_timeout)
     @pytest.mark.respx(base_url=base_url)
     def test_retrying_status_errors_doesnt_leak(self, respx_mock: MockRouter, client: Orb) -> None:
         respx_mock.post("/customers").mock(return_value=httpx.Response(500))
@@ -772,7 +772,7 @@ def test_retrying_status_errors_doesnt_leak(self, respx_mock: MockRouter, client
         assert _get_open_connections(client) == 0
 
     @pytest.mark.parametrize("failures_before_success", [0, 2, 4])
-    @mock.patch("orb._base_client.BaseClient._calculate_retry_timeout", _low_retry_timeout)
+    @mock.patch("orb_sdk._base_client.BaseClient._calculate_retry_timeout", _low_retry_timeout)
     @pytest.mark.respx(base_url=base_url)
     @pytest.mark.parametrize("failure_mode", ["status", "exception"])
     def test_retries_taken(
@@ -803,7 +803,7 @@ def retry_handler(_request: httpx.Request) -> httpx.Response:
         assert int(response.http_request.headers.get("x-stainless-retry-count")) == failures_before_success
 
     @pytest.mark.parametrize("failures_before_success", [0, 2, 4])
-    @mock.patch("orb._base_client.BaseClient._calculate_retry_timeout", _low_retry_timeout)
+    @mock.patch("orb_sdk._base_client.BaseClient._calculate_retry_timeout", _low_retry_timeout)
     @pytest.mark.respx(base_url=base_url)
     def test_omit_retry_count_header(self, client: Orb, failures_before_success: int, respx_mock: MockRouter) -> None:
         client = client.with_options(max_retries=4)
@@ -826,7 +826,7 @@ def retry_handler(_request: httpx.Request) -> httpx.Response:
         assert len(response.http_request.headers.get_list("x-stainless-retry-count")) == 0
 
     @pytest.mark.parametrize("failures_before_success", [0, 2, 4])
-    @mock.patch("orb._base_client.BaseClient._calculate_retry_timeout", _low_retry_timeout)
+    @mock.patch("orb_sdk._base_client.BaseClient._calculate_retry_timeout", _low_retry_timeout)
     @pytest.mark.respx(base_url=base_url)
     def test_overwrite_retry_count_header(
         self, client: Orb, failures_before_success: int, respx_mock: MockRouter
@@ -851,7 +851,7 @@ def retry_handler(_request: httpx.Request) -> httpx.Response:
         assert response.http_request.headers.get("x-stainless-retry-count") == "42"
 
     @pytest.mark.parametrize("failures_before_success", [0, 2, 4])
-    @mock.patch("orb._base_client.BaseClient._calculate_retry_timeout", _low_retry_timeout)
+    @mock.patch("orb_sdk._base_client.BaseClient._calculate_retry_timeout", _low_retry_timeout)
     @pytest.mark.respx(base_url=base_url)
     def test_retries_taken_new_response_class(
         self, client: Orb, failures_before_success: int, respx_mock: MockRouter
@@ -1630,7 +1630,7 @@ async def test_parse_retry_after_header(
         calculated = async_client._calculate_retry_timeout(remaining_retries, options, headers)
         assert calculated == pytest.approx(timeout, 0.5 * 0.875)  # pyright: ignore[reportUnknownMemberType]
 
-    @mock.patch("orb._base_client.BaseClient._calculate_retry_timeout", _low_retry_timeout)
+    @mock.patch("orb_sdk._base_client.BaseClient._calculate_retry_timeout", _low_retry_timeout)
     @pytest.mark.respx(base_url=base_url)
     async def test_retrying_timeout_errors_doesnt_leak(self, respx_mock: MockRouter, async_client: AsyncOrb) -> None:
         respx_mock.post("/customers").mock(side_effect=httpx.TimeoutException("Test timeout error"))
@@ -1642,7 +1642,7 @@ async def test_retrying_timeout_errors_doesnt_leak(self, respx_mock: MockRouter,
 
         assert _get_open_connections(async_client) == 0
 
-    @mock.patch("orb._base_client.BaseClient._calculate_retry_timeout", _low_retry_timeout)
+    @mock.patch("orb_sdk._base_client.BaseClient._calculate_retry_timeout", _low_retry_timeout)
     @pytest.mark.respx(base_url=base_url)
     async def test_retrying_status_errors_doesnt_leak(self, respx_mock: MockRouter, async_client: AsyncOrb) -> None:
         respx_mock.post("/customers").mock(return_value=httpx.Response(500))
@@ -1654,7 +1654,7 @@ async def test_retrying_status_errors_doesnt_leak(self, respx_mock: MockRouter,
         assert _get_open_connections(async_client) == 0
 
     @pytest.mark.parametrize("failures_before_success", [0, 2, 4])
-    @mock.patch("orb._base_client.BaseClient._calculate_retry_timeout", _low_retry_timeout)
+    @mock.patch("orb_sdk._base_client.BaseClient._calculate_retry_timeout", _low_retry_timeout)
     @pytest.mark.respx(base_url=base_url)
     @pytest.mark.parametrize("failure_mode", ["status", "exception"])
     async def test_retries_taken(
@@ -1685,7 +1685,7 @@ def retry_handler(_request: httpx.Request) -> httpx.Response:
         assert int(response.http_request.headers.get("x-stainless-retry-count")) == failures_before_success
 
     @pytest.mark.parametrize("failures_before_success", [0, 2, 4])
-    @mock.patch("orb._base_client.BaseClient._calculate_retry_timeout", _low_retry_timeout)
+    @mock.patch("orb_sdk._base_client.BaseClient._calculate_retry_timeout", _low_retry_timeout)
     @pytest.mark.respx(base_url=base_url)
     async def test_omit_retry_count_header(
         self, async_client: AsyncOrb, failures_before_success: int, respx_mock: MockRouter
@@ -1710,7 +1710,7 @@ def retry_handler(_request: httpx.Request) -> httpx.Response:
         assert len(response.http_request.headers.get_list("x-stainless-retry-count")) == 0
 
     @pytest.mark.parametrize("failures_before_success", [0, 2, 4])
-    @mock.patch("orb._base_client.BaseClient._calculate_retry_timeout", _low_retry_timeout)
+    @mock.patch("orb_sdk._base_client.BaseClient._calculate_retry_timeout", _low_retry_timeout)
     @pytest.mark.respx(base_url=base_url)
     async def test_overwrite_retry_count_header(
         self, async_client: AsyncOrb, failures_before_success: int, respx_mock: MockRouter
@@ -1735,7 +1735,7 @@ def retry_handler(_request: httpx.Request) -> httpx.Response:
         assert response.http_request.headers.get("x-stainless-retry-count") == "42"
 
     @pytest.mark.parametrize("failures_before_success", [0, 2, 4])
-    @mock.patch("orb._base_client.BaseClient._calculate_retry_timeout", _low_retry_timeout)
+    @mock.patch("orb_sdk._base_client.BaseClient._calculate_retry_timeout", _low_retry_timeout)
     @pytest.mark.respx(base_url=base_url)
     async def test_retries_taken_new_response_class(
         self, async_client: AsyncOrb, failures_before_success: int, respx_mock: MockRouter

tests/test_client.py

@@ -37,7 +37,7 @@ def test_extract_response_type_direct_classes() -> None:
 def test_extract_response_type_direct_class_missing_type_arg() -> None:
     with pytest.raises(
         RuntimeError,
-        match="Expected type <class 'orb._response.AsyncAPIResponse'> to have a type argument at index 0 but it did not",
+        match="Expected type <class 'orb_sdk._response.AsyncAPIResponse'> to have a type argument at index 0 but it did not",
     ):
         extract_response_type(AsyncAPIResponse)