[26.0] AttributeError when history is None during tool execution

[mvdbeek]

When executing tools via the API without a history_id and no active session history, the history variable is None, causing a crash at history.add_pending_items(). This raises a clear error early in the service layer and adds a defensive guard in execute.py.

Fixes #22018

How to test the changes?

(Select all options that apply)

• I've included appropriate automated tests.
• This is a refactoring of components with existing test coverage.
• Instructions for manual testing are as follows:
  1. [add testing steps and prerequisites here if you didn't write automated tests covering all your changes]

License

• I agree to license these and all my past contributions to the core galaxy codebase under the MIT license.

[jmchilton]

FAILED lib/galaxy_test/api/test_tools_upload.py::TestToolsUpload::test_abort_fetch_job - AssertionError: assert 'error' == 'discarded'

I'm assuming this can't be related - but I don't recall ever seeing it fail before. I'll rerun that test.

[mvdbeek]

FAILED lib/galaxy_test/api/test_tools_upload.py::TestToolsUpload::test_abort_fetch_job - AssertionError: assert 'error' == 'discarded'

I'm assuming this can't be related - but I don't recall ever seeing it fail before. I'll rerun that test.

httpstat.us down / unreachable etc. I think you fixes this on the dev branch ?