feat(executions): add result_ref kwarg to report_outcome#19
Closed
govindkavaturi-art wants to merge 1 commit intomainfrom
Closed
feat(executions): add result_ref kwarg to report_outcome#19govindkavaturi-art wants to merge 1 commit intomainfrom
govindkavaturi-art wants to merge 1 commit intomainfrom
Conversation
The server's /v1/executions/{id}/outcome endpoint already accepts a
`result_ref` evidence field (see OutcomeRequest in cueapi/cueapi
app/schemas/outcome.py). The SDK's report_outcome was missing it,
forcing callers who want to attach a reference ID to fall back to
either raw HTTP or using an unrelated field like external_id.
Single-line addition in executions.py: new kwarg, forwarded to the
POST body when provided, same shape as every other evidence field.
Companion change in cueapi-worker 0.3.0 (monorepo PR in cueapi/cueapi)
exposes the same field on its WorkerAPIClient so worker-transport
handlers can surface result_ref via the new CUEAPI_OUTCOME_FILE
pattern.
No behavioral change for existing callers — kwarg is optional and
defaults to None.
Member
Author
|
Closing as already-shipped. Verified:
Flagging as a parity artifact — no gap, no work needed. |
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
1 participant
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.
Single-line kwarg addition to the SDK's
report_outcomeso callers can passresult_ref(already supported server-side perOutcomeRequestin the main cueapi monorepo'sapp/schemas/outcome.py).Companion to the cueapi-worker 0.3.0 PR in cueapi/cueapi, which adds the same field to its internal
WorkerAPIClientso the newCUEAPI_OUTCOME_FILEpattern can surface result_ref.NoneNo version bump here — Argus cuts the SDK tag on its own after the monorepo bits land.