Skip to content

Fix workflow lint crash when tool_state is dict instead of JSON string#1626

Merged
jmchilton merged 1 commit intogalaxyproject:masterfrom
jmchilton:json_fix
Mar 19, 2026
Merged

Fix workflow lint crash when tool_state is dict instead of JSON string#1626
jmchilton merged 1 commit intogalaxyproject:masterfrom
jmchilton:json_fix

Conversation

@jmchilton
Copy link
Copy Markdown
Member

Some .ga workflows have tool_state as a dict rather than a JSON-encoded string. Handle both forms to avoid json.loads TypeError.

Some .ga workflows have tool_state as a dict rather than a JSON-encoded
string. Handle both forms to avoid json.loads TypeError.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@jmchilton jmchilton added the bug label Mar 19, 2026
@mvdbeek
Copy link
Copy Markdown
Member

mvdbeek commented Mar 19, 2026

Test error is surely unrelated

@jmchilton jmchilton marked this pull request as ready for review March 19, 2026 16:43
@jmchilton jmchilton merged commit 9add87a into galaxyproject:master Mar 19, 2026
14 of 15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants