Remove force-sorting for query_tickets API and mention sorting in README

Author: zyv

README.md

@@ -31,6 +31,11 @@ TracApiVersion(epoch=1, major=2, minor=0)
 datetime.datetime(2025, 2, 27, 13, 37, 11, 171873, tzinfo=TzInfo(UTC))
 ```
 
+> [!IMPORTANT]
+> Trac APIs (e.g. `query_tickets`) do not return IDs and/or objects sorted in alphanumeric order!
+>
+> They can either be returned in a custom order (such as priority levels) or grouped by certain fields (such as ticket IDs). If you want to iterate on the objects in a particular order, always remember to sort them appropriately after you call the API.
+
 ### Customizing models
 
 #### Changing default string type

src/trac_rpc/client.py

@@ -145,7 +145,7 @@ def query_tickets(self, query: str = "", per_page: int = 0, page_number: int | N
             *((f"page={page_number}",) if page_number is not None else ()),
             f"max={per_page}",
         )
-        return sorted(self._request(TracRequest(method="ticket.query", params=["&".join(pieces)]), list[int]))
+        return self._request(TracRequest(method="ticket.query", params=["&".join(pieces)]), list[int])
 
     def get_ticket_attachments(self, ticket_id: int) -> TracTicketAttachments:
         """

tests/test_methods.py

@@ -25,15 +25,15 @@ def test_query_tickets(kwargs: dict, expected: str, api_client: ApiClient, respx
     respx_mock.post().mock(
         return_value=httpx.Response(
             status_code=httpx.codes.OK,
-            text="""{"error": null, "result": [1, 2, 3], "id": null}""",
+            text="""{"error": null, "result": [3, 2, 1], "id": null}""",
         )
     )
 
     def get_last_request_params() -> str:
         (param,) = json.loads(respx_mock.calls.last.request.content)["params"]
         return param
 
-    assert api_client.query_tickets(**kwargs) == [1, 2, 3]
+    assert sorted(api_client.query_tickets(**kwargs)) == [1, 2, 3]
     assert get_last_request_params() == expected