Skip to content

馃挜 Includes input/output for nexus op describe JSON - #1225

Merged
S15 merged 2 commits into
mainfrom
samm/sanoDescribePayloads
Oct 2, 2026
Merged

S15 merged 2 commits into
mainfrom
samm/sanoDescribePayloads

Conversation

@S15

@S15 S15 commented Oct 2, 2026 •

Copy link
Copy Markdown
Contributor

What changed?

Includes input and output for nexus operation describe json output. Matches activity describe.

This changes the JSON output, moving the existing output under the 'info' property and including top-level input, result or failure fields.

This also changes the --raw output to include the Info wrapper.

Checklist

Stability

  • Breaking changes are marked with 馃挜 in the PR title and release notes
  • Changes to JSON output (-o json / -o jsonl) are treated as breaking changes

Design

  • This feature does not depend on Cloud-only APIs or behavior (it works against an OSS server)

Behavior

  • Results go to stdout; errors and warnings go to stderr
  • Error messages are lowercase with no trailing punctuation

Tests

  • Added functional test(s) (SharedServerSuite)
  • Added unit test(s) (func TestXxx) where applicable

Manual tests

Happy path

temporal nexus operation describe \
  --operation-id myOperationId \
  --output json

Successful Operation:

{
  "runId": "...",
  "info": {
    "operationId": "myOperationId",
    "runId": "...",
    "endpoint": "myEndpoint",
    "service": "myService",
    "operation": "myOperation",
    "status": "NEXUS_OPERATION_EXECUTION_STATUS_COMPLETED",
    "attempt": 1,
    "scheduleTime": "2026-10-02T22:11:15.505571Z",
    "closeTime": "2026-10-02T22:11:17.524073Z",
    "lastAttemptCompleteTime": "2026-10-02T22:11:15.512824Z",
    "executionDuration": "2.018502s",
    "requestId": "...",
    "operationToken": "...",
    "stateTransitionCount": "3",
    "searchAttributes": {},
    "userMetadata": {
      "summary": "My test operation"
    },
    "links": [
      {
        "workflowEvent": {
          "namespace": "default",
          "workflowId": "...",
          "runId": "...",
          "eventRef": {
            "eventId": "1",
            "eventType": "EVENT_TYPE_WORKFLOW_EXECUTION_STARTED"
          }
        }
      }
    ],
    "stateSizeBytes": "1029"
  },
  "input": {
    "Name": "World"
  },
  "result": {
    "Greeting": "Hello World"
  }
}

Failed operation:

{
  "runId": "...",
  "info": {
    "operationId": "myOperationId",
    "runId": "...",
    "endpoint": "myEndpoint",
    "service": "myService",
    "operation": "myOperation",
    "status": "NEXUS_OPERATION_EXECUTION_STATUS_FAILED",
    "attempt": 1,
    "scheduleTime": "2026-10-02T22:18:49.121318Z",
    "closeTime": "2026-10-02T22:18:49.125985Z",
    "lastAttemptCompleteTime": "2026-10-02T22:18:49.125985Z",
    "lastAttemptFailure": {
      "message": "expected failure",
      "source": "GoSDK",
      "applicationFailureInfo": {
        "type": "OperationError",
        "nonRetryable": true
      }
    },
    "executionDuration": "0.004667s",
    "requestId": "...",
    "stateTransitionCount": "2",
    "searchAttributes": {},
    "userMetadata": {
      "summary": "My failing operation"
    },
    "stateSizeBytes": "811"
  },
  "input": {
    "Reason": "expected failure"
  },
  "failure": {
    "message": "expected failure",
    "source": "GoSDK",
    "applicationFailureInfo": {
      "type": "OperationError",
      "nonRetryable": true
    }
  },
}

@S15
S15 marked this pull request as ready for review October 2, 2026 18:58
@S15
S15 requested a review from a team as a code owner October 2, 2026 18:58
@ks-temporal

Copy link
Copy Markdown
Contributor

I approved the code changes. Since this changes the cli output, it will be useful to show a sample test commands and output in the PR description, as recommended by the PR template. If you decide to do this, please update the PR description with the example before merge. Thanks.

@S15
S15 added this pull request to the merge queue Oct 2, 2026
Merged via the queue into main with commit daa3f46 Oct 2, 2026
11 checks passed
@S15
S15 deleted the samm/sanoDescribePayloads branch October 2, 2026 22:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants