Remove "pdf" from Summarize method names

Author: datalogics-cgreen

src/pdfrest/client.py

@@ -2132,7 +2132,7 @@ def query_pdf_info(
         raw_payload = self._send_request(request)
         return PdfRestInfoResponse.model_validate(raw_payload)
 
-    def summarize_pdf_text(
+    def summarize_text(
         self,
         file: PdfRestFile | Sequence[PdfRestFile],
         *,
@@ -2179,7 +2179,7 @@ def summarize_pdf_text(
         raw_payload = self._send_request(request)
         return SummarizePdfTextResponse.model_validate(raw_payload)
 
-    def summarize_pdf_text_to_file(
+    def summarize_text_to_file(
         self,
         file: PdfRestFile | Sequence[PdfRestFile],
         *,
@@ -3142,7 +3142,7 @@ async def query_pdf_info(
         raw_payload = await self._send_request(request)
         return PdfRestInfoResponse.model_validate(raw_payload)
 
-    async def summarize_pdf_text(
+    async def summarize_text(
         self,
         file: PdfRestFile | Sequence[PdfRestFile],
         *,
@@ -3189,7 +3189,7 @@ async def summarize_pdf_text(
         raw_payload = await self._send_request(request)
         return SummarizePdfTextResponse.model_validate(raw_payload)
 
-    async def summarize_pdf_text_to_file(
+    async def summarize_text_to_file(
         self,
         file: PdfRestFile | Sequence[PdfRestFile],
         *,

tests/live/test_live_summarize_pdf_text.py

@@ -8,7 +8,7 @@
 from ..resources import get_test_resource_path
 
 
-def test_live_summarize_pdf_text_success(
+def test_live_summarize_text_success(
     pdfrest_api_key: str,
     pdfrest_live_base_url: str,
 ) -> None:
@@ -18,7 +18,7 @@ def test_live_summarize_pdf_text_success(
         base_url=pdfrest_live_base_url,
     ) as client:
         uploaded = client.files.create_from_paths([resource])[0]
-        response = client.summarize_pdf_text(
+        response = client.summarize_text(
             uploaded,
             target_word_count=40,
             summary_format="overview",
@@ -29,7 +29,7 @@ def test_live_summarize_pdf_text_success(
     assert response.input_id == uploaded.id
 
 
-def test_live_summarize_pdf_text_to_file_success(
+def test_live_summarize_text_to_file_success(
     pdfrest_api_key: str,
     pdfrest_live_base_url: str,
 ) -> None:
@@ -39,7 +39,7 @@ def test_live_summarize_pdf_text_to_file_success(
         base_url=pdfrest_live_base_url,
     ) as client:
         uploaded = client.files.create_from_paths([resource])[0]
-        response = client.summarize_pdf_text_to_file(
+        response = client.summarize_text_to_file(
             uploaded,
             target_word_count=40,
             summary_format="overview",
@@ -56,7 +56,7 @@ def test_live_summarize_pdf_text_to_file_success(
 
 
 @pytest.mark.asyncio
-async def test_live_async_summarize_pdf_text_success(
+async def test_live_async_summarize_text_success(
     pdfrest_api_key: str,
     pdfrest_live_base_url: str,
 ) -> None:
@@ -66,7 +66,7 @@ async def test_live_async_summarize_pdf_text_success(
         base_url=pdfrest_live_base_url,
     ) as client:
         uploaded = (await client.files.create_from_paths([resource]))[0]
-        response = await client.summarize_pdf_text(
+        response = await client.summarize_text(
             uploaded,
             target_word_count=30,
             summary_format="overview",
@@ -77,7 +77,7 @@ async def test_live_async_summarize_pdf_text_success(
     assert response.input_id == uploaded.id
 
 
-def test_live_summarize_pdf_text_invalid_format(
+def test_live_summarize_text_invalid_format(
     pdfrest_api_key: str,
     pdfrest_live_base_url: str,
 ) -> None:
@@ -88,14 +88,14 @@ def test_live_summarize_pdf_text_invalid_format(
     ) as client:
         uploaded = client.files.create_from_paths([resource])[0]
         with pytest.raises(PdfRestApiError, match=r"(?i)summary"):
-            client.summarize_pdf_text(
+            client.summarize_text(
                 uploaded,
                 extra_body={"summary_format": "invalid-style"},
             )
 
 
 @pytest.mark.asyncio
-async def test_live_async_summarize_pdf_text_invalid_format(
+async def test_live_async_summarize_text_invalid_format(
     pdfrest_api_key: str,
     pdfrest_live_base_url: str,
 ) -> None:
@@ -106,7 +106,7 @@ async def test_live_async_summarize_pdf_text_invalid_format(
     ) as client:
         uploaded = (await client.files.create_from_paths([resource]))[0]
         with pytest.raises(PdfRestApiError, match=r"(?i)summary"):
-            await client.summarize_pdf_text(
+            await client.summarize_text(
                 uploaded,
                 extra_body={"summary_format": "invalid-style"},
             )

tests/test_summarize_pdf_text.py

@@ -66,7 +66,7 @@ def test_summarize_payload_invalid_page_range() -> None:
         SummarizePdfTextPayload.model_validate({"files": [file_repr], "pages": ["5-2"]})
 
 
-def test_summarize_pdf_text_json_success(monkeypatch: pytest.MonkeyPatch) -> None:
+def test_summarize_text_json_success(monkeypatch: pytest.MonkeyPatch) -> None:
     monkeypatch.delenv("PDFREST_API_KEY", raising=False)
     input_file = _make_text_file(str(PdfRestFileID.generate(1)))
     payload_dump = SummarizePdfTextPayload.model_validate(
@@ -100,7 +100,7 @@ def handler(request: httpx.Request) -> httpx.Response:
 
     transport = httpx.MockTransport(handler)
     with PdfRestClient(api_key=VALID_API_KEY, transport=transport) as client:
-        response = client.summarize_pdf_text(
+        response = client.summarize_text(
             input_file,
             target_word_count=120,
             summary_format="bullet_points",
@@ -115,7 +115,7 @@ def handler(request: httpx.Request) -> httpx.Response:
     assert response.input_id == input_file.id
 
 
-def test_summarize_pdf_text_to_file_success(
+def test_summarize_text_to_file_success(
     monkeypatch: pytest.MonkeyPatch,
 ) -> None:
     monkeypatch.delenv("PDFREST_API_KEY", raising=False)
@@ -158,7 +158,7 @@ def handler(request: httpx.Request) -> httpx.Response:
 
     transport = httpx.MockTransport(handler)
     with PdfRestClient(api_key=VALID_API_KEY, transport=transport) as client:
-        response = client.summarize_pdf_text_to_file(
+        response = client.summarize_text_to_file(
             input_file,
             target_word_count=200,
             summary_format="bullet_points",
@@ -174,7 +174,7 @@ def handler(request: httpx.Request) -> httpx.Response:
     assert response.input_id == input_file.id
 
 
-def test_summarize_pdf_text_to_file_request_customization(
+def test_summarize_text_to_file_request_customization(
     monkeypatch: pytest.MonkeyPatch,
 ) -> None:
     monkeypatch.delenv("PDFREST_API_KEY", raising=False)
@@ -223,7 +223,7 @@ def handler(request: httpx.Request) -> httpx.Response:
 
     transport = httpx.MockTransport(handler)
     with PdfRestClient(api_key=VALID_API_KEY, transport=transport) as client:
-        response = client.summarize_pdf_text_to_file(
+        response = client.summarize_text_to_file(
             input_file,
             extra_query={"trace": "true"},
             extra_headers={"X-Debug": "sync"},
@@ -246,7 +246,7 @@ def handler(request: httpx.Request) -> httpx.Response:
 
 
 @pytest.mark.asyncio
-async def test_async_summarize_pdf_text_success(
+async def test_async_summarize_text_success(
     monkeypatch: pytest.MonkeyPatch,
 ) -> None:
     monkeypatch.delenv("PDFREST_API_KEY", raising=False)
@@ -275,7 +275,7 @@ def handler(request: httpx.Request) -> httpx.Response:
 
     transport = httpx.MockTransport(handler)
     async with AsyncPdfRestClient(api_key=ASYNC_API_KEY, transport=transport) as client:
-        response = await client.summarize_pdf_text(input_file)
+        response = await client.summarize_text(input_file)
 
     assert seen == {"post": 1}
     assert isinstance(response, SummarizePdfTextResponse)
@@ -284,7 +284,7 @@ def handler(request: httpx.Request) -> httpx.Response:
 
 
 @pytest.mark.asyncio
-async def test_async_summarize_pdf_text_to_file_success(
+async def test_async_summarize_text_to_file_success(
     monkeypatch: pytest.MonkeyPatch,
 ) -> None:
     monkeypatch.delenv("PDFREST_API_KEY", raising=False)
@@ -322,7 +322,7 @@ def handler(request: httpx.Request) -> httpx.Response:
 
     transport = httpx.MockTransport(handler)
     async with AsyncPdfRestClient(api_key=ASYNC_API_KEY, transport=transport) as client:
-        response = await client.summarize_pdf_text_to_file(input_file)
+        response = await client.summarize_text_to_file(input_file)
 
     assert seen == {"post": 1, "get": 1}
     assert isinstance(response, PdfRestFileBasedResponse)