Merge origin/main into chore/decommission-speakeasy-scaffolding

Resolve RELEASES.md modify/delete conflict by keeping the deletion; main's
new entries (0.46.1, 0.46.2) are Speakeasy ledger boilerplate and the
human-facing notes already live in CHANGELOG.md.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

Author: awalker4

.genignore

@@ -31,7 +31,8 @@ jobs:
 
       - name: Run Claude Code
         id: claude
-        uses: anthropics/claude-code-action@beta
+        uses: anthropics/claude-code-action@v1
         with:
           anthropic_api_key: ${{ secrets.GH_ANTHROPIC_API_KEY }}
-          allowed_tools: "Bash(git:*),View,GlobTool,GrepTool,BatchTool"
+          claude_args: |
+            --allowedTools "Bash(git:*),Read,Glob,Grep"

.github/workflows/claude.yml

@@ -1,3 +1,13 @@
+## 0.46.2
+
+### Fixes
+* Accept the API URL the Transform Platform hands you as `server_url`. The app's API Keys page and the docs give you `https://platform-api.transform.unstructured.io/api/v1`, which works with curl but 404'd every Platform call in the SDK: the URL cleaner only stripped a path for `unstructuredapp.io` hosts, so the `/api/v1` survived and the operation's own `/api/v1/jobs/` was appended on top, producing `/api/v1/api/v1/jobs/`. Hosts under `unstructured.io` are now recognized too, and are matched on domain boundaries so a lookalike host like `unstructuredapp.io.example.com` keeps its path and scheme. A `server_url` passed to an individual operation is cleaned as well — previously only the client-level URL was, so `client.jobs.list_jobs(request={}, server_url=...)` still 404'd.
+
+## 0.46.1
+
+### Fixes
+* Fail loudly when a split-PDF chunk returns HTTP 200 with an empty JSON body in NDJSON elements-file mode. The chunk used to be logged and skipped, so the combined `elements_file` was silently short by those pages while the call still returned 200 — and `split_pdf_allow_failed=False` did not catch it, because an empty 200 counts as a successful chunk. Recombination now raises `EmptyChunkResponseError` (a `ValueError`), matching the buffered path, which raises `JSONDecodeError` on the same response. Emptiness is judged against the chunk's own `Content-Type`: JSON has no empty document (a chunk with no elements is `[]`), while `application/x-ndjson` encodes zero records as zero lines, so an empty NDJSON chunk is well formed and still contributes nothing.
+
 ## 0.46.0
 
 ### Features

CHANGELOG.md

@@ -3,14 +3,15 @@
 import logging
 import re
 
+import httpx
 import pytest
 import requests
-import httpx
-from httpx import Response, ConnectError
+from httpx import ConnectError, Response
 
 from _test_unstructured_client.unit_utils import FixtureRequest, Mock, method_mock
 from unstructured_client import UnstructuredClient
-from unstructured_client.models import shared, operations
+from unstructured_client._hooks.custom.clean_server_url_hook import clean_server_url
+from unstructured_client.models import operations, shared
 from unstructured_client.models.errors import SDKError
 from unstructured_client.utils.retries import BackoffStrategy, RetryConfig
 
@@ -34,7 +35,10 @@ def test_unit_retry_with_backoff_does_retry(caplog):
 
     def mock_post(request):
         request_count[0] += 1
-        if request.url == "https://api.unstructuredapp.io/general/v0/general" and request.method == "POST":
+        if (
+            request.url == "https://api.unstructuredapp.io/general/v0/general"
+            and request.method == "POST"
+        ):
             return Response(502, request=request)
 
     transport = httpx.MockTransport(mock_post)
@@ -69,7 +73,10 @@ def test_unit_backoff_strategy_logs_retries_5XX(status_code: int, caplog):
     )
 
     def mock_post(request):
-        if request.url == "https://api.unstructuredapp.io/general/v0/general" and request.method == "POST":
+        if (
+            request.url == "https://api.unstructuredapp.io/general/v0/general"
+            and request.method == "POST"
+        ):
             return Response(status_code, request=request)
 
     transport = httpx.MockTransport(mock_post)
@@ -83,11 +90,13 @@ def mock_post(request):
         partition_parameters=shared.PartitionParameters(files=files)
     )
 
-    with pytest.raises(Exception):
+    with pytest.raises(Exception):  # noqa: B017
         session.general.partition(request=req, retries=retries)
 
-    pattern = re.compile(f"Failed to process a request due to API server error with status code {status_code}. "
-                        "Attempting retry number 1 after sleep.")
+    pattern = re.compile(
+        f"Failed to process a request due to API server error with status code {status_code}. "
+        "Attempting retry number 1 after sleep."
+    )
     assert bool(pattern.search(caplog.text))
 
 
@@ -103,9 +112,11 @@ def mock_post(request):
         [502, True],
         [503, True],
         [504, True],
-    ]
+    ],
 )
-def test_unit_number_of_retries_in_failed_requests(status_code: int, expect_retry: bool):
+def test_unit_number_of_retries_in_failed_requests(
+    status_code: int, expect_retry: bool
+):
     filename = "README.md"
     backoff_strategy = BackoffStrategy(
         initial_interval=1, max_interval=10, exponent=1.5, max_elapsed_time=300
@@ -115,17 +126,19 @@ def test_unit_number_of_retries_in_failed_requests(status_code: int, expect_retr
     )
 
     number_of_requests = [0]
+
     def mock_post(request):
-        if request.url == "https://api.unstructuredapp.io/general/v0/general" and request.method == "POST":
+        if (
+            request.url == "https://api.unstructuredapp.io/general/v0/general"
+            and request.method == "POST"
+        ):
             number_of_requests[0] += 1
             return Response(status_code, request=request)
 
-
     transport = httpx.MockTransport(mock_post)
     client = httpx.Client(transport=transport)
     session = UnstructuredClient(api_key_auth=FAKE_KEY, client=client)
 
-
     with open(filename, "rb") as f:
         files = shared.Files(content=f.read(), file_name=filename)
 
@@ -166,11 +179,13 @@ def mock_post(request):
         partition_parameters=shared.PartitionParameters(files=files)
     )
 
-    with pytest.raises(Exception):
+    with pytest.raises(Exception):  # noqa: B017
         session.general.partition(request=req, retries=retries)
 
-    pattern = re.compile("Failed to process a request due to transport error .*? "
-                         "Attempting retry number 1 after sleep.")
+    pattern = re.compile(
+        "Failed to process a request due to transport error .*? "
+        "Attempting retry number 1 after sleep."
+    )
     assert bool(pattern.search(caplog.text))
 
 
@@ -187,31 +202,93 @@ def mock_post(request):
     ],
 )
 def test_unit_clean_server_url_fixes_malformed_paid_api_url(server_url: str):
-    client = UnstructuredClient(
-        server_url=server_url,
-        api_key_auth=FAKE_KEY,
-    )
     assert (
-        client.general.sdk_configuration.server_url
+        clean_server_url(server_url)
         == "https://unstructured-000mock.api.unstructuredapp.io"
     )
 
 
+@pytest.mark.parametrize(
+    "server_url",
+    [
+        # -- the value the Transform Platform's API Keys page hands you --
+        "https://platform-api.transform.unstructured.io/api/v1",
+        "http://platform-api.transform.unstructured.io/api/v1",
+        "platform-api.transform.unstructured.io/api/v1",
+        # -- well-formed url --
+        "https://platform-api.transform.unstructured.io",
+        "platform-api.transform.unstructured.io",
+    ],
+)
+def test_unit_clean_server_url_fixes_malformed_transform_platform_url(server_url: str):
+    assert (
+        clean_server_url(server_url) == "https://platform-api.transform.unstructured.io"
+    )
+
+
+@pytest.mark.parametrize(
+    "server_url,expected_url",
+    [
+        # -- the terminal root dot is a valid fully qualified name and still ours, so
+        # -- the path goes; the host is left exactly as the caller wrote it, because the
+        # -- dot is a deliberate DNS choice that changes the Host header and SNI --
+        (
+            "https://platform-api.transform.unstructured.io./api/v1",
+            "https://platform-api.transform.unstructured.io.",
+        ),
+        (
+            "https://unstructured-000mock.api.unstructuredapp.io./general/v0/general",
+            "https://unstructured-000mock.api.unstructuredapp.io.",
+        ),
+    ],
+)
+def test_unit_clean_server_url_handles_a_fully_qualified_host(
+    server_url: str, expected_url: str
+):
+    assert clean_server_url(server_url) == expected_url
+
+
+@pytest.mark.parametrize(
+    "server_url,expected_url",
+    [
+        # -- a host that merely CONTAINS an Unstructured domain is not ours, so its
+        # -- path and scheme are left alone --
+        (
+            "http://unstructuredapp.io.example.com/api/v1",
+            "http://unstructuredapp.io.example.com/api/v1",
+        ),
+        (
+            "http://not-unstructured.io/api/v1",
+            "http://not-unstructured.io/api/v1",
+        ),
+    ],
+)
+def test_unit_clean_server_url_leaves_lookalike_domains_alone(
+    server_url: str, expected_url: str
+):
+    assert clean_server_url(server_url) == expected_url
+
+
 @pytest.mark.parametrize(
     "server_url,expected_url",
     [
         ("http://localhost:8000", "http://localhost:8000"),
         ("localhost:8000", "http://localhost:8000"),
-        ("localhost:8000/general/v0/general", "http://localhost:8000/general/v0/general"),
-        ("http://localhost:8000/general/v0/general", "http://localhost:8000/general/v0/general"),
+        (
+            "localhost:8000/general/v0/general",
+            "http://localhost:8000/general/v0/general",
+        ),
+        (
+            "http://localhost:8000/general/v0/general",
+            "http://localhost:8000/general/v0/general",
+        ),
     ],
 )
-def test_unit_clean_server_url_fixes_non_unst_domain_url(server_url: str, expected_url: str):
-    client = UnstructuredClient(
-        server_url=server_url,
-        api_key_auth=FAKE_KEY,
-    )
-    assert client.general.sdk_configuration.server_url == expected_url
+def test_unit_clean_server_url_fixes_non_unst_domain_url(
+    server_url: str, expected_url: str
+):
+    assert clean_server_url(server_url) == expected_url
+
 
 @pytest.mark.parametrize(
     "server_url",
@@ -222,10 +299,11 @@ def test_unit_clean_server_url_fixes_non_unst_domain_url(server_url: str, expect
         "unstructured-000mock.api.unstructuredapp.io/general/v0/general",
     ],
 )
-def test_unit_clean_server_url_fixes_malformed_urls_with_positional_arguments(server_url: str):
-    client = UnstructuredClient(FAKE_KEY, server_url=server_url)
+def test_unit_clean_server_url_fixes_malformed_urls_with_positional_arguments(
+    server_url: str,
+):
     assert (
-        client.general.sdk_configuration.server_url
+        clean_server_url(server_url)
         == "https://unstructured-000mock.api.unstructuredapp.io"
     )
 
@@ -247,11 +325,8 @@ def mock_post(request):
     )
     with pytest.raises(SDKError, match="API error occurred: Status 401"):
         session.general.partition(request=req)
-    
-    assert any(
-        "Server responded with 401"
-        in message for message in caplog.messages
-    )
+
+    assert any("Server responded with 401" in message for message in caplog.messages)
 
 
 # -- fixtures --------------------------------------------------------------------------------