Skip to content
This repository was archived by the owner on Jan 5, 2026. It is now read-only.

Discard dispatch API response body#109

Open
pablots99 wants to merge 1 commit into
convictional:masterfrom
pablots99:fix/jq-workflow-id-integer-overflow
Open

Discard dispatch API response body#109
pablots99 wants to merge 1 commit into
convictional:masterfrom
pablots99:fix/jq-workflow-id-integer-overflow

Conversation

@pablots99
Copy link
Copy Markdown

@pablots99 pablots99 commented Dec 19, 2025

GitHub changed the workflow dispatch API to return a JSON response body:

{ "workflow_run_id": 20365302298, "run_url": "https://api.github.com/repos/.../actions/runs/20365302298", "html_url": "https://github.com/.../actions/runs/20365302298" }

Previously, this endpoint returned 204 No Content with an empty body. And the docs still show that.
trigger_workflow function captures all stdout to extract run IDs, the api() function echo its response to stdout, this new JSON response was being captured and mixed with the actual run IDs.

Solution
Redirect the dispatch API response to null

@pablots99 pablots99 marked this pull request as draft December 19, 2025 09:48
@pablots99 pablots99 marked this pull request as ready for review December 19, 2025 09:54
@silviucuruciucfp
Copy link
Copy Markdown

any change of this getting released today? por favor

@kepten
Copy link
Copy Markdown

kepten commented Dec 19, 2025

Until it's not released, you can use this patched version like uses: convictional/trigger-workflow-and-wait@ee0ac1964d5ec33b3bb2cab11f87bece4dedb80b

@stephenmontgomery
Copy link
Copy Markdown

Confirmed above works - thanks!

@nitesh-kumar-ba
Copy link
Copy Markdown

nitesh-kumar-ba commented Dec 19, 2025

Any plans to merge this and release a new version?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants