From 5c3a592304e8c893b57315d7d98fd30f059c8c05 Mon Sep 17 00:00:00 2001 From: John Chilton Date: Sat, 13 Jun 2026 10:04:08 -0400 Subject: [PATCH] Fix upload_file_from_url passing decoded id to FetchDataPayload FetchDataPayload.history_id is a DecodedDatabaseIdField, which expects the encoded id string and decodes it itself. upload_file_from_url was pre-decoding history_id to an int, tripping the field's isinstance(str) validator and failing every call. Pass the encoded id straight through, matching invoke_workflow's handling of the same field type. Co-Authored-By: Claude Opus 4.8 (1M context) --- lib/galaxy/agents/operations.py | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/lib/galaxy/agents/operations.py b/lib/galaxy/agents/operations.py index 2270f297660..ebdb3f9105e 100644 --- a/lib/galaxy/agents/operations.py +++ b/lib/galaxy/agents/operations.py @@ -424,9 +424,8 @@ class AgentOperationsManager: dbkey: str = "?", file_name: str | None = None, ) -> dict[str, Any]: - decoded_history_id = self.trans.security.decode_id(history_id) fetch_payload = FetchDataPayload( - history_id=decoded_history_id, + history_id=history_id, targets=[ DataElementsTarget( destination=HdaDestination(type="hdas"),