Skip to content

Preserve assistant messages with tool calls#21545

Merged
jmchilton merged 6 commits intogalaxyproject:devfrom
guerler:plugins_adapter.000
Jan 12, 2026
Merged

Preserve assistant messages with tool calls#21545
jmchilton merged 6 commits intogalaxyproject:devfrom
guerler:plugins_adapter.000

Conversation

@guerler
Copy link
Copy Markdown
Contributor

@guerler guerler commented Jan 7, 2026

Newer models can return assistant messages with both content and tool_calls. The plugin chat adapter now forwards both fields so tool-based workflows keep working.

The PR also improves error handling for JupyterLite by showing the actual provider error message instead of a generic one.

Before:
image

Now:
image

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.

@guerler guerler added this to the 26.0 milestone Jan 7, 2026
@guerler guerler force-pushed the plugins_adapter.000 branch 3 times, most recently from dc204f1 to cf33f9b Compare January 8, 2026 07:24
@guerler guerler force-pushed the plugins_adapter.000 branch 3 times, most recently from a83809a to fe96da1 Compare January 8, 2026 09:00
@guerler guerler force-pushed the plugins_adapter.000 branch from fe96da1 to fdc6576 Compare January 8, 2026 09:02
@guerler guerler marked this pull request as ready for review January 8, 2026 10:50
@guerler
Copy link
Copy Markdown
Contributor Author

guerler commented Jan 8, 2026

Failing tests are unrelated.

@guerler guerler requested a review from jmchilton January 9, 2026 14:51
@jmchilton jmchilton merged commit dee1889 into galaxyproject:dev Jan 12, 2026
64 of 71 checks passed
@itisAliRH itisAliRH deleted the plugins_adapter.000 branch January 15, 2026 15:22
@guerler guerler added the release-testing-26.0 PRs marked for testing for the 26.0 release and issues stemming from release testing label Jan 29, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/API kind/bug release-testing-26.0 PRs marked for testing for the 26.0 release and issues stemming from release testing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants