fix: return action results from action_invoke - #12
Merged
sciabarracom merged 1 commit intoAug 29, 2026
Merged
Conversation
Generated-by: OpenAI Codex
Contributor
Author
|
Hi maintainers, a gentle follow-up on this PR. It fixes the default action_invoke path so callers receive the invoked action result, aligning the implementation with the documented tool contract, and includes a process-boundary regression test. The license workflow is currently awaiting approval to run. I am happy to address any feedback or adjust the approach. Thanks! |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
ops invokealiasopsprocess boundaryRoot cause
action_invokeusedops action invoke. In the OpenServerless CLI,actionis the genericwsk actionwrapper, so this path invokes withblocking=false&result=false. Its successful output contains only an activation ID.The CLI's dedicated
invokealias expands towsk action invoke -r, which invokes withblocking=true&result=trueand prints the action result.Because this MCP exposes no follow-up activation-result tool, the old success response was a dead end for an agent: the action ran, but its result was unavailable.
Live verification
I exercised the checksum-verified Apache OpenServerless CLI release
v0.9.1-2607122150.devagainst a local mock OpenWhisk API:blocking=false&result=false; MCP outputok: invoked /_/v1/demo with id activation-123blocking=true&result=true; MCP output{ "answer": 42 }The request body remained
{ "key": "value" }, confirming parameter forwarding is unchanged.Validation
npm test— 14 tests passednpm run typecheckgit diff --checkAI assistance
This contribution was developed with OpenAI Codex assistance. The submitter verified the upstream CLI contract, live behavior, patch, and tests. The commit includes a
Generated-by: OpenAI Codextrailer.