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

@pablots99 pablots99 commented Dec 19, 2025

Copy link
Copy Markdown

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

kepten commented Dec 19, 2025

Copy link
Copy Markdown

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

nitesh-kumar-ba commented Dec 19, 2025

Copy link
Copy Markdown

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